@principal-ai/subsystems-react 0.41.0 → 0.43.0

package/src/subsystem/C4Graph.tsx

@@ -3,9 +3,8 @@
  *
  * Prototype test case: feed it a `C4Model` from `toC4()` and it draws the
  * system as a compound frame, its containers (or components) as boxes, and
- * externals/actors outside the system. Static `relations` render as solid
- * edges; `walkthrough` hops render as dashed flow edges. Click a box to list
- * the source components it rolled up.
+ * externals/actors outside the system. `walkthrough` hops render as flow
+ * edges. Click a box to list the source components it rolled up.
  *
  * Same interaction language as the other graphs: React Flow + ELK + the shared
  * `GRAPH_NAV_PROPS` chrome. Deliberately small — no measurement passes, no

package/src/subsystem/IssueList.tsx

@@ -2,7 +2,7 @@
  * SubsystemIssueList — the diagnostics sidebar list.
  *
  * A catalogue of verification issues (audit findings), grouped by *target* so
- * every broken component / relation / module / flow clusters together. Each
+ * every broken component / module / flow clusters together. Each
  * card is severity-colored, names its kind, states the message, and offers the
  * deterministic fix inline when one exists.
  *
@@ -38,7 +38,6 @@ export type SubsystemIssueSeverity = 'error' | 'info';
 
 export type SubsystemIssueTargetKind =
   | 'component'
-  | 'relation'
   | 'module'
   | 'walkthrough'
   | 'step'
@@ -48,8 +47,8 @@ export type SubsystemIssueTargetKind =
 /**
  * The four verification layers, in order. Mirrors the docs' progression:
  * constructs → static topology → dynamic topology → walkthrough.
- * Static topology = relations; dynamic topology = process (runtime) +
- * package/module (containment).
+ * Static topology = package/module (containment); dynamic topology = process
+ * (runtime).
  */
 export type SubsystemIssueCategory =
   | 'repo'
@@ -90,7 +89,7 @@ export const SUBSYSTEM_ISSUE_CATEGORY_LABEL: Record<
   walkthrough: 'Walkthrough',
 };
 
-/** Default `kind` → layer. `boundary_*` (module containment + process) is dynamic. */
+/** Default `kind` → layer. Module containment is static; process nest is dynamic. */
 const KIND_CATEGORY: Record<string, SubsystemIssueCategory> = {
   missing_file: 'construct',
   symbol_ambiguous: 'construct',
@@ -102,10 +101,7 @@ const KIND_CATEGORY: Record<string, SubsystemIssueCategory> = {
   signature_unconfirmed: 'construct',
   repo_unresolved: 'repo',
   graphify_unavailable: 'repo',
-  topology_broken_endpoint: 'static-topology',
-  topology_import_unconfirmed: 'static-topology',
-  topology_relation_unconfirmed: 'static-topology',
-  boundary_module_file_mismatch: 'dynamic-topology',
+  boundary_module_file_mismatch: 'static-topology',
   boundary_process_nest_disagree: 'dynamic-topology',
   walkthrough: 'walkthrough',
 };
@@ -189,7 +185,7 @@ export interface SubsystemIssueTarget {
   id?: string;
   /** Display label (component name, `from → to`, module path, flow title). */
   label: string;
-  /** Optional sub-label (relation type, step number, …). */
+  /** Optional sub-label (step number, …). */
   detail?: string;
   /**
    * For a `step` target: the walkthrough id, plus a 0-based index into that
@@ -231,8 +227,8 @@ export function humanizeIssueKind(kind: string): string {
  * Curated labels. Construct-layer findings stay fully qualified ("Construct
  * declaration file missing", "Construct type mismatch") rather than relying on
  * the category header — this UI is new and the extra words read clearer.
- * Topology findings drop the `topology_`/`boundary_` prefix since the layer
- * already scopes them. Explicit `issue.kindLabel` wins.
+ * Boundary findings drop the `boundary_` prefix since the layer already
+ * scopes them. Explicit `issue.kindLabel` wins.
  */
 const KIND_LABEL: Record<string, string> = {
   missing_file: 'Construct declaration file missing',
@@ -245,9 +241,6 @@ const KIND_LABEL: Record<string, string> = {
   signature_unconfirmed: 'Construct signature unconfirmed',
   repo_unresolved: 'No repository clone',
   graphify_unavailable: 'Graphify cache unavailable',
-  topology_broken_endpoint: 'Construct relationship broken',
-  topology_import_unconfirmed: 'Import unconfirmed',
-  topology_relation_unconfirmed: 'Relationship unconfirmed',
   boundary_module_file_mismatch: 'Declaration file outside module',
   boundary_process_nest_disagree: 'Module spans multiple process contexts',
   walkthrough: 'Step issue',
@@ -261,8 +254,8 @@ export function issueKindLabel(issue: SubsystemIssue): string {
 /**
  * Display order within the list, by verification pipeline — not severity.
  * Mirrors the per-component check order (file → symbol → declaration → type →
- * signature), then topology (integrity → corroboration → membership) and the
- * remaining layers. Within a stage, "unconfirmed" precedes "mismatch": you
+ * signature), then the boundary layers (module membership → process nest) and
+ * the remaining layers. Within a stage, "unconfirmed" precedes "mismatch": you
  * reach a verdict before you can find it wrong.
  */
 const KIND_ORDER: Record<string, number> = {
@@ -278,12 +271,9 @@ const KIND_ORDER: Record<string, number> = {
   construct_mismatch: 15,
   signature_unconfirmed: 16,
   signature_mismatch: 17,
-  // Static topology — edge integrity → corroboration → membership
-  topology_broken_endpoint: 20,
-  topology_relation_unconfirmed: 21,
-  topology_import_unconfirmed: 22,
+  // Static topology — package/module membership
   boundary_module_file_mismatch: 23,
-  // Runtime topology
+  // Runtime topology — process containment
   boundary_process_nest_disagree: 30,
   // Walkthrough
   walkthrough: 40,

package/src/subsystem/SubsystemComponentGraph.tsx

@@ -39,7 +39,6 @@ import {
   deriveGraphEdges,
   isConstructsOnlyModel,
   moduleGroupNodeId,
-  isRelationMechanism,
   isWalkthroughMechanism,
   edgeColor,
   MECHANISM_DESCRIPTIONS,
@@ -52,7 +51,6 @@ import {
   type SubsystemGraphifyRelation,
   type SubsystemNodeIssue,
   type SubsystemRegionIssue,
-  type SubsystemRelation,
   type SubsystemWalkthrough,
 } from './model';
 import { ConstructsCatalog } from './ConstructsCatalog';
@@ -137,15 +135,18 @@ type DrawerTarget =
 
 /**
  * Per-model UI state persisted to `localStorage`, keyed by `persistKey`.
- * Only the walkthrough working set is stored — which flows are expanded and
- * which flow/step is selected — so tabbing away from a model and back lands
- * you where you left off. Transient things (hover, drag, camera) are not saved.
+ * Stores the walkthrough working set (which flows are expanded and which
+ * flow/step is selected) and which sidebar/diagnostics tab is showing, so
+ * tabbing away from a model and back lands you where you left off. Transient
+ * things (hover, drag, camera) are not saved.
  */
 interface PersistedViewState {
   expandedWalkthroughs?: string[];
   focusedWalkthroughId?: string | null;
   focusedStepIndex?: number | null;
   sidebarWidth?: number;
+  sidebarView?: 'files' | 'walkthroughs';
+  diagnosticsTab?: 'issues' | 'agents';
 }
 
 const VIEW_STATE_PREFIX = 'principal.subsystems.viewState.';
@@ -179,8 +180,6 @@ function writeViewState(
 
 export interface SubsystemComponentGraphProps {
   components: SubsystemComponent[];
-  /** Topology relations (structural / module / type). Display edges are derived with walkthrough hops. */
-  relations: SubsystemRelation[];
   /**
    * Ordered runtime walkthroughs — one walkthrough per flow. When present the
    * sidebar's bottom half offers a Files/Walkthroughs toggle:
@@ -195,7 +194,7 @@ export interface SubsystemComponentGraphProps {
    * graphify-native relations (raw static-graph edges: `imports`, `contains`,
    * `re_exports`, …). Display-only, drawn with the separate
    * `GRAPHIFY_RELATION_COLOR` palette so they read as derived facts, distinct
-   * from authored subsystem mechanisms. Visible in the `relations` edge view.
+   * from authored subsystem mechanisms. Visible in the `graphify` edge view.
    */
   graphifyRelations?: readonly SubsystemGraphifyRelation[];
   /**
@@ -236,16 +235,15 @@ export interface SubsystemComponentGraphProps {
    */
   showSingletonFrames?: boolean;
   /**
-   * Which edge vocabulary the canvas draws. The relation and walkthrough
-   * vocabularies are disjoint, so a graph carrying both shows one or the
-   * other — never both. Edges outside the view are hidden (labels go too).
-   * - `relations`: topology relation edges (`extends`, `implements`, …)
+   * Which edge source the canvas draws. The graphify and walkthrough sources
+   * are disjoint, so a graph carrying both shows one or the other — never
+   * both. Edges outside the view are hidden (labels go too).
+   * - `graphify`: graphify-native static edges (`imports`, `contains`, …)
    * - `walkthroughs`: walkthrough hop edges (`calls`, `feeds`, …), including
    *   step numbers when a flow is focused/hovered
    * Leave unset to let the sidebar's Files / Walkthroughs tab drive it: Files
-   * draws topology relations, Walkthroughs draws runtime hops. Without visible
-   * tabs, defaults to `walkthroughs` for a flow-only model (walkthroughs but
-   * no relations), otherwise `relations`.
+   * draws graphify edges, Walkthroughs draws runtime hops. Without visible
+   * tabs, defaults to `walkthroughs`.
    */
   edgeView?: SubsystemEdgeView;
   /** Subsystem title displayed in the sidebar. */
@@ -345,7 +343,7 @@ export interface SubsystemComponentGraphProps {
   /**
    * Click an issue. The graph first focuses the target on the canvas itself —
    * a component target is selected and framed (any edge / walkthrough focus
-   * that would hide it is cleared), a relation target frames its edge, and a
+   * that would hide it is cleared), and a
    * module target frames that boundary frame — then this fires so the host can
    * open its own detail. Collapsing the card again reverses that (deselect +
    * zoom out). Other target kinds (flow / repo) have no node to frame and just
@@ -519,7 +517,7 @@ interface InnerProps extends SubsystemComponentGraphProps {
   measured: { w: number; h: number } | null;
 }
 
-function Inner({ components, relations, walkthroughs, graphifyRelations, orderByLine, initialWalkthroughId, onReorderWalkthroughs, onSelect, onEdgeSelect, measured: _measured, maxNodeWidth, showEdgeLabels, showSingletonFrames = true, edgeView, title, hideSidebar, walkthroughStepMode = 'focus', autoPlayWalkthroughs = false, walkthroughAutoPlayIntervalMs = WALKTHROUGH_PLAY_PAUSE_MS, zoomOnWalkthroughFocus = true, walkthroughFocusDurationMs = 300, graphTitle, showWalkthroughTitle = false, description, canvasOverlay, sidebarExtra, sidebarAfterDescription, diagnostic, issues, showIssues, focusIssueCategory, onSelectIssue, onApplyIssueFix, onHoverIssue, renderFileView, renderFileViewer, renderWalkthroughViewer, onFileSelect, componentVerification, onInspectSymbol, boundaryColors, hideDrawer = false, persistKey, liveEvents, agentsPanel }: InnerProps) {
+function Inner({ components, walkthroughs, graphifyRelations, orderByLine, initialWalkthroughId, onReorderWalkthroughs, onSelect, onEdgeSelect, measured: _measured, maxNodeWidth, showEdgeLabels, showSingletonFrames = true, edgeView, title, hideSidebar, walkthroughStepMode = 'focus', autoPlayWalkthroughs = false, walkthroughAutoPlayIntervalMs = WALKTHROUGH_PLAY_PAUSE_MS, zoomOnWalkthroughFocus = true, walkthroughFocusDurationMs = 300, graphTitle, showWalkthroughTitle = false, description, canvasOverlay, sidebarExtra, sidebarAfterDescription, diagnostic, issues, showIssues, focusIssueCategory, onSelectIssue, onApplyIssueFix, onHoverIssue, renderFileView, renderFileViewer, renderWalkthroughViewer, onFileSelect, componentVerification, onInspectSymbol, boundaryColors, hideDrawer = false, persistKey, liveEvents, agentsPanel }: InnerProps) {
   const { theme } = useTheme();
   const { fitView, fitBounds } = useReactFlow();
   const viewport = useViewport();
@@ -527,8 +525,8 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
   // graph is its own tab/mount, so `persistKey` is stable for a mount.
   const persisted = useMemo(() => readViewState(persistKey), [persistKey]);
   const graphEdges = useMemo(
-    () => deriveGraphEdges({ relations, walkthroughs }),
-    [relations, walkthroughs],
+    () => deriveGraphEdges({ walkthroughs }),
+    [walkthroughs],
   );
   const [built, setBuilt] = useState<{
     nodes: Node[];
@@ -577,31 +575,26 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
   } | null>(null);
   // Sidebar bottom half: which panel is shown when walkthroughs exist.
   const [sidebarView, setSidebarView] = useState<'files' | 'walkthroughs'>(() =>
-    walkthroughs?.length ? 'walkthroughs' : 'files',
+    persisted.sidebarView ?? (walkthroughs?.length ? 'walkthroughs' : 'files'),
   );
   // Diagnostics area tab: issues list vs the Maintain agent pipeline.
-  const [diagnosticsTab, setDiagnosticsTab] = useState<'issues' | 'agents'>('issues');
-  // Temporary edge-vocabulary override. Focusing a relation finding whose edge
-  // belongs to the *other* vocabulary flips the canvas so the edge is actually
-  // visible; it reverts when the focus is undone (see `unfocusIssueTarget`) or
-  // when the diagnostics list closes. Null = use the host-pinned / tab view.
-  const [edgeViewOverride, setEdgeViewOverride] = useState<SubsystemEdgeView | null>(
-    null,
+  const [diagnosticsTab, setDiagnosticsTab] = useState<'issues' | 'agents'>(
+    () => persisted.diagnosticsTab ?? 'issues',
   );
-  // One edge vocabulary at a time. When the caller doesn't pick, the sidebar's
-  // Files / Walkthroughs tab picks: Files draws topology relation edges,
+  // One edge source at a time. When the caller doesn't pick, the sidebar's
+  // Files / Walkthroughs tab picks: Files draws graphify static edges,
   // Walkthroughs draws runtime hop edges. Without visible tabs (no walkthroughs,
-  // or a sidebar-less embed) fall back to the model: a flow-only graph defaults
-  // to walkthrough edges rather than empty, everything else to relations.
+  // or a sidebar-less embed) fall back to walkthrough edges.
   const sidebarTabsVisible = !hideSidebar && (walkthroughs?.length ?? 0) > 0;
   const resolvedEdgeView: SubsystemEdgeView =
-    edgeViewOverride ??
     edgeView ??
     (sidebarTabsVisible
-      ? (sidebarView === 'walkthroughs' ? 'walkthroughs' : 'relations')
-      : relations.length === 0 && (walkthroughs?.length ?? 0) > 0
+      ? (sidebarView === 'walkthroughs' ? 'walkthroughs' : 'graphify')
+      : (walkthroughs?.length ?? 0) > 0
         ? 'walkthroughs'
-        : 'relations');
+        : (graphifyRelations?.length ?? 0) > 0
+          ? 'graphify'
+          : 'walkthroughs');
   // Walkthrough flows the user has expanded (via the title row). Closed by
   // default so a graph with several flows doesn't dump every step list at once.
   // Restored from the persisted view state when a `persistKey` is set.
@@ -681,6 +674,14 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
     if (!persistKey) return;
     writeViewState(persistKey, { focusedStepIndex });
   }, [persistKey, focusedStepIndex]);
+  useEffect(() => {
+    if (!persistKey) return;
+    writeViewState(persistKey, { sidebarView });
+  }, [persistKey, sidebarView]);
+  useEffect(() => {
+    if (!persistKey) return;
+    writeViewState(persistKey, { diagnosticsTab });
+  }, [persistKey, diagnosticsTab]);
   // Ref mirror of `selected` so the SUBSYSTEM_CALLBACKS click handler (a
   // closure over the effect deps) can toggle without a stale value.
   const selectedRef = useRef<SubsystemComponent | null>(null);
@@ -794,15 +795,13 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
   // The pane stays hidden until Pass 2 completes. Key off layout-affecting
   // fields only — declarationRef updates after verify must not re-run ELK.
   const layoutKey = useMemo(
-    () => subsystemGraphLayoutKey({ components, relations, walkthroughs, graphifyRelations }),
-    [components, relations, walkthroughs, graphifyRelations],
+    () => subsystemGraphLayoutKey({ components, walkthroughs, graphifyRelations }),
+    [components, walkthroughs, graphifyRelations],
   );
   const componentsRef = useRef(components);
-  const relationsRef = useRef(relations);
   const walkthroughsRef = useRef(walkthroughs);
   const graphifyRelationsRef = useRef(graphifyRelations);
   componentsRef.current = components;
-  relationsRef.current = relations;
   walkthroughsRef.current = walkthroughs;
   graphifyRelationsRef.current = graphifyRelations;
 
@@ -823,7 +822,6 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
     pass2GenRef.current += 1;
     const doc = {
       components: componentsRef.current,
-      relations: relationsRef.current,
       walkthroughs: walkthroughsRef.current,
     };
     void buildSubsystemGraph(doc, {
@@ -880,7 +878,7 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
     const measuredHeights = new Map(leafNodes.map((n) => [n.id, dims.get(n.id)!.height]));
     const gen = ++pass2GenRef.current;
     void buildSubsystemGraph(
-      { components, relations, walkthroughs },
+      { components, walkthroughs },
       { maxNodeWidth, showEdgeLabels, measuredWidths, measuredHeights, showSingletonFrames, graphifyRelations, orderByLine },
     )
       .then(({ nodes, edges: e, absoluteRects }) => {
@@ -894,7 +892,7 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
         // Reveal Pass 1 layout rather than leaving the cover up forever.
         setLayoutReady(true);
       });
-  }, [built.nodes, components, relations, walkthroughs, graphifyRelations, orderByLine, maxNodeWidth, showEdgeLabels]);
+  }, [built.nodes, components, walkthroughs, graphifyRelations, orderByLine, maxNodeWidth, showEdgeLabels]);
 
   // After Pass 1 commits, try Pass 2 immediately with retained measurements.
   // Same-id live updates often get no new React Flow `dimensions` events, so
@@ -978,7 +976,7 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
 
   // Resolve an issue's component target to the matching node. `id` is the
   // stable alias; `label` may be an alias, name, or symbol. Non-component
-  // targets (relation / module / flow / repo) have no single node → null.
+  // targets (module / flow / repo) have no single node → null.
   const issueComponent = useCallback(
     (issue: SubsystemIssue): SubsystemComponent | null => {
       const target = issue.target;
@@ -996,33 +994,6 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
     [components],
   );
 
-  // Resolve a `relation` issue target to the display edge it flags. The target
-  // names the two endpoints (`from → to`); match an edge joining both, in
-  // either direction, preferring the target's `detail` mechanism when it names
-  // one. Null when no display edge matches.
-  const issueEdge = useCallback(
-    (issue: SubsystemIssue): Edge | null => {
-      const target = issue.target;
-      if (target?.kind !== 'relation') return null;
-      const parts = `${target.id ?? ''} → ${target.label}`
-        .split('→')
-        .map((p) => p.replace(/\(\)$/, '').trim())
-        .filter(Boolean);
-      const joins = (e: Edge): boolean =>
-        parts.includes(e.source) && parts.includes(e.target);
-      const matches = baseEdges.filter(joins);
-      const byMechanism = target.detail
-        ? matches.filter(
-            (e) =>
-              (e.data as { mechanism?: string } | undefined)?.mechanism ===
-              target.detail,
-          )
-        : matches;
-      return byMechanism[0] ?? matches[0] ?? null;
-    },
-    [baseEdges],
-  );
-
   // A `module` issue target names a boundary frame, so focusing the card frames
   // that region on the canvas. Null when the target names no frame (a singleton
   // dropped from the layout, or a region key nothing declares).
@@ -1147,7 +1118,6 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
   // the canvas keeps while everything else dims. Derived from each finding's
   // target:
   //   component   → that node
-  //   relation    → the edge it flags (+ its two endpoint nodes)
   //   module      → every component in that module
   //   walkthrough → the flow's nodes + its hop edges
   //   repo        → every component of that repo
@@ -1167,20 +1137,6 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
       if (t.kind === 'component') {
         const a = resolveAlias(t.id ?? t.label);
         if (a) nodeIds.add(a);
-      } else if (t.kind === 'relation') {
-        // The target names the two endpoints (`from → to`); light the edge and
-        // its endpoints. Falls back to endpoints alone if no edge matches.
-        const edge = issueEdge(issue);
-        if (edge) {
-          edgeIds.add(edge.id);
-          nodeIds.add(edge.source);
-          nodeIds.add(edge.target);
-        } else {
-          for (const part of `${t.id ?? ''} → ${t.label}`.split('→')) {
-            const a = resolveAlias(part);
-            if (a) nodeIds.add(a);
-          }
-        }
       } else if (t.kind === 'module') {
         const key = t.id ?? t.label;
         const prefix = `${key.replace(/\/$/, '')}/`;
@@ -1219,7 +1175,7 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
       if (nodeIds.has(e.source) && nodeIds.has(e.target)) edgeIds.add(e.id);
     }
     return { nodeIds, edgeIds };
-  }, [issues, expandedIssueCategories, components, walkthroughs, baseEdges, resolveAlias, issueEdge]);
+  }, [issues, expandedIssueCategories, components, walkthroughs, resolveAlias]);
   const issueFocusNodeIds = issueFocus?.nodeIds ?? null;
   const issueFocusEdgeIds = issueFocus?.edgeIds ?? null;
 
@@ -1466,15 +1422,15 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
       };
     };
     // Edges outside the selected view are hidden entirely (labels included).
-    // graphify-native edges belong to the relations view (they are static
+    // graphify-native edges belong to the graphify view (they are static
     // topology, not runtime hops) regardless of their verb.
     const edgeInView = (e: Edge): boolean => {
       const d = e.data as
         | { mechanism?: string; provenance?: SubsystemEdgeProvenance }
         | undefined;
       const mechanism = d?.mechanism ?? 'uses';
-      if (resolvedEdgeView === 'relations') {
-        return d?.provenance === 'graphify' || isRelationMechanism(mechanism);
+      if (resolvedEdgeView === 'graphify') {
+        return d?.provenance === 'graphify';
       }
       return isWalkthroughMechanism(mechanism);
     };
@@ -1573,7 +1529,6 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
     setSelectedEdgeId(null);
     setFocusedWalkthroughId(null);
     setFocusedStepIndex(null);
-    setEdgeViewOverride(null);
     issueModuleFocusRef.current = null;
   }, []);
 
@@ -1599,7 +1554,6 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
   useEffect(() => {
     if (!issuesActive) {
       setExpandedIssueCategories([]);
-      setEdgeViewOverride(null);
     }
   }, [issuesActive]);
   const toggleIssues = () => {
@@ -2068,11 +2022,7 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
     if (playable.length === 0) return;
     // Only run the cycles that can actually be shown in the current edge view.
     const inView = playable.filter((tl) =>
-      tl.steps.some((s) =>
-        resolvedEdgeView === 'relations'
-          ? isRelationMechanism(s.mechanism)
-          : isWalkthroughMechanism(s.mechanism),
-      ),
+      tl.steps.some((s) => isWalkthroughMechanism(s.mechanism)),
     );
     if (inView.length === 0) return;
     const cyc = inView;
@@ -2241,16 +2191,14 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
   );
 
   // Issue click → select + frame the target on the canvas. Component targets
-  // frame their node; relation targets frame their edge (endpoints + the routed
-  // line); other kinds (module / flow / repo) have no single element and just
-  // forward. Selections / focus that would hide the target are cleared first,
-  // then the camera flies to it on the next frame. The host's `onSelectIssue`
-  // still fires afterwards for its own detail.
+  // frame their node; other kinds (module / flow / repo) have no single element
+  // and just forward. Selections / focus that would hide the target are cleared
+  // first, then the camera flies to it on the next frame. The host's
+  // `onSelectIssue` still fires afterwards for its own detail.
   const focusIssueTarget = useCallback(
     (issue: SubsystemIssue) => {
       const comp = issueComponent(issue);
-      const edge = comp ? null : issueEdge(issue);
-      const step = comp || edge ? null : issueStep(issue);
+      const step = comp ? null : issueStep(issue);
       if (comp) {
         setSelected(comp);
         setSelectedEdgeId(null);
@@ -2268,27 +2216,6 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
             maxZoom: 1.5,
           });
         });
-      } else if (edge) {
-        setSelected(null);
-        setFocusedWalkthroughId(null);
-        setFocusedStepIndex(null);
-        setHoveredWalkthroughStep(null);
-        setSelectedEdgeId(edge.id);
-        issueModuleFocusRef.current = null;
-        // Flip to the vocabulary this edge belongs to, so a hop flagged in the
-        // relations view (or vice versa) is actually drawn while focused.
-        const d = edge.data as
-          | { mechanism?: string; provenance?: SubsystemEdgeProvenance }
-          | undefined;
-        const mechanism = d?.mechanism ?? 'uses';
-        setEdgeViewOverride(
-          d?.provenance === 'graphify' || isRelationMechanism(mechanism)
-            ? 'relations'
-            : 'walkthroughs',
-        );
-        requestAnimationFrame(() => {
-          fitFocusBounds(new Set([edge.id]));
-        });
       } else if (step) {
         // A step finding focuses the step itself, through the same entry point
         // the walkthrough panel uses — so the drawer, the step numbering, the
@@ -2316,14 +2243,12 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
     },
     [
       issueComponent,
-      issueEdge,
       issueStep,
       issueModuleNodeId,
       focusWalkthroughStep,
       onSelect,
       onSelectIssue,
       fitView,
-      fitFocusBounds,
       fitNodeRects,
     ],
   );
@@ -2342,14 +2267,9 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
         setSelected(null);
         setSelectedEdgeId(null);
       } else {
-        const edge = issueEdge(issue);
-        const step = edge ? null : issueStep(issue);
-        const moduleNodeId = edge || step ? null : issueModuleNodeId(issue);
-        if (edge) {
-          if (selectedEdgeIdRef.current !== edge.id) return;
-          setSelectedEdgeId(null);
-          setEdgeViewOverride(null);
-        } else if (step) {
+        const step = issueStep(issue);
+        const moduleNodeId = step ? null : issueModuleNodeId(issue);
+        if (step) {
           // Only unwind a step focus this card still owns. A dim-mode graph
           // keeps its step focus in hover state instead (owned by the pointer,
           // not the card), and correctly leaves that alone here.
@@ -2375,7 +2295,7 @@ function Inner({ components, relations, walkthroughs, graphifyRelations, orderBy
         fitView({ padding: 0.1, duration: 300, minZoom: 0.05, maxZoom: 2 });
       });
     },
-    [issueComponent, issueEdge, issueStep, issueModuleNodeId, fitView],
+    [issueComponent, issueStep, issueModuleNodeId, fitView],
   );
 
   // Construct tokens inside a walkthrough snippet. A step's line is an edge
@@ -3349,7 +3269,6 @@ export function SubsystemComponentGraph(props: SubsystemComponentGraphProps) {
 
   const constructsOnly = isConstructsOnlyModel({
     components: props.components,
-    relations: props.relations,
     walkthroughs: props.walkthroughs,
     graphifyRelations: props.graphifyRelations,
   });