fix(ci+compliance): ruff format + Dijkstra RED items (panel split, dead param)

CI went red on commit 21ba756 with ruff format check failing on 5 files
(ast_extractors.py, codebase_graph.py, workflow_graph_source_native_ast.py,
test_ap_bridge_enable_precedence.py, test_workflow_graph_source_native_ast.py).
`ruff format` applied — cosmetic only, no logic changes.

Dijkstra compliance audit (2026-04-24) flagged two RED items (block
merge per §10 High-stakes) that I introduced in the humanization chain:

**RED 1 — workflow_graph_panel.js at 579 LOC violated both global
§4.1 (500) and project (300) limits.** I added +141 LOC across 0135887
(humanization rows) and 21ba756 (ZPD raw-key bridge).

Split into two modules:
- `ui/unified/js/workflow_graph_panel.js` — framework primitives,
  `buildSidePanel`, plain-description + heat + stage + technical-
  details renderers. Exports `JUG._wfgPanelHelpers` for the renderer
  module to consume. **362 LOC** (under the global 500-line cap;
  above the project 300 cap — YELLOW, follow-up split planned).
- `ui/unified/js/workflow_graph_panel_renderers.js` — the ten
  per-kind `render<Kind>` functions + dispatch table under
  `JUG._wfgRenderers`. File and Discussion renderers additionally
  split into `_render<Kind>Identity/Domains/Relationships/...` helpers
  to stay under the project 40-line method rule. **332 LOC**.

Panel's `show()` now fetches the renderer via `rendererFor(kind)` which
delegates to `JUG._wfgRenderers.get(kind)`. Load order in
`unified-viz.html`: humanize → panel → panel_renderers, so the helper
registry + humanizer exist before the renderers are registered.

**RED 2 — dead `ctx` parameter.** `plainDescription(n, ctx)` in
humanize.js advertised graph-context lookup in its docstring but
never read `ctx`; `renderPlainDescription(body, n, ctx)` in panel.js
forwarded it. Both signatures tightened to drop the unused arg
(§9 anti-pattern enforcement).

Smoke-tested via Node eval: all three modules load in order, every
exported registry (wfg, humanize, renderers, panel helpers) is
present, all 10 renderers execute against a synthetic body/node/ctx
triple without error.

Status against Dijkstra audit:
- RED 1 (panel.js > 500 lines): CLEARED.
- RED 2 (dead ctx param): CLEARED.
- YELLOW (panel.js / humanize.js / renderers.js > 300 project cap,
  several method-size violations): documented, scheduled for a
  follow-up split that moves the neighbor helpers into their own
  module and breaks plainDescription/renderTechnicalDetails into
  per-kind describers.
- GREEN (pre-existing 300+ Python files inherited from earlier
  commits): noted, not in scope for this chain.

Author: cdeust

mcp_server/core/ast_extractors.py

@@ -301,9 +301,7 @@ def walk(node: Node, class_scope: str) -> None:
                 fn_name = _text(name_node, source) if name_node else ""
                 body = child.child_by_field_name("body") or child
                 if fn_name:
-                    qname = (
-                        f"{class_scope}.{fn_name}" if class_scope else fn_name
-                    )
+                    qname = f"{class_scope}.{fn_name}" if class_scope else fn_name
                     calls: list[str] = []
                     seen: set[str] = set()
                     for call_type in _CALL_NODE_TYPES:

mcp_server/core/codebase_graph.py

@@ -225,9 +225,7 @@ def build_resolved_call_edges(
                 if key in seen_pair:
                     continue
                 seen_pair.add(key)
-                edges.append(
-                    (analysis.path, caller_qname, target_file, target_qname)
-                )
+                edges.append((analysis.path, caller_qname, target_file, target_qname))
     return edges
 
 

mcp_server/infrastructure/workflow_graph_source_native_ast.py

@@ -74,9 +74,7 @@ def ast_available(self) -> bool:
         """Tree-sitter proper is installed (deeper extraction)."""
         return ast_available()
 
-    def load_symbols(
-        self, file_paths: Iterable[str]
-    ) -> list[dict[str, Any]]:
+    def load_symbols(self, file_paths: Iterable[str]) -> list[dict[str, Any]]:
         """Return one symbol row per function/class/method definition
         found in each readable file. Skips unreadable files silently.
         """
@@ -97,9 +95,7 @@ def load_symbols(
                 )
         return symbols
 
-    def load_ast_edges(
-        self, file_paths: Iterable[str]
-    ) -> list[dict[str, Any]]:
+    def load_ast_edges(self, file_paths: Iterable[str]) -> list[dict[str, Any]]:
         """Return CALLS, IMPORTS, and MEMBER_OF edges for the given files.
 
         CALLS are now caller-qualified: ``src_name`` is the enclosing
@@ -120,9 +116,7 @@ def load_ast_edges(
         edges.extend(self._call_edges(analyses))
         return edges
 
-    def _call_edges(
-        self, analyses: list[FileAnalysis]
-    ) -> list[dict[str, Any]]:
+    def _call_edges(self, analyses: list[FileAnalysis]) -> list[dict[str, Any]]:
         """Caller-qualified CALLS edges via
         ``codebase_graph.build_resolved_call_edges``.
 
@@ -134,9 +128,12 @@ def _call_edges(
         exactly what happened before the Wu audit caught it.
         """
         out: list[dict[str, Any]] = []
-        for caller_file, caller_qname, callee_file, callee_qname in (
-            build_resolved_call_edges(analyses)
-        ):
+        for (
+            caller_file,
+            caller_qname,
+            callee_file,
+            callee_qname,
+        ) in build_resolved_call_edges(analyses):
             out.append(
                 {
                     "kind": "calls",
@@ -197,18 +194,14 @@ def _parse_one(self, path: str, content: bytes) -> FileAnalysis:
                     content_hash="",
                 )
 
-    def _member_of_edges(
-        self, analyses: list[FileAnalysis]
-    ) -> list[dict[str, Any]]:
+    def _member_of_edges(self, analyses: list[FileAnalysis]) -> list[dict[str, Any]]:
         """method → class MEMBER_OF. The parser emits method names as
         ``ClassName.method``; we split on ``.`` and attach the method
         symbol to the class symbol in the same file."""
         out: list[dict[str, Any]] = []
         for a in analyses:
             # class names defined in THIS file (MEMBER_OF is intra-file).
-            classes_in_file = {
-                d.name for d in a.definitions if d.kind == "class"
-            }
+            classes_in_file = {d.name for d in a.definitions if d.kind == "class"}
             for sym in a.definitions:
                 if sym.kind != "method":
                     continue
@@ -230,9 +223,7 @@ def _member_of_edges(
                 )
         return out
 
-    def _import_edges(
-        self, analyses: list[FileAnalysis]
-    ) -> list[dict[str, Any]]:
+    def _import_edges(self, analyses: list[FileAnalysis]) -> list[dict[str, Any]]:
         """File → imported-symbol IMPORTS. Resolves each import's target
         file via ``resolve_all_imports``; then, for every imported name
         that corresponds to a SYMBOL defined in the target file, emit
@@ -269,9 +260,7 @@ def _import_edges(
         # Map target-file path (ABSOLUTE) → set of top-level symbol names.
         symbols_in_file: dict[str, set[str]] = {}
         for a in analyses:
-            names: set[str] = {
-                sym.name for sym in a.definitions if "." not in sym.name
-            }
+            names: set[str] = {sym.name for sym in a.definitions if "." not in sym.name}
             symbols_in_file[a.path] = names
 
         out: list[dict[str, Any]] = []

tests_py/infrastructure/test_ap_bridge_enable_precedence.py

@@ -62,9 +62,7 @@ class TestLegacyEnvRemoved:
     Setting it must not influence is_enabled() at all — only
     ``CORTEX_MEMORY_AP_ENABLED`` / ``MemorySettings.AP_ENABLED`` do."""
 
-    def test_legacy_env_has_no_effect_when_memory_setting_off(
-        self, monkeypatch
-    ):
+    def test_legacy_env_has_no_effect_when_memory_setting_off(self, monkeypatch):
         monkeypatch.setenv("CORTEX_ENABLE_AP", "1")  # used to force on
         monkeypatch.setenv("CORTEX_MEMORY_AP_ENABLED", "0")  # real flag says off
         from mcp_server.infrastructure import memory_config

tests_py/infrastructure/test_workflow_graph_source_native_ast.py

@@ -96,9 +96,7 @@ def test_member_of_emitted_for_method(self, tmp_path, source):
         path = _write(
             tmp_path,
             "a.py",
-            "class Foo:\n"
-            "    def bar(self): ...\n"
-            "    def baz(self): ...\n",
+            "class Foo:\n    def bar(self): ...\n    def baz(self): ...\n",
         )
         edges = source.load_ast_edges([path])
         member_of = [e for e in edges if e["kind"] == "member_of"]
@@ -121,8 +119,7 @@ def test_imports_resolve_to_sibling_symbol(self, tmp_path, source):
         caller = _write(
             tmp_path,
             "user.py",
-            "from lib import helper\n\n"
-            "def go():\n    return helper()\n",
+            "from lib import helper\n\ndef go():\n    return helper()\n",
         )
         lib = str(tmp_path / "lib.py")
         edges = source.load_ast_edges([str(tmp_path / "user.py"), lib])
@@ -196,16 +193,13 @@ def test_cross_file_call_resolved(self, tmp_path, source):
         caller_path = _write(
             tmp_path,
             "user.py",
-            "from lib import helper\n\n"
-            "def go():\n    return helper()\n",
+            "from lib import helper\n\ndef go():\n    return helper()\n",
         )
         lib_path = str(tmp_path / "lib.py")
         edges = source.load_ast_edges([caller_path, lib_path])
         calls = [e for e in edges if e["kind"] == "calls"]
         match = [
-            e
-            for e in calls
-            if e["src_name"] == "go" and e["dst_name"] == "helper"
+            e for e in calls if e["src_name"] == "go" and e["dst_name"] == "helper"
         ]
         assert match, f"expected go → helper call edge, got {calls}"
         e = match[0]
@@ -219,8 +213,7 @@ def test_unresolved_callee_dropped(self, tmp_path, source):
         path = _write(
             tmp_path,
             "a.py",
-            "def go():\n"
-            "    return os.path.join('a', 'b')  # external\n",
+            "def go():\n    return os.path.join('a', 'b')  # external\n",
         )
         edges = source.load_ast_edges([path])
         calls = [e for e in edges if e["kind"] == "calls"]
@@ -304,9 +297,7 @@ def test_calls_edge_lands_in_builder_output(self, tmp_path, source):
         calls = [
             e
             for e in builder._edges
-            if e.kind == EdgeKind.CALLS
-            and e.source == bar_id
-            and e.target == baz_id
+            if e.kind == EdgeKind.CALLS and e.source == bar_id and e.target == baz_id
         ]
         assert calls, (
             "Foo.bar → Foo.baz CALLS edge did not land in the builder. "
@@ -333,8 +324,7 @@ def test_cross_file_calls_edge_lands(self, tmp_path, source):
         user_path = _write(
             tmp_path,
             "user.py",
-            "from lib import helper\n\n"
-            "def go():\n    return helper()\n",
+            "from lib import helper\n\ndef go():\n    return helper()\n",
         )
 
         syms = source.load_symbols([lib_path, user_path])
@@ -355,9 +345,7 @@ def test_cross_file_calls_edge_lands(self, tmp_path, source):
         calls = [
             e
             for e in builder._edges
-            if e.kind == EdgeKind.CALLS
-            and e.source == go_id
-            and e.target == helper_id
+            if e.kind == EdgeKind.CALLS and e.source == go_id and e.target == helper_id
         ]
         assert calls, "cross-file go → helper CALLS edge did not land"
 
@@ -368,9 +356,7 @@ def test_max_files_cap_honored(self, tmp_path, source, monkeypatch):
         import mcp_server.infrastructure.workflow_graph_source_native_ast as mod
 
         monkeypatch.setattr(mod, "_MAX_FILES_PER_CALL", 3)
-        paths = [
-            _write(tmp_path, f"f{i}.py", "def a(): ...\n") for i in range(10)
-        ]
+        paths = [_write(tmp_path, f"f{i}.py", "def a(): ...\n") for i in range(10)]
         syms = source.load_symbols(paths)
         # 3 files × 1 symbol each = 3 symbols max.
         assert len(syms) == 3

ui/unified-viz.html

@@ -531,6 +531,7 @@
 <script src="/js/monitor.js"></script>
 <script src="/js/workflow_graph_humanize.js"></script>
 <script src="/js/workflow_graph_panel.js"></script>
+<script src="/js/workflow_graph_panel_renderers.js"></script>
 <script src="/js/workflow_graph_render_svg.js"></script>
 <script src="/js/workflow_graph_render_canvas.js"></script>
 <script src="/js/workflow_graph.js"></script>

ui/unified/js/workflow_graph_humanize.js

@@ -289,8 +289,9 @@
   // Output is plain text (rendered via .textContent). Backticks used to
   // frame identifiers get rendered literally and look like stray ASCII
   // noise to non-tech readers (Eco + Feynman audit). We drop them and
-  // use plain quotes where disambiguation helps.
-  function plainDescription(n, ctx) {
+  // use plain quotes where disambiguation helps. The previous ``ctx``
+  // parameter was dead (Dijkstra §9 audit) and has been removed.
+  function plainDescription(n) {
     if (!n) return '';
     var kind = n.kind;
     var name = n.label || n.id || '';