sourcecode 1.32.0__tar.gz → 1.32.1__tar.gz

{sourcecode-1.32.0 → sourcecode-1.32.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sourcecode
-Version: 1.32.0
+Version: 1.32.1
 Summary: Deterministic codebase context for AI coding agents
 License:                                  Apache License
                                    Version 2.0, January 2004

{sourcecode-1.32.0 → sourcecode-1.32.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "sourcecode"
-version = "1.32.0"
+version = "1.32.1"
 description = "Deterministic codebase context for AI coding agents"
 readme = "README.md"
 requires-python = ">=3.9"

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/__init__.py

@@ -1,3 +1,3 @@
 """sourcecode — Deterministic codebase context maps for AI coding agents."""
 
-__version__ = "1.32.0"
+__version__ = "1.32.1"

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/architecture_analyzer.py

@@ -152,14 +152,15 @@ LAYER_PATTERNS: dict[str, dict[str, list[str]]] = {
 
 # Higher value = wins when score ties
 _PATTERN_PRIORITY: dict[str, int] = {
-    "cqrs":       8,
-    "clean":      7,
-    "onion":      6,
-    "hexagonal":  5,
-    "monorepo":   4,
-    "mvc":        3,
-    "layered":    2,
-    "fullstack":  1,
+    "cqrs":               8,
+    "clean":              7,
+    "onion":              6,
+    "hexagonal":          5,
+    "monorepo":           4,
+    "spring_mvc_layered": 3,
+    "mvc":                3,
+    "layered":            2,
+    "fullstack":          1,
 }
 
 
@@ -248,9 +249,9 @@ class ArchitectureAnalyzer:
         pattern, layers = self._detect_layers(filtered)
         if pattern in (None, "flat", "unknown"):
             if pattern == "flat":
-                limitations.append("Patron de capas no detectado: proyecto con estructura plana")
+                limitations.append("Layer pattern not detected: project has a flat directory structure")
             elif pattern == "unknown":
-                limitations.append("Patron de capas no reconocido: estructura de directorios sin senales claras")
+                limitations.append("Unrecognized layer pattern: directory structure has no clear architectural signals")
 
         # Step 3b: monorepo override — workspace config is hard evidence.
         # Overrides all weak inferred patterns; only truly specialised patterns
@@ -269,7 +270,7 @@ class ArchitectureAnalyzer:
                 pattern = "monorepo"
                 layers = mono_layers
                 limitations.append(
-                    "Workspace config detectado — arquitectura refleja topologia de paquetes"
+                    "Workspace config detected — architecture reflects package topology"
                 )
                 ws_files = [p for p in sm.file_paths if p.split("/")[-1] in _WORKSPACE_CONFIG_FILES]
                 evidence.append({
@@ -570,28 +571,76 @@ class ArchitectureAnalyzer:
                 ))
             return best_pattern, layers
 
-        # 2. Microservices structural detection (before file-naming heuristics)
+        # 2. Spring domain-module detection (petclinic-style: deep common prefix + feature dirs)
+        spring_result = self._detect_spring_domain_modules(source_paths)
+        if spring_result is not None:
+            return spring_result
+
+        # 3. Microservices structural detection (before file-naming heuristics)
         microservices_result = self._detect_microservices(source_paths)
         if microservices_result is not None:
             return microservices_result
 
-        # 3. Functional file-naming heuristic: *_analyzer.py, cli.py, schema.py, …
+        # 5. Functional file-naming heuristic: *_analyzer.py, cli.py, schema.py, …
         func_result = self._detect_layered_functional(source_paths)
         if func_result is not None:
             return func_result
 
-        # 4. Modular sub-package heuristic: ≥2 distinct named sub-packages
+        # 6. Modular sub-package heuristic: ≥2 distinct named sub-packages
         modular_result = self._detect_modular(source_paths)
         if modular_result is not None:
             return modular_result
 
-        # 5. Fallback: flat (shallow) vs truly unknown (deep but unrecognised)
+        # 7. Fallback: flat (shallow) vs truly unknown (deep but unrecognised)
         max_depth = max(
             (len(p.replace("\\", "/").split("/")) - 1 for p in source_paths),
             default=0,
         )
         return ("flat" if max_depth <= 2 else "unknown"), []
 
+    def _detect_spring_domain_modules(
+        self, paths: list[str]
+    ) -> Optional[tuple[str, list[ArchitectureLayer]]]:
+        """Detect Spring Boot domain-organized packages (petclinic-style).
+
+        When all source paths share a deep common package prefix
+        (e.g. src/main/java/org/springframework/samples/petclinic/),
+        strips that prefix and detects feature/domain modules in the remainder.
+        Requires ≥3 distinct domain directories to avoid false positives.
+        """
+        if len(paths) < 6:
+            return None
+
+        parts_list = [p.replace("\\", "/").split("/") for p in paths]
+        min_depth = min(len(p) for p in parts_list)
+        common_depth = 0
+        for i in range(min_depth - 1):
+            seg = parts_list[0][i]
+            if all(pl[i] == seg for pl in parts_list):
+                common_depth = i + 1
+            else:
+                break
+
+        if common_depth < 3:
+            return None
+
+        module_files: dict[str, list[str]] = {}
+        for orig, pl in zip(paths, parts_list):
+            remaining = pl[common_depth:]
+            if remaining and remaining[0] not in _GENERIC_NAMES:
+                module_files.setdefault(remaining[0], []).append(orig)
+
+        meaningful = {k: v for k, v in module_files.items() if len(v) >= 2}
+        if len(meaningful) < 3:
+            return None
+
+        return "spring_mvc_layered", [
+            ArchitectureLayer(
+                name=k, pattern="spring_mvc_layered", files=v, confidence="medium"
+            )
+            for k, v in meaningful.items()
+        ]
+
     def _detect_microservices(
         self, paths: list[str]
     ) -> Optional[tuple[str, list[ArchitectureLayer]]]:

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/classifier.py

@@ -118,6 +118,10 @@ class TypeClassifier:
         ):
             return "webapp"
 
+        _SERVERSIDE_TEMPLATE_FRAMEWORKS = frozenset({"Thymeleaf", "FreeMarker"})
+        if framework_names & _SERVERSIDE_TEMPLATE_FRAMEWORKS:
+            return "web_mvc"
+
         if framework_names & _API_FRAMEWORKS:
             return "api"
 

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/cli.py

@@ -3824,7 +3824,7 @@ def mcp_init(
         raise typer.Exit(code=1)
 
     typer.echo("MCP integration active.")
-    typer.echo("  Note:    repo_path debe usar forward slashes: C:/Users/... o /ruta/unix")
+    typer.echo("  Note:    repo_path must use forward slashes: C:/Users/... or /unix/path")
     typer.echo("")
 
     # Post-write: validate config and warn if client not running
@@ -3976,7 +3976,7 @@ def mcp_status() -> None:
     typer.echo(sep)
     typer.echo("  Note:    'configured' and 'running' are checked independently.")
     typer.echo("           A running app still needs restart after first-time config.")
-    typer.echo("  Path:    repo_path debe usar forward slashes: C:/Users/... o /ruta/unix")
+    typer.echo("  Path:    repo_path must use forward slashes: C:/Users/... or /unix/path")
     typer.echo("  Setup:   sourcecode mcp init")
     typer.echo("  Remove:  sourcecode mcp remove")
 

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/context_summarizer.py

@@ -28,6 +28,7 @@ _STACK_LABELS: dict[str, str] = {
 
 _TYPE_LABELS: dict[str, str] = {
     "api": "REST API",
+    "web_mvc": "Spring MVC web app",
     "webapp": "Web app",
     "fullstack": "Full-stack app",
     "cli": "CLI tool",

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/dependency_analyzer.py

@@ -1118,7 +1118,7 @@ class DependencyAnalyzer:
         try:
             tree = ET.parse(pom)
         except (ET.ParseError, OSError):
-            return [], ["java: error al parsear pom.xml"]
+            return [], ["java: error parsing pom.xml"]
 
         root_elem = tree.getroot()
         ns_match = re.match(r"\{[^}]+\}", root_elem.tag)
@@ -1217,10 +1217,10 @@ class DependencyAnalyzer:
                 try:
                     content = gradle_file.read_text(encoding="utf-8", errors="replace")
                 except OSError:
-                    return [], [f"gradle: error al leer {filename}"]
+                    return [], [f"gradle: error reading {filename}"]
                 props = self._parse_gradle_properties(root, content)
                 records = self._parse_gradle_dependencies(content, props, filename)
-                return records, ["gradle: sin lockfile compatible; dependencias transitivas no disponibles"]
+                return records, ["gradle: no compatible lockfile found; transitive dependencies unavailable"]
         return [], []
 
     def _parse_gradle_properties(self, root: Path, content: str) -> dict[str, str]:

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/detectors/java.py

@@ -23,6 +23,7 @@ _MAX_ANNOTATION_ENTRY_POINTS = 1000
 _REST_CONTROLLER_RE = re.compile(r'@RestController\b')
 _MVC_CONTROLLER_RE = re.compile(r'@Controller\b')
 _REQUEST_MAPPING_RE = re.compile(r'@RequestMapping\b')
+_ANY_MAPPING_RE = re.compile(r'@(?:Request|Get|Post|Put|Delete|Patch)Mapping\b')
 _CONTROLLER_ADVICE_RE = re.compile(r'@ControllerAdvice\b')
 _WEB_FILTER_RE = re.compile(r'@WebFilter\b')
 _FILTER_BEAN_RE = re.compile(r'FilterRegistrationBean\b')
@@ -298,6 +299,10 @@ class JavaDetector(AbstractDetector):
             frameworks.append(FrameworkDetection(name="Jakarta EE", source=source))
         if "mybatis" in text:
             frameworks.append(FrameworkDetection(name="MyBatis", source=source))
+        if "thymeleaf" in text:
+            frameworks.append(FrameworkDetection(name="Thymeleaf", source=source))
+        if "freemarker" in text:
+            frameworks.append(FrameworkDetection(name="FreeMarker", source=source))
         if "spring-boot-starter-security" in text or "spring-security-core" in text:
             frameworks.append(FrameworkDetection(name="Spring Security", source=source))
         if "spring-boot-starter-data-jpa" in text or "spring-data-jpa" in text:
@@ -470,7 +475,7 @@ class JavaDetector(AbstractDetector):
                 path=rel_path, stack="java", kind="exception_handler",
                 source="annotation", confidence="medium",
             )]
-        if _MVC_CONTROLLER_RE.search(content) and _REQUEST_MAPPING_RE.search(content):
+        if _MVC_CONTROLLER_RE.search(content) and _ANY_MAPPING_RE.search(content):
             http_path_match = _HTTP_PATH_RE.search(content)
             http_path = http_path_match.group(1) if http_path_match else None
             verb_match = _REQUEST_METHOD_VERB_RE.search(content)

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/prepare_context.py

@@ -981,12 +981,18 @@ class TaskContextBuilder:
 
             dep_records, dep_summary = DependencyAnalyzer().analyze(self.root)
             primary_eco = stacks[0].stack if stacks else ""
-            direct = [
+            _direct_raw = [
                 d for d in dep_records
                 if d.scope != "transitive" and d.source in {"manifest", "lockfile"}
                 and (d.role or "unknown") in {"runtime", "parsing", "serialization", "observability", "infra"}
                 and d.scope not in {"dev"}
             ]
+            _seen_dep: set[str] = set()
+            direct = []
+            for _d in _direct_raw:
+                if _d.name not in _seen_dep:
+                    _seen_dep.add(_d.name)
+                    direct.append(_d)
             # Rank by framework centrality: core infra (ORM, Spring) > serialization > other.
             # Penalise vendored tooling (closure-compiler, shaded utilities) so that
             # Hibernate/JPA/Solr appear before minor build-time dependencies.

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/repository_ir.py

@@ -1697,6 +1697,9 @@ def _canonical_subsystem_pkg(fqn: str) -> str:
     (even if uppercase) to force at least 2-segment grouping.
     """
     _TOP_LEVEL = {"com", "org", "net", "io", "java", "javax"}
+    # Well-known framework namespaces that are not application boundaries;
+    # go one level deeper (depth 5) so callers get the actual app module.
+    _FRAMEWORK_NS = {"springframework", "apache", "eclipse", "google", "jetbrains"}
     parts: list[str] = []
     for segment in fqn.split("."):
         if "#" in segment or (segment and segment[0].isupper()):
@@ -1705,6 +1708,8 @@ def _canonical_subsystem_pkg(fqn: str) -> str:
     if not parts:
         return fqn.rsplit(".", 1)[0] if "." in fqn else fqn
     if parts[0] in _TOP_LEVEL and len(parts) >= 3:
+        if len(parts) >= 5 and len(parts) > 3 and parts[1] in _FRAMEWORK_NS:
+            return ".".join(parts[:5])
         return ".".join(parts[:3])
     # Prevent bare TLD collapse: "org" or "com" alone as subsystem key is meaningless
     # and groups ALL classes under that TLD into a single giant component.
@@ -3338,7 +3343,7 @@ def compute_blast_radius(
             _seen_mapper_fqns.add(fqn)
             _mapper_entry: dict = {
                 "fqn": fqn,
-                "role": role or "mapper",
+                "role": role or ("mapper" if symbol_kind == "mapper_interface" else "repository"),
                 "source_file": node_dict.get("source_file") or "",
             }
             if canonical != fqn:

{sourcecode-1.32.0 → sourcecode-1.32.1}/src/sourcecode/summarizer.py

@@ -279,6 +279,7 @@ class ProjectSummarizer:
     _TYPE_LABELS: dict[str, str] = {
         "cli": "CLI",
         "api": "API",
+        "web_mvc": "Spring MVC web app",
         "webapp": "Web application",
         "library": "Library",
         "monorepo": "Monorepo",