fix(cli): add reconfiguration guidance to help and post-onboard output by latenighthackathon · Pull Request #1725 · NVIDIA/NemoClaw

latenighthackathon · 2026-04-10T04:24:39Z

Summary

New users hit openclaw configure inside the sandbox and get a read-only error with no guidance on what to do instead. This adds clear reconfiguration instructions in two places.

Problem

openclaw.json is read-only inside the sandbox (Landlock enforced) by design. When users try openclaw configure, they see:

Error: 'openclaw configure' cannot modify config inside the sandbox.
The sandbox config is read-only (Landlock enforced) for security.

No NemoClaw documentation or CLI output explains the three available reconfiguration paths. Users are left guessing.

Changes

nemoclaw --help: Added a "Reconfiguration (after onboard)" section listing the three paths:
- Runtime model switch: openshell inference set
- Policy presets: nemoclaw <name> policy-add
- Credential/image changes: nemoclaw credentials reset <KEY> then nemoclaw onboard
Post-onboard dashboard: Added a "To change settings later" block at the end of the completion output so users see the reconfiguration paths immediately after their first onboard.

Test plan

Closes #1699

Signed-off-by: latenighthackathon latenighthackathon@users.noreply.github.com

Summary by CodeRabbit

Documentation
- Added a "To change settings later" section to onboarding output with copy‑paste reconfiguration commands for model routing, sandbox policies, and credential reset.
- Extended CLI help with a "Reconfiguration (after onboard)" section describing how to change inference models/providers, apply network/policy updates to running sandboxes, and when re‑onboarding is needed.
- Confirmed runtime behavior and public APIs remain unchanged.

coderabbitai · 2026-04-10T04:24:54Z

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

@coderabbitai resume to resume automatic reviews.
@coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

▶️ Resume reviews
🔍 Trigger review

📝 Walkthrough

Walkthrough

Added post-onboard reconfiguration guidance to two CLI outputs (dashboard and help): copy-paste commands for changing model routing, adding sandbox policies, and resetting credentials then re-onboarding. No functional, API, or control-flow changes.

Changes

Cohort / File(s)	Summary
Onboard dashboard text `src/lib/onboard.ts`	Appended a "To change settings later:" section to the dashboard output with three copy-paste reconfiguration command hints: change model routing via `openshell inference set ...`, apply policies via `nemoclaw <name> policy-add`, and reset credentials then run `nemoclaw onboard`. Text-only; no logic/API changes.
CLI help text `src/nemoclaw.ts`	Expanded `help()` output with a "Reconfiguration (after onboard)" section documenting the same runtime guidance and clarifying that `openclaw.json` is read-only inside the sandbox (Landlock) and that OpenClaw config changes require re-onboarding. Text-only change.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 I nudged the README, left commands to paste,

Change the model, add policy, reset with haste,
Re-onboard once, and your settings will stick —
A hop, a tweak, then back to the trick! 🥕✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main changes: adding reconfiguration guidance to both help output and post-onboard output, which aligns with the PR's primary objective.
Linked Issues check	✅ Passed	The PR implements documentation-based solutions (`#1699` requirements) by providing clear reconfiguration paths in help and post-onboard output, effectively surfacing existing mechanisms.
Out of Scope Changes check	✅ Passed	All changes are focused on adding documentation and UI guidance for reconfiguration; no unrelated code changes or scope creep detected.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

🧹 Nitpick comments (1)

src/nemoclaw.ts (1)

1384-1384: Verb choice: "Change" would be more accurate than "Add".

The documented workflow (nemoclaw credentials reset <KEY> then nemoclaw onboard) is for changing existing credentials (reset removes them, re-onboard re-prompts), not adding new ones. The verb "Add" is misleading here.

Suggested wording fix

-    ${D}Add credentials, messaging channels, or sandbox image settings:${R}
+    ${D}Change credentials, messaging channels, or sandbox image settings:${R}
      nemoclaw credentials reset <KEY>   ${D}then${R}   nemoclaw onboard

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/nemoclaw.ts` at line 1384, The message text currently uses the verb "Add"
but should say "Change" to accurately describe the workflow; locate the
template/string in src/nemoclaw.ts that contains "${D}Change credentials,
messaging channels, or sandbox image settings:${R}" (or the version currently
using "Add") and replace "Add" with "Change" so the CLI help/readme reflects the
correct action for the "nemoclaw credentials reset <KEY> then nemoclaw onboard"
flow.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/nemoclaw.ts`:
- Line 1384: The message text currently uses the verb "Add" but should say
"Change" to accurately describe the workflow; locate the template/string in
src/nemoclaw.ts that contains "${D}Change credentials, messaging channels, or
sandbox image settings:${R}" (or the version currently using "Add") and replace
"Add" with "Change" so the CLI help/readme reflects the correct action for the
"nemoclaw credentials reset <KEY> then nemoclaw onboard" flow.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c3d220d6-c05f-4fe5-861e-9f8a5ac35969

📥 Commits

Reviewing files that changed from the base of the PR and between 9f549db and 765fbda.

📒 Files selected for processing (2)

src/lib/onboard.ts
src/nemoclaw.ts

wscurran · 2026-04-10T14:25:55Z

✨ Thanks for submitting this PR, which proposes a fix for a documentation issue and may improve the overall user experience — the added guidance and explanations are clear and easy to follow.

Possibly related open issues:

#1699 Provide a path to edit openclaw.json after setup, or a simple way to fully specify it during onboarding

wscurran · 2026-04-15T18:05:09Z

Thanks for contributing — adding reconfiguration guidance to the help output and post-onboard messaging fills a real discoverability gap for users who need to change their setup after initial onboard. The codebase has changed significantly since this was opened, including a TypeScript migration (#1673). Could you rebase onto the current main?

latenighthackathon · 2026-04-16T04:35:10Z

@wscurran Thanks for picking this up — you're right, the branch had gotten stale. Rebased onto latest origin/main — single clean commit (5046888d) against the current TypeScript-migrated src/nemoclaw.ts and src/lib/onboard.ts, docstrings preserved, signed-off. Cheers!

prekshivyas

Good UX improvement — fills a real gap for users hitting read-only sandbox config. @latenighthackathon can you rebase onto main?

latenighthackathon · 2026-04-17T01:34:02Z

@prekshivyas Rebased onto latest origin/main — single clean commit (ea584eeb), signed.

Cheers!

prekshivyas · 2026-04-20T21:58:18Z

@latenighthackathon please give access to update branch and this should be good to go

New users hit 'openclaw configure' inside the sandbox and get a read-only error with no guidance on what to do instead. Add a Reconfiguration section to 'nemoclaw --help' and a 'To change settings later' block to the post-onboard dashboard output, covering the three main reconfiguration paths: runtime model switch via openshell inference set, policy presets via nemoclaw policy-add, and credential reset + re-onboard for image-level changes. Closes NVIDIA#1699 Signed-off-by: latenighthackathon <latenighthackathon@users.noreply.github.com>

latenighthackathon · 2026-04-21T13:21:19Z

@prekshivyas — maintainer_can_modify has been on since 2026-04-18, but GitHub's Update Branch likely hit the rebaseable: false edge because the branch had drifted. Rebased locally onto current main — single signed commit b3fcdcd8, same diff. Cheers!

coderabbitai Bot reviewed Apr 10, 2026

View reviewed changes

wscurran added documentation labels Apr 10, 2026

wscurran added the status: rebase label Apr 15, 2026

latenighthackathon force-pushed the fix/reconfigure-guidance branch from 6aeb362 to 5046888 Compare April 16, 2026 04:32

prekshivyas self-assigned this Apr 16, 2026

prekshivyas approved these changes Apr 16, 2026

View reviewed changes

latenighthackathon force-pushed the fix/reconfigure-guidance branch from 5046888 to ea584ee Compare April 17, 2026 01:33

latenighthackathon mentioned this pull request Apr 18, 2026

docs: use CUDA base image for GPU smoke test in spark-install #1226

Merged

3 tasks

latenighthackathon force-pushed the fix/reconfigure-guidance branch from ea584ee to b3fcdcd Compare April 21, 2026 13:21

ericksoa merged commit 60c5cc7 into NVIDIA:main Apr 22, 2026
14 checks passed

oparoz mentioned this pull request Apr 22, 2026

Provide a path to edit openclaw.json after setup, or a simple way to fully specify it during onboarding #1699

Closed

2 tasks

cv added the v0.0.23 label Apr 22, 2026

latenighthackathon deleted the fix/reconfigure-guidance branch April 22, 2026 11:07

wscurran added area: cli Command line interface, flags, terminal UX, or output bug-fix PR fixes a bug or regression needs: rebase PR needs rebase or conflict resolution area: docs Documentation, examples, guides, or docs build and removed NemoClaw CLI labels Jun 3, 2026

wscurran removed the documentation label Jun 3, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(cli): add reconfiguration guidance to help and post-onboard output#1725

fix(cli): add reconfiguration guidance to help and post-onboard output#1725
ericksoa merged 1 commit into
NVIDIA:mainfrom
latenighthackathon:fix/reconfigure-guidance

latenighthackathon commented Apr 10, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Apr 10, 2026 •

edited

Loading

Reviews paused

Walkthrough

Changes

Estimated code review effort

Poem

❌ Failed checks (1 warning)

Uh oh!

coderabbitai Bot left a comment

Uh oh!

wscurran commented Apr 10, 2026

Uh oh!

wscurran commented Apr 15, 2026

Uh oh!

latenighthackathon commented Apr 16, 2026

Uh oh!

prekshivyas left a comment

Uh oh!

latenighthackathon commented Apr 17, 2026

Uh oh!

prekshivyas commented Apr 20, 2026

Uh oh!

latenighthackathon commented Apr 21, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

latenighthackathon commented Apr 10, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Problem

Changes

Test plan

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Apr 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviews paused

Walkthrough

Changes

Estimated code review effort

Poem

❌ Failed checks (1 warning)

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

wscurran commented Apr 10, 2026

Uh oh!

wscurran commented Apr 15, 2026

Uh oh!

latenighthackathon commented Apr 16, 2026

Uh oh!

prekshivyas left a comment

Choose a reason for hiding this comment

Uh oh!

latenighthackathon commented Apr 17, 2026

Uh oh!

prekshivyas commented Apr 20, 2026

Uh oh!

latenighthackathon commented Apr 21, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

latenighthackathon commented Apr 10, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Apr 10, 2026 •

edited

Loading