fix: preserve local file links

Author: burugo

plugins/codex/scripts/sync.py

@@ -425,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(strip_local_markdown_link_targets(text)).strip()
+    value = redact_text(file_url_local_markdown_link_targets(text)).strip()
     if value:
         blocks.append({"type": "text", "text": value})
 
@@ -558,12 +558,12 @@ def should_skip_user_message(content: Any) -> bool:
     return any(text.startswith(prefix) for prefix in skipped_prefixes)
 
 
-def strip_local_markdown_link_targets(text: str) -> str:
+def file_url_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 f"[{label}]({Path(target).as_uri()})"
         return match.group(0)
 
     return LOCAL_MARKDOWN_LINK_RE.sub(replace, text)

plugins/codex/tests/test_plugin_scripts.py

@@ -322,7 +322,7 @@ def test_build_payload_skips_loaded_skill_content_block(self):
         self.assertNotIn("Spec-Driven Planning", raw_payload)
         self.assertNotIn("SKILL.md", raw_payload)
 
-    def test_build_payload_strips_local_markdown_link_targets(self):
+    def test_build_payload_rewrites_local_markdown_link_targets_as_file_urls(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)"
@@ -351,8 +351,10 @@ def test_build_payload_strips_local_markdown_link_targets(self):
             )
 
         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))
+        self.assertEqual(
+            content,
+            "use [$claude-code-setup:spec-driven-planning](file:///Users/buru/Library/Mobile%20Documents/com~apple~CloudDocs/dotfiles/config/claude/skills/spec-driven-planning/SKILL.md)",
+        )
 
     def test_build_payload_includes_raw_repo_url_from_git_remote(self):
         from sync import build_payload_from_hook