dcdpr / jp / 24518011221

67%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(cli, workspace): rename multi-target variants and scope help (#565)

Rename `ConversationTarget::Session` and `ConversationTarget::Pinned` to
`AllSession` and `AllPinned` to better reflect their semantics — these
targets expand to all conversations in a session or all pinned
conversations, distinguishing them from the single-target `Session` and
`LatestPinned` aliases.

The `ConversationIds` trait gains a `supports_session()` method
(mirroring the existing `is_multi()`), and `ConversationLoadRequest`
gains a `session` field. Both are threaded through `resolve_targets` and
the `TargetHelp` error variant so that help output is correctly tailored
to the command's capabilities.

The "Multi-Target Keywords" section in argument help text is now
conditional on the command supporting multi-target mode. Similarly, the
`TargetHelp` error carries `session` and `multi` flags so the displayed
help page matches what the command actually supports. Single-target
commands now also reject `+session` and `+pinned` at parse time with a
clear error message.

`conversation use` is updated to `PositionalIds<true, false>` so it
correctly advertises session-keyword support, and its output now
includes the conversation title when switching: "Switched active
conversation from X to Y: title". Pinned conversations in `conversation
ls` are no longer rendered bold, only blue.

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

Coverage Stats

37 of 75 new or added lines in 5 files covered. (49.33%)

4 existing lines in 1 file now uncovered.

22678 of 35330 relevant lines covered (64.19%)

183.97 hits per line