Skip to content

docs: record the stale-head trap and the patch-id landing audit - #9738

Merged
proggeramlug merged 6 commits into
mainfrom
train122
Sep 4, 2026
Merged

docs: record the stale-head trap and the patch-id landing audit#9738
proggeramlug merged 6 commits into
mainfrom
train122

Conversation

@proggeramlug

Copy link
Copy Markdown
Contributor

Follow-up to #9737. Records in MERGE_GUIDE.md the two process failures that train caught, so the next session does not repeat them:

  • Re-fetch at assembly time, not just at audit time. Auditing and building a train are minutes-to-hours apart, and reviewed follow-ups land in that window. That is how trains 119/120 shipped the pre-review version of fix(gc): right-size idle arena capacity #9731.
  • Audit what landed by patch-id. git rev-list origin/main..<pr-head> is useless after a rebase-merge — it rewrites every SHA, so every landed PR reads as unlanded. git cherry -v compares patch-ids and survives it, with a note on its own false positives when a train reshaped commit boundaries.

64/64 lint gates.

Ralph Küpper added 6 commits September 4, 2026 17:03
The reordered initializer scan is a superset, not a replacement: a
self-recursive arrow, a named function expression, and a binding whose
initializer both creates a closure over it and is called immediately all
passed before this change and must keep passing. Verified identical on
both arms of the same-commit A/B.
@proggeramlug
proggeramlug merged commit ef79a02 into main Sep 4, 2026
@proggeramlug
proggeramlug deleted the train122 branch September 4, 2026 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant