uncoded 0.7.0__tar.gz → 0.7.1__tar.gz

{uncoded-0.7.0 → uncoded-0.7.1}/.uncoded/namespace.yaml

@@ -78,7 +78,7 @@ src/:
         name:
         annotation:
         value_source:
-        is_type_alias:
+        is_pep695_alias:
       StubClass:
         name:
         bases:
@@ -114,6 +114,7 @@ tests/:
       test_writes_namespace_map_stubs_and_instruction_file:
       test_idempotent_second_run:
       test_dedupes_when_claude_md_is_symlink_to_agents_md:
+      test_instruction_file_outside_project_uses_absolute_path:
       test_error_when_no_pyproject_toml:
       test_error_when_source_root_missing:
       test_error_when_uncoded_section_missing:
@@ -155,7 +156,7 @@ tests/:
       test_reads_configured_list:
       test_empty_list_is_respected:
   test_extract.py:
-    TestExtractModule:
+    TestExtractModuleFromSource:
       test_classes_and_functions:
       test_async_functions_and_methods:
       test_empty_module:
@@ -163,10 +164,13 @@ tests/:
       test_type_alias_classic:
       test_type_alias_pep695:
       test_tuple_unpacking_skipped:
+      test_chained_assignment_skipped:
       test_unannotated_class_variable:
+      test_class_tuple_unpacking_skipped:
       test_annotated_attributes:
       test_property_classified_as_attribute:
       test_property_setter_and_deleter_suppressed:
+      test_non_property_decorator_classified_as_method:
       test_preserves_source_order:
     TestIterAndExtract:
       test_basic_walk:
@@ -174,7 +178,7 @@ tests/:
       test_includes_init_with_symbols:
       test_skips_empty_init:
       test_skips_syntax_errors:
-    TestExtractModules:
+    TestExtractModulesFromFiles:
       test_returns_module_info_per_parseable_file:
       test_preserves_source_order:
       test_module_with_only_constants_is_kept:
@@ -232,6 +236,7 @@ tests/:
       test_merges_into_existing_claude_settings:
       test_does_not_overwrite_existing_serena_project_yml:
       test_does_not_duplicate_on_second_merge:
+      test_appends_missing_allowed_tool_to_existing_settings:
       test_setup_uses_root_name_when_no_pyproject:
       test_setup_reads_name_from_pyproject:
     TestRepoDogfooding:
@@ -263,7 +268,7 @@ tests/:
       test_class_with_attributes_and_methods:
       test_class_with_bases:
       test_class_no_bases:
-      test_kwargs_and_varargs:
+      test_extract_params_covers_input_kind:
       test_imports_collected:
       test_syntax_error_raises:
       test_source_order_preserved:
@@ -275,6 +280,8 @@ tests/:
       test_type_alias_classic:
       test_type_alias_pep695:
       test_tuple_unpacking_skipped:
+      test_ann_assign_non_name_target_skipped:
+      test_class_tuple_unpacking_skipped:
       test_class_with_unannotated_attribute:
       test_property_rendered_as_attribute:
       test_property_without_return_annotation:
@@ -284,9 +291,9 @@ tests/:
       test_imports_rendered:
       test_rendered_stub_has_no_line_range_comments:
       test_async_function_prefix:
-      test_function_with_annotations:
+      test_render_param_covers_input_kind:
+      test_return_annotation_rendered:
       test_class_with_bases:
-      test_class_no_bases:
       test_class_with_no_members_renders_body:
       test_attribute_with_annotation:
       test_method_indented:
@@ -298,6 +305,7 @@ tests/:
       test_constant_bare_annotation_rendered:
       test_type_alias_pep695_rendered:
       test_unannotated_class_attribute_rendered:
+      test_renders_valid_python_for_representative_source:
     TestBuildStubs:
       _setup:
       _build:
@@ -309,6 +317,7 @@ tests/:
       test_no_op_when_clean:
       test_reports_count_on_first_build:
       test_reports_zero_when_clean:
+      test_skips_module_with_no_symbols:
     TestBuildStubsCheckMode:
       _setup:
       _build:
@@ -322,6 +331,7 @@ tests/:
       test_prunes_orphan_stubs:
       test_project_root_anchors_writes_independent_of_cwd:
       test_project_root_anchors_orphan_pruning_independent_of_cwd:
+      test_source_root_outside_project_root_skips_cleanup:
   test_sync.py:
     TestSyncFile:
       test_creates_missing_file:

{uncoded-0.7.0 → uncoded-0.7.1}/.uncoded/stubs/src/uncoded/stubs.pyi

@@ -68,7 +68,7 @@ class StubAssignment:
     name: str
     annotation: str | None = None
     value_source: str | None = None
-    is_type_alias: bool = False
+    is_pep695_alias: bool = False
 
 class StubClass:
     name: str

{uncoded-0.7.0 → uncoded-0.7.1}/.uncoded/stubs/tests/test_cli.pyi

@@ -20,6 +20,9 @@ class TestSyncApplyMode:
     def test_dedupes_when_claude_md_is_symlink_to_agents_md(self, tmp_path, monkeypatch, capsys):
         ...
 
+    def test_instruction_file_outside_project_uses_absolute_path(self, tmp_path, monkeypatch):
+        ...
+
     def test_error_when_no_pyproject_toml(self, tmp_path, monkeypatch, capsys):
         ...
 

{uncoded-0.7.0 → uncoded-0.7.1}/.uncoded/stubs/tests/test_extract.pyi

@@ -3,7 +3,7 @@
 import textwrap
 from uncoded.extract import extract_module, extract_modules, iter_source_files
 
-class TestExtractModule:
+class TestExtractModuleFromSource:
     def test_classes_and_functions(self):
         ...
 
@@ -25,9 +25,15 @@ class TestExtractModule:
     def test_tuple_unpacking_skipped(self):
         ...
 
+    def test_chained_assignment_skipped(self):
+        ...
+
     def test_unannotated_class_variable(self):
         ...
 
+    def test_class_tuple_unpacking_skipped(self):
+        ...
+
     def test_annotated_attributes(self):
         ...
 
@@ -37,6 +43,9 @@ class TestExtractModule:
     def test_property_setter_and_deleter_suppressed(self):
         ...
 
+    def test_non_property_decorator_classified_as_method(self):
+        ...
+
     def test_preserves_source_order(self):
         ...
 
@@ -56,7 +65,7 @@ class TestIterAndExtract:
     def test_skips_syntax_errors(self, tmp_path, capsys):
         ...
 
-class TestExtractModules:
+class TestExtractModulesFromFiles:
     def test_returns_module_info_per_parseable_file(self):
         ...
 

{uncoded-0.7.0 → uncoded-0.7.1}/.uncoded/stubs/tests/test_serena_setup.pyi

@@ -3,7 +3,7 @@
 import json
 from pathlib import Path
 import yaml
-from uncoded.serena_setup import SERENA_ALLOWED_TOOLS, SERENA_VERSION, setup
+from uncoded.serena_setup import MCP_SERVER_SERENA, SERENA_ALLOWED_TOOLS, SERENA_VERSION, setup
 
 REPO_ROOT = Path(__file__).parent.parent
 EXPECTED_EXCLUDED_TOOLS = ...
@@ -46,6 +46,9 @@ class TestSetup:
     def test_does_not_duplicate_on_second_merge(self, tmp_path):
         ...
 
+    def test_appends_missing_allowed_tool_to_existing_settings(self, tmp_path):
+        ...
+
     def test_setup_uses_root_name_when_no_pyproject(self, tmp_path):
         ...
 

{uncoded-0.7.0 → uncoded-0.7.1}/.uncoded/stubs/tests/test_stubs.pyi

@@ -28,7 +28,7 @@ class TestExtractStub:
     def test_class_no_bases(self):
         ...
 
-    def test_kwargs_and_varargs(self):
+    def test_extract_params_covers_input_kind(self, source, expected):
         ...
 
     def test_imports_collected(self):
@@ -64,6 +64,12 @@ class TestExtractStub:
     def test_tuple_unpacking_skipped(self):
         ...
 
+    def test_ann_assign_non_name_target_skipped(self):
+        ...
+
+    def test_class_tuple_unpacking_skipped(self):
+        ...
+
     def test_class_with_unannotated_attribute(self):
         ...
 
@@ -89,13 +95,13 @@ class TestRenderStub:
     def test_async_function_prefix(self):
         ...
 
-    def test_function_with_annotations(self):
+    def test_render_param_covers_input_kind(self, params, expected):
         ...
 
-    def test_class_with_bases(self):
+    def test_return_annotation_rendered(self):
         ...
 
-    def test_class_no_bases(self):
+    def test_class_with_bases(self):
         ...
 
     def test_class_with_no_members_renders_body(self):
@@ -131,6 +137,9 @@ class TestRenderStub:
     def test_unannotated_class_attribute_rendered(self):
         ...
 
+    def test_renders_valid_python_for_representative_source(self):
+        ...
+
 class TestBuildStubs:
     def _setup(self, tmp_path):
         ...
@@ -162,6 +171,9 @@ class TestBuildStubs:
     def test_reports_zero_when_clean(self, tmp_path):
         ...
 
+    def test_skips_module_with_no_symbols(self, tmp_path):
+        ...
+
 class TestBuildStubsCheckMode:
     def _setup(self, tmp_path):
         ...
@@ -196,3 +208,6 @@ class TestWriteStubs:
 
     def test_project_root_anchors_orphan_pruning_independent_of_cwd(self, tmp_path, monkeypatch):
         ...
+
+    def test_source_root_outside_project_root_skips_cleanup(self, tmp_path):
+        ...

{uncoded-0.7.0 → uncoded-0.7.1}/AGENTS.md

@@ -47,8 +47,11 @@ uv run uncoded check
 # Generate Serena + ty MCP and Claude Code configuration
 uv run uncoded setup
 
-# Run tests
+# Run tests (branch coverage enforced; see [tool.coverage.report] in pyproject.toml)
 uv run pytest
+
+# Run a subset of tests without the coverage gate
+uv run pytest tests/test_stubs.py --no-cov
 ```
 
 <!-- uncoded:start -->

{uncoded-0.7.0 → uncoded-0.7.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: uncoded
-Version: 0.7.0
+Version: 0.7.1
 Summary: Symbol indexes and language-server setup for AI-assisted code navigation
 Project-URL: Homepage, https://github.com/alimanfoo/uncoded
 Project-URL: Repository, https://github.com/alimanfoo/uncoded
@@ -14,6 +14,7 @@ Requires-Python: >=3.12
 Requires-Dist: pyyaml>=6.0
 Provides-Extra: dev
 Requires-Dist: pre-commit>=3.0; extra == 'dev'
+Requires-Dist: pytest-cov>=6.0; extra == 'dev'
 Requires-Dist: pytest>=8.0; extra == 'dev'
 Description-Content-Type: text/markdown
 
@@ -229,13 +230,20 @@ uv sync --extra dev
 uv run pre-commit install
 ```
 
-Run the tests:
+Run the tests (branch coverage is enforced; see `[tool.coverage.report]` in
+`pyproject.toml` for the threshold):
 
 ```
 uv run pytest
 ```
 
-Run all checks (the same suite CI runs):
+To run a subset of tests without the coverage gate:
+
+```
+uv run pytest tests/test_stubs.py --no-cov
+```
+
+Run the same checks CI's lint job runs:
 
 ```
 uv run pre-commit run --all-files

{uncoded-0.7.0 → uncoded-0.7.1}/README.md

@@ -210,13 +210,20 @@ uv sync --extra dev
 uv run pre-commit install
 ```
 
-Run the tests:
+Run the tests (branch coverage is enforced; see `[tool.coverage.report]` in
+`pyproject.toml` for the threshold):
 
 ```
 uv run pytest
 ```
 
-Run all checks (the same suite CI runs):
+To run a subset of tests without the coverage gate:
+
+```
+uv run pytest tests/test_stubs.py --no-cov
+```
+
+Run the same checks CI's lint job runs:
 
 ```
 uv run pre-commit run --all-files

{uncoded-0.7.0 → uncoded-0.7.1}/pyproject.toml

@@ -25,6 +25,7 @@ classifiers = [
 dev = [
     "pre-commit>=3.0",
     "pytest>=8.0",
+    "pytest-cov>=6.0",
 ]
 
 [project.scripts]
@@ -42,6 +43,15 @@ source-roots = ["src", "tests"]
 
 [tool.pytest.ini_options]
 testpaths = ["tests"]
+addopts = "--cov"
+
+[tool.coverage.run]
+source = ["src"]
+branch = true
+
+[tool.coverage.report]
+show_missing = true
+fail_under = 100
 
 [tool.ruff]
 src = ["src"]

{uncoded-0.7.0 → uncoded-0.7.1}/src/uncoded/extract.py

@@ -87,7 +87,7 @@ def extract_module(source: str, rel_path: str) -> ModuleInfo:
             name = _assign_target_name(node)
             if name:
                 constants.append(name)
-        elif isinstance(node, ast.TypeAlias) and isinstance(node.name, ast.Name):
+        elif isinstance(node, ast.TypeAlias):
             constants.append(node.name.id)
 
     return ModuleInfo(

{uncoded-0.7.0 → uncoded-0.7.1}/src/uncoded/stubs.py

@@ -43,7 +43,7 @@ class StubAssignment:
     name: str
     annotation: str | None = None
     value_source: str | None = None
-    is_type_alias: bool = False
+    is_pep695_alias: bool = False
 
 
 @dataclass
@@ -119,12 +119,10 @@ def _extract_assignment(
     unpacking or attribute assignment), which we can't represent cleanly.
     """
     if isinstance(node, ast.TypeAlias):
-        if not isinstance(node.name, ast.Name):
-            return None
         return StubAssignment(
             name=node.name.id,
             value_source=_render_value(node.value),
-            is_type_alias=True,
+            is_pep695_alias=True,
         )
 
     if isinstance(node, ast.AnnAssign):
@@ -166,8 +164,6 @@ def _property_attribute(
     return StubAssignment(
         name=node.name,
         annotation=ast.unparse(node.returns) if node.returns else None,
-        value_source=None,
-        is_type_alias=False,
     )
 
 
@@ -250,7 +246,7 @@ def _render_function(func: StubFunction, indent: str = "") -> list[str]:
 
 def _format_assignment_body(a: StubAssignment) -> str:
     """Render the 'name [: type] [= value]' portion of an assignment."""
-    if a.is_type_alias:
+    if a.is_pep695_alias:
         return f"type {a.name} = {a.value_source}"
     head = f"{a.name}: {a.annotation}" if a.annotation else a.name
     if a.value_source is None:
@@ -375,7 +371,10 @@ def _write_stubs(
         if existing.resolve() in expected:
             continue
         display = existing.relative_to(project_root)
-        if remove_file(display, project_root=project_root, check=check):
+        # .pyi may have been removed between rglob and remove_file
+        if remove_file(  # pragma: no branch
+            display, project_root=project_root, check=check
+        ):
             changes += 1
 
     if check:

{uncoded-0.7.0 → uncoded-0.7.1}/tests/test_cli.py

@@ -103,6 +103,32 @@ class TestSyncApplyMode:
         ]
         assert instruction_lines == ["Updated AGENTS.md"]
 
+    def test_instruction_file_outside_project_uses_absolute_path(
+        self, tmp_path, monkeypatch
+    ):
+        project = tmp_path / "project"
+        project.mkdir()
+        (project / "src").mkdir()
+        (project / "pyproject.toml").write_text(
+            textwrap.dedent(
+                """\
+                [project]
+                name = "demo"
+
+                [tool.uncoded]
+                source-roots = ["src"]
+                instruction-files = ["../outside.md"]
+                """
+            )
+        )
+        monkeypatch.chdir(project)
+
+        assert cli._sync() == 0
+
+        outside_file = tmp_path / "outside.md"
+        assert outside_file.exists()
+        assert "<!-- uncoded:start -->" in outside_file.read_text()
+
     def test_error_when_no_pyproject_toml(self, tmp_path, monkeypatch, capsys):
         # Pins the problem statement, the recovery hint pointing at
         # [tool.uncoded] source-roots, and the absence of any absolute

{uncoded-0.7.0 → uncoded-0.7.1}/tests/test_extract.py

@@ -3,7 +3,7 @@ import textwrap
 from uncoded.extract import extract_module, extract_modules, iter_source_files
 
 
-class TestExtractModule:
+class TestExtractModuleFromSource:
     def test_classes_and_functions(self):
         source = textwrap.dedent("""\
             class MyClass:
@@ -99,6 +99,13 @@ class TestExtractModule:
 
         assert result.constants == []
 
+    def test_chained_assignment_skipped(self):
+        source = "x = y = 1\n"
+
+        result = extract_module(source, "chain.py")
+
+        assert result.constants == []
+
     def test_unannotated_class_variable(self):
         source = textwrap.dedent("""\
             class Registry:
@@ -112,6 +119,16 @@ class TestExtractModule:
         cls = result.classes[0]
         assert cls.attributes == ["items", "_cache", "count"]
 
+    def test_class_tuple_unpacking_skipped(self):
+        source = textwrap.dedent("""\
+            class C:
+                a, b = 1, 2
+        """)
+
+        result = extract_module(source, "c.py")
+
+        assert result.classes[0].attributes == []
+
     def test_annotated_attributes(self):
         source = textwrap.dedent("""\
             from dataclasses import dataclass
@@ -172,6 +189,20 @@ class TestExtractModule:
         assert cls.attributes == ["path"]
         assert cls.methods == []
 
+    def test_non_property_decorator_classified_as_method(self):
+        source = textwrap.dedent("""\
+            class Util:
+                @staticmethod
+                def helper():
+                    pass
+        """)
+
+        result = extract_module(source, "util.py")
+
+        cls = result.classes[0]
+        assert cls.methods == ["helper"]
+        assert cls.attributes == []
+
     def test_preserves_source_order(self):
         source = textwrap.dedent("""\
             def zebra():
@@ -284,7 +315,7 @@ class TestIterAndExtract:
         assert "SyntaxError" in err
 
 
-class TestExtractModules:
+class TestExtractModulesFromFiles:
     def test_returns_module_info_per_parseable_file(self):
         files = [
             ("def foo(): pass\n", "src/a.py"),

{uncoded-0.7.0 → uncoded-0.7.1}/tests/test_serena_setup.py

@@ -4,6 +4,7 @@ from pathlib import Path
 import yaml
 
 from uncoded.serena_setup import (
+    MCP_SERVER_SERENA,
     SERENA_ALLOWED_TOOLS,
     SERENA_VERSION,
     setup,
@@ -173,6 +174,26 @@ class TestSetup:
         for tool in SERENA_ALLOWED_TOOLS:
             assert claude["permissions"]["allow"].count(tool) == 1
 
+    def test_appends_missing_allowed_tool_to_existing_settings(self, tmp_path):
+        (tmp_path / ".mcp.json").write_text(
+            json.dumps({"mcpServers": {"serena": MCP_SERVER_SERENA}})
+        )
+        claude_path = tmp_path / ".claude" / "settings.json"
+        claude_path.parent.mkdir()
+        missing_tool = SERENA_ALLOWED_TOOLS[-1]
+        claude_path.write_text(
+            json.dumps(
+                {
+                    "enabledMcpjsonServers": ["serena"],
+                    "permissions": {"allow": list(SERENA_ALLOWED_TOOLS[:-1])},
+                }
+            )
+        )
+        self._run(tmp_path)
+        data = json.loads(claude_path.read_text())
+        assert missing_tool in data["permissions"]["allow"]
+        assert set(data["permissions"]["allow"]) >= set(SERENA_ALLOWED_TOOLS)
+
     def test_setup_uses_root_name_when_no_pyproject(self, tmp_path):
         # No ``pyproject.toml`` anywhere from ``tmp_path`` up to the
         # filesystem root, so ``read_project_name(start=tmp_path)`` falls