Skip to content

Fix CLI help safety and test config discovery#88

Merged
PennyroyalTea merged 1 commit into
mainfrom
bs-help-no-execute-tests-discovery
May 14, 2026
Merged

Fix CLI help safety and test config discovery#88
PennyroyalTea merged 1 commit into
mainfrom
bs-help-no-execute-tests-discovery

Conversation

@PennyroyalTea
Copy link
Copy Markdown
Collaborator

Summary

  • restore Commander help handling so --help exits before command actions run
  • add recursive test config discovery for tests push via --config-dir
  • add e2e coverage for help safety and folder-discovered test configs

Verification

  • npm run build
  • npm run lint (passes with existing warnings in unrelated test files)
  • npm run test:e2e -- --runTestsByPath src/tests/cli.e2e.test.ts --testNamePattern "help|tests push command|push-tools command"
  • npm test -- --runTestsByPath src/tests/tests/test-commands.integration.test.ts

@PennyroyalTea PennyroyalTea force-pushed the bs-help-no-execute-tests-discovery branch from 6e59bf2 to a0e6dea Compare May 14, 2026 20:40
@PennyroyalTea PennyroyalTea merged commit 18e3c39 into main May 14, 2026
4 checks passed
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