Skip to content

fix(runtime-host): preserve transcript projection diagnostics - #5601

Open
jackeyfaker77 wants to merge 1 commit into
apache:mainfrom
jackeyfaker77:codex/5572-projection-diagnostics
Open

jackeyfaker77 wants to merge 1 commit into
apache:mainfrom
jackeyfaker77:codex/5572-projection-diagnostics

Conversation

@jackeyfaker77

Copy link
Copy Markdown
Contributor

Summary

Durable transcript projection failures currently discard the read model's hard diagnostics and throw only "projection is incomplete". Preserve them in RuntimeReadModelError and include session/invocation context, diagnostic codes, event IDs, and reasons in its message so boundedFailureDiagnostic can record the cause. Raw diagnostic detail stays out of the message; existing log redaction and truncation still apply. Soft diagnostics remain readable, and client error responses are unchanged.

This complements the page/bootstrap logging changes in #5573 and #5600. Test fixtures also close their database stores before cleanup so Windows SQLite locks do not mask assertion failures.

Refs #5572

Verification

  • New regressions for unsupported_event, incomplete_event, and tool_use_id_mismatch failed against the original reader; the soft-diagnostic case passed.
  • Rebuilt transcript reader and session continuity coordinator suites: 59/59 passed, including both paging directions, diagnostic preservation, redaction, raw-detail omission, and soft-diagnostic reads.
  • npm run build, npm run typecheck, npm run lint, and npm run format:check passed.
  • Desktop and UI knip checks passed.
  • Full-repository npm test was not run.

AI use

  • No generative tool made a substantive contribution
  • Generative tooling made a substantive contribution

Tool(s) and scope: Codex implemented the diagnostic preservation and regression tests, fixed test database cleanup, ran validation, and prepared this PR.

Checklist

  • Tests cover the change and fail without it
  • Lint, format, typecheck and the affected suites pass locally

Does this PR entail a change in behavior?

  • Yes - described under Summary above
  • No

@github-actions github-actions Bot added the effort/M Under 500 readable lines label Sep 22, 2026

This branch has not been deployed

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

Labels

effort/M Under 500 readable lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant