tmux-ide 2.9.0-beta.20 → 2.9.0-beta.21

package/packages/daemon/dist/tui/mirror/resize-transaction.js

@@ -12,6 +12,7 @@ function nonEmpty(value, label) {
 }
 function sameTarget(state, observation) {
     return (state.operationId === observation.operationId &&
+        state.authorityGeneration === observation.authorityGeneration &&
         state.workspaceName === observation.workspaceName &&
         state.semanticPaneId === observation.semanticPaneId &&
         state.axis === observation.axis);
@@ -19,9 +20,9 @@ function sameTarget(state, observation) {
 /**
  * Renderer-local pane-resize transaction.
  *
- * Pointer motion only changes the preview state. Release authors exactly one
- * semantic intent and keeps that preview visible until the matching observed
- * operation settles or rejects. The controller owns no timers, transport, or
+ * Pointer motion submits the first target immediately, then retains only the
+ * latest desired size while receipt and canonical layout settle the pending
+ * operation. Release closes the gesture and flushes that latest target. The controller owns no timers, transport, or
  * rendering effects beyond the functions supplied by its caller.
  */
 export class ResizeTransactionController {
@@ -35,6 +36,8 @@ export class ResizeTransactionController {
     #projection = null;
     #revision = 0;
     #disposed = false;
+    #dragOpen = false;
+    #submittedCells = null;
     constructor(options) {
         if (!Number.isFinite(options.timeoutMs) || options.timeoutMs <= 0) {
             throw new RangeError("resize transaction timeout must be positive");
@@ -54,6 +57,8 @@ export class ResizeTransactionController {
         const workspaceName = nonEmpty(input.workspaceName, "workspaceName");
         const semanticPaneId = nonEmpty(input.semanticPaneId, "semanticPaneId");
         const canonicalCells = positiveCells(input.canonicalCells, "canonicalCells");
+        this.#dragOpen = true;
+        this.#submittedCells = null;
         this.#projection = createOptimisticProjection({
             generation: authorityGeneration,
             revision: this.#revision,
@@ -71,30 +76,36 @@ export class ResizeTransactionController {
         });
         return true;
     }
-    /** Purely local preview update. It never calls submit or schedules work. */
+    /** First/latest coalescing: at most one mutation awaits canonical settlement. */
     move(previewCells) {
-        if (this.#state.phase !== "dragging")
+        if (!this.#dragOpen || this.#state.phase === "idle")
             return false;
         const next = positiveCells(previewCells, "previewCells");
         if (next === this.#state.previewCells)
             return false;
         this.#emit({ ...this.#state, previewCells: next });
+        if (this.#state.phase === "dragging")
+            this.#submit();
         return true;
     }
-    /**
-     * Submit once. Duplicate physical release/up/drop events return the existing
-     * operation id without authoring another mutation.
-     */
+    /** Duplicate releases never author another mutation. */
     release() {
+        this.#dragOpen = false;
         if (this.#state.phase === "pending")
             return this.#state.operationId;
+        if (this.#state.phase !== "dragging")
+            return null;
+        const canonicalCells = this.#state.canonicalCells;
+        this.#emit({ phase: "idle", canonicalCells, outcome: null });
+        return null;
+    }
+    #submit() {
         if (this.#state.phase !== "dragging")
             return null;
         const dragging = this.#state;
-        if (dragging.previewCells === dragging.canonicalCells) {
-            this.#emit({ phase: "idle", canonicalCells: dragging.canonicalCells, outcome: null });
+        if (dragging.previewCells === dragging.canonicalCells)
             return null;
-        }
+        this.#submittedCells = dragging.previewCells;
         const operationId = nonEmpty(this.#options.operationId(), "operationId");
         const submittedAt = this.#options.now();
         if (!this.#projection)
@@ -146,10 +157,15 @@ export class ResizeTransactionController {
             message: rejection.message,
         });
     }
-    /** Escape/cancel is only meaningful before release. */
+    /** Discard queued motion; an already dispatched mutation cannot be undone. */
     cancelDrag() {
-        if (this.#state.phase !== "dragging")
+        if (!this.#dragOpen || this.#state.phase === "idle")
             return false;
+        this.#dragOpen = false;
+        if (this.#state.phase === "pending") {
+            this.#emit({ ...this.#state, previewCells: this.#submittedCells });
+            return true;
+        }
         const canonicalCells = this.#state.canonicalCells;
         this.#emit({ phase: "idle", canonicalCells, outcome: null });
         return true;
@@ -160,6 +176,8 @@ export class ResizeTransactionController {
     }
     /** Retire one runtime lane without permanently disposing the controller. */
     retire() {
+        this.#dragOpen = false;
+        this.#submittedCells = null;
         this.#clearTimeout();
         this.#projection = null;
         if (this.#state.phase !== "idle" || this.#state.canonicalCells !== null) {
@@ -172,7 +190,8 @@ export class ResizeTransactionController {
             !this.#projection?.pending.some((operation) => operation.operationId === observation.operationId))
             return false;
         const cells = positiveCells(observation.cells, "observed cells");
-        const operationId = this.#state.operationId;
+        const pending = this.#state;
+        const operationId = pending.operationId;
         if (!this.#projection)
             return false;
         this.#revision += 1;
@@ -182,11 +201,19 @@ export class ResizeTransactionController {
             value: cells,
         }, { observedOperationIds: [operationId], nowMs: this.#options.now() });
         this.#clearTimeout();
-        this.#emit({
-            phase: "idle",
-            canonicalCells: cells,
-            outcome: { kind: "settled", operationId, source, cells },
-        });
+        const hasLatest = pending.previewCells !== this.#submittedCells && pending.previewCells !== cells;
+        if (this.#dragOpen || hasLatest) {
+            this.#emit({ ...pending, phase: "dragging", canonicalCells: cells });
+            if (hasLatest)
+                this.#submit();
+        }
+        else {
+            this.#emit({
+                phase: "idle",
+                canonicalCells: cells,
+                outcome: { kind: "settled", operationId, source, cells },
+            });
+        }
         return true;
     }
     #revertTimedOut(operationId) {
@@ -202,6 +229,7 @@ export class ResizeTransactionController {
             this.#projection = reconcileOptimisticOperation(this.#projection, operationId, reason.kind === "timed-out" ? "timed-out" : "rejected");
         }
         const canonicalCells = this.#state.canonicalCells;
+        this.#dragOpen = false;
         this.#clearTimeout();
         this.#emit({
             phase: "idle",

package/packages/daemon/dist/tui/mirror/runtime/application-appearance-owner.js

@@ -1,14 +1,32 @@
+import { createAutomaticContrastPass } from "../automatic-contrast.js";
 import { VISUAL_THEME_PRESETS, findVisualThemePreset } from "@tmux-ide/contracts";
 import { batch, createSignal } from "solid-js";
 import { updateAppConfig } from "../../../lib/app-config.js";
 import { createSemanticThemeStore, createTerminalPaletteProjection, deriveSystemVisualHostDefaults, } from "../theme.js";
 import { createApplicationTransientNoteOwner } from "./application-transient-note-owner.js";
 function hostDefaults(snapshot) {
-    if (snapshot.availability !== "available")
-        return null;
-    return deriveSystemVisualHostDefaults(snapshot);
+    return (deriveSystemVisualHostDefaults(snapshot) ?? { appearance: snapshot.detectedMode, overrides: {} });
 }
 export function createAppearanceOwner(config, renderer, terminalPaletteOwner) {
+    const [automaticContrast, setAutomaticContrast] = createSignal(config.theme.automaticContrast ?? true);
+    const correct = createAutomaticContrastPass();
+    const postProcess = (buffer) => {
+        if (automaticContrast())
+            correct(buffer);
+    };
+    renderer.addPostProcessFn?.(postProcess);
+    const setContrast = (enabled) => {
+        if (enabled === automaticContrast())
+            return;
+        setAutomaticContrast(enabled);
+        renderer.requestRender?.();
+    };
+    const toggleAutomaticContrast = () => {
+        if (pickerOpen()) {
+            setContrast(!automaticContrast());
+            setPickerError(null);
+        }
+    };
     const initialHostPalette = terminalPaletteOwner.getSnapshot();
     const store = createSemanticThemeStore(config.theme, {
         rendererMode: renderer.themeMode,
@@ -65,10 +83,18 @@ export function createAppearanceOwner(config, renderer, terminalPaletteOwner) {
     const apply = (id) => {
         const preset = findVisualThemePreset(id);
         const mode = preset?.appearance ?? id;
-        store.configure({ ...config.theme, mode, preset: preset?.id });
+        const wasSystem = theme().setting === "system";
+        store.configure({
+            ...config.theme,
+            mode,
+            preset: preset?.id,
+        });
+        if (mode === "system" && !wasSystem)
+            void terminalPaletteOwner.refresh();
         setPickerSelection(id);
     };
     let originalSelection = pickerSelection();
+    let originalContrast = automaticContrast();
     const preview = (id) => {
         if (!pickerOpen())
             return;
@@ -79,6 +105,7 @@ export function createAppearanceOwner(config, renderer, terminalPaletteOwner) {
     const cancelPicker = () => {
         if (!pickerOpen())
             return;
+        setContrast(originalContrast);
         apply(originalSelection);
         setPickerOpen(false);
         setPickerError(null);
@@ -89,6 +116,7 @@ export function createAppearanceOwner(config, renderer, terminalPaletteOwner) {
         try {
             updateAppConfig({
                 theme: {
+                    automaticContrast: automaticContrast(),
                     mode: theme().setting,
                     preset: findVisualThemePreset(pickerSelection())?.id ?? "",
                 },
@@ -105,7 +133,7 @@ export function createAppearanceOwner(config, renderer, terminalPaletteOwner) {
         const order = ["dark", "light", "system"];
         const next = order[(order.indexOf(theme().setting) + 1) % order.length];
         apply(next);
-        updateAppConfig({ theme: { mode: next, preset: "" } });
+        updateAppConfig({ theme: { mode: next, preset: "", automaticContrast: automaticContrast() } });
         notice.publish(`theme → ${next}`);
     };
     return {
@@ -116,6 +144,8 @@ export function createAppearanceOwner(config, renderer, terminalPaletteOwner) {
         palette,
         hostPalette,
         setTransientNote: notice.publish,
+        automaticContrast,
+        toggleAutomaticContrast,
         pickerOpen,
         pickerError,
         pickerQuery,
@@ -125,6 +155,7 @@ export function createAppearanceOwner(config, renderer, terminalPaletteOwner) {
             if (pickerOpen())
                 return;
             originalSelection = pickerSelection();
+            originalContrast = automaticContrast();
             setPickerQuery("");
             setPickerError(null);
             setPickerOpen(true);
@@ -139,7 +170,9 @@ export function createAppearanceOwner(config, renderer, terminalPaletteOwner) {
                 return true;
             const name = event.name.toLowerCase();
             const choices = pickerOptions();
-            if (name === "escape")
+            if (event.ctrl && name === "a")
+                toggleAutomaticContrast();
+            else if (name === "escape")
                 cancelPicker();
             else if (name === "return" || name === "enter")
                 savePicker();
@@ -166,6 +199,7 @@ export function createAppearanceOwner(config, renderer, terminalPaletteOwner) {
         },
         cycleTheme,
         dispose() {
+            renderer.removePostProcessFn?.(postProcess);
             stopTheme();
             stopRendererTheme();
             stopHostPalette();

package/packages/daemon/dist/tui/mirror/runtime/application-machine-sidebar.jsx

@@ -68,6 +68,7 @@ export function ApplicationMachineSidebar(props) {
             }))
             : []),
     ]));
+    const rowsByKey = createMemo(() => new Map(rows().map((row) => [row.key, row])));
     const index = () => Math.max(0, rows().findIndex((row) => row.key === selectedKey()));
     const controlGroup = () => {
         const group = rows()[index()]?.group;
@@ -281,49 +282,69 @@ export function ApplicationMachineSidebar(props) {
                 revealFocusedRow();
             };
         }} height={machineHeight()} width={props.width} scrollX={false} scrollY={true} horizontalScrollbarOptions={{ visible: false }}>
-        <For each={rows()}>
-          {(row) => (<box width={Math.max(1, props.width - 1)} height={row.agent ? (row.agentHeading ? 3 : 2) : 1} flexShrink={0} flexDirection="column">
-              <Show when={row.agentHeading}>
-                <text height={1} fg={props.theme.roles.text.secondary}>
-                  {" "}
-                  Agents
-                </text>
-              </Show>
-              <NavigationRow theme={props.theme} id={`machine:${row.key}`} width={Math.max(1, props.width - 1)} labelColor={!row.agent && !row.session ? fleetHostColor(row.group) : undefined} label={row.agent
-                ? row.agent.name
-                : row.session
-                    ? friendlySessionLabel(row.session.name)
-                    : row.group.label} marker={row.agent
-                ? active(row)
-                    ? row.agent.attention
-                        ? "›!"
-                        : "›"
-                    : row.agent.attention
-                        ? "!"
-                        : "•"
-                : row.session
-                    ? props.model.favorites?.().includes(row.session.id)
-                        ? " ★"
-                        : active(row)
-                            ? " ›"
-                            : "  "
-                    : collapsed().has(preferenceKey(row.group))
-                        ? "▸"
-                        : "▾"} detail={row.agent
-                ? row.group.state !== "ready" || row.agent.disabled
-                    ? "unavailable"
-                    : `[${terminalAgentStatusLabel(row.agent.activity)}]`
-                : row.session
-                    ? row.group.state !== "ready" || row.session.disabled
+        <For each={rows().map((row) => row.key)}>
+          {(key) => {
+            // The semantic key owns the native row; fresh snapshots update its
+            // props instead of destroying and recreating identical renderables.
+            const current = createMemo((previous) => rowsByKey().get(key) ?? previous, rowsByKey().get(key));
+            const row = {
+                key,
+                get group() {
+                    return current().group;
+                },
+                get session() {
+                    return current().session;
+                },
+                get agent() {
+                    return current().agent;
+                },
+                get agentHeading() {
+                    return current().agentHeading;
+                },
+            };
+            return (<box width={Math.max(1, props.width - 1)} height={row.agent ? (row.agentHeading ? 3 : 2) : 1} flexShrink={0} flexDirection="column">
+                <Show when={row.agentHeading}>
+                  <text height={1} fg={props.theme.roles.text.secondary}>
+                    {" "}
+                    Agents
+                  </text>
+                </Show>
+                <NavigationRow theme={props.theme} id={`machine:${row.key}`} width={Math.max(1, props.width - 1)} labelColor={!row.agent && !row.session ? fleetHostColor(row.group) : undefined} label={row.agent
+                    ? row.agent.name
+                    : row.session
+                        ? friendlySessionLabel(row.session.name)
+                        : row.group.label} marker={row.agent
+                    ? active(row)
+                        ? row.agent.attention
+                            ? "›!"
+                            : "›"
+                        : row.agent.attention
+                            ? "!"
+                            : "•"
+                    : row.session
+                        ? props.model.favorites?.().includes(row.session.id)
+                            ? " ★"
+                            : active(row)
+                                ? " ›"
+                                : "  "
+                        : collapsed().has(preferenceKey(row.group))
+                            ? "▸"
+                            : "▾"} detail={row.agent
+                    ? row.group.state !== "ready" || row.agent.disabled
                         ? "unavailable"
-                        : `${row.session.paneCount}p${row.group.agents?.length ? ` ${activity(row).label}` : ""}`
-                    : row.group.agents?.length && row.group.state === "ready"
-                        ? activity(row).label
-                        : `${connectionDetail(row.group)}${row.group.agents?.length ? " ?" : ""}`} selected={Boolean((row.session || row.agent) && active(row))} focused={Boolean(focused() && row.key === selectedKey())} attention={row.agent?.attention ?? activity(row).kind === "attention"} onActivate={(source) => activate(row, source)}/>
-              <Show when={row.agent}>
-                {(agent) => (<text height={1} fg={props.theme.roles.text.muted} content={clipTerminal(`  ${friendlySessionLabel(agent().sessionName)}`, Math.max(1, props.width - 1))}/>)}
-              </Show>
-            </box>)}
+                        : `[${terminalAgentStatusLabel(row.agent.activity)}]`
+                    : row.session
+                        ? row.group.state !== "ready" || row.session.disabled
+                            ? "unavailable"
+                            : `${row.session.paneCount}p${row.group.agents?.length ? ` ${activity(row).label}` : ""}`
+                        : row.group.agents?.length && row.group.state === "ready"
+                            ? activity(row).label
+                            : `${connectionDetail(row.group)}${row.group.agents?.length ? " ?" : ""}`} selected={Boolean((row.session || row.agent) && active(row))} focused={Boolean(focused() && row.key === selectedKey())} attention={row.agent?.attention ?? activity(row).kind === "attention"} onActivate={(source) => activate(row, source)}/>
+                <Show when={row.agent}>
+                  {(agent) => (<text height={1} fg={props.theme.roles.text.muted} content={clipTerminal(`  ${friendlySessionLabel(agent().sessionName)}`, Math.max(1, props.width - 1))}/>)}
+                </Show>
+              </box>);
+        }}
         </For>
       </scrollbox>
       <For each={agentHeight() > 0 ? [true] : []}>