Skip to content

docs(quickstart): explain curl|bash TTY acceptance and flag form (closes #3275)#4167

Closed
latenighthackathon wants to merge 2 commits into
NVIDIA:mainfrom
latenighthackathon:docs/3351-fresh-yes-curl-bash-v2
Closed

docs(quickstart): explain curl|bash TTY acceptance and flag form (closes #3275)#4167
latenighthackathon wants to merge 2 commits into
NVIDIA:mainfrom
latenighthackathon:docs/3351-fresh-yes-curl-bash-v2

Conversation

@latenighthackathon

@latenighthackathon latenighthackathon commented May 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

Piping the installer through bash strips the TTY, so the third-party software notice cannot prompt and the install exits before doing anything. The quickstart only showed the env-var acceptance form, with no explanation of why it is required under a pipe. This documents both non-interactive acceptance forms and the reason behind them.

Problem

curl -fsSL https://www.nvidia.com/nemoclaw.sh | bash in a headless shell or CI exits before installing because the notice has no TTY to prompt on, and the quickstart did not surface the workaround (#3275).

What changed

docs/get-started/quickstart.mdx now explains the TTY-stripping behavior and shows two non-interactive acceptance forms: the existing env-var form on the bash side of the pipe, and the flag form via bash -s -- --yes-i-accept-third-party-software. The nemoclaw-user-get-started skill mirror is regenerated to match.

Scope note

This PR previously also carried --fresh/--yes/--probe-only reference edits to close #3233. Those flags are now documented in docs/reference/commands.mdx upstream and #3233 has been closed as completed, so that portion is dropped to avoid duplicating the reference. This PR is now scoped to the #3275 quickstart gap only.

Test plan

  • Docs render check on docs/get-started/quickstart.mdx
  • bash -s -- --yes-i-accept-third-party-software flag form verified against scripts/install.sh argument handling

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

@latenighthackathon latenighthackathon mentioned this pull request May 25, 2026
Closed
3 tasks
@copy-pr-bot

copy-pr-bot Bot commented May 25, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@coderabbitai

coderabbitai Bot commented May 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Walkthrough

This PR updates documentation for NemoClaw's curl-piped installer across two locations, clarifying that piping strips the TTY in headless/CI contexts and providing two explicit methods to accept the third-party software notice: environment variables or command-line flags.

Changes

NemoClaw Quickstart Installation Documentation

Layer / File(s) Summary
Quickstart curl|bash third-party acceptance patterns
docs/get-started/quickstart.mdx, .agents/skills/nemoclaw-user-get-started/SKILL.md		 Expands installation instructions with TTY-stripping clarification and two piped-command variants to deliver third-party acceptance: setting NEMOCLAW_NON_INTERACTIVE and NEMOCLAW_ACCEPT_THIRD_PARTY_SOFTWARE environment variables on the bash side, or passing --yes-i-accept-third-party-software via bash -s --.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

  • ericksoa
  • jyaunches

Poem

🐰 A curl and a bash, through the pipeline it flows,
With flags and with vars, third-party accepts,
No TTY? No problem! The docs now all know,
Headless installs smooth, with explicit checkpoints.
Piped and polished, the quickstart's complete! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The PR addresses the primary objectives from both linked issues: documenting --fresh/--yes/--probe-only flags and updating curl|bash quickstart guidance for non-interactive installs.
Out of Scope Changes check ✅ Passed All changes are within scope of the linked issues. The PR updates documentation and the generated skill mirror as required, with no unrelated code alterations.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main documentation update to the quickstart guide regarding curl|bash TTY behavior and acceptance flags.

✏️ 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.

@wscurran

wscurran commented May 26, 2026

Copy link
Copy Markdown
Contributor

✨ Thanks for submitting this detailed PR about documenting the --fresh, --yes, and piped installer flags for the NemoClaw CLI. This proposes a way to improve the user-facing reference for the onboard flow, addressing gaps that have caused issues for QA and bug reporters.

Related open issues:

1 similar comment
@wscurran

wscurran commented May 26, 2026

Copy link
Copy Markdown
Contributor

✨ Thanks for submitting this detailed PR about documenting the --fresh, --yes, and piped installer flags for the NemoClaw CLI. This proposes a way to improve the user-facing reference for the onboard flow, addressing gaps that have caused issues for QA and bug reporters.

Related open issues:

@latenighthackathon latenighthackathon force-pushed the docs/3351-fresh-yes-curl-bash-v2 branch from bb56c3c to 3388204 Compare May 27, 2026 03:07
@latenighthackathon
docs(quickstart): explain curl|bash TTY acceptance and flag form (NVI…
f7e67c4 
…DIA#3275)

Piping the installer through bash strips the TTY, so the third-party
software notice cannot prompt and the install exits before doing
anything. Document both non-interactive acceptance forms: the env-var
form on the bash side of the pipe, and the flag form via
bash -s -- --yes-i-accept-third-party-software.

Scoped to NVIDIA#3275. The --fresh/--yes/--probe-only documentation that the
earlier revision of this branch also carried is now present in
docs/reference/commands.mdx upstream (NVIDIA#3233 closed as completed), so
those edits are dropped to avoid duplicating the reference.

Signed-off-by: latenighthackathon <latenighthackathon@users.noreply.github.com>
@latenighthackathon latenighthackathon force-pushed the docs/3351-fresh-yes-curl-bash-v2 branch from 3388204 to f7e67c4 Compare May 28, 2026 06:05
@latenighthackathon latenighthackathon changed the title docs(reference,quickstart): explain --fresh/--yes and the curl|bash flag form (closes #3233, #3275) docs(quickstart): explain curl|bash TTY acceptance and flag form (closes #3275) May 28, 2026
@latenighthackathon

latenighthackathon commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #4451, which documents the non-TTY install acceptance in the quickstart (including the bash -s -- --yes-i-accept-third-party-software flag form). Closing in favor of that. Cheers!

@latenighthackathon latenighthackathon deleted the docs/3351-fresh-yes-curl-bash-v2 branch May 29, 2026 14:22
@wscurran wscurran added area: cli Command line interface, flags, terminal UX, or output bug-fix PR fixes a bug or regression area: docs Documentation, examples, guides, or docs build and removed NemoClaw CLI labels Jun 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

area: cli Command line interface, flags, terminal UX, or output area: docs Documentation, examples, guides, or docs build bug-fix PR fixes a bug or regression

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[All Platforms][Docs] commands.md missing documentation for nemoclaw onboard --fresh, --yes, and <name> connect --probe-only

2 participants

@latenighthackathon @wscurran