npm - @adcops/autocore-react - Versions diffs - 3.5.15 → 3.6.3 - Mend

@adcops/autocore-react 3.5.15 → 3.6.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (39) hide show

package/src/components/varpick/varpick.css CHANGED Viewed

@@ -1,12 +1,66 @@
 /*
  * Variable picker.
  *
- * Colours come from the PrimeReact theme's variables so the dialog matches
- * whatever theme the machine is running, light or dark. Every value has a
- * literal fallback: an un-themed host (the playground, a test harness) must
- * still render legible text rather than black on black.
+ * ## The token bridge
+ *
+ * Every colour resolves `--ac-*` first, then the PrimeReact theme variable,
+ * then a literal. That ordering is the whole point:
+ *
+ *   - **`--ac-*`** is autocore's own design-token layer, which 4.0 supplies.
+ *     When it exists, this file needs no edit to follow it.
+ *   - **PrimeReact's `--surface-*` / `--text-color`** is what 3.5 actually
+ *     defines today, so the picker matches whatever theme the machine runs,
+ *     light or dark.
+ *   - **The literal** keeps an un-themed host legible — the playground, the
+ *     jsdom test harness — rather than rendering black on black.
+ *
+ * The token NAMES and their fallbacks are copied deliberately from
+ * `@adcops/autocore-seq-react`'s `styles.css`. The sequence editor's picker and
+ * this one sit one tab apart doing the identical thing; if they read different
+ * token names, 4.0 would have to satisfy two sets and the two panels could
+ * drift to different colours while both looked "correct".
+ *
+ * ## Why the tokens are declared twice
+ *
+ * The field lives inside the TIS editor's form, but PrimeReact renders the
+ * Dialog through a PORTAL to `<body>` — so the two halves are not in one
+ * subtree and a single scoping root cannot cover both. Hence the declaration on
+ * `.ac-varpick` (the dialog, which passes that className) and on
+ * `.ac-varpick__input` (the field). Declaring them on `:root` instead would
+ * push a dozen names into every consuming application's global scope for the
+ * benefit of one component.
  */
+.ac-varpick,
+.ac-varpick__input {
+    /* -- surfaces ---------------------------------------------------- */
+    --vp-sunk:           var(--ac-surface-sunk,         var(--surface-50,     #f8f9fa));
+    --vp-overlay:        var(--ac-surface-overlay,      var(--surface-overlay,#ffffff));
+    --vp-hover:          var(--ac-hover,                var(--surface-hover,  rgba(127, 127, 127, 0.08)));
+    /* -- ink --------------------------------------------------------- */
+    --vp-ink:            var(--ac-on-surface,           var(--text-color,           #111827));
+    --vp-ink-muted:      var(--ac-on-surface-muted,     var(--text-color-secondary, #6b7280));
+    --vp-ink-subtle:     var(--ac-on-surface-subtle,    var(--text-color-secondary, #8b95a1));
+    /* -- lines ------------------------------------------------------- */
+    --vp-border:         var(--ac-border,               var(--surface-border, rgba(127, 127, 127, 0.25)));
+    --vp-divider:        var(--ac-divider,              var(--surface-border, rgba(127, 127, 127, 0.18)));
+    /* -- accents ----------------------------------------------------- */
+    --vp-primary:        var(--ac-primary,              var(--primary-color,  #2563eb));
+    --vp-primary-subtle: var(--ac-primary-subtle,       var(--highlight-bg,   rgba(37, 99, 235, 0.14)));
+    --vp-success:        var(--ac-success,              var(--green-500,      #15803d));
+    --vp-warning:        var(--ac-warning,              var(--orange-500,     #b45309));
+    --vp-info:           var(--ac-info,                 var(--blue-500,       #0369a1));
+    --vp-focus-ring:     var(--ac-focus-ring,           rgba(37, 99, 235, 0.45));
+    /* -- type -------------------------------------------------------- */
+    --vp-mono:           var(--ac-font-mono,            ui-monospace, "Cascadia Mono", Menlo, Consolas, monospace);
+    --vp-font-sm:        var(--ac-text-sm,              0.8125rem);
+    --vp-font-xs:        var(--ac-text-xs,              0.75rem);
+}
 .ac-varpick__input {
     display: flex;
     align-items: stretch;
@@ -31,8 +85,8 @@
 }
 .ac-varpick__hint {
-    font-size: 0.8125rem;
-    color: var(--text-color-secondary, #6b7280);
+    font-size: var(--vp-font-sm);
+    color: var(--vp-ink-muted);
     margin-bottom: 0.75rem;
     line-height: 1.4;
 }
@@ -41,7 +95,7 @@
 .ac-varpick__empty {
     padding: 1.5rem 0.25rem;
-    color: var(--text-color-secondary, #6b7280);
+    color: var(--vp-ink-muted);
     font-size: 0.875rem;
 }
@@ -64,24 +118,24 @@
     gap: 0.5rem;
     padding-bottom: 0.25rem;
     margin-bottom: 0.25rem;
-    border-bottom: 1px solid var(--surface-border, rgba(127, 127, 127, 0.25));
+    border-bottom: 1px solid var(--vp-divider);
     position: sticky;
     top: 0;
-    background: var(--surface-overlay, #ffffff);
+    background: var(--vp-overlay);
     z-index: 1;
 }
 .ac-varpick__grouptitle {
-    font-size: 0.75rem;
+    font-size: var(--vp-font-xs);
     font-weight: 700;
     letter-spacing: 0.04em;
     text-transform: uppercase;
-    color: var(--text-color-secondary, #6b7280);
+    color: var(--vp-ink-muted);
 }
 .ac-varpick__grouphint {
-    font-size: 0.75rem;
-    color: var(--text-color-secondary, #8b95a1);
+    font-size: var(--vp-font-xs);
+    color: var(--vp-ink-subtle);
 }
 /*
@@ -102,21 +156,21 @@
     border: 1px solid transparent;
     border-radius: 3px;
     background: transparent;
-    color: var(--text-color, #111827);
+    color: var(--vp-ink);
     font: inherit;
     cursor: pointer;
 }
-.ac-varpick__row:hover { background: var(--surface-hover, rgba(127, 127, 127, 0.08)); }
+.ac-varpick__row:hover { background: var(--vp-hover); }
 .ac-varpick__row:focus-visible {
-    outline: 2px solid var(--primary-color, #2563eb);
+    outline: 2px solid var(--vp-focus-ring);
     outline-offset: -2px;
 }
 .ac-varpick__row--on {
-    background: var(--highlight-bg, rgba(37, 99, 235, 0.14));
-    border-color: var(--primary-color, #2563eb);
+    background: var(--vp-primary-subtle);
+    border-color: var(--vp-primary);
 }
 .ac-varpick__row--off { opacity: 0.55; cursor: default; }
@@ -130,19 +184,19 @@
 }
 .ac-varpick__name {
-    font-family: var(--font-family-mono, ui-monospace, SFMono-Regular, Menlo, monospace);
+    font-family: var(--vp-mono);
     font-weight: 600;
 }
 .ac-varpick__label { font-size: 0.875rem; }
 .ac-varpick__chip {
-    font-size: 0.75rem;
+    font-size: var(--vp-font-xs);
     padding: 0 6px;
     border-radius: 999px;
-    background: var(--surface-100, rgba(127, 127, 127, 0.12));
-    border: 1px solid var(--surface-border, rgba(127, 127, 127, 0.25));
-    color: var(--text-color-secondary, #6b7280);
+    background: var(--vp-sunk);
+    border: 1px solid var(--vp-border);
+    color: var(--vp-ink-muted);
     /* A unit symbol's case is meaningful: "mm/s", "N", "MPa". */
     text-transform: none;
 }
@@ -150,18 +204,18 @@
 /* The current reading. Nothing turns an unfamiliar name into an obvious one
  * faster than seeing what it says right now. */
 .ac-varpick__live {
-    font-family: var(--font-family-mono, ui-monospace, SFMono-Regular, Menlo, monospace);
-    font-size: 0.75rem;
-    color: var(--green-600, #15803d);
+    font-family: var(--vp-mono);
+    font-size: var(--vp-font-xs);
+    color: var(--vp-success);
 }
 .ac-varpick__desc {
-    font-size: 0.8125rem;
-    color: var(--text-color-secondary, #6b7280);
+    font-size: var(--vp-font-sm);
+    color: var(--vp-ink-muted);
     line-height: 1.35;
 }
-.ac-varpick__why { font-size: 0.8125rem; color: var(--orange-600, #b45309); }
-.ac-varpick__note { font-size: 0.8125rem; color: var(--blue-600, #0369a1); }
+.ac-varpick__why { font-size: var(--vp-font-sm); color: var(--vp-warning); }
+.ac-varpick__note { font-size: var(--vp-font-sm); color: var(--vp-info); }
 .ac-varpick__foot { display: flex; justify-content: flex-end; gap: 0.5rem; }

package/src/hooks/index.ts CHANGED Viewed

@@ -22,3 +22,7 @@ export type { UseGmVariablesResult, UseGmVariablesOptions, GmIpcInvoker } from '
 // a TIS cycle column, and the one nothing used to check.
 export { useSequenceProducers, collectStoredNames } from './useSequenceProducers';
 export type { UseSequenceProducersResult, UseSequenceProducersOptions } from './useSequenceProducers';
+export {
+    useMethodSequence, sequenceTopic, emptySequence, isSequenceShaped,
+} from './useMethodSequence';
+export type { MethodSequenceIo, UseMethodSequenceOptions } from './useMethodSequence';

package/src/hooks/useMethodSequence.ts ADDED Viewed

@@ -0,0 +1,128 @@
+/**
+ * useMethodSequence — read, write and delete the `.seq.json` half of a test
+ * method.
+ *
+ * ## One method, two files
+ *
+ * A sequence-backed test method is its `test_methods.json` schema entry PLUS
+ * `methods/<method_id>.seq.json`, the procedure the sequence engine runs. The
+ * filename is DERIVED from the method id and never stored, so the two halves
+ * cannot point at different things — see `autocore_tis::config::sequence_file`
+ * and `method_path()` in gen4_ac's `process_main.rs`, which spell the same
+ * derivation.
+ *
+ * Every method-lifecycle action therefore has to move both halves together:
+ *
+ *   New       — scaffold a blank procedure, or the method is unrunnable and
+ *               Start refuses with "has no sequence on this machine".
+ *   Duplicate — copy the procedure under the new id, or the copy silently runs
+ *               nothing.
+ *   Delete    — remove the procedure, or an orphan file is left behind that a
+ *               later method of the same name would inherit.
+ *   Im/Export — carry the procedure, because a schema whose cycle columns
+ *               nothing produces records blank rows forever.
+ *
+ * That last one is why import/export is a METHOD operation and not a sequence
+ * operation: half a method is not portable.
+ *
+ * The control program has no filesystem access, so all of this rides the
+ * datastore servelet over IPC, exactly as {@link useSequenceProducers} reads it.
+ */
+import { useCallback, useContext, useMemo } from 'react';
+import { EventEmitterContext } from '../core/EventEmitterContext';
+import { MessageType } from '../hub/CommandMessage';
+/** Where the sequence library lives in the datastore. Keep in step with
+ *  `useSequenceProducers`, `autocore_tis::config::sequence_file`, and
+ *  gen4_ac's `method_path()`. */
+const METHOD_DIR = 'methods';
+const METHOD_EXT = '.seq.json';
+/** The datastore topic for a method's procedure. */
+export function sequenceTopic(methodId: string): string {
+    return `datastore.${METHOD_DIR}/${methodId}${METHOD_EXT}`;
+}
+/** A blank procedure, shaped so `validateMethod` and the editor accept it. */
+export function emptySequence(methodId: string): Record<string, unknown> {
+    return { name: methodId, format_version: 1, steps: [] };
+}
+/** Is this datastore payload plausibly a `.seq.json` method? */
+export function isSequenceShaped(v: any): boolean {
+    return !!v && Array.isArray(v.steps);
+}
+export interface UseMethodSequenceOptions {
+    /** Override the IPC layer (tests, playground). */
+    invoker?: (topic: string, messageType: number, payload: object) => Promise<any>;
+}
+export interface MethodSequenceIo {
+    /** The procedure for `methodId`, or null when there is none. */
+    read: (methodId: string) => Promise<any | null>;
+    /**
+     * Write `sequence` as the procedure for `methodId`.
+     *
+     * `name` is rewritten to `methodId` on the way out, unconditionally. The
+     * method id owns the identity; the `name` inside the file is a leftover
+     * from when the sequence library named itself, and a copy that keeps its
+     * source's name is a file claiming to be a method it is not.
+     */
+    write: (methodId: string, sequence: any) => Promise<void>;
+    /** Remove the procedure for `methodId`. Resolves even if there was none. */
+    remove: (methodId: string) => Promise<void>;
+    /** Copy `from`'s procedure to `to`. No-op when `from` has none. */
+    copy: (from: string, to: string) => Promise<void>;
+}
+export function useMethodSequence(options?: UseMethodSequenceOptions): MethodSequenceIo {
+    const ctx = useContext(EventEmitterContext);
+    const invoke = options?.invoker
+        ?? ((topic: string, messageType: number, payload: object) =>
+            ctx.invoke(topic as any, messageType as any, payload as any));
+    const read = useCallback(async (methodId: string) => {
+        if (!methodId) return null;
+        try {
+            const resp: any = await invoke(sequenceTopic(methodId), MessageType.Read, {});
+            if (resp?.success === false) return null;
+            const seq = resp?.data ?? resp;
+            return isSequenceShaped(seq) ? seq : null;
+        } catch {
+            return null;
+        }
+        // eslint-disable-next-line react-hooks/exhaustive-deps
+    }, [ctx]);
+    const write = useCallback(async (methodId: string, sequence: any) => {
+        const value = { ...(sequence ?? {}), name: methodId };
+        await invoke(sequenceTopic(methodId), MessageType.Write, {
+            value,
+            options: { create_dirs: true },
+        });
+        // eslint-disable-next-line react-hooks/exhaustive-deps
+    }, [ctx]);
+    const remove = useCallback(async (methodId: string) => {
+        try {
+            await invoke(sequenceTopic(methodId), MessageType.Control, { action: 'delete' });
+        } catch {
+            // A method with no procedure is a legitimate state (a control-backed
+            // method, or one that was never authored). Failing the delete of the
+            // schema because the file it never had could not be removed would
+            // leave the operator unable to finish.
+        }
+        // eslint-disable-next-line react-hooks/exhaustive-deps
+    }, [ctx]);
+    const copy = useCallback(async (from: string, to: string) => {
+        const seq = await read(from);
+        if (seq) await write(to, seq);
+    }, [read, write]);
+    return useMemo(() => ({ read, write, remove, copy }), [read, write, remove, copy]);
+}
+export default useMethodSequence;

package/tools/tests/dist/{TestFieldDialog-BvHAr2GQ.js → TestFieldDialog-Dph2oPfe.js} RENAMED Viewed

@@ -31489,6 +31489,7 @@ var VariablePicker = ({ visible, catalog, title, hint, initialQuery, onCancel, o
 		visible,
 		onHide: onCancel,
 		footer,
+		className: "ac-varpick",
 		style: {
 			width: "44rem",
 			maxWidth: "95vw"

package/tools/tests/dist/autocore-react.css CHANGED Viewed

@@ -90,12 +90,66 @@
 /*
  * Variable picker.
  *
- * Colours come from the PrimeReact theme's variables so the dialog matches
- * whatever theme the machine is running, light or dark. Every value has a
- * literal fallback: an un-themed host (the playground, a test harness) must
- * still render legible text rather than black on black.
+ * ## The token bridge
+ *
+ * Every colour resolves `--ac-*` first, then the PrimeReact theme variable,
+ * then a literal. That ordering is the whole point:
+ *
+ *   - **`--ac-*`** is autocore's own design-token layer, which 4.0 supplies.
+ *     When it exists, this file needs no edit to follow it.
+ *   - **PrimeReact's `--surface-*` / `--text-color`** is what 3.5 actually
+ *     defines today, so the picker matches whatever theme the machine runs,
+ *     light or dark.
+ *   - **The literal** keeps an un-themed host legible — the playground, the
+ *     jsdom test harness — rather than rendering black on black.
+ *
+ * The token NAMES and their fallbacks are copied deliberately from
+ * `@adcops/autocore-seq-react`'s `styles.css`. The sequence editor's picker and
+ * this one sit one tab apart doing the identical thing; if they read different
+ * token names, 4.0 would have to satisfy two sets and the two panels could
+ * drift to different colours while both looked "correct".
+ *
+ * ## Why the tokens are declared twice
+ *
+ * The field lives inside the TIS editor's form, but PrimeReact renders the
+ * Dialog through a PORTAL to `<body>` — so the two halves are not in one
+ * subtree and a single scoping root cannot cover both. Hence the declaration on
+ * `.ac-varpick` (the dialog, which passes that className) and on
+ * `.ac-varpick__input` (the field). Declaring them on `:root` instead would
+ * push a dozen names into every consuming application's global scope for the
+ * benefit of one component.
  */
+.ac-varpick,
+.ac-varpick__input {
+    /* -- surfaces ---------------------------------------------------- */
+    --vp-sunk:           var(--ac-surface-sunk,         var(--surface-50,     #f8f9fa));
+    --vp-overlay:        var(--ac-surface-overlay,      var(--surface-overlay,#ffffff));
+    --vp-hover:          var(--ac-hover,                var(--surface-hover,  rgba(127, 127, 127, 0.08)));
+    /* -- ink --------------------------------------------------------- */
+    --vp-ink:            var(--ac-on-surface,           var(--text-color,           #111827));
+    --vp-ink-muted:      var(--ac-on-surface-muted,     var(--text-color-secondary, #6b7280));
+    --vp-ink-subtle:     var(--ac-on-surface-subtle,    var(--text-color-secondary, #8b95a1));
+    /* -- lines ------------------------------------------------------- */
+    --vp-border:         var(--ac-border,               var(--surface-border, rgba(127, 127, 127, 0.25)));
+    --vp-divider:        var(--ac-divider,              var(--surface-border, rgba(127, 127, 127, 0.18)));
+    /* -- accents ----------------------------------------------------- */
+    --vp-primary:        var(--ac-primary,              var(--primary-color,  #2563eb));
+    --vp-primary-subtle: var(--ac-primary-subtle,       var(--highlight-bg,   rgba(37, 99, 235, 0.14)));
+    --vp-success:        var(--ac-success,              var(--green-500,      #15803d));
+    --vp-warning:        var(--ac-warning,              var(--orange-500,     #b45309));
+    --vp-info:           var(--ac-info,                 var(--blue-500,       #0369a1));
+    --vp-focus-ring:     var(--ac-focus-ring,           rgba(37, 99, 235, 0.45));
+    /* -- type -------------------------------------------------------- */
+    --vp-mono:           var(--ac-font-mono,            ui-monospace, "Cascadia Mono", Menlo, Consolas, monospace);
+    --vp-font-sm:        var(--ac-text-sm,              0.8125rem);
+    --vp-font-xs:        var(--ac-text-xs,              0.75rem);
+}
 .ac-varpick__input {
     display: flex;
     align-items: stretch;
@@ -120,8 +174,8 @@
 }
 .ac-varpick__hint {
-    font-size: 0.8125rem;
-    color: var(--text-color-secondary, #6b7280);
+    font-size: var(--vp-font-sm);
+    color: var(--vp-ink-muted);
     margin-bottom: 0.75rem;
     line-height: 1.4;
 }
@@ -130,7 +184,7 @@
 .ac-varpick__empty {
     padding: 1.5rem 0.25rem;
-    color: var(--text-color-secondary, #6b7280);
+    color: var(--vp-ink-muted);
     font-size: 0.875rem;
 }
@@ -153,24 +207,24 @@
     gap: 0.5rem;
     padding-bottom: 0.25rem;
     margin-bottom: 0.25rem;
-    border-bottom: 1px solid var(--surface-border, rgba(127, 127, 127, 0.25));
+    border-bottom: 1px solid var(--vp-divider);
     position: sticky;
     top: 0;
-    background: var(--surface-overlay, #ffffff);
+    background: var(--vp-overlay);
     z-index: 1;
 }
 .ac-varpick__grouptitle {
-    font-size: 0.75rem;
+    font-size: var(--vp-font-xs);
     font-weight: 700;
     letter-spacing: 0.04em;
     text-transform: uppercase;
-    color: var(--text-color-secondary, #6b7280);
+    color: var(--vp-ink-muted);
 }
 .ac-varpick__grouphint {
-    font-size: 0.75rem;
-    color: var(--text-color-secondary, #8b95a1);
+    font-size: var(--vp-font-xs);
+    color: var(--vp-ink-subtle);
 }
 /*
@@ -191,21 +245,21 @@
     border: 1px solid transparent;
     border-radius: 3px;
     background: transparent;
-    color: var(--text-color, #111827);
+    color: var(--vp-ink);
     font: inherit;
     cursor: pointer;
 }
-.ac-varpick__row:hover { background: var(--surface-hover, rgba(127, 127, 127, 0.08)); }
+.ac-varpick__row:hover { background: var(--vp-hover); }
 .ac-varpick__row:focus-visible {
-    outline: 2px solid var(--primary-color, #2563eb);
+    outline: 2px solid var(--vp-focus-ring);
     outline-offset: -2px;
 }
 .ac-varpick__row--on {
-    background: var(--highlight-bg, rgba(37, 99, 235, 0.14));
-    border-color: var(--primary-color, #2563eb);
+    background: var(--vp-primary-subtle);
+    border-color: var(--vp-primary);
 }
 .ac-varpick__row--off { opacity: 0.55; cursor: default; }
@@ -219,19 +273,19 @@
 }
 .ac-varpick__name {
-    font-family: var(--font-family-mono, ui-monospace, SFMono-Regular, Menlo, monospace);
+    font-family: var(--vp-mono);
     font-weight: 600;
 }
 .ac-varpick__label { font-size: 0.875rem; }
 .ac-varpick__chip {
-    font-size: 0.75rem;
+    font-size: var(--vp-font-xs);
     padding: 0 6px;
     border-radius: 999px;
-    background: var(--surface-100, rgba(127, 127, 127, 0.12));
-    border: 1px solid var(--surface-border, rgba(127, 127, 127, 0.25));
-    color: var(--text-color-secondary, #6b7280);
+    background: var(--vp-sunk);
+    border: 1px solid var(--vp-border);
+    color: var(--vp-ink-muted);
     /* A unit symbol's case is meaningful: "mm/s", "N", "MPa". */
     text-transform: none;
 }
@@ -239,19 +293,19 @@
 /* The current reading. Nothing turns an unfamiliar name into an obvious one
  * faster than seeing what it says right now. */
 .ac-varpick__live {
-    font-family: var(--font-family-mono, ui-monospace, SFMono-Regular, Menlo, monospace);
-    font-size: 0.75rem;
-    color: var(--green-600, #15803d);
+    font-family: var(--vp-mono);
+    font-size: var(--vp-font-xs);
+    color: var(--vp-success);
 }
 .ac-varpick__desc {
-    font-size: 0.8125rem;
-    color: var(--text-color-secondary, #6b7280);
+    font-size: var(--vp-font-sm);
+    color: var(--vp-ink-muted);
     line-height: 1.35;
 }
-.ac-varpick__why { font-size: 0.8125rem; color: var(--orange-600, #b45309); }
-.ac-varpick__note { font-size: 0.8125rem; color: var(--blue-600, #0369a1); }
+.ac-varpick__why { font-size: var(--vp-font-sm); color: var(--vp-warning); }
+.ac-varpick__note { font-size: var(--vp-font-sm); color: var(--vp-info); }
 .ac-varpick__foot { display: flex; justify-content: flex-end; gap: 0.5rem; }
 /*$vite$:1*/

package/tools/tests/dist/tis-editor.entry.js CHANGED Viewed

@@ -1,4 +1,4 @@
-import { A as useMergeProps, B as PrimeReact$2, C as Portal, D as ComponentBase, E as IconBase, F as useUnmountEffect, G as IconUtils, H as ariaLabel$2, I as useUpdateEffect, J as ZIndexUtils, K as ObjectUtils, L as FilterMatchMode$2, M as useOverlayListener, N as usePrevious, O as useHandleStyle, P as useStyle, Q as __toESM, R as FilterOperator, S as Tooltip, T as SpinnerIcon, U as localeOption, V as PrimeReactContext, W as DomHandler, X as require_client, Y as classNames, Z as require_react, _ as OverlayService, b as InputNumber, c as useEditDraft, d as require_jsx_runtime, f as Button, g as CSSTransition, h as VirtualScroller, i as chartFieldCatalog, j as useMountEffect, k as useEventListener, l as FormRow, m as CheckIcon, p as Dropdown, q as UniqueComponentId, s as VariableInput, t as TestFieldDialog, u as Dialog, v as TimesIcon, w as Ripple, x as InputText, y as ChevronDownIcon, z as FilterService } from "./TestFieldDialog-BvHAr2GQ.js";
+import { A as useMergeProps, B as PrimeReact$2, C as Portal, D as ComponentBase, E as IconBase, F as useUnmountEffect, G as IconUtils, H as ariaLabel$2, I as useUpdateEffect, J as ZIndexUtils, K as ObjectUtils, L as FilterMatchMode$2, M as useOverlayListener, N as usePrevious, O as useHandleStyle, P as useStyle, Q as __toESM, R as FilterOperator, S as Tooltip, T as SpinnerIcon, U as localeOption, V as PrimeReactContext, W as DomHandler, X as require_client, Y as classNames, Z as require_react, _ as OverlayService, b as InputNumber, c as useEditDraft, d as require_jsx_runtime, f as Button, g as CSSTransition, h as VirtualScroller, i as chartFieldCatalog, j as useMountEffect, k as useEventListener, l as FormRow, m as CheckIcon, p as Dropdown, q as UniqueComponentId, s as VariableInput, t as TestFieldDialog, u as Dialog, v as TimesIcon, w as Ripple, x as InputText, y as ChevronDownIcon, z as FilterService } from "./TestFieldDialog-Dph2oPfe.js";
 //#region node_modules/primereact/icons/arrowdown/index.esm.js
 var import_client = require_client();
 var import_react = /* @__PURE__ */ __toESM(require_react());

package/tools/tests/dist/varpick.entry.js CHANGED Viewed

@@ -1,4 +1,4 @@
-import { Q as __toESM, X as require_client, Z as require_react, a as configSourceCatalog, d as require_jsx_runtime, i as chartFieldCatalog, n as AuthoringContext, o as cycleSourceCatalog, r as boundCatalog, t as TestFieldDialog } from "./TestFieldDialog-BvHAr2GQ.js";
+import { Q as __toESM, X as require_client, Z as require_react, a as configSourceCatalog, d as require_jsx_runtime, i as chartFieldCatalog, n as AuthoringContext, o as cycleSourceCatalog, r as boundCatalog, t as TestFieldDialog } from "./TestFieldDialog-Dph2oPfe.js";
 //#region src/hooks/useSequenceProducers.ts
 var import_client = require_client();
 var import_react = /* @__PURE__ */ __toESM(require_react(), 1);

package/tools/tests/varpick.test.mjs CHANGED Viewed

@@ -173,6 +173,11 @@ check('typing still works', inputIn('Source').value, 'gm.stale_binding');
 await click(pickerButtonIn('Source'));
 ok('the picker opened', dialogs().length > 1, `dialogs=${dialogs().length}`);
+// The dialog is PORTALED to <body>, so it inherits nothing from the field's
+// subtree — the token bridge in varpick.css is declared on this class, and
+// without it every colour would fall through to its literal.
+ok('the dialog carries the token-bridge class',
+    picker().classList.contains('ac-varpick'), picker().className);
 ok('the sequence group is present',
     picker().textContent.includes('Recorded by the sequence'));