Skip to content

fix(cli): correct help tool names for bash output/kill#120

Merged
frostming merged 3 commits into
bubbuild:mainfrom
Andy963:fix/help-tool-names
Mar 17, 2026
Merged

fix(cli): correct help tool names for bash output/kill#120
frostming merged 3 commits into
bubbuild:mainfrom
Andy963:fix/help-tool-names

Conversation

@Andy963
Copy link
Copy Markdown
Contributor

@Andy963 Andy963 commented Mar 17, 2026

What / Why

  • ,help currently suggests ,bash_output and ,kill_bash, but the actual internal commands are ,bash.output and ,bash.kill.
  • This mismatch causes users to copy/paste from help and immediately hit “unknown command” behavior.

Changes

  • Update src/bub/builtin/tools.py help text to list ,bash.output and ,bash.kill.
  • Update bash tool docstring to reference bash.output (keeps docs consistent).
  • Add regression test tests/test_cli_help.py to ensure help text stays in sync.

Verification

  • uv run ruff check .
  • uv run mypy src
  • uv run pytest -q

Comment thread src/bub/builtin/tools.py Outdated
@frostming frostming merged commit c7ffdb1 into bubbuild:main Mar 17, 2026
5 checks passed
@Andy963 Andy963 deleted the fix/help-tool-names branch March 20, 2026 01:56
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.

2 participants