Skip to content

Batch ingest follow-ups from PR #38 review (transcript retention, probe transcript ring, no-op lastResult) #39

Description

@qiffang

Non-blocking follow-ups consolidated by architect-1 from the PR #38 tri-review (#runner:c710d0b8). Item 3 of the original list (stale stop record) was promoted into PR #38 itself and is done.

  1. Transcript retention/GC — <dataDir>/transcripts/ gains one file per completed batch (including fully adjudicated ones) with no retention policy; unbounded growth.
  2. Violation-path transcripts — splitByViolationPaths calls parseEventResults(batch, summary, ""), so unknowns produced on the violation path carry the transcript-unavailable marker even when a real transcript was persisted for the turn. Thread the actual path through.
  3. Probe-batch transcript ring — the stop record's transcript list saturates at 3 during the first streak; on a probe re-trip the record carries the previous streak's paths and excludes the newest probe transcript (the one the operator most needs). Use ring semantics (keep the most recent ≤3).
  4. Suspended auto-flush no-op overwrites last_flush — the guard no-op returns {stopped, -1} and, if routed through scheduleNext, refreshes lastResult/lastFlushAt each timer cycle while suspended, overwriting stopped_reason="unknown_streak" forensics (currently structurally unreachable; cosmetic). Either skip recording no-ops or use a dedicated reason.

🤖 Generated with Claude Code

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions