chore: consolidate dev skills into onsager-ai/dev-skills#302
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR removes several previously vendored, cross-repo “dev-process” skills from this repository, relying instead on the consolidated global install from onsager-ai/dev-skills. It also updates remaining repo-local skills to link to the new canonical locations and clears the pinned skills list in skills-lock.json.
Changes:
- Remove vendored skills (
plan-dag,agent-browser,github-integration,parallel-worktrees,issue-spec) now hosted inonsager-ai/dev-skills. - Update surviving
lean-spec-*skill docs to link to the new upstream URLs (avoiding broken relative links). - Clear
skills-lock.jsonto indicate no skills are pinned by this repo.
Reviewed changes
Copilot reviewed 37 out of 37 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| skills-lock.json | Clears pinned skills list to an empty set. |
| .agents/skills/plan-dag/SKILL.md | Removes vendored plan-dag skill doc (migrated upstream). |
| .agents/skills/plan-dag/scripts/svg-to-png.mjs | Removes plan-dag rasterization helper (migrated upstream). |
| .agents/skills/plan-dag/scripts/plan-dag-render.test.sh | Removes plan-dag test script (migrated upstream). |
| .agents/skills/plan-dag/scripts/plan-dag-render.py | Removes plan-dag renderer (migrated upstream). |
| .agents/skills/plan-dag/fixtures/happy.json | Removes plan-dag test fixture (migrated upstream). |
| .agents/skills/plan-dag/fixtures/cycle.json | Removes plan-dag test fixture (migrated upstream). |
| .agents/skills/plan-dag/fixtures/bad.json | Removes plan-dag test fixture (migrated upstream). |
| .agents/skills/parallel-worktrees/SKILL.md | Removes vendored parallel-worktrees skill doc (migrated upstream). |
| .agents/skills/parallel-worktrees/references/worktree-lifecycle.md | Removes parallel-worktrees reference doc (migrated upstream). |
| .agents/skills/parallel-worktrees/references/github-pr-sync.md | Removes parallel-worktrees reference doc (migrated upstream). |
| .agents/skills/parallel-worktrees/references/agent-coordination.md | Removes parallel-worktrees reference doc (migrated upstream). |
| .agents/skills/lean-spec-pre-push/SKILL.md | Updates cross-skill links to point to onsager-ai/dev-skills. |
| .agents/skills/lean-spec-pr-lifecycle/SKILL.md | Updates cross-skill links to point to onsager-ai/dev-skills. |
| .agents/skills/lean-spec-dev-process/SKILL.md | Updates cross-skill links to point to onsager-ai/dev-skills. |
| .agents/skills/issue-spec/templates/issue-spec-template.md | Removes vendored issue-spec template (migrated upstream). |
| .agents/skills/issue-spec/SKILL.md | Removes vendored issue-spec skill doc (migrated upstream). |
| .agents/skills/issue-spec/references/spec-format.md | Removes vendored issue-spec reference doc (migrated upstream). |
| .agents/skills/issue-spec/.upstream-source | Removes upstream source marker for vendored issue-spec. |
| .agents/skills/github-integration/templates/copilot-setup-steps.yml | Removes vendored template (migrated upstream). |
| .agents/skills/github-integration/SKILL.md | Removes vendored github-integration skill doc (migrated upstream). |
| .agents/skills/github-integration/references/copilot-setup-steps.md | Removes vendored reference doc (migrated upstream). |
| .agents/skills/github-integration/references/cloud-auth.md | Removes vendored reference doc (migrated upstream). |
| .agents/skills/agent-browser/templates/form-automation.sh | Removes vendored agent-browser template (migrated upstream). |
| .agents/skills/agent-browser/templates/capture-workflow.sh | Removes vendored agent-browser template (migrated upstream). |
| .agents/skills/agent-browser/templates/authenticated-session.sh | Removes vendored agent-browser template (migrated upstream). |
| .agents/skills/agent-browser/SKILL.md | Removes vendored agent-browser skill doc (migrated upstream). |
| .agents/skills/agent-browser/references/video-recording.md | Removes vendored agent-browser reference doc (migrated upstream). |
| .agents/skills/agent-browser/references/snapshot-refs.md | Removes vendored agent-browser reference doc (migrated upstream). |
| .agents/skills/agent-browser/references/session-management.md | Removes vendored agent-browser reference doc (migrated upstream). |
| .agents/skills/agent-browser/references/proxy-support.md | Removes vendored agent-browser reference doc (migrated upstream). |
| .agents/skills/agent-browser/references/commands.md | Removes vendored agent-browser reference doc (migrated upstream). |
| .agents/skills/agent-browser/references/authentication.md | Removes vendored agent-browser reference doc (migrated upstream). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
1
to
4
| { | ||
| "version": 1, | ||
| "skills": { | ||
| "github-integration": { | ||
| "source": "codervisor/forge", | ||
| "sourceType": "github", | ||
| "computedHash": "14357468e9d8ef12bb896d08228f938bcc0e85a7eff758baec01f9edd2547e95" | ||
| }, | ||
| "parallel-worktrees": { | ||
| "source": "codervisor/forge", | ||
| "sourceType": "github", | ||
| "computedHash": "1848a67829d211a61b007f18d6f91690816c140b738b00265cd2f8fe46a556c6" | ||
| }, | ||
| "plan-dag": { | ||
| "source": "onsager-ai/onsager-skills", | ||
| "sourceType": "github", | ||
| "skillPath": "skills/plan-dag/SKILL.md", | ||
| "computedHash": "2dfad0d270654877aa3a5d663afcd80e69f8c238af0ccb170991f732075ed692" | ||
| } | ||
| } | ||
| "skills": {} | ||
| } |
Remove vendored copies of cross-repo dev-process skills (plan-dag,
agent-browser, github-integration, parallel-worktrees, issue-spec) —
these now install globally via 'npx skills add -g
onsager-ai/dev-skills --skill * -a claude-code'. The local
dev-process skills (lean-spec-dev-process, lean-spec-pre-push,
lean-spec-pr-lifecycle, leanspec-development, watch-ci) stay local.
- Drop .agents/skills/{plan-dag,agent-browser,github-integration,
parallel-worktrees,issue-spec}
- Drop corresponding .claude/skills/* symlinks
- skills-lock.json becomes empty (all three pinned skills migrated)
- Rewrite local skill cross-links to removed skills as absolute URLs
pointing at onsager-ai/dev-skills
tikazyq
force-pushed
the
chore/dev-skills-consolidation
branch
from
7eaa88f to
619e797
Compare
3 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.
Summary
Removes vendored copies of cross-repo dev-process skills now consolidated into
onsager-ai/dev-skills. Each developer machine installs them once globally:npx skills add -g onsager-ai/dev-skills --skill '*' -a claude-codeSkills removed (new home: onsager-ai/dev-skills)
plan-dag.agents/skills/plan-dag+.claude/skills/plan-dagsymlinkagent-browser.agents/skills/agent-browsergithub-integration.agents/skills/github-integration+.claude/skills/github-integrationsymlinkparallel-worktrees.agents/skills/parallel-worktrees+.claude/skills/parallel-worktreessymlinkissue-spec.agents/skills/issue-spec+.claude/skills/issue-specsymlink (byte-identical duplicate of canonical)Skills remaining (repo-local dev process)
lean-spec-dev-process,lean-spec-pre-push,lean-spec-pr-lifecycle,leanspec-development,watch-ci— these carry the lean-spec-specific area taxonomy, provider-impact / i18n discipline, and CI workflows, so they stay in.agents/skills/(with.claude/skills/symlinks).skills-lock.json
All three pinned skills (
plan-dag,github-integration,parallel-worktrees) were migrated, so the file becomes:{ "version": 1, "skills": {} }Kept the file to make it explicit that lean-spec pins no skills today; remove if cleaner.
Local skill cross-links
Rewrote relative
../issue-spec/SKILL.md/../github-integration/SKILL.mdstyle links in the survivinglean-spec-*skills to absolute URLs againstonsager-ai/dev-skillsso they don't 404 after removal.Test plan
npx skills add -g onsager-ai/dev-skills --skill '*' -a claude-codelands the 12 skills under~/.claude/skills/.🤖 Generated with Claude Code