refactor(skills): rename migrate → purchasely-migrate

[kherembourg]

Renames the migrate skill to purchasely-migrate, consistent with purchasely-integrate / purchasely-review / purchasely-debug (disambiguation for the skills CLI).

• skills/migrate/ → skills/purchasely-migrate/ (git mv, name: purchasely-migrate).
• commands/migrate.md keeps /purchasely:migrate and now follows the integrate command pattern (reads skills/purchasely-migrate/SKILL.md).
• Updated the build-gate guard path, the iOS/Android v5-api-reference.md prose, and CLAUDE.md (Found 3 skills → Found 4 skills).
• CHANGELOG [Unreleased] entry added.

Validated: npm test (guard) passes, skills add . --list reports the 4 purchasely-* skills, markdownlint (excl. MD060) clean, all skill reference links resolve.

🤖 Generated with Claude Code

[greptile-apps]

Greptile Summary

Renames the migrate skill to purchasely-migrate for naming consistency with the other three purchasely-* skills, updates the slash command to follow the integrate command pattern, and tightens the build-gate guard path accordingly.

• skills/migrate/SKILL.md moved to skills/purchasely-migrate/SKILL.md with the frontmatter name updated; the guard script, both v5 API reference files, and commands/migrate.md are all updated in sync.
• CLAUDE.md's skills.sh validation comment is updated to "Found 4 skills", but the agentskill.sh publishing section still says "three skills" in two places, has a JSON example capped at 3 entries, and the public-pages checklist omits the new @purchasely/purchasely-migrate URL.
• CHANGELOG entry correctly documents the rename and provides a consumer migration path (npx skills update).

Confidence Score: 4/5

All functional changes — the skill rename, the guard path, and the command body — are correct and internally consistent. The only gap is a set of stale prose references in CLAUDE.md's agentskill.sh section that still describe three skills where there are now four.

The rename is cleanly executed across every file that matters for runtime behaviour. The sole gap is documentation: the agentskill.sh publishing section of CLAUDE.md counts three skills in two prose sentences, in the JSON sample response, and in the public-pages checklist. A developer following that section to verify or re-trigger a publish will see a misleading count, but this has no effect on the skills themselves or the build gate.

CLAUDE.md — the agentskill.sh publishing section (lines 33–78) still reflects the old three-skill inventory.

Important Files Changed

Filename	Overview
CHANGELOG.md	Adds a [Unreleased] Changed entry documenting the migrate → purchasely-migrate rename with a migration note for consumers; correct and complete.
CLAUDE.md	skills.sh validation line updated to 'Found 4 skills' but the agentskill.sh section still says 'three skills' in two places, the JSON example response shows only 3 entries with found:3, and the public-pages checklist omits the new purchasely-migrate URL.
purchasely/commands/migrate.md	Refactored to follow the integrate command pattern; now correctly reads skills/purchasely-migrate/SKILL.md; matches integrate.md exactly.
purchasely/skills/purchasely-migrate/SKILL.md	Renamed from skills/migrate/SKILL.md; only the frontmatter name field changed (migrate → purchasely-migrate); all skill content preserved.
scripts/guard-known-bad-snippets.mjs	Build-gate path updated to purchasely/skills/purchasely-migrate/SKILL.md; guard logic unchanged and correct.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["/purchasely:migrate command"] --> B["commands/migrate.md"]
    B --> C["Read skills/purchasely-migrate/SKILL.md"]
    C --> D["purchasely-migrate skill executes migration steps"]

    E["guard-known-bad-snippets.mjs"] --> F{"All 4 SKILL.md paths present?"}
    F -->|Yes| G["npm test passes ✓"]
    F -->|No| H["Build gate fails ✗"]

    I["skills CLI / agentskill.sh"] --> J["Discovers purchasely/skills/**/SKILL.md"]
    J --> K["purchasely-debug\npurchasely-integrate\npurchasely-migrate\npurchasely-review"]

    style D fill:#d4edda
    style G fill:#d4edda
    style H fill:#f8d7da

Loading

Comments Outside Diff (1)

1. CLAUDE.md, line 33-88 (link)

   Stale "three skills" references in the agentskill.sh / skills.sh publishing sections

   The PR correctly updated the skills.sh validation line (now "Found 4 skills"), but the surrounding agentskill.sh and skills.sh prose was not updated. Line 33 still reads "The three skills are published on agentskill.sh", line 82 reads "The same three skills are also installable through the skills CLI", the JSON response example (lines 56-62) shows only three slug entries with "found": 3, and the public-pages checklist (lines 75-78) is missing https://agentskill.sh/@purchasely/purchasely-migrate. Anyone following this guide for a re-scan or to verify the published skill list will see misleading counts.

   Prompt To Fix With AI

   This is a comment left during a code review.
   Path: CLAUDE.md
   Line: 33-88
   
   Comment:
   **Stale "three skills" references in the agentskill.sh / skills.sh publishing sections**
   
   The PR correctly updated the `skills.sh` validation line (now "Found 4 skills"), but the surrounding `agentskill.sh` and `skills.sh` prose was not updated. Line 33 still reads "The three skills are published on agentskill.sh", line 82 reads "The same three skills are also installable through the `skills` CLI", the JSON response example (lines 56-62) shows only three `slug` entries with `"found": 3`, and the public-pages checklist (lines 75-78) is missing `https://agentskill.sh/@purchasely/purchasely-migrate`. Anyone following this guide for a re-scan or to verify the published skill list will see misleading counts.
   
   How can I resolve this? If you propose a fix, please make it concise.

   Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

   

Prompt To Fix All With AI

Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
CLAUDE.md:33-88
**Stale "three skills" references in the agentskill.sh / skills.sh publishing sections**

The PR correctly updated the `skills.sh` validation line (now "Found 4 skills"), but the surrounding `agentskill.sh` and `skills.sh` prose was not updated. Line 33 still reads "The three skills are published on agentskill.sh", line 82 reads "The same three skills are also installable through the `skills` CLI", the JSON response example (lines 56-62) shows only three `slug` entries with `"found": 3`, and the public-pages checklist (lines 75-78) is missing `https://agentskill.sh/@purchasely/purchasely-migrate`. Anyone following this guide for a re-scan or to verify the published skill list will see misleading counts.

_{Reviews (1): Last reviewed commit: "refactor(skills): rename migrate skill t..." | Re-trigger Greptile}