@pellux/goodvibes-tui 1.13.0 → 1.14.0

package/src/input/onboarding/onboarding-wizard-external-surface-extra-specs.ts

@@ -1,130 +0,0 @@
-import type { ExternalSurfaceSpec } from './onboarding-wizard-types.ts';
-
-export const WEBHOOK_SURFACE_SPEC: ExternalSurfaceSpec = {
-  id: 'webhook',
-  enabledFieldId: 'external-services.webhook',
-  enabledConfigKey: 'surfaces.webhook.enabled',
-  label: 'Outbound webhook surface',
-  hint: 'Enable outbound webhook delivery targets.',
-  defaultEnabled: (snapshot) => snapshot?.config.surfaces.webhook.enabled ?? false,
-  fields: [
-    {
-      id: 'external-services.webhook.default-target',
-      configKey: 'surfaces.webhook.defaultTarget',
-      kind: 'text',
-      label: 'Default webhook target',
-      hint: 'Fallback URL used for outbound webhook deliveries.',
-      placeholder: 'https://example.com/goodvibes',
-      defaultValue: (snapshot) => snapshot?.config.surfaces.webhook.defaultTarget ?? '',
-    },
-    {
-      id: 'external-services.webhook.secret',
-      configKey: 'surfaces.webhook.secret',
-      kind: 'masked',
-      label: 'Webhook signing secret',
-      hint: 'Secret used to sign outbound webhook payloads.',
-      placeholder: 'secret',
-      defaultValue: (snapshot) => snapshot?.config.surfaces.webhook.secret ?? '',
-    },
-    {
-      id: 'external-services.webhook.timeout-ms',
-      configKey: 'surfaces.webhook.timeoutMs',
-      kind: 'text',
-      valueType: 'number',
-      label: 'Webhook timeout ms',
-      hint: 'Request timeout for outbound webhook deliveries.',
-      placeholder: '10000',
-      defaultNumber: 10000,
-      min: 1000,
-      max: 60000,
-      defaultValue: (snapshot) => String(snapshot?.config.surfaces.webhook.timeoutMs ?? 10000),
-    },
-  ],
-};
-
-export const HOME_ASSISTANT_SURFACE_SPEC: ExternalSurfaceSpec = {
-  id: 'homeassistant',
-  enabledFieldId: 'external-services.homeassistant',
-  enabledConfigKey: 'surfaces.homeassistant.enabled',
-  label: 'Home Assistant surface',
-  hint: 'Enable the Home Assistant companion surface, daemon callbacks, and event delivery.',
-  defaultEnabled: (snapshot) => snapshot?.config.surfaces.homeassistant.enabled ?? false,
-  fields: [
-    {
-      id: 'external-services.homeassistant.instance-url',
-      configKey: 'surfaces.homeassistant.instanceUrl',
-      kind: 'text',
-      label: 'Home Assistant URL',
-      hint: 'Base URL of the Home Assistant instance.',
-      placeholder: 'http://homeassistant.local:8123',
-      defaultValue: (snapshot) => snapshot?.config.surfaces.homeassistant.instanceUrl ?? '',
-    },
-    {
-      id: 'external-services.homeassistant.access-token',
-      configKey: 'surfaces.homeassistant.accessToken',
-      kind: 'masked',
-      label: 'Home Assistant access token',
-      hint: 'Long-lived Home Assistant access token or goodvibes:// secret reference.',
-      placeholder: 'long-lived access token',
-      defaultValue: (snapshot) => snapshot?.config.surfaces.homeassistant.accessToken ?? '',
-    },
-    {
-      id: 'external-services.homeassistant.webhook-secret',
-      configKey: 'surfaces.homeassistant.webhookSecret',
-      kind: 'masked',
-      label: 'Home Assistant webhook secret',
-      hint: 'Shared secret used to verify inbound Home Assistant callbacks.',
-      placeholder: 'webhook secret',
-      defaultValue: (snapshot) => snapshot?.config.surfaces.homeassistant.webhookSecret ?? '',
-    },
-    {
-      id: 'external-services.homeassistant.default-conversation-id',
-      configKey: 'surfaces.homeassistant.defaultConversationId',
-      kind: 'text',
-      label: 'Default conversation ID',
-      hint: 'Default Home Assistant conversation id used for route binding.',
-      placeholder: 'goodvibes',
-      defaultValue: (snapshot) => snapshot?.config.surfaces.homeassistant.defaultConversationId ?? 'goodvibes',
-    },
-    {
-      id: 'external-services.homeassistant.remote-session-ttl-ms',
-      configKey: 'surfaces.homeassistant.remoteSessionTtlMs',
-      kind: 'text',
-      valueType: 'number',
-      label: 'Remote session idle TTL ms',
-      hint: 'How long an idle Home Assistant remote conversation session remains open before the daemon closes it.',
-      placeholder: '1200000',
-      defaultNumber: 1_200_000,
-      min: 60_000,
-      max: 86_400_000,
-      defaultValue: (snapshot) => String(snapshot?.config.surfaces.homeassistant.remoteSessionTtlMs ?? 1_200_000),
-    },
-    {
-      id: 'external-services.homeassistant.device-id',
-      configKey: 'surfaces.homeassistant.deviceId',
-      kind: 'text',
-      label: 'Home Assistant device ID',
-      hint: 'Stable device identifier exposed by the GoodVibes daemon.',
-      placeholder: 'goodvibes-daemon',
-      defaultValue: (snapshot) => snapshot?.config.surfaces.homeassistant.deviceId ?? 'goodvibes-daemon',
-    },
-    {
-      id: 'external-services.homeassistant.device-name',
-      configKey: 'surfaces.homeassistant.deviceName',
-      kind: 'text',
-      label: 'Home Assistant device name',
-      hint: 'Display name for the GoodVibes daemon device in Home Assistant.',
-      placeholder: 'GoodVibes Daemon',
-      defaultValue: (snapshot) => snapshot?.config.surfaces.homeassistant.deviceName ?? 'GoodVibes Daemon',
-    },
-    {
-      id: 'external-services.homeassistant.event-type',
-      configKey: 'surfaces.homeassistant.eventType',
-      kind: 'text',
-      label: 'Home Assistant event type',
-      hint: 'Event type used for daemon-to-Home Assistant deliveries.',
-      placeholder: 'goodvibes_message',
-      defaultValue: (snapshot) => snapshot?.config.surfaces.homeassistant.eventType ?? 'goodvibes_message',
-    },
-  ],
-};

package/src/runtime/render-scheduler.ts

@@ -1,90 +0,0 @@
-/**
- * render-scheduler.ts — same-tick render coalescing for the TUI shell (WO-208).
- *
- * main.ts fans out its own direct render() calls across turn/stream/input wiring
- * (~14 direct invocation contexts). Each one previously ran a full synchronous
- * Compositor.composite() the instant it was called. When several fire within a
- * single event-loop tick — a streaming burst is the canonical case — only the
- * LAST frame is ever visible; the earlier composites are immediately overwritten.
- * This scheduler collapses every schedule() call made within one tick into a
- * single composite flushed on the microtask queue, so the tick produces exactly
- * one frame. That frame is byte-identical to what the last synchronous render()
- * would have produced, because the coalesced flush runs after all of the tick's
- * synchronous state mutations, reading the same final state the last direct
- * render() would have read.
- *
- * This is deliberately SEPARATE from bootstrap-core's requestRender(), which is a
- * cross-tick, 16ms-throttled (~60fps) coalescer for the panel/input/runtime
- * fan-out — that one caps repaint RATE across ticks. This one collapses the
- * WITHIN-tick burst with no added latency: a microtask flushes at the tail of the
- * current tick, before the event loop yields to I/O, so the frame still lands in
- * the same turn it was requested in.
- *
- * flushNow() preserves a synchronous immediate path for callers that genuinely
- * need the frame composited before they return. Terminal resize is the known
- * case: the resize handler resets the compositor diff and must repaint against
- * the new dimensions synchronously rather than waiting for the microtask.
- * flushNow() also clears any pending coalesced flush, so a resize (or any
- * immediate paint) followed by an already-queued microtask never double-composites.
- */
-
-export interface RenderScheduler {
-  /**
-   * Coalesced request: schedule a composite for the current tick. N calls within
-   * one tick collapse into exactly one composite, flushed on the microtask queue.
-   */
-  readonly schedule: () => void;
-  /**
-   * Immediate request: run the composite synchronously right now, and cancel any
-   * pending coalesced flush so the tick still composites only once. Use only where
-   * a caller genuinely requires synchronous output (terminal resize).
-   */
-  readonly flushNow: () => void;
-}
-
-/**
- * Build a render scheduler around `renderNow` (the synchronous composite closure).
- *
- * `scheduleFlush` is the deferral primitive; it defaults to `queueMicrotask` so
- * bursts coalesce within the current tick. Tests and benchmarks inject a manual
- * queue to flush deterministically.
- */
-export function createRenderScheduler(
-  renderNow: () => void,
-  scheduleFlush: (flush: () => void) => void = queueMicrotask,
-  /**
-   * When provided and returning true, every composite path becomes a no-op.
-   * Wired to process-lifecycle's isTerminalRestored(): once the exit teardown
-   * has handed the terminal back to the shell, a late frame (async shutdown
-   * races, stray timers) would paint cursor-positioned content over the
-   * user's primary screen and strand the next prompt mid-screen.
-   */
-  isReleased: () => boolean = () => false,
-): RenderScheduler {
-  let scheduled = false;
-
-  const flush = (): void => {
-    // A synchronous flushNow() (or a superseding immediate paint) before this
-    // microtask ran clears `scheduled`, turning this into a no-op so a single
-    // tick never composites twice.
-    if (!scheduled) return;
-    scheduled = false;
-    if (isReleased()) return;
-    renderNow();
-  };
-
-  const schedule = (): void => {
-    if (scheduled) return;
-    scheduled = true;
-    scheduleFlush(flush);
-  };
-
-  const flushNow = (): void => {
-    // Satisfy any pending coalesced flush, then composite synchronously.
-    scheduled = false;
-    if (isReleased()) return;
-    renderNow();
-  };
-
-  return { schedule, flushNow };
-}