Skip to content

Genericize skill-repo owner and de-link private references - #60

Merged
dmccoystephenson merged 1 commit into
mainfrom
oss-audit
Jul 27, 2026
Merged

dmccoystephenson merged 1 commit into
mainfrom
oss-audit

Conversation

@dmccoystephenson

@dmccoystephenson dmccoystephenson commented Jul 27, 2026 •

Copy link
Copy Markdown
Member

Summary

Part of prep to open-source this repo (follow-up to #59, the MIT license switch). Fixes two blockers for external usage:

  • Hardcoded username: the generated <slug>-dev-loop skill previously wrote dmccoystephenson/<slug>-dev-loop literally in four places (skill-repo header, both self-audit gh issue commands, and the abort-budget gap-issue instruction). Anyone else running /create-dev-loop would generate a skill that tries to file issues against the maintainer's account instead of their own. Replaced with a new {{SKILL_REPO_OWNER}} placeholder derived from gh api user -q .login at generation time, with a substitution-table row. Step 7's own instructions (not template output) now use the same $OWNER pattern Step 6 already established, instead of a literal username.
  • Private reference implementations: README.md, CLAUDE.md, and RESEARCH.md pointed contributors at example-l-dev-loop, example-q-dev-loop, and example-skills-catalog/CONVENTIONS.md as if they were inspectable — all three are private repos used for personal validation. Removed the dead "Related skills" section from the README, and reworded the CLAUDE.md/RESEARCH.md mentions to name them as private maintainer fixtures with a fallback instruction for contributors who don't have access (validate against any repo you maintain; note in the PR description which skills may need a retrofit pass).

Test plan

  • grep -rn dmccoystephenson create-dev-loop.md README.md RESEARCH.md CLAUDE.md returns no matches
  • README "What it does" Step list still 1:1 with create-dev-loop.md's seven Steps (per CLAUDE.md's doc-sync rule)
  • New {{SKILL_REPO_OWNER}} placeholder has a substitution-table row, consistent with the other {{placeholder}} rows
  • Not run end-to-end against a real repo (no automated test suite for this project; per CLAUDE.md, real validation is running /create-dev-loop live)

drafted by Claude on behalf of Daniel Stephenson

- Add {{SKILL_REPO_OWNER}} placeholder (gh api user -q .login) and use it
  everywhere the generated skill previously hardcoded dmccoystephenson as
  the skill-tracker repo owner, so the template works for any user.
- Make Step 7's my-claude-skills entry use the same dynamic $OWNER as
  Step 6 instead of a literal username.
- Stop presenting dpm-dev-loop, herald-dev-loop, and my-claude-skills as
  inspectable references in README/CLAUDE.md/RESEARCH.md — they're the
  maintainer's private repos; external contributors get a fallback
  instruction instead of a dead link.
@dmccoystephenson
dmccoystephenson merged commit 2f4b0b5 into main Jul 27, 2026
@dmccoystephenson
dmccoystephenson deleted the oss-audit branch July 27, 2026 00:52
This was referenced Jul 27, 2026
dmccoystephenson added a commit that referenced this pull request Aug 3, 2026
- Add {{SKILL_REPO_OWNER}} placeholder (gh api user -q .login) and use it
  everywhere the generated skill previously hardcoded dmccoystephenson as
  the skill-tracker repo owner, so the template works for any user.
- Make Step 7's a-private-repo-3 entry use the same dynamic $OWNER as
  Step 6 instead of a literal username.
- Stop presenting a-private-repo-1, a-private-repo-2, and a-private-repo-3 as
  inspectable references in README/CLAUDE.md/RESEARCH.md — they're the
  maintainer's private repos; external contributors get a fallback
  instruction instead of a dead link.
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