agor-live 0.17.2 → 0.17.3

package/dist/core/db/index.js

@@ -523,6 +523,8 @@ var init_task = __esm({
     "use strict";
     init_esm_shims();
     TaskStatus = {
+      QUEUED: "queued",
+      // Task created but not yet running (waiting for executor to drain queue)
       CREATED: "created",
       RUNNING: "running",
       STOPPING: "stopping",
@@ -553,6 +555,21 @@ function normalizeRole(role) {
   if (role === "owner") return ROLES.SUPERADMIN;
   return role || ROLES.MEMBER;
 }
+function toAgenticToolsStatus(stored) {
+  if (!stored) return void 0;
+  const out = {};
+  for (const [tool, fields] of Object.entries(stored)) {
+    if (!fields) continue;
+    const flags = {};
+    for (const [field, value] of Object.entries(fields)) {
+      if (value) flags[field] = true;
+    }
+    if (Object.keys(flags).length > 0) {
+      out[tool] = flags;
+    }
+  }
+  return Object.keys(out).length > 0 ? out : void 0;
+}
 var ROLES, ROLE_RANK;
 var init_user = __esm({
   "src/types/user.ts"() {
@@ -759,6 +776,7 @@ var init_schema_postgres = __esm({
         completed_at: t.timestamp("completed_at"),
         status: text("status", {
           enum: [
+            "queued",
             "created",
             "running",
             "stopping",
@@ -770,6 +788,8 @@ var init_schema_postgres = __esm({
             "stopped"
           ]
         }).notNull(),
+        // Queue position (lower drains first); only populated for status='queued'
+        queue_position: integer("queue_position"),
         // User attribution
         created_by: varchar("created_by", { length: 36 }).notNull().default("anonymous"),
         // MD5 of SDK session file at task completion (only populated when stateless_fs_mode is enabled)
@@ -779,7 +799,12 @@ var init_schema_postgres = __esm({
       (table) => ({
         sessionIdx: index("tasks_session_idx").on(table.session_id),
         statusIdx: index("tasks_status_idx").on(table.status),
-        createdIdx: index("tasks_created_idx").on(table.created_at)
+        createdIdx: index("tasks_created_idx").on(table.created_at),
+        queueIdx: index("tasks_queue_idx").on(table.session_id, table.status, table.queue_position),
+        // Partial unique index — defense-in-depth for `tasks.createPending` race
+        // serialization. Only QUEUED rows are constrained; CREATED/RUNNING/done
+        // rows have NULL queue_position and are unaffected.
+        queuedPositionUnique: uniqueIndex("tasks_queued_position_unique").on(table.session_id, table.queue_position).where(sql`${table.status} = 'queued'`)
       })
     );
     serializedSessions = pgTable(
@@ -839,11 +864,9 @@ var init_schema_postgres = __esm({
         // First 200 chars for list views
         // Parent tool use ID (for nested tool calls - e.g., Task tool spawning Read/Grep)
         parent_tool_use_id: text("parent_tool_use_id"),
-        // Message queueing fields
-        status: text("status", { enum: ["queued"] }),
-        // 'queued' or null (normal message)
-        queue_position: integer("queue_position"),
-        // Position in queue (1, 2, 3, ...)
+        // NOTE: queueing moved off `messages` and onto `tasks.status='queued'` as
+        // of migration sqlite/0040 (postgres/0030). The legacy `status` and
+        // `queue_position` columns are gone — see `tasks.queue_position` instead.
         // Full data (JSON blob)
         data: t.json("data").$type().notNull()
       },
@@ -851,8 +874,7 @@ var init_schema_postgres = __esm({
         // Indexes for efficient lookups
         sessionIdx: index("messages_session_id_idx").on(table.session_id),
         taskIdx: index("messages_task_id_idx").on(table.task_id),
-        sessionIndexIdx: index("messages_session_index_idx").on(table.session_id, table.index),
-        queueIdx: index("messages_queue_idx").on(table.session_id, table.status, table.queue_position)
+        sessionIndexIdx: index("messages_session_index_idx").on(table.session_id, table.index)
       })
     );
     boards = pgTable(
@@ -1497,6 +1519,7 @@ var init_schema_sqlite = __esm({
         completed_at: t2.timestamp("completed_at"),
         status: text2("status", {
           enum: [
+            "queued",
             "created",
             "running",
             "stopping",
@@ -1508,6 +1531,8 @@ var init_schema_sqlite = __esm({
             "stopped"
           ]
         }).notNull(),
+        // Queue position (lower drains first); only populated for status='queued'
+        queue_position: integer2("queue_position"),
         // User attribution
         created_by: text2("created_by", { length: 36 }).notNull().default("anonymous"),
         // MD5 of SDK session file at task completion (only populated when stateless_fs_mode is enabled)
@@ -1517,7 +1542,12 @@ var init_schema_sqlite = __esm({
       (table) => ({
         sessionIdx: index2("tasks_session_idx").on(table.session_id),
         statusIdx: index2("tasks_status_idx").on(table.status),
-        createdIdx: index2("tasks_created_idx").on(table.created_at)
+        createdIdx: index2("tasks_created_idx").on(table.created_at),
+        queueIdx: index2("tasks_queue_idx").on(table.session_id, table.status, table.queue_position),
+        // Partial unique index — defense-in-depth for `tasks.createPending` race
+        // serialization. Only QUEUED rows are constrained; CREATED/RUNNING/done
+        // rows have NULL queue_position and are unaffected.
+        queuedPositionUnique: uniqueIndex2("tasks_queued_position_unique").on(table.session_id, table.queue_position).where(sql2`${table.status} = 'queued'`)
       })
     );
     serializedSessions2 = sqliteTable(
@@ -1577,11 +1607,9 @@ var init_schema_sqlite = __esm({
         // First 200 chars for list views
         // Parent tool use ID (for nested tool calls - e.g., Task tool spawning Read/Grep)
         parent_tool_use_id: text2("parent_tool_use_id"),
-        // Message queueing fields
-        status: text2("status", { enum: ["queued"] }),
-        // 'queued' or null (normal message)
-        queue_position: integer2("queue_position"),
-        // Position in queue (1, 2, 3, ...)
+        // NOTE: queueing moved off `messages` and onto `tasks.status='queued'` as
+        // of migration sqlite/0040 (postgres/0030). The legacy `status` and
+        // `queue_position` columns are gone — see `tasks.queue_position` instead.
         // Full data (JSON blob)
         data: t2.json("data").$type().notNull()
       },
@@ -1589,8 +1617,7 @@ var init_schema_sqlite = __esm({
         // Indexes for efficient lookups
         sessionIdx: index2("messages_session_id_idx").on(table.session_id),
         taskIdx: index2("messages_task_id_idx").on(table.task_id),
-        sessionIndexIdx: index2("messages_session_index_idx").on(table.session_id, table.index),
-        queueIdx: index2("messages_queue_idx").on(table.session_id, table.status, table.queue_position)
+        sessionIndexIdx: index2("messages_session_index_idx").on(table.session_id, table.index)
       })
     );
     boards2 = sqliteTable(
@@ -5473,8 +5500,6 @@ var MessagesRepository = class {
       content: row.data.content,
       tool_uses: row.data.tool_uses,
       parent_tool_use_id: row.parent_tool_use_id || void 0,
-      status: row.status,
-      queue_position: row.queue_position ?? void 0,
       metadata: row.data.metadata
     };
   }
@@ -5493,8 +5518,6 @@ var MessagesRepository = class {
       timestamp: new Date(message.timestamp),
       content_preview: message.content_preview,
       parent_tool_use_id: message.parent_tool_use_id || null,
-      status: message.status || null,
-      queue_position: message.queue_position ?? null,
       data: {
         content: message.content,
         tool_uses: message.tool_uses,
@@ -5593,68 +5616,6 @@ var MessagesRepository = class {
   async delete(messageId) {
     await deleteFrom(this.db, messages3).where(eq10(messages3.message_id, messageId)).run();
   }
-  /**
-   * Create a queued message
-   * NOTE: Queued messages always store prompt as string content
-   * This ensures compatibility with prompt execution endpoint
-   *
-   * @param sessionId - Session to queue message for
-   * @param prompt - Message content (string)
-   * @param metadata - Optional metadata (e.g., is_agor_callback, source, child_session_id)
-   */
-  async createQueued(sessionId, prompt, metadata) {
-    if (!prompt || typeof prompt !== "string") {
-      throw new Error("Prompt must be a non-empty string");
-    }
-    const { generateId: generateId2 } = await Promise.resolve().then(() => (init_ids(), ids_exports));
-    const result = await select(this.db).from(messages3).where(and6(eq10(messages3.session_id, sessionId), eq10(messages3.status, "queued"))).all();
-    const maxPosition = result.reduce(
-      (max, row) => Math.max(max, row.queue_position || 0),
-      0
-    );
-    const nextPosition = maxPosition + 1;
-    const messageType = metadata?.is_agor_callback ? "system" : "user";
-    const message = {
-      message_id: generateId2(),
-      session_id: sessionId,
-      type: messageType,
-      role: "user",
-      // Always 'user' for right-side positioning
-      index: -1,
-      // Not in conversation yet
-      timestamp: (/* @__PURE__ */ new Date()).toISOString(),
-      content_preview: prompt.substring(0, 200),
-      content: prompt,
-      // Always string for queued messages
-      status: "queued",
-      queue_position: nextPosition,
-      task_id: void 0,
-      metadata
-      // Include metadata (is_agor_callback, source, child_session_id, etc.)
-    };
-    return this.create(message);
-  }
-  /**
-   * Find queued messages for a session
-   */
-  async findQueued(sessionId) {
-    const { asc: asc2 } = await import("drizzle-orm");
-    const rows = await select(this.db).from(messages3).where(and6(eq10(messages3.session_id, sessionId), eq10(messages3.status, "queued"))).orderBy(asc2(messages3.queue_position)).all();
-    return rows.map((r) => this.rowToMessage(r));
-  }
-  /**
-   * Get next queued message
-   */
-  async getNextQueued(sessionId) {
-    const queued = await this.findQueued(sessionId);
-    return queued[0] || null;
-  }
-  /**
-   * Delete queued message (when processing or user cancels)
-   */
-  async deleteQueued(messageId) {
-    await this.delete(messageId);
-  }
 };
 
 // src/db/repositories/repos.ts
@@ -6331,7 +6292,7 @@ init_esm_shims();
 init_types();
 init_config_manager();
 init_ids();
-import { and as and9, desc as desc2, eq as eq14, inArray as inArray2, isNotNull as isNotNull2, isNull as isNull2, like as like9, or as or3, sql as sql7 } from "drizzle-orm";
+import { and as and9, desc as desc2, eq as eq14, inArray as inArray2, isNotNull as isNotNull2, like as like9, or as or3, sql as sql7 } from "drizzle-orm";
 init_database_wrapper();
 init_schema();
 var SessionRepository = class {
@@ -6817,14 +6778,7 @@ var SessionRepository = class {
       for (const sessionId of sessionIds) {
         const query = select(this.db, {
           data: messagesTable.data
-        }).from(messagesTable).where(
-          and9(
-            eq14(messagesTable.session_id, sessionId),
-            eq14(messagesTable.role, "assistant"),
-            isNull2(messagesTable.status)
-            // Exclude queued messages
-          )
-        );
+        }).from(messagesTable).where(and9(eq14(messagesTable.session_id, sessionId), eq14(messagesTable.role, "assistant")));
         const lastMessage = await query.orderBy(desc2(messagesTable.index)).limit(1).one();
         if (lastMessage) {
           const messageData = lastMessage.data;
@@ -7112,6 +7066,7 @@ var TaskRepository = class {
       task_id: row.task_id,
       session_id: row.session_id,
       status: row.status,
+      queue_position: row.queue_position ?? void 0,
       created_at: new Date(row.created_at).toISOString(),
       completed_at: row.completed_at ? new Date(row.completed_at).toISOString() : void 0,
       created_by: row.created_by,
@@ -7139,11 +7094,11 @@ var TaskRepository = class {
       // Always use server timestamp, ignore client-provided value
       completed_at: task.completed_at ? new Date(task.completed_at) : void 0,
       status: task.status ?? TaskStatus.CREATED,
+      queue_position: task.queue_position ?? null,
       created_by: task.created_by ?? "anonymous",
       session_md5: task.session_md5 ?? null,
       data: {
-        description: task.description ?? "",
-        full_prompt: task.full_prompt ?? task.description ?? "",
+        full_prompt: task.full_prompt ?? "",
         message_range: task.message_range ?? {
           start_index: 0,
           end_index: 0,
@@ -7165,8 +7120,10 @@ var TaskRepository = class {
         computed_context_window: task.computed_context_window,
         // Cumulative context window (computed by tool.computeContextWindow())
         report: task.report,
-        permission_request: task.permission_request
+        permission_request: task.permission_request,
         // Permission state for UI approval flow
+        metadata: task.metadata
+        // Generic metadata bag (e.g., is_agor_callback, source)
       }
     };
   }
@@ -7293,7 +7250,12 @@ var TaskRepository = class {
   }
   /**
    * Find orphaned tasks (running, stopping, awaiting permission, or awaiting input)
-   * These are tasks that were interrupted when daemon stopped
+   * These are tasks that were interrupted when daemon stopped.
+   *
+   * NOTE: QUEUED tasks are intentionally NOT considered orphans — they were
+   * never spawned, so they have no executor to recover. The startup queue
+   * drainer (see register-routes.ts processNextQueuedTask) picks them up
+   * once any session goes idle. See never-lose-prompt §C.
    */
   async findOrphaned() {
     try {
@@ -7345,6 +7307,7 @@ var TaskRepository = class {
         const insertData = this.taskToInsert(merged);
         await update(txAsDb(tx), tasks3).set({
           status: insertData.status,
+          queue_position: insertData.queue_position,
           completed_at: insertData.completed_at,
           session_md5: insertData.session_md5,
           data: insertData.data
@@ -7379,6 +7342,92 @@ var TaskRepository = class {
       );
     }
   }
+  /**
+   * Create a pending task — either CREATED (will spawn immediately) or
+   * QUEUED (will drain later) — owning the sentinel defaults that the
+   * caller would otherwise have to assemble by hand.
+   *
+   * For QUEUED tasks, `queue_position = max(queue_position) + 1` is computed
+   * inside a transaction so concurrent writers don't both observe the same
+   * max and collide. (The schema also carries a partial unique index on
+   * `(session_id, queue_position) WHERE status='queued'` as a belt-and-
+   * suspenders against transaction-isolation surprises.)
+   *
+   * Sentinel contract: while a task carries `message_range.start_index = -1`
+   * and `git_state.sha_at_start = ''`, it has not yet been pinned to real
+   * conversation/git state. spawnTaskExecutor is the sole place that
+   * overwrites these on the way to RUNNING.
+   */
+  async createPending(input) {
+    const taskBase = {
+      session_id: input.session_id,
+      full_prompt: input.full_prompt,
+      created_by: input.created_by,
+      status: input.status,
+      metadata: input.metadata,
+      // Sentinels — overwritten by spawnTaskExecutor at the status → RUNNING
+      // transition. While `start_index === -1` / `sha_at_start === ''`, the
+      // task is intentionally unpinned.
+      message_range: {
+        start_index: -1,
+        end_index: -1,
+        start_timestamp: (/* @__PURE__ */ new Date()).toISOString()
+      },
+      git_state: {
+        ref_at_start: "",
+        sha_at_start: ""
+      },
+      tool_use_count: 0
+    };
+    if (input.status === TaskStatus.CREATED) {
+      return this.create(taskBase);
+    }
+    return this.db.transaction(async (tx) => {
+      const positionRow = await select(txAsDb(tx), {
+        maxPos: sql8`max(${tasks3.queue_position})`
+      }).from(tasks3).where(sql8`${tasks3.session_id} = ${input.session_id} AND ${tasks3.status} = 'queued'`).one();
+      const nextPosition = (positionRow?.maxPos ?? 0) + 1;
+      const insertData = this.taskToInsert({
+        ...taskBase,
+        queue_position: nextPosition
+      });
+      await insert(txAsDb(tx), tasks3).values(insertData).run();
+      const row = await select(txAsDb(tx)).from(tasks3).where(eq16(tasks3.task_id, insertData.task_id)).one();
+      if (!row) {
+        throw new RepositoryError("Failed to retrieve created queued task");
+      }
+      return this.rowToTask(row);
+    });
+  }
+  /**
+   * Find all QUEUED tasks for a session, ordered by queue_position ascending.
+   */
+  async findQueued(sessionId) {
+    try {
+      const rows = await select(this.db).from(tasks3).where(sql8`${tasks3.session_id} = ${sessionId} AND ${tasks3.status} = 'queued'`).orderBy(tasks3.queue_position).all();
+      return rows.map((row) => this.rowToTask(row));
+    } catch (error) {
+      throw new RepositoryError(
+        `Failed to find queued tasks: ${error instanceof Error ? error.message : String(error)}`,
+        error
+      );
+    }
+  }
+  /**
+   * Return the next QUEUED task to drain (lowest queue_position) for a session,
+   * or null if none.
+   */
+  async getNextQueued(sessionId) {
+    try {
+      const row = await select(this.db).from(tasks3).where(sql8`${tasks3.session_id} = ${sessionId} AND ${tasks3.status} = 'queued'`).orderBy(tasks3.queue_position).limit(1).one();
+      return row ? this.rowToTask(row) : null;
+    } catch (error) {
+      throw new RepositoryError(
+        `Failed to get next queued task: ${error instanceof Error ? error.message : String(error)}`,
+        error
+      );
+    }
+  }
   /**
    * Count tasks for a session
    */
@@ -7779,7 +7828,7 @@ var UserApiKeysRepository = class {
 init_esm_shims();
 init_database_wrapper();
 init_schema();
-import { and as and13, eq as eq19, isNull as isNull3 } from "drizzle-orm";
+import { and as and13, eq as eq19, isNull as isNull2 } from "drizzle-orm";
 function rowToToken(row) {
   return {
     user_id: row.user_id ?? null,
@@ -7795,7 +7844,7 @@ function rowToToken(row) {
 }
 function matchKey(userId, serverId) {
   return and13(
-    userId === null ? isNull3(userMcpOauthTokens3.user_id) : eq19(userMcpOauthTokens3.user_id, userId),
+    userId === null ? isNull2(userMcpOauthTokens3.user_id) : eq19(userMcpOauthTokens3.user_id, userId),
     eq19(userMcpOauthTokens3.mcp_server_id, serverId)
   );
 }
@@ -7947,6 +7996,7 @@ var UserMCPOAuthTokenRepository = class {
 
 // src/db/repositories/users.ts
 init_esm_shims();
+init_types();
 import { eq as eq20, like as like12 } from "drizzle-orm";
 
 // src/config/env-vars.ts
@@ -7986,8 +8036,9 @@ var UsersRepository = class {
     this.db = db;
   }
   /**
-   * Convert database row to User type
-   * Note: Converts encrypted API keys (strings) to boolean flags for API exposure
+   * Convert database row to User type.
+   * Converts the encrypted `agentic_tools` blob to a boolean presence DTO so
+   * decrypted credentials never leave this repository.
    */
   rowToUser(row) {
     return {
@@ -8003,13 +8054,8 @@ var UsersRepository = class {
       must_change_password: row.must_change_password,
       avatar: row.data.avatar,
       preferences: row.data.preferences,
-      // Convert encrypted keys to boolean flags (true = key exists, false/undefined = no key)
-      api_keys: row.data.api_keys ? {
-        ANTHROPIC_API_KEY: !!row.data.api_keys.ANTHROPIC_API_KEY,
-        OPENAI_API_KEY: !!row.data.api_keys.OPENAI_API_KEY,
-        GEMINI_API_KEY: !!row.data.api_keys.GEMINI_API_KEY,
-        COPILOT_GITHUB_TOKEN: !!row.data.api_keys.COPILOT_GITHUB_TOKEN
-      } : void 0,
+      // Convert encrypted per-tool credential blobs into boolean presence flags.
+      agentic_tools: toAgenticToolsStatus(row.data.agentic_tools),
       // Convert stored env vars to presence + scope metadata (never exposes secrets).
       // Handles both legacy string form and v0.5 object form via normalizeStoredEnvMap.
       // The schema stores `scope` as a generic string (no SQL CHECK constraint); the
@@ -8053,10 +8099,17 @@ var UsersRepository = class {
       data: {
         avatar: user.avatar,
         preferences: user.preferences,
-        // Use raw API keys if provided (for internal operations like setApiKey)
-        api_keys: user.api_keys_raw,
-        env_vars: void 0,
-        // Not implemented yet
+        // Encrypted per-tool credentials. Only forwarded when caller passes the
+        // raw shape (internal credential mutators); regular updates leave it undefined,
+        // letting the merge in `update()` reuse the existing on-disk blob.
+        // Cast: schema declares `opencode: Record<string, never>` (no fields by
+        // contract); StoredAgenticTools widens that to string values for shape
+        // uniformity. Runtime never writes opencode, so the cast is safe.
+        agentic_tools: user.agentic_tools_raw,
+        // Same pass-through as agentic_tools: env_vars are encrypted blobs
+        // not represented on the public DTO. `update()` threads the raw value
+        // from the existing row so a generic field update doesn't wipe them.
+        env_vars: user.env_vars_raw,
         default_agentic_config: user.default_agentic_config
       }
     };
@@ -8168,7 +8221,14 @@ var UsersRepository = class {
         );
       }
     }
+    const rawRow = await this.getRawRow(fullId);
     const merged = { ...current, ...updates };
+    if (rawRow?.data.agentic_tools) {
+      merged.agentic_tools_raw = rawRow.data.agentic_tools;
+    }
+    if (rawRow?.data.env_vars) {
+      merged.env_vars_raw = rawRow.data.env_vars;
+    }
     const insertData = this.userToInsert(merged);
     await update(this.db, users3).set({
       ...insertData,
@@ -8203,96 +8263,103 @@ var UsersRepository = class {
     }
   }
   /**
-   * Get decrypted API key for a user and service
+   * Get the full decrypted credential bag for a single agentic tool.
    *
-   * @param userId - User ID
-   * @param service - Service name ('anthropic', 'openai', 'gemini')
-   * @returns Decrypted API key or null if not found
+   * Returns `null` when the user has no stored config for that tool.
+   * Fields that fail to decrypt are dropped from the returned object and
+   * logged — callers see "missing field" rather than a thrown error so a
+   * single corrupt value doesn't poison an entire SDK spawn.
    */
-  async getApiKey(userId, service) {
+  async getToolConfig(userId, tool) {
     const row = await this.getRawRow(userId);
-    if (!row || !row.data.api_keys) {
-      return null;
-    }
-    const keyMap = {
-      anthropic: "ANTHROPIC_API_KEY",
-      openai: "OPENAI_API_KEY",
-      gemini: "GEMINI_API_KEY",
-      copilot: "COPILOT_GITHUB_TOKEN"
-    };
-    const encryptedKey = row.data.api_keys[keyMap[service]];
-    if (!encryptedKey) {
-      return null;
+    if (!row) return null;
+    const stored = row.data.agentic_tools;
+    const fields = stored?.[tool];
+    if (!fields || Object.keys(fields).length === 0) return null;
+    const out = {};
+    for (const [field, encrypted] of Object.entries(fields)) {
+      if (!encrypted) continue;
+      try {
+        out[field] = decryptApiKey(encrypted);
+      } catch (error) {
+        console.error(
+          `[users] Failed to decrypt ${tool}.${field} for user ${userId.substring(0, 8)}: ${error.message}`
+        );
+      }
     }
+    return Object.keys(out).length > 0 ? out : null;
+  }
+  /**
+   * Get a single decrypted credential field for a tool.
+   *
+   * Returns `null` when the field is unset OR when decryption fails (logged).
+   * Throws only on storage-layer errors, not on missing/corrupt values.
+   */
+  async getToolConfigField(userId, tool, field) {
+    const row = await this.getRawRow(userId);
+    if (!row) return null;
+    const stored = row.data.agentic_tools;
+    const encrypted = stored?.[tool]?.[field];
+    if (!encrypted) return null;
     try {
-      return decryptApiKey(encryptedKey);
+      return decryptApiKey(encrypted);
     } catch (error) {
-      throw new RepositoryError(`Failed to decrypt API key for service ${service}`, error);
+      console.error(
+        `[users] Failed to decrypt ${tool}.${field} for user ${userId.substring(0, 8)}: ${error.message}`
+      );
+      return null;
     }
   }
   /**
-   * Set encrypted API key for a user and service
+   * Set (encrypt + persist) a single credential field for a tool.
    *
-   * @param userId - User ID
-   * @param service - Service name ('anthropic', 'openai', 'gemini')
-   * @param apiKey - Plaintext API key to encrypt and store
+   * Field names are env-var-shaped (e.g. ANTHROPIC_API_KEY, ANTHROPIC_BASE_URL)
+   * and are stored encrypted regardless of whether the value is a secret —
+   * keeping the on-disk shape uniform avoids decrypt-vs-plain branching at
+   * read time. UI controls own the text-vs-password rendering distinction.
    */
-  async setApiKey(userId, service, apiKey) {
+  async setToolConfigField(userId, tool, field, value) {
     const fullId = await this.resolveId(userId);
     const row = await this.getRawRow(fullId);
     if (!row) {
       throw new EntityNotFoundError("User", userId);
     }
-    const keyMap = {
-      anthropic: "ANTHROPIC_API_KEY",
-      openai: "OPENAI_API_KEY",
-      gemini: "GEMINI_API_KEY",
-      copilot: "COPILOT_GITHUB_TOKEN"
-    };
-    const encryptedKey = encryptApiKey(apiKey);
-    const updatedApiKeys = {
-      ...row.data.api_keys || {},
-      [keyMap[service]]: encryptedKey
-    };
-    const user = this.rowToUser(row);
-    const updateData = {
-      ...user,
-      api_keys_raw: updatedApiKeys
+    const stored = row.data.agentic_tools ?? {};
+    const next = {
+      ...stored,
+      [tool]: {
+        ...stored[tool] ?? {},
+        [field]: encryptApiKey(value)
+      }
     };
-    const insertData = this.userToInsert(updateData);
     await update(this.db, users3).set({
-      ...insertData,
+      data: { ...row.data, agentic_tools: next },
       updated_at: /* @__PURE__ */ new Date()
     }).where(eq20(users3.user_id, fullId)).run();
   }
   /**
-   * Delete API key for a user and service
+   * Delete a single credential field for a tool.
    *
-   * @param userId - User ID
-   * @param service - Service name ('anthropic', 'openai', 'gemini')
+   * If the tool's bucket becomes empty after the delete, the bucket itself is
+   * removed so `data.agentic_tools` doesn't accumulate empty objects.
    */
-  async deleteApiKey(userId, service) {
+  async deleteToolConfigField(userId, tool, field) {
     const fullId = await this.resolveId(userId);
     const row = await this.getRawRow(fullId);
     if (!row) {
       throw new EntityNotFoundError("User", userId);
     }
-    const keyMap = {
-      anthropic: "ANTHROPIC_API_KEY",
-      openai: "OPENAI_API_KEY",
-      gemini: "GEMINI_API_KEY",
-      copilot: "COPILOT_GITHUB_TOKEN"
-    };
-    const updatedApiKeys = { ...row.data.api_keys || {} };
-    delete updatedApiKeys[keyMap[service]];
-    const user = this.rowToUser(row);
-    const updateData = {
-      ...user,
-      api_keys_raw: updatedApiKeys
-    };
-    const insertData = this.userToInsert(updateData);
+    const stored = row.data.agentic_tools ?? {};
+    const toolFields = { ...stored[tool] ?? {} };
+    delete toolFields[field];
+    const next = { ...stored };
+    if (Object.keys(toolFields).length > 0) {
+      next[tool] = toolFields;
+    } else {
+      delete next[tool];
+    }
     await update(this.db, users3).set({
-      ...insertData,
+      data: { ...row.data, agentic_tools: next },
       updated_at: /* @__PURE__ */ new Date()
     }).where(eq20(users3.user_id, fullId)).run();
   }
@@ -8304,18 +8371,7 @@ init_types();
 init_ids();
 init_database_wrapper();
 init_schema();
-import {
-  and as and14,
-  desc as desc3,
-  eq as eq21,
-  getTableColumns,
-  inArray as inArray3,
-  isNotNull as isNotNull3,
-  isNull as isNull4,
-  like as like13,
-  or as or4,
-  sql as sql9
-} from "drizzle-orm";
+import { and as and14, desc as desc3, eq as eq21, getTableColumns, inArray as inArray3, isNotNull as isNotNull3, like as like13, or as or4, sql as sql9 } from "drizzle-orm";
 var WorktreeRepository = class {
   constructor(db) {
     this.db = db;
@@ -8843,14 +8899,7 @@ var WorktreeRepository = class {
       for (const sessionId of sessionIds) {
         const query = select(this.db, {
           data: messagesTable.data
-        }).from(messagesTable).where(
-          and14(
-            eq21(messagesTable.session_id, sessionId),
-            eq21(messagesTable.role, "assistant"),
-            isNull4(messagesTable.status)
-            // Exclude queued messages
-          )
-        );
+        }).from(messagesTable).where(and14(eq21(messagesTable.session_id, sessionId), eq21(messagesTable.role, "assistant")));
         const lastMessage = await query.orderBy(desc3(messagesTable.index)).limit(1).one();
         if (lastMessage) {
           const messageData = lastMessage.data;