sourcecode 1.31.2__py3-none-any.whl → 1.31.4__py3-none-any.whl

sourcecode/__init__.py

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

sourcecode/cli.py

@@ -1849,6 +1849,11 @@ def prepare_context_cmd(
         "--include-config",
         help="(generate-tests) Include tooling config files (*.conf.js, .eslintrc*, etc.) in test_gaps. Excluded by default.",
     ),
+    all_gaps: bool = typer.Option(
+        False,
+        "--all",
+        help="(generate-tests) Return the full test_gaps list without truncating to top 20.",
+    ),
 ) -> None:
     """Task-specific context for AI coding agents.
 
@@ -1930,7 +1935,7 @@ def prepare_context_cmd(
             _sys.stderr.flush()
     _t0 = _time.perf_counter()
     try:
-        output = builder.build(task, since=since, symptom=symptom, fast=fast, include_config=include_config)
+        output = builder.build(task, since=since, symptom=symptom, fast=fast, include_config=include_config, all_gaps=all_gaps)
     finally:
         _progress.finish()
     _t_total = (_time.perf_counter() - _t0) * 1000
@@ -1957,8 +1962,8 @@ def prepare_context_cmd(
         },
         "explain": {
             "project_summary": True, "architecture_summary": True,
-            "relevant_files": True, "key_dependencies": True,
-            "gaps": True, "confidence": True,
+            "relevant_files": False, "key_dependencies": True,
+            "gaps": False, "confidence": True,
             "suspected_areas": False, "improvement_opportunities": False,
             "test_gaps": False, "code_notes_summary": False,
             "changed_files": False, "affected_entry_points": False,
@@ -2467,6 +2472,11 @@ def _extract_java_endpoints(root: "Path") -> "dict[str, Any]":
     _CLASS_PATH_RE = _re.compile(
         r'@RequestMapping\s*\(\s*(?:value\s*=\s*)?["\']([^"\']+)["\']',
     )
+    # Handles array syntax: @RequestMapping({"/v1/foo", "/v1/bar"})
+    _CLASS_ARRAY_PATH_RE = _re.compile(r'@RequestMapping\s*\(\s*\{([^}]*)\}')
+    _QUOTED_STR_RE = _re.compile(r'"([^"]+)"')
+    _REQUEST_METHOD_VERB_RE = _re.compile(r'method\s*=\s*RequestMethod\.([A-Z]+)')
+    _VALUE_PATH_RE = _re.compile(r'value\s*=\s*"([^"]+)"')
 
     _HTTP_METHOD_MAP = {
         "Get": "GET", "Post": "POST", "Put": "PUT",
@@ -2502,27 +2512,66 @@ def _extract_java_endpoints(root: "Path") -> "dict[str, Any]":
         cls_m = _CLASS_RE.search(content)
         class_name = cls_m.group(1) if cls_m else java_file.stem
 
-        # Extract class-level base path from @RequestMapping on the class
-        class_base = ""
+        # Extract class-level base path and locate class body start
         lines = content.splitlines()
+
+        # First pass: find class/interface declaration line index
+        class_body_start = 0
         for i, line in enumerate(lines):
-            if "@RequestMapping" in line and i < len(lines) - 1:
-                # Check if next non-blank line is class declaration or it's on same block
-                block = "\n".join(lines[max(0, i - 1): i + 5])
+            stripped_l = line.strip()
+            if (not stripped_l.startswith("//") and not stripped_l.startswith("*")
+                    and ("class " in stripped_l or "interface " in stripped_l)
+                    and _CLASS_RE.search(stripped_l)):
+                class_body_start = i + 1
+                break
+
+        # Second pass: extract class-level @RequestMapping path (only before class body).
+        # Supports both single-string and array syntax (Bug #1B).
+        search_end = class_body_start if class_body_start else len(lines)
+        class_bases: list[str] = [""]  # default: no prefix
+        for i in range(search_end):
+            sl = lines[i].strip()
+            if sl.startswith("//") or sl.startswith("*"):
+                continue
+            if "@RequestMapping" in lines[i]:
+                # Cap block to search_end so method annotations inside the class body
+                # cannot be matched as the class-level prefix (Bug #1B).
+                block = "\n".join(lines[max(0, i - 1): min(i + 5, search_end + 1)])
                 if "class " in block or "interface " in block:
                     path_m = _CLASS_PATH_RE.search(block)
                     if path_m:
-                        class_base = path_m.group(1).rstrip("/")
-                    break
-
-        # Extract method-level endpoints
-        # Parse line-by-line to associate annotations with methods
+                        class_bases = [path_m.group(1).rstrip("/")]
+                    else:
+                        arr_m = _CLASS_ARRAY_PATH_RE.search(block)
+                        if arr_m:
+                            paths = _QUOTED_STR_RE.findall(arr_m.group(1))
+                            if paths:
+                                class_bases = [p.rstrip("/") for p in paths]
+                break
+
+        # Extract method-level endpoints starting from inside class body.
+        # Bug #1A fix: track block comments and skip // lines before annotation checks.
         pending_annotations: list[tuple[str, str]] = []  # (http_verb, path_suffix)
         pending_filtro: Optional[str] = None
+        in_block_comment = False
 
-        for i, line in enumerate(lines):
+        for i in range(class_body_start, len(lines)):
+            line = lines[i]
             stripped = line.strip()
 
+            # Block comment state tracking (Bug #1A)
+            if in_block_comment:
+                if "*/" in stripped:
+                    in_block_comment = False
+                continue
+            if stripped.startswith("/*"):
+                if "*/" not in stripped:
+                    in_block_comment = True
+                continue
+            # Skip line comments (Bug #1A)
+            if stripped.startswith("//"):
+                continue
+
             # Check for @M3FiltroSeguridad
             fm = _FILTRO_RE.search(stripped)
             if fm:
@@ -2535,6 +2584,19 @@ def _extract_java_endpoints(root: "Path") -> "dict[str, Any]":
                 verb_key = hm.group(1)
                 http_verb = _HTTP_METHOD_MAP.get(verb_key, "GET")
                 path_suffix = (hm.group(2) or "").strip()
+
+                # For @RequestMapping: resolve HTTP method from method= attribute
+                if verb_key == "Request":
+                    annotation_block = "\n".join(lines[i:min(i + 5, len(lines))])
+                    vm = _REQUEST_METHOD_VERB_RE.search(annotation_block)
+                    if vm:
+                        http_verb = vm.group(1)
+                    # If path not captured (method= precedes value=), try value= extraction
+                    if not path_suffix:
+                        vp = _VALUE_PATH_RE.search(annotation_block)
+                        if vp:
+                            path_suffix = vp.group(1)
+
                 pending_annotations.append((http_verb, path_suffix))
                 continue
 
@@ -2544,30 +2606,30 @@ def _extract_java_endpoints(root: "Path") -> "dict[str, Any]":
                 handler = mm.group(1) if mm else ""
                 if handler and not handler.startswith("class"):
                     for http_verb, path_suffix in pending_annotations:
-                        full_path = (class_base + "/" + path_suffix).replace("//", "/").rstrip("/") or "/"
-                        if not full_path.startswith("/"):
-                            full_path = "/" + full_path
-                        key = (class_name, handler, http_verb)
-                        if key not in seen:
-                            seen.add(key)
-                            entry: dict[str, Any] = {
-                                "method": http_verb,
-                                "path": full_path,
-                                "controller": class_name,
-                                "handler": handler,
-                            }
-                            if pending_filtro:
-                                entry["required_permission"] = pending_filtro
-                            endpoints.append(entry)
+                        for _cb in class_bases:  # Bug #1B: one endpoint per class prefix
+                            full_path = (_cb + "/" + path_suffix).replace("//", "/").rstrip("/") or "/"
+                            if not full_path.startswith("/"):
+                                full_path = "/" + full_path
+                            key = (class_name, handler, http_verb, _cb)
+                            if key not in seen:
+                                seen.add(key)
+                                entry: dict[str, Any] = {
+                                    "method": http_verb,
+                                    "path": full_path,
+                                    "controller": class_name,
+                                    "handler": handler,
+                                }
+                                if pending_filtro:
+                                    entry["required_permission"] = pending_filtro
+                                endpoints.append(entry)
                     pending_annotations = []
                     pending_filtro = None
                     continue
 
-            # Non-annotation, non-method line — reset if it's a closing brace or blank
-            if stripped in ("}", "{", "") or stripped.startswith("//") or stripped.startswith("*"):
-                if stripped == "}":
-                    pending_annotations = []
-                    pending_filtro = None
+            # Non-annotation, non-method line — reset on closing brace
+            if stripped == "}":
+                pending_annotations = []
+                pending_filtro = None
 
     endpoints.sort(key=lambda e: (e.get("controller", ""), e.get("path", "")))
     undocumented = sum(1 for e in endpoints if "required_permission" not in e)

sourcecode/prepare_context.py

@@ -721,7 +721,7 @@ class TaskContextBuilder:
     def __init__(self, root: Path) -> None:
         self.root = root
 
-    def build(self, task_name: str, *, since: Optional[str] = None, symptom: Optional[str] = None, fast: bool = False, include_config: bool = False) -> TaskOutput:
+    def build(self, task_name: str, *, since: Optional[str] = None, symptom: Optional[str] = None, fast: bool = False, include_config: bool = False, all_gaps: bool = False) -> TaskOutput:
         if task_name not in TASKS:
             raise ValueError(
                 f"Unknown task '{task_name}'. Available: {', '.join(TASKS)}"
@@ -1763,14 +1763,18 @@ class TaskContextBuilder:
                         "_rank": _pub_count + _ann_count * 2,
                     })
 
-                _java_candidates.sort(key=lambda x: -x["_rank"])
+                _java_candidates.sort(
+                    key=lambda x: -(x["public_method_count"] * (1.5 if x["has_spring_annotations"] else 1.0))
+                )
+                _top = _java_candidates if all_gaps else _java_candidates[:20]
                 test_gaps = [
                     {
                         "path": c["path"],
                         "public_method_count": c["public_method_count"],
                         "has_spring_annotations": c["has_spring_annotations"],
+                        "rank_score": round(c["public_method_count"] * (1.5 if c["has_spring_annotations"] else 1.0), 1),
                     }
-                    for c in _java_candidates
+                    for c in _top
                 ]
             else:
                 # Non-Java algorithm (unchanged)
@@ -2206,8 +2210,67 @@ class TaskContextBuilder:
                 "refactor": max(15, min(30, _repo_size // 120)),
             }
             _budget = _task_budget.get(task_name, 15)
-            _selected = _ctx.select_subgraph(_ns, contracts=[], budget=_budget, min_score=0.15)
+            _selected = list(_ctx.select_subgraph(_ns, contracts=[], budget=_budget, min_score=0.15))
             _rf_map = {path: rf for _, path, rf in scored}
+
+            # Bug #3: onboard must cover ≥3 arch layers (controllers/services/domain/repositories).
+            if task_name == "onboard":
+                def _arch_layer(p: str) -> str:
+                    n = Path(p).name.lower()
+                    if "controller" in n:
+                        return "controllers"
+                    if "repository" in n or "mapper" in n or "dao" in n:
+                        return "repositories"
+                    if "service" in n:
+                        return "services"
+                    pn = p.replace("\\", "/")
+                    if "entity" in n or "/entity/" in pn or "/domain/" in pn or "/model/" in pn:
+                        return "domain"
+                    return "other"
+
+                _REQUIRED = {"controllers", "services", "repositories", "domain"}
+                _covered = {_arch_layer(p) for p in _selected} & _REQUIRED
+                _missing = _REQUIRED - _covered
+                if len(_covered) < 3 and _missing:
+                    _sel_set = set(_selected)
+                    # First pass: inject from already-scored files
+                    for _, _p, _ in sorted(scored, key=lambda x: -x[0]):
+                        if len(_covered) >= 3:
+                            break
+                        if _p in _sel_set or _p not in _rf_map:
+                            continue
+                        _layer = _arch_layer(_p)
+                        if _layer in _missing:
+                            _selected.append(_p)
+                            _sel_set.add(_p)
+                            _covered.add(_layer)
+                            _missing.discard(_layer)
+                    # Second pass: fallback scan of all_paths when scored files
+                    # don't cover enough layers (e.g. all Java files scored ≤ 0
+                    # due to auxiliary/example package detection).
+                    if len(_covered) < 3 and _missing:
+                        _NON_TEST = ("/test/", "/tests/", "/spec/")
+                        for _p in all_paths:
+                            if len(_covered) >= 3:
+                                break
+                            if _p in _sel_set:
+                                continue
+                            if any(s in _p.replace("\\", "/") for s in _NON_TEST):
+                                continue
+                            _layer = _arch_layer(_p)
+                            if _layer not in _missing:
+                                continue
+                            _rf_map[_p] = RelevantFile(
+                                path=_p,
+                                role="source",
+                                score=0.1,
+                                reason="layer coverage (onboard)",
+                            )
+                            _selected.append(_p)
+                            _sel_set.add(_p)
+                            _covered.add(_layer)
+                            _missing.discard(_layer)
+
             return [_rf_map[p] for p in _selected if p in _rf_map]
         except Exception:
             return [f for _, _, f in scored[:15]]

{sourcecode-1.31.2.dist-info → sourcecode-1.31.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sourcecode
-Version: 1.31.2
+Version: 1.31.4
 Summary: Deterministic codebase context for AI coding agents
 License:                                  Apache License
                                    Version 2.0, January 2004
@@ -225,7 +225,7 @@ Description-Content-Type: text/markdown
 
 **Deterministic, behavior-aware codebase context for AI agents and PR review.**
 
-![Version](https://img.shields.io/badge/version-1.31.2-blue)
+![Version](https://img.shields.io/badge/version-1.31.4-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---
@@ -261,7 +261,7 @@ pipx install sourcecode
 
 ```bash
 sourcecode version
-# sourcecode 1.31.2
+# sourcecode 1.31.4
 ```
 
 ---

{sourcecode-1.31.2.dist-info → sourcecode-1.31.4.dist-info}/RECORD

@@ -1,10 +1,10 @@
-sourcecode/__init__.py,sha256=BlIGNpIZ2KvAHJhMqzRGPET0plDw5AYOTlGs0F8h4hU,103
+sourcecode/__init__.py,sha256=P8MbrmTVGyjN1Uy_38VxYTdrjIoEPOf9I4Xn71cCRs8,103
 sourcecode/adaptive_scanner.py,sha256=XffluXKzJUXrMtjEiAOnSNPZnztdIcts17T9ouHeID0,10521
 sourcecode/architecture_analyzer.py,sha256=MyBa0Hf5HmkudZQDLKrjcWDKETXETXl0mQX1swtTwAA,39091
 sourcecode/architecture_summary.py,sha256=z34_6v7cSwy98cof2UVciGho7SCrZ93tiqMmq5WNzRQ,20405
 sourcecode/ast_extractor.py,sha256=XgrZg2DcWcUm9r87cRG3KGO7IK2TIL_N-CvhSbUmmh4,49901
 sourcecode/classifier.py,sha256=-0t0HLc9L9UleMLfclfLM3AXhBjUb_AYyBPDbvgWtac,7755
-sourcecode/cli.py,sha256=qYMuo7DY6erJD5HwjU1Rot07xR54kXil4YzVytyU1Po,120897
+sourcecode/cli.py,sha256=dcHIYDvDpcIzbExxKuemtElk3RClLRWaaO2RU9R8Hoc,123945
 sourcecode/code_notes_analyzer.py,sha256=y1MJBnPZHYp4i6cQCXUb9ATIyifS_qMQWjw_8lPkpsU,9215
 sourcecode/confidence_analyzer.py,sha256=ZUn-Nywi5TEQcuozqK_vfOyPT-a1dYYO42elAtVFV-k,16412
 sourcecode/context_scorer.py,sha256=QpChSpsmaAYz91rXA4Ue5xzQmNz_ZboZN09YOHScq1U,14679
@@ -22,7 +22,7 @@ sourcecode/git_analyzer.py,sha256=0Gyj-vMpIIN4nfriKXVRouNYBeJ59s6pQDX2Xu9Pq-U,13
 sourcecode/graph_analyzer.py,sha256=iUK-7pSV-cvGqqD2hENdYmhnm0wcXFEyK-xnu5ul8OU,62515
 sourcecode/metrics_analyzer.py,sha256=m0ENgtqKeBL17kUIK3fmGkgo7UfXBNHxCMj0H_Y5K7c,22750
 sourcecode/pr_comment_renderer.py,sha256=smHslxiG14lrytCkq5nFrFu-qTHgA-t-LFYfdrfjz2o,14423
-sourcecode/prepare_context.py,sha256=vmZnoFneipRCE-hP6drvhqNXb3FwLrVub449e0MzT0Y,179908
+sourcecode/prepare_context.py,sha256=Eid3Wmh4lmtr20pmobcCU0yzjRXhFql2rEmfGUgkFpU,183214
 sourcecode/progress.py,sha256=qn30sWaHOkjTgXsSBmiPkz7Rsbwc5oSlIe6JNEMYp_k,3149
 sourcecode/ranking_engine.py,sha256=ZAucq_YX2KkWUuAZf4P0lhtQ_38vEFnUhuGtSZd1S0E,12970
 sourcecode/redactor.py,sha256=xuGcadGEHaPw4qZXlMDvzMCsr4VOkdp3oBQptHyJk8c,2884
@@ -72,8 +72,8 @@ sourcecode/telemetry/consent.py,sha256=wLMvGNJeSSyZoNkQXpoUioY6mMv4Qdvuw7S9jAEWn
 sourcecode/telemetry/events.py,sha256=oEvvulfsv5GIDWG2174gSS6tNB95w38AIYiYeifGKlE,2294
 sourcecode/telemetry/filters.py,sha256=Asa71oRl7q3Wt_FMwuufIZJFzSYdgRNKS8LHCIyFeYE,4805
 sourcecode/telemetry/transport.py,sha256=KJeIPCPWMdmbCP3ySGs2iUlia34U6vWne2dZsUezesw,1560
-sourcecode-1.31.2.dist-info/METADATA,sha256=CaHPC5ti8mv3vU8hOaq1vV5lvAHw8kPIH4IEvbk3x2k,29083
-sourcecode-1.31.2.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
-sourcecode-1.31.2.dist-info/entry_points.txt,sha256=ex3F9rmbXeyDIoFQHtkEqTsKSaJow8F0LrVu8XfIktQ,57
-sourcecode-1.31.2.dist-info/licenses/LICENSE,sha256=7DdHrU9Z_3e7dSvq4ISijZNjnuHo5NIHNiHDouMQ9JU,10491
-sourcecode-1.31.2.dist-info/RECORD,,
+sourcecode-1.31.4.dist-info/METADATA,sha256=JZXcuZYPozOjtbIY_xnVgUNwMF1aEU12mV4m2T__E1A,29083
+sourcecode-1.31.4.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
+sourcecode-1.31.4.dist-info/entry_points.txt,sha256=ex3F9rmbXeyDIoFQHtkEqTsKSaJow8F0LrVu8XfIktQ,57
+sourcecode-1.31.4.dist-info/licenses/LICENSE,sha256=7DdHrU9Z_3e7dSvq4ISijZNjnuHo5NIHNiHDouMQ9JU,10491
+sourcecode-1.31.4.dist-info/RECORD,,