Repository navigation
chore: batch buglog entries for the 2026-08-29 merges - #1455
Merged
Merged
Conversation
Append the fifty buglog entries carried in the bodies of the pull requests merged to main on 2026-08-29. The diff is .wolf/buglog.jsonl and nothing else, per the protocol in .claude/rules/openwolf.md. Entries are copied verbatim from their source pull request bodies. No line is rewritten and no field is invented. Entries already present on main, including the thirty six landed by the 2026-08-28 batch in #1342, are skipped. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WyEwUxZCArdn1ZUDkTvuQ1
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Review limit reachedNext included review available in 43 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
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 |
#1432 and #1448 merged after the first enumeration pass and both carried an entry. #1432 contributes two well formed JSON entries. #1448 wrote its entry as prose key and value lines rather than a JSON object, so it is reformatted here into one line with its content preserved verbatim and nothing added. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WyEwUxZCArdn1ZUDkTvuQ1
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
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
This is the batched buglog follow-up for the pull requests merged to
mainon 2026-08-29. Its diff is.wolf/buglog.jsonland nothing else.Per
.claude/rules/openwolf.md, every fixed bug, error, failed test or failed build must be logged, but the line may never be appended on a fix branch.merge=unionin.gitattributesresolves concurrent appends locally and is ignored by GitHub's server side merge, so two branches that both appended land in hard conflict there. An unmergeable pull request gets norefs/pull/N/merge, nopull_requestrun and therefore zero checks, and the required status gate then blocks the merge for a reason the page never states (issue #873). Each fix accordingly carried its entry in its own pull request body, and this pull request copies them ontomainin one batch, which the protocol explicitly prefers over one pull request per entry.Scope examined
Sixty one pull requests merged to
mainon 2026-08-29. Fifty of them carried at least one entry, for eighty five entries in total. Thirty two of those were already onmainand are skipped, leaving fifty three appended here from thirty six pull requests.#1432 and #1448 merged after the first enumeration pass and were folded in afterwards, in a second commit on this branch.
The largest block of skips comes from #1342, the equivalent batch for the 2026-08-28 merges, which merged earlier the same day and already landed thirty six entries covering #1257, #1268, #1276, #1277, #1287, #1292, #1293, #1294, #1296, #1301, #1303, #1305, #1313, #1335 and #1337.
What landed
Fifty three entries appended, one JSON object per line, append only. The 232 pre-existing lines are byte identical to
origin/main(verified by hashing the first 232 lines of the result against the base file). Every line in the resulting file parses as JSON and carrieserror_message,root_cause,fixandtags.Entries are copied verbatim from their source pull request bodies. Nothing was rewritten, no field was invented, and no field was added.
One entry needed repair. #1448 wrote its
Buglog entrysection as prose key and value lines (error_message: ...,root_cause: ...) rather than as a JSON object, so it was reformatted into a single line here. Every value is its author's own text with the body's line wrapping unwrapped, and theid,date,title,verification,tagsandprkeys it named are preserved. Nothing was added and nothing was dropped. The other eighty four extracted entries parsed as JSON on the first attempt with all four required fields present.Merged pull requests that carried no entry
Eleven of the sixty one. Recorded here because the gap is itself the useful signal.
it.failsmarkers reading as red is a real defect that was fixed here and should have carried an entrySix of the eleven are Dependabot bumps and one is the previous batch, so the genuine protocol gaps are #1364, #1383, #1384 and #1398. Of those, #1383 is the one worth a follow-up: it fixed a real defect class (a stale expected-failure marker reads as a red "Expect test to fail" and gets dismissed as pre-existing) and left no record.
A twelfth pull request, #1448, is not counted as a gap. It did write an entry, just not as JSON; see the repair note above.
Entries skipped as already present
Thirty two. Thirty of them matched an entry already on
mainonerror_message,idorfix. Two more from #1278 are semantic duplicates that an exact match would have missed, and were skipped after reading the landed entries they duplicate:streaming content_block_start omits text fieldentry is covered by the consolidatedbug-2026-08-28-anthropic-sdk-wire-conformanceentry landed from fix: make the Anthropic surface conform to the real SDK wire contract (#1274, #1260, #1261, #1259) #1296, whose root cause names the sameomitemptyonStreamContentBlock.Text.GET /v1/models leaked an upstream provider nameentry is covered byBUG-1284, landed from fix: keep upstream provider identity out of catalogue metadata (#1284) #1300, which names the samepublic.model_aliases.summarypublication path.#1278's third entry, on
top_kforwarding producing a 400, is not covered anywhere onmainand is appended here. #1342 recorded #1278 as fully "merged into #1296", which was accurate for two of its three entries.Note on entry quality
One appended entry is thin: #1277's parity re-score record carries
error_messageofn/aand a root cause of "console had no privacy/data-policy surface at all". It is a parity gap record rather than a defect record. It is included exactly as written rather than embellished, per the protocol's preference for the author's own words.Test plan
origin/main, diff is.wolf/buglog.jsonland nothing elseerror_message,root_cause,fixandtags.wolf/telemetry (anatomy.md,memory.md,token-ledger.json,hooks/_session.json,buglog.json) in the commit.github/workflows/ci.yml