Skip to content

chore: remove legacy maintainer loop - #8

Merged
BramVR merged 1 commit into
mainfrom
claude/remove-legacy-loop
Jul 22, 2026
Merged

chore: remove legacy maintainer loop#8
BramVR merged 1 commit into
mainfrom
claude/remove-legacy-loop

Conversation

@BramVR

@BramVR BramVR commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Brings origin/main in line with the local main checkout, which has been carrying this change unpushed.

Why

b54218a1 "chore: remove legacy maintainer loop" was committed locally but never reached origin/main. Because PRs #6 and #7 were squash-merged, git sees no shared history between the local commits and their squashed counterparts, so a plain git pull cannot reconcile the two sides. Cherry-picking the one commit that is genuinely missing is the clean way to close the gap.

After this lands, origin/main holds everything, and the local checkout can simply reset onto it.

What changed

Removes the legacy current-repository bram-maintainer-loop skill (412 lines) and its prompts/bram-maintainer-project-loop.md, then repoints every $bram-maintainer-loop reference at $bram-maintainer-loop-v2 across AGENTS.MD, README.md, VISION.md, config/bram-loop-repos.txt, and the github-project-triage, tdd, to-issues, to-prd skills. Maintainer orchestration now routes only through v2.

Conflict resolutions

The cherry-pick hit four conflicts, all caused by PRs #6 and #7 having reshaped the same regions:

  • README.md — the original commit edited the hand-maintained Codex symlink list that feat: add sync-skills skill mirror #6 deleted in favour of the scripts/sync-skills section. Kept the sync-skills version; the edited line is obsolete because the mirror is generated.
  • skills/codex-first/SKILL.md — both sides rewrote the maintainer pointer line. Kept feat: adopt codex-huge-context, refresh codex-first #7's wording ("Whole-repo/multi-repo maintainer coordination"), which already said $bram-maintainer-loop-v2.
  • AGENTS.MD — kept feat: add sync-skills skill mirror #6's scripts/sync-skills rule and took this commit's -v2 rename in the upstream-sync preserve list.
  • CHANGELOG.md — kept all four entries, newest first.

Verification

scripts/validate-skills passes at 41 skills, down from 42 exactly as expected with bram-maintainer-loop gone. No non-historical $bram-maintainer-loop references survive — the only remaining mentions are past changelog entries, which correctly describe what happened at the time.

Copilot AI review requested due to automatic review settings July 22, 2026 12:48
@BramVR
BramVR merged commit 2ca8886 into main Jul 22, 2026
2 checks passed
@BramVR
BramVR deleted the claude/remove-legacy-loop branch July 22, 2026 12:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Aligns the repo with the locally committed (previously unpushed) change by removing the legacy bram-maintainer-loop skill and routing all maintainer-orchestration references through bram-maintainer-loop-v2.

Changes:

  • Removed the legacy skills/bram-maintainer-loop skill (and its agent config) plus the associated prompts/bram-maintainer-project-loop.md.
  • Repointed documentation and skill guidance that referenced bram-maintainer-loop to bram-maintainer-loop-v2.
  • Added a changelog entry documenting the removal and updated the flagged-repos config header accordingly.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
VISION.md Updates vision wording to reference maintainer-loop v2 guardrails.
skills/to-prd/SKILL.md Updates loop-mode guidance to point at bram-maintainer-loop-v2.
skills/to-issues/SKILL.md Updates loop-mode guidance to point at bram-maintainer-loop-v2.
skills/tdd/SKILL.md Updates loop-mode guidance to point at bram-maintainer-loop-v2.
skills/github-project-triage/SKILL.md Updates autonomous-mode trigger text to refer to bram-maintainer-loop-v2.
skills/bram-maintainer-loop/SKILL.md Removes the legacy current-repo maintainer loop skill.
skills/bram-maintainer-loop/agents/openai.yaml Removes the legacy skill’s OpenAI agent interface metadata.
README.md Updates references and “preserve” list to bram-maintainer-loop-v2.
prompts/bram-maintainer-project-loop.md Removes legacy project-loop prompt that invoked the old loop.
config/bram-loop-repos.txt Updates header comment to indicate v2 flagged repos.
CHANGELOG.md Documents removal of the legacy loop and prompt.
AGENTS.MD Updates upstream-sync preserve list to bram-maintainer-loop-v2.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants