@descryy/adapter-python 0.3.3 → 0.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@descryy/adapter-python",
-  "version": "0.3.3",
+  "version": "0.4.0",
   "type": "module",
   "description": "Python adapter. Parses with Python's own ast in one subprocess (DEC-062) — no grammar to keep in sync, and annotations arrive as source text.",
   "license": "UNLICENSED",
@@ -36,8 +36,8 @@
   },
   "dependencies": {
     "@descryy/adapter-common": "0.2.0",
-    "@descryy/ir": "0.6.3",
-    "@descryy/adapter-openapi": "0.3.3",
-    "@descryy/adapter-sql": "0.3.3"
+    "@descryy/ir": "0.7.0",
+    "@descryy/adapter-openapi": "0.4.0",
+    "@descryy/adapter-sql": "0.4.0"
   }
 }

package/src/extract.py

@@ -443,6 +443,10 @@ class Collector(ast.NodeVisitor):
                     "module": alias.name,
                     "name": None,
                     "local": alias.asname or alias.name.split(".")[0],
+                    # `import a.b.c` binds `a`; `import a.b.c as a` binds
+                    # `a.b.c`. Nothing in resolution can tell those apart from
+                    # `local` and `module`, and the dependency boundary has to.
+                    "asname": alias.asname,
                     "line": node.lineno,
                 }
             )
@@ -460,6 +464,7 @@ class Collector(ast.NodeVisitor):
                     "module": node.module,
                     "name": alias.name,
                     "local": alias.asname or alias.name,
+                    "asname": alias.asname,
                     "line": node.lineno,
                 }
             )
@@ -1295,6 +1300,43 @@ def _alembic_revision(tree: ast.Module) -> dict | None:
     return {"revision": revision, "downRevision": down_revision}
 
 
+def _assigned_names(tree: ast.AST) -> list[str]:
+    """Every name this file binds itself, anywhere in it.
+
+    The dependency boundary needs this and nothing else does: an import can be
+    shadowed (`import json` then `json = load_config()`), and an import table
+    cannot see the shadow. A name in here makes the Node side decline to
+    attribute a call through it -- see `external.ts`.
+
+    **File-level rather than scope-precise, deliberately.** A blunter set
+    refuses MORE sites, and rule 2 prices a wrong attribution far above a
+    missing one. Measured cost of the bluntness: 2 attributions of 4,838 on
+    dispatch, 1 of 4,543 on plane.
+
+    Import statements are not included -- they are the table this guards, not a
+    shadow of it.
+    """
+    names: set[str] = set()
+    for node in ast.walk(tree):
+        if isinstance(node, ast.Name) and isinstance(node.ctx, ast.Store):
+            # Assignment, `for`, `with ... as`, a walrus, a comprehension
+            # target, an augmented or annotated assignment -- all one case.
+            names.add(node.id)
+        elif isinstance(node, ast.arg):
+            names.add(node.arg)
+        elif isinstance(node, (ast.FunctionDef, ast.AsyncFunctionDef, ast.ClassDef)):
+            names.add(node.name)
+        elif isinstance(node, ast.ExceptHandler) and node.name:
+            names.add(node.name)
+        elif isinstance(node, ast.MatchAs) and node.name:
+            names.add(node.name)
+        elif isinstance(node, ast.MatchStar) and node.name:
+            names.add(node.name)
+        elif isinstance(node, ast.MatchMapping) and node.rest:
+            names.add(node.rest)
+    return sorted(names)
+
+
 def collect(path: str, source: str, deep_call_modules: frozenset[str] = frozenset()) -> dict:
     tree = ast.parse(source, filename=path)
     collector = Collector(path)
@@ -1347,6 +1389,7 @@ def collect(path: str, source: str, deep_call_modules: frozenset[str] = frozense
         )
 
     env_reads, env_dynamic_reads = _env_reads(tree, collector.imports)
+
     migration_ops = _migration_ops(tree, collector.imports)
     migration_revision = _alembic_revision(tree)
 
@@ -1366,6 +1409,7 @@ def collect(path: str, source: str, deep_call_modules: frozenset[str] = frozense
         "envDynamicReads": env_dynamic_reads,
         "migrationOps": migration_ops,
         "migrationRevision": migration_revision,
+        "assignedNames": _assigned_names(tree),
     }