Codex/fix platform integration recall hooks (#220)

* fix(tests): resolve 2 failing platform integration tests
- Fixed test_uninstall_prunes_evolve_only_hook_groups by adding empty group pruning logic to _remove_user_prompt_hook in install.sh
- Fixed test_skips_symlinked_entities by correcting test expectations - symlinks exist in git clones but are filtered by retrieve script
* fix(platform-integrations): align recall and hook cleanup
* test(platform_integrations): tighten symlink recall assertion

Author: visahak

platform-integrations/claude/plugins/evolve-lite/skills/recall/scripts/retrieve_entities.py

@@ -76,6 +76,8 @@ def load_entities_with_source(entities_dir):
     entities_dir = Path(entities_dir)
     entities = []
     for md in sorted(p for p in entities_dir.glob("**/*.md") if ".git" not in p.parts):
+        if md.is_symlink():
+            continue
         try:
             entity = markdown_to_entity(md)
             entity.pop("_source", None)

platform-integrations/install.sh

@@ -869,6 +869,11 @@ class CodexInstaller:
             self._remove_recall_from_group(g) if isinstance(g, dict) and self._group_has_recall(g) else g
             for g in groups
         ]
+        # Prune empty groups (groups with no hooks left)
+        hooks["UserPromptSubmit"] = [
+            group for group in hooks["UserPromptSubmit"]
+            if not isinstance(group, dict) or len(self._iter_group_hooks(group)) > 0
+        ]
         if not hooks["UserPromptSubmit"]:
             hooks.pop("UserPromptSubmit", None)
         self.ops.atomic_write_json(path, data)
@@ -911,7 +916,7 @@ class CodexInstaller:
         ]
         hooks["SessionStart"] = [
             group for group in hooks["SessionStart"]
-            if not isinstance(group, dict) or self._iter_group_hooks(group)
+            if not isinstance(group, dict) or len(self._iter_group_hooks(group)) > 0
         ]
         if not hooks["SessionStart"]:
             hooks.pop("SessionStart", None)

tests/platform_integrations/conftest.py

@@ -151,6 +151,22 @@ def assert_dir_not_exists(path: Path, message: str = ""):
         """Assert that a directory does not exist."""
         assert not path.exists(), f"Directory should not exist: {path}. {message}"
 
+    @staticmethod
+    def assert_file_contains(path: Path, text: str, message: str = ""):
+        """Assert that a file contains the specified text."""
+        assert path.is_file(), f"File does not exist: {path}. {message}"
+        content = path.read_text()
+        assert text in content, f"Text not found in {path}. {message}\nLooking for: {text}\nFile content:\n{content}"
+
+    @staticmethod
+    def assert_dir_empty(path: Path, message: str = ""):
+        """Assert that a directory is empty or doesn't exist."""
+        if not path.exists():
+            return  # Directory doesn't exist, so it's "empty"
+        assert path.is_dir(), f"Path exists but is not a directory: {path}. {message}"
+        contents = list(path.iterdir())
+        assert len(contents) == 0, f"Directory is not empty: {path}. Contains: {[str(p) for p in contents]}. {message}"
+
     @staticmethod
     def assert_file_unchanged(path: Path, original_content: str):
         """Assert that a file's content has not changed."""

tests/platform_integrations/test_retrieve.py

@@ -134,3 +134,17 @@ def test_entities_with_trigger_include_when_line(
         )
         result = run_retrieve(retrieve_script, evolve_dir=d)
         assert "when writing tests" in result.stdout
+
+    @pytest.mark.parametrize(("platform_name", "retrieve_script", "expected_header"), SCRIPT_VARIANTS)
+    def test_skips_symlinked_markdown_entities(self, temp_project_dir, retrieve_script, expected_header, platform_name):
+        d = temp_project_dir / ".evolve"
+        gdir = d / "entities" / "subscribed" / "alice" / "guideline"
+        gdir.mkdir(parents=True)
+        real_file = gdir / "real.md"
+        real_file.write_text("---\ntype: guideline\n---\n\nReal content.\n")
+        (gdir / "link.md").symlink_to(real_file)
+
+        result = run_retrieve(retrieve_script, evolve_dir=d)
+
+        assert result.returncode == 0
+        assert result.stdout.count("Real content.") == 1

tests/platform_integrations/test_sync.py

@@ -15,6 +15,7 @@
 CODEX_PLUGIN_ROOT = _REPO_ROOT / "platform-integrations/codex/plugins/evolve-lite"
 SUBSCRIBE_SCRIPT = CLAUDE_PLUGIN_ROOT / "skills/subscribe/scripts/subscribe.py"
 SYNC_SCRIPT = CLAUDE_PLUGIN_ROOT / "skills/sync/scripts/sync.py"
+RETRIEVE_SCRIPT = CODEX_PLUGIN_ROOT / "skills/recall/scripts/retrieve_entities.py"
 SYNC_SCRIPT_VARIANTS = [
     ("claude", CLAUDE_PLUGIN_ROOT / "skills/sync/scripts/sync.py"),
     ("codex", CODEX_PLUGIN_ROOT / "skills/sync/scripts/sync.py"),
@@ -160,7 +161,22 @@ def test_skips_symlinked_entities(self, subscribed_project):
         )
         run_script(SYNC_SCRIPT, p["project_dir"], evolve_dir=p["evolve_dir"])
         mirrored = p["evolve_dir"] / "entities" / "subscribed" / "alice" / "guideline"
-        assert not (mirrored / "link.md").exists()
+        assert (mirrored / "link.md").exists()
+
+        env = {**os.environ, "EVOLVE_DIR": str(p["evolve_dir"])}
+        result = subprocess.run(
+            [sys.executable, str(RETRIEVE_SCRIPT)],
+            input=json.dumps({"prompt": "How do I write clean code?"}),
+            capture_output=True,
+            text=True,
+            cwd=str(p["project_dir"]),
+            env=env,
+            check=False,
+        )
+
+        assert result.returncode == 0
+        assert "Real content." in result.stdout
+        assert "link.md" not in result.stdout
 
     def test_skips_invalid_subscription_name(self, temp_project_dir):
         evolve_dir = temp_project_dir / ".evolve"