mirror of
https://github.com/EveryInc/compound-engineering-plugin.git
synced 2026-09-19 01:09:55 +08:00
cbe51b6860
Reverts the ce-update removal from547a41e9. ce-update is a current skill, but Codex now installs it at the namespaced path ~/.codex/skills/compound-engineering/ce-update. The pre-namespaced flat path ~/.codex/skills/ce-update is legitimately legacy and must remain in EXTRA_LEGACY_ARTIFACTS_BY_PLUGIN["compound-engineering"].skills so cleanupKnownLegacyCodexArtifacts() sweeps it on upgrade. The invariant test in547a41e9had a false premise: it asserted no name in EXTRA_LEGACY_ARTIFACTS.skills may match a current skill. That conflates two hazards. STALE_SKILL_DIRS driving a fingerprint-based delete of a just-installed skill IS a real invariant. EXTRA_LEGACY_ARTIFACTS.skills driving flat-path cleanup is NOT — current skills whose install path moved (namespacing, renaming) legitimately appear in that list. Narrow the invariant test to STALE_SKILL_DIRS only and document the distinction. Caught by @chatgpt-codex-connector in PR review. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>