Skip to content

docs: add DeepAgents variant blocks to Windows Prerequisites (#7307) - #7316

Merged
miyoungc merged 1 commit into
mainfrom
fix/7307-windows-prep-deepagents-blocks
Jul 21, 2026
Merged

docs: add DeepAgents variant blocks to Windows Prerequisites (#7307)#7316
miyoungc merged 1 commit into
mainfrom
fix/7307-windows-prep-deepagents-blocks

Conversation

@jason-ma-nv

@jason-ma-nv jason-ma-nv commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

The Windows Prerequisites page (windows-preparation.mdx) carried AgentOnly
blocks for openclaw and hermes in the intro, hardware-requirements, and Next
Step sections but none for deepagents, so DeepAgents readers saw generic or
OpenClaw-specific content without their variant's quickstart link or hardware
guidance. This adds the three missing DeepAgents variant blocks.

Related Issue

Fixes #7307

Changes

  • Added three AgentOnly variant="deepagents" blocks to
    docs/get-started/windows-preparation.mdx, mirroring the existing
    openclaw/hermes blocks:
    • Page intro: quickstart pointer.
    • Prerequisites → hardware requirements: DeepAgents hardware line.
    • Next Step: manual-WSL navigation paragraph to the quickstart.
  • Uses the established "Quickstart with Deep Agents" label (as in docs/index.mdx
    and the local-inference pages) and the same ../quickstart route the
    openclaw/hermes blocks use. No new canonical content — this fills documented
    coverage gaps for an existing, supported variant.

Type of Change

  • Code change (feature, bug fix, or refactor)
  • Code change with doc updates
  • Doc only (prose changes, no code sample modifications)
  • Doc only (prose changes, no code sample modifications)

Quality Gates

  • Tests added or updated for changed behavior
  • Existing tests cover changed behavior — justification:
  • Tests not applicable — justification: documentation content parity; no code behavior changes.
  • Docs updated for user-facing behavior changes
  • Docs not applicable — justification:
  • Sensitive paths changed (security, policy, credentials, preflight, onboarding, inference, runner, sandbox, or messaging)
  • Sensitive-path review completed or maintainer-approved waiver recorded — reviewer/approval link/justification:
  • Non-success, skipped, or missing CI check accepted by maintainer — check name, approval link, and follow-up issue:

Verification

  • PR description includes a Signed-off-by: line and every commit appears as Verified in GitHub
  • Normal pre-commit, commit-msg, and pre-push hooks passed, or npm run check:diff passed when hooks were skipped or unavailable
  • Targeted behavior tests pass for the current change set, or tests are marked not applicable above — command/result: n/a (docs-only); coverage verified via the docs build below.
  • Applicable broad gate passed — justification: docs-only content parity change.
  • Quality Gates section completed with required justifications or waivers
  • No secrets, API keys, or credentials committed
  • npm run docs builds without warnings (doc changes only) — result: npm run docs → 0 errors, exit 0, check-docs-published-routes: OK, locally and on a fresh clone + npm ci on an Ubuntu x86_64 host. (The 2 fern warnings are pre-existing and repo-wide: FERN_TOKEN-gated redirect check and a global light-mode accent contrast ratio; neither references this page.)
  • Doc pages follow the style guide (doc changes only)
  • New doc pages include SPDX header and frontmatter (new pages only)

Signed-off-by: Jason Ma jama@nvidia.com

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Added Windows preparation guidance for Deep Agents.
    • Included Deep Agents hardware requirements and instructions to continue with the quickstart.
    • Added next steps for installing NemoClaw and launching the first sandbox.

windows-preparation.mdx carried AgentOnly blocks for openclaw and hermes in the
intro, hardware-requirements, and Next Step sections but none for deepagents, so
DeepAgents readers saw generic or OpenClaw-specific content without their
variant's quickstart link or hardware guidance.

Add the three missing `AgentOnly variant="deepagents"` blocks, mirroring the
existing openclaw/hermes blocks and using the established "Quickstart with Deep
Agents" label and the same ../quickstart route. No new canonical content: this
fills documented-variant coverage gaps for an existing, supported variant.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 978413bc-a07d-4d40-bed1-4a57c813b864

📥 Commits

Reviewing files that changed from the base of the PR and between d5fa194 and f2e5b73.

📒 Files selected for processing (1)
  • docs/get-started/windows-preparation.mdx

📝 Walkthrough

Walkthrough

The Windows preparation page adds Deep Agents-specific introductory guidance, hardware requirements, and next-step instructions alongside the existing OpenClaw and Hermes content.

Changes

Deep Agents Windows documentation

Layer / File(s) Summary
Add Deep Agents preparation guidance
docs/get-started/windows-preparation.mdx
Adds Deep Agents instructions before quickstart, a hardware requirements reference, and next-step guidance to continue with the Deep Agents quickstart and install NemoClaw.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: miyoungc, zyang-dev

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main docs change: adding DeepAgents variant blocks to Windows prerequisites.
Linked Issues check ✅ Passed The PR adds all three requested DeepAgents blocks: intro, hardware guidance, and next-step navigation.
Out of Scope Changes check ✅ Passed The change is limited to the requested documentation page and does not introduce unrelated edits.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/7307-windows-prep-deepagents-blocks

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

PR Review Advisor — Informational

Advisor assessment: Informational / high confidence
Next action: No advisor follow-up needed.
Findings: 0 blockers · 0 warnings · 0 suggestions
Status: No actionable findings remain in the canonical review ledger.

Model lanes

  • GPT-5.6 Terra (primary): Completed · high confidence · 0 blockers · 0 warnings · 0 suggestions
  • Nemotron 3 Ultra (second opinion): Completed · high confidence · 0 blockers · 0 warnings · 0 suggestions
  • Model comparison: normalized findings match; normalized E2E selections differ; severity counts match.

Nemotron output stays in workflow artifacts and does not change the assessment above.

E2E guidance

Advisory only. E2E / PR Gate selects and runs jobs independently.

Recommended E2E: None

1 optional E2E recommendation
  • docs-validation

Workflow run details

This automated review informs maintainers. Warnings and suggestions do not require a response. A maintainer decides whether to merge.

@coderabbitai coderabbitai Bot mentioned this pull request Jul 21, 2026
22 tasks
@wscurran
wscurran requested a review from miyoungc July 21, 2026 15:50
@wscurran wscurran added area: docs Documentation, examples, guides, or docs build chore Build, CI, dependency, or tooling maintenance integration: dcode LangChain Deep Code integration behavior labels Jul 21, 2026
@miyoungc
miyoungc merged commit 8f533b4 into main Jul 21, 2026
70 checks passed
@miyoungc
miyoungc deleted the fix/7307-windows-prep-deepagents-blocks branch July 21, 2026 16:27
jyaunches added a commit that referenced this pull request Jul 21, 2026
## Summary

Complete the planned v0.0.90 changelog after the security remediation in
#7286 merged.
The release entry now records the remediated OpenClaw dependency
boundary and links to the detailed dependency review.

## Related Issue

Related to #5591 and follows #7286.

## Changes

- Add the OpenClaw core, Slack, and Microsoft Teams transitive
dependency remediation to the v0.0.90 summary and release bullets.
- Record fail-closed archive validation and the reviewed residual audit
findings.
- Link the detailed OpenClaw 2026.6.10 dependency review.
- Source summary: #7286 -> `docs/changelog/2026-07-20.mdx`.
- Reconciled #7316 as documentation-only coverage for existing Deep
Agents behavior; its merged source page is already the canonical record
and requires no separate release behavior entry.

## Type of Change

- [ ] Code change (feature, bug fix, or refactor)
- [ ] Code change with doc updates
- [x] Doc only (prose changes, no code sample modifications)
- [ ] Doc only (includes code sample changes)

## Quality Gates

- [ ] Tests added or updated for changed behavior
- [x] Existing tests cover changed behavior — justification:
`test/changelog-docs.test.ts` validates changelog structure and links,
and the full docs build validates generated content and routes.
- [ ] Tests not applicable — justification:
- [x] Docs updated for user-facing behavior changes
- [ ] Docs not applicable — justification:
- [ ] Sensitive paths changed (security, policy, credentials, preflight,
onboarding, inference, runner, sandbox, or messaging)
- [ ] Sensitive-path review completed or maintainer-approved waiver
recorded — reviewer/approval link/justification:
- [ ] Non-success, skipped, or missing CI check accepted by maintainer —
check name, approval link, and follow-up issue:

## DGX Station Hardware Evidence

- [ ] Tested on DGX Station
- Tested commit: not applicable
- Station profile/scenario: not applicable
- Result: not applicable
- Supporting evidence: not applicable

## Verification

- [x] PR description includes a `Signed-off-by:` line and every commit
appears as `Verified` in GitHub
- [x] Normal `pre-commit`, `commit-msg`, and `pre-push` hooks passed, or
`npm run check:diff` passed when hooks were skipped or unavailable
- [x] Targeted behavior tests pass for the current change set, or tests
are marked not applicable above — `npx vitest run
test/changelog-docs.test.ts`: 6/6 passed
- [x] Applicable broad gate passed — `npm run docs`: 0 errors; the two
repository-wide pre-existing Fern warnings remain
- [x] Quality Gates section completed with required justifications or
waivers
- [x] No secrets, API keys, or credentials committed
- [x] `npm run docs` builds without warnings (doc changes only) — 0
errors; the two repository-wide pre-existing Fern warnings remain
- [x] Doc pages follow the [style
guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md)
(doc changes only)
- [ ] New doc pages include SPDX header and frontmatter (new pages only)

---

Signed-off-by: Julie Yaunches <jyaunches@nvidia.com>


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Documentation**
* Clarified inference provider routing and credential handling for
managed agent images.
  * Added migration guidance ahead of the upcoming fallback removal.
* Documented security updates for OpenClaw, Slack, and Microsoft Teams
installations, including verified package remediation and archive
validation.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Signed-off-by: Julie Yaunches <jyaunches@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: docs Documentation, examples, guides, or docs build chore Build, CI, dependency, or tooling maintenance integration: dcode LangChain Deep Code integration behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DeepAgents Windows Prerequisites page missing three AgentOnly variant blocks

3 participants