refactor(passes): remove --switches flag (raw RT data available via draws)

Author: BANANASJIM

src/rdc/_skills/SKILL.md

@@ -68,7 +68,6 @@ rdc events --json | jq '.[] | select(.type == "DrawIndexed")'
 ```bash
 rdc passes                          # TSV table
 rdc passes --json                   # includes load_ops/store_ops per pass
-rdc passes --switches               # adds RT_SWITCHES column (TBR optimization)
 rdc passes --deps --table           # per-pass READS/WRITES/LOAD/STORE
 ```
 

src/rdc/_skills/references/commands-quick-ref.md

@@ -652,7 +652,6 @@ List render passes.
 | `--dot` | Graphviz DOT output (requires --deps). | flag |  |
 | `--graph` | Human-readable graph (requires --deps). | flag |  |
 | `--table` | Per-pass I/O table (requires --deps). | flag |  |
-| `--switches` | Show RT switch count per pass (TBR flush risk). | flag |  |
 | `--no-header` | Omit TSV header | flag |  |
 | `--jsonl` | JSONL output | flag |  |
 | `-q, --quiet` | Print primary key column only | flag |  |

src/rdc/commands/resources.py

@@ -173,20 +173,13 @@ def resource_cmd(resid: int, use_json: bool) -> None:
     default=False,
     help="Per-pass I/O table (requires --deps).",
 )
-@click.option(
-    "--switches",
-    is_flag=True,
-    default=False,
-    help="Show RT switch count per pass (TBR flush risk).",
-)
 @list_output_options
 def passes_cmd(  # noqa: PLR0913
     use_json: bool,
     deps: bool,
     dot: bool,
     graph: bool,
     table: bool,
-    switches: bool,
     no_header: bool,
     use_jsonl: bool,
     quiet: bool,
@@ -204,10 +197,7 @@ def passes_cmd(  # noqa: PLR0913
         _passes_deps(use_json, dot, graph, table)
         return
 
-    params: dict[str, Any] = {}
-    if switches:
-        params["switches"] = True
-    result = call("passes", params)
+    result = call("passes", {})
     tree: dict[str, Any] = result.get("tree", {})
     if use_json:
         write_json(tree)
@@ -221,8 +211,6 @@ def passes_cmd(  # noqa: PLR0913
             sys.stdout.write(str(p.get("name", "")) + "\n")
     else:
         header = ["NAME", "DRAWS", "DISPATCHES", "TRIANGLES", "BEGIN_EID", "END_EID"]
-        if switches:
-            header.append("RT_SWITCHES")
         tsv_rows = [
             [
                 p.get("name", "-"),
@@ -232,7 +220,6 @@ def passes_cmd(  # noqa: PLR0913
                 p.get("begin_eid", "-"),
                 p.get("end_eid", "-"),
             ]
-            + ([p.get("rt_switches", {}).get("count", 0)] if switches else [])
             for p in passes
         ]
         write_tsv(tsv_rows, header=header, no_header=no_header)

src/rdc/handlers/query.py

@@ -200,12 +200,6 @@ def _handle_passes(
     actions = state.adapter.get_root_actions()
     tree = get_pass_hierarchy(actions, state.structured_file)
 
-    if params.get("switches"):
-        from rdc.services.query_service import _count_rt_switches
-
-        for p in tree.get("passes", []):
-            p["rt_switches"] = _count_rt_switches(actions, p["begin_eid"], p["end_eid"])
-
     return _result_response(request_id, {"tree": tree}), True
 
 

src/rdc/services/query_service.py

@@ -774,53 +774,6 @@ def _flush() -> None:
     return passes
 
 
-def _count_rt_switches(
-    actions: list[Any],
-    begin_eid: int,
-    end_eid: int,
-) -> dict[str, Any]:
-    """Count render-target switches within a pass EID range.
-
-    Args:
-        actions: Root action list from ReplayController.
-        begin_eid: First EID of the pass (inclusive).
-        end_eid: Last EID of the pass (inclusive).
-
-    Returns:
-        Dict with ``count`` (int) and ``switches`` list of dicts
-        containing ``eid``, ``from_targets``, and ``to_targets``.
-    """
-    leaf_actions: list[Any] = []
-
-    def _collect(nodes: list[Any]) -> None:
-        for a in nodes:
-            flags = int(a.flags)
-            if flags & _DRAW_OR_DISPATCH_OR_CLEAR:
-                if begin_eid <= a.eventId <= end_eid:
-                    leaf_actions.append(a)
-            if a.children:
-                _collect(a.children)
-
-    _collect(actions)
-    leaf_actions.sort(key=lambda a: a.eventId)
-
-    switches: list[dict[str, Any]] = []
-    prev_key: tuple[int, ...] | None = None
-    for a in leaf_actions:
-        key = _rt_key(a)
-        if prev_key is not None and key != prev_key:
-            switches.append(
-                {
-                    "eid": a.eventId,
-                    "from_targets": prev_key,
-                    "to_targets": key,
-                }
-            )
-        prev_key = key
-
-    return {"count": len(switches), "switches": switches}
-
-
 def _pass_list_with_fallback(actions: list[Any], sf: Any = None) -> list[dict[str, Any]]:
     """Build pass list, merging explicit passes with gap-filling synthetic passes."""
     explicit = _build_pass_list(actions, sf)