docs: add CONTRIBUTING.md#4079
Conversation
📝 WalkthroughWalkthroughThis PR adds four planning and documentation files to the NemoClaw repository fork. The changes establish the current repository state, map its structure, define PR candidate selection criteria with a prioritized list of ten candidates, and provide a detailed action plan for five selected PRs including fork steps and verification commands. ChangesNemoClaw Repository Planning Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment Warning |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@00_STATE.md`:
- Line 1: Add the required SPDX license header to the top of 00_STATE.md: insert
a one-line SPDX copyright/ license declaration (including
SPDX-License-Identifier: Apache-2.0) and a short copyright holder line as the
first lines of the file so the file complies with the repository rule for all
source files.
In `@01_REPO_MAP.md`:
- Line 1: Add the required SPDX copyright/license header for Apache-2.0 at the
top of the Markdown file before the H1; specifically, open the file that
contains the line "# 01_REPO_MAP.md — NemoClaw" and insert the standardized SPDX
header (copyright notice + SPDX-License-Identifier: Apache-2.0) as the very
first lines above the existing H1.
In `@05_PR_CANDIDATES.md`:
- Line 1: Add the required SPDX license header at the very top of the file by
inserting the SPDX copyright and license lines (e.g., include an
SPDX-FileCopyrightText line for copyright holder and an SPDX-License-Identifier:
Apache-2.0 line) before the existing content or heading in 05_PR_CANDIDATES.md
so the file complies with the repository rule that every source/markdown file
include an SPDX header.
In `@06_SELECTED_5_PR_PLAN.md`:
- Line 1: The file 06_SELECTED_5_PR_PLAN.md is missing the required SPDX license
header; prepend a standard SPDX copyright notice and the "Apache-2.0" SPDX
identifier at the very top of 06_SELECTED_5_PR_PLAN.md so the file complies with
the repository rule that every source file (including .md) contains an SPDX
header for copyright and Apache-2.0 licensing.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 22ac1cbe-e5b1-4ef7-8233-95f76af326e2
📒 Files selected for processing (4)
00_STATE.md01_REPO_MAP.md05_PR_CANDIDATES.md06_SELECTED_5_PR_PLAN.md
| @@ -0,0 +1,124 @@ | |||
| # 00_STATE.md — NemoClaw | |||
There was a problem hiding this comment.
Add the required SPDX license header at the top of the file.
00_STATE.md currently starts directly with the title and is missing the required SPDX copyright/license header.
Suggested fix
+<!-- SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. -->
+<!-- SPDX-License-Identifier: Apache-2.0 -->
+
# 00_STATE.md — NemoClawAs per coding guidelines, "**/*.{js,ts,tsx,jsx,sh,yaml,yml,json,md,mdx}: Every source file must include an SPDX license header for copyright and Apache-2.0 license".
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| # 00_STATE.md — NemoClaw | |
| <!-- SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. --> | |
| <!-- SPDX-License-Identifier: Apache-2.0 --> | |
| # 00_STATE.md — NemoClaw |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@00_STATE.md` at line 1, Add the required SPDX license header to the top of
00_STATE.md: insert a one-line SPDX copyright/ license declaration (including
SPDX-License-Identifier: Apache-2.0) and a short copyright holder line as the
first lines of the file so the file complies with the repository rule for all
source files.
| @@ -0,0 +1,266 @@ | |||
| # 01_REPO_MAP.md — NemoClaw | |||
There was a problem hiding this comment.
Missing SPDX header in this Markdown source file.
Please add the SPDX copyright/license header before the H1.
As per coding guidelines, "**/*.{js,ts,tsx,jsx,sh,yaml,yml,json,md,mdx}: Every source file must include an SPDX license header for copyright and Apache-2.0 license".
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@01_REPO_MAP.md` at line 1, Add the required SPDX copyright/license header for
Apache-2.0 at the top of the Markdown file before the H1; specifically, open the
file that contains the line "# 01_REPO_MAP.md — NemoClaw" and insert the
standardized SPDX header (copyright notice + SPDX-License-Identifier:
Apache-2.0) as the very first lines above the existing H1.
| @@ -0,0 +1,133 @@ | |||
| # 05_PR_CANDIDATES.md — NemoClaw | |||
There was a problem hiding this comment.
Add SPDX license metadata at file start.
05_PR_CANDIDATES.md is missing the required SPDX header.
As per coding guidelines, "**/*.{js,ts,tsx,jsx,sh,yaml,yml,json,md,mdx}: Every source file must include an SPDX license header for copyright and Apache-2.0 license".
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@05_PR_CANDIDATES.md` at line 1, Add the required SPDX license header at the
very top of the file by inserting the SPDX copyright and license lines (e.g.,
include an SPDX-FileCopyrightText line for copyright holder and an
SPDX-License-Identifier: Apache-2.0 line) before the existing content or heading
in 05_PR_CANDIDATES.md so the file complies with the repository rule that every
source/markdown file include an SPDX header.
| @@ -0,0 +1,232 @@ | |||
| # 06_SELECTED_5_PR_PLAN.md — NemoClaw | |||
There was a problem hiding this comment.
SPDX header is required but absent.
Please prepend the standard SPDX copyright and Apache-2.0 identifier at the top of this file.
As per coding guidelines, "**/*.{js,ts,tsx,jsx,sh,yaml,yml,json,md,mdx}: Every source file must include an SPDX license header for copyright and Apache-2.0 license".
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@06_SELECTED_5_PR_PLAN.md` at line 1, The file 06_SELECTED_5_PR_PLAN.md is
missing the required SPDX license header; prepend a standard SPDX copyright
notice and the "Apache-2.0" SPDX identifier at the very top of
06_SELECTED_5_PR_PLAN.md so the file complies with the repository rule that
every source file (including .md) contains an SPDX header for copyright and
Apache-2.0 licensing.
1 participant
Summary
Adds CONTRIBUTING.md
Testing
N/A
Summary by CodeRabbit
Release Notes