Repository navigation
docs(responses): register and document citation replay coverage (carry #7608) - #7865
Conversation
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: Ingwannu <186453546+Ingwannu@users.noreply.github.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 24 minutes. View limit details
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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
✅ Deterministic PR hygiene checks passed. |
|
Maintainer integration into |
Summary
Carries #7608 by @Ingwannu onto current
dev. The runtime repair forurl_citationtitles already landed in #7573, and its regressiontests/responses/responses-url-citation-title-repair.test.tsis ondev, but the owning structure doc and the explicit test registration were never merged.structure/transports/response-citations.md(outbound citation repair insrc/adapters/openai-responses/request-strips.ts, generated titles insrc/bridge/response-json.tsandsrc/bridge/sse.ts) and its manifest entry;structure/INDEX.mdis regenerated.structure/transports/responses.md(same line, no growth).scripts/test-layout/layout.jsonandtests/fixtures/test-layout-expected.jsonon existing lines.The original branch cannot receive maintainer pushes and conflicted with
devinstructure/manifest.json,structure/transports/responses.mdand the generated index; both sides were kept.Co-authored-by: Ingwannu 186453546+Ingwannu@users.noreply.github.com
Verification
Bun 1.4.4, Windows:
bun scripts/test.ts tests/responses/responses-url-citation-title-repair.test.ts tests/ci-workflows/file-size-ratchet.test.ts tests/ci-workflows/structure-ssot.test.ts tests/test-layout.test.ts tests/test-layout-tooling.test.ts: 98 pass / 0 fail.bun run structure:indexandbun run structure:check: pass.bun run test(documentation and registration only; left to PR CI).Checklist