boinger / confvis / 24609176026

88%

Build Type

push

github

Committed by boinger

Commit Message

feat(history): cap history growth via Prune/PruneRef + --history-max-entries

AppendToFile and AppendToRef had no retention: history grew forever, and
Last(n) had to load every entry into memory before returning the last N.
For long-lived CI dogfood pipelines this is slow over time and wasteful
at read time.

Add history.Prune(path, maxEntries) and history.PruneGitRef(ref,
maxEntries) — symmetric helpers for the two storage backends. File
backend uses write-to-sibling-tempfile + atomic rename so readers never
see a partial file. Ref backend reuses the existing blob
read-modify-write flow. Both are no-ops when the input has <=maxEntries
or when maxEntries <=0.

Expose via a new --history-max-entries flag on `confvis gauge`,
default 5000 (0 or negative disables pruning). CLI wires through
GaugeDeps as HistoryPruner / GitRefPruner so tests can inject mocks.
appendToHistory calls the pruner after a successful append; prune
failures surface as errors, preserving the pre-prune append so the
next run retries.

10 new tests (5 file-backend, 5 git-ref-backend) cover trims-to-last-N,
no-op-under-cap, disabled-on-zero-or-negative, missing-input-no-op,
and order preservation.

Coverage Stats

50 of 82 new or added lines in 5 files covered. (60.98%)

25 existing lines in 1 file now uncovered.

4453 of 5140 relevant lines covered (86.63%)

15.02 hits per line