docs(skills): one common copy of the skill packs; project keeps blender-director #38

Merged
zwitschi merged 1 commits from skills-layout-0.1.6 into main 2026-11-10 08:37:59 +01:00
Owner

Fixes the startup warnings: 94 skill name collisions, plus blender-director never loading.

Cause

  • The 94 symlinks in ~/.agents/skills/<pack> (added 2026-11-06) and a full second copy in ~/.pi/agent/skills/blender-skills/ were the same packs under two roots. Pi keeps the first discovery and warns on every start; the loser is invisible.
  • blender-director/SKILL.md declared metadata.role twice (specialist, then orchestrator). Duplicate YAML keys are fatal, so the pack was dropped silently while the other 94 loaded.

Layout now

Where What
~/.pi/agent/skills/ (common) 93 upstream packs, flat and one level down so they are discovered; shared references/; upstream LICENSE
.agents/skills/blender-skills/blender-director/ (repo) the project-specific orchestrator only

blender-director now references the shared notes by absolute path (~/.pi/agent/skills/references/…), since it no longer sits beside them. The 94 symlinks are removed.

Verified

Walked all three discovery roots (~/.pi/agent/skills, ~/.agents/skills, project .agents/skills, dot-dirs skipped) and parsed every frontmatter: 97 skills loaded, 0 malformed, 0 collisions.

AGENTS.md and Docs/toolchain.md describe the split and warn against copying the common packs back into the repository.

Fixes the startup warnings: 94 skill name collisions, plus `blender-director` never loading. ### Cause - The 94 symlinks in `~/.agents/skills/<pack>` (added 2026-11-06) and a full second copy in `~/.pi/agent/skills/blender-skills/` were **the same packs under two roots**. Pi keeps the first discovery and warns on every start; the loser is invisible. - `blender-director/SKILL.md` declared `metadata.role` **twice** (`specialist`, then `orchestrator`). Duplicate YAML keys are fatal, so the pack was dropped silently while the other 94 loaded. ### Layout now | Where | What | | --- | --- | | `~/.pi/agent/skills/` (common) | 93 upstream packs, flat and one level down so they are discovered; shared `references/`; upstream `LICENSE` | | `.agents/skills/blender-skills/blender-director/` (repo) | the project-specific orchestrator only | `blender-director` now references the shared notes by absolute path (`~/.pi/agent/skills/references/…`), since it no longer sits beside them. The 94 symlinks are removed. ### Verified Walked all three discovery roots (`~/.pi/agent/skills`, `~/.agents/skills`, project `.agents/skills`, dot-dirs skipped) and parsed every frontmatter: **97 skills loaded, 0 malformed, 0 collisions**. `AGENTS.md` and `Docs/toolchain.md` describe the split and warn against copying the common packs back into the repository.
zwitschi added 1 commit 2026-11-10 08:37:50 +01:00
The 94 symlinks in ~/.agents/skills and the full second copy in
~/.pi/agent/skills/blender-skills were the same packs under two roots, so Pi
reported a collision for every one of them on every start and silently dropped
the loser.

- The 93 upstream packs, the shared references/ notes and the LICENSE now live
  once, flat and one level down, in the common agent folder ~/.pi/agent/skills/.
- The repository keeps only the project-specific pack:
  .agents/skills/blender-skills/blender-director/. Its ../references/ links now
  point at ~/.pi/agent/skills/references/ because it no longer sits beside them.
  Its SKILL.md declared metadata.role twice (specialist, orchestrator), which is
  fatal YAML - the pack had never actually loaded.
- AGENTS.md and Docs/toolchain.md describe the split and warn against copying the
  common packs back into the repository.

Verified by walking all three discovery roots and parsing every frontmatter:
97 skills loaded, 0 malformed, 0 collisions.
zwitschi force-pushed skills-layout-0.1.6 from 19ed4fefd4 to 963be11a70 2026-11-10 08:37:50 +01:00 Compare
zwitschi merged commit 266fc1bb0c into main 2026-11-10 08:37:59 +01:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: zwitschi/gig-line#38