bug: squad.agent.md SKILL EXTRACTION still directs to legacy .squad/skills/ path (follow-up to #354)

[diberry]

Bug: squad.agent.md SKILL EXTRACTION still directs agents to .squad/skills/ (legacy path)

Related Issues

• PRD: Migrate Squad Skills from .squad/skills/ to .copilot/skills/ #354 (closed) — PRD: Migrate Squad Skills from .squad/skills/ to .copilot/skills/
• feat: skills should be created in .copilot/skills/ and prompt user to reload #353 (closed) — feat: skills should be created in .copilot/skills/
• fix: import skills to .copilot/skills/ instead of .ai-team/skills/ #849 (merged) — fix: import skills to .copilot/skills/ instead of .ai-team/skills/

Problem

The SDK/CLI migration to .copilot/skills/ is complete — skill-source.ts, init.ts, tools/index.ts, templates, and tests all use .copilot/skills/ as the primary path. However, squad.agent.md line 825 still instructs agents to write skills to the legacy path:

SKILL EXTRACTION: If you found a reusable pattern, write/update
    .squad/skills/{skill-name}/SKILL.md (read templates/skill.md for format).

This means every agent session that extracts a skill writes it to .squad/skills/ where:

• The SDK's write_skill tool won't find it (writes to .copilot/skills/)
• squad init and squad upgrade won't sync it (targets .copilot/skills/)
• Copilot's native skill discovery won't see it
• The skill-source.ts fallback will read it, but only because of legacy compat code

Additional locations in squad.agent.md with the same issue

The spawn template also tells agents to check .squad/skills/ for existing skills:

If .squad/skills/ has relevant SKILL.md files, read them before working.

And the Skill-aware routing section:

Skill-aware routing: Before spawning, check .squad/skills/ for skills relevant...

And the Plugin Marketplace section:

Install: copy to .squad/skills/{plugin-name}/SKILL.md

Also affected: collections.ts state schema

packages/squad-sdk/src/state/collections.ts line 251 still resolves skills to .squad/skills:

const skillsDir = `${this.rootDir}/.squad/skills`;

Fix

1. squad.agent.md: Change all .squad/skills/ references to .copilot/skills/ in:
   • SKILL EXTRACTION instruction (line 825)
   • Spawn template skill-read instruction
   • Skill-aware routing section
   • Plugin Marketplace install path
2. collections.ts: Update skillsDir to resolve to .copilot/skills
3. Keep skill-source.ts fallback to .squad/skills/ for backward compat (already correct)

Impact

Every squad session that extracts a skill creates it in the wrong directory. The skill works due to legacy fallback but diverges from the SDK's canonical path, causing confusion about where skills live.

[sytone]

See #798

[diberry]

✈️ Flight Triage

Category: Bug — quick fix, no PRD needed.

Assessment: Well-scoped bug with clear fix list (4 locations in squad.agent.md + 1 in collections.ts). The issue body already serves as a complete implementation spec. All .squad/skills/ references need updating to .copilot/skills/.

Priority: P1 — every agent session that extracts a skill writes to the wrong directory.
Effort: ~15 minutes.
Assignment: squad:procedures (template/agent instructions) + squad:eecom (collections.ts)

Action: Implement directly — no design work needed.