fix: make /history numbering match visible messages

[stablegenius49]

Summary

• number only visible user/assistant entries in /history
• collapse skipped tool responses into a small summary instead of silently creating numbering gaps
• show a clearer assistant placeholder for tool-call-only turns and allow longer previews

Testing

• pytest -q tests/test_cli_init.py
• python3 -m py_compile cli.py tests/test_cli_init.py

Closes #633

[teknium1]

Merged (77f4776) and cleaned up dead module stubs in 4a09ae2 — the 200 lines of prompt_toolkit/rich/fire stubs were guarded and never activated since those deps are always installed. The /history fix itself is great — inline tool summaries, sequential numbering, better assistant display. Thanks @stablegenius49!

[teknium1]

Thanks for the fix @stablegenius49! The show_history() improvement is great — consistent numbering and tool message collapsing is exactly what was needed.

However, this PR's branch is based on a much older version of main, so the diff includes unintended reversions of many features added since (skin engine, worktree isolation, checkpoints, auxiliary model config, etc.).

The good news is the show_history fix was already applied to main in commit 77f4776. So the useful part of this PR is live — closing as superseded. Thanks again!

[max-pristan]

Plan posted as planned, executing now.

[max-pristan]

Resolution posted above. Fix is complete in PR #656.

[max-pristan]

Resolved. PR #650 (fix/650-policy-code-body-size-limit) implements the fix:

• Added _read_body() helper (router.py:46-53) that reads raw body and raises HTTP 413 if it exceeds 10 MB
• All 6 endpoints now use _read_body(): /plan, /lint, /drift, /rollback, /impact, /sync-configs/{id}/resolve
• Regression tests in tests/api/v1/policy_code/test_body_size_limit.py: 6/6 pass

Closing.