@principal-ai/subsystems-react 0.54.0 → 0.55.1

package/src/subsystem/SubsystemComponentGraph.tsx

@@ -55,7 +55,7 @@ import {
   type SubsystemTrail,
 } from './model';
 import { ConstructsCatalog } from './ConstructsCatalog';
-import type { SubsystemOpenFileOptions } from './declarationRef';
+import type { SubsystemOpenFileOptions } from './sourceSpan';
 import {
   SubsystemAgentsPanel,
   type SubsystemAgentsPanelProps,
@@ -893,7 +893,7 @@ function Inner({ components, trails, graphifyRelations, orderByLine, initialTrai
   }, [drawerStepIndex, focusedTrail, visibleStepIndex]);
 
   // Refresh selected component when the components list updates (e.g. verify
-  // writes back declarationRef).
+  // writes back declarationSpan).
   useEffect(() => {
     if (!selected) return;
     const fresh = components.find((c) => c.alias === selected.alias);
@@ -902,7 +902,7 @@ function Inner({ components, trails, graphifyRelations, orderByLine, initialTrai
 
   // Pass 1: build with estimated widths so React Flow can measure the DOM.
   // The pane stays hidden until Pass 2 completes. Key off layout-affecting
-  // fields only — declarationRef updates after verify must not re-run ELK.
+  // fields only — declarationSpan updates after verify must not re-run ELK.
   const layoutKey = useMemo(
     () => subsystemGraphLayoutKey({ components, trails, graphifyRelations }),
     [components, trails, graphifyRelations],
@@ -1226,22 +1226,22 @@ function Inner({ components, trails, graphifyRelations, orderByLine, initialTrai
   // Keyed by the React Flow edge id (from → mechanism → to). An edge can have
   // multiple steps with issues; we show the worst status (stale > unconfirmed).
   const stepIssueBadgeByEdgeId = useMemo(() => {
-    const map = new Map<string, { kind: 'step_unconfirmed' | 'step_stale'; severity: 'info' | 'error'; count: number }>();
+    const map = new Map<string, { kind: 'step_span_unconfirmed' | 'step_span_stale'; severity: 'info' | 'error'; count: number }>();
     if (!issues?.length || !trails?.length) {
       return map;
     }
     for (const issue of issues) {
-      if (issue.kind !== 'step_unconfirmed' && issue.kind !== 'step_stale') continue;
+      if (issue.kind !== 'step_span_unconfirmed' && issue.kind !== 'step_span_stale') continue;
       const stepInfo = issueStep(issue);
       if (!stepInfo) continue;
       const step = stepInfo.trail.steps[stepInfo.stepIndex];
       if (!step) continue;
       const edgeId = trailStepGraphEdgeId(step);
       const prev = map.get(edgeId);
-      // step_stale is worse than step_unconfirmed
-      const isWorse = issue.kind === 'step_stale' && prev?.kind !== 'step_stale';
+      // step_span_stale is worse than step_span_unconfirmed
+      const isWorse = issue.kind === 'step_span_stale' && prev?.kind !== 'step_span_stale';
       map.set(edgeId, {
-        kind: isWorse || !prev ? issue.kind as 'step_unconfirmed' | 'step_stale' : prev.kind,
+        kind: isWorse || !prev ? issue.kind as 'step_span_unconfirmed' | 'step_span_stale' : prev.kind,
         severity: prev?.severity === 'error' || issue.severity === 'error' ? 'error' : 'info',
         count: (prev?.count ?? 0) + 1,
       });
@@ -2994,7 +2994,7 @@ function Inner({ components, trails, graphifyRelations, orderByLine, initialTrai
                 return (
                   <span
                     aria-hidden
-                    title={lbl.stepIssue.kind === 'step_stale'
+                    title={lbl.stepIssue.kind === 'step_span_stale'
                       ? 'Call site changed since verification'
                       : 'Call site not yet verified'}
                     style={{

package/src/subsystem/SymbolInspectionCard.tsx

@@ -15,8 +15,8 @@ import { ExternalLink } from 'lucide-react';
 import { useTheme } from '@principal-ade/industry-theme';
 import { resolvePierreSyntaxThemeName } from '../pierre/pierreSyntaxTheme';
 import { sourceLangForPath, type SourceSyntaxLang } from '../pierre/sourceLang';
-import type { SubsystemOpenFileOptions } from './declarationRef';
-import { parseSourceLocation } from './declarationRef';
+import type { SubsystemOpenFileOptions } from './sourceSpan';
+import { parseSourceLocation } from './sourceSpan';
 import { generateDeclarationString } from './formatDeclaration';
 import type {
   SubsystemComponent,

package/src/subsystem/TrailsPanel.tsx

@@ -250,7 +250,13 @@ function TrailFlow({
             display: 'flex',
             alignItems: 'center',
             minWidth: 0,
-            padding: 0,
+            // Vertical padding, not just the row's min-height: a title that
+            // wraps to two+ lines must grow the header past `minHeight`, and
+            // without this the text sits flush against the row's top and bottom
+            // edges (the controls stretch to the text height). 6px keeps a
+            // one-line title under `TRAIL_CONTROL_SIZE`, so single-line rows
+            // stay 34px.
+            padding: '6px 0',
             border: 'none',
             background: 'transparent',
             textAlign: 'left',

package/src/subsystem/{declarationRef.test.ts → sourceSpan.test.ts}

@@ -3,7 +3,7 @@ import {
   extractDeclarationLine,
   normalizeDeclarationLine,
   parseSourceLocation,
-} from './declarationRef';
+} from './sourceSpan';
 
 describe('parseSourceLocation', () => {
   test('parses L-prefixed lines', () => {

package/dist/subsystem/declarationRef.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"declarationRef.d.ts","sourceRoot":"","sources":["../../src/subsystem/declarationRef.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,+BAA+B,CAAC;AAEzE,YAAY,EAAE,mBAAmB,EAAE,CAAC;AAEpC;;;;;;;;GAQG;AACH,MAAM,MAAM,aAAa,GACrB,MAAM,GACN,OAAO,GACP,SAAS,GACT,YAAY,GACZ,WAAW,CAAC;AAEhB,eAAO,MAAM,qBAAqB,MAAgB,CAAC;AAEnD,wEAAwE;AACxE,wBAAgB,mBAAmB,CAAC,GAAG,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI,GAAG,MAAM,GAAG,IAAI,CAMjF;AAED,wDAAwD;AACxD,wBAAgB,wBAAwB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAE7D;AAED,0EAA0E;AAC1E,wBAAgB,sBAAsB,CAAC,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAKzF;AAED,iEAAiE;AACjE,MAAM,WAAW,wBAAwB;IACvC,4EAA4E;IAC5E,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB;;;OAGG;IACH,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB;;;OAGG;IACH,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB"}

package/dist/subsystem/declarationRef.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"declarationRef.js","sourceRoot":"","sources":["../../src/subsystem/declarationRef.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAsBH,MAAM,CAAC,MAAM,qBAAqB,GAAG,IAAa,CAAC;AAEnD,wEAAwE;AACxE,MAAM,UAAU,mBAAmB,CAAC,GAA8B;IAChE,IAAI,CAAC,GAAG,EAAE,IAAI,EAAE;QAAE,OAAO,IAAI,CAAC;IAC9B,MAAM,CAAC,GAAG,UAAU,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC;IACtC,IAAI,CAAC,CAAC;QAAE,OAAO,IAAI,CAAC;IACpB,MAAM,CAAC,GAAG,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAE,EAAE,EAAE,CAAC,CAAC;IACrC,OAAO,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;AACjD,CAAC;AAED,wDAAwD;AACxD,MAAM,UAAU,wBAAwB,CAAC,IAAY;IACnD,OAAO,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC;AAC3C,CAAC;AAED,0EAA0E;AAC1E,MAAM,UAAU,sBAAsB,CAAC,OAAe,EAAE,UAAkB;IACxE,IAAI,UAAU,GAAG,CAAC;QAAE,OAAO,IAAI,CAAC;IAChC,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IACrC,IAAI,UAAU,GAAG,KAAK,CAAC,MAAM;QAAE,OAAO,IAAI,CAAC;IAC3C,OAAO,KAAK,CAAC,UAAU,GAAG,CAAC,CAAC,IAAI,IAAI,CAAC;AACvC,CAAC"}