Repository navigation
refactor(imports): retire 8 David skills, make 10 David/Matt skills Stack-owned - #55
Draft
thecolormaroun wants to merge 1 commit into
Draft
thecolormaroun wants to merge 1 commit into
thecolormaroun wants to merge 1 commit into
Conversation
…tack-owned #51 kept 14 David skills and Matt's merge-conflict skill at their last reviewed versions after upstream removed them. This resolves each one. Retired (built for David's own setup, or duplicated): david-pi-web-search, david-pi-custom-model, david-push-skill-to-github, david-distribute-skill-to-all-agents, david-run-deep-swe, david-level-up, david-short, david-teach. Stack-owned, moved out of skills/imported into their family roots with names, families and commands unchanged: david-youtube-transcript, david-research-prompt, david-google-safe-browsing, david-cyber-audit, david-create-readonly-db-role, david-prompt-me, david-brain-to-docs, david-agent-self-scheduling, david-folder-specific-claude-and-agents-md, matt-resolving-merge-conflicts. Each keeps its origin and MIT notice. The new stack-owned-imports source row records them as repository-owned-capability, outside the maintenance allowlist. The classifier no longer rewrites the hand-reviewed stageshow manifest, so a rerun is a no-op again. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
#51 kept 14 David skills and Matt's merge-conflict skill at their last reviewed versions after upstream removed them. This PR decides each one for good: 8 retired, 10 Stack-owned. Draft; not merged, not installed.
Retired
I read each SKILL.md first and kept none of them.
david-pi-web-searchdavid-pi-custom-model~/.pi/agent. No Pi on this machine.david-push-skill-to-github~/.agentsrepo through cmux panes.david-distribute-skill-to-all-agentsdavid-run-deep-swe~/.zshrc.david-level-upnotes/learning/david-knowledge.md, tuned to David.david-shortdavid-teachmatt-teach, which stays.Stack-owned
These no longer track upstream. Each moved out of
skills/imported/into its family root. Canonical name, family, command, role and trust are unchanged, sostack:<name>still works.references/source.mdkeeps the origin path, the last inspected commit and the full MIT notice, and the SKILL.md header now says "Stack-owned, adapted from …".david-youtube-transcript,david-research-promptskills/knowledge/david-google-safe-browsing,david-cyber-auditskills/review/david-create-readonly-db-role,david-prompt-me,david-brain-to-docsskills/engineering/david-agent-self-schedulingskills/orchestration/david-folder-specific-claude-and-agents-mdskills/platform/matt-resolving-merge-conflictsskills/planning/. Matt removed the skill in 1.3.0 (CHANGELOG) and kept only an archived docs page.The 12 other David skills still exist upstream and stay on the maintenance lane here. The next PR changes that.
What else changed
registry/maintenance-imports.json: every David retained target is gone, and so is Matt's merge-conflict one. Matt's olderwriting-great-skillsretention andCONTEXT-FORMAT.mdare untouched.registry/maintenance-sources.jsonandconfig/stack-maintenance.json: a newstack-owned-importsrow (repository-owned-capability) lists the 10 paths, and its id is inlegacy_target_ids. The paths sit outside the maintenance diff allowlist, so the lane can't write to them.scripts/classify-capabilities.py: retired and converted names left the hardcoded family sets, because path now decides family. It also now skips manifests from import providers it has no rules for. Onmain, a rerun rewrotebenjitaylor/stageshow's hand-reviewed manifest (family, context, validation evidence) and so was not idempotent. A new test reruns the classifier on a copy and requires zero manifest changes.registry/capabilities.json, the layout migration, the reconciliation artifact andconfig/runtime-targets.jsoncatalog_digest. Retired skills left the layout record; converted ones record their new destination.docs/stack-maintenance.mdrecords both lists with reasons. The router'simported-skills.mdreference was updated to match, and I dropped a staledavid-online-shoppingrow from it.Watch out
ad5bedd,skills/<category>/<name>becameskills/<name>), after the audit Preserve approved imports during upstream maintenance #51 was built from. The 12 David skills this PR keeps point at paths that no longer exist. The next PR makes David pick-only, which clears it.Validation
python3 -m unittest discover -s tests: 531 tests, all pass.build-capability-registry.py --check,classify-capabilities.py --check,apply-skill-layout.py --dry-run(0 pending moves) andsync-upstreams.pypass. A classifier rerun leaves the tree unchanged.🤖 Generated with Claude Code