codeboarding 0.13.7__tar.gz → 0.13.8__tar.gz

{codeboarding-0.13.7/codeboarding.egg-info → codeboarding-0.13.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codeboarding
-Version: 0.13.7
+Version: 0.13.8
 Summary: Interactive Diagrams for Code
 Author: CodeBoarding Team
 License-Expression: MIT

{codeboarding-0.13.7 → codeboarding-0.13.8/codeboarding.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codeboarding
-Version: 0.13.7
+Version: 0.13.8
 Summary: Interactive Diagrams for Code
 Author: CodeBoarding Team
 License-Expression: MIT

{codeboarding-0.13.7 → codeboarding-0.13.8}/codeboarding.egg-info/SOURCES.txt

@@ -189,6 +189,7 @@ tests/test_github_action.py
 tests/test_install.py
 tests/test_logging_config.py
 tests/test_main.py
+tests/test_persisted_analysis_artifacts.py
 tests/test_pyproject_packages.py
 tests/test_registry_coverage.py
 tests/test_telemetry_events.py

{codeboarding-0.13.7 → codeboarding-0.13.8}/constants.py

@@ -1,6 +1,18 @@
 """Application-level constants for CodeBoarding."""
 
 CODEBOARDING_DIR_NAME = ".codeboarding"
+ANALYSIS_FILENAME = "analysis.json"
+FINGERPRINT_FILENAME = "fingerprint.json"
+STATIC_ANALYSIS_PKL = "static_analysis.pkl"
+STATIC_ANALYSIS_SHA = "static_analysis.sha"
+CODEBOARDING_VERSION_FILENAME = "codeboarding_version.json"
+PERSISTED_ANALYSIS_ARTIFACT_FILENAMES = (
+    ANALYSIS_FILENAME,
+    FINGERPRINT_FILENAME,
+    STATIC_ANALYSIS_PKL,
+    STATIC_ANALYSIS_SHA,
+    CODEBOARDING_VERSION_FILENAME,
+)
 # Ignore-file names read from a repo (``.gitignore`` at the repo root,
 # ``.codeboardingignore`` under ``CODEBOARDING_DIR_NAME``).
 GITIGNORE_FILENAME = ".gitignore"

{codeboarding-0.13.7 → codeboarding-0.13.8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "codeboarding"
-version = "0.13.7"
+version = "0.13.8"
 description = "Interactive Diagrams for Code"
 readme = "PYPI.md"
 license = "MIT"

{codeboarding-0.13.7 → codeboarding-0.13.8}/static_analyzer/analysis_cache.py

@@ -29,11 +29,12 @@ from typing import TYPE_CHECKING, Any
 
 from filelock import FileLock
 
+from constants import STATIC_ANALYSIS_PKL, STATIC_ANALYSIS_SHA
+from repo_utils.path_utils import to_absolute_path, to_relative_path
 from static_analyzer.analysis_result import AnalysisData, InvalidatedAnalysis, InvalidatedEdge
 from static_analyzer.graph import CallGraph, Edge, EdgeKind
 from static_analyzer.lsp_client.diagnostics import FileDiagnosticsMap
 from static_analyzer.node import Node
-from repo_utils.path_utils import to_absolute_path, to_relative_path
 
 if TYPE_CHECKING:
     from static_analyzer.analysis_result import StaticAnalysisResults
@@ -44,8 +45,6 @@ logger = logging.getLogger(__name__)
 # Run-artifact filenames. Stored in ``<repo>/.codeboarding/`` (sibling of
 # ``analysis.json``), not under ``cache/`` — losing them costs a full LSP
 # re-index, so they're not safe to wipe with the rest of the cache.
-STATIC_ANALYSIS_PKL = "static_analysis.pkl"
-STATIC_ANALYSIS_SHA = "static_analysis.sha"
 STATIC_ANALYSIS_LOCK = "static_analysis.lock"
 # Legacy location ``StaticAnalysisCache`` wrote to before the run-artifact
 # split. Kept for one-time read fallback so CLI users transition smoothly.

{codeboarding-0.13.7 → codeboarding-0.13.8}/static_analyzer/language_results.py

@@ -25,6 +25,11 @@ class ControlFlowGraph:
         if self.graph is None:
             self.graph = other
             return
+        if self.graph is other:
+            # Why: two engine configs can share a language (two tsconfig projects give
+            # two TypeScript engines) and warm-start hands both the same cached graph.
+            # Iterating a container while adding to it is unbounded, not just wasteful.
+            return
         for node in other.nodes.values():
             self.graph.add_node(node)
         for edge in other.edges:
@@ -55,6 +60,11 @@ class ControlFlowGraph:
         # Re-added via the API so alias-resolution and node-existence guards apply post-merge.
         for src, dst, kind in getattr(other, "reference_edges", ()):
             self.graph.add_reference_edge(src, dst, EdgeKind(kind))
+        # Nodes and call edges are keyed, reference edges are appended, so only they
+        # survive a repeat merge. Dedup after the loop rather than per-add: the stored
+        # tuple is post-alias-resolution, and a membership scan per edge is quadratic.
+        # getattr for the same reason as above: pre-reference-edge caches lack the field.
+        self.graph.reference_edges = list(dict.fromkeys(getattr(self.graph, "reference_edges", ())))
         self.graph.method_cluster_paths.merge(other.method_cluster_paths)
 
     def visit_paths(self, fn: Callable[[str], str]) -> None:
@@ -126,9 +136,16 @@ class SourceFiles:
     paths: list[str] | None = None
 
     def extend(self, files: list[str]) -> None:
+        # Why: called once per engine config, and two engines of one language are
+        # handed the same cached list on warm-start. Duplicates persist into the pkl,
+        # so an undeduped extend doubles the list on every subsequent run.
         if self.paths is None:
             self.paths = []
-        self.paths.extend(files)
+        known = set(self.paths)
+        for path in files:
+            if path not in known:
+                known.add(path)
+                self.paths.append(path)
 
     def visit_paths(self, fn: Callable[[str], str]) -> None:
         if self.paths is None:

codeboarding-0.13.8/tests/test_persisted_analysis_artifacts.py

@@ -0,0 +1,29 @@
+from constants import (
+    ANALYSIS_FILENAME,
+    CODEBOARDING_VERSION_FILENAME,
+    FINGERPRINT_FILENAME,
+    PERSISTED_ANALYSIS_ARTIFACT_FILENAMES,
+    STATIC_ANALYSIS_PKL,
+    STATIC_ANALYSIS_SHA,
+)
+from static_analyzer.analysis_cache import (
+    STATIC_ANALYSIS_PKL as COMPAT_STATIC_ANALYSIS_PKL,
+    STATIC_ANALYSIS_SHA as COMPAT_STATIC_ANALYSIS_SHA,
+)
+from utils import ANALYSIS_FILENAME as COMPAT_ANALYSIS_FILENAME
+from utils import FINGERPRINT_FILENAME as COMPAT_FINGERPRINT_FILENAME
+
+
+def test_persisted_analysis_artifact_manifest_and_compatibility_imports():
+    assert PERSISTED_ANALYSIS_ARTIFACT_FILENAMES == (
+        "analysis.json",
+        "fingerprint.json",
+        "static_analysis.pkl",
+        "static_analysis.sha",
+        "codeboarding_version.json",
+    )
+    assert CODEBOARDING_VERSION_FILENAME == "codeboarding_version.json"
+    assert COMPAT_ANALYSIS_FILENAME == ANALYSIS_FILENAME
+    assert COMPAT_FINGERPRINT_FILENAME == FINGERPRINT_FILENAME
+    assert COMPAT_STATIC_ANALYSIS_PKL == STATIC_ANALYSIS_PKL
+    assert COMPAT_STATIC_ANALYSIS_SHA == STATIC_ANALYSIS_SHA

{codeboarding-0.13.7 → codeboarding-0.13.8}/utils.py

@@ -7,14 +7,12 @@ import uuid
 from collections.abc import Iterable
 from pathlib import Path
 
-from constants import CODEBOARDING_DIR_NAME
+from constants import ANALYSIS_FILENAME, CODEBOARDING_DIR_NAME, FINGERPRINT_FILENAME
 from repo_utils.path_utils import to_absolute_path, to_relative_path
 
 logger = logging.getLogger(__name__)
 
 RUN_OUTPUT_DIR_NAME = "run-output"
-ANALYSIS_FILENAME = "analysis.json"
-FINGERPRINT_FILENAME = "fingerprint.json"
 
 
 class CFGGenerationError(Exception):