Skip to content

fix: allow informational options before setup - #53

Open
xuu33030 wants to merge 1 commit into
humanmade:mainfrom
xuu33030:fix/issue-47-info-options
Open

xuu33030 wants to merge 1 commit into
humanmade:mainfrom
xuu33030:fix/issue-47-info-options

Conversation

@xuu33030

Copy link
Copy Markdown

Describe the PR

Fixes #47 by allowing --help, -h, --version, and -v to be handled before the configuration load and setup gate.

The CLI now normalizes the short informational aliases before passing them to yargs, while normal command execution continues to receive the configuration object. The entry point is also importable for focused regression tests without starting the CLI as a side effect.

Verification

  • npm test
  • npm run smoke in a clean clone with the patch applied
  • Manual clean-environment checks for --help, -h, --version, and -v: all exit 0 without configuration files or setup prompts
  • git diff --check

Notes

No documentation change is required; this restores standard informational CLI behavior.

For Altis Team Use

  • Acceptance criteria met
  • Automatic regression tests added
  • Code changes follow existing project patterns
  • Documentation updated (not required for this behavior)
  • Module release created/scheduled (maintainer decision)
  • Backport labels added (maintainer decision)

AI assistance was used during implementation; the change was locally reviewed and verified with the commands listed above.

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.

altis-cli --help / --version should work without configuration

1 participant