Skip to content

feat(help): add browser structured help#1404

Merged
jackwener merged 1 commit intomainfrom
feat/help-optionspec-browser
May 7, 2026
Merged

feat(help): add browser structured help#1404
jackwener merged 1 commit intomainfrom
feat/help-optionspec-browser

Conversation

@jackwener
Copy link
Copy Markdown
Owner

Summary

  • add shared Commander argument/option spec extraction for structured help
  • wire opencli browser --help -f yaml|json to emit all browser leaf commands, including nested tab, get, and dialog commands
  • wire individual browser leaf commands, e.g. opencli browser click --help -f yaml, to structured help with command options, namespace options, and root global options

Verification

  • npm test -- --run src/cli.test.ts src/help.test.ts
  • npm run typecheck
  • npm run build
  • git diff --check
  • empty-HOME smoke: node dist/src/main.js browser --help -f yaml
  • empty-HOME smoke: node dist/src/main.js browser click --help -f yaml

@jackwener jackwener force-pushed the feat/help-optionspec-browser branch from f65f444 to 9d2113b Compare May 7, 2026 18:12
@jackwener jackwener merged commit 34f793f into main May 7, 2026
11 checks passed
@jackwener jackwener deleted the feat/help-optionspec-browser branch May 7, 2026 18:16
Copy link
Copy Markdown
Owner Author

@jackwener jackwener left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (cannot --approve since shared GH identity). Verified commit 9d2113b in thread #OpenCLI:968cb389 — all 3 intermediate-parent subtrees (browser tab/get/dialog) emit structured YAML with correct group/leaves/options layering; namespace root description preserved via originalBrowserDescription snapshot (applyRootSubcommandSummaries would otherwise overwrite it); 134/134 unit tests + 11/11 CI green; spec abstraction ready for A1 (daemon/plugin/adapter).

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.

1 participant