@vellumai/assistant 0.3.6 → 0.3.8

package/src/tools/executor.ts

@@ -699,15 +699,18 @@ export class ToolExecutor {
     } catch (err) {
       const durationMs = Date.now() - startTime;
       const msg = err instanceof Error ? err.message : String(err);
-      const isExpected = err instanceof PermissionDeniedError || err instanceof ToolError || err instanceof TokenExpiredError;
-
-      const errorCategory = err instanceof PermissionDeniedError
-        ? 'permission_denied' as const
-        : err instanceof TokenExpiredError
-          ? 'auth' as const
-          : err instanceof ToolError
-            ? 'tool_failure' as const
-            : 'unexpected' as const;
+      const isAbort = err instanceof Error && err.name === 'AbortError';
+      const isExpected = isAbort || err instanceof PermissionDeniedError || err instanceof ToolError || err instanceof TokenExpiredError;
+
+      const errorCategory = isAbort
+        ? 'tool_failure' as const
+        : err instanceof PermissionDeniedError
+          ? 'permission_denied' as const
+          : err instanceof TokenExpiredError
+            ? 'auth' as const
+            : err instanceof ToolError
+              ? 'tool_failure' as const
+              : 'unexpected' as const;
 
       emitLifecycleEvent(context, {
         type: 'error',

package/src/tools/subagent/message.ts

@@ -23,9 +23,16 @@ export async function executeSubagentMessage(
     };
   }
 
-  const sent = manager.sendMessage(subagentId, content);
+  const result = manager.sendMessage(subagentId, content);
 
-  if (!sent) {
+  if (result === 'queue_full') {
+    return {
+      content: `Subagent "${subagentId}" message queue is full. Please wait for current messages to be processed.`,
+      isError: true,
+    };
+  }
+
+  if (result !== 'sent') {
     return {
       content: `Could not send message to subagent "${subagentId}". It may not exist or be in a terminal state.`,
       isError: true,

package/src/util/logger.ts

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, readdirSync, unlinkSync } from 'node:fs';
+import { chmodSync, existsSync, mkdirSync, readdirSync, unlinkSync } from 'node:fs';
 import { join } from 'node:path';
 import { Writable } from 'node:stream';
 import pino from 'pino';
@@ -67,6 +67,8 @@ function buildRotatingLogger(config: LogFileConfig): pino.Logger {
   const today = formatDate(new Date());
   const filePath = logFilePathForDate(config.dir, new Date());
   const fileStream = pino.destination({ dest: filePath, sync: false, mkdir: true, mode: 0o600 });
+  // Tighten permissions on pre-existing log files that may have been created with looser modes
+  try { chmodSync(filePath, 0o600); } catch { /* best-effort */ }
 
   activeLogDate = today;
   activeLogFileConfig = config;
@@ -130,7 +132,10 @@ function getRootLogger(): pino.Logger {
     }
 
     try {
-      const fileStream = pino.destination({ dest: getLogPath(), sync: false, mkdir: true, mode: 0o600 });
+      const logPath = getLogPath();
+      const fileStream = pino.destination({ dest: logPath, sync: false, mkdir: true, mode: 0o600 });
+      // Tighten permissions on pre-existing log files that may have been created with looser modes
+      try { chmodSync(logPath, 0o600); } catch { /* best-effort */ }
 
       if (getDebugMode()) {
         const prettyStream = pinoPretty({ destination: 2 });