Skip to content

feat: output dashboard token on onboarding completion#88

Closed
WuKongAI-CMU wants to merge 1 commit into
NVIDIA:mainfrom
WuKongAI-CMU:feat/show-dashboard-token
Closed

feat: output dashboard token on onboarding completion#88
WuKongAI-CMU wants to merge 1 commit into
NVIDIA:mainfrom
WuKongAI-CMU:feat/show-dashboard-token

Conversation

@WuKongAI-CMU
Copy link
Copy Markdown
Contributor

Summary

Fixes #53 — onboarding now outputs the dashboard auth token so users can access the dashboard immediately.

  • Reads the token from openshell gateway info --json or falls back to the OpenShell config file
  • Displays a ready-to-click URL with the token appended as a fragment (#token=...)
  • Shows the raw token value for users who need to copy it separately
  • Gracefully omits the token line when no token is configured

Test plan

  • Existing tests pass (19/19)
  • Manual: run nemoclaw onboard → summary should show dashboard URL with token
  • Manual: open the displayed URL in browser → dashboard should authenticate automatically

🤖 Generated with Claude Code

After onboarding, the summary now includes the dashboard auth token
and a ready-to-click URL with the token appended as a fragment.
The token is read from the OpenShell gateway info or config file.

Before:
  Dashboard    http://localhost:18789/

After:
  Dashboard    http://localhost:18789/#token=abc123...
  Token        abc123...

Closes NVIDIA#53

Signed-off-by: peteryuqin <peter.yuqin@gmail.com>
@WuKongAI-CMU
Copy link
Copy Markdown
Contributor Author

Closing in favor of #87 which addresses the same issue.

cv added a commit that referenced this pull request May 1, 2026
Add a source-shape detector and budget gate, then replace production-source-text assertions across the CLI/plugin test suite with behavior coverage. Behavior replacements use subprocess fakes, temp fixtures, shell snippet execution, emitted files, stdout/stderr/exit-code assertions, and direct helper behavior instead of asserting on implementation strings.

Experiments: kept runs #1-#88
Metric: source_shape_cases 317 → 0 (-100%); source_shape_assertions 819 → 0; source_shape_files 23 → 0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Installer should output dashboard token on completion

1 participant