Reduce merge-queue wait to 1 min; add MIGRATION.md entry doc + .vscode README#16
Draft
DenWin wants to merge 2 commits into
Draft
Reduce merge-queue wait to 1 min; add MIGRATION.md entry doc + .vscode README#16DenWin wants to merge 2 commits into
DenWin wants to merge 2 commits into
Conversation
Queued entries now merge after at most 1 minute instead of 5 — a solo repo gains nothing from batching entries longer. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vi2ifWUY3MXfbcbNFnsS8V
MIGRATION.md is the entry document for adopting the template's concepts in a repo not created from it: a concept-to-doc map plus the migration steps in dependency order. AGENTS.md's retrofit section now points there instead of duplicating the steps (one source, no drift). .vscode/README.md fills the one gap in per-folder docs — AGENTS.md's mechanism map already linked it, but the file did not exist; .gitignore now tracks it alongside the shipped settings. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vi2ifWUY3MXfbcbNFnsS8V
2 tasks
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.
Goal
Two things from the repo-concepts review:
gains nothing from batching that long — drop the merge-queue wait to
1 minute.
a repo not created from it (AGENTS.md's retrofit section was a terse
list), and the concepts inventory found one documentation gap: AGENTS.md's
mechanism map links
.vscode/README.md, which did not exist.Scope
setup/Protect-MainBranch.ps1—min_entries_to_merge_wait_minutes5 → 1;setup/Protect-MainBranch.Tests.ps1— new assertion pinning the value(test-first).
MIGRATION.md(new) — entry document: concept→doc map plus migration stepsin dependency order, linking each per-folder reference doc.
AGENTS.md— retrofit section now points to MIGRATION.md instead ofduplicating the steps (one source, no drift).
.vscode/README.md(new) — fills the missing per-folder doc;.gitignore— one-line exception so the README is tracked.These boundaries are correct because the wait-time change plus its test live
entirely in
setup/, and the rest is documentation only.Risk & rollback
Low. The docs have no runtime behavior. The wait-time change only affects the
ruleset payload the one-time setup script POSTs — it applies to a repo when
setup/Protect-MainBranch.ps1is (re-)run there with admin +gh; note thescripts cannot execute from the sandbox this PR was authored in (no
gh).Rollback: revert the commit; an already-applied ruleset can be adjusted in
repo settings → Rules.
Evidence
32 passed, 0 failed, including the new
waits at most 1 minute before merging queued entriesassertion.each exits 1 with
GitHub CLI (gh) is not installed.as designed.MD060: alignedrule; full
pre-commitsuite runs in CI on this PR.🤖 Generated with Claude Code
https://claude.ai/code/session_01Vi2ifWUY3MXfbcbNFnsS8V
Generated by Claude Code