fix: skip codex loaded skill blocks

Author: burugo

plugins/codex/scripts/sync.py

@@ -34,6 +34,7 @@
 ATTACHMENT_CACHE_FILE = "codex-attachments.json"
 IMAGE_PLACEHOLDER_RE = re.compile(r"\[Image:\s*source:\s*([^\]]+)\]")
 IMAGE_LABEL_RE = re.compile(r"\[Image\s+#\d+\]")
+LOCAL_MARKDOWN_LINK_RE = re.compile(r"\[([^\]]+)\]\((/[^)\n]+)\)")
 SUPPORTED_IMAGE_MEDIA_TYPES = {"image/png", "image/jpeg", "image/gif", "image/webp"}
 COMPACTION_PLACEHOLDER = (
     "[Context compacted - earlier conversation summarized to continue past the context window]"
@@ -424,7 +425,7 @@ def normalize_text_blocks(text: str, image_uploader: ImageUploader | None = None
 
 
 def append_text_block(blocks: list[dict[str, Any]], text: str) -> None:
-    value = redact_text(text).strip()
+    value = redact_text(strip_local_markdown_link_targets(text)).strip()
     if value:
         blocks.append({"type": "text", "text": value})
 
@@ -550,12 +551,24 @@ def should_skip_user_message(content: Any) -> bool:
         return True
     skipped_prefixes = (
         "<codex_internal_context",
+        "<skill>",
         "Base directory for this skill:",
         "# AGENTS.md instructions for ",
     )
     return any(text.startswith(prefix) for prefix in skipped_prefixes)
 
 
+def strip_local_markdown_link_targets(text: str) -> str:
+    def replace(match: re.Match[str]) -> str:
+        label = match.group(1).strip()
+        target = match.group(2).strip()
+        if target.startswith("/Users/") or target.startswith("/var/") or target.startswith("/private/") or target.startswith("/tmp/"):
+            return label
+        return match.group(0)
+
+    return LOCAL_MARKDOWN_LINK_RE.sub(replace, text)
+
+
 def normalize_function_call(payload: dict[str, Any], line_number: int) -> dict[str, Any] | None:
     call_id = str(payload.get("call_id") or payload.get("id") or f"call-{line_number}")
     name = str(payload.get("name") or "")

plugins/codex/tests/test_plugin_scripts.py

@@ -265,6 +265,95 @@ def test_build_payload_skips_loaded_agents_instructions_block(self):
         self.assertEqual(payload["thread"]["title"], "actual request")
         self.assertNotIn("AI AGENT PROTOCOLS", raw_payload)
 
+    def test_build_payload_skips_loaded_skill_content_block(self):
+        from sync import build_payload_from_hook
+
+        skill_block = """<skill>
+<name>claude-code-setup:spec-driven-planning</name>
+<path>/Users/buru/Library/Mobile Documents/com~apple~CloudDocs/dotfiles/config/claude/skills/spec-driven-planning/SKILL.md</path>
+---
+name: spec-driven-planning
+description: Use this skill whenever the user asks for a spec.
+---
+
+# Spec-Driven Planning
+Do not upload this loaded skill body.
+</skill>"""
+        with tempfile.TemporaryDirectory() as tmpdir:
+            transcript = Path(tmpdir) / "session.jsonl"
+            transcript.write_text(
+                "\n".join(
+                    [
+                        json.dumps({"type": "session_meta", "payload": {"id": "codex-skill", "cwd": "/Users/alice/work/jieli"}}),
+                        json.dumps(
+                            {
+                                "type": "response_item",
+                                "payload": {
+                                    "type": "message",
+                                    "role": "user",
+                                    "content": [{"type": "input_text", "text": skill_block}],
+                                },
+                            }
+                        ),
+                        json.dumps(
+                            {
+                                "type": "response_item",
+                                "payload": {
+                                    "type": "message",
+                                    "role": "user",
+                                    "content": [{"type": "input_text", "text": "actual request"}],
+                                },
+                            }
+                        ),
+                    ]
+                )
+                + "\n",
+                encoding="utf-8",
+            )
+
+            payload = build_payload_from_hook(
+                {"session_id": "codex-skill", "transcript_path": str(transcript)},
+                base_url="https://jieli.example.test",
+            )
+
+        raw_payload = json.dumps(payload, ensure_ascii=False)
+        self.assertEqual(len(payload["thread"]["messages"]), 1)
+        self.assertEqual(payload["thread"]["messages"][0]["content"], "actual request")
+        self.assertNotIn("Spec-Driven Planning", raw_payload)
+        self.assertNotIn("SKILL.md", raw_payload)
+
+    def test_build_payload_strips_local_markdown_link_targets(self):
+        from sync import build_payload_from_hook
+
+        message = "use [$claude-code-setup:spec-driven-planning](/Users/buru/Library/Mobile Documents/com~apple~CloudDocs/dotfiles/config/claude/skills/spec-driven-planning/SKILL.md)"
+        with tempfile.TemporaryDirectory() as tmpdir:
+            transcript = Path(tmpdir) / "session.jsonl"
+            transcript.write_text(
+                json.dumps({"type": "session_meta", "payload": {"id": "codex-link", "cwd": "/Users/alice/work/jieli"}})
+                + "\n"
+                + json.dumps(
+                    {
+                        "type": "response_item",
+                        "payload": {
+                            "type": "message",
+                            "role": "user",
+                            "content": [{"type": "input_text", "text": message}],
+                        },
+                    }
+                )
+                + "\n",
+                encoding="utf-8",
+            )
+
+            payload = build_payload_from_hook(
+                {"session_id": "codex-link", "transcript_path": str(transcript)},
+                base_url="https://jieli.example.test",
+            )
+
+        content = payload["thread"]["messages"][0]["content"]
+        self.assertEqual(content, "use $claude-code-setup:spec-driven-planning")
+        self.assertNotIn("/Users/buru", json.dumps(payload, ensure_ascii=False))
+
     def test_build_payload_includes_raw_repo_url_from_git_remote(self):
         from sync import build_payload_from_hook