feat(reasoning): consolidate replay, opt-in tag parsing, and summary policy

[luvs01]

Summary

Consolidate the reasoning roundtrip changes from #5449, #5205 and #5491 into one attributed draft, preserving their distinct behavior changes:

• Retain the existing empty-reasoning placeholder for signed thinking-only assistant turns even when there is no tool call. Explicit model opt-outs and serving-identity replay boundaries remain intact.
• Add inlineThinkTagModels as an explicit model opt-in for Chat output that encodes reasoning in inline tags. Operator lists, including [], override registry defaults for this new field. Unlisted models pass through unchanged.
• Preserve answer whitespace and parse repeated blocks iteratively. Once an initial reasoning block activates the format, later tags remain delimiters even inside code fences; prose or a code fence before initial activation leaves the response unchanged. This is a declared opt-in format, not a Markdown intent detector. Kiro retains its existing single-block behavior.
• Change the omitted-summary display default only for validated reasoning effort. Explicit summary: "none" stays hidden; invalid or absent effort does not accidentally opt in. Combo-injected defaults add summary auto only when the caller did not specify a summary.

All three other-author originals remain open and unchanged. Current dev's extracted model-field classifier is preserved, and the source's size-cap increase is excluded. No provider preset is newly enabled.

Verification

• Head 8028c913ab00ed5b402cb0ab9efee65fbe9e7abb, based on dev a6b54297b03fc218105273887ce147c901848d89. Four source commits preserve author and source SHA. Three carries have equal stable patch IDs; the initial inline-parser carry adapts the current classifier and excludes the cap change. Final integration commit retains all contributor trailers.
• First 13-file run: 389 pass / 7 fail. A subsequent affected five-file run was 202 pass / 3 fail and exposed the actual explicit-empty-list opt-out defect. The corrected final four-file scope passed 110 / 110. Earlier passing parser/Kiro/replay/bridge cases were unchanged; overlapping counts are not added or described as a final complete-suite pass.
• Regressions cover valid/invalid effort, explicit summary none/auto, empty-list override, transport mismatch, signed empty replay, split tags, whitespace, Unicode, EOF/disposal and a 12,000-block single-chunk versus split-chunk comparison. Hidden/visible bridge output is checked through continuation serialization.
• Final typecheck, structure, privacy, unchanged-cap ratchet and whitespace checks passed. Documentation built successfully: 497 pages. Independent static review found no new defect in the inspected boundaries; it did not execute tests or live provider calls.
• Full prepush, changed-import-graph coverage, hosted CI, live-provider acceptance and the required independent security review of the provider-editor policy change remain outstanding. This PR stays draft and does not attest to merge or release readiness.

Checklist

• Scope stays focused and avoids unrelated cleanup.
• Docs or release notes were updated when needed.
• Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Co-authored-by: Joonsuh Park trckstr4422@gmail.com
Co-authored-by: Daniel Sjöstrand 16033062+Danielsjostrand1979@users.noreply.github.com
Co-authored-by: alexph-dev alexph-dev@users.noreply.github.com
Co-authored-by: Yum-wu 1172989563@qq.com

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

✅ Deterministic PR hygiene checks passed.

[lidge-jun]

리뷰 · 우선순위 61 / 80

이 PR은 생각(reasoning)이 한 바퀴 돌 때 깨지던 세 가지 고침을 한 draft로 모은 작업입니다. 첫째, 서명한 thinking-only 어시스턴트 턴에 도구 호출이 없어도 빈 reasoning_content 자리표시를 남겨, DeepSeek처럼 재전송을 거부하는 업스트림이 다음 요청을 400으로 끊지 않게 합니다. 둘째, 게이트웨이가 생각을 <think> 같은 태그로 답변 본문에 섞어 보낼 때, 운영자가 inlineThinkTagModels에 모델만 넣으면 openai-chat이 그 블록을 다시 reasoning으로 가르게 합니다. 기본은 꺼져 있고, []로 레지스트리 기본값을 막을 수 있으며, 목록에 없는 모델은 글자 하나 안 바꿉니다. 파서는 Kiro용 코드를 inline-think-tags.ts로 빼 공유하고, Kiro는 예전처럼 첫 블록만, Chat opt-in은 여러 블록을 이어서 처리합니다. 셋째, Responses에서 summary를 안 적었을 때, 검증된 effort가 있을 때만 원시 생각을 보이게 바꾸고, combo가 넣는 기본값도 호출자가 summary를 안 고른 경우에만 auto를 붙입니다. base는 dev이고, 원작자 커밋을 cherry-pick으로 유지했으며, 프리셋을 새로 켠 제공자는 없습니다. 관련 테스트·구조 문서·설정 문서도 같이 들어 있습니다.

라인 - src/server/auth-cors.ts의 inlineThinkTagModels: "editor": 제공자 편집기 정책 표면이 늘어났는데, PR 체크리스트와 본문이 독립 보안 리뷰를 아직 미완료로 적어 두었습니다. draft로 두는 선택은 맞지만, ready 전에 이 한 줄 정책 변경을 반드시 별도 확인해야 합니다.
라인 - src/combos/request.ts의 summary: "auto" 주입: 예전에는 effort만 넣던 combo 자식 요청이 이제 summary까지 갖게 됩니다. 의도된 표시 기본값 정렬이지만, 생략(=숨김)에 의존하던 클라이언트·브리지 경로가 있으면 체감이 바뀝니다.
라인 - InlineThinkTagParser 활성화 후 코드 펜스 안 태그: 첫 <think>로 포맷이 켜지면 이후 태그는 마크다운을 보지 않고 구분자로만 씁니다. 문서·테스트에 명시된 opt-in 계약이지만, 게이트웨이가 예제 XML을 답에 섞으면 답이 잘릴 수 있으니 운영자 안내가 필요합니다.
라인 - 원본 #5449, #5205, #5491이 아직 open: 이 draft가 정본으로 남을 계획이면, 병합(또는 대체 확정) 때 원본을 close해 중복·무효 PR이 남지 않게 정리하는 편이 낫습니다.

메인테이너의 판단이 필요한 지점

이 한 PR에 replay placeholder, inline 태그 복구, summary/effort 표시 정책을 묶을지, 아니면 합친 검증만 유지하고 원본 세 갈래를 따로 머지할지. 또 combo의 summary: "auto" 기본 주입을 “표시를 살리는 의도”로 받아들일지, 아니면 생략을 계속 숨김으로 둘지. inlineThinkTagModels를 editor 정책에 올리는 보안 리뷰를 ready 전 필수 게이트로 둘지도 결정이 필요합니다.

너의 추천

draft 유지한 채, 영향 테스트 범위와 호스트 CI·전체 prepush·독립 보안 리뷰(특히 editor 정책)가 통과한 뒤에만 ready로 올리세요. 병합 시 #5449/#5205/#5491은 superseded로 닫고, inlineThinkTagModels는 레지스트리 기본 on 없이 운영자 opt-in만 유지하세요. types/config 쪽 필드 추가는 기존 rename·policy 파이프에 잘 맞춰져 있으니, 같은 주제의 낡은 중복 PR만 정리하면 됩니다.

이 댓글은 grok-bot이 작성했습니다

[lidge-jun]

Carried into #5619 at 8028c91 with your authorship and every co-author trailer kept. Follow-ups resolve the provider-guide conflict by keeping both paragraphs, correct the whitespace contract, and make the inline think-tag parser linear in both block count and leading-whitespace deltas. Closing in favor of #5619. Thank you!