umputun / agterm / 33439642769

97%

Build Type

push

github

Committed by umputun

Commit Message

fix(session): narrow the context validator and correct stale title-bar prose

validateContext, contextByteLimit and SessionContextValidation were public with
no caller outside agtermCore - only the module itself and its @testable tests.
agtermCore ships as a .library to the agterm-linux fork, so a public symbol is
one a fork can start depending on and narrowing later breaks that build. the two
closest analogues, String.trimmedOrNil and CommandRestore.hasControlCharacter,
are both internal.

two comments still described line two of the title bar as the cwd, which stopped
being unconditionally true when the context took that line: titleLabel's own doc
and ToolbarMode's. both now point at TitlebarComposition, which owns the rule.

also drops the "where the sidebar row has no room" justification from the CLI
help and the four doc surfaces. the sidebar row is narrow, but that was never
the argument - it carries the session's identity, and room is beside the point.
the docs now say what the command does without arguing a case.

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

27 existing lines in 5 files now uncovered.

9201 of 9427 relevant lines covered (97.6%)

4328558.21 hits per line