Skip to content

fix(skills): derive bug-triage target repo from the checkout - #3

Closed
bfoxhound wants to merge 2 commits into
mainfrom
fix/bug-triage-derive-repo
Closed

bfoxhound wants to merge 2 commits into
mainfrom
fix/bug-triage-derive-repo

Conversation

@bfoxhound

Copy link
Copy Markdown
Owner

Summary

skills/bug-triage/SKILL.md hard-coded --repo aoagents/ReverbCode (upstream's product repo) in ~20 gh commands. Anyone running the skill from this fork would search for duplicates, file issues, comment, upload assets, and open PRs against that foreign repo.

This PR derives the target repo from the checkout instead:

  • Pre-flight now defines REPO=$(gh repo view --json nameWithOwner -q .nameWithOwner) (with git remote get-url origin noted as the equivalent) and instructs a sanity-check echo before filing anything.
  • All gh issue / pr / label / api / search commands use $REPO.
  • Appendix B (Remote Code Inspection, no local clone) sets REPO=<owner>/<repo> explicitly, since there is no checkout to derive from there.

Intentional omissions

  • Prose describing the product as ReverbCode (stack description, port :3001 guidance, pitfalls) is untouched — only repo-slug wiring changed.
  • One mention of the old aoagents/ReverbCode slug remains deliberately, in the Target-repo bullet, explaining why hard-coding it was a bug.
  • ao spawn --project reverbcode in §5f is a project registration name, not a GitHub slug, and is out of scope.

Test

Docs-only. grep -c 'aoagents/ReverbCode' → 1 (the explanatory mention); all 22 command sites now use $REPO.

🤖 Generated with Claude Code

bfoxhound and others added 2 commits July 27, 2026 21:37
The bug-triage skill hard-coded --repo aoagents/ReverbCode (upstream's
product repo) in every gh command, so anyone running the skill from this
fork would search for duplicates and file issues against a foreign repo.

Define REPO=$(gh repo view --json nameWithOwner -q .nameWithOwner) in
pre-flight and use $REPO throughout. Appendix B (remote inspection, no
local clone) sets REPO explicitly since there is no checkout to derive
from. Prose that names the product (ReverbCode) is left intact; one
explanatory mention of the old slug remains deliberately in the
Target-repo bullet.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Test Coverage Report

No TypeScript source files changed in this PR.

@bfoxhound

Copy link
Copy Markdown
Owner Author

Closing unmerged: superseded — both fixes landed via #4 and #5 (merged 07-28). Branch kept for reference.

@bfoxhound bfoxhound closed this Aug 4, 2026
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