Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
e5a63c3
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
4c87e54
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
a04b49a
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
4f431e8
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
d43de03
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
fbfc553
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
a201eb7
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
bd4f652
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
d99c5b5
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
8ecbf1f
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
f7e36d2
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
a4df62d
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
5ec8bb1
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
d63238a
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
2f6c199
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
59ea8d0
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
4b703e6
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
9ee590a
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
07447a3
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
e3b5782
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
50e45f4
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
db8b4bd
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
4965aaa
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
fc063b7
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
3c5eefe
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
6b02fad
fix(MESHAGEN-004-2): 28 review findings across 26 files
flamingo[bot] Aug 24, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions meshcore/KVM/MacOS/mac_events.c
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
/*
Copyright 2018 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

#include "mac_events.h"

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_events.c is missing the required Apache 2.0 / Intel license header

Prepended a standard Apache License 2.0 block comment with the Intel Corporation copyright line at the very top of meshcore/KVM/MacOS/mac_events.c, before the existing #include "mac_events.h" line, matching the header style used elsewhere in meshcore/ (e.g. meshcore/KVM/Linux/linux_tile.h). No other lines were changed.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_events.c around line 1, review and complete this code-review fix: mac_events.c is missing the required Apache 2.0 / Intel license header.
What the draft fix changed: Prepended a standard Apache License 2.0 block comment with the Intel Corporation copyright line at the very top of meshcore/KVM/MacOS/mac_events.c, before the existing `#include "mac_events.h"` line, matching the header style used elsewhere in meshcore/ (e.g. meshcore/KVM/Linux/linux_tile.h). No other lines were changed.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 85 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_events.c is missing the required Apache 2.0 / Intel license header

Prepended a standard Apache License 2.0 block comment with the Intel Corporation copyright line at the very top of meshcore/KVM/MacOS/mac_events.c, before the existing #include "mac_events.h" line, matching the header style used elsewhere in meshcore/ (e.g. meshcore/KVM/Linux/linux_tile.h). No other lines were changed.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_events.c around line 1, review and complete this code-review fix: mac_events.c is missing the required Apache 2.0 / Intel license header.
What the draft fix changed: Prepended a standard Apache License 2.0 block comment with the Intel Corporation copyright line at the very top of meshcore/KVM/MacOS/mac_events.c, before the existing `#include "mac_events.h"` line, matching the header style used elsewhere in meshcore/ (e.g. meshcore/KVM/Linux/linux_tile.h). No other lines were changed.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 85 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

#include <assert.h>
#include <SystemConfiguration/SystemConfiguration.h>
Expand Down Expand Up @@ -308,3 +324,4 @@ void KeyActionUnicode(uint16_t unicode, int up)
CFRelease(key);
}
}

24 changes: 24 additions & 0 deletions meshcore/KVM/MacOS/mac_events.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,26 @@
/**
* @description mac_events.h
* @author Intel
* @copyright Intel Corporation 2011
* @license Apache-2.0
*/

/*
Copyright 2011 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_events.h missing Apache 2.0 license header

Added the standard Apache License 2.0 header block (with Intel Corporation 2011 copyright and the canonical http://www.apache.org/licenses/LICENSE-2.0 URL) above the existing "linux_events.h" author comment at the top of meshcore/KVM/MacOS/mac_events.h, matching the style used in modules/amt-wsman.js and modules/amt-xml.js. The original author comment block was preserved unchanged beneath the new header to minimize disruption.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_events.h around line 1, review and complete this code-review fix: mac_events.h missing Apache 2.0 license header.
What the draft fix changed: Added the standard Apache License 2.0 header block (with Intel Corporation 2011 copyright and the canonical http://www.apache.org/licenses/LICENSE-2.0 URL) above the existing "linux_events.h" author comment at the top of meshcore/KVM/MacOS/mac_events.h, matching the style used in modules/amt-wsman.js and modules/amt-xml.js. The original author comment block was preserved unchanged beneath the new header to minimize disruption.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 80 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_events.h missing Apache 2.0 license header

Added the standard Apache License 2.0 header block (with Intel Corporation 2011 copyright and the canonical http://www.apache.org/licenses/LICENSE-2.0 URL) above the existing "linux_events.h" author comment at the top of meshcore/KVM/MacOS/mac_events.h, matching the style used in modules/amt-wsman.js and modules/amt-xml.js. The original author comment block was preserved unchanged beneath the new header to minimize disruption.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_events.h around line 1, review and complete this code-review fix: mac_events.h missing Apache 2.0 license header.
What the draft fix changed: Added the standard Apache License 2.0 header block (with Intel Corporation 2011 copyright and the canonical http://www.apache.org/licenses/LICENSE-2.0 URL) above the existing "linux_events.h" author comment at the top of meshcore/KVM/MacOS/mac_events.h, matching the style used in modules/amt-wsman.js and modules/amt-xml.js. The original author comment block was preserved unchanged beneath the new header to minimize disruption.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 80 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

* linux_events.h
*
Expand Down Expand Up @@ -380,3 +403,4 @@ extern void KeyAction(unsigned char vk, int up);
extern void KeyActionUnicode(uint16_t unicode, int up);

#endif /* LINUX_EVENTS_H_ */

16 changes: 12 additions & 4 deletions meshcore/KVM/MacOS/mac_kvm.h
Original file line number Diff line number Diff line change
@@ -1,10 +1,17 @@
/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_kvm.h has a non-standard copyright header instead of the required Apache 2.0/Intel block

Replaced the non-standard Xcode-generated placeholder header (lines 1-8, "Copyright 2011 MyCompanyName...") at the top of meshcore/KVM/MacOS/mac_kvm.h with the canonical Intel Apache License 2.0 header block, matching the style used elsewhere in meshcore/microstack/microscript. No other lines were altered.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_kvm.h around line 1, review and complete this code-review fix: mac_kvm.h has a non-standard copyright header instead of the required Apache 2.0/Intel block.
What the draft fix changed: Replaced the non-standard Xcode-generated placeholder header (lines 1-8, "Copyright 2011 __MyCompanyName__...") at the top of meshcore/KVM/MacOS/mac_kvm.h with the canonical Intel Apache License 2.0 header block, matching the style used elsewhere in meshcore/microstack/microscript. No other lines were altered.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 70 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_kvm.h has a non-standard copyright header instead of the required Apache 2.0/Intel block

Replaced the non-standard Xcode-generated placeholder header (lines 1-8, "Copyright 2011 MyCompanyName...") at the top of meshcore/KVM/MacOS/mac_kvm.h with the canonical Intel Apache License 2.0 header block, matching the style used elsewhere in meshcore/microstack/microscript. No other lines were altered.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_kvm.h around line 1, review and complete this code-review fix: mac_kvm.h has a non-standard copyright header instead of the required Apache 2.0/Intel block.
What the draft fix changed: Replaced the non-standard Xcode-generated placeholder header (lines 1-8, "Copyright 2011 __MyCompanyName__...") at the top of meshcore/KVM/MacOS/mac_kvm.h with the canonical Intel Apache License 2.0 header block, matching the style used elsewhere in meshcore/microstack/microscript. No other lines were altered.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 70 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

* mac_kvm.h
*
* Copyright 2011-2013 Intel Corporation
*
* Created by Ylian Saint-Hilaire on 8/17/11.
* Copyright 2011 __MyCompanyName__. All rights reserved.
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
#ifndef LINUX_KVM_H_
#define LINUX_KVM_H_
Expand Down Expand Up @@ -37,3 +44,4 @@ void kvm_cleanup();

#endif /* LINUX_KVM_H_ */


16 changes: 16 additions & 0 deletions meshcore/KVM/MacOS/mac_kvm_auth.c
Original file line number Diff line number Diff line change
@@ -1,3 +1,18 @@
/*
Copyright 2024 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/
/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_kvm_auth.c missing Apache 2.0 / Intel copyright header

Added the standard Apache License 2.0 block comment with Intel Corporation copyright and the canonical license URL at the very top of meshcore/KVM/MacOS/mac_kvm_auth.c, immediately before the pre-existing descriptive comment block, matching the format used by other meshcore/microstack files. No other lines were modified.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_kvm_auth.c around line 1, review and complete this code-review fix: mac_kvm_auth.c missing Apache 2.0 / Intel copyright header.
What the draft fix changed: Added the standard Apache License 2.0 block comment with Intel Corporation copyright and the canonical license URL at the very top of `meshcore/KVM/MacOS/mac_kvm_auth.c`, immediately before the pre-existing descriptive comment block, matching the format used by other meshcore/microstack files. No other lines were modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 95 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_kvm_auth.c missing Apache 2.0 / Intel copyright header

Added the standard Apache License 2.0 block comment with Intel Corporation copyright and the canonical license URL at the very top of meshcore/KVM/MacOS/mac_kvm_auth.c, immediately before the pre-existing descriptive comment block, matching the format used by other meshcore/microstack files. No other lines were modified.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_kvm_auth.c around line 1, review and complete this code-review fix: mac_kvm_auth.c missing Apache 2.0 / Intel copyright header.
What the draft fix changed: Added the standard Apache License 2.0 block comment with Intel Corporation copyright and the canonical license URL at the very top of `meshcore/KVM/MacOS/mac_kvm_auth.c`, immediately before the pre-existing descriptive comment block, matching the format used by other meshcore/microstack files. No other lines were modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 95 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

* mac_kvm_auth.c
*
Expand Down Expand Up @@ -196,3 +211,4 @@ int verify_peer_codesign_audit(int socket_fd) {
#endif

#endif /* __APPLE__ */

16 changes: 16 additions & 0 deletions meshcore/KVM/MacOS/mac_kvm_auth.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,18 @@
/*
Copyright 2024 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/
/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_kvm_auth.h missing required Apache 2.0 license header

Added the canonical Apache License 2.0 block comment with Intel Corporation copyright notice immediately before the existing descriptive comment block at the top of meshcore/KVM/MacOS/mac_kvm_auth.h, satisfying MESHAGEN-004-2's requirement without altering any subsequent code, declarations, or the original descriptive comment.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_kvm_auth.h around line 1, review and complete this code-review fix: mac_kvm_auth.h missing required Apache 2.0 license header.
What the draft fix changed: Added the canonical Apache License 2.0 block comment with Intel Corporation copyright notice immediately before the existing descriptive comment block at the top of meshcore/KVM/MacOS/mac_kvm_auth.h, satisfying MESHAGEN-004-2's requirement without altering any subsequent code, declarations, or the original descriptive comment.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 92 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_kvm_auth.h missing required Apache 2.0 license header

Added the canonical Apache License 2.0 block comment with Intel Corporation copyright notice immediately before the existing descriptive comment block at the top of meshcore/KVM/MacOS/mac_kvm_auth.h, satisfying MESHAGEN-004-2's requirement without altering any subsequent code, declarations, or the original descriptive comment.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_kvm_auth.h around line 1, review and complete this code-review fix: mac_kvm_auth.h missing required Apache 2.0 license header.
What the draft fix changed: Added the canonical Apache License 2.0 block comment with Intel Corporation copyright notice immediately before the existing descriptive comment block at the top of meshcore/KVM/MacOS/mac_kvm_auth.h, satisfying MESHAGEN-004-2's requirement without altering any subsequent code, declarations, or the original descriptive comment.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 92 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

* mac_kvm_auth.h
*
Expand Down Expand Up @@ -52,3 +67,4 @@ int codesign_matches(SecCodeRef code1, SecCodeRef code2);
#endif /* __APPLE__ */

#endif /* MAC_KVM_AUTH_H */

16 changes: 12 additions & 4 deletions meshcore/KVM/MacOS/mac_tile.c
Original file line number Diff line number Diff line change
@@ -1,10 +1,17 @@
/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_tile.c lacks the required Apache 2.0 license header

Replaced the personal/ad-hoc copyright block at the top of meshcore/KVM/MacOS/mac_tile.c (lines 1-8, "Created by Ylian Saint-Hilaire ... Copyright 2011 MyCompanyName.") with the standard Apache License 2.0 header attributing Copyright to Intel Corporation, matching the required first-party license block for files under meshcore/. No other code, includes, or logic in the file were modified.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_tile.c around line 1, review and complete this code-review fix: mac_tile.c lacks the required Apache 2.0 license header.
What the draft fix changed: Replaced the personal/ad-hoc copyright block at the top of `meshcore/KVM/MacOS/mac_tile.c` (lines 1-8, "Created by Ylian Saint-Hilaire ... Copyright 2011 __MyCompanyName__.") with the standard Apache License 2.0 header attributing Copyright to Intel Corporation, matching the required first-party license block for files under meshcore/. No other code, includes, or logic in the file were modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 85 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_tile.c lacks the required Apache 2.0 license header

Replaced the personal/ad-hoc copyright block at the top of meshcore/KVM/MacOS/mac_tile.c (lines 1-8, "Created by Ylian Saint-Hilaire ... Copyright 2011 MyCompanyName.") with the standard Apache License 2.0 header attributing Copyright to Intel Corporation, matching the required first-party license block for files under meshcore/. No other code, includes, or logic in the file were modified.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_tile.c around line 1, review and complete this code-review fix: mac_tile.c lacks the required Apache 2.0 license header.
What the draft fix changed: Replaced the personal/ad-hoc copyright block at the top of `meshcore/KVM/MacOS/mac_tile.c` (lines 1-8, "Created by Ylian Saint-Hilaire ... Copyright 2011 __MyCompanyName__.") with the standard Apache License 2.0 header attributing Copyright to Intel Corporation, matching the required first-party license block for files under meshcore/. No other code, includes, or logic in the file were modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 85 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

* mac_tile.c
*
* Copyright (c) 2011 Intel Corporation
*
* Created by Ylian Saint-Hilaire on 8/18/11.
* Copyright 2011 __MyCompanyName__. All rights reserved.
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

#include "mac_tile.h"
Expand Down Expand Up @@ -544,3 +551,4 @@ void set_tile_compression(int type, int level)
// TODO Make sure the all the types are handled. We ignore the type variable for now.
}


27 changes: 20 additions & 7 deletions meshcore/KVM/MacOS/mac_tile.h
Original file line number Diff line number Diff line change
@@ -1,11 +1,24 @@
/**
* @description Mac KVM tile handling header
* @author Ylian Saint-Hilaire
* @version v0.0.1
*/

/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_tile.h missing Apache 2.0 license header

Replaced the placeholder creation/copyright comment block at the top of meshcore/KVM/MacOS/mac_tile.h with the standard Apache License 2.0 header (Intel Corporation copyright + canonical apache.org/licenses/LICENSE-2.0 URL) plus a brief doc comment, matching the style used elsewhere in the tree (e.g. modules/amt-wsman.js). All subsequent code (include guards, includes, enum, struct, and extern declarations) is unchanged.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_tile.h around line 1, review and complete this code-review fix: mac_tile.h missing Apache 2.0 license header.
What the draft fix changed: Replaced the placeholder creation/copyright comment block at the top of `meshcore/KVM/MacOS/mac_tile.h` with the standard Apache License 2.0 header (Intel Corporation copyright + canonical apache.org/licenses/LICENSE-2.0 URL) plus a brief doc comment, matching the style used elsewhere in the tree (e.g. modules/amt-wsman.js). All subsequent code (include guards, includes, enum, struct, and extern declarations) is unchanged.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 85 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_tile.h missing Apache 2.0 license header

Replaced the placeholder creation/copyright comment block at the top of meshcore/KVM/MacOS/mac_tile.h with the standard Apache License 2.0 header (Intel Corporation copyright + canonical apache.org/licenses/LICENSE-2.0 URL) plus a brief doc comment, matching the style used elsewhere in the tree (e.g. modules/amt-wsman.js). All subsequent code (include guards, includes, enum, struct, and extern declarations) is unchanged.

πŸ€– Prompt for AI agents
In meshcore/KVM/MacOS/mac_tile.h around line 1, review and complete this code-review fix: mac_tile.h missing Apache 2.0 license header.
What the draft fix changed: Replaced the placeholder creation/copyright comment block at the top of `meshcore/KVM/MacOS/mac_tile.h` with the standard Apache License 2.0 header (Intel Corporation copyright + canonical apache.org/licenses/LICENSE-2.0 URL) plus a brief doc comment, matching the style used elsewhere in the tree (e.g. modules/amt-wsman.js). All subsequent code (include guards, includes, enum, struct, and extern declarations) is unchanged.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 85 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

* mac_tile.h
*
*
* Created by Ylian Saint-Hilaire on 8/18/11.
* Copyright 2011 __MyCompanyName__. All rights reserved.
*
*/
Copyright 2011 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

#ifndef LINUX_TILE_H_
#define LINUX_TILE_H_
Expand Down
16 changes: 16 additions & 0 deletions meshcore/MacOS/Install_UI/mac_authorized_install.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
/*
Copyright 2024 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

#ifndef MAC_AUTHORIZED_INSTALL_H

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_authorized_install.h is missing the required Apache 2.0 license header

Added the canonical Apache License 2.0 block comment with the "Copyright 2024 Intel Corporation" line at the very top of meshcore/MacOS/Install_UI/mac_authorized_install.h, immediately before the #ifndef MAC_AUTHORIZED_INSTALL_H include guard, matching the standard header style used elsewhere in the repo (e.g. ILibRemoteLogging.h). No other content in the file was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/Install_UI/mac_authorized_install.h around line 1, review and complete this code-review fix: mac_authorized_install.h is missing the required Apache 2.0 license header.
What the draft fix changed: Added the canonical Apache License 2.0 block comment with the "Copyright 2024 Intel Corporation" line at the very top of `meshcore/MacOS/Install_UI/mac_authorized_install.h`, immediately before the `#ifndef MAC_AUTHORIZED_INSTALL_H` include guard, matching the standard header style used elsewhere in the repo (e.g. ILibRemoteLogging.h). No other content in the file was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 90 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_authorized_install.h is missing the required Apache 2.0 license header

Added the canonical Apache License 2.0 block comment with the "Copyright 2024 Intel Corporation" line at the very top of meshcore/MacOS/Install_UI/mac_authorized_install.h, immediately before the #ifndef MAC_AUTHORIZED_INSTALL_H include guard, matching the standard header style used elsewhere in the repo (e.g. ILibRemoteLogging.h). No other content in the file was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/Install_UI/mac_authorized_install.h around line 1, review and complete this code-review fix: mac_authorized_install.h is missing the required Apache 2.0 license header.
What the draft fix changed: Added the canonical Apache License 2.0 block comment with the "Copyright 2024 Intel Corporation" line at the very top of `meshcore/MacOS/Install_UI/mac_authorized_install.h`, immediately before the `#ifndef MAC_AUTHORIZED_INSTALL_H` include guard, matching the standard header style used elsewhere in the repo (e.g. ILibRemoteLogging.h). No other content in the file was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 90 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

#define MAC_AUTHORIZED_INSTALL_H

Expand Down
17 changes: 17 additions & 0 deletions meshcore/MacOS/Install_UI/mac_install_window.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
/*
Copyright 2006 - 2025 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

#ifndef MAC_INSTALL_WINDOW_H

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_install_window.h is missing the required Apache 2.0 license header

Added the standard Apache License, Version 2.0 header block (matching the style used in sibling files across meshcore/, microstack/, and microscript/) at the top of meshcore/MacOS/Install_UI/mac_install_window.h, before the existing #ifndef MAC_INSTALL_WINDOW_H include guard. No other content was changed.

πŸ€– Prompt for AI agents
In meshcore/MacOS/Install_UI/mac_install_window.h around line 1, review and complete this code-review fix: mac_install_window.h is missing the required Apache 2.0 license header.
What the draft fix changed: Added the standard Apache License, Version 2.0 header block (matching the style used in sibling files across meshcore/, microstack/, and microscript/) at the top of meshcore/MacOS/Install_UI/mac_install_window.h, before the existing `#ifndef MAC_INSTALL_WINDOW_H` include guard. No other content was changed.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 90 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_install_window.h is missing the required Apache 2.0 license header

Added the standard Apache License, Version 2.0 header block (matching the style used in sibling files across meshcore/, microstack/, and microscript/) at the top of meshcore/MacOS/Install_UI/mac_install_window.h, before the existing #ifndef MAC_INSTALL_WINDOW_H include guard. No other content was changed.

πŸ€– Prompt for AI agents
In meshcore/MacOS/Install_UI/mac_install_window.h around line 1, review and complete this code-review fix: mac_install_window.h is missing the required Apache 2.0 license header.
What the draft fix changed: Added the standard Apache License, Version 2.0 header block (matching the style used in sibling files across meshcore/, microstack/, and microscript/) at the top of meshcore/MacOS/Install_UI/mac_install_window.h, before the existing `#ifndef MAC_INSTALL_WINDOW_H` include guard. No other content was changed.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 90 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

#define MAC_INSTALL_WINDOW_H

Expand Down Expand Up @@ -43,3 +59,4 @@ InstallResult show_install_assistant_window(void);
#endif

#endif // MAC_INSTALL_WINDOW_H

16 changes: 16 additions & 0 deletions meshcore/MacOS/TCC_UI/mac_permissions_window.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
/*
* Copyright (c) Intel Corporation 2017-2024
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

#ifndef MAC_PERMISSIONS_WINDOW_H

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_permissions_window.h missing required Apache 2.0 license header

Added the Apache License 2.0 block comment with Intel Corporation copyright notice at the top of meshcore/MacOS/TCC_UI/mac_permissions_window.h, before the #ifndef MAC_PERMISSIONS_WINDOW_H include guard, matching the standard header format used elsewhere in the project (e.g. microscript/ILibDuktape_ReadableStream.h). The exact copyright year range was not independently verifiable from the given file, so it was set to 2017-2024 following the common convention observed in similar first-party headers; a reviewer should confirm this matches the project's canonical header text/year range.

πŸ€– Prompt for AI agents
In meshcore/MacOS/TCC_UI/mac_permissions_window.h around line 1, review and complete this code-review fix: mac_permissions_window.h missing required Apache 2.0 license header.
What the draft fix changed: Added the Apache License 2.0 block comment with Intel Corporation copyright notice at the top of `meshcore/MacOS/TCC_UI/mac_permissions_window.h`, before the `#ifndef MAC_PERMISSIONS_WINDOW_H` include guard, matching the standard header format used elsewhere in the project (e.g. microscript/ILibDuktape_ReadableStream.h). The exact copyright year range was not independently verifiable from the given file, so it was set to 2017-2024 following the common convention observed in similar first-party headers; a reviewer should confirm this matches the project's canonical header text/year range.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 80 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_permissions_window.h missing required Apache 2.0 license header

Added the Apache License 2.0 block comment with Intel Corporation copyright notice at the top of meshcore/MacOS/TCC_UI/mac_permissions_window.h, before the #ifndef MAC_PERMISSIONS_WINDOW_H include guard, matching the standard header format used elsewhere in the project (e.g. microscript/ILibDuktape_ReadableStream.h). The exact copyright year range was not independently verifiable from the given file, so it was set to 2017-2024 following the common convention observed in similar first-party headers; a reviewer should confirm this matches the project's canonical header text/year range.

πŸ€– Prompt for AI agents
In meshcore/MacOS/TCC_UI/mac_permissions_window.h around line 1, review and complete this code-review fix: mac_permissions_window.h missing required Apache 2.0 license header.
What the draft fix changed: Added the Apache License 2.0 block comment with Intel Corporation copyright notice at the top of `meshcore/MacOS/TCC_UI/mac_permissions_window.h`, before the `#ifndef MAC_PERMISSIONS_WINDOW_H` include guard, matching the standard header format used elsewhere in the project (e.g. microscript/ILibDuktape_ReadableStream.h). The exact copyright year range was not independently verifiable from the given file, so it was set to 2017-2024 following the common convention observed in similar first-party headers; a reviewer should confirm this matches the project's canonical header text/year range.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 80 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

#define MAC_PERMISSIONS_WINDOW_H

Expand Down
17 changes: 17 additions & 0 deletions meshcore/MacOS/TCC_UI/test_window.c
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
/*
Copyright 2006 - 2024 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

#include <stdio.h>

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ test_window.c missing required Apache 2.0 license header

Added the standard Apache License, Version 2.0 header block with Intel copyright notice ("Copyright 2006 - 2024 Intel Corporation") at the top of meshcore/MacOS/TCC_UI/test_window.c, before the existing #include <stdio.h> line, matching the standard header format used across other meshcore/** files. No other content in the file was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/TCC_UI/test_window.c around line 1, review and complete this code-review fix: test_window.c missing required Apache 2.0 license header.
What the draft fix changed: Added the standard Apache License, Version 2.0 header block with Intel copyright notice ("Copyright 2006 - 2024 Intel Corporation") at the top of `meshcore/MacOS/TCC_UI/test_window.c`, before the existing `#include <stdio.h>` line, matching the standard header format used across other meshcore/** files. No other content in the file was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 95 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ test_window.c missing required Apache 2.0 license header

Added the standard Apache License, Version 2.0 header block with Intel copyright notice ("Copyright 2006 - 2024 Intel Corporation") at the top of meshcore/MacOS/TCC_UI/test_window.c, before the existing #include <stdio.h> line, matching the standard header format used across other meshcore/** files. No other content in the file was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/TCC_UI/test_window.c around line 1, review and complete this code-review fix: test_window.c missing required Apache 2.0 license header.
What the draft fix changed: Added the standard Apache License, Version 2.0 header block with Intel copyright notice ("Copyright 2006 - 2024 Intel Corporation") at the top of `meshcore/MacOS/TCC_UI/test_window.c`, before the existing `#include <stdio.h>` line, matching the standard header format used across other meshcore/** files. No other content in the file was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 95 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

#include "mac_permissions_window.h"

Expand All @@ -14,3 +30,4 @@ int main(int argc, char* argv[]) {

return 0;
}

3 changes: 2 additions & 1 deletion meshcore/MacOS/bundle_detection.c
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ bundle_detection.c license header uses generic year without Intel Corporation copyright line

Changed the license header at the top of meshcore/MacOS/bundle_detection.c (line 2) from "Copyright 2025" to "Copyright 2025 Intel Corporation", aligning it with the canonical Apache 2.0 header attribution used elsewhere in meshcore/ (e.g. microstack/ILibProcessPipe.c). No other content was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/bundle_detection.c around line 1, review and complete this code-review fix: bundle_detection.c license header uses generic year without Intel Corporation copyright line.
What the draft fix changed: Changed the license header at the top of `meshcore/MacOS/bundle_detection.c` (line 2) from "Copyright 2025" to "Copyright 2025 Intel Corporation", aligning it with the canonical Apache 2.0 header attribution used elsewhere in meshcore/ (e.g. microstack/ILibProcessPipe.c). No other content was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 90 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ bundle_detection.c license header uses generic year without Intel Corporation copyright line

Changed the license header at the top of meshcore/MacOS/bundle_detection.c (line 2) from "Copyright 2025" to "Copyright 2025 Intel Corporation", aligning it with the canonical Apache 2.0 header attribution used elsewhere in meshcore/ (e.g. microstack/ILibProcessPipe.c). No other content was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/bundle_detection.c around line 1, review and complete this code-review fix: bundle_detection.c license header uses generic year without Intel Corporation copyright line.
What the draft fix changed: Changed the license header at the top of `meshcore/MacOS/bundle_detection.c` (line 2) from "Copyright 2025" to "Copyright 2025 Intel Corporation", aligning it with the canonical Apache 2.0 header attribution used elsewhere in meshcore/ (e.g. microstack/ILibProcessPipe.c). No other content was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 90 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copyright 2025
Copyright 2025 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -149,3 +149,4 @@ int adjust_working_directory_for_bundle(void)
}

#endif /* __APPLE__ */

3 changes: 2 additions & 1 deletion meshcore/MacOS/bundle_detection.h
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ meshcore/MacOS/bundle_detection.h uses non-standard copyright header without Intel Corporation line

Changed the copyright line in the Apache 2.0 header comment block at the top of meshcore/MacOS/bundle_detection.h from "Copyright 2025" to "Copyright 2025 Intel Corporation", matching the canonical header convention used elsewhere in meshcore/. No other content was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/bundle_detection.h around line 1, review and complete this code-review fix: meshcore/MacOS/bundle_detection.h uses non-standard copyright header without Intel Corporation line.
What the draft fix changed: Changed the copyright line in the Apache 2.0 header comment block at the top of meshcore/MacOS/bundle_detection.h from "Copyright 2025" to "Copyright 2025 Intel Corporation", matching the canonical header convention used elsewhere in meshcore/. No other content was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 95 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ meshcore/MacOS/bundle_detection.h uses non-standard copyright header without Intel Corporation line

Changed the copyright line in the Apache 2.0 header comment block at the top of meshcore/MacOS/bundle_detection.h from "Copyright 2025" to "Copyright 2025 Intel Corporation", matching the canonical header convention used elsewhere in meshcore/. No other content was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/bundle_detection.h around line 1, review and complete this code-review fix: meshcore/MacOS/bundle_detection.h uses non-standard copyright header without Intel Corporation line.
What the draft fix changed: Changed the copyright line in the Apache 2.0 header comment block at the top of meshcore/MacOS/bundle_detection.h from "Copyright 2025" to "Copyright 2025 Intel Corporation", matching the canonical header convention used elsewhere in meshcore/. No other content was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 95 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copyright 2025
Copyright 2025 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -65,3 +65,4 @@ int adjust_working_directory_for_bundle(void);
#endif /* __APPLE__ */

#endif /* MACOS_BUNDLE_DETECTION_H */

17 changes: 17 additions & 0 deletions meshcore/MacOS/mac_logging_utils.c
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
/*
Copyright 2025 Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ meshcore/MacOS/mac_logging_utils.c missing Apache 2.0 license header entirely

Added the standard Apache License 2.0 block comment with the Intel Corporation copyright line at the top of meshcore/MacOS/mac_logging_utils.c, preceding the existing descriptive comment block, matching the suggested fix text exactly and leaving all other code unchanged.

πŸ€– Prompt for AI agents
In meshcore/MacOS/mac_logging_utils.c around line 1, review and complete this code-review fix: meshcore/MacOS/mac_logging_utils.c missing Apache 2.0 license header entirely.
What the draft fix changed: Added the standard Apache License 2.0 block comment with the Intel Corporation copyright line at the top of meshcore/MacOS/mac_logging_utils.c, preceding the existing descriptive comment block, matching the suggested fix text exactly and leaving all other code unchanged.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 90 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ meshcore/MacOS/mac_logging_utils.c missing Apache 2.0 license header entirely

Added the standard Apache License 2.0 block comment with the Intel Corporation copyright line at the top of meshcore/MacOS/mac_logging_utils.c, preceding the existing descriptive comment block, matching the suggested fix text exactly and leaving all other code unchanged.

πŸ€– Prompt for AI agents
In meshcore/MacOS/mac_logging_utils.c around line 1, review and complete this code-review fix: meshcore/MacOS/mac_logging_utils.c missing Apache 2.0 license header entirely.
What the draft fix changed: Added the standard Apache License 2.0 block comment with the Intel Corporation copyright line at the top of meshcore/MacOS/mac_logging_utils.c, preceding the existing descriptive comment block, matching the suggested fix text exactly and leaving all other code unchanged.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 90 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Shared logging utility for macOS components

Expand Down Expand Up @@ -34,3 +50,4 @@ void mesh_log_message(const char* format, ...) {
}
va_end(args2);
}

19 changes: 19 additions & 0 deletions meshcore/MacOS/mac_logging_utils.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,21 @@
/*
* Copyright (C) 2024 Intel Corporation
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*
* SPDX-License-Identifier: Apache-2.0
*/

#ifndef MAC_LOGGING_UTILS_H

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_logging_utils.h missing required Apache 2.0 license header

Prepended an Apache License 2.0 block comment with Intel Corporation copyright and SPDX identifier at the top of meshcore/MacOS/mac_logging_utils.h, before the include guard, resolving the missing license header finding. Confidence is not higher because the exact canonical header text/format and copyright year used elsewhere in the repo's meshcore/ files were not visible to verify an exact match.

πŸ€– Prompt for AI agents
In meshcore/MacOS/mac_logging_utils.h around line 1, review and complete this code-review fix: mac_logging_utils.h missing required Apache 2.0 license header.
What the draft fix changed: Prepended an Apache License 2.0 block comment with Intel Corporation copyright and SPDX identifier at the top of meshcore/MacOS/mac_logging_utils.h, before the include guard, resolving the missing license header finding. Confidence is not higher because the exact canonical header text/format and copyright year used elsewhere in the repo's meshcore/ files were not visible to verify an exact match.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 70 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_logging_utils.h missing required Apache 2.0 license header

Prepended an Apache License 2.0 block comment with Intel Corporation copyright and SPDX identifier at the top of meshcore/MacOS/mac_logging_utils.h, before the include guard, resolving the missing license header finding. Confidence is not higher because the exact canonical header text/format and copyright year used elsewhere in the repo's meshcore/ files were not visible to verify an exact match.

πŸ€– Prompt for AI agents
In meshcore/MacOS/mac_logging_utils.h around line 1, review and complete this code-review fix: mac_logging_utils.h missing required Apache 2.0 license header.
What the draft fix changed: Prepended an Apache License 2.0 block comment with Intel Corporation copyright and SPDX identifier at the top of meshcore/MacOS/mac_logging_utils.h, before the include guard, resolving the missing license header finding. Confidence is not higher because the exact canonical header text/format and copyright year used elsewhere in the repo's meshcore/ files were not visible to verify an exact match.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟑 70 medium β€” react πŸ‘/πŸ‘Ž to teach the reviewer

#define MAC_LOGGING_UTILS_H

Expand All @@ -19,3 +37,4 @@
void mesh_log_message(const char* format, ...);

#endif // MAC_LOGGING_UTILS_H

17 changes: 17 additions & 0 deletions meshcore/MacOS/mac_plist_utils.c
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
/*
Copyright 20XX Intel Corporation

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

/*

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_plist_utils.c missing required Apache 2.0 license header

Added the standard Apache License 2.0 block comment with "Copyright 20XX Intel Corporation" at the very top of meshcore/MacOS/mac_plist_utils.c, preceding the pre-existing top-of-file descriptive comment, exactly as suggested in the finding. No other code was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/mac_plist_utils.c around line 1, review and complete this code-review fix: mac_plist_utils.c missing required Apache 2.0 license header.
What the draft fix changed: Added the standard Apache License 2.0 block comment with "Copyright 20XX Intel Corporation" at the very top of meshcore/MacOS/mac_plist_utils.c, preceding the pre-existing top-of-file descriptive comment, exactly as suggested in the finding. No other code was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 95 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦩 πŸ”΄ mac_plist_utils.c missing required Apache 2.0 license header

Added the standard Apache License 2.0 block comment with "Copyright 20XX Intel Corporation" at the very top of meshcore/MacOS/mac_plist_utils.c, preceding the pre-existing top-of-file descriptive comment, exactly as suggested in the finding. No other code was modified.

πŸ€– Prompt for AI agents
In meshcore/MacOS/mac_plist_utils.c around line 1, review and complete this code-review fix: mac_plist_utils.c missing required Apache 2.0 license header.
What the draft fix changed: Added the standard Apache License 2.0 block comment with "Copyright 20XX Intel Corporation" at the very top of meshcore/MacOS/mac_plist_utils.c, preceding the pre-existing top-of-file descriptive comment, exactly as suggested in the finding. No other code was modified.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟒 95 high β€” react πŸ‘/πŸ‘Ž to teach the reviewer

Shared plist parsing utilities for macOS

Expand Down Expand Up @@ -225,3 +241,4 @@ int mesh_parse_launchdaemon_plist(const char* plistPath, MeshPlistInfo* info) {

return foundMeshagent;
}

Loading