uncoded 1.1.0__tar.gz → 1.2.0__tar.gz

{uncoded-1.1.0 → uncoded-1.2.0}/.uncoded/namespace.yaml

@@ -55,14 +55,16 @@ src/:
       iter_source_files:
       extract_modules:
     instruction_files.py:
-      MARKER_START:
       MARKER_END:
-      MARKER_DOCS_START:
       MARKER_DOCS_END:
+      MARKER_START_PREFIX:
+      MARKER_DOCS_START_PREFIX:
       DEFAULT_INSTRUCTION_FILES:
       _CODE_SECTION_BODY:
+      MARKER_START:
       SECTION_CODE:
       _DOCS_SECTION_BODY:
+      MARKER_DOCS_START:
       SECTION_DOCS:
       _apply_section:
       sync_instruction_file:
@@ -276,6 +278,9 @@ tests/:
       test_check_returns_one_when_docs_yaml_stale:
       test_check_returns_one_when_stubs_should_be_removed:
       test_check_returns_one_when_docs_yaml_should_be_removed:
+      test_doc_only_does_not_write_skill:
+      test_doc_only_removes_preexisting_skill:
+      test_check_returns_one_when_skill_should_be_removed:
       test_idempotent_doc_only:
     _init_repo:
     _init_doc_repo:
@@ -415,6 +420,12 @@ tests/:
     TestSyncInstructionFileProjectRootAnchor:
       test_project_root_anchors_create_independent_of_cwd:
       test_project_root_anchors_update_of_existing_file:
+    TestSyncInstructionFileFingerprint:
+      test_reflowed_body_survives_sync:
+      test_reflowed_body_passes_check:
+      test_different_fingerprint_refreshes_section:
+      test_plain_marker_refreshes_once_then_stable:
+      test_prose_mention_of_prefix_before_section_is_ignored:
   test_namespace_map.py:
     TestBuildMap:
       test_single_file:

{uncoded-1.1.0 → uncoded-1.2.0}/.uncoded/stubs/src/uncoded/instruction_files.pyi

@@ -1,20 +1,24 @@
 # src/uncoded/instruction_files.py
 
+import hashlib
+import re
 from importlib.resources import files
 from pathlib import Path
 from uncoded.sync import sync_file
 
-MARKER_START = '<!-- uncoded:start -->'
 MARKER_END = '<!-- uncoded:end -->'
-MARKER_DOCS_START = '<!-- uncoded:docs:start -->'
 MARKER_DOCS_END = '<!-- uncoded:docs:end -->'
+MARKER_START_PREFIX = '<!-- uncoded:start'
+MARKER_DOCS_START_PREFIX = '<!-- uncoded:docs:start'
 DEFAULT_INSTRUCTION_FILES = [Path('CLAUDE.md'), Path('AGENTS.md')]
 _CODE_SECTION_BODY = (files('uncoded') / 'dispatch_rule.md').read_text(encoding='utf-8').rstrip('\n')
+MARKER_START = ...
 SECTION_CODE = f'{MARKER_START}\n{_CODE_SECTION_BODY}\n{MARKER_END}\n'
 _DOCS_SECTION_BODY = (files('uncoded') / 'docs_rule.md').read_text(encoding='utf-8').rstrip('\n')
+MARKER_DOCS_START = ...
 SECTION_DOCS = f'{MARKER_DOCS_START}\n{_DOCS_SECTION_BODY}\n{MARKER_DOCS_END}\n'
 
-def _apply_section(text: str, start: str, end: str, body: str | None) -> str:
+def _apply_section(text: str, start: str, end: str, body: str | None, *, prefix: str) -> str:
     ...
 
 def sync_instruction_file(path: Path, *, code_section: str | None, docs_section: str | None, project_root: Path, check: bool) -> bool:

{uncoded-1.1.0 → uncoded-1.2.0}/.uncoded/stubs/src/uncoded/skill.pyi

@@ -8,5 +8,5 @@ SKILL_OUTPUTS = ...
 LEGACY_SKILL_OUTPUTS = ...
 _SKILL_CONTENT = (files('uncoded') / 'coherence_review.md').read_text(encoding='utf-8')
 
-def sync_skill(*, project_root: Path, check: bool) -> bool:
+def sync_skill(*, project_root: Path, check: bool, build: bool) -> bool:
     ...

{uncoded-1.1.0 → uncoded-1.2.0}/.uncoded/stubs/tests/test_cli.pyi

@@ -6,6 +6,7 @@ from pathlib import Path
 from unittest import mock
 import pytest
 from uncoded import cli
+from uncoded.instruction_files import MARKER_START
 from uncoded.skill import SKILL_OUTPUTS
 
 def _init_repo(tmp_path, monkeypatch, source_roots):
@@ -203,5 +204,14 @@ class TestSyncDocRoots:
     def test_check_returns_one_when_docs_yaml_should_be_removed(self, tmp_path, monkeypatch):
         ...
 
+    def test_doc_only_does_not_write_skill(self, tmp_path, monkeypatch):
+        ...
+
+    def test_doc_only_removes_preexisting_skill(self, tmp_path, monkeypatch):
+        ...
+
+    def test_check_returns_one_when_skill_should_be_removed(self, tmp_path, monkeypatch):
+        ...
+
     def test_idempotent_doc_only(self, tmp_path, monkeypatch):
         ...

{uncoded-1.1.0 → uncoded-1.2.0}/.uncoded/stubs/tests/test_instruction_files.pyi

@@ -91,3 +91,19 @@ class TestSyncInstructionFileProjectRootAnchor:
 
     def test_project_root_anchors_update_of_existing_file(self, tmp_path, monkeypatch):
         ...
+
+class TestSyncInstructionFileFingerprint:
+    def test_reflowed_body_survives_sync(self, tmp_path):
+        ...
+
+    def test_reflowed_body_passes_check(self, tmp_path):
+        ...
+
+    def test_different_fingerprint_refreshes_section(self, tmp_path):
+        ...
+
+    def test_plain_marker_refreshes_once_then_stable(self, tmp_path):
+        ...
+
+    def test_prose_mention_of_prefix_before_section_is_ignored(self, tmp_path):
+        ...

{uncoded-1.1.0 → uncoded-1.2.0}/AGENTS.md

@@ -64,7 +64,7 @@ uv run pytest
 uv run pytest tests/test_stubs.py --no-cov
 ```
 
-<!-- uncoded:start -->
+<!-- uncoded:start sha256=5d50217d -->
 ## How to read and edit code in this codebase
 
 This repo uses [uncoded](https://github.com/alimanfoo/uncoded) to maintain
@@ -171,7 +171,7 @@ Edit, and grep stay correct:
 The dispatch rule turns on the search term: a symbol name → the index; a
 regex or free-text phrase → grep.
 <!-- uncoded:end -->
-<!-- uncoded:docs:start -->
+<!-- uncoded:docs:start sha256=6a530a01 -->
 ## How to read documentation in this codebase
 
 `.uncoded/docs.yaml` is an orientation outline: it lists every Markdown file

{uncoded-1.1.0 → uncoded-1.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: uncoded
-Version: 1.1.0
+Version: 1.2.0
 Summary: Symbol index with associated code reading tools for AI coding agent navigation
 Project-URL: Homepage, https://github.com/alimanfoo/uncoded
 Project-URL: Repository, https://github.com/alimanfoo/uncoded

{uncoded-1.1.0 → uncoded-1.2.0}/src/uncoded/cli.py

@@ -117,6 +117,13 @@ def _sync(*, start: Path | None = None, check: bool = False) -> int:
             project_root=project_root,
             check=check,
         )
+    # The skill needs namespace.yaml and stubs to run, so it builds and
+    # removes with the other code artefacts.
+    changes += sync_skill(
+        project_root=project_root,
+        check=check,
+        build=bool(config.source_roots),
+    )
 
     # Doc artefacts — build when doc_roots configured, else remove.
     if config.doc_roots:
@@ -190,8 +197,6 @@ def _sync(*, start: Path | None = None, check: bool = False) -> int:
             check=check,
         )
 
-    changes += sync_skill(project_root=project_root, check=check)
-
     if check:
         if changes:
             print(f"Index out of date: {changes} file(s) would change.")

{uncoded-1.1.0 → uncoded-1.2.0}/src/uncoded/instruction_files.py

@@ -9,37 +9,63 @@ each), or symlink one to the other (sync dedupes by inode and writes once).
 This module owns delimited sections in any such file and keeps them in sync.
 """
 
+import hashlib
+import re
 from importlib.resources import files
 from pathlib import Path
 
 from uncoded.sync import sync_file
 
-MARKER_START = "<!-- uncoded:start -->"
 MARKER_END = "<!-- uncoded:end -->"
-MARKER_DOCS_START = "<!-- uncoded:docs:start -->"
 MARKER_DOCS_END = "<!-- uncoded:docs:end -->"
+MARKER_START_PREFIX = "<!-- uncoded:start"
+MARKER_DOCS_START_PREFIX = "<!-- uncoded:docs:start"
 
 DEFAULT_INSTRUCTION_FILES = [Path("CLAUDE.md"), Path("AGENTS.md")]
 
+# The opening markers carry a short sha256 stamp of uncoded's canonical
+# section body, computed once at module load. On sync, the on-disk
+# opening-marker line is compared to the canonical marker: a matching stamp
+# means this version's wording is already planted, so the body is left alone
+# and a formatter's reflow survives; a differing stamp means the wording
+# changed (e.g. on upgrade) and the whole section is replaced.
 _CODE_SECTION_BODY = (
     (files("uncoded") / "dispatch_rule.md").read_text(encoding="utf-8").rstrip("\n")
 )
+MARKER_START = (
+    f"{MARKER_START_PREFIX} sha256="
+    f"{hashlib.sha256(_CODE_SECTION_BODY.encode()).hexdigest()[:8]} -->"
+)
 SECTION_CODE = f"{MARKER_START}\n{_CODE_SECTION_BODY}\n{MARKER_END}\n"
 
 _DOCS_SECTION_BODY = (
     (files("uncoded") / "docs_rule.md").read_text(encoding="utf-8").rstrip("\n")
 )
+MARKER_DOCS_START = (
+    f"{MARKER_DOCS_START_PREFIX} sha256="
+    f"{hashlib.sha256(_DOCS_SECTION_BODY.encode()).hexdigest()[:8]} -->"
+)
 SECTION_DOCS = f"{MARKER_DOCS_START}\n{_DOCS_SECTION_BODY}\n{MARKER_DOCS_END}\n"
 
 
-def _apply_section(text: str, start: str, end: str, body: str | None) -> str:
+def _apply_section(
+    text: str, start: str, end: str, body: str | None, *, prefix: str
+) -> str:
     """Apply, replace, or remove the delimited section in text.
 
-    When body is a string: replaces the section if already present,
-    appends it otherwise. When body is None: removes the section if
-    present, leaves text unchanged otherwise.
+    Locates an existing section by prefix, anchored to the start of a line,
+    so marker-like text in prose is not mistaken for a section opener. This
+    matches both old plain markers and current fingerprinted ones.
+
+    When body is a string:
+    - absent → append the canonical section;
+    - found and opening-marker line matches start → return text unchanged
+      (a formatter's reflow of the body is tolerated);
+    - found and opening-marker line differs → replace the whole section.
+    When body is None: remove the section if present.
     """
-    s = text.find(start)
+    m = re.search(r"^" + re.escape(prefix), text, re.MULTILINE)
+    s = m.start() if m else -1
     e = text.find(end)
     section_found = s != -1 and e != -1 and s < e
 
@@ -51,13 +77,19 @@ def _apply_section(text: str, start: str, end: str, body: str | None) -> str:
         return before + after
     else:
         if section_found:
+            line_end = text.find("\n", s)
+            existing_opening = text[s:line_end] if line_end != -1 else text[s:]
+            if existing_opening == start:
+                # Opening-marker stamp matches — leave the text untouched so
+                # a formatter's reflow of the body is preserved.
+                return text
             before = text[:s]
             after = text[e + len(end) :].lstrip("\n")
             return before + body + after
         else:
             stripped = text.rstrip("\n")
-            prefix = stripped + "\n\n" if stripped else ""
-            return prefix + body
+            lead = stripped + "\n\n" if stripped else ""
+            return lead + body
 
 
 def sync_instruction_file(
@@ -86,6 +118,18 @@ def sync_instruction_file(
     if not target.exists() and code_section is None and docs_section is None:
         return False
     existing = target.read_text() if target.exists() else ""
-    updated = _apply_section(existing, MARKER_START, MARKER_END, code_section)
-    updated = _apply_section(updated, MARKER_DOCS_START, MARKER_DOCS_END, docs_section)
+    updated = _apply_section(
+        existing,
+        MARKER_START,
+        MARKER_END,
+        code_section,
+        prefix=MARKER_START_PREFIX,
+    )
+    updated = _apply_section(
+        updated,
+        MARKER_DOCS_START,
+        MARKER_DOCS_END,
+        docs_section,
+        prefix=MARKER_DOCS_START_PREFIX,
+    )
     return sync_file(path, updated, project_root=project_root, check=check)

{uncoded-1.1.0 → uncoded-1.2.0}/src/uncoded/skill.py

@@ -18,17 +18,24 @@ LEGACY_SKILL_OUTPUTS = [
 _SKILL_CONTENT = (files("uncoded") / "coherence_review.md").read_text(encoding="utf-8")
 
 
-def sync_skill(*, project_root: Path, check: bool) -> bool:
-    """Write the coherence-review skill file to all supported agent locations.
+def sync_skill(*, project_root: Path, check: bool, build: bool) -> bool:
+    """Sync the coherence-review skill file to all supported agent locations.
 
-    The skill files are written under ``project_root``. Log lines still
-    name each configured path as given, so messages stay
-    project-relative regardless of where the caller is running from.
+    When build is True, write the skill files under project_root.
+    When build is False, remove any existing skill files. In both cases,
+    remove legacy skill files left by older versions of uncoded.
     """
-    results = [
-        sync_file(path, _SKILL_CONTENT, project_root=project_root, check=check)
-        for path in SKILL_OUTPUTS
-    ]
+    results: list[bool] = []
+    if build:
+        results.extend(
+            sync_file(path, _SKILL_CONTENT, project_root=project_root, check=check)
+            for path in SKILL_OUTPUTS
+        )
+    else:
+        results.extend(
+            remove_file(path, project_root=project_root, check=check)
+            for path in SKILL_OUTPUTS
+        )
     results.extend(
         remove_file(path, project_root=project_root, check=check)
         for path in LEGACY_SKILL_OUTPUTS

{uncoded-1.1.0 → uncoded-1.2.0}/tests/test_cli.py

@@ -14,6 +14,7 @@ from unittest import mock
 import pytest
 
 from uncoded import cli
+from uncoded.instruction_files import MARKER_START
 from uncoded.skill import SKILL_OUTPUTS
 
 
@@ -93,7 +94,7 @@ class TestSyncApplyMode:
         assert cli._sync() == 0
 
         # The section is written through the symlink to AGENTS.md.
-        assert "<!-- uncoded:start -->" in agents.read_text()
+        assert MARKER_START in agents.read_text()
 
         # Exactly one user-facing line for the instruction file, naming
         # the canonical AGENTS.md.
@@ -877,6 +878,8 @@ class TestSyncDocRoots:
 
         assert (tmp_path / ".uncoded" / "namespace.yaml").exists()
         assert (tmp_path / ".uncoded" / "stubs").exists()
+        for path in SKILL_OUTPUTS:
+            assert (tmp_path / path).exists()
 
         # Drop source-roots.
         (tmp_path / "pyproject.toml").write_text(
@@ -893,6 +896,8 @@ class TestSyncDocRoots:
         assert cli._sync() == 0
         assert not (tmp_path / ".uncoded" / "namespace.yaml").exists()
         assert not (tmp_path / ".uncoded" / "stubs").exists()
+        for path in SKILL_OUTPUTS:
+            assert not (tmp_path / path).exists()
 
     def test_doc_root_removal_cleans_docs_yaml(self, tmp_path, monkeypatch):
         # First sync with doc-roots; then drop them.
@@ -998,6 +1003,34 @@ class TestSyncDocRoots:
         (tmp_path / "src").mkdir()
         assert cli._sync(check=True) == 1
 
+    def test_doc_only_does_not_write_skill(self, tmp_path, monkeypatch):
+        _init_doc_repo(tmp_path, monkeypatch)
+        assert cli._sync() == 0
+        for path in SKILL_OUTPUTS:
+            assert not (tmp_path / path).exists()
+
+    def test_doc_only_removes_preexisting_skill(self, tmp_path, monkeypatch):
+        _init_doc_repo(tmp_path, monkeypatch)
+        for path in SKILL_OUTPUTS:
+            skill_path = tmp_path / path
+            skill_path.parent.mkdir(parents=True, exist_ok=True)
+            skill_path.write_text("old skill\n")
+        assert cli._sync() == 0
+        for path in SKILL_OUTPUTS:
+            assert not (tmp_path / path).exists()
+
+    def test_check_returns_one_when_skill_should_be_removed(
+        self, tmp_path, monkeypatch
+    ):
+        _init_doc_repo(tmp_path, monkeypatch)
+        for path in SKILL_OUTPUTS:
+            skill_path = tmp_path / path
+            skill_path.parent.mkdir(parents=True, exist_ok=True)
+            skill_path.write_text("old skill\n")
+        assert cli._sync(check=True) == 1
+        for path in SKILL_OUTPUTS:
+            assert (tmp_path / path).exists()
+
     def test_idempotent_doc_only(self, tmp_path, monkeypatch):
         _init_doc_repo(tmp_path, monkeypatch)
         cli._sync()

{uncoded-1.1.0 → uncoded-1.2.0}/tests/test_instruction_files.py

@@ -60,8 +60,9 @@ class TestSyncInstructionFile:
         assert SECTION_CODE in content
 
     def test_replaces_existing_code_section(self, tmp_path):
+        # An old plain marker (no fingerprint) is replaced with the current section.
         path = tmp_path / "CLAUDE.md"
-        old_section = f"{MARKER_START}\nold content\n{MARKER_END}\n"
+        old_section = f"<!-- uncoded:start -->\nold content\n{MARKER_END}\n"
         path.write_text(f"# My Project\n\n{old_section}")
         sync_instruction_file(
             path, code_section=SECTION_CODE, docs_section=None, project_root=tmp_path
@@ -72,8 +73,9 @@ class TestSyncInstructionFile:
         assert "# My Project" in content
 
     def test_preserves_content_after_code_section(self, tmp_path):
+        # Content after the old section is preserved when replacing.
         path = tmp_path / "CLAUDE.md"
-        old_section = f"{MARKER_START}\nold\n{MARKER_END}\n"
+        old_section = f"<!-- uncoded:start -->\nold\n{MARKER_END}\n"
         path.write_text(f"{old_section}\n## Other section\n")
         sync_instruction_file(
             path, code_section=SECTION_CODE, docs_section=None, project_root=tmp_path
@@ -124,11 +126,12 @@ class TestSyncInstructionFile:
         assert content.index(MARKER_START) < content.index(MARKER_DOCS_START)
 
     def test_both_sections_replaced_independently(self, tmp_path):
-        # Each section is updated without disturbing the other.
+        # Old plain markers for both sections are each replaced without
+        # disturbing the other.
         path = tmp_path / "CLAUDE.md"
         path.write_text(
-            f"# Title\n\n{MARKER_START}\nold code\n{MARKER_END}\n\n"
-            f"{MARKER_DOCS_START}\nold docs\n{MARKER_DOCS_END}\n"
+            f"# Title\n\n<!-- uncoded:start -->\nold code\n{MARKER_END}\n\n"
+            f"<!-- uncoded:docs:start -->\nold docs\n{MARKER_DOCS_END}\n"
         )
         sync_instruction_file(
             path,
@@ -295,3 +298,83 @@ class TestSyncInstructionFileProjectRootAnchor:
         content = (tmp_path / rel).read_text()
         assert "# My Project" in content
         assert SECTION_CODE in content
+
+
+class TestSyncInstructionFileFingerprint:
+    def test_reflowed_body_survives_sync(self, tmp_path):
+        # Within a version, a formatter's reflow of the section body does
+        # not trigger a rewrite — the opening marker fingerprint still matches.
+        path = tmp_path / "CLAUDE.md"
+        sync_instruction_file(
+            path, code_section=SECTION_CODE, docs_section=None, project_root=tmp_path
+        )
+        # Simulate a markdown formatter reflowing the body between the markers.
+        path.write_text(f"{MARKER_START}\nReflowed body.\n{MARKER_END}\n")
+        result = sync_instruction_file(
+            path, code_section=SECTION_CODE, docs_section=None, project_root=tmp_path
+        )
+        assert result is False
+        assert "Reflowed body." in path.read_text()
+
+    def test_reflowed_body_passes_check(self, tmp_path):
+        # check mode also reports no change when the opening marker matches.
+        path = tmp_path / "CLAUDE.md"
+        sync_instruction_file(
+            path, code_section=SECTION_CODE, docs_section=None, project_root=tmp_path
+        )
+        path.write_text(f"{MARKER_START}\nReflowed body.\n{MARKER_END}\n")
+        result = sync_instruction_file(
+            path,
+            code_section=SECTION_CODE,
+            docs_section=None,
+            project_root=tmp_path,
+            check=True,
+        )
+        assert result is False
+
+    def test_different_fingerprint_refreshes_section(self, tmp_path):
+        # A section from an older uncoded version (different fingerprint)
+        # is replaced with the current canonical section.
+        path = tmp_path / "CLAUDE.md"
+        old_marker = "<!-- uncoded:start sha256=deadbeef -->"
+        path.write_text(f"{old_marker}\nOld wording.\n{MARKER_END}\n")
+        result = sync_instruction_file(
+            path, code_section=SECTION_CODE, docs_section=None, project_root=tmp_path
+        )
+        assert result is True
+        content = path.read_text()
+        assert SECTION_CODE in content
+        assert "Old wording." not in content
+
+    def test_plain_marker_refreshes_once_then_stable(self, tmp_path):
+        # An old plain marker (no fingerprint) is refreshed on the first
+        # sync after upgrading uncoded, then stays stable.
+        path = tmp_path / "CLAUDE.md"
+        path.write_text(f"<!-- uncoded:start -->\nold body\n{MARKER_END}\n")
+        result = sync_instruction_file(
+            path, code_section=SECTION_CODE, docs_section=None, project_root=tmp_path
+        )
+        assert result is True
+        assert SECTION_CODE in path.read_text()
+        result = sync_instruction_file(
+            path, code_section=SECTION_CODE, docs_section=None, project_root=tmp_path
+        )
+        assert result is False
+
+    def test_prose_mention_of_prefix_before_section_is_ignored(self, tmp_path):
+        # A line that contains the marker prefix inside prose (not at column 0)
+        # must not be mistaken for the section opener. Only a line that starts
+        # at column 0 is a valid opener.
+        path = tmp_path / "CLAUDE.md"
+        prose = "Use `<!-- uncoded:start` markers to delimit sections.\n\n"
+        old_section = f"<!-- uncoded:start -->\nold body\n{MARKER_END}\n"
+        path.write_text(f"{prose}{old_section}")
+        result = sync_instruction_file(
+            path, code_section=SECTION_CODE, docs_section=None, project_root=tmp_path
+        )
+        # The real section (old plain marker) is replaced; the prose is preserved.
+        assert result is True
+        content = path.read_text()
+        assert "<!-- uncoded:start` markers" in content
+        assert "old body" not in content
+        assert SECTION_CODE in content

{uncoded-1.1.0 → uncoded-1.2.0}/tests/test_skill.py

@@ -18,43 +18,43 @@ class TestSyncSkill:
         assert "name: uncoded-review\n" not in _SKILL_CONTENT
 
     def test_writes_skill_files(self, tmp_path):
-        sync_skill(project_root=tmp_path, check=False)
+        sync_skill(project_root=tmp_path, check=False, build=True)
         for path in SKILL_OUTPUTS:
             skill_path = tmp_path / path
             assert skill_path.exists()
             assert skill_path.read_text() == _SKILL_CONTENT
 
     def test_creates_parent_directories(self, tmp_path):
-        sync_skill(project_root=tmp_path, check=False)
+        sync_skill(project_root=tmp_path, check=False, build=True)
         for path in SKILL_OUTPUTS:
             assert (tmp_path / path).parent.is_dir()
 
     def test_returns_true_on_first_write(self, tmp_path):
-        assert sync_skill(project_root=tmp_path, check=False) is True
+        assert sync_skill(project_root=tmp_path, check=False, build=True) is True
 
     def test_returns_false_when_already_in_sync(self, tmp_path):
-        sync_skill(project_root=tmp_path, check=False)
-        assert sync_skill(project_root=tmp_path, check=False) is False
+        sync_skill(project_root=tmp_path, check=False, build=True)
+        assert sync_skill(project_root=tmp_path, check=False, build=True) is False
 
     def test_idempotent(self, tmp_path):
-        sync_skill(project_root=tmp_path, check=False)
+        sync_skill(project_root=tmp_path, check=False, build=True)
         mtimes = [(tmp_path / path).stat().st_mtime_ns for path in SKILL_OUTPUTS]
-        sync_skill(project_root=tmp_path, check=False)
+        sync_skill(project_root=tmp_path, check=False, build=True)
         assert [
             (tmp_path / path).stat().st_mtime_ns for path in SKILL_OUTPUTS
         ] == mtimes
 
     def test_check_mode_does_not_write(self, tmp_path):
-        sync_skill(project_root=tmp_path, check=True)
+        sync_skill(project_root=tmp_path, check=True, build=True)
         for path in SKILL_OUTPUTS:
             assert not (tmp_path / path).exists()
 
     def test_check_mode_reports_change_when_missing(self, tmp_path):
-        assert sync_skill(project_root=tmp_path, check=True) is True
+        assert sync_skill(project_root=tmp_path, check=True, build=True) is True
 
     def test_check_mode_reports_no_change_when_in_sync(self, tmp_path):
-        sync_skill(project_root=tmp_path, check=False)
-        assert sync_skill(project_root=tmp_path, check=True) is False
+        sync_skill(project_root=tmp_path, check=False, build=True)
+        assert sync_skill(project_root=tmp_path, check=True, build=True) is False
 
     def test_removes_legacy_skill_files(self, tmp_path):
         for path in LEGACY_SKILL_OUTPUTS:
@@ -62,7 +62,7 @@ class TestSyncSkill:
             legacy_path.parent.mkdir(parents=True, exist_ok=True)
             legacy_path.write_text("old skill\n")
 
-        assert sync_skill(project_root=tmp_path, check=False) is True
+        assert sync_skill(project_root=tmp_path, check=False, build=True) is True
 
         for path in LEGACY_SKILL_OUTPUTS:
             assert not (tmp_path / path).exists()
@@ -73,7 +73,7 @@ class TestSyncSkill:
             legacy_path.parent.mkdir(parents=True, exist_ok=True)
             legacy_path.write_text("old skill\n")
 
-        assert sync_skill(project_root=tmp_path, check=True) is True
+        assert sync_skill(project_root=tmp_path, check=True, build=True) is True
 
         for path in LEGACY_SKILL_OUTPUTS:
             assert (tmp_path / path).exists()
@@ -87,7 +87,7 @@ class TestSyncSkillProjectRootAnchor:
         sub.mkdir()
         monkeypatch.chdir(sub)
 
-        sync_skill(project_root=tmp_path, check=False)
+        sync_skill(project_root=tmp_path, check=False, build=True)
 
         for path in SKILL_OUTPUTS:
             assert (tmp_path / path).exists()
@@ -106,7 +106,7 @@ class TestSyncSkillProjectRootAnchor:
             legacy_path.parent.mkdir(parents=True, exist_ok=True)
             legacy_path.write_text("old skill\n")
 
-        sync_skill(project_root=tmp_path, check=False)
+        sync_skill(project_root=tmp_path, check=False, build=True)
 
         for path in LEGACY_SKILL_OUTPUTS:
             assert not (tmp_path / path).exists()