Relocate cross-organ skill installer into bin/#14
Merged
Conversation
Move the skill installer (install.sh) and line-count guard (check_skill_line_counts.sh) into PyAutoBrain/bin/. They are organism-wide infrastructure, not admin_jammy's own tooling, and admin_jammy is slated to leave PyAutoLabs/. - Repoint the vestigial admin_jammy discovery root to an absolute path so it self-skips once admin_jammy is gone; PYAUTO_ROOT (../..) still resolves to the PyAutoLabs root from bin/. - Add bin/README.md carrying the relocated installer docs. - Note the installer + guard in the top-level README. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Relocates the cross-organ Claude skill installer and line-count guard into PyAutoBrain/bin/, aligning this organism-wide tooling with the Brain’s orchestration role and updating documentation to point to the new location.
Changes:
- Adds
bin/install.shto auto-discover and symlink skills/commands from organism repos into~/.claude/, with pruning of stale/broken PyAuto-root symlinks. - Adds
bin/check_skill_line_counts.shto enforce primary skill/command markdown size limits across the same discovery roots. - Updates top-level and
bin/documentation to describe the new location and usage.
Reviewed changes
Copilot reviewed 1 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| README.md | Documents that the cross-organ installer/guard now live under bin/. |
| bin/README.md | Adds usage and behavioral documentation for the installer and line-count guard. |
| bin/install.sh | New cross-organ installer script: discovers skills/commands per root and links them into ~/.claude/. |
| bin/check_skill_line_counts.sh | New guard script: checks primary skill/command markdown files against a line limit and exits non-zero on violations. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
Move the cross-organ Claude skill installer (
install.sh) and line-count guard (check_skill_line_counts.sh) fromadmin_jammy/skills/intoPyAutoBrain/bin/, alongside the existingpyauto-brainCLI.They are organism-wide infrastructure, not admin_jammy's own tooling — after the organism re-home admin_jammy hosts no skills of its own, and it is a separate personal repo (
Jammy2211/admin_jammy) slated to leavePyAutoLabs/. PyAutoBrain, the reasoning/orchestration organ, is the right home.Changes
bin/install.sh,bin/check_skill_line_counts.sh— relocated. The self-referencing$SCRIPT_DIRadmin discovery root is replaced with an absolute$PYAUTO_ROOT/admin_jammy/skills, which is now a vestigial root that auto-skips once admin_jammy leaves.PYAUTO_ROOT="$SCRIPT_DIR/../.."still resolves to thePyAutoLabs/root frombin/(same depth asskills/), so behaviour is unchanged today.bin/README.md— installer docs relocated here.README.md— note the installer + guard now live inbin/.Verification
bash PyAutoBrain/bin/install.sh→ exit 0; discovers all organ roots, admin_jammy vestigial root skipped cleanly, symlinks LINK'd.readlink -f ~/.claude/skills/start_dev→ resolves intoPyAutoBrain/skills/.bash PyAutoBrain/bin/check_skill_line_counts.sh→ exit 0, "OK: all 27 primary skill files within 200 lines".Companion PRs
This is the merge-first PR of the set. Merge before:
admin_jammy— removes the old copies + repoints its README.PyAutoMind— repoints doc references.🤖 Generated with Claude Code