Skip to content

docs: align Station coding-agent path with installer - #7264

Merged
miyoungc merged 2 commits into
mainfrom
docs/station-installer-model-alignment
Jul 20, 2026
Merged

docs: align Station coding-agent path with installer#7264
miyoungc merged 2 commits into
mainfrom
docs/station-installer-model-alignment

Conversation

@miyoungc

@miyoungc miyoungc commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

Align the DGX Station coding-agent instructions with the versioned installer. The ordinary Station Express path now remains authoritative for its Nemotron Ultra default, while DeepSeek remains available only through the installer’s explicit --station-deepseek override.

Changes

  • #7061 -> docs/resources/prompt-assets/dgx-station.md: Replace the DeepSeek-only provider-preseeded path with the installer’s Ultra-default and DeepSeek-override contract.
  • Delegate Station qualification, host preparation, third-party notice, Express summary, and reboot or login resume to the selected maintained release’s installer.
  • Stop the coding-agent path from pre-setting provider, model, non-interactive, approval, or Express-skip variables, and stop it from running the Station preparation helper separately.
  • Preserve Ultra’s approximately 352 GB disclosure, shared Station host-safety boundaries, the temporary metadata override guard, and Deferred support status.
  • Pin every starter-prompt platform asset URL to the immutable asset commit and update its SHA-256 contract.
  • Update focused tests to require both installer-supported Station model outcomes and reject the former DeepSeek-only path.

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 (includes code sample changes)

Quality Gates

  • Tests added or updated for changed behavior
  • Existing tests cover changed behavior — justification:
  • Tests not applicable — justification:
  • 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; this PR does not change scripts/prepare-dgx-station-host.sh or runtime behavior.
  • Station profile/scenario: Not applicable.
  • Result: Not applicable.
  • Supporting evidence: Installer behavior is covered by the focused automated Express tests below.

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 — npx vitest run test/starter-prompt-docs.test.ts test/install-express-prompt.test.ts (88 passed, 1 skipped)
  • Applicable broad gate passed — npm test for broad runtime/test-harness changes; npm run check for repo-wide validation/coverage changes — command/result: Not run; this is a focused documentation contract 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) — passed with 0 errors and 1 Fern warning
  • Doc pages follow the style guide (doc changes only)
  • New doc pages include SPDX header and frontmatter (new pages only)

Signed-off-by: Miyoung Choi miyoungc@nvidia.com

Summary by CodeRabbit

  • Documentation

    • Updated DGX Station guidance to use the official Station Express installer after hardware detection.
    • Added support for Nemotron Ultra by default and DeepSeek V4 Flash as an explicit option.
    • Clarified installer consent, interactive prompts, reboot/resume behavior, and fallback provider selection.
    • Refreshed platform instruction links and clarified installer-specific prompt handling.
  • Tests

    • Updated validation for revised Station Express instructions, consent messaging, and asset revisions.

miyoungc added 2 commits July 20, 2026 12:49
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
@miyoungc miyoungc added area: docs Documentation, examples, guides, or docs build platform: dgx-station Affects DGX Station hardware or workflows labels Jul 20, 2026
@miyoungc miyoungc self-assigned this Jul 20, 2026
@copy-pr-bot

copy-pr-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

DGX Station documentation now uses the Station Express installer flow, with updated model selection, consent, interactive prompt, and preparation rules. Starter-prompt asset pins and contract tests are updated to validate the new content and behavior.

Changes

DGX Station Express flow

Layer / File(s) Summary
Station Express instructions
docs/resources/prompt-assets/dgx-station.md
Replaces the prepared-host DeepSeek instructions with installer-driven Express model selection, consent, secure terminal, confirmation, and resume rules.
Starter-prompt wiring
docs/resources/starter-prompt.md
Updates platform asset revisions and adds DGX Station exceptions for visible installer prompts and installer-managed consent.
Pinned asset and contract validation
test/starter-prompt-docs.test.ts
Repins remote assets and updates Station content, ordering, model-selection, and absence assertions.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

  • NVIDIA/NemoClaw#6883: Introduces the installer and model-selection flow reflected in these Station instructions.
  • NVIDIA/NemoClaw#7061: Updates the same DGX Station asset and related starter-prompt contract tests.
  • NVIDIA/NemoClaw#7263: Modifies the same Station assets, pinning, and validation expectations.

Suggested labels: area: install, area: onboarding

Suggested reviewers: cv

🚥 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 reflects the main change: aligning the DGX Station coding-agent path with the installer.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/station-installer-model-alignment

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

@github-code-quality

github-code-quality Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Code Coverage Overview

Languages: TypeScript

TypeScript / code-coverage/plugin

The overall coverage in commit 580e84f in the docs/station-install... branch remains at 96%, unchanged from commit ac120c8 in the main branch.

TypeScript / code-coverage/cli

The overall coverage in commit 580e84f in the docs/station-install... branch remains at 80%, unchanged from commit ad26c30 in the main branch.

Show a code coverage summary of the most impacted files.
File main ad26c30 docs/station-install... 580e84f +/-
src/lib/inferen...lama/process.ts 100% 50% -50%
src/lib/inferen...er-lifecycle.ts 71% 65% -6%
src/lib/platform.ts 89% 84% -5%
src/lib/actions...annel-status.ts 93% 91% -2%
src/lib/actions...test-helpers.ts 65% 63% -2%
src/lib/adapter...shell/client.ts 88% 90% +2%
src/lib/messagi...onfig-parser.ts 50% 100% +50%
src/lib/messagi...-health-eval.ts 0% 88% +88%
src/lib/messagi...tatus-health.ts 0% 92% +92%
src/lib/messagi.../hooks/index.ts 0% 100% +100%

Updated July 20, 2026 20:03 UTC

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Jul 20, 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.

@miyoungc
miyoungc marked this pull request as ready for review July 20, 2026 20:01

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 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 `@docs/resources/prompt-assets/dgx-station.md`:
- Around line 20-46: Update the DGX Station Express documentation bullets so
each sentence is on its own Markdown source line, especially the multi-sentence
bullets covering host preparation, model downloads, installer options, and
confirmation behavior. Preserve the existing wording and paragraph structure;
only split the affected sentences without changing their meaning.

In `@docs/resources/starter-prompt.md`:
- Line 126: Split the sentence in the DGX Station asset guidance into two
separate source lines: keep the exception statement on one line and the
installer-prompt instruction on the next, preserving the existing wording and
meaning.
🪄 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: ab920bf6-dac0-4625-89a6-49e4aa3127bb

📥 Commits

Reviewing files that changed from the base of the PR and between 888c3d9 and 580e84f.

📒 Files selected for processing (3)
  • docs/resources/prompt-assets/dgx-station.md
  • docs/resources/starter-prompt.md
  • test/starter-prompt-docs.test.ts

Comment thread docs/resources/prompt-assets/dgx-station.md
Comment thread docs/resources/starter-prompt.md
@miyoungc
miyoungc merged commit fc2fde7 into main Jul 20, 2026
91 of 92 checks passed
@miyoungc
miyoungc deleted the docs/station-installer-model-alignment branch July 20, 2026 21:00
@jyaunches jyaunches mentioned this pull request Jul 20, 2026
22 tasks
jyaunches added a commit that referenced this pull request Jul 20, 2026
<!-- markdownlint-disable MD041 -->
## Summary

Add the canonical `## v0.0.90` entry to `docs/changelog/2026-07-20.mdx`
before the release tag is planned.
The update also corrects the documented custom-image migration window so
the compatibility fallback that first ships in v0.0.90 remains available
until v0.0.91.

## Changes

- Add the v0.0.90 summary and detailed release bullets for managed-image
routing, provider-reset recovery, WhatsApp health reporting, and DGX
Station guidance.
- Keep the newest release first in the shared dated changelog and use
root-absolute links to the canonical OpenClaw routes.
- Correct `docs/reference/commands.mdx` to state that the legacy image
route selector remains supported through v0.0.90 and is removed in
v0.0.91.
- Release source summary:
- [#7264](#7264) ->
`docs/resources/prompt-assets/dgx-station.md`,
`docs/changelog/2026-07-20.mdx`: Record the versioned Station installer
path, Nemotron 3 Ultra 550B default, and explicit DeepSeek override.
- [#7261](#7261) ->
`docs/get-started/dgx-station-preparation.mdx`,
`docs/manage-sandboxes/recover-rebuild-sandboxes.mdx`,
`docs/changelog/2026-07-20.mdx`: Include the OpenIB, legacy recovery,
and Additional Setup documentation follow-ups.
- [#7232](#7232) ->
`docs/changelog/2026-07-20.mdx`: Document provider-reset recovery for
wrapped OpenShell attachment diagnostics.
- [#7189](#7189) ->
`docs/reference/commands.mdx`, `docs/changelog/2026-07-20.mdx`: Document
the managed-image route-selector rename and correct its one-release
migration window.
- [#7015](#7015) ->
`docs/changelog/2026-07-20.mdx`: Document corrected OpenClaw WhatsApp
health reporting.
- No additional user-facing page update is needed for
[#7193](#7193),
[#7110](#7110),
[#6783](#6783), or
[#7263](#7263) because they
change contributor governance, internal CI or release automation, or
editorial style without changing supported user behavior.
- [#7242](#7242) and
[#7225](#7225) are already
ancestors of and documented in v0.0.89, so this entry does not duplicate
them despite their stale v0.0.90 labels.

## 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 the dated changelog heading,
SPDX form, version order, and published links.
- [ ] 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; this PR does not change
`scripts/prepare-dgx-station-host.sh` or runtime behavior.
- 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 passed).
- [ ] Applicable broad gate passed — `npm test` for broad
runtime/test-harness changes; `npm run check` for repo-wide
validation/coverage changes — command/result: Not run; this is a focused
documentation-only change.
- [x] Quality Gates section completed with required justifications or
waivers
- [x] No secrets, API keys, or credentials committed
- [ ] `npm run docs` builds without warnings (doc changes only) — passed
with 0 errors and two unrelated baseline warnings for unauthenticated
redirect checks and the existing light-mode contrast ratio.
- [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**
- Added release notes for v0.0.90 covering inference routing, credential
reset behavior, WhatsApp status detection, and DGX Station coding-agent
guidance.
- Updated custom Dockerfile guidance to document continued support for
the legacy provider argument through v0.0.90.
- Clarified that legacy declarations must be renamed to
`NEMOCLAW_INFERENCE_PROVIDER_ID` before v0.0.91.
  - Added and refreshed related documentation links.

<!-- 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 platform: dgx-station Affects DGX Station hardware or workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants