docs(integration): record the branch retirement and audit the stale refs - #666
Open
RonenMars wants to merge 1 commit into
Open
docs(integration): record the branch retirement and audit the stale refs#666RonenMars wants to merge 1 commit into
RonenMars wants to merge 1 commit into
Conversation
…efs [skip-ci] Closes the decision left open by repo-health brief 06: the long-lived integration branch is retired, every open PR having been landed on main individually. Records the six refs deleted, the two backups kept, and the file-level audit that made deletion safe — across sixteen surviving integration-named refs, not one holds a file main has never had. Notes that their apparent unique content is the Maestro flows and stray logs main deliberately removed in #531, #474 and #458, so a PR from any of them would be a revert wearing a recovery title. [skip-ci]
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.
Closes the decision left open by
repo-health/06-integration-branch-decision.md, which asked for a decision to be recorded and for #575/#580 to get a path that is not manual conflict resolution.Both are settled: the long-lived integration branch is retired, every open PR having been landed on
mainindividually (20 merged, 4 closed unmerged), and #575/#580 were closed rather than conflict-resolved because their content was already onmainvia #578/#579.Why no PR should ever be opened from the remaining branches
Sixteen integration-named refs still exist. Each shows 16–19 non-doc files that
maindoes not have, which reads like unlanded work. It is not.The test that settles it is not "is this ref merged" — none of them are ancestors of
main, because every PR that fed them was squash-merged under a new SHA. The test is whether the ref holds a filemainhas never had:Across all sixteen refs the count is zero.
Their apparent unique content is the Maestro flows and stray logs that
maindeliberately removed in #531 (test(e2e): delete the maestro flows that nothing runs), #474 and #458.A PR from any of them would revert those three merged PRs under a title that reads like recovery.
Also recorded
e2e/live-chat-tab.yamlcarried an API key, removed frommainwhen #531 deleted the flow — but the stale refs predate that deletion and still carry it, verified at 3API_KEYlines each.Retiring them reduces how many refs hold the blob; it does not remediate it, since any existing clone already has the history. Rotation is the fix.
The detached
integration-merge-android-r8worktree sits on a commit titledfix(android): pin expo-modules-core for R8 ColorCompatthat contains no Android code — its whole diff is fourios/Podfile.lockSPEC CHECKSUM lines, the path-dependent noiseCLAUDE.mddocuments. Worth knowing before someone goes looking for that fix.Scope
Documentation only. This PR deletes nothing. The recommendation is to drop all sixteen, with two held back for an owner's call rather than an audit result:
archive/26-07-2026.18-44-integrationis explicitly named archive, andfeat/in-chat-search-integration/feat/live-external-sessions-integrationare feature branches that merely contain the word.