@monster-orange/den 0.4.11 → 0.4.13

package/src/ui/server/editor-open-pref.ts

@@ -0,0 +1,41 @@
+/**
+ * Where a file path in an agent's pane opens (AG-4b): ⌘-click (Ctrl-click
+ * elsewhere) opens it in this editor by its URL scheme, at the line
+ * (vscode://file/…, cursor://file/…, zed://file/…), or, with `none` (the
+ * default), copies its absolute path. A person's view preference, stored
+ * in the user config as `editor_open` (the `editor` key is already the
+ * repo editor's, ↗) and served by /api/settings as `editor_open`. Nothing
+ * here runs an editor: the browser hands the URL to the OS.
+ */
+import type { UserConfig } from "../../shared/den-config.js";
+
+export const EDITOR_OPENS = ["vscode", "cursor", "zed", "none"] as const;
+export type EditorOpen = (typeof EDITOR_OPENS)[number];
+export const EDITOR_OPEN_DEFAULT: EditorOpen = "none";
+
+const isEditorOpen = (v: unknown): v is EditorOpen =>
+  typeof v === "string" && (EDITOR_OPENS as readonly string[]).includes(v);
+
+/** The stored choice, or none when unset or hand-broken. */
+export function editorOpen(user: UserConfig): EditorOpen {
+  const v = user.editor_open;
+  return isEditorOpen(v) ? v : EDITOR_OPEN_DEFAULT;
+}
+
+/**
+ * Apply a POSTed `editor_open` to `user` (in place): an editor sets it,
+ * null clears it. Anything else: the error to answer with.
+ */
+export function patchEditorOpen(
+  user: UserConfig,
+  v: unknown,
+): { ok: true } | { ok: false; error: string } {
+  if (v !== null && !isEditorOpen(v))
+    return {
+      ok: false,
+      error: `editor_open must be one of ${EDITOR_OPENS.map((e) => `"${e}"`).join(", ")}, or null`,
+    };
+  if (v === null) delete user.editor_open;
+  else user.editor_open = v;
+  return { ok: true };
+}

package/src/ui/server/index.ts

@@ -256,6 +256,8 @@ import {
   sshTarget,
 } from "../../shared/cli-command.js";
 import { graphPrefs, patchStoredGraph } from "./graph-prefs.js";
+import { agentsLayout, patchAgentsLayout } from "./agents-layout-pref.js";
+import { editorOpen, patchEditorOpen } from "./editor-open-pref.js";
 import { graphPageHtml } from "./graph-page.js";
 import { mergeShellPrefs, shellPrefs, storedShell } from "./shell-prefs.js";
 import { distributionSettings, userWebPort } from "./settings-distribution.js";
@@ -4467,6 +4469,10 @@ async function handleRequest(
   //   graph: { direction, group, density, panelWidth } (see graph-prefs.ts)
   //   appearance: { theme: id | null, mode: system|dark|light } (appearance-prefs.ts)
   //   shell: { pins, collapsed, hidden, order, dock, dockHeight } (shell-prefs.ts)
+  //   agents_layout: focus | tabs | tile, the Agents tab's layout (AG-4b,
+  //     agents-layout-pref.ts; user.yml agents.layout); null clears it
+  //   editor_open: vscode | cursor | zed | none, where a path in an agent's
+  //     pane opens (AG-4b, editor-open-pref.ts); null clears it (none)
   //   language: the UI's language, en | ja | es (ui-language.ts); null or
   //     "en" clears it (English is the default)
   // It writes config/local/user.yml only. `distribution` is read-only: what
@@ -4512,6 +4518,8 @@ async function handleRequest(
         graph: graphPrefs(user.graph),
         appearance: appearancePrefs(user.appearance),
         shell: shellPrefs(user.shell),
+        agents_layout: agentsLayout(user),
+        editor_open: editorOpen(user),
         distribution: distributionSettings(getDistRoot()),
         // Set from the command line (`<cli> config`), shown read-only.
         user_web_port: userWebPort(user as Record<string, unknown>),
@@ -4537,8 +4545,20 @@ async function handleRequest(
           shell?: unknown;
           language?: unknown;
           machine_name?: unknown;
+          agents_layout?: unknown;
+          editor_open?: unknown;
         };
         const user = readUserConfig();
+        // AG-4b: the Agents tab's layout, a view preference only.
+        if (body.agents_layout !== undefined) {
+          const l = patchAgentsLayout(user, body.agents_layout);
+          if (!l.ok) return send400(res, l.error);
+        }
+        // AG-4b: where a path link in an agent's pane opens.
+        if (body.editor_open !== undefined) {
+          const e = patchEditorOpen(user, body.editor_open);
+          if (!e.ok) return send400(res, e.error);
+        }
         // SEC-2a: web.machine_name (user.yml); null or "" clears it.
         if (body.machine_name !== undefined) {
           let name: string | null;
@@ -4648,6 +4668,8 @@ async function handleRequest(
           shell: shellPrefs(user.shell),
           language: uiLanguage(user.language),
           machine_name: machineNameSetting(user),
+          agents_layout: agentsLayout(user),
+          editor_open: editorOpen(user),
         });
       } catch (e) {
         return send400(res, `Invalid request: ${(e as Error).message}`);

package/src/ui/server/public/agent-start.js

@@ -37,9 +37,10 @@ export function personaOptions(personas) {
     .join("");
 }
 
-/** The help line under the persona select: the picked persona's role. */
+/** The help line under the persona select: the picked persona's description. */
 export function personaHelp(personas, id) {
-  return (personas ?? []).find((p) => p.id === id)?.role ?? "";
+  const p = (personas ?? []).find((x) => x.id === id);
+  return p?.description ?? p?.role ?? "";
 }
 
 /** A runtime's name for people ("Claude Code"); its id when it has none. */