roam-code 12.32__tar.gz → 12.34__tar.gz

{roam_code-12.32 → roam_code-12.34}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: roam-code
-Version: 12.32
+Version: 12.34
 Summary: Instant codebase comprehension for AI coding agents
 Author: CosmoHac
 License-Expression: Apache-2.0

{roam_code-12.32 → roam_code-12.34}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "roam-code"
-version = "12.32"
+version = "12.34"
 description = "Instant codebase comprehension for AI coding agents"
 readme = "README.md"
 requires-python = ">=3.9"

{roam_code-12.32 → roam_code-12.34}/src/roam/commands/cmd_describe.py

@@ -353,7 +353,6 @@ _DOMAIN_STOP_WORDS = frozenset(
         "context",
         "hook",
         # DOM events / UI interactions
-        "click",
         "focus",
         "blur",
         "hover",

{roam_code-12.32 → roam_code-12.34}/src/roam/commands/cmd_n1.py

@@ -39,13 +39,11 @@ from roam.output.formatter import json_envelope, loc, to_json
 
 # Patterns that identify ORM model classes by parent class or trait
 _MODEL_PARENTS = {
-    # Laravel / Eloquent
+    # Laravel / Eloquent / Django (both use bare "Model" — single membership)
     "Model",
     "Eloquent",
     "Authenticatable",
-    # Django
     "models.Model",
-    "Model",
     # Rails / ActiveRecord
     "ApplicationRecord",
     "ActiveRecord::Base",
@@ -129,7 +127,6 @@ _RELATIONSHIP_CALLS = {
     "load",
     "fetch",
     "findOrFail",
-    "find",
 }
 
 # I/O operations that indicate a DB query or network call
@@ -154,7 +151,6 @@ _IO_INDICATORS = {
     # HTTP
     "fetch",
     "request",
-    "get",
     "post",
     "put",
     # File

{roam_code-12.32 → roam_code-12.34}/src/roam/commands/cmd_orphan_routes.py

@@ -468,17 +468,20 @@ def _analyse_orphan_routes(project_root: Path, conn, limit: int) -> dict:
         # Deduplicate
         all_matching = list(set(all_matching))
 
-        # Remove the route files themselves and the controller file from matches
+        # Remove the route files themselves and the controller file from matches.
+        # `_ctrl` and `_route_files` are bound as defaults so the closure
+        # captures THIS iteration's values (per-loop B023 fix).
         controller_name = route.get("controller") or ""
 
-        def _is_self_reference(file_path: str) -> bool:
+        def _is_self_reference(
+            file_path: str,
+            _ctrl: str = controller_name,
+            _route_files: set = route_file_prefixes,
+        ) -> bool:
             norm = file_path.replace("\\", "/")
-            # Exclude the route definition files
-            if norm in route_file_prefixes:
+            if norm in _route_files:
                 return True
-            # Exclude the controller file itself
-            if controller_name and controller_name.lower() in norm.lower():
-                # Check it actually looks like a controller path
+            if _ctrl and _ctrl.lower() in norm.lower():
                 if "controller" in norm.lower() or "http" in norm.lower():
                     return True
             return False

{roam_code-12.32 → roam_code-12.34}/src/roam/commands/resolve.py

@@ -414,12 +414,14 @@ def find_symbol_with_alternatives(conn: sqlite3.Connection, name: str) -> tuple[
 
         signals = _ambiguity_signals(conn, rows)
 
-        def _score(r):
+        # Bind `signals` as default so the closure captures THIS iteration's
+        # value (avoids the late-binding-loop closure pitfall flagged by B023).
+        def _score(r, _sig=signals):
             return (
-                signals["ref"].get(r["id"], 0),
-                signals["pr"].get(r["id"], 0),
-                signals["cc"].get(r["id"], 0),
-                signals["churn"].get(r["file_id"], 0) if "file_id" in r.keys() else 0,
+                _sig["ref"].get(r["id"], 0),
+                _sig["pr"].get(r["id"], 0),
+                _sig["cc"].get(r["id"], 0),
+                _sig["churn"].get(r["file_id"], 0) if "file_id" in r.keys() else 0,
                 _path_rank(r["file_path"]),
                 -r["id"],
             )

{roam_code-12.32 → roam_code-12.34}/src/roam/competitor_site_data.py

@@ -1362,8 +1362,8 @@ MAP_METADATA: dict[str, dict[str, object]] = {
         "relationship": "self",
         "peer": True,
         "graph": "PageRank + Tarjan + Louvain + layers",
-        "note": "Graph algorithms (PageRank, SCC, Louvain, Fiedler) on tree-sitter ASTs fused with git history in SQLite. 136 MCP tools, 187 CLI commands. 19 Python idiom detectors (v12.7+). 54 algo detectors (12.32).",
-        "version_evaluated": "12.32",
+        "note": "Graph algorithms (PageRank, SCC, Louvain, Fiedler) on tree-sitter ASTs fused with git history in SQLite. 136 MCP tools, 187 CLI commands. 19 Python idiom detectors (v12.7+). 54 algo detectors (12.34).",
+        "version_evaluated": "12.34",
         "repo_url": "https://github.com/Cranot/roam-code",
     },
     "CKB/CodeMCP": {

{roam_code-12.32 → roam_code-12.34}/src/roam/graph/clone_detect.py

@@ -158,29 +158,19 @@ def _jaccard_bags(a: Counter, b: Counter) -> float:
 
 _FUNCTION_NODE_TYPES = frozenset(
     {
-        # Python
+        # Python / C / C++ all share `function_definition`
         "function_definition",
-        # JS/TS
+        # JS/TS / Swift share `function_declaration`
         "function_declaration",
         "method_definition",
         "arrow_function",
-        # Java/C#/Kotlin/Scala
+        # Java/C#/Kotlin/Scala / Go / PHP all share `method_declaration`
         "method_declaration",
         "constructor_declaration",
-        # Go
-        "function_declaration",
-        "method_declaration",
         # Rust
         "function_item",
-        # C/C++
-        "function_definition",
         # Ruby
         "method",
-        # PHP
-        "function_definition",
-        "method_declaration",
-        # Swift
-        "function_declaration",
     }
 )
 

{roam_code-12.32 → roam_code-12.34}/src/roam/index/complexity.py

@@ -28,20 +28,13 @@ _CONTROL_FLOW = {
     "with_statement",
     "match_statement",
     # JS/TS
-    "if_statement",
-    "for_statement",
     "for_in_statement",
-    "while_statement",
     "do_statement",
     "switch_statement",
-    "try_statement",
     "catch_clause",
     # Java/C#/Go/Rust
-    "for_statement",
     "enhanced_for_statement",
     "foreach_statement",
-    "while_statement",
-    "do_statement",
     "if_expression",
     "match_expression",
     # General

{roam_code-12.32 → roam_code-12.34}/src/roam/languages/foxpro_lang.py

@@ -329,8 +329,6 @@ _VFP_BUILTINS = frozenset(
         "TRIM",
         "UPPER",
         # ── Date / Time ──
-        "CDOW",
-        "CMONTH",
         "CTOD",
         "CTOT",
         "DATE",

{roam_code-12.32 → roam_code-12.34}/src/roam/languages/hcl_lang.py

@@ -359,16 +359,18 @@ class HclExtractor(LanguageExtractor):
                 if mb:
                     current_block = mb.group(2)
 
-            def _add(target: str):
-                key = (target, str(ln))
+            # B023: bind ln + current_block as defaults so the closure
+            # captures THIS iteration's values (loop-variable late-binding fix).
+            def _add(target: str, _ln: int = ln, _cur: str | None = current_block):
+                key = (target, str(_ln))
                 if key not in seen:
                     seen.add(key)
                     refs.append(
                         self._make_reference(
                             target_name=target,
                             kind="call",
-                            line=ln,
-                            source_name=current_block,
+                            line=_ln,
+                            source_name=_cur,
                         )
                     )
 

{roam_code-12.32 → roam_code-12.34}/src/roam/mcp-server-card.json

@@ -2,7 +2,7 @@
   "$schema": "https://schemas.modelcontextprotocol.io/server-card/v1",
   "name": "roam-code",
   "display_name": "roam — instant codebase intelligence",
-  "version": "12.32",
+  "version": "12.34",
   "description": "Architectural sight for AI agents before they edit. Pre-indexes symbols, call graphs, dependencies, architecture layers, and git history into a local SQLite DB. 136 MCP tools, 10 resources, 5 prompts, 27 languages. 100% local, zero API keys.",
   "vendor": {
     "name": "Cranot",

{roam_code-12.32 → roam_code-12.34}/src/roam/retrieve/seeds.py

@@ -306,8 +306,6 @@ _NL_EXTRA_STOPWORDS = frozenset(
         "often",
         "actually",
         "though",
-        "until",
-        "while",
     }
 )
 

{roam_code-12.32 → roam_code-12.34}/src/roam/rules/engine.py

@@ -41,11 +41,40 @@ def _load_yaml(path: Path) -> dict | None:
         return None
 
 
+def _coerce_scalar(val: str) -> object:
+    """Best-effort YAML scalar coercion: bool / int / float / quoted string."""
+    val = val.strip().strip('"').strip("'")
+    if val.lower() == "true":
+        return True
+    if val.lower() == "false":
+        return False
+    try:
+        return int(val)
+    except ValueError:
+        try:
+            return float(val)
+        except ValueError:
+            return val
+
+
 def _parse_simple_yaml(path: Path) -> dict | None:
     """Minimal YAML subset parser for rule files (no PyYAML dependency).
 
-    Handles flat key-value pairs, lists as ``[a, b]``, and nested maps
-    introduced by indented keys under a parent key.
+    Handles:
+    * flat key-value pairs
+    * inline lists ``[a, b, c]``
+    * nested maps under an indented key
+    * list-of-dicts shape introduced by ``- key: value`` items (the
+      ``rules:`` block in rules.yml is the canonical use)
+
+    12.34 (2026-05-06) — added list-of-dicts support; CI 12.33 failed
+    on Python 3.9 (PyYAML not installed) because
+    ``test_load_rules_yaml_simple`` uses that shape.
+
+    Frames track ``(indent, container, kind, parent_key)``. When we hit
+    a ``- key: value`` item and the current frame is an empty dict, we
+    promote it to a list inside the parent container at the recorded
+    ``parent_key``, then push a new dict frame for the item.
     """
     try:
         text = path.read_text(encoding="utf-8")
@@ -53,52 +82,80 @@ def _parse_simple_yaml(path: Path) -> dict | None:
         return None
 
     result: dict = {}
-    stack: list[tuple[int, dict]] = [(0, result)]
+    # Frame: (indent, container, kind, parent_dict, parent_key)
+    # The root frame has parent_dict=None / parent_key=None.
+    stack: list[tuple[int, object, str, object, object]] = [(0, result, "dict", None, None)]
 
-    for line in text.split("\n"):
-        stripped = line.strip()
+    for raw in text.split("\n"):
+        stripped = raw.strip()
         if not stripped or stripped.startswith("#"):
             continue
 
-        indent = len(line) - len(line.lstrip())
+        indent = len(raw) - len(raw.lstrip())
 
-        # Pop stack to matching indent
         while len(stack) > 1 and indent < stack[-1][0]:
             stack.pop()
 
+        if stripped.startswith("- "):
+            after_dash = stripped[2:].strip()
+            top = stack[-1]
+            top_indent, top_container, top_kind, top_parent, top_pkey = top
+
+            # If current frame is an empty dict that was opened by a parent
+            # `key:` (no value), promote it to a list and replace it in the
+            # parent. This is how `rules:\n  - id: ...` becomes
+            # `{"rules": [{"id": ...}]}` instead of `{"rules": {"- id": ...}}`.
+            if (
+                top_kind == "dict"
+                and isinstance(top_container, dict)
+                and not top_container
+                and top_parent is not None
+                and top_pkey is not None
+            ):
+                new_list: list = []
+                top_parent[top_pkey] = new_list
+                stack[-1] = (top_indent, new_list, "list", top_parent, top_pkey)
+                top_container = new_list
+                top_kind = "list"
+
+            if top_kind == "list" and isinstance(top_container, list):
+                if ":" in after_dash:
+                    new_item: dict = {}
+                    top_container.append(new_item)
+                    key, _, val = after_dash.partition(":")
+                    key = key.strip()
+                    if val.strip():
+                        new_item[key] = _coerce_scalar(val)
+                    else:
+                        new_item[key] = {}
+                    # Push the new item dict so subsequent same-indent keys
+                    # populate it. Indent of children = indent + 2.
+                    stack.append((indent + 2, new_item, "dict", top_container, len(top_container) - 1))
+                else:
+                    top_container.append(_coerce_scalar(after_dash))
+            continue
+
         if ":" not in stripped:
             continue
 
         key, _, val = stripped.partition(":")
         key = key.strip()
+        val_raw = val
         val = val.strip()
 
-        current = stack[-1][1]
+        _i, container, kind, _pp, _pk = stack[-1]
+        if kind != "dict" or not isinstance(container, dict):
+            continue
 
         if not val:
-            # Start a nested dict
             child: dict = {}
-            current[key] = child
-            stack.append((indent + 2, child))
+            container[key] = child
+            stack.append((indent + 2, child, "dict", container, key))
         elif val.startswith("[") and val.endswith("]"):
-            # Inline list
             items = [v.strip().strip('"').strip("'") for v in val[1:-1].split(",") if v.strip()]
-            current[key] = items
+            container[key] = items
         else:
-            val = val.strip('"').strip("'")
-            if val.lower() == "true":
-                parsed_val: object = True
-            elif val.lower() == "false":
-                parsed_val = False
-            else:
-                try:
-                    parsed_val = int(val)
-                except ValueError:
-                    try:
-                        parsed_val = float(val)
-                    except ValueError:
-                        parsed_val = val
-            current[key] = parsed_val
+            container[key] = _coerce_scalar(val_raw)
 
     return result if result else None
 

{roam_code-12.32 → roam_code-12.34}/src/roam/search/tfidf.py

@@ -71,7 +71,6 @@ _STOPWORDS = frozenset(
         "self",
         "return",
         "import",
-        "from",
         "def",
         "class",
         "function",

{roam_code-12.32 → roam_code-12.34}/src/roam_code.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: roam-code
-Version: 12.32
+Version: 12.34
 Summary: Instant codebase comprehension for AI coding agents
 Author: CosmoHac
 License-Expression: Apache-2.0

{roam_code-12.32 → roam_code-12.34}/tests/test_mcp_server.py

@@ -450,9 +450,10 @@ class TestExpandToolset:
         from roam.mcp_server import expand_toolset
 
         result = expand_toolset(preset="core")
-        # v12.0=27, v12.1=33 (+5 oracles +taint_classify), v12.6=35 (+py-pivot 2),
-        # v12.16=36 (+roam_catalog).
-        assert result["tool_count"] == 41
+        # v12.0=27, v12.1=33 (+5 oracles +taint_classify), v12.6=35
+        # (+py-pivot 2), v12.16=36 (+roam_catalog), v12.19=41
+        # (+5 agent wrappers), v12.28=49 (+8 Agent Review v2 tools).
+        assert result["tool_count"] == 49
 
     def test_invalid_preset(self):
         from roam.mcp_server import expand_toolset