codex-coach 0.1.4 → 0.1.5

package/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "codex-coach",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "Local-first Codex usage coach that analyzes logs and suggests workflow improvements.",
   "author": {
     "name": "Codex Coach Contributors"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codex-coach",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "Local-first Codex usage coach and plugin",
   "license": "MIT",
   "author": "Codex Coach Contributors",

package/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "codex-coach"
-version = "0.1.4"
+version = "0.1.5"
 description = "Local-first Codex usage coach and plugin"
 readme = "README.md"
 requires-python = ">=3.10"

package/src/codex_coach/__init__.py

@@ -1,3 +1,3 @@
 """Local-first Codex usage coach."""
 
-__version__ = "0.1.4"
+__version__ = "0.1.5"

package/src/codex_coach/cli.py

@@ -166,8 +166,11 @@ def _doctor(paths) -> int:
     print(f"command: {command} {'OK' if command.exists() else 'not installed'}")
     plugin = paths.home / "plugins" / "codex-coach" / ".codex-plugin" / "plugin.json"
     print(f"plugin: {plugin} {'OK' if plugin.exists() else 'not installed'}")
-    skill = paths.home / ".agents" / "skills" / "codex-coach" / "SKILL.md"
+    skill = paths.codex_home / "skills" / "codex-coach" / "SKILL.md"
+    legacy_skill = paths.home / ".agents" / "skills" / "codex-coach" / "SKILL.md"
     print(f"skill: {skill} {'OK' if skill.exists() else 'not installed'}")
+    if legacy_skill.exists():
+        print(f"legacy_skill: {legacy_skill} duplicate")
     return 0 if paths.codex_home.exists() else 1
 
 

package/src/codex_coach/install.py

@@ -19,7 +19,7 @@ def install_from_source(source_root: Path, paths: CoachPaths, *, schedule: str =
         _copy_app(source_root, paths.app_dir)
     _install_command(paths)
     plugin_path = _install_plugin(source_root, paths.home)
-    skill_paths = _install_user_skills(source_root, paths.home)
+    skill_paths = _install_user_skills(source_root, paths)
     marketplace = _update_marketplace(paths.home, plugin_path)
     scheduler = _write_scheduler(paths, schedule=schedule)
     return {
@@ -90,20 +90,28 @@ def _install_plugin(source_root: Path, home: Path) -> Path:
     return plugin_root
 
 
-def _install_user_skills(source_root: Path, home: Path) -> list[Path]:
+def _install_user_skills(source_root: Path, paths: CoachPaths) -> list[Path]:
     skill_source = source_root / "skills" / PLUGIN_NAME
-    targets = [
-        home / ".agents" / "skills" / PLUGIN_NAME,
-        home / ".codex" / "skills" / PLUGIN_NAME,
-    ]
-    installed: list[Path] = []
-    for target in targets:
-        if target.exists():
-            shutil.rmtree(target)
-        target.parent.mkdir(parents=True, exist_ok=True)
-        shutil.copytree(skill_source, target)
-        installed.append(target)
-    return installed
+    target = paths.codex_home / "skills" / PLUGIN_NAME
+    if target.exists():
+        shutil.rmtree(target)
+    target.parent.mkdir(parents=True, exist_ok=True)
+    shutil.copytree(skill_source, target)
+    _remove_legacy_duplicate_skill(paths.home, target)
+    return [target]
+
+
+def _remove_legacy_duplicate_skill(home: Path, canonical_target: Path) -> None:
+    legacy_target = home / ".agents" / "skills" / PLUGIN_NAME
+    if legacy_target.resolve() == canonical_target.resolve() or not legacy_target.exists():
+        return
+    skill_file = legacy_target / "SKILL.md"
+    try:
+        skill_text = skill_file.read_text(encoding="utf-8")
+    except OSError:
+        return
+    if f"name: {PLUGIN_NAME}" in skill_text:
+        shutil.rmtree(legacy_target)
 
 
 def _update_marketplace(home: Path, plugin_path: Path) -> Path: