docs(providers): describe OpenCode Go session affinity and the Pi compat flag - #3888
Conversation
…treaming filter After #3868 the streaming filter keeps a malformed START verbatim when a later START opens a real span, but stripCitationMarkers still paired the first START with that later span's END and deleted everything between. bridge.ts re-strips the accumulated text for output_text.done and output_item.done, so the terminal text disagreed with the concatenated deltas. Walk START-delimited segments in the whole-string path too, and share the 4096 span bound with the whole-string path (an over-bound span that is terminated late is malformed text in both), and assert delta-vs-whole equality across several chunkings. Found by the lane A fresh-base composition audit on dev d00615d. Co-authored-by: luvs01 <27862058+luvs01@users.noreply.github.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. |
리뷰 · 우선순위 38 / 80이 PR은 코드가 아니라 문서만 고칩니다. 이미 베이스가 추가된 문단 내용은 HEAD 코드와 대체로 일치합니다. Go는 안정적인 세션 식별자가 필요하고, Codex 스레드/세션 헤더가 없으면 클라이언트의 라인 단위로 보면 문장 자체에 코드와 어긋나는 거짓말은 거의 없습니다. 다만 영어 메인테이너의 판단이 필요한 지점
너의 추천 이 댓글은 grok-bot이 작성했습니다 |
Summary
Docs-only follow-up to #3880 (carry of #3858 by @makesomethingshit): the OpenCode Go session-affinity paragraph and the Pi
compat.sendSessionAffinityHeadersnote fordocs-site/src/content/docs/guides/providers.md. Lane A handed this hunk off because the file is owned by the main lane in the release train; the patch applied cleanly on the current head.Verification
git apply --checkclean onorigin/dev@76436a3ee+ fix(responses): make whole-string citation stripping agree with the streaming filter #3882.Checklist
Depends on #3882 (base
codex/rt-m7-citation-terminal). Review this PR's diff only.Maintainer integration decision (MAINTAINERS.md, dev-only admin integration): @lidge-jun integrates #3882 (+ docs follow-up #3888) into
dev. Exact-head evidence for #3882 ate96b5c52e: Cross-platform CI run 34114667385 — Linux test 1/4–4/4, macOS 1/2, 2/2, gates, storage policy, api usage, keyring ×3, npm-global ×3, docker smoke = success; Windows shards and macos control deferred to the final release-train head by maintainer policy. #3888 is docs-only on top of #3882. Independent review PASS at the amended head (see body). Local suites NOT RUN. Maintainer integration, not self-approval.