@xemahq/ui-kernel 0.10.0 → 0.11.1

package/src/vista/GenericCatalogueSurface.tsx

@@ -30,11 +30,15 @@ interface GenericSurfaceField {
 }
 
 function stringify(value: unknown): string {
-  if (typeof value === 'string') return value;
+  if (typeof value === 'string') {
+    return value;
+  }
   if (typeof value === 'number' || typeof value === 'boolean') {
     return String(value);
   }
-  if (value === null || value === undefined) return '—';
+  if (value === null || value === undefined) {
+    return '—';
+  }
   try {
     return JSON.stringify(value);
   } catch {
@@ -43,9 +47,7 @@ function stringify(value: unknown): string {
 }
 
 function isPlainObject(value: unknown): value is Record<string, unknown> {
-  return (
-    typeof value === 'object' && value !== null && !Array.isArray(value)
-  );
+  return typeof value === 'object' && value !== null && !Array.isArray(value);
 }
 
 /**

package/src/vista/VistaPanel.tsx

@@ -114,10 +114,16 @@ export function VistaTabBar({
       indicator?: boolean;
       icon?: OverflowTab['icon'];
     } = { value: tab.tabId, label: tab.title };
-    if (tab.pinned === true) entry.pinned = true;
-    if (indicatorTabIds?.includes(tab.tabId) === true) entry.indicator = true;
+    if (tab.pinned === true) {
+      entry.pinned = true;
+    }
+    if (indicatorTabIds?.includes(tab.tabId) === true) {
+      entry.indicator = true;
+    }
     const icon = surfaceKindIcon(tab.kind);
-    if (icon !== undefined) entry.icon = icon;
+    if (icon !== undefined) {
+      entry.icon = icon;
+    }
     return entry;
   });
   return (
@@ -143,7 +149,9 @@ export function VistaSurfaceBody({
   tab,
 }: Readonly<{ tab: VistaTab }>): ReactElement {
   const resolution = vistaSurfaceRegistry.resolve(tab);
-  if (resolution.matched) return resolution.rendered;
+  if (resolution.matched) {
+    return resolution.rendered;
+  }
   return (
     <SurfaceFallbackCard
       kind={resolution.kind}

package/src/vista/html/CustomUiFrame.spec.tsx

@@ -34,7 +34,9 @@ const STYLE_CSS = ':root{--x:1}';
 function parseIframe(markup: string): Element {
   const doc = new DOMParser().parseFromString(markup, 'text/html');
   const iframe = doc.querySelector('iframe');
-  if (!iframe) throw new Error('no iframe rendered');
+  if (!iframe) {
+    throw new Error('no iframe rendered');
+  }
   return iframe;
 }
 

package/src/vista/html/CustomUiFrame.tsx

@@ -153,7 +153,9 @@ export function CustomUiFrame<TIntent>({
   // re-render, per the HostProps contract). Before ready, the frameReady
   // handler above pushes the current props — so no update is lost.
   useEffect(() => {
-    if (!readyRef.current) return;
+    if (!readyRef.current) {
+      return;
+    }
     iframeRef.current?.contentWindow?.postMessage(
       hostPropsMessage(componentProps),
       '*',

package/src/vista/html/custom-ui-frame-bridge.ts

@@ -52,10 +52,14 @@ export function classifyFrameMessage<TIntent>(
 ): ClassifiedFrameMessage<TIntent> {
   // Gate 1: source identity. Reject if we have no frame yet or the message did
   // not come from THIS frame's window.
-  if (frameWindow === null || event.source !== frameWindow) return IGNORED;
+  if (frameWindow === null || event.source !== frameWindow) {
+    return IGNORED;
+  }
 
   const data = event.data;
-  if (data === null || typeof data !== 'object') return IGNORED;
+  if (data === null || typeof data !== 'object') {
+    return IGNORED;
+  }
   const envelope = data as {
     protocol?: unknown;
     type?: unknown;
@@ -64,7 +68,9 @@ export function classifyFrameMessage<TIntent>(
   };
 
   // Gate 2: protocol version.
-  if (envelope.protocol !== CUSTOM_UI_FRAME_PROTOCOL) return IGNORED;
+  if (envelope.protocol !== CUSTOM_UI_FRAME_PROTOCOL) {
+    return IGNORED;
+  }
 
   // Gate 3: closed message-type set.
   switch (envelope.type) {

package/src/vista/html/custom-ui-intents.spec.ts

@@ -104,7 +104,9 @@ describe('createNavigateIntentSet', () => {
       kind: CustomUiIntentKind.Navigate,
       href: '/apps/x',
     });
-    if (intent === null) throw new Error('expected a parsed navigate intent');
+    if (intent === null) {
+      throw new Error('expected a parsed navigate intent');
+    }
     set.perform(intent);
     expect(navigate).toHaveBeenCalledWith({
       kind: CustomUiIntentKind.Navigate,

package/src/vista/html/custom-ui-intents.ts

@@ -72,13 +72,19 @@ export interface CustomUiIntentHandlers {
  */
 function isHostRelativePath(href: string): boolean {
   // Must be rooted at '/'.
-  if (!href.startsWith('/')) return false;
+  if (!href.startsWith('/')) {
+    return false;
+  }
   // A backslash is a protocol-relative/scheme trick after WHATWG normalisation
   // ("/\\evil.com" -> //evil.com -> foreign origin); whitespace / control
   // chars can also confuse the parser. Reject any before resolving.
-  if (href.includes('\\')) return false;
+  if (href.includes('\\')) {
+    return false;
+  }
   // eslint-disable-next-line no-control-regex
-  if (/[\u0000-\u0020\u007f]/.test(href)) return false;
+  if (/[\u0000-\u0020\u007f]/.test(href)) {
+    return false;
+  }
   let resolved: URL;
   try {
     resolved = new URL(href, window.location.origin);
@@ -96,12 +102,18 @@ function isHostRelativePath(href: string): boolean {
  * an out-of-policy value all return `null` — the caller rejects it.
  */
 export function parseCustomUiIntent(raw: unknown): CustomUiIntent | null {
-  if (raw === null || typeof raw !== 'object') return null;
+  if (raw === null || typeof raw !== 'object') {
+    return null;
+  }
   const candidate = raw as { kind?: unknown; href?: unknown };
   switch (candidate.kind) {
     case CustomUiIntentKind.Navigate:
-      if (typeof candidate.href !== 'string') return null;
-      if (!isHostRelativePath(candidate.href)) return null;
+      if (typeof candidate.href !== 'string') {
+        return null;
+      }
+      if (!isHostRelativePath(candidate.href)) {
+        return null;
+      }
       return { kind: CustomUiIntentKind.Navigate, href: candidate.href };
     default:
       return null;
@@ -118,15 +130,7 @@ export function performCustomUiIntent(
   intent: CustomUiIntent,
   handlers: CustomUiIntentHandlers,
 ): void {
-  switch (intent.kind) {
-    case CustomUiIntentKind.Navigate:
-      handlers[CustomUiIntentKind.Navigate](intent);
-      return;
-    default: {
-      const exhaustive: never = intent.kind;
-      throw new Error(`Unhandled custom-UI intent kind: ${String(exhaustive)}`);
-    }
-  }
+  handlers[CustomUiIntentKind.Navigate](intent);
 }
 
 /** The built-in intent set: host-relative navigation and nothing else. */

package/src/vista/surface-catalogue.ts

@@ -55,16 +55,22 @@ export function __resetSurfaceCatalogueSyncForTests(): void {
 export async function syncSurfaceCatalogue(
   fetchCatalogue: SurfaceCatalogueFetcher,
 ): Promise<void> {
-  if (synced) return;
+  if (synced) {
+    return;
+  }
   synced = true;
   try {
     const entries = await fetchCatalogue();
     for (const entry of entries) {
       // Only biome-contributed kinds need registering here; first-party kinds
       // are already registered by the static builtins.
-      if (entry.source !== SurfaceKindSource.Biome) continue;
+      if (entry.source !== SurfaceKindSource.Biome) {
+        continue;
+      }
       // Never shadow a biome's own (or an already-registered) renderer.
-      if (vistaSurfaceRegistry.has(entry.kind)) continue;
+      if (vistaSurfaceRegistry.has(entry.kind)) {
+        continue;
+      }
       vistaSurfaceRegistry.register(entry.kind, {
         version: entry.version,
         validate: (payload) =>

package/src/vista/surfaces/FormSurface.tsx

@@ -48,10 +48,7 @@ import { captureValuesToStringMap } from '../../ui/capture/submit-values';
 import { useSchemaValidation } from '../../ui/capture/use-schema-validation';
 import { cn } from '../../ui/cn';
 import { Button } from '../../ui/primitives/button';
-import {
-  useVistaFormHost,
-  type VistaFormOutcome,
-} from '../vista-form-context';
+import { useVistaFormHost, type VistaFormOutcome } from '../vista-form-context';
 
 import type { CaptureValues } from '../../ui/capture/field-spec';
 import type { VistaFormPayload } from '../vista-payloads';
@@ -117,9 +114,13 @@ export function FormSurface(payload: Readonly<VistaFormPayload>): ReactElement {
   }, []);
 
   const handleSubmit = useCallback((): void => {
-    if (host === null) return;
+    if (host === null) {
+      return;
+    }
     setShowFieldErrors(true);
-    if (!isValid) return;
+    if (!isValid) {
+      return;
+    }
     setServiceError(null);
     setPhase(FormPhase.Working);
     // The values object is a PARAMETER here and nowhere else. It is never
@@ -138,7 +139,9 @@ export function FormSurface(payload: Readonly<VistaFormPayload>): ReactElement {
   }, [host, isValid, payload.fields, payload.ref, settle, failed, values]);
 
   const handleCheckStatus = useCallback((): void => {
-    if (host === null) return;
+    if (host === null) {
+      return;
+    }
     setServiceError(null);
     setPhase(FormPhase.Working);
     void host
@@ -167,10 +170,24 @@ export function FormSurface(payload: Readonly<VistaFormPayload>): ReactElement {
   // an event, not a timer — there is no polling loop and no timeout.
   const isLaunch = payload.launch !== undefined;
   useEffect(() => {
-    if (!isLaunch || phase === FormPhase.Settled || host === null) return;
+    if (!isLaunch || phase === FormPhase.Settled || host === null) {
+      return;
+    }
     const onFocus = (): void => handleCheckStatus();
-    globalThis.addEventListener?.('focus', onFocus);
-    return () => globalThis.removeEventListener?.('focus', onFocus);
+    const addEventListener: unknown = Reflect.get(globalThis, 'addEventListener');
+    if (typeof addEventListener !== 'function') {
+      return;
+    }
+    Reflect.apply(addEventListener, globalThis, ['focus', onFocus]);
+    return () => {
+      const removeEventListener: unknown = Reflect.get(
+        globalThis,
+        'removeEventListener',
+      );
+      if (typeof removeEventListener === 'function') {
+        Reflect.apply(removeEventListener, globalThis, ['focus', onFocus]);
+      }
+    };
     // `handleCheckStatus` is stable per (host, payload) and re-subscribing on a
     // phase change is intended: a settled surface stops listening.
   }, [isLaunch, phase, host, handleCheckStatus]);

package/src/vista/surfaces/TableSurface.tsx

@@ -9,9 +9,10 @@
 import type { VistaTablePayload } from '../vista-payloads';
 import type { ReactElement } from 'react';
 
-
 function cellText(value: unknown): string {
-  if (value === null || value === undefined) return '';
+  if (value === null || value === undefined) {
+    return '';
+  }
   if (typeof value === 'object') {
     try {
       return JSON.stringify(value);
@@ -62,7 +63,7 @@ export function TableSurface({
             <tr
               // Rows carry no identity in the generic contract; position is the
               // only stable key available.
-               
+
               key={rowIndex}
             >
               {columns.map((column, columnIndex) => (

package/src/vista/surfaces/blob-surfaces.tsx

@@ -29,7 +29,9 @@ function BlobBody({
   payload: VistaBlobPayload;
   render: (url: string) => ReactNode;
 }>): ReactElement {
-  if (payload.url !== undefined) return <>{render(payload.url)}</>;
+  if (payload.url !== undefined) {
+    return <>{render(payload.url)}</>;
+  }
   // `validateBlobPayload` guarantees exactly one of url/path.
   return <VistaFileUrl path={payload.path as string} render={render} />;
 }
@@ -51,7 +53,11 @@ export function ImageSurface(payload: VistaBlobPayload): ReactElement {
           <img
             src={url}
             alt={payload.alt ?? 'Preview'}
-            style={{ maxWidth: '100%', maxHeight: '100%', objectFit: 'contain' }}
+            style={{
+              maxWidth: '100%',
+              maxHeight: '100%',
+              objectFit: 'contain',
+            }}
           />
         </div>
       )}
@@ -113,7 +119,11 @@ export function MediaSurface(payload: VistaBlobPayload): ReactElement {
         >
           {isAudio ? (
             // eslint-disable-next-line jsx-a11y/media-has-caption
-            <audio src={url} controls style={{ width: '100%', maxWidth: 520 }} />
+            <audio
+              src={url}
+              controls
+              style={{ width: '100%', maxWidth: 520 }}
+            />
           ) : (
             // eslint-disable-next-line jsx-a11y/media-has-caption
             <video

package/src/vista/surfaces/elicitation-transport.ts

@@ -66,7 +66,9 @@ function asRecord(value: unknown): UnknownRecord | null {
  */
 function unwrapEnvelope(body: unknown): UnknownRecord | null {
   const record = asRecord(body);
-  if (record === null) return null;
+  if (record === null) {
+    return null;
+  }
   const inner = asRecord(record.data);
   return inner ?? record;
 }
@@ -82,14 +84,20 @@ export function extractServiceMessage(body: unknown, status: number): string {
     record === null
       ? undefined
       : (record.message ?? asRecord(record.error)?.message ?? record.error);
-  if (typeof candidate === 'string' && candidate.length > 0) return candidate;
+  if (typeof candidate === 'string' && candidate.length > 0) {
+    return candidate;
+  }
   if (Array.isArray(candidate)) {
     const joined = candidate
       .filter((m): m is string => typeof m === 'string')
       .join(' ');
-    if (joined.length > 0) return joined;
+    if (joined.length > 0) {
+      return joined;
+    }
+  }
+  if (typeof body === 'string' && body.length > 0) {
+    return body;
   }
-  if (typeof body === 'string' && body.length > 0) return body;
   return `The service rejected the submission (HTTP ${status}).`;
 }
 
@@ -112,11 +120,15 @@ async function readBody(response: Response): Promise<unknown> {
  */
 function toOutcome(body: unknown, ref: string): VistaFormOutcome | null {
   const record = unwrapEnvelope(body);
-  if (record === null) return null;
+  if (record === null) {
+    return null;
+  }
   const submitted =
     record.submitted === true ||
     (typeof record.status === 'string' && record.status === 'submitted');
-  if (!submitted) return null;
+  if (!submitted) {
+    return null;
+  }
   const connectionId =
     typeof record.connectionId === 'string' && record.connectionId.length > 0
       ? record.connectionId
@@ -133,7 +145,9 @@ function authHeaders(
     Authorization: `Bearer ${accessToken}`,
     Accept: 'application/json',
   };
-  if (json) headers['Content-Type'] = 'application/json';
+  if (json) {
+    headers['Content-Type'] = 'application/json';
+  }
   return headers;
 }
 

package/src/vista/surfaces/file-surface-utils.ts

@@ -37,8 +37,12 @@ export interface VistaFileGroup {
 }
 
 function compareDirs(a: string, b: string): number {
-  if (a === '') return -1;
-  if (b === '') return 1;
+  if (a === '') {
+    return -1;
+  }
+  if (b === '') {
+    return 1;
+  }
   return a.localeCompare(b);
 }
 
@@ -50,8 +54,11 @@ export function groupFilesByFolder(
     const slash = file.path.lastIndexOf('/');
     const dir = slash < 0 ? '' : file.path.slice(0, slash);
     const bucket = groups.get(dir);
-    if (bucket) bucket.push(file);
-    else groups.set(dir, [file]);
+    if (bucket) {
+      bucket.push(file);
+    } else {
+      groups.set(dir, [file]);
+    }
   }
   return [...groups.keys()].sort(compareDirs).map((dir) => ({
     dir,

package/src/vista/surfaces/register-surfaces.tsx

@@ -60,7 +60,9 @@ import {
  */
 function validateMediaPayload(raw: unknown): VistaBlobPayload | null {
   const payload = validateBlobPayload(raw);
-  if (payload === null) return null;
+  if (payload === null) {
+    return null;
+  }
   return payload.contentType === undefined ? null : payload;
 }
 

package/src/vista/surfaces/text-surfaces.tsx

@@ -35,7 +35,9 @@ function TextBody({
   payload: VistaTextPayload;
   render: (content: string) => ReactNode;
 }>): ReactElement {
-  if (payload.content !== undefined) return <>{render(payload.content)}</>;
+  if (payload.content !== undefined) {
+    return <>{render(payload.content)}</>;
+  }
   // `validateTextPayload` guarantees exactly one of content/path.
   return <VistaFileText path={payload.path as string} render={render} />;
 }
@@ -73,10 +75,13 @@ export function CodeSurface(payload: VistaTextPayload): ReactElement {
  * parses and shown VERBATIM when it does not (a malformed JSON file is shown as
  * it is, never swallowed).
  */
-export function JsonSurface({ data }: Readonly<{ data: unknown }>): ReactElement {
+export function JsonSurface({
+  data,
+}: Readonly<{ data: unknown }>): ReactElement {
   let text: string;
   try {
-    text = JSON.stringify(data, null, 2) ?? String(data);
+    const serialized: unknown = JSON.stringify(data, null, 2);
+    text = typeof serialized === 'string' ? serialized : String(data);
   } catch {
     text = String(data);
   }
@@ -89,10 +94,18 @@ export function JsonSurface({ data }: Readonly<{ data: unknown }>): ReactElement
 
 /** Colour a unified-diff line by its leading marker. */
 function diffLineColor(line: string): string | undefined {
-  if (line.startsWith('+++') || line.startsWith('---')) return 'hsl(var(--ink-3))';
-  if (line.startsWith('@@')) return 'hsl(var(--info))';
-  if (line.startsWith('+')) return 'hsl(var(--success))';
-  if (line.startsWith('-')) return 'hsl(var(--destructive, var(--danger)))';
+  if (line.startsWith('+++') || line.startsWith('---')) {
+    return 'hsl(var(--ink-3))';
+  }
+  if (line.startsWith('@@')) {
+    return 'hsl(var(--info))';
+  }
+  if (line.startsWith('+')) {
+    return 'hsl(var(--success))';
+  }
+  if (line.startsWith('-')) {
+    return 'hsl(var(--destructive, var(--danger)))';
+  }
   return undefined;
 }
 
@@ -105,7 +118,7 @@ export function DiffSurface(payload: VistaTextPayload): ReactElement {
           {content.split('\n').map((line, index) => (
             <div
               // A diff has no stable per-line identity other than its position.
-               
+
               key={index}
               style={{ color: diffLineColor(line) }}
             >

package/src/vista/use-vista-tabs.ts

@@ -128,14 +128,18 @@ export function useVistaTabs({
   activeRef.current = activeTabId;
 
   const activate = useCallback((tabId: string): void => {
-    if (!tabsRef.current.some((t) => t.tabId === tabId)) return;
+    if (!tabsRef.current.some((t) => t.tabId === tabId)) {
+      return;
+    }
     setActiveTabId(tabId);
   }, []);
 
   const open = useCallback((surface: VistaTab): string => {
     setOpenedTabs((prev) => {
       const index = prev.findIndex((t) => t.tabId === surface.tabId);
-      if (index === -1) return [...prev, surface];
+      if (index === -1) {
+        return [...prev, surface];
+      }
       // Re-opening the same tabId REPLACES it (the `VistaSurface` contract).
       const next = [...prev];
       next[index] = surface;
@@ -153,7 +157,9 @@ export function useVistaTabs({
       // overwritten on the next reconcile, so it is REFUSED rather than
       // silently lost. Ownership is decided synchronously from the ref, not
       // inside the state updater (which React may invoke twice).
-      if (!openedTabsRef.current.some((t) => t.tabId === tabId)) return false;
+      if (!openedTabsRef.current.some((t) => t.tabId === tabId)) {
+        return false;
+      }
       setOpenedTabs((prev) =>
         prev.map((tab) => (tab.tabId === tabId ? { ...tab, ...patch } : tab)),
       );
@@ -164,7 +170,9 @@ export function useVistaTabs({
 
   const close = useCallback((tabId: string): boolean => {
     const tab = tabsRef.current.find((t) => t.tabId === tabId);
-    if (tab === undefined || tab.pinned === true) return false;
+    if (tab === undefined || tab.pinned === true) {
+      return false;
+    }
     setOpenedTabs((prev) => prev.filter((t) => t.tabId !== tabId));
     setClosedHostTabIds((prev) =>
       prev.includes(tabId) ? prev : [...prev, tabId],

package/src/vista/vista-code-renderer.tsx

@@ -13,10 +13,8 @@
  * pane is the intended rendering, and with a provider the host's renderer is
  * the intended rendering. Neither is an error path.
  */
-import {
-  type ReactElement,
-  type ReactNode,
-} from 'react';
+import { type ReactElement, type ReactNode } from 'react';
+
 import {
   VistaCodeRendererContext,
   type VistaCodeRenderer,
@@ -25,7 +23,10 @@ import {
 export function VistaCodeRendererProvider({
   renderer,
   children,
-}: Readonly<{ renderer: VistaCodeRenderer; children: ReactNode }>): ReactElement {
+}: Readonly<{
+  renderer: VistaCodeRenderer;
+  children: ReactNode;
+}>): ReactElement {
   return (
     <VistaCodeRendererContext.Provider value={renderer}>
       {children}

package/src/vista/vista-file-source.tsx

@@ -12,12 +12,8 @@
  * provider renders a visible error, never a blank pane and never a silent
  * "content unavailable".
  */
-import {
-  useEffect,
-  useState,
-  type ReactElement,
-  type ReactNode,
-} from 'react';
+import { useEffect, useState, type ReactElement, type ReactNode } from 'react';
+
 import {
   VistaFileSourceContext,
   useVistaFileSource,
@@ -43,10 +39,7 @@ type LoadState =
 
 const LOADING: LoadState = { status: 'loading' };
 
-function useResolvedFile(
-  path: string,
-  mode: 'text' | 'url',
-): LoadState {
+function useResolvedFile(path: string, mode: 'text' | 'url'): LoadState {
   const source = useVistaFileSource();
   const revision = source?.revision;
   const [state, setState] = useState<LoadState>(LOADING);
@@ -66,10 +59,14 @@ function useResolvedFile(
       mode === 'text' ? source.readText(path) : source.resolveUrl(path);
     promise.then(
       (value) => {
-        if (!cancelled) setState({ status: 'ready', value });
+        if (!cancelled) {
+          setState({ status: 'ready', value });
+        }
       },
       (error: unknown) => {
-        if (cancelled) return;
+        if (cancelled) {
+          return;
+        }
         setState({
           status: 'error',
           message: error instanceof Error ? error.message : String(error),
@@ -122,7 +119,9 @@ export function VistaFileText({
   render: (content: string) => ReactNode;
 }>): ReactElement {
   const state = useResolvedFile(path, 'text');
-  if (state.status !== 'ready') return <LoadNotice state={state} />;
+  if (state.status !== 'ready') {
+    return <LoadNotice state={state} />;
+  }
   return <>{render(state.value)}</>;
 }
 
@@ -135,6 +134,8 @@ export function VistaFileUrl({
   render: (url: string) => ReactNode;
 }>): ReactElement {
   const state = useResolvedFile(path, 'url');
-  if (state.status !== 'ready') return <LoadNotice state={state} />;
+  if (state.status !== 'ready') {
+    return <LoadNotice state={state} />;
+  }
   return <>{render(state.value)}</>;
 }

package/src/vista/vista-form-host.tsx

@@ -26,14 +26,9 @@
  * provider renders a visible refusal and cannot submit. Fail-closed — a form
  * that silently posts nowhere would be worse than one that says it cannot.
  */
-import {
-  type ReactElement,
-  type ReactNode,
-} from 'react';
-import {
-  VistaFormHostContext,
-  type VistaFormHost,
-} from './vista-form-context';
+import { type ReactElement, type ReactNode } from 'react';
+
+import { VistaFormHostContext, type VistaFormHost } from './vista-form-context';
 
 /**
  * Everything a settled elicitation reports. Closed, scalar-only, and