mirror of
https://github.com/NVIDIA/NemoClaw.git
synced 2026-07-03 03:37:16 +00:00
49f3163ed2
<!-- markdownlint-disable MD041 --> ## Summary Adds a `nemo-deepagents` host CLI alias for the LangChain Deep Agents Code runtime and teaches onboarding to accept common user-facing agent aliases. This lets users run `nemo-deepagents onboard` or shorter forms such as `nemoclaw onboard --agent dcode`, `--agent deepagents`, and `--agent langchain` instead of remembering `langchain-deepagents-code`. ## Changes - Registered `nemo-deepagents` as an npm bin that reuses the existing `bin/nemoclaw.js` launcher and selects Deep Agents Code when invoked through that symlink. - Added canonical agent alias resolution for `--agent` and `NEMOCLAW_AGENT`, including `nemohermes`, `dcode`, `deepagents`, `deepagents-code`, `langchain`, and related variants. - Updated CLI branding, installer/update branding, onboard `--agent` help, and unknown-agent diagnostics for the alias set. - Updated Deep Agents quickstart, command reference, and platform support docs; adjusted platform-doc validation to allow documented aliases. - Added/updated tests for CLI alias behavior, onboarding arg parsing/help, agent definition alias resolution, update copy, oclif help, and platform-doc validation. ## Type of Change - [ ] Code change (feature, bug fix, or refactor) - [x] Code change with doc updates - [ ] Doc only (prose changes, no code sample modifications) - [ ] Doc only (includes code sample changes) ## Quality Gates <!-- Check all that apply. For any "covered by existing tests", "not applicable", or waiver entry, add a brief justification on the same line or in the Changes section. --> - [x] Tests added or updated for changed behavior - [ ] Existing tests cover changed behavior — justification: - [ ] Tests not applicable — justification: - [x] Docs updated for user-facing behavior changes - [ ] Docs not applicable — justification: - [x] Sensitive paths changed (security, policy, credentials, preflight, onboarding, inference, runner, sandbox, or messaging) - [x] Sensitive-path review completed or maintainer-approved waiver recorded — reviewer/approval link/justification: self-review; changes are limited to agent selector canonicalization, launcher branding, installer alias selection, docs, and tests, with no credential handling or network-policy enforcement changes. - [ ] Non-success, skipped, or missing CI check accepted by maintainer — check name, approval link, and follow-up issue: ## Verification <!-- Check each item you ran and confirmed. Leave unchecked items you skipped. Doc-only changes do not require npm test unless you ran it. --> - [x] PR description includes the DCO sign-off declaration and every commit appears as `Verified` in GitHub - [x] Git hooks passed during commit and push, or `npx prek run --from-ref main --to-ref HEAD` passes - [x] Targeted tests pass for changed behavior - [ ] Full `npm test` passes (broad runtime changes only) - [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) - [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) Verification commands run: - `npm run build:cli` - `npx vitest run test/nemohermes-alias.test.ts test/nemo-deepagents-alias.test.ts test/update.test.ts test/cli-oclif-compatibility.test.ts src/lib/cli/branding.test.ts src/lib/actions/update.test.ts src/lib/agent/defs.test.ts src/lib/onboard/legacy-command.test.ts src/lib/onboard/agent-flag-help.test.ts src/lib/onboard/command-support.test.ts test/cli/dispatch-basics.test.ts test/generate-platform-docs.test.ts --reporter=dot` - `NEMOCLAW_RUN_INSTALLER_TESTS=1 npx vitest run --project installer-integration test/install-preflight.test.ts --reporter=dot` - `npm run typecheck:cli` - `npm run validate:configs` - `python3 scripts/generate-platform-docs.py --check` - `npm run docs` (passed; Fern reported two hidden warnings) - Commit hooks passed; latest push hooks passed. --- <!-- DCO sign-off is required in this PR description, and every commit must appear as Verified in GitHub. Run: git config user.name && git config user.email --> Signed-off-by: Carlos Villela <cvillela@nvidia.com> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added a `nemo-deepagents` CLI alias for Deep Agents with matching branding, help, onboarding, uninstall, and update flows. * **Documentation** * Updated quickstart and command/reference docs to use the Deep Agents CLI wrapper and clarified `--agent` alias vs canonical name behavior. * Refreshed platform support/onboarding matrix entries for the new alias. * **Bug Fixes** * Improved agent selector alias resolution and enhanced “unknown agent” guidance with supported alias mappings. * Sanitized Deep Agents update environment variables during installs. * **Tests** * Added/expanded coverage for Deep Agents alias behavior and alias canonicalization. <!-- end of auto-generated comment: release notes by coderabbit.ai --> |
||
|---|---|---|
| .. | ||
| lib | ||
| nemoclaw.js | ||
| nemohermes.js | ||