@joewinke/jatui 0.15.0 → 0.16.0

package/dist/video/settings.js

@@ -1,102 +1,19 @@
 /**
- * Settings plumbing for VideoPlayer (mos-5lzns.28) — ported discipline from
- * extension-library's `settings.js`, adapted to a prop-driven component.
+ * Settings for VideoPlayer — the barrel (mos-5lzns.28 skeleton, filled in by
+ * mos-5lzns.35). The pieces live in `settings/`:
  *
- * The extension's registry exists so the settings panel is generated from one
- * array and "a setting added there appears in the UI by existing". jatui's
- * player is prop-driven (the HOST owns the panel), so this module owns the
- * smaller half of the contract that still matters here:
+ * · `registry.ts` — `VIDEO_SETTINGS`: every standing choice the player's
+ *   automatic behaviours read, with its label, hint, group, the prop it
+ *   drives and the availability gate. `coerce` never throws.
+ * · `store.ts` — the host-supplied `VideoSettingsStore` (get/set) and the
+ *   session that reads/writes through it. jatui never persists on its own.
+ * · `panel.ts` — which rows a given player can offer, and how values read.
+ * · `SettingsPanel.svelte` — the panel, generated wholly from the registry.
  *
- * · `coerce` never throws and never returns junk. A bool accepts
- *   true/"1"/"true"; a range parses and clamps to [min, max]; an enum must
- *   match a declared option or the value is discarded. Anything
- *   unrecognised returns undefined and the caller falls back to the
- *   declared default. A corrupted blob degrades to defaults, not to a
- *   broken player.
- * · One registry (`VIDEO_SETTINGS`) lists every knob the player's automatic
- *   behaviours read, with the sub-task that owns each. A knob that is here
- *   but unread by any module is a switch that visibly does nothing — the
- *   extension calls that "declared but not yet wired" and refuses to fix it
- *   quietly. Keep the same discipline: land the knob WITH its reader.
- *
- * Nothing under src/lib/video/ reads a setting directly except through the
- * value its owning feature passes in as an `opts` argument — the advisor
- * rule (see README). That keeps every module node-testable and keeps the
- * arbitration in the component.
- */
-/**
- * The registry. Populated as the behaviours land — .30 oneAxis, .31 audio,
- * .32 rate/speaker/silence, .33 night, .34 presence. An empty registry is
- * the honest state while every behaviour is prop-driven.
+ * Nothing under src/lib/video/ reads a setting directly: the component maps
+ * each value onto the prop it names and the feature reads that — the
+ * advisor rule (see README).
  */
-export const VIDEO_SETTINGS = [
-    // Read by VideoPlayer's `oneAxis` prop → video/band/axis.ts resolveAxis().
-    { id: 'player.oneAxis', type: 'bool', default: true, owner: 'mos-5lzns.30', wired: true },
-    // Read by VideoPlayer's `loudness` / `loudnessTarget` / `quietListening` /
-    // `limiter` props → video/audio/* (graph.createAudioSession, rider.*). As with
-    // night, the DEFAULT is the host's opt-in: an absent prop means nothing is
-    // ever routed through WebAudio on that player.
-    { id: 'audio.level', type: 'bool', default: true, owner: 'mos-5lzns.31', wired: true },
-    { id: 'audio.target', type: 'range', default: -20, min: -30, max: -12, step: 2, owner: 'mos-5lzns.31', wired: true },
-    { id: 'audio.quiet', type: 'bool', default: true, owner: 'mos-5lzns.31', wired: true },
-    { id: 'audio.limiter', type: 'bool', default: true, owner: 'mos-5lzns.31', wired: true },
-    // Read by VideoPlayer's `smartSpeed` / `speakerPace` (+ `speakerRates`) /
-    // `skipAds` (+ `adSpans`) props → video/rate/* (silenceMapFor + silenceFactor,
-    // speakerRateMap + speakerFactor, normalizeAdSpans + adSkipTarget), composed
-    // by rate/compose.ts composeRate. All default off: an automatic rate change
-    // is the host's opt-in.
-    { id: 'player.smartSpeed', type: 'bool', default: false, owner: 'mos-5lzns.32', wired: true },
-    { id: 'player.speakerRate', type: 'bool', default: false, owner: 'mos-5lzns.32', wired: true },
-    { id: 'player.skipAds', type: 'bool', default: false, owner: 'mos-5lzns.32', wired: true },
-    // Read by VideoPlayer's `night` / `nightDepth` / `biasLight` / `predictive` /
-    // `strobeGuard` props → video/night/* (clock.nightArmed, levels.nightLevels,
-    // meter.biasCells, track.lumaAhead / inFlash / composeNight). Unlike the
-    // extension, night.mode's DEFAULT here is the host's opt-in: an absent `night`
-    // prop means the feature does not exist on that player.
-    { id: 'night.mode', type: 'enum', default: 'auto', options: ['auto', 'on', 'off'], owner: 'mos-5lzns.33', wired: true },
-    { id: 'night.depth', type: 'range', default: 0.7, min: 0.2, max: 1, step: 0.1, owner: 'mos-5lzns.33', wired: true },
-    { id: 'night.bias', type: 'bool', default: true, owner: 'mos-5lzns.33', wired: true },
-    { id: 'player.predictive', type: 'bool', default: true, owner: 'mos-5lzns.33', wired: true },
-    { id: 'player.strobeGuard', type: 'bool', default: true, owner: 'mos-5lzns.33', wired: true },
-    // Read by VideoPlayer's `windDown` / `windDownAfter` / `windDownNightOnly` /
-    // `idleDetection` props → video/presence/winddown.ts (shouldBeginWindDown,
-    // windDownAfterMs) + presence/idle.ts (startDesktopWatch). As with night,
-    // the DEFAULT is the host's opt-in: an absent `windDown` prop means the
-    // feature does not exist on that player. `sleep.desktopIdle` defaults OFF
-    // because turning it on is what asks the browser for a permission — the
-    // host calls `requestIdleDetection()` from the click that turns it on.
-    { id: 'sleep.enabled', type: 'bool', default: true, owner: 'mos-5lzns.34', wired: true },
-    { id: 'sleep.after', type: 'range', default: 30, min: 10, max: 90, step: 5, owner: 'mos-5lzns.34', wired: true },
-    { id: 'sleep.nightOnly', type: 'bool', default: true, owner: 'mos-5lzns.34', wired: true },
-    { id: 'sleep.desktopIdle', type: 'bool', default: false, owner: 'mos-5lzns.34', wired: true },
-    // Read by VideoPlayer's `resumeSnap` prop (with `resumeKey` present) →
-    // video/presence/resume.ts planResumeSnap → advisors/resume-point snapResume.
-    { id: 'player.resumeSnap', type: 'bool', default: true, owner: 'mos-5lzns.34', wired: true },
-    // Read by VideoPlayer's `openMotion` prop (with `openFrom` present) →
-    // video/presence/flip.ts flipOpen. Off ⇒ the poster still paints.
-    { id: 'player.openMotion', type: 'bool', default: true, owner: 'mos-5lzns.34', wired: true },
-];
-/** Coerce a raw stored value against a spec — never throws, never returns junk. */
-export function coerce(spec, raw) {
-    if (spec.type === 'bool') {
-        if (raw === true || raw === '1' || raw === 'true')
-            return true;
-        if (raw === false || raw === '0' || raw === 'false')
-            return false;
-        return undefined;
-    }
-    if (spec.type === 'range') {
-        const n = typeof raw === 'number' ? raw : Number(raw);
-        if (!Number.isFinite(n))
-            return undefined;
-        const min = spec.min ?? -Infinity;
-        const max = spec.max ?? Infinity;
-        return Math.min(max, Math.max(min, n));
-    }
-    // enum: must match a declared option or the value is discarded
-    if (typeof raw !== 'string')
-        return undefined;
-    if (!spec.options?.includes(raw))
-        return undefined;
-    return raw;
-}
+export * from './settings/registry';
+export * from './settings/store';
+export * from './settings/panel';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@joewinke/jatui",
-  "version": "0.15.0",
+  "version": "0.16.0",
   "private": false,
   "description": "Shared Svelte 5 component library for JAT projects",
   "type": "module",