Skip to content

fix: import skills to .copilot/skills/ instead of .ai-team/skills/#849

Merged
bradygaster merged 2 commits intodevfrom
squad/fix-skills-import-path
Apr 11, 2026
Merged

fix: import skills to .copilot/skills/ instead of .ai-team/skills/#849
bradygaster merged 2 commits intodevfrom
squad/fix-skills-import-path

Conversation

@bradygaster
Copy link
Copy Markdown
Owner

Skills import was writing to .ai-team/skills/ but the canonical location is .copilot/skills/. This fixes the last 6 test failures blocking the insider release.

Skills canonical location is .copilot/skills/, not .ai-team/skills/.
Import was writing to the wrong path.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 5, 2026 09:12
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 5, 2026

🟢 Impact Analysis — PR #849

Risk tier: 🟢 LOW

📊 Summary

Metric Count
Files changed 1
Files added 0
Files modified 1
Files deleted 0
Modules touched 1

🎯 Risk Factors

  • 1 files changed (≤5 → LOW)
  • 1 module(s) touched (≤1 → LOW)

📦 Modules Affected

root (1 file)
  • index.cjs

This report is generated automatically for every PR. See #733 for details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 5, 2026

🛫 PR Readiness Check

ℹ️ This comment updates on each push. Last checked: commit 878277e

⚠️ 2 item(s) to address before review

Status Check Details
Single commit 1 commit — clean history
Not in draft Ready for review
Branch up to date Up to date with dev
Copilot review No Copilot review yet — it may still be processing
Changeset present No source files changed — changeset not required
Scope clean No .squad/ or docs/proposals/ files
No merge conflicts No merge conflicts
Copilot threads resolved No Copilot review threads
CI passing 12 check(s) still running

This check runs automatically on every push. Fix any ❌ items and push again.
See CONTRIBUTING.md and PR Requirements for details.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes the squad import command to write imported skills into the canonical Copilot skills location (.copilot/skills/) rather than the legacy .ai-team/skills/, aligning import behavior with existing export behavior and unblocking failing tests.

Changes:

  • Update import-time skills destination to .copilot/skills/<skill>/SKILL.md.
  • Add a clarifying comment indicating .copilot/skills/ is the canonical location.

Comment thread index.cjs Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@bradygaster bradygaster merged commit cf8ffdc into dev Apr 11, 2026
tamirdresher pushed a commit that referenced this pull request Apr 21, 2026
)

* fix: import skills to .copilot/skills/ instead of .ai-team/skills/

Skills canonical location is .copilot/skills/, not .ai-team/skills/.
Import was writing to the wrong path.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Apply suggestion from @Copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants