@spencer-kit/coder-studio 0.4.5 → 0.4.7

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 0.4.7
+
+### Patch Changes
+
+- [#64](https://github.com/spencerkit/coder-studio/pull/64) [`d14fe08`](https://github.com/spencerkit/coder-studio/commit/d14fe08d7861652a9290559d5a59aa766c286309) Thanks [@pallyoung](https://github.com/pallyoung)! - Refine workspace theming and session ergonomics by adding pane drag reordering,
+  stabilizing update checks, hardening PTY color environment isolation, and
+  polishing shared desktop surfaces across the app.
+
+## 0.4.6
+
+### Patch Changes
+
+- [#62](https://github.com/spencerkit/coder-studio/pull/62) [`535c3c0`](https://github.com/spencerkit/coder-studio/commit/535c3c09cc5895e3b3b949067633f8a6bb3644f8) Thanks [@pallyoung](https://github.com/pallyoung)! - Improve desktop workspace ergonomics by adding keyboard pane navigation,
+  supporting workspace path drops into terminal sessions, and launching themed
+  PTYs with terminal-aware background environment hints.
+
 ## 0.4.5
 
 ### Patch Changes

package/dist/esm/bin.mjs

@@ -6892,7 +6892,8 @@ var init_manager3 = __esm({
             ...cmd.env,
             CODER_STUDIO_SESSION_ID: sessionId
           },
-          title: req.provider.displayName
+          title: req.provider.displayName,
+          themeBackground: req.themeBackground
         };
         const terminal = this.deps.terminalMgr.create(terminalSpec);
         const active = new ActiveSession({
@@ -12152,6 +12153,39 @@ var init_types2 = __esm({
 function isTerminalTraceEnabled() {
   return process.env.CODER_STUDIO_TERMINAL_TRACE === "1";
 }
+function parseHexColor(input2) {
+  const trimmed = input2.trim();
+  if (!trimmed.startsWith("#")) {
+    return null;
+  }
+  const hex = trimmed.slice(1);
+  let r;
+  let g;
+  let b;
+  if (hex.length === 3) {
+    r = Number.parseInt(hex[0] + hex[0], 16);
+    g = Number.parseInt(hex[1] + hex[1], 16);
+    b = Number.parseInt(hex[2] + hex[2], 16);
+  } else if (hex.length === 6 || hex.length === 8) {
+    r = Number.parseInt(hex.slice(0, 2), 16);
+    g = Number.parseInt(hex.slice(2, 4), 16);
+    b = Number.parseInt(hex.slice(4, 6), 16);
+  } else {
+    return null;
+  }
+  if ([r, g, b].some((v) => Number.isNaN(v))) {
+    return null;
+  }
+  return { r, g, b };
+}
+function computeColorFgBg(background) {
+  const rgb = parseHexColor(background);
+  if (!rgb) {
+    return void 0;
+  }
+  const luma = (0.299 * rgb.r + 0.587 * rgb.g + 0.114 * rgb.b) / 255;
+  return luma > 0.5 ? "0;15" : "15;0";
+}
 function countOccurrences(text, needle) {
   return text.split(needle).length - 1;
 }
@@ -12208,6 +12242,7 @@ var init_manager5 = __esm({
        */
       create(spec) {
         const id = generateId();
+        const derivedColorFgBg = spec.themeBackground ? computeColorFgBg(spec.themeBackground) : void 0;
         const terminalEnv = {
           ...Object.fromEntries(
             Object.entries(process.env).filter((e) => e[1] != null)
@@ -12215,8 +12250,16 @@ var init_manager5 = __esm({
           TERM: "xterm-256color",
           COLORTERM: "truecolor",
           FORCE_COLOR: "3",
+          ...derivedColorFgBg ? { COLORFGBG: derivedColorFgBg } : {},
           ...spec.env
         };
+        delete terminalEnv.COLORFGBG;
+        if (derivedColorFgBg) {
+          terminalEnv.COLORFGBG = derivedColorFgBg;
+        }
+        if (spec.env?.COLORFGBG) {
+          terminalEnv.COLORFGBG = spec.env.COLORFGBG;
+        }
         let pty;
         try {
           pty = this.deps.ptyHost.spawn(spec.argv, {
@@ -12616,7 +12659,7 @@ var init_update_service = __esm({
   "packages/server/src/update/update-service.ts"() {
     "use strict";
     init_src3();
-    UpdateService = class {
+    UpdateService = class _UpdateService {
       constructor(deps) {
         this.deps = deps;
         this.now = deps.now ?? Date.now;
@@ -12638,12 +12681,14 @@ var init_update_service = __esm({
         this.spawnDetachedWorkerImpl = deps.spawnDetachedWorker;
       }
       deps;
+      static CHECK_TIMEOUT_MS = 15e3;
       now;
       runtime;
       updateWorkerLogFilePath;
       runLatestVersionLookup;
       spawnDetachedWorkerImpl;
       scheduleTimer = null;
+      inFlightCheck = null;
       start() {
         this.reconcileOnStartup();
         this.reloadScheduleFromSettings();
@@ -12674,10 +12719,8 @@ var init_update_service = __esm({
         this.scheduleTimer.unref?.();
       }
       getStateView() {
-        return {
-          ...this.deps.updateStateRepo.get(),
-          ...this.getSupportInfo()
-        };
+        const persisted = this.deps.updateStateRepo.get();
+        return this.composeStateView(persisted);
       }
       getPrepareInstallState() {
         const state = this.getStateView();
@@ -12694,36 +12737,15 @@ var init_update_service = __esm({
         if (current.updateStatus === "installing" || current.updateStatus === "restarting") {
           throw createBusyError("Update installation is already in progress");
         }
-        if (current.updateStatus === "checking") {
+        if (this.inFlightCheck) {
           throw createBusyError("Update check is already in progress");
         }
-        this.persistAndBroadcast({
-          updateStatus: "checking",
-          finishedAt: null,
-          errorSummary: null
-        });
+        this.inFlightCheck = this.runCheckForUpdates();
+        this.broadcastStateChange();
         try {
-          const latestVersion = await this.runLatestVersionLookup(this.runtime.packageName);
-          const availability = compareVersions(latestVersion, this.runtime.currentVersion) > 0 ? "update_available" : "up_to_date";
-          return this.persistAndBroadcast({
-            currentVersion: this.runtime.currentVersion,
-            latestVersion,
-            availability,
-            updateStatus: "idle",
-            lastCheckedAt: this.now(),
-            errorSummary: null,
-            requiresManualStep: false,
-            manualCommand: null
-          });
-        } catch (error) {
-          const message = error instanceof Error ? error.message : String(error);
-          return this.persistAndBroadcast({
-            currentVersion: this.runtime.currentVersion,
-            availability: "check_failed",
-            updateStatus: "idle",
-            lastCheckedAt: this.now(),
-            errorSummary: message
-          });
+          return await this.inFlightCheck;
+        } finally {
+          this.inFlightCheck = null;
         }
       }
       prepareInstall() {
@@ -12738,6 +12760,9 @@ var init_update_service = __esm({
         if (state.updateStatus === "installing" || state.updateStatus === "restarting") {
           throw createBusyError("Update installation is already in progress");
         }
+        if (this.inFlightCheck) {
+          throw createBusyError("Update check is already in progress");
+        }
         const targetVersion = input2.targetVersion ?? state.latestVersion;
         if (!targetVersion) {
           throw createValidationError("update_no_target", "No target version is available");
@@ -12812,6 +12837,15 @@ var init_update_service = __esm({
             errorSummary: null
           });
         }
+        if (current.updateStatus === "checking") {
+          return this.persistAndBroadcast({
+            currentVersion: this.runtime.currentVersion,
+            availability: "check_failed",
+            updateStatus: "failed",
+            finishedAt: this.now(),
+            errorSummary: "Update check did not complete before the service restarted"
+          });
+        }
         if (current.updateStatus === "installing" || current.updateStatus === "restarting") {
           return this.persistAndBroadcast({
             currentVersion: this.runtime.currentVersion,
@@ -12862,18 +12896,6 @@ var init_update_service = __esm({
           hasActiveWork: runningTerminalCount > 0 || runningSessionCount > 0 || runningSupervisorCount > 0
         };
       }
-      persistAndBroadcast(patch) {
-        const snapshot = this.deps.updateStateRepo.update((current) => ({
-          ...patch,
-          currentVersion: patch.currentVersion ?? current.currentVersion
-        }));
-        const view = {
-          ...snapshot,
-          ...this.getSupportInfo()
-        };
-        this.deps.broadcaster.broadcast("update.state.changed", view);
-        return view;
-      }
       buildManualCommand(targetVersion) {
         return [
           `${this.runtime.npmCommand ?? "npm"} install -g ${this.runtime.packageName}@${targetVersion}`,
@@ -12903,6 +12925,83 @@ var init_update_service = __esm({
         });
         child.unref();
       }
+      composeStateView(snapshot, options) {
+        if (options?.includeInFlightCheck !== false && this.inFlightCheck) {
+          return {
+            ...snapshot,
+            ...this.getSupportInfo(),
+            updateStatus: "checking",
+            errorSummary: null
+          };
+        }
+        return {
+          ...snapshot,
+          ...this.getSupportInfo()
+        };
+      }
+      broadcastStateChange() {
+        this.deps.broadcaster.broadcast("update.state.changed", this.getStateView());
+      }
+      async runCheckForUpdates() {
+        try {
+          const latestVersion = await this.withCheckTimeout(
+            this.runLatestVersionLookup(this.runtime.packageName)
+          );
+          const availability = compareVersions(latestVersion, this.runtime.currentVersion) > 0 ? "update_available" : "up_to_date";
+          return this.persistAndBroadcast(
+            {
+              currentVersion: this.runtime.currentVersion,
+              latestVersion,
+              availability,
+              updateStatus: "idle",
+              lastCheckedAt: this.now(),
+              errorSummary: null,
+              requiresManualStep: false,
+              manualCommand: null
+            },
+            false
+          );
+        } catch (error) {
+          const message = error instanceof Error ? error.message : String(error);
+          return this.persistAndBroadcast(
+            {
+              currentVersion: this.runtime.currentVersion,
+              availability: "check_failed",
+              updateStatus: "idle",
+              lastCheckedAt: this.now(),
+              errorSummary: message
+            },
+            false
+          );
+        }
+      }
+      async withCheckTimeout(promise) {
+        let timeoutHandle = null;
+        try {
+          return await Promise.race([
+            promise,
+            new Promise((_, reject) => {
+              timeoutHandle = setTimeout(() => {
+                reject(new Error(`Update check timed out after ${_UpdateService.CHECK_TIMEOUT_MS}ms`));
+              }, _UpdateService.CHECK_TIMEOUT_MS);
+              timeoutHandle.unref?.();
+            })
+          ]);
+        } finally {
+          if (timeoutHandle) {
+            clearTimeout(timeoutHandle);
+          }
+        }
+      }
+      persistAndBroadcast(patch, includeInFlightCheck = true) {
+        const snapshot = this.deps.updateStateRepo.update((current) => ({
+          ...patch,
+          currentVersion: patch.currentVersion ?? current.currentVersion
+        }));
+        const view = this.composeStateView(snapshot, { includeInFlightCheck });
+        this.deps.broadcaster.broadcast("update.state.changed", view);
+        return view;
+      }
     };
   }
 });
@@ -13759,7 +13858,8 @@ var init_terminal = __esm({
         workspaceId: z6.string(),
         cols: z6.number().int().positive().optional(),
         rows: z6.number().int().positive().optional(),
-        cwdPath: z6.string().optional()
+        cwdPath: z6.string().optional(),
+        themeBackground: z6.string().regex(/^#[0-9a-fA-F]{3,8}$/).optional()
       }),
       async (args, ctx) => {
         const workspace = ctx.workspaceMgr.get(args.workspaceId);
@@ -13797,7 +13897,8 @@ var init_terminal = __esm({
           title: shell.title,
           cwd,
           cols: args.cols ?? 120,
-          rows: args.rows ?? 30
+          rows: args.rows ?? 30,
+          themeBackground: args.themeBackground
         });
         return terminal;
       }
@@ -15369,7 +15470,8 @@ var init_session2 = __esm({
       z12.object({
         workspaceId: z12.string(),
         providerId: z12.string(),
-        draft: z12.string().optional()
+        draft: z12.string().optional(),
+        themeBackground: z12.string().regex(/^#[0-9a-fA-F]{3,8}$/).optional()
       }),
       async (args, ctx) => {
         const workspace = ctx.workspaceMgr.get(args.workspaceId);
@@ -15397,7 +15499,8 @@ var init_session2 = __esm({
           workspacePath: workspace.path,
           providerId: args.providerId,
           provider,
-          draft: args.draft
+          draft: args.draft,
+          themeBackground: args.themeBackground
         });
       }
     );