fix(cli): align hook dry-run with installed command
Your AI's guard dog to stop it from going rogue.
Brought to you by:
doberman
Originally created by: slegarraga
Closes [#429]
DASHBOARD_COMMAND in both install and uninstall dry-run previews so the displayed SessionStart command is the same one the installer writes.doberman session-summary and no longer mentions doberman dashboard..venv/bin/pytest tests/unit/test_install_hooks.py — 44 passed..venv/bin/pytest tests/unit -k 'install_hooks or cli_help or cli_encode_safe' — 117 passed, 1 skipped..venv/bin/ruff check src tests — passed..venv/bin/ruff format --check src/doberman/cli/main.py tests/unit/test_install_hooks.py — passed.
Tickets: #429
Tickets: #471
Tickets: #493
Tickets: #495
Tickets: #497
Tickets: #509
Ticket changed by: fu351
Ticket changed by: fu351
Ticket changed by: fu351
Originally posted by: fu351
Merged, thanks @slegarraga! Importing
DASHBOARD_COMMANDinto the preview means the dry run can never drift from what the installer writes again, which was the whole bug. One thing on the landing. Your branch and [#472] both added an import at the same line in cli/main.py, so I merged main into a copy of your branch in our repo (land/pr463, [#493]) and resolved that one line. Your commits and authorship are untouched.Related
Tickets:
#472Tickets:
#493Originally posted by: slegarraga
Thank you for handling the landing conflict so carefully and for preserving the commits and authorship. The landing branch was a thoughtful way to resolve the shared import anchor without disturbing either change. I really appreciate the detailed review — glad the shared constant now keeps preview and installation behavior aligned by construction!