Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

nemoclaw sandbox mcp usage omits the supported update subcommand

Closed
  • Is something wrong?
#12,349 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

@ericcurtin is already working on this.

Since Sep 28, 2026.

  • #12384 by @ericcurtin — open

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
45/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
typescript
Domain
cli

Research direction

Start with src/commands/sandbox/mcp.ts and compare its static usage strings with the dispatcher in src/lib/actions/sandbox/mcp-bridge.ts. Run src/commands/sandbox/mcp.test.ts and the CLI reproduction commands; done means both sandbox MCP help paths list update consistently and the focused tests pass.

Written by the indexing model from the issue text.

Description

Investigation Summary
  • src/commands/sandbox/mcp.ts lists <add|list|status|restart|remove|migrate> in both static usage (line 13) and the missing-sandbox error (line 32).
  • The dispatcher handles update (src/lib/actions/sandbox/mcp-bridge.ts, case "update"), and its own usage error and nemoclaw <name> mcp --help list <add|update|list|status|restart|remove|migrate>. The global help and the docs also describe mcp update.
  • No open issue or PR covers this.
Description

nemoclaw sandbox mcp and nemoclaw sandbox mcp --help omit the supported update subcommand from the MCP action list, so the two help entry points disagree.

Reproduction Steps
  1. Build the CLI on main (6721c27): npm run build:cli.
  2. Run node bin/nemoclaw.js sandbox mcp. It prints Usage: nemoclaw <sandbox> mcp <add|list|status|restart|remove|migrate> [args...] and exits 2.
  3. Run node bin/nemoclaw.js sandbox mcp --help. The USAGE line also omits update.
  4. Run node bin/nemoclaw.js alpha mcp --help. It prints <add|update|list|status|restart|remove|migrate>.
Environment
  • OS: macOS 26.6 (Apple M5, arm64)
  • Node.js: v24.18.0
  • NemoClaw: main at 6721c27
  • Docker: not required
Debug Output

Not applicable. The output comes from static usage strings, before any runtime or gateway call.

Proposed fix

A signed, tested fix is on sandeepstele:fix/mcp-usage-update (commit 2e1ea5f):

  • It adds update to both strings in src/commands/sandbox/mcp.ts.
  • It adds two tests to src/commands/sandbox/mcp.test.ts, for the usage metadata and the missing-sandbox error. Both fail without the fix.

A fork PR cannot pass the required openshell-sdk-package CI job, so I'm offering the branch for a maintainer to adopt.

Signed-off-by: Sandeep Satheesh [email protected]

Dominant language
TypeScript
Stars
22.7k
Forks
3.1k
Avg merge
1d 2h
Merged PRs (30d)
666

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from NVIDIA/NemoClaw

All issues in NVIDIA/NemoClaw

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.