Menu

#471 fix(cli): preview the real session-summary hook

closed
nobody
duplicate (3)
2026-08-27
2026-08-26
Anonymous
No

Originally created by: slegarraga

Closes [#429]

What

  • Imports DASHBOARD_COMMAND from the installer so install-hooks and uninstall-hooks dry runs display the same command the merge logic writes.
  • Replaces the hardcoded stale preview (doberman dashboard) with doberman session-summary.

Tests

  • Extends tests/unit/test_install_hooks.py to assert the install dry run contains the real command and no longer contains the hidden alias.

Verification

  • .venv/bin/pytest tests/unit/test_install_hooks.py — 43 passed.
  • .venv/bin/ruff check src/doberman/cli/main.py tests/unit/test_install_hooks.py
  • .venv/bin/ruff format --check src/doberman/cli/main.py tests/unit/test_install_hooks.py

Related

Tickets: #429

Discussion

  • Anonymous

    Anonymous - 2026-08-27

    Originally posted by: fu351

    Closing this as a duplicate of [#463], @slegarraga, the main.py change is byte for byte the same. I'm keeping [#463] because its test also covers uninstall-hooks --dry-run, so it goes red if either preview drifts from the installed command again. Review coming on [#463].

     

    Related

    Tickets: #463

  • Anonymous

    Anonymous - 2026-08-27

    Ticket changed by: fu351

    • status: open --> closed
     

Log in to post a comment.