docs: change alpha statement into banner, add disclaimer#1209
Conversation
📝 WalkthroughWalkthroughReplaced embedded alpha-statement includes with explicit disclaimer content and theme announcement entries: added a "Notice and Disclaimer" to README and docs pages, removed the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
README.md (1)
1-1:⚠️ Potential issue | 🟠 MajorAdd the required SPDX Markdown header at the top of this file.
This file currently starts at Line 1 with the H1, so it misses the required SPDX HTML comment header for Markdown sources.
Proposed fix
+<!-- + SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. + SPDX-License-Identifier: Apache-2.0 +--> + # 🦞 NVIDIA NemoClaw: Reference Stack for Running OpenClaw in OpenShellAs per coding guidelines, "Include SPDX license header at the top of every source file ... Use ... HTML comments for Markdown files."
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` at line 1, Add the required SPDX HTML comment header at the very top of README.md (before the existing H1 line) by inserting the SPDX license identifier comment (e.g., <!-- SPDX-License-Identifier: <LICENSE> -->) so the Markdown file includes the mandated SPDX header; update the placeholder <LICENSE> to the project's actual SPDX identifier to match the repository license.
🧹 Nitpick comments (2)
docs/index.md (1)
193-193: Split the disclaimer into one sentence per line and rewrite passive phrasing.Line 193 packs many sentences into one line and uses passive voice in multiple places (for example, “are governed” / “is provided”). Please keep one sentence per source line and switch to active voice where possible.
As per coding guidelines, "Every sentence must end with a period. One sentence per line in source" and "Active voice required. Flag passive constructions."
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/index.md` at line 193, Split the long disclaimer paragraph into separate sentences with one sentence per source line and convert passive constructions to active voice; specifically, rewrite the disclaimer block (the disclaimer paragraph that begins "This software automatically retrieves, accesses or interacts with external materials.") so each sentence ends with a period and stands on its own line, replace passive phrases like "are governed" and "is provided" with active phrasing (e.g., "Separate terms govern those materials" and "The author provides this software 'AS IS' without warranty"), and keep the original meaning and legal content while enforcing one-sentence-per-line and active-voice requirements.docs/conf.py (1)
99-100: Remove emoji and routine bolding from the announcement copy.The announcement text uses a bell emoji and
<strong>emphasis for routine warning text. LLM pattern detected.Proposed fix
- "announcement": ( - "🔔 NemoClaw is <strong>alpha software</strong>. APIs and behavior" - " may change without notice. Do not use in production." - ), + "announcement": ( + "NemoClaw is alpha software. APIs and behavior may change without notice. " + "Do not use in production." + ),As per coding guidelines, "LLM-generated patterns ... Emoji in documentation prose" and "Unnecessary bold on routine instructions ... should be flagged."
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/conf.py` around lines 99 - 100, Update the announcement string that currently includes a bell emoji and HTML <strong> tags so it reads as plain text without emoji or markup; replace the concatenated string fragment "🔔 NemoClaw is <strong>alpha software</strong>. APIs and behavior" " may change without notice. Do not use in production." with a plain-text sentence like "NemoClaw is alpha software. APIs and behavior may change without notice. Do not use in production." so the literal in the module (the announcement constant/string in docs/conf.py) contains no emoji or HTML emphasis.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@README.md`:
- Line 1: Add the required SPDX HTML comment header at the very top of README.md
(before the existing H1 line) by inserting the SPDX license identifier comment
(e.g., <!-- SPDX-License-Identifier: <LICENSE> -->) so the Markdown file
includes the mandated SPDX header; update the placeholder <LICENSE> to the
project's actual SPDX identifier to match the repository license.
---
Nitpick comments:
In `@docs/conf.py`:
- Around line 99-100: Update the announcement string that currently includes a
bell emoji and HTML <strong> tags so it reads as plain text without emoji or
markup; replace the concatenated string fragment "🔔 NemoClaw is <strong>alpha
software</strong>. APIs and behavior" " may change without notice. Do not use in
production." with a plain-text sentence like "NemoClaw is alpha software. APIs
and behavior may change without notice. Do not use in production." so the
literal in the module (the announcement constant/string in docs/conf.py)
contains no emoji or HTML emphasis.
In `@docs/index.md`:
- Line 193: Split the long disclaimer paragraph into separate sentences with one
sentence per source line and convert passive constructions to active voice;
specifically, rewrite the disclaimer block (the disclaimer paragraph that begins
"This software automatically retrieves, accesses or interacts with external
materials.") so each sentence ends with a period and stands on its own line,
replace passive phrases like "are governed" and "is provided" with active
phrasing (e.g., "Separate terms govern those materials" and "The author provides
this software 'AS IS' without warranty"), and keep the original meaning and
legal content while enforcing one-sentence-per-line and active-voice
requirements.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: ec4f7e45-716a-4dc3-933d-3765fc51870b
📒 Files selected for processing (5)
README.mddocs/about/overview.mddocs/about/release-notes.mddocs/conf.pydocs/index.md
💤 Files with no reviewable changes (2)
- docs/about/release-notes.md
- docs/about/overview.md
<!-- markdownlint-disable MD041 --> ## Summary Preview: https://nvidia.github.io/NemoClaw/pr-preview/pr-1209/ ## Related Issue <!-- Link to the issue: Fixes #NNN or Closes #NNN. Remove this section if none. --> ## Changes <!-- Bullet list of key changes. --> ## Type of Change <!-- Check the one that applies. --> - [ ] Code change for a new feature, bug fix, or refactor. - [ ] Code change with doc updates. - [x] Doc only. Prose changes without code sample modifications. - [ ] Doc only. Includes code sample changes. ## Testing <!-- What testing was done? --> - [ ] `npx prek run --all-files` passes (or equivalently `make check`). - [ ] `npm test` passes. - [x] `make docs` builds without warnings. (for doc-only changes) ## Checklist ### General - [ ] I have read and followed the [contributing guide](https://github.com/NVIDIA/NemoClaw/blob/main/CONTRIBUTING.md). - [ ] I have read and followed the [style guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md). (for doc-only changes) ### Code Changes <!-- Skip if this is a doc-only PR. --> - [ ] Formatters applied — `npx prek run --all-files` auto-fixes formatting (or `make format` for targeted runs). - [ ] Tests added or updated for new or changed behavior. - [ ] No secrets, API keys, or credentials committed. - [ ] Doc pages updated for any user-facing behavior changes (new commands, changed defaults, new features, bug fixes that contradict existing docs). ### Doc Changes <!-- Skip if this PR has no doc changes. --> - [ ] Follows the [style guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md). Try running the `update-docs` agent skill to draft changes while complying with the style guide. For example, prompt your agent with "`/update-docs` catch up the docs for the new changes I made in this PR." - [ ] New pages include SPDX license header and frontmatter, if creating a new page. - [ ] Cross-references and links verified. --- <!-- DCO sign-off (required by CI). Replace with your real name and email. --> Signed-off-by: Your Name <your-email@example.com> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Updated README and site docs with a new Notice & Disclaimer clarifying the software’s alpha status, advising against production use, and adding warranty and liability disclaimers. * Added a prominent announcement/warning on the docs homepage and release notes, removed the prior embedded alpha include, and noted that the software may access external materials subject to separate terms—users must verify suitability and compliance. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
<!-- markdownlint-disable MD041 --> ## Summary Preview: https://nvidia.github.io/NemoClaw/pr-preview/pr-1209/ ## Related Issue <!-- Link to the issue: Fixes #NNN or Closes #NNN. Remove this section if none. --> ## Changes <!-- Bullet list of key changes. --> ## Type of Change <!-- Check the one that applies. --> - [ ] Code change for a new feature, bug fix, or refactor. - [ ] Code change with doc updates. - [x] Doc only. Prose changes without code sample modifications. - [ ] Doc only. Includes code sample changes. ## Testing <!-- What testing was done? --> - [ ] `npx prek run --all-files` passes (or equivalently `make check`). - [ ] `npm test` passes. - [x] `make docs` builds without warnings. (for doc-only changes) ## Checklist ### General - [ ] I have read and followed the [contributing guide](https://github.com/NVIDIA/NemoClaw/blob/main/CONTRIBUTING.md). - [ ] I have read and followed the [style guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md). (for doc-only changes) ### Code Changes <!-- Skip if this is a doc-only PR. --> - [ ] Formatters applied — `npx prek run --all-files` auto-fixes formatting (or `make format` for targeted runs). - [ ] Tests added or updated for new or changed behavior. - [ ] No secrets, API keys, or credentials committed. - [ ] Doc pages updated for any user-facing behavior changes (new commands, changed defaults, new features, bug fixes that contradict existing docs). ### Doc Changes <!-- Skip if this PR has no doc changes. --> - [ ] Follows the [style guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md). Try running the `update-docs` agent skill to draft changes while complying with the style guide. For example, prompt your agent with "`/update-docs` catch up the docs for the new changes I made in this PR." - [ ] New pages include SPDX license header and frontmatter, if creating a new page. - [ ] Cross-references and links verified. --- <!-- DCO sign-off (required by CI). Replace with your real name and email. --> Signed-off-by: Your Name <your-email@example.com> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Updated README and site docs with a new Notice & Disclaimer clarifying the software’s alpha status, advising against production use, and adding warranty and liability disclaimers. * Added a prominent announcement/warning on the docs homepage and release notes, removed the prior embedded alpha include, and noted that the software may access external materials subject to separate terms—users must verify suitability and compliance. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
<!-- markdownlint-disable MD041 --> ## Summary Preview: https://nvidia.github.io/NemoClaw/pr-preview/pr-1209/ ## Related Issue <!-- Link to the issue: Fixes #NNN or Closes #NNN. Remove this section if none. --> ## Changes <!-- Bullet list of key changes. --> ## Type of Change <!-- Check the one that applies. --> - [ ] Code change for a new feature, bug fix, or refactor. - [ ] Code change with doc updates. - [x] Doc only. Prose changes without code sample modifications. - [ ] Doc only. Includes code sample changes. ## Testing <!-- What testing was done? --> - [ ] `npx prek run --all-files` passes (or equivalently `make check`). - [ ] `npm test` passes. - [x] `make docs` builds without warnings. (for doc-only changes) ## Checklist ### General - [ ] I have read and followed the [contributing guide](https://github.com/NVIDIA/NemoClaw/blob/main/CONTRIBUTING.md). - [ ] I have read and followed the [style guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md). (for doc-only changes) ### Code Changes <!-- Skip if this is a doc-only PR. --> - [ ] Formatters applied — `npx prek run --all-files` auto-fixes formatting (or `make format` for targeted runs). - [ ] Tests added or updated for new or changed behavior. - [ ] No secrets, API keys, or credentials committed. - [ ] Doc pages updated for any user-facing behavior changes (new commands, changed defaults, new features, bug fixes that contradict existing docs). ### Doc Changes <!-- Skip if this PR has no doc changes. --> - [ ] Follows the [style guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md). Try running the `update-docs` agent skill to draft changes while complying with the style guide. For example, prompt your agent with "`/update-docs` catch up the docs for the new changes I made in this PR." - [ ] New pages include SPDX license header and frontmatter, if creating a new page. - [ ] Cross-references and links verified. --- <!-- DCO sign-off (required by CI). Replace with your real name and email. --> Signed-off-by: Your Name <your-email@example.com> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Updated README and site docs with a new Notice & Disclaimer clarifying the software’s alpha status, advising against production use, and adding warranty and liability disclaimers. * Added a prominent announcement/warning on the docs homepage and release notes, removed the prior embedded alpha include, and noted that the software may access external materials subject to separate terms—users must verify suitability and compliance. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Summary
Preview: https://nvidia.github.io/NemoClaw/pr-preview/pr-1209/
Related Issue
Changes
Type of Change
Testing
npx prek run --all-filespasses (or equivalentlymake check).npm testpasses.make docsbuilds without warnings. (for doc-only changes)Checklist
General
Code Changes
npx prek run --all-filesauto-fixes formatting (ormake formatfor targeted runs).Doc Changes
update-docsagent skill to draft changes while complying with the style guide. For example, prompt your agent with "/update-docscatch up the docs for the new changes I made in this PR."Signed-off-by: Your Name your-email@example.com
Summary by CodeRabbit