Repository navigation
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Deterministic PR hygiene checks passed. |
|
Triage note (2026-10-24, against Needs fixes before this can land: refresh onto dev containing #7607, preserve its size repair, obtain passing replacement-head CI, and complete draft readiness. No runtime reimplementation is needed. Still needed on dev: partial. Runtime repair already landed through #7573 / Exact-head CI at triage time: Please refresh onto current |
Carries #7608 onto current dev: adds the response-citations structure doc, links it from responses.md and registers the existing url-citation title-repair regression in the test layout. Co-authored-by: JUN <243035832+lidge-jun@users.noreply.github.com> Co-authored-by: Ingwannu <186453546+Ingwannu@users.noreply.github.com>
Summary
Register the existing URL-citation replay regression in both test inventories and document the outbound citation projection introduced by #7573. The decoded-checkpoint contract now distinguishes checkpoint framing from citation repair on OpenAI-operated outbound history. This changes documentation and test registration only.
The branch merges trusted
dev@6672cfb4a2b9a1dc72e26ed159c1d70368fe0ca7, preserving #7607's layout size repair and the subsequent test-layout shards. The original runtime fix is already on dev; no runtime reimplementation or file-size limit increase is included.Verification
Updated head:
5b3b87894cf6e2b0e7e70ae1f0b795f728edafec. Validation ran on the resulting merged tree with official Bun 1.4.4+38afee6c8, a frozen install with lifecycle scripts disabled, private temporary homes, and one CPU/RAM/process-limited workload at a time.bun test --isolate tests/responses/responses-url-citation-title-repair.test.ts tests/test-layout.test.ts tests/test-layout-tooling.test.ts tests/ci-workflows/file-size-ratchet.test.ts: 47 pass, 0 fail, 685 assertions. This includes the size gate that failed on the old head.bun scripts/structure-ssot.ts --fixandbun scripts/structure-ssot.ts: exit 0; generated index agrees with the manifest.bun scripts/privacy-scan.ts: exit 0 (ordinary repository privacy gate).git diff --check: exit 0.bun node_modules/typescript/bin/tsc --noEmit): incomplete; the enforced 300-second workload deadline killed it (about 1.2 GiB peak; no successful result). Limits were not raised. The Bun dependency wrapper's ignored postinstall was not used.No live provider request, client-file synchronization, service restart, security scanner, merge, release or deployment was performed. The main dirty checkout and protected runtime configuration were preserved. The push used
core.hooksPath=/dev/null; the explicit checks and incomplete typecheck are recorded above.Checklist
This PR remains Draft pending successful typecheck/current-head CI and review readiness. Previous-head CI results do not qualify the new head.