docs(devlog): post-2.49 reconciliation — 101 units to _fin + scope-cleanup records - #4125
Merged
Conversation
Contributor
|
Important Review skippedToo many files! This PR contains 800 files, which is 500 over the limit of 300. To get a review, reduce the PR to 300 files or fewer by splitting it into smaller PRs or changing its base branch. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: ⛔ Files ignored due to path filters (38)
📒 Files selected for processing (800)
You can disable this status message by setting the 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 |
Contributor
|
✅ Deterministic PR hygiene checks passed. |
github-actions
Bot
added
the
documentation
Owner
Author
리뷰 · 우선순위 58 / 80설명 이 PR은 제품 코드를 건드리지 않는 2.49 출하 직후 정리용 문서 PR이다. 지금 구체적으로는 세 덩어리다. 첫째, 현재 다만 숫자 표기가 지금 checkout과 어긋나는 곳이 두 군데 있고, disposition 표가 방금 열린 #4123을 빠뜨렸다. 내용은 방향이 맞으니 작은 숫자/표 수정 후 머지가 가장 값싸다. 제품 리스크는 사실상 없다.
메인테이너의 판단이 필요한 지점
너의 추천 이 댓글은 grok-bot이 작성했습니다 |
Ingwannu
requested changes
Sep 9, 2026
Ingwannu
left a comment
Ingwannu
left a comment
Owner
There was a problem hiding this comment.
Targeted review of the new reconciliation/disposition records; this is not a claim that all 837 changed paths have been independently audited.
Please correct 010_reconciliation.md against the immutable trees: git ls-tree -d --name-only 57077ca3:devlog/_plan counts 157 and the head counts 57; _fin counts 429 -> 528. These are not the stated 158 -> 57 / 432 -> 533. Distinguish renamed units from already-present/merged duplicates so the terminal audit is reproducible.
More importantly, move #4121 out of CLEAR-FIX/no-PR into POLICY/upstream clarification in 030_issue_disposition.md. The issue proposes fabricating OpenCode client/session headers to pass an upstream gate whose message says that the free tier is OpenCode-only. Generic Zen interoperability documentation is not proof that this exact keyless tier authorizes that workaround. I left the supporting source/clarification request on #4121; the work queue must not pre-authorize implementing that bypass as a clear bug fix.
Date the open-issue inventory as an explicit capture time rather than a perpetual count; reconcile #4123 and #4126 if taking a new snapshot. Keep branch/worktree cleanup described as a historical record only—this PR must not authorize further deletions. No runtime code delta was found outside devlog, but docs-only CI does not validate these factual claims.
3 tasks
lidge-jun
added a commit
that referenced
this pull request
Sep 9, 2026
Closes #4149. The exhausted-combo 413 mapping no longer requires stream === true. It carries the loop's own classifyOverflow verdict forward through lastFailureClassifiesOverflow, so a hopping 413 (err_free_prompt_cap) is classified on both wire shapes while a local input_admission_refused keeps its own non-streaming diagnostic. A second commit repoints three deferred-verb route ownerDocs at devlog/_fin. The post-2.49 devlog reconciliation in #4125 moved those units and left the registry naming the old _plan paths, which turned "route exemptions stay honest" red on dev itself. Exact-head CI at 5095f1b: Cross-platform CI success, Enforce PR target branch success (run 34406302801, the latest at this SHA; the two earlier failures were the escaped-newline PR body, fixed before that run), PR hygiene success, PR Labeler success, React Doctor success. Only the CodeRabbit review bot is outstanding. The new regression asserts both combo targets were hit, which is what separates the exhausted path from the stop path already covered.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Post-2.49 housekeeping before the next bug/improvement round:
_planunits moved to_fin(158 -> 57 open units). Every move was verified by read-only recon lanes against public git history — terminal outcome recorded in the unit AND landed ancestry on origin/dev or origin/main. 6 abandoned units and 7 landed-but-unrecorded units received a short terminal doc (090_terminal.md/090_closeout.md) citing the merge evidence before the move.020_branch_worktree_cleanup.mddocuments the offline cleanup already performed locally (144 merged local branches, 54 clean tmp worktrees, 8 merged/closed origin branches deleted with per-item PR/ancestry evidence).030_issue_disposition.mdis the post-2.49 open-issue table (57 issues): 8 CLEAR-FIX (5 without PRs: Spark model-specific 5h header window is stored as account-level short quota on Pro accounts #4122, opencode-free: Zen rejects requests without x-opencode-session (free tier lock-in) #4121, [Bug]: revoked-but-time-valid pool credential keepslastCodexValidationStatus: "ok"— dashboard reports the account healthy indefinitely #4120, [Bug]: Mid-thread model switch from a native ChatGPT model to a routed provider dies with unreadable_encrypted_agent_task; agentTaskRecovery never runs because it is gated on threadSpawn #4089, Non-streaming 413 never reaches the terminal context-overflow mapping, so compaction fails with a raw 413 #4112), 8 POLICY, 9 LIVE-PROBE, 32 IMPROVEMENT — the input for the next-round scope interview.No product code changes; devlog/docs only.
Verification
find devlog/_finfor doubled directories: none (two nesting accidents found and fixed before commit).bun run typecheck/bun run test: NOT RUN (docs-only change; local suite deliberately skipped per maintainer instruction — CI is the gate).Checklist