@otto-assistant/otto 0.7.15 → 0.7.16

package/dist/commands/discord-commands-group-a.test.js

@@ -429,6 +429,40 @@ describe('/model', () => {
       Select a provider:"
     `);
     });
+    test('shows provider list error details', async () => {
+        const textChannel = { id: 'channel-1', type: ChannelType.GuildText };
+        const { command } = createCommand({ channel: textChannel });
+        mockGetOttoMetadata.mockResolvedValue({ projectDirectory: '/repo' });
+        mockInitializeOpencodeForDirectory.mockResolvedValue(() => {
+            return {
+                provider: {
+                    list: vi.fn(async () => {
+                        return {
+                            error: {
+                                name: 'UnknownError',
+                                data: {
+                                    message: 'Cursor token refresh failed: {"code":"internal","message":"Error"}',
+                                },
+                            },
+                        };
+                    }),
+                },
+            };
+        });
+        await handleModelCommand({
+            interaction: command,
+            appId: 'app-1',
+        });
+        expect(command.editReply.mock.calls).toMatchInlineSnapshot(`
+      [
+        [
+          {
+            "content": "Failed to fetch providers: Cursor token refresh failed: {"code":"internal","message":"Error"}",
+          },
+        ],
+      ]
+    `);
+    });
 });
 describe('/agent', () => {
     test('shows selectable agents and current channel override', async () => {

package/dist/commands/login.js

@@ -13,6 +13,7 @@ import crypto from 'node:crypto';
 import { initializeOpencodeForDirectory, getOpencodeServerPort, } from '../opencode.js';
 import { resolveTextChannel, getOttoMetadata } from '../discord-utils.js';
 import { createLogger, LogPrefix } from '../logger.js';
+import { formatOpenCodeResponseError } from '../errors.js';
 import { buildPaginatedOptions, parsePaginationValue } from './paginated-select.js';
 const loginLogger = createLogger(LogPrefix.LOGIN);
 // ── Context store ───────────────────────────────────────────────
@@ -141,7 +142,9 @@ export async function handleLoginCommand({ interaction, }) {
             directory: projectDirectory,
         });
         if (!providersResponse.data) {
-            await interaction.editReply({ content: 'Failed to fetch providers' });
+            await interaction.editReply({
+                content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
+            });
             return;
         }
         const { all: allProviders, connected } = providersResponse.data;
@@ -272,7 +275,10 @@ async function handleProviderStep(interaction, ctx, hash, providerId) {
         }
         const providersResponse = await getClient().provider.list({ directory: ctx.dir });
         if (!providersResponse.data) {
-            await interaction.editReply({ content: 'Failed to fetch providers', components: [] });
+            await interaction.editReply({
+                content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
+                components: [],
+            });
             return;
         }
         const { all: allProviders, connected } = providersResponse.data;

package/dist/commands/model-variant.js

@@ -14,6 +14,7 @@ import { getCurrentModelInfo, ensureSessionPreferencesSnapshot, } from './model.
 import { getRuntime } from '../session-handler/thread-session-runtime.js';
 import { getThinkingValuesForModel } from '../thinking-utils.js';
 import { createLogger, LogPrefix } from '../logger.js';
+import { formatOpenCodeResponseError } from '../errors.js';
 const logger = createLogger(LogPrefix.MODEL);
 const pendingVariantContexts = new Map();
 /** 10 minute TTL for pending contexts to prevent unbounded map growth */
@@ -121,7 +122,9 @@ export async function handleModelVariantCommand({ interaction, appId, }) {
         return;
     }
     if (!providersResponse.data) {
-        await interaction.editReply({ content: 'Failed to fetch providers' });
+        await interaction.editReply({
+            content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
+        });
         return;
     }
     const { providerID, modelID, model: fullModelId } = currentModelInfo;

package/dist/commands/model.js

@@ -8,6 +8,7 @@ import { getDefaultModel } from '../session-handler/model-utils.js';
 import { getRuntime } from '../session-handler/thread-session-runtime.js';
 import { getThinkingValuesForModel } from '../thinking-utils.js';
 import { createLogger, LogPrefix } from '../logger.js';
+import { formatOpenCodeResponseError } from '../errors.js';
 import * as errore from 'errore';
 import { buildPaginatedOptions, parsePaginationValue } from './paginated-select.js';
 const modelLogger = createLogger(LogPrefix.MODEL);
@@ -270,7 +271,7 @@ export async function handleModelCommand({ interaction, appId, }) {
         ]);
         if (!providersResponse.data) {
             await interaction.editReply({
-                content: 'Failed to fetch providers',
+                content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
             });
             return;
         }
@@ -393,7 +394,10 @@ export async function handleProviderSelectMenu(interaction) {
         }
         const providersResponse = await getClient().provider.list({ directory: context.dir });
         if (!providersResponse.data) {
-            await interaction.editReply({ content: 'Failed to fetch providers', components: [] });
+            await interaction.editReply({
+                content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
+                components: [],
+            });
             return;
         }
         const { all: allProviders, connected } = providersResponse.data;
@@ -434,7 +438,7 @@ export async function handleProviderSelectMenu(interaction) {
         });
         if (!providersResponse.data) {
             await interaction.editReply({
-                content: 'Failed to fetch providers',
+                content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
                 components: [],
             });
             return;

package/dist/discord-bot.js

@@ -43,11 +43,11 @@ import * as errore from "errore";
 import { createLogger, formatErrorWithStack, LogPrefix } from "./logger.js";
 import { writeHeapSnapshot, startHeapMonitor } from "./heap-monitor.js";
 import { startTaskRunner } from "./task-runner.js";
+import { setGlobalDispatcher, Agent } from "undici";
 // Increase connection pool to prevent deadlock when multiple sessions have open SSE streams.
 // Each session's event.subscribe() holds a connection; without enough connections,
 // regular HTTP requests (question.reply, session.prompt) get blocked → deadlock.
-// undici is a transitive dep from discord.js — not listed in our package.json.
-// Types are declared in src/undici.d.ts.
+setGlobalDispatcher(new Agent({ headersTimeout: 0, bodyTimeout: 0, connections: 500 }));
 const discordLogger = createLogger(LogPrefix.DISCORD);
 const voiceLogger = createLogger(LogPrefix.VOICE);
 // Well-known WebSocket and Discord Gateway close codes for diagnostic logging.

package/dist/errors.js

@@ -155,3 +155,25 @@ export class GitCommandError extends createTaggedError({
     message: 'Git command failed: $command',
 }) {
 }
+export function formatOpenCodeResponseError(error) {
+    if (error && typeof error === 'object') {
+        if ('data' in error &&
+            error.data &&
+            typeof error.data === 'object' &&
+            'message' in error.data) {
+            return String(error.data.message);
+        }
+        if ('errors' in error &&
+            Array.isArray(error.errors) &&
+            error.errors.length > 0) {
+            return JSON.stringify(error.errors);
+        }
+        if ('message' in error && typeof error.message === 'string') {
+            return error.message;
+        }
+        if ('name' in error && typeof error.name === 'string') {
+            return error.name;
+        }
+    }
+    return 'Unknown OpenCode API error';
+}

package/dist/opencode.js

@@ -39,7 +39,7 @@ const STARTUP_STDERR_TAIL_LIMIT = 30;
 const STARTUP_STDERR_LINE_MAX_LENGTH = 120;
 const STARTUP_ERROR_REASON_MAX_LENGTH = 1500;
 const ANSI_ESCAPE_REGEX = /[\u001B\u009B][[\]()#;?]*(?:(?:(?:[a-zA-Z\d]*(?:;[a-zA-Z\d]*)*)?\u0007)|(?:(?:\d{1,4}(?:;\d{0,4})*)?[\dA-PR-TZcf-nq-uy=><~]))/g;
-async function requestHealthcheck({ url, }) {
+async function requestHealthcheck({ url, timeoutMs = 2000, }) {
     return new Promise((resolve, reject) => {
         const req = http.request(url, {
             method: 'GET',
@@ -58,6 +58,13 @@ async function requestHealthcheck({ url, }) {
                 });
             });
         });
+        // Without a timeout, a stalled connection (TCP accepted during opencode
+        // startup before the HTTP layer is ready) hangs forever, wedging
+        // waitForServer on its first iteration and blocking the entire single
+        // server startup promise. Abort and let the poll loop retry instead.
+        req.setTimeout(timeoutMs, () => {
+            req.destroy(new Error(`healthcheck timed out after ${timeoutMs}ms`));
+        });
         req.on('error', reject);
         req.end();
     });

package/package.json

@@ -7,7 +7,7 @@
   },
   "module": "index.ts",
   "type": "module",
-  "version": "0.7.15",
+  "version": "0.7.16",
   "scripts": {
     "dev": "tsx src/bin.ts",
     "prepublishOnly": "pnpm build",
@@ -52,8 +52,7 @@
     "opencode-cached-provider": "workspace:^",
     "opencode-deterministic-provider": "workspace:^",
     "prisma": "7.4.2",
-    "tsx": "^4.20.5",
-    "undici": "^8.0.2"
+    "tsx": "^4.20.5"
   },
   "dependencies": {
     "@ai-sdk/google": "^3.0.53",
@@ -88,6 +87,7 @@
     "proper-lockfile": "^4.1.2",
     "string-dedent": "^3.0.2",
     "traforo": "^0.4.0",
+    "undici": "^8.0.2",
     "ws": "^8.19.0",
     "xdg-basedir": "^5.1.0",
     "yaml": "^2.8.3",

package/src/commands/discord-commands-group-a.test.ts

@@ -512,6 +512,44 @@ describe('/model', () => {
     `,
     )
   })
+
+  test('shows provider list error details', async () => {
+    const textChannel = { id: 'channel-1', type: ChannelType.GuildText }
+    const { command } = createCommand({ channel: textChannel })
+
+    mockGetOttoMetadata.mockResolvedValue({ projectDirectory: '/repo' })
+    mockInitializeOpencodeForDirectory.mockResolvedValue(() => {
+      return {
+        provider: {
+          list: vi.fn(async () => {
+            return {
+              error: {
+                name: 'UnknownError',
+                data: {
+                  message: 'Cursor token refresh failed: {"code":"internal","message":"Error"}',
+                },
+              },
+            }
+          }),
+        },
+      }
+    })
+
+    await handleModelCommand({
+      interaction: command as never,
+      appId: 'app-1',
+    })
+
+    expect(command.editReply.mock.calls).toMatchInlineSnapshot(`
+      [
+        [
+          {
+            "content": "Failed to fetch providers: Cursor token refresh failed: {"code":"internal","message":"Error"}",
+          },
+        ],
+      ]
+    `)
+  })
 })
 
 describe('/agent', () => {

package/src/commands/login.ts

@@ -34,6 +34,7 @@ import {
 } from '../opencode.js'
 import { resolveTextChannel, getOttoMetadata } from '../discord-utils.js'
 import { createLogger, LogPrefix } from '../logger.js'
+import { formatOpenCodeResponseError } from '../errors.js'
 import { buildPaginatedOptions, parsePaginationValue } from './paginated-select.js'
 
 const loginLogger = createLogger(LogPrefix.LOGIN)
@@ -266,7 +267,9 @@ export async function handleLoginCommand({
     })
 
     if (!providersResponse.data) {
-      await interaction.editReply({ content: 'Failed to fetch providers' })
+      await interaction.editReply({
+        content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
+      })
       return
     }
 
@@ -416,7 +419,10 @@ async function handleProviderStep(
     }
     const providersResponse = await getClient().provider.list({ directory: ctx.dir })
     if (!providersResponse.data) {
-      await interaction.editReply({ content: 'Failed to fetch providers', components: [] })
+      await interaction.editReply({
+        content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
+        components: [],
+      })
       return
     }
     const { all: allProviders, connected } = providersResponse.data

package/src/commands/model-variant.ts

@@ -34,6 +34,7 @@ import {
 import { getRuntime } from '../session-handler/thread-session-runtime.js'
 import { getThinkingValuesForModel } from '../thinking-utils.js'
 import { createLogger, LogPrefix } from '../logger.js'
+import { formatOpenCodeResponseError } from '../errors.js'
 
 const logger = createLogger(LogPrefix.MODEL)
 
@@ -184,7 +185,9 @@ export async function handleModelVariantCommand({
   }
 
   if (!providersResponse.data) {
-    await interaction.editReply({ content: 'Failed to fetch providers' })
+    await interaction.editReply({
+      content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
+    })
     return
   }
 

package/src/commands/model.ts

@@ -30,6 +30,7 @@ import { getDefaultModel } from '../session-handler/model-utils.js'
 import { getRuntime } from '../session-handler/thread-session-runtime.js'
 import { getThinkingValuesForModel } from '../thinking-utils.js'
 import { createLogger, LogPrefix } from '../logger.js'
+import { formatOpenCodeResponseError } from '../errors.js'
 import * as errore from 'errore'
 import { buildPaginatedOptions, parsePaginationValue } from './paginated-select.js'
 
@@ -447,7 +448,7 @@ export async function handleModelCommand({
 
     if (!providersResponse.data) {
       await interaction.editReply({
-        content: 'Failed to fetch providers',
+        content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
       })
       return
     }
@@ -596,7 +597,10 @@ export async function handleProviderSelectMenu(
     }
     const providersResponse = await getClient().provider.list({ directory: context.dir })
     if (!providersResponse.data) {
-      await interaction.editReply({ content: 'Failed to fetch providers', components: [] })
+      await interaction.editReply({
+        content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
+        components: [],
+      })
       return
     }
     const { all: allProviders, connected } = providersResponse.data
@@ -640,7 +644,7 @@ export async function handleProviderSelectMenu(
 
     if (!providersResponse.data) {
       await interaction.editReply({
-        content: 'Failed to fetch providers',
+        content: `Failed to fetch providers: ${formatOpenCodeResponseError(providersResponse.error)}`,
         components: [],
       })
       return

package/src/discord-bot.ts

@@ -130,11 +130,13 @@ import * as errore from "errore";
 import { createLogger, formatErrorWithStack, LogPrefix } from "./logger.js";
 import { writeHeapSnapshot, startHeapMonitor } from "./heap-monitor.js";
 import { startTaskRunner } from "./task-runner.js";
+import { setGlobalDispatcher, Agent } from "undici";
 // Increase connection pool to prevent deadlock when multiple sessions have open SSE streams.
 // Each session's event.subscribe() holds a connection; without enough connections,
 // regular HTTP requests (question.reply, session.prompt) get blocked → deadlock.
-// undici is a transitive dep from discord.js — not listed in our package.json.
-// Types are declared in src/undici.d.ts.
+setGlobalDispatcher(
+  new Agent({ headersTimeout: 0, bodyTimeout: 0, connections: 500 }),
+);
 
 const discordLogger = createLogger(LogPrefix.DISCORD);
 const voiceLogger = createLogger(LogPrefix.VOICE);

package/src/errors.ts

@@ -168,6 +168,37 @@ export class GitCommandError extends createTaggedError({
   message: 'Git command failed: $command',
 }) {}
 
+export function formatOpenCodeResponseError(error: unknown): string {
+  if (error && typeof error === 'object') {
+    if (
+      'data' in error &&
+      error.data &&
+      typeof error.data === 'object' &&
+      'message' in error.data
+    ) {
+      return String(error.data.message)
+    }
+
+    if (
+      'errors' in error &&
+      Array.isArray(error.errors) &&
+      error.errors.length > 0
+    ) {
+      return JSON.stringify(error.errors)
+    }
+
+    if ('message' in error && typeof error.message === 'string') {
+      return error.message
+    }
+
+    if ('name' in error && typeof error.name === 'string') {
+      return error.name
+    }
+  }
+
+  return 'Unknown OpenCode API error'
+}
+
 // ═══════════════════════════════════════════════════════════════════════════
 // UNION TYPES - For function signatures
 // ═══════════════════════════════════════════════════════════════════════════

package/src/opencode.ts

@@ -82,8 +82,10 @@ const ANSI_ESCAPE_REGEX =
 
 async function requestHealthcheck({
   url,
+  timeoutMs = 2000,
 }: {
   url: string
+  timeoutMs?: number
 }): Promise<{ status: number; body: string }> {
   return new Promise((resolve, reject) => {
     const req = http.request(
@@ -107,6 +109,13 @@ async function requestHealthcheck({
         })
       },
     )
+    // Without a timeout, a stalled connection (TCP accepted during opencode
+    // startup before the HTTP layer is ready) hangs forever, wedging
+    // waitForServer on its first iteration and blocking the entire single
+    // server startup promise. Abort and let the poll loop retry instead.
+    req.setTimeout(timeoutMs, () => {
+      req.destroy(new Error(`healthcheck timed out after ${timeoutMs}ms`))
+    })
     req.on('error', reject)
     req.end()
   })