nudge-ui 0.2.5 → 0.2.6

package/dist/inspector/canvas/rendererMessageRouter.d.ts

@@ -5,6 +5,5 @@ export interface CanvasRendererMessage {
     message: RendererToControllerMessage;
 }
 type CanvasRendererMessageListener = (event: CanvasRendererMessage) => void;
-/** Subscribes to same-origin messages from registered Canvas renderer frames. */
 export declare function subscribeCanvasRendererMessages(listener: CanvasRendererMessageListener): () => void;
 export {};

package/dist/inspector/canvas/rendererMessageRouter.js

@@ -26,7 +26,6 @@ function routeRendererMessage(event) {
     for (const listener of listeners)
         listener(routed);
 }
-/** Subscribes to same-origin messages from registered Canvas renderer frames. */
 export function subscribeCanvasRendererMessages(listener) {
     if (listeners.size === 0)
         window.addEventListener("message", routeRendererMessage);

package/dist/inspector/canvas/rendererSelectionProxy.d.ts

@@ -1,8 +1,4 @@
 import type { ElementClickMessage } from "./frameProtocol.js";
 import type { FrameConnection } from "./frameConnection.js";
-/**
- * Selects the exact DOM node the user clicked inside a card's frame, so the
- * controller mutates that physical node. If it no longer exists, the card is
- * selected instead of binding the click to an unrelated node.
- */
+/** Selects the exact DOM node the user clicked. Falls back to the card when that node is gone. */
 export declare function handleElementClick(msg: ElementClickMessage, frame: FrameConnection): void;

package/dist/inspector/canvas/rendererSelectionProxy.js

@@ -1,11 +1,7 @@
 import { getSelectedElements, setSelectedElement, toggleSelectedElement } from "../selection/selectionStore.js";
 import { selectCard } from "./canvasStore.js";
 import { inspectComponentTargets } from "../componentSemantics/index.js";
-/**
- * Selects the exact DOM node the user clicked inside a card's frame, so the
- * controller mutates that physical node. If it no longer exists, the card is
- * selected instead of binding the click to an unrelated node.
- */
+/** Selects the exact DOM node the user clicked. Falls back to the card when that node is gone. */
 export function handleElementClick(msg, frame) {
     const el = frame.element(msg);
     if (!el) {
@@ -24,8 +20,7 @@ export function handleElementClick(msg, frame) {
         domElement: el,
         componentTargets,
     };
-    // setSelectedElement must run before any side effect that could reload the
-    // host page, otherwise the selection would be discarded by the unload.
+    // Must run before any side effect that could reload the host page, which would drop the selection.
     const current = getSelectedElements();
     if (msg.additive && current.length > 0 && current.every((candidate) => candidate.domElement.ownerDocument === el.ownerDocument)) {
         toggleSelectedElement(selected);

package/dist/inspector/canvas/rendererStylesheet.d.ts

@@ -1,5 +1,4 @@
 import { type ReplaceStylesMessage } from "./frameProtocol.js";
-/** Installs renderer diagnostics only after the dev-only renderer bootstrap. */
 export declare function startRendererProjectionDiagnostics(): () => void;
 export declare function getLastAppliedRevision(): number;
 export declare function resetRendererRevision(): void;

package/dist/inspector/canvas/rendererStylesheet.js

@@ -9,16 +9,24 @@ import { applyStructuralProjection, getStructuralProjectionReports, isStructural
 import { applyTextContentProjection, getTextProjectionReports, subscribeTextProjectionDiagnostics, } from "../projection/textProjection.js";
 import { isTextContentChangeListValue } from "../changes/types.js";
 import { applyDraftProjection, } from "../projection/draftProjection.js";
-import { adoptStylesheet, allowsStyleElements } from "../runtime/ownedStylesheets.js";
+import { adoptStylesheet, allowsStyleElements, getAdoptedStylesheet } from "../runtime/ownedStylesheets.js";
 import { applyRuntimeIdentityClaims, isRuntimeIdentityClaim } from "../runtime/runtimeIdentityClaims.js";
 const SHEET_ID = "nudge-ui-styles";
+let adoptedCss = null;
 function replaceRendererCss(css) {
     if (!allowsStyleElements(document)) {
+        // Structural and text projections resend unchanged CSS; rewriting it would discard cached resolutions.
+        const adopted = getAdoptedStylesheet(document, SHEET_ID);
+        if (adopted && adoptedCss === css && document.adoptedStyleSheets.includes(adopted))
+            return;
         adoptStylesheet(document, SHEET_ID, css);
+        adoptedCss = css;
         notifyBrowserStylesheetChange(document);
         return;
     }
     let sheet = document.getElementById(SHEET_ID);
+    if (sheet?.textContent === css)
+        return;
     if (!sheet) {
         sheet = document.createElement("style");
         sheet.id = SHEET_ID;
@@ -124,7 +132,6 @@ function sendProjectionApplied(revision) {
     };
     sendToParent(msg);
 }
-/** Installs renderer diagnostics only after the dev-only renderer bootstrap. */
 export function startRendererProjectionDiagnostics() {
     if (!isNudgeUiDev() || activeDiagnosticsOwner)
         return () => undefined;

package/dist/inspector/canvas/resizeHandleScale.d.ts

@@ -1,13 +1,4 @@
-/**
- * Keep the resize handle usable while the board is zoomed without allowing
- * inverse scaling to make it disproportionately large or small.
- */
+/** Keeps the resize handle usable at any zoom without letting inverse scaling make it disproportionately large or small. */
 export declare const CANVAS_RESIZE_HANDLE_MIN_SCALE: number;
 export declare const CANVAS_RESIZE_HANDLE_MAX_SCALE = 2;
-/**
- * Returns the scale to apply inside the board's zoomed coordinate space.
- *
- * The lower bound matches the inverse of the largest supported camera zoom,
- * and the upper bound follows the toolbar compensation policy.
- */
 export declare function getCanvasResizeHandleScale(zoom: number): number;

package/dist/inspector/canvas/resizeHandleScale.js

@@ -1,17 +1,8 @@
 import { MAX_CAMERA_ZOOM } from "./viewStore.js";
 import { CANVAS_TOOLBAR_MAX_SCALE } from "./toolbarScale.js";
-/**
- * Keep the resize handle usable while the board is zoomed without allowing
- * inverse scaling to make it disproportionately large or small.
- */
+/** Keeps the resize handle usable at any zoom without letting inverse scaling make it disproportionately large or small. */
 export const CANVAS_RESIZE_HANDLE_MIN_SCALE = 1 / MAX_CAMERA_ZOOM;
 export const CANVAS_RESIZE_HANDLE_MAX_SCALE = CANVAS_TOOLBAR_MAX_SCALE;
-/**
- * Returns the scale to apply inside the board's zoomed coordinate space.
- *
- * The lower bound matches the inverse of the largest supported camera zoom,
- * and the upper bound follows the toolbar compensation policy.
- */
 export function getCanvasResizeHandleScale(zoom) {
     if (!Number.isFinite(zoom) || zoom <= 0) {
         return CANVAS_RESIZE_HANDLE_MAX_SCALE;

package/dist/inspector/canvas/sessionStore.d.ts

@@ -33,7 +33,6 @@ export interface DurableSession {
 export type { SerializableChange, SerializableComponentChange, SerializableElementChange, SerializableTextContentChange, SerializableTokenChange, SerializableTokenRef, } from "../changes/codecs.js";
 export interface HydrationResult {
     restored: boolean;
-    changeCount: number;
 }
 export declare function serializeSession(): DurableSession;
 export declare function persistSession(): boolean;
@@ -42,13 +41,8 @@ export declare function hydrateSession(): HydrationResult;
 export declare function clearSelectedFrameChanges(): Promise<void>;
 export declare function clearSession(): void;
 export declare function enableAutoSave(): void;
-/**
- * Coalesces the synchronous full-session `JSON.stringify` + `localStorage`
- * write behind a trailing timer so commits never block on persistence. A
- * refresh or close inside the debounce window still flushes on `beforeunload`.
- */
+/** Coalesces the synchronous full-session write behind a trailing timer; `beforeunload` still flushes it. */
 export declare function scheduleAutoSave(): void;
-export declare function scheduleCanvasSave(): void;
 export declare function flushAutoSave(): void;
 export declare function getRestoreCount(): number;
 export declare function setRestoreCount(count: number): void;

package/dist/inspector/canvas/sessionStore.js

@@ -70,10 +70,7 @@ export function persistSession() {
         return false;
     return persistSessionUnchecked();
 }
-/**
- * Serializes and writes the session without the write-lease gate. The public
- * persistence path checks the lease before calling this helper.
- */
+/** Skips the write-lease gate; the public persistence path checks the lease first. */
 function persistSessionUnchecked() {
     if (!projectId())
         return false;
@@ -84,56 +81,56 @@ function persistSessionUnchecked() {
         return true;
     }
     catch {
-        // Storage unavailable or quota exceeded — silently ignore
+        // Storage unavailable or quota exceeded
         return false;
     }
 }
 export function hydrateSession() {
     if (!projectId())
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     let raw = null;
     try {
         raw = inspectorStorage()?.getItem(storageKey(projectId())) ?? null;
     }
     catch {
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     if (!raw)
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     let session;
     try {
         session = JSON.parse(raw);
     }
     catch {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     if (!session || typeof session !== "object") {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     const s = session;
     if (typeof s.schemaVersion !== "number" || s.schemaVersion !== SCHEMA_VERSION) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     if (typeof s.projectId !== "string" || s.projectId !== projectId()) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     if (typeof s.mode !== "string" || (s.mode !== "inspect" && s.mode !== "canvas")) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     if (!isSameOriginUrl(s.inspectUrl)) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     const inspectRouteChanged = s.mode === "inspect" && s.inspectUrl !== window.location.href;
     const cards = Array.isArray(s.cards) ? s.cards : null;
     if (!cards) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     const serializableCards = [];
     for (const card of cards) {
@@ -148,7 +145,7 @@ export function hydrateSession() {
             candidate.width <= 0 ||
             candidate.height <= 0) {
             safeDiscard();
-            return { restored: false, changeCount: 0 };
+            return { restored: false };
         }
         const c = card;
         serializableCards.push({
@@ -164,7 +161,7 @@ export function hydrateSession() {
     }
     if (new Set(serializableCards.map((card) => card.id)).size !== serializableCards.length) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     const cardsById = new Map(serializableCards.map((card) => [card.id, card]));
     const cameraRaw = s.camera;
@@ -173,18 +170,18 @@ export function hydrateSession() {
         !isFiniteNumber(cameraRaw.y) ||
         !isFiniteNumber(cameraRaw.zoom)) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     const presentation = s.presentation ?? "focus";
     const groups = Array.isArray(s.groups) ? s.groups.filter((group) => group?.kind !== "linked") : s.groups ?? [];
     if ((presentation !== "focus" && presentation !== "canvas") || !Array.isArray(groups) || !groups.every(isFrameGroup)) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     const clipboardHandoff = s.clipboardHandoff;
     if (!isClipboardHandoffSnapshot(clipboardHandoff)) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     const camera = {
         x: cameraRaw.x,
@@ -198,24 +195,20 @@ export function hydrateSession() {
             : undefined;
     if (focusedCardId === undefined) {
         safeDiscard();
-        return { restored: false, changeCount: 0 };
+        return { restored: false };
     }
     hydrateCanvasStore(s.mode, serializableCards, camera, focusedCardId, presentation, groups);
     hydrateClipboardHandoff(clipboardHandoff);
-    // A different URL means the user intentionally navigated while Inspect was
-    // active. Keep the durable edits, but adopt the new route instead of
-    // sending the user back to the previous page. On refresh, the URLs already
-    // match and restoration remains unchanged.
+    // A changed URL means the user navigated while Inspect was active: keep the edits but adopt the new route.
     if (inspectRouteChanged)
         persistSessionUnchecked();
-    return { restored: true, changeCount: 0 };
+    return { restored: true };
 }
 function safeDiscard() {
     try {
         inspectorStorage()?.removeItem(storageKey(projectId()));
     }
     catch {
-        // ignore
     }
 }
 /** Clears the active draft, its comments, and project sketches while preserving canvas geometry and other drafts. */
@@ -242,7 +235,6 @@ export function clearSession() {
         inspectorStorage()?.removeItem(storageKey(projectId()));
     }
     catch {
-        // ignore
     }
     clearComments();
     clearActiveDraft();
@@ -272,11 +264,7 @@ export function enableAutoSave() {
     autoSaveEnabled = true;
     window.addEventListener("beforeunload", flushAutoSave);
 }
-/**
- * Coalesces the synchronous full-session `JSON.stringify` + `localStorage`
- * write behind a trailing timer so commits never block on persistence. A
- * refresh or close inside the debounce window still flushes on `beforeunload`.
- */
+/** Coalesces the synchronous full-session write behind a trailing timer; `beforeunload` still flushes it. */
 export function scheduleAutoSave() {
     if (!autoSaveEnabled)
         return;
@@ -291,7 +279,6 @@ export function scheduleAutoSave() {
             autoSaveDirty = false;
     }, AUTOSAVE_DEBOUNCE_MS);
 }
-export function scheduleCanvasSave() { scheduleAutoSave(); }
 export function flushAutoSave() {
     if (autosaveTimer !== null) {
         clearTimeout(autosaveTimer);

package/dist/inspector/canvas/staleChangeDetector.js

@@ -148,10 +148,7 @@ function applyStaleResults(changes, attempts) {
             continue;
         }
         const routes = evidence.get(i);
-        // A document that does not contain the target says nothing about
-        // staleness. Publish target-missing only when the change matches nowhere;
-        // a Canvas-only match is verified, not a host conflict. When nowhere,
-        // every live document reports the same miss.
+        // Publish target-missing only when no live document matches; a Canvas-only match is verified, not a host conflict.
         if (!routes || routes.length === 0) {
             for (const attempt of attempts.values()) {
                 publishPreviewDiagnostic(attempt, changeKey(change), buildStaleResult(change.selector, getRequestedValue(change)));

package/dist/inspector/canvas/toolbarScale.d.ts

@@ -1,7 +1,4 @@
-/**
- * Keep canvas card controls usable while the board is zoomed out without
- * letting them grow without bound relative to the card.
- */
+/** Keeps card controls usable while the board is zoomed out, without letting them outgrow the card. */
 export declare const CANVAS_TOOLBAR_MAX_SCALE = 2;
 export declare const CANVAS_TOOLBAR_SCALE_CAP_ZOOM: number;
 export declare function getCanvasToolbarScale(zoom: number): number;

package/dist/inspector/canvas/toolbarScale.js

@@ -1,7 +1,4 @@
-/**
- * Keep canvas card controls usable while the board is zoomed out without
- * letting them grow without bound relative to the card.
- */
+/** Keeps card controls usable while the board is zoomed out, without letting them outgrow the card. */
 export const CANVAS_TOOLBAR_MAX_SCALE = 2;
 export const CANVAS_TOOLBAR_SCALE_CAP_ZOOM = 1 / CANVAS_TOOLBAR_MAX_SCALE;
 export function getCanvasToolbarScale(zoom) {

package/dist/inspector/canvas/useCanvasLayoutAnimation.d.ts

@@ -1,5 +1,4 @@
 import { type RefObject } from "react";
 import { type CanvasCard } from "./canvasStore.js";
 import { type CanvasCamera, type CanvasPresentation } from "./viewStore.js";
-/** Animates frame creation and layout changes without delaying direct manipulation. */
 export declare function useCanvasLayoutAnimation(boardRef: RefObject<HTMLDivElement | null>, cards: readonly CanvasCard[], camera: CanvasCamera, presentation: CanvasPresentation): void;

package/dist/inspector/canvas/useCanvasLayoutAnimation.js

@@ -4,7 +4,6 @@ import {} from "./canvasStore.js";
 import { setCanvasLayoutTransitioning } from "./viewStore.js";
 const SLIDE_OPTIONS = { duration: 300, easing: "ease-out" };
 const CAMERA_OPTIONS = { duration: 150, easing: "ease-out" };
-/** Animates frame creation and layout changes without delaying direct manipulation. */
 export function useCanvasLayoutAnimation(boardRef, cards, camera, presentation) {
     const previous = useRef({ cards, camera, presentation });
     const animations = useRef([]);

package/dist/inspector/canvas/viewStore.js

@@ -2,7 +2,7 @@ import { useSyncExternalStore } from "react";
 export const MIN_CAMERA_ZOOM = 0.25;
 export const MAX_CAMERA_ZOOM = 3;
 export const DEFAULT_CAMERA = { x: 0, y: 0, zoom: 1 };
-/** How the canvas is viewed. Frames, groups, and selection live in the canvas store. */
+/** Frames, groups, and selection live in the canvas store. */
 let mode = "inspect";
 let presentation = "focus";
 let presentationTransitioning = false;

package/dist/inspector/canvas/wheelPan.d.ts

@@ -1,4 +1,4 @@
-/** Converts wheel units to pixel distances for consistent canvas panning. */
+/** deltaMode 1 is lines (16px) and 2 is pages, so both need converting to pixels. */
 export declare function wheelPanDelta(event: Pick<WheelEvent, "deltaX" | "deltaY" | "deltaMode" | "shiftKey">, viewport: {
     width: number;
     height: number;

package/dist/inspector/canvas/wheelPan.js

@@ -1,4 +1,4 @@
-/** Converts wheel units to pixel distances for consistent canvas panning. */
+/** deltaMode 1 is lines (16px) and 2 is pages, so both need converting to pixels. */
 export function wheelPanDelta(event, viewport) {
     const xUnit = event.deltaMode === 1 ? 16 : event.deltaMode === 2 ? viewport.width : 1;
     const yUnit = event.deltaMode === 1 ? 16 : event.deltaMode === 2 ? viewport.height : 1;

package/dist/inspector/canvas/workspaceLease.d.ts

@@ -10,11 +10,7 @@ export declare function acquireLease(): boolean;
 export declare function requestTakeover(): boolean;
 /** Enable lease checks for state-changing controller actions in this document. */
 export declare function enableWriteGuard(): void;
-/**
- * Unit-level modules are also used without the browser controller bootstrap.
- * Once the controller enables the guard, only the current lease owner may
- * mutate or persist its canonical workspace state.
- */
+/** Off by default so unit-level modules work without the controller bootstrap. */
 export declare function canWriteWorkspace(): boolean;
 export declare function releaseLease(): void;
 export declare function hasWriteLease(): boolean;

package/dist/inspector/canvas/workspaceLease.js

@@ -49,7 +49,6 @@ function deleteLease() {
         localStorage.removeItem(leaseKey());
     }
     catch {
-        /* ignore */
     }
 }
 export function isLeaseExpired(lease, now = Date.now()) {
@@ -67,7 +66,6 @@ function notifyOwnershipListeners() {
             listener(snapshot);
         }
         catch {
-            /* ignore */
         }
     }
 }
@@ -149,11 +147,7 @@ export function requestTakeover() {
 export function enableWriteGuard() {
     writeGuardEnabled = true;
 }
-/**
- * Unit-level modules are also used without the browser controller bootstrap.
- * Once the controller enables the guard, only the current lease owner may
- * mutate or persist its canonical workspace state.
- */
+/** Off by default so unit-level modules work without the controller bootstrap. */
 export function canWriteWorkspace() {
     return !writeGuardEnabled || hasWriteLease();
 }

package/dist/inspector/changes/codecs.js

@@ -35,7 +35,8 @@ export function isSerializableElementChangeValue(value) {
         || (value.runtimeEvidence !== undefined && !isRuntimeElementEvidenceValue(value.runtimeEvidence))
         || (value.sourceEvidence !== undefined && !isSourceElementEvidenceValue(value.sourceEvidence))
         || !isEditScopeValue(value.scope)
-        || (value.sourceInstanceCount !== undefined && (!Number.isSafeInteger(value.sourceInstanceCount) || value.sourceInstanceCount < 1))
+        || (value.sourceInstanceCount !== undefined
+            && (!isNonNegativeSafeInteger(value.sourceInstanceCount) || value.sourceInstanceCount < 1))
         || (value.instanceOverride !== undefined && value.scope !== "rendered-instance")
         || (value.scope === "rendered-instance" && !isRenderedInstanceOverride(value.instanceOverride))
         || (value.state !== undefined && !isElementStateValue(value.state))) {

package/dist/inspector/changes/draftPreview.d.ts

@@ -1,12 +1,7 @@
 import type { DraftChangesSnapshot, DraftProjectionEvent } from "../drafts/draftStore.js";
-/** The host document's preview of the active draft. */
 export interface DraftPreview {
     project(snapshot: DraftChangesSnapshot, event: DraftProjectionEvent): void;
     dispose(): void;
 }
-/**
- * Projects the active draft into the host document and verifies the managed
- * CSS off the commit path (0044). Verification re-checks the full surviving
- * set: a new rule can change cascade or specificity for earlier changes.
- */
+/** Verification re-checks the full surviving set: a new rule can change cascade or specificity for earlier changes. */
 export declare function createDraftPreview(readActive: () => DraftChangesSnapshot): DraftPreview;

package/dist/inspector/changes/draftPreview.js

@@ -1,23 +1,16 @@
 import { getSelectedElement } from "../selection/selectionStore.js";
 import { changeKey } from "./model.js";
-import { verifyManagedStyleProjection } from "./managedStyleProjection.js";
-import { isPreviewableChange } from "./types.js";
-import { beginPreviewAttempt, clearPreviewDiagnostics, getHostPreviewDocument, publishPreviewDiagnostic, } from "./previewDiagnostics.js";
+import { beginPreviewAttempt, clearPreviewDiagnostics, getHostPreviewDocument, } from "./previewDiagnostics.js";
+import { verifyPreviewChanges } from "./previewVerification.js";
 import { clearStructuralProjectionReports, pruneStructuralProjectionReports } from "../projection/structuralProjection.js";
 import { applyHostDraftProjection, compileDraftProjection } from "../projection/draftProjection.js";
 import { isEditorShellDocument } from "../runtime/editorShell.js";
 import { isOriginalPreviewActive, subscribeOriginalPreview } from "../shell/originalPreview.js";
-/**
- * Projects the active draft into the host document and verifies the managed
- * CSS off the commit path (0044). Verification re-checks the full surviving
- * set: a new rule can change cascade or specificity for earlier changes.
- */
+/** Verification re-checks the full surviving set: a new rule can change cascade or specificity for earlier changes. */
 export function createDraftPreview(readActive) {
     let pendingTargets = new Map();
     let handle = null;
     function reapply(snapshot) {
-        // While hold-to-view-original is active the host document intentionally
-        // shows the page without inspector changes. Releasing re-projects.
         if (isOriginalPreviewActive())
             return;
         if (!isEditorShellDocument())
@@ -25,30 +18,25 @@ export function createDraftPreview(readActive) {
     }
     function flushVerification() {
         handle = null;
-        // Probing the original preview would report the held-back changes as
-        // conflicts. Keep the pending targets; releasing re-verifies them.
+        // Probing the original preview would report held-back changes as conflicts; keep the targets for re-verification on release.
         if (isOriginalPreviewActive())
             return;
         const targets = pendingTargets;
         pendingTargets = new Map();
-        // The editor shell has no application document. Renderer diagnostics are
-        // owned by renderer projection reports.
+        // The editor shell has no application document to verify; renderer projection reports cover it.
         if (targets.size === 0 || isEditorShellDocument())
             return;
         const current = readActive();
         const attempt = beginPreviewAttempt(getHostPreviewDocument(), current.revision);
         if (!attempt)
             return;
-        for (const change of current.changes) {
-            if (!isPreviewableChange(change))
-                continue;
-            const key = changeKey(change);
-            if (!targets.has(key))
-                continue;
-            const result = verifyManagedStyleProjection(change, targets.get(key) ?? null);
-            if (result)
-                publishPreviewDiagnostic(attempt, key, result);
-        }
+        const targeted = current.changes.filter((change) => targets.has(changeKey(change)));
+        verifyPreviewChanges({
+            changes: targeted,
+            document,
+            attempt,
+            targetFor: (change) => targets.get(changeKey(change)) ?? null,
+        });
     }
     function scheduleVerification() {
         if (handle !== null)
@@ -70,8 +58,7 @@ export function createDraftPreview(readActive) {
         for (const key of snapshot.changes.map(changeKey)) {
             if (!pendingTargets.has(key))
                 added = true;
-            // Keep the latest commit context for a merged key. The element may be in
-            // an iframe, so this is also the document for verification.
+            // Latest commit context wins for a merged key; the element may live in an iframe, which fixes the verification document.
             pendingTargets.set(key, verificationTargets?.get(key) ?? selectedElement);
         }
         if (added)
@@ -93,8 +80,7 @@ export function createDraftPreview(readActive) {
             return;
         }
         reapply(snapshot);
-        // A flush during the hold kept its targets without scheduling, so marking
-        // alone would find every key present and schedule nothing.
+        // A flush during the hold kept its targets without scheduling, so markForVerification would find every key present and schedule nothing.
         markForVerification(snapshot);
         if (pendingTargets.size > 0)
             scheduleVerification();
@@ -117,8 +103,7 @@ export function createDraftPreview(readActive) {
                     markForVerification(snapshot);
                     return;
                 case "reset":
-                    // A deferred verification must not inspect a new record with the
-                    // selection context of the cleared one.
+                    // A deferred verification must not inspect a new record with the selection context of the cleared one.
                     pendingTargets.clear();
                     clearPreviewDiagnostics();
                     clearStructuralProjectionReports();

package/dist/inspector/changes/editModel.d.ts

@@ -1,14 +1,11 @@
 import type { TokenContextWrapper } from "../../css/model/index.js";
 export type EditScope = "source-site" | "rendered-instance";
-/** A JSX instrumentation site. This is stable across documents. */
+/** JSX instrumentation site; stable across documents. */
 export interface SourceSiteRef {
     cid: string;
     src: string;
 }
-/**
- * Serializable identity for one rendered output. It deliberately contains no
- * DOM reference or generated node id: each document resolves it independently.
- */
+/** Deliberately free of DOM references and generated node ids; each document resolves it independently. */
 export interface RenderedInstanceRef {
     sourceSite: SourceSiteRef;
     locator: {
@@ -22,14 +19,10 @@ export interface RenderedInstanceRef {
         tagName?: string;
         domPath?: string | null;
         elementId?: string | null;
-        /**
-         * The component invocation whose entire output is this element. Present
-         * only when evidence alone cannot tell identical outputs apart.
-         */
+        /** Present only when evidence alone cannot tell identical outputs apart. */
         invocation?: InvocationRef;
     };
 }
-/** A component callsite, as stamped by the compiler, and its component name. */
 export interface InvocationRef {
     callsiteId: string;
     componentName: string;
@@ -44,9 +37,7 @@ export interface StyleRuleContext {
 }
 export declare function isRenderedInstanceRef(value: unknown): value is RenderedInstanceRef;
 export declare function isRenderedInstanceOverride(value: unknown): value is RenderedInstanceOverride;
-/** Validates the serializable context carried by a token change. */
 export declare function isStyleRuleContextValue(value: unknown): value is StyleRuleContext;
-/** A source site shared by text and rendered-instance evidence. */
 export type TextProjectionSourceSite = SourceSiteRef;
 export interface TextProjectionTarget {
     sourceSite: TextProjectionSourceSite;
@@ -62,7 +53,6 @@ export interface TextProjectionTarget {
     textNodePath?: readonly number[];
 }
 export type TextProjectionScope = EditScope;
-/** Bounded semantic evidence retained alongside an instance text projection. */
 export interface TextBindingEvidence {
     callsiteId: string;
     componentName: string;

package/dist/inspector/changes/editModel.js

@@ -17,7 +17,7 @@ export function isRenderedInstanceRef(value) {
         || !hasOnlyKeys(locator, ["kind", "occurrence", "props", "text", "ariaLabel", "tagName", "domPath", "elementId", "invocation"]))
         return false;
     return locator.kind === "evidence"
-        && Number.isSafeInteger(locator.occurrence) && locator.occurrence >= 0
+        && isNonNegativeSafeInteger(locator.occurrence)
         && (typeof locator.props === "string" || locator.props === null)
         && (typeof locator.text === "string" || locator.text === null)
         && (locator.ariaLabel === undefined || typeof locator.ariaLabel === "string" || locator.ariaLabel === null)
@@ -32,7 +32,6 @@ export function isRenderedInstanceOverride(value) {
         && typeof value.id === "string"
         && isRenderedInstanceRef(value.target);
 }
-/** Validates the serializable context carried by a token change. */
 export function isStyleRuleContextValue(value) {
     if (!isRecord(value) || !hasOnlyKeys(value, ["wrappers"]))
         return false;