refactor(cli): add shared oclif command base

[cv]

Summary

Introduce a shared NemoClaw oclif command base so common parser conventions live in one place instead of every command class redefining them.

Stack Navigation

• Position: 33 of 60
• Previous PR: #2925 — refactor(cli): render public oclif help
• Next PR: #2927 — refactor(cli): use oclif flag relationships

Changes

• Added NemoClawCommand with common -h/--help base flag handling.
• Centralized JSON formatting used by oclif JSON-enabled commands.
• Migrated low-risk list, status, and maintenance command adapters to the shared base.

Type of Change

• Code change (feature, bug fix, or refactor)
• Code change with doc updates
• Doc only (prose changes, no code sample modifications)
• Doc only (includes code sample changes)

Verification

• npx prek run --all-files passes
• npm test passes
• Tests added or updated for new or changed behavior
• No secrets, API keys, or credentials committed
• Docs updated for user-facing behavior changes
• make docs builds without warnings (doc changes only)
• Doc pages follow the style guide (doc changes only)
• New doc pages include SPDX header and frontmatter (new pages only)

---

Signed-off-by: Carlos Villela cvillela@nvidia.com

Summary by CodeRabbit

• Refactor
  • Restructured CLI command architecture for backup, upgrade, garbage collection, list, and status operations. All commands continue to function with consistent help and JSON output support.

[copy-pr-bot]

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 85c9f684-e5fa-4b4d-92a1-9296610c505e

📥 Commits

Reviewing files that changed from the base of the PR and between cd56a9e and 4222730.

📒 Files selected for processing (4)

• src/lib/list-command.ts
• src/lib/maintenance-cli-commands.ts
• src/lib/nemoclaw-oclif-command.ts
• src/lib/status-command.ts

---

📝 Walkthrough

Walkthrough

A new shared base class NemoClawCommand is introduced, consolidating help flag definition and JSON logging behavior previously duplicated across CLI commands. Four command classes are refactored to extend this base class and remove redundant implementations.

Changes

CLI Command Consolidation

Layer / File(s)	Summary
Base Class Introduction src/lib/nemoclaw-oclif-command.ts	New NemoClawCommand extends @oclif/core's Command, defines baseFlags with -h/--help, and provides protected logJson(json: unknown) helper for formatted JSON output.
Command Migration src/lib/list-command.ts, src/lib/status-command.ts, src/lib/maintenance-cli-commands.ts	Four command classes (ListCommand, StatusCommand, BackupAllCommand, UpgradeSandboxesCommand, GarbageCollectImagesCommand) updated to extend NemoClawCommand instead of Command.
Flag & Method Cleanup src/lib/list-command.ts, src/lib/status-command.ts, src/lib/maintenance-cli-commands.ts	static flags changed from declaring help flag to empty objects; protected logJson() methods removed from individual commands (now provided by base class).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Commands once scattered, now they're neat,
A base class brings them all to meet.
Help and JSON, shared with care,
Less duplication in the air! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'refactor(cli): add shared oclif command base' accurately describes the primary change: introducing a shared NemoClawCommand base class for oclif commands to centralize common conventions.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/oclif-first-command-base

---

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

[cjagwani]

Approved as part of stack from #2886 (auto-approve: CI green, no scope/behavior flags).