Skip to content

Latest commit

 

History

History
274 lines (226 loc) · 34.9 KB

File metadata and controls

274 lines (226 loc) · 34.9 KB

Credits

When a maintainer lands another author's pull request by reimplementing, carrying, or rebasing it, the resulting commit is authored by the maintainer. The contributor's name survives only through a Co-authored-by trailer — that trailer is what GitHub reads for the contributor graph, the repository's contributor list, and the author's own profile activity.

Some of those landings carry the trailer. Others state the debt in the commit body and omit it:

  53c09a247  "Clean reimplementation of #3193"   Co-authored-by: alan7629 ...
  5734a1caf  "Reimplements #2797 by @rrmlima."   (no contributor trailer)

Both sentences are equally sincere. Only the first is data.

The commits below are inside published release tags and behind branch rulesets that block force-pushes, so the trailers cannot be added retroactively without invalidating every tag and clone — MAINTAINERS.md states the same principle in the other direction: authorship credit in git history is not rewritten. This file is the forward repair.

Every entry cites the maintainer's own words from the closing comment, pull-request description, or landing commit. Nothing here is inferred from a diff.

This file is not a contributor list. Most contributions merged normally, with authorship intact, and need no entry. Absence from this page means the ordinary path worked.

Carried work

Code, design, or tests from these pull requests shipped.

Pull request Author Landed as What landed
#1801 @jonathanli12 cb48c2e11 "carries all three of its unique tests" — the Cursor code-mode contract
#2123 @chilung-cgu ef7b3c9cf "Your account loop and the reuse of getTokenForAccountQuotaProbe are what shipped"
#2655 @TooSpace 607042b02 "re-implemented on current dev from your design"
#2693 @yxr1995-maker d829215af, bdc1e97bb "carries your fix forward with the three review blockers closed"
#2734 @TooSpace 88c427522 "That carry keeps the adaptive effort-mode design"
#2744 @yxr1995-maker 8877df0ee "Your diagnosis held up"; the landed fix reimplements it narrowly
#2796 @rrmlima bb3321ca8 "Reimplements #2796 by @rrmlima"
#2797 @rrmlima 5734a1caf "Reimplements #2797 by @rrmlima"
#2812 @gaoran1209 c986d1d20 "Reimplements #2812 by @gaoran1209 with the maintainer's blocker addressed"
#2867 @Ingwannu 8d1dc1f5d "That landed change includes this PR's strict LoadState parsing"
#2870 @luvs01 de91dfde4 "the coalescing design here is right, and it is carried forward"
#2884 @chilung-cgu eb52973c5 "Completes contributor PR #2884"; the exact-name approach carried as-is
#3000 @MarcTCruz fecb77a91 "Your central insight" — the refresh lock and the file it protects live under different homes
#3039 @ntdatt812 b14b741dc "keeps your production logic exactly as written — the Windows budget, the waited guard, and the grace probe"
#3041 @ntdatt812 b46164e78 "carries your three merge-loop tests … they came from this PR"
#3067 @ntdatt812 b14b741dc "keeps your diagnosis and your relocation", with the remedy narrowed
#3078 @Veritas-7 0ef04e640 "reimplements both of your production hunks on dev"
#3142 @olddonkey 52d941640 "That carry keeps the measurement/refusal work and ships the guard default-off"
#3300 @S0RYUASUKA 15b43e51c the same two test files made hermetic
#3284 @mdwsk88 3d3c4fe26 "Core implementation is already on dev via #3286 (3d3c4fe26), including the suffix wire ladder, picker collapse, Google adapter coverage"

2026-09-07 follow-up: missing or malformed trailers

These additional landings are present in the audited 3,000-commit window. The linked landing descriptions or commit messages identify what was taken.

Pull request Author Landed as What landed
#1748 @Blushyes e3bbf5321 Scoped reimplementation of the outbound Fake-IP discovery fix; the source PR author is @Blushyes, correcting the name in the landing message.
#1842 @luvs01 e1e431332 Public OAuth error projection and preservation of typed authentication failures.
#1889 @dbc-hbin ea16f8613 The remaining x-goog-api-client removal and the contributor's header assertions.
#1896 @luvyoun0224-beep 5f2b93979 The functions-namespace parser flattening; the hardcoded-name guidance was not taken.
#1899 @ntdatt812 fb5ceee35 The catalog-writer temp-path binding, extended with ordered write/harden/publish assertions.
#1920 @Yuxin-Qiao 34b167367 Scoped native and replay Computer Use tool-result normalization.
#1932 @harryzhou2000 f2b507f83 The transient bare-401 concept and test scaffolding; the literal trailer "PR #1932" identifies no account.
#2027 @yzxcj797 5445ce3e6, 293494cfa "Absorbed from #2027" — OpenCode Go identity follows its destination.
#2040 @Ingwannu 16345ab8b "implementation and tests, with two corrections".
#2053 @Ingwannu f4ad13922 "Carries @Ingwannu's #2053 unchanged."
#2056 @Ingwannu 9b0c5a02d The shortPercent known-quota and scoring changes.
#2075 @olddonkey 647b98eb8 The native Chat fast-capability gate, reconciled with the exact-ID contract.
#2077 @ntdatt812 f9c224b70 "Both patches are @ntdatt812's work from #2100 and #2077, applied unchanged."
#2082 @yzxcj797 06cdbc109, 057e8575c AgentRouter opening-turn framing, with exact hostname and separate-block corrections.
#2099 @yzxcj797 81492fd10 The repro-shaped prompt-cache-retention test fixture; the runtime contract came from #2102.
#2100 @ntdatt812 f9c224b70 "Both patches are @ntdatt812's work from #2100 and #2077, applied unchanged."
#2101 @Ingwannu 0bce9516d The entitlement module and full test suite, with maintainer corrections.
#2102 @lilinxiong 5904178c3 The prompt-cache-retention implementation, narrowed to the exact native model family.
#2104 @olddonkey 7cd270dc2 "Carries @olddonkey's #2104 unchanged."
#2105 @lilinxiong a0635eaa2 "Carries @lilinxiong's #2105 implementation and tests."
#2109 @drakonkat d2493a147 The shared base-URL override implementation from #2109 and #2110.
#2110 @drakonkat d2493a147 The shared base-URL override implementation from #2109 and #2110.
#2122 @chilung-cgu 6a6efa928 Configuration union, schema and migration design for retainModels; #2860 supplied the retention predicate.
#2127 @agentHits 1adcfde0c "Carries @agentHits's #2127 unchanged."
#2131 @bet4it 8ff77e11e "Carries @bet4it's #2131 implementation and tests."
#2155 @waw4303 64ba54edb, 772d375fe Resolved tool-call padding handling, with per-field provenance and diagnostic corrections.
#2227 @olddonkey 63d387cae The Grok OAuth registry flip, structure documentation and test conversions.
#2432 @mdwsk88 850afb2e9 The omit-sentinel documentation and provider wire comments.
#2639 @bet4it fefeb0501 The Responses item status backfill; queued/in_progress mapping was corrected separately.
#2647 @darwintree e1e6ec04f Command Code profile metadata, reconciled with the intervening catalog update.
#2663 @Eleven-is-cool cb9bb9b76 Bare code-mode helper calls through exec: "the implementation is yours, unchanged".
#2938 @luvs01 8427efe6e The failed-wrapper diagnosis and linear-scan design, with six behavioral divergences corrected.
#3069 @justin-mc-lai a0d386b49 The two query/queries parity commits, followed by malformed-history boundary fixes.
#3329 @Veritas-7 3ac310782 Combo cooldown/wait design with corrected reset metadata, clocks and Retry-After handling.
#3407 @turin-dev 3b3fe21d4 The desired-state Codex switch, observed-state badge and effective-home path.
#3421 @Skyline-23 89c0a64fe The Compose/container foundation, with loopback binding and generated compatibility identity.
#3469 @agentHits c44e187ee Google location-error classification, carried through #3547 and corrected for error precedence.
#3487 @Ingwannu f8ba644f3 The bounded Kiro fallback-execution assertion at the migrated test path.
#3489 @Flowershangfromthebranches 55395a9dc Canonical-final-URL discovery injection alongside independently proxy-bound IPv6 handling.
#3528 @benedictusrey bef04efbc The effort CLI command, exact selectors and distinct live/offline failures.
#3531 @benedictusrey 45045623b The agy alias, with captured discovery authority retained.

2026-09-07 follow-up: unlinked trailers

These commits contain a contributor name, but GitHub's commit-author mapping does not resolve that trailer to the source PR author. No personal addresses are reproduced here. The forward correction uses account-linked noreply identities.

Pull request Author Landed as What landed
#2817 @gulup 6fe46312c The opt-in upstream Responses WebSocket transport and six carried commits.
#3148 @Veritas-7 865a36ef0 The two subscription-launch admission-key fixes, plus connected-launch reconciliation.
#3293 @Veritas-7 3a9c4d297 The missing claude-fable-5-1 model metadata and accompanying usage-cost test.

The correction commit records these contributors and the earlier Carried work authors as co-authors. This is forward attribution: the old commit objects, their original dates and release tags are unchanged.

2026-09-07 follow-up: four-track source-to-landing attribution

At the owner's request, this audit makes the original PR titles, authors and delivered slices explicit for the four follow-up tracks after #3771. All linked landing commits are ancestors of 5759d9ea2f1e7281cdc01eb9628f2e0a123fb59c. The human contributors already resolve through reachable source commits or merge trailers; a merge commit with no repeated trailer does not erase its parents' authorship. This forward record strengthens discoverability without claiming that every earlier landing omitted credit or rewriting existing commits and tags.

Original pull request Original author Landed through Delivered scope
#3769: fix(responses): fallback to routed compaction on 404 and enable quota failover on incomplete terminal @ideabib #3791 (fcf07446aa) Quota/incomplete attribution only; native compact 404 fallback remains outside this landing.
#3736: fix: preserve compaction progress and use a 600s stall budget @Hylouis233 #3792 (823ffeb771) Content-free buffered-compaction progress; the proposed global 600-second default was not adopted.
#3744: fix(server): opt the compact route out of the request idle timeout @mashfromband #3792 (823ffeb771) Accepted compact-request lifetime, with complete-body admission and bounded response-body inactivity.
#3740: fix(responses): answer a wrapped WebSocket rejection with its HTTP status @FredAmartey #3793 (110623ecfc) Precommit wrapped WebSocket rejection status; mid-turn failures retain their separate boundary.
#3779: fix(chat): preserve completion semantics in JSON-to-SSE fallback @Ingwannu #3803 (ac4a7659fd) JSON-to-stream tools, reasoning, usage and finish-reason preservation, extended across both fallback paths.
#3730: feat(claude): gate routed protocol compatibility @yansigit #3806 (4255bfac61), #3808 (5759d9ea2f) Opt-in translated Messages compatibility and bounded diagnostics; integrated with the final fixture layer.
#3747: fix(container): persist Codex home separately from OCX state @Ingwannu #3788 (ad5285e415) Separate persisted Codex home under the read-only container root, with serializer and documentation corrections.
#3324: docs(skill): keep access-key secrets out of agent sessions @luvs01 #3789 (26fa36424a) Agent-facing secret-bearing command and rotation-recipe restrictions, including aliases and management API spellings.
#3632: feat(config): add exclusive initialize-if-missing primitive @yansigit #3796 (443310e5dc), #3802 (f89b815090) Exclusive initial configuration publication and its real setup consumer, with filesystem and cancellation corrections.
#3728: feat(quota): show subscription credits in capacity bars @yansigit #3798 (b72b8ea6c8) Subscription-credit quota rows, including duplicate-label and displayed-row urgency handling.
#3250: perf(logs): poll request history incrementally @chilung-cgu #3800 (57211f43d4) Incremental request-history polling, extended to preserve changed requests and reset behavior.
#3383: feat(models): add main picker ordering controls @x3M3x #3801 (8615f1a1c9) Picker-order controls and isolated saves; unrelated source-PR Windows changes are not credited as part of this layer.

The source commits for @yansigit also credit Yumi. That original automation attribution remains in the reachable history; this audit does not invent a GitHub account mapping for its unlinked automation identity. The forward human trailers use the source authors' verified numeric GitHub account identities.

A delivered slice is not a statement that every requirement in its original PR or umbrella issue is complete. The table deliberately retains the unadopted scope.

Report and diagnosis

These fixes exist because of the report. The branch's own approach was not the vehicle, and each author was told why at the time — recording them as carried code would misstate what happened in the other direction.

Pull request Author Fix landed as Maintainer's words
#2925 @ncepuee 1d9b389c1 "Credit to @ncepuee, whose #2925 identified this and argued the split"
#3006 @Ingwannu 870a2adb6 "your PR correctly identified the broken invariant and verified the target was unused"
#3038 @L-Y-J e9d198a3c "the defect is real and #3107 exists because you found it"
#3040 @ntdatt812 330470e74 "The defect you found is real"
#3117 @olddonkey b46164e78 "Thank you for the focused report and tests"
#3143 @Ingwannu 408652698 "The diagnosis here was yours and it was right"
#3223 @alex-jordan547 d23eab43a "The report itself was what made the fix quick; the wire capture pointed straight at the cause"

Four-track reports and diagnostic evidence

These issue authors supplied the reports or observations used by the follow-up work. They are acknowledged as reporters, separately from the carried PR authors.

Report Reporter Follow-up Contribution
#3778 @turin-dev #3786 Non-atomic cleanup-manifest failure report.
#3746 @juzijia #3788 Read-only container Codex-home persistence failure.
#3770 @turin-dev #3803 JSON-to-SSE completion-semantics loss.
#3767 @turin-dev #3805 Refusal loss across Chat projections.
#3775 @leonclab #3804 Unsupported effort report; the delivered fix is limited to proven native capability aliases.
#3661 @Hu9956 #3794 Encrypted-task recovery failure classes; the landed change exposes bounded reasons.
#3522 @stephen-drew #3790 Same-process Windows spill failure evidence; the landed change separates timeout origins.
#3781 @jaychou0642-create #3799 Canonical-destination/Fake-IP quota-path investigation; field acceptance remains separate.

Diagnostic-only delivery does not establish that the reported runtime failure has been resolved.

Closed as landed, carry not stated

Two more were closed with a landing commit and nothing further. The landing is recorded; what was taken is not, and inventing an answer would be the same inaccuracy this file exists to correct.

Two further source PRs were closed with a landing reference but without an explicit statement of what was carried. Their authors are acknowledged here; they are not counted as carried code solely from that closure:

How this stays accurate

This page is a repair, not a process. The process is missing_coauthor_credit in .github/scripts/pr-hygiene.cjs: a pull request whose own text says it reimplements, supersedes, carries, or rebases another author's pull request fails the hygiene gate until a Co-authored-by trailer names that author. New entries here should be unnecessary.

If you find a landing that belongs on this page, open an issue. Being missed is the defect this file documents, not a claim you have to argue for.

A gap the gate does not close

The gate checks that a trailer is present. It cannot check that the trailer resolves to the account it names.

A 2026-09-04 backlog review found carry PR #3374, carrying #3333 by @blackjune67, with:

Co-authored-by: hajune <contributor@work-domain.example.test>

(The address is masked here — privacy:scan blocks real contributor emails in the tree. What matters is its shape: a personal work address, not a GitHub-linked one.)

That is the git identity on the contributor's own commits, so it looks correct in every review. But GitHub attributes co-authors by account-linked email, and that address is linked to no account — so the trailer would have credited nobody, and the contributor would have been invisible on their own patch. The gate passed it, because a trailer was there.

It was corrected before the merge to the contributor's account-linked users.noreply.github.com address, which is why there is no table row for it above.

The lesson generalizes: when carrying work, take the trailer address from the author's GitHub account (the numeric-id users.noreply.github.com form is always safe), not from the commit metadata on their branch. A contributor who commits under a work email is the normal case, not an edge case.

Verify the landing, not just the proposal

The 2026-09-07 audit read exactly 3,000 commits reachable from 7d8523eed75a67f7a4a15b533744fcd0e6059aa8, ending at 53130de4e540fbfcf2629079effb851af54e989e, and followed source-PR descriptions, closure comments and GitHub commit-author mappings. Normally merged work, credited cherry-picks, independent fixes and report-only acknowledgements were kept distinct from the carried-work tables.

A PR description or an intermediate branch commit can contain the right trailer and still lose it when a custom squash message replaces that text. Before calling a carry credited, inspect the actual landing commit: its final Co-authored-by trailer must remain present and resolve to the source author's GitHub account. The existing presence gate alone does not establish either fact.