dcdpr / jp / 28378805656

69%

Build Type

push

github

Committed by web-flow

Commit Message

fix(cli, conversation): Show stable raw turn numbers in print headers (#810)

Turn headers shown by `jp conversation print` now display the
pre-projection (raw) turn number rather than the position in the
projected iterator. This means the number in the header matches the
argument `jp conversation compact --from/--to` accepts, so users can
reference a printed turn directly in a compact command without having to
account for how many earlier turns were collapsed.

In the compacted view, a summary turn that stands in for several raw
turns shows the full range it replaced, e.g. `turns 2–5, 12 minutes
ago`, while a single-turn summary continues to show `turn N`. The
`TurnSelection::Index` path also resolves by raw turn number now, so `jp
conversation print --turn N` is consistent with the header.

A new `TurnOrigin` enum is returned by `apply_projection()` and threaded
through to `TurnRenderer::render_turn`, carrying the mapping from each
projected turn back to its raw source turn(s). When no compaction is
present every turn maps to `TurnOrigin::Kept` of its own index, so
behaviour is unchanged for uncompacted views.

Signed-off-by: Jean Mertz <git@jeanmertz.com>

Coverage Stats

73 of 77 new or added lines in 4 files covered. (94.81%)

2 existing lines in 1 file now uncovered.

36956 of 54936 relevant lines covered (67.27%)

388.95 hits per line