Skip to content

Persist inference errors on empty partials and honest exec turn counts - #552

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-6917-trace-instrumentation-gaps-inference-error-partials-carry-no
Aug 23, 2026
Merged

Persist inference errors on empty partials and honest exec turn counts#552
TheGreatAxios merged 1 commit into
mainfrom
cl-6917-trace-instrumentation-gaps-inference-error-partials-carry-no

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • partial.jsonl records for inference-error now include error (category, message, statusCode when present), including cycles that streamed no text.
  • Exec mid-run and terminal run.json snapshots read turnsUsed from runSink.getTurnCount() (and persist on each turn boundary) instead of the initial zero.
  • Docs note that latest is a symlink, not a session, so naive globs of the project sessions directory must skip it.

Verification

  • bun run typecheck passes
  • bun test src/session/stream-journal.test.ts src/session/run-sink.test.ts — 30 pass
  • Pre-commit typecheck and build pass

Fixes CL-6917

@linear-code

linear-code Bot commented Aug 23, 2026

Copy link
Copy Markdown
@TheGreatAxios
TheGreatAxios merged commit 31ea722 into main Aug 23, 2026
5 checks passed
TheGreatAxios added a commit that referenced this pull request Aug 27, 2026
…-gaps-inference-error-partials-carry-no

Persist inference errors on empty partials and honest exec turn counts
TheGreatAxios added a commit that referenced this pull request Aug 27, 2026
…-gaps-inference-error-partials-carry-no

Persist inference errors on empty partials and honest exec turn counts
@TheGreatAxios
TheGreatAxios deleted the cl-6917-trace-instrumentation-gaps-inference-error-partials-carry-no branch August 27, 2026 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant