npm - dominds - Versions diffs - 1.16.1 → 1.16.2 - Mend

dominds 1.16.1 → 1.16.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (175) hide show

package/dist/llm/api-quirks.js CHANGED Viewed

@@ -2,40 +2,321 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.normalizeProviderApiQuirks = normalizeProviderApiQuirks;
 exports.createLlmFailureQuirkHandlerSession = createLlmFailureQuirkHandlerSession;
+const persistence_errors_1 = require("../persistence-errors");
 const DOMINDS_LLM_EMPTY_RESPONSE_ERROR_CODE = 'DOMINDS_LLM_EMPTY_RESPONSE';
 const XCODE_BEST_EMPTY_RESPONSE_SINGLE_RETRY_DELAY_MS = 3000;
 const XCODE_BEST_EMPTY_RESPONSE_GIVE_UP_THRESHOLD = 5;
+const XCODE_BEST_GATEWAY_HTML_502_RETRY_MESSAGE = 'xcode.best gateway returned an HTML 502 Bad Gateway page; retrying conservatively.';
+const XCODE_BEST_UNEXPECTED_EOF_RETRY_MESSAGE = 'xcode.best upstream stream ended unexpectedly (unexpected EOF); retrying conservatively.';
+const LOCAL_FILE_IO_ERROR_CODES = new Set(['ENOENT', 'ENOTDIR', 'EISDIR', 'EACCES', 'EPERM']);
+const LOCAL_FILE_IO_SYSCALLS = new Set([
+    'open',
+    'read',
+    'write',
+    'close',
+    'stat',
+    'lstat',
+    'fstat',
+    'rename',
+    'truncate',
+    'mkdir',
+    'readdir',
+]);
+function isXcodeBestGatewayHtml502Failure(failure, error) {
+    const status = failure.status ?? readErrorStatus(error);
+    if (status !== 502) {
+        return false;
+    }
+    const message = readErrorMessage(error) ?? failure.message;
+    const lowerMessage = message.toLowerCase();
+    if (!lowerMessage.includes('bad gateway')) {
+        return false;
+    }
+    if (lowerMessage.includes('<!doctype html')) {
+        return true;
+    }
+    return lowerMessage.includes('<html') || lowerMessage.includes('cloudflare');
+}
+function isXcodeBestUnexpectedEofFailure(failure, error) {
+    return ((errorChainIncludesMessageFragment(error, 'unexpected eof') ||
+        failure.message.toLowerCase().includes('unexpected eof')) &&
+        !hasLikelyLocalFileErrorContext(error));
+}
+function getErrorChain(error) {
+    const queue = [error];
+    const visited = new Set();
+    const chain = [];
+    while (queue.length > 0) {
+        const current = queue.shift();
+        if (current === undefined) {
+            continue;
+        }
+        if (typeof current === 'object' && current !== null) {
+            if (visited.has(current)) {
+                continue;
+            }
+            visited.add(current);
+        }
+        chain.push(current);
+        const nestedError = readNestedError(current);
+        if (nestedError !== undefined) {
+            queue.push(nestedError);
+        }
+        const cause = readErrorCause(current);
+        if (cause !== undefined) {
+            queue.push(cause);
+        }
+    }
+    return chain;
+}
+function readErrorStatus(error) {
+    for (const current of getErrorChain(error)) {
+        if (isRecord(current)) {
+            if ('status' in current && typeof current.status === 'number') {
+                return current.status;
+            }
+            if ('statusCode' in current && typeof current.statusCode === 'number') {
+                return current.statusCode;
+            }
+        }
+    }
+    return undefined;
+}
+function readErrorCause(error) {
+    if (error instanceof Error) {
+        const withCause = error;
+        return withCause.cause;
+    }
+    if (isRecord(error) && 'cause' in error) {
+        return error.cause;
+    }
+    return undefined;
+}
+function readNestedError(error) {
+    if (isRecord(error) && 'error' in error && isRecord(error.error)) {
+        return error.error;
+    }
+    return undefined;
+}
+function readErrorPath(error) {
+    for (const current of getErrorChain(error)) {
+        if (current instanceof Error) {
+            const withPath = current;
+            if (typeof withPath.path === 'string' && withPath.path.trim() !== '') {
+                return withPath.path;
+            }
+        }
+        if (isRecord(current)) {
+            if ('path' in current && typeof current.path === 'string' && current.path.trim() !== '') {
+                return current.path;
+            }
+        }
+    }
+    return undefined;
+}
+function readErrorSyscall(error) {
+    for (const current of getErrorChain(error)) {
+        if (current instanceof Error) {
+            const withSyscall = current;
+            if (typeof withSyscall.syscall === 'string' && withSyscall.syscall.trim() !== '') {
+                return withSyscall.syscall;
+            }
+        }
+        if (isRecord(current)) {
+            if ('syscall' in current &&
+                typeof current.syscall === 'string' &&
+                current.syscall.trim() !== '') {
+                return current.syscall;
+            }
+        }
+    }
+    return undefined;
+}
+function readErrorMessage(error) {
+    for (const current of getErrorChain(error)) {
+        if (current instanceof Error) {
+            const trimmed = current.message.trim();
+            if (trimmed.length > 0) {
+                return trimmed;
+            }
+            if (current.name.trim().length > 0) {
+                return current.name;
+            }
+        }
+        if (typeof current === 'string') {
+            const trimmed = current.trim();
+            if (trimmed.length > 0) {
+                return trimmed;
+            }
+        }
+        if (isRecord(current)) {
+            if ('message' in current && typeof current.message === 'string') {
+                const trimmed = current.message.trim();
+                if (trimmed.length > 0) {
+                    return trimmed;
+                }
+            }
+        }
+    }
+    return undefined;
+}
+function errorChainIncludesMessageFragment(error, fragment) {
+    const lowerFragment = fragment.toLowerCase();
+    for (const current of getErrorChain(error)) {
+        if (current instanceof Error) {
+            if (current.message.toLowerCase().includes(lowerFragment)) {
+                return true;
+            }
+            continue;
+        }
+        if (typeof current === 'string') {
+            if (current.toLowerCase().includes(lowerFragment)) {
+                return true;
+            }
+            continue;
+        }
+        if (isRecord(current) && 'message' in current && typeof current.message === 'string') {
+            if (current.message.toLowerCase().includes(lowerFragment)) {
+                return true;
+            }
+        }
+    }
+    return false;
+}
+function isRecord(value) {
+    return typeof value === 'object' && value !== null && !Array.isArray(value);
+}
+function hasLikelyLocalFileErrorContext(error) {
+    const persistenceFileError = (0, persistence_errors_1.findDomindsPersistenceFileError)(error);
+    if (persistenceFileError) {
+        return true;
+    }
+    const pathValue = readErrorPath(error);
+    const syscall = readErrorSyscall(error);
+    if (typeof syscall === 'string' && LOCAL_FILE_IO_SYSCALLS.has(syscall)) {
+        return true;
+    }
+    const code = readErrorCode(error);
+    if (typeof code === 'string' &&
+        LOCAL_FILE_IO_ERROR_CODES.has(code) &&
+        isLikelyDomindsFilePath(pathValue)) {
+        return true;
+    }
+    if (isLikelyDomindsFilePath(pathValue)) {
+        return true;
+    }
+    return false;
+}
+function isLikelyDomindsFilePath(filePath) {
+    if (typeof filePath !== 'string') {
+        return false;
+    }
+    const normalized = filePath.split('\\').join('/').trim().toLowerCase();
+    if (normalized === '') {
+        return false;
+    }
+    return (normalized.includes('/.dialogs/') ||
+        normalized.includes('/.minds/') ||
+        normalized.endsWith('/latest.yaml') ||
+        normalized.endsWith('/dialog.yaml') ||
+        normalized.endsWith('/q4h.yaml') ||
+        normalized.endsWith('.jsonl'));
+}
+function readErrorCode(error) {
+    for (const current of getErrorChain(error)) {
+        if (current instanceof Error) {
+            const withCode = current;
+            if (typeof withCode.code === 'string' && withCode.code.trim() !== '') {
+                return withCode.code;
+            }
+            if (typeof withCode.errno === 'string' && withCode.errno.trim() !== '') {
+                return withCode.errno;
+            }
+        }
+        if (isRecord(current)) {
+            if ('code' in current && typeof current.code === 'string' && current.code.trim() !== '') {
+                return current.code;
+            }
+            if ('errno' in current && typeof current.errno === 'string' && current.errno.trim() !== '') {
+                return current.errno;
+            }
+        }
+    }
+    return undefined;
+}
+function buildXcodeBestEmptyResponseGiveUpText(providerConfig, provider) {
+    const providerName = providerConfig.name.trim().length > 0 ? providerConfig.name : provider;
+    const summaryTextI18n = {
+        zh: `${providerName} 在同一对话上下文中连续返回 empty response。` +
+            `Dominds 已在 ${String(XCODE_BEST_EMPTY_RESPONSE_GIVE_UP_THRESHOLD)} 次 empty response 后停止继续重试，因为这通常表示 provider 侧该对话上下文已经卡住；` +
+            '如果直接点继续，大概率仍然无真实进展；更建议结合真实情况灵活尝试多种新的指令，例如改写问题、补充上下文、换一个切入方式。',
+        en: `${providerName} returned empty responses repeatedly for the same dialog context. ` +
+            `Dominds stopped retrying after ${String(XCODE_BEST_EMPTY_RESPONSE_GIVE_UP_THRESHOLD)} empty responses because this usually means the provider-side conversation ` +
+            'context is stuck; simply pressing Continue is still unlikely to make real progress, ' +
+            'so it is better to try different fresh instructions based on the real situation, such as reframing the ask, adding context, or changing the angle.',
+    };
+    return {
+        providerName,
+        summaryTextI18n,
+        recoveryAction: { kind: 'diligence_push_once' },
+    };
+}
 function createXcodeBestFailureQuirkHandlerSession(providerConfig) {
     let consecutiveEmptyResponseCount = 0;
+    let consumedDiligencePushRecoverySinceLastSuccess = false;
     return {
+        quirkName: 'xcode.best',
         onFailure(args) {
-            if (args.failure.code !== DOMINDS_LLM_EMPTY_RESPONSE_ERROR_CODE) {
-                consecutiveEmptyResponseCount = 0;
-                return { kind: 'default' };
+            const { providerName, summaryTextI18n, recoveryAction } = buildXcodeBestEmptyResponseGiveUpText(providerConfig, args.provider);
+            if (args.failure.code === DOMINDS_LLM_EMPTY_RESPONSE_ERROR_CODE) {
+                consecutiveEmptyResponseCount += 1;
+                if (consecutiveEmptyResponseCount < XCODE_BEST_EMPTY_RESPONSE_GIVE_UP_THRESHOLD) {
+                    return {
+                        kind: 'single_retry',
+                        delayMs: XCODE_BEST_EMPTY_RESPONSE_SINGLE_RETRY_DELAY_MS,
+                    };
+                }
+                return {
+                    kind: 'give_up',
+                    message: `${providerName} returned empty responses repeatedly for the same dialog context; ` +
+                        'automatic retries were stopped; simply continuing is still unlikely to make real progress, so it is better to flexibly try different fresh instructions based on the real situation.',
+                    summaryTextI18n,
+                    recoveryAction: consumedDiligencePushRecoverySinceLastSuccess
+                        ? { kind: 'none' }
+                        : recoveryAction,
+                };
             }
-            consecutiveEmptyResponseCount += 1;
-            if (consecutiveEmptyResponseCount < XCODE_BEST_EMPTY_RESPONSE_GIVE_UP_THRESHOLD) {
+            consecutiveEmptyResponseCount = 0;
+            if (isXcodeBestUnexpectedEofFailure(args.failure, args.error)) {
                 return {
-                    kind: 'single_retry',
-                    delayMs: XCODE_BEST_EMPTY_RESPONSE_SINGLE_RETRY_DELAY_MS,
+                    kind: 'retry_strategy',
+                    retryStrategy: 'conservative',
+                    message: XCODE_BEST_UNEXPECTED_EOF_RETRY_MESSAGE,
                 };
             }
-            const providerName = providerConfig.name.trim().length > 0 ? providerConfig.name : args.provider;
-            const summaryTextI18n = {
-                zh: `${providerName} 在同一对话上下文中连续返回 empty response。` +
-                    `Dominds 已在 ${String(XCODE_BEST_EMPTY_RESPONSE_GIVE_UP_THRESHOLD)} 次 empty response 后停止继续重试，因为这通常表示 provider 侧该对话上下文已经卡住；` +
-                    '继续调整常规重试策略也无助于恢复。请新开对话继续。',
-                en: `${providerName} returned empty responses repeatedly for the same dialog context. ` +
-                    `Dominds stopped retrying after ${String(XCODE_BEST_EMPTY_RESPONSE_GIVE_UP_THRESHOLD)} empty responses because this usually means the provider-side conversation ` +
-                    'context is stuck; adjusting the normal retry policies will not recover it. ' +
-                    'Start a new dialog to continue.',
-            };
-            return {
-                kind: 'give_up',
-                message: `${providerName} returned empty responses repeatedly for the same dialog context; ` +
-                    'automatic retries will not help until a new dialog is started.',
-                summaryTextI18n,
-            };
+            if (isXcodeBestGatewayHtml502Failure(args.failure, args.error)) {
+                return {
+                    kind: 'retry_strategy',
+                    retryStrategy: 'conservative',
+                    message: XCODE_BEST_GATEWAY_HTML_502_RETRY_MESSAGE,
+                };
+            }
+            return { kind: 'default' };
+        },
+        onRequestSucceeded() {
+            consecutiveEmptyResponseCount = 0;
+            consumedDiligencePushRecoverySinceLastSuccess = false;
+        },
+        onRecoveryActionUsed(usage) {
+            if (usage.action.kind !== 'diligence_push_once') {
+                return;
+            }
+            // This state intentionally spans multiple request invocations in the same driver run, even if
+            // the dialog opens a new course in between; provider/API retry heuristics are treated as
+            // independent from course boundaries. Once the driver accepts and reserves this automatic
+            // recovery path for the current stop decision, consider it consumed immediately.
+            consecutiveEmptyResponseCount = 0;
+            consumedDiligencePushRecoverySinceLastSuccess = true;
         },
     };
 }
@@ -69,14 +350,31 @@ function createLlmFailureQuirkHandlerSession(providerConfig) {
         return undefined;
     }
     return {
+        quirkName: 'aggregate',
         onFailure(args) {
             for (const session of sessions) {
                 const handling = session.onFailure(args);
                 if (handling.kind !== 'default') {
-                    return handling;
+                    return {
+                        ...handling,
+                        sourceQuirk: handling.sourceQuirk ?? session.quirkName,
+                    };
                 }
             }
             return { kind: 'default' };
         },
+        onRequestSucceeded() {
+            for (const session of sessions) {
+                session.onRequestSucceeded?.();
+            }
+        },
+        onRecoveryActionUsed(usage) {
+            for (const session of sessions) {
+                if (usage.sourceQuirk !== undefined && usage.sourceQuirk !== session.quirkName) {
+                    continue;
+                }
+                session.onRecoveryActionUsed?.(usage);
+            }
+        },
     };
 }

package/dist/llm/defaults.yaml CHANGED Viewed

@@ -7,7 +7,8 @@
 # - apiQuirks: optional provider/gateway quirk profile(s) for non-standard transport behavior
 #   and provider-specific failure handling that can override the driver's default retry disposition.
 #   Example: `apiQuirks: xcode.best` currently covers both keepalive/heartbeat stream quirks and
-#   repeated empty-response handling for the same dialog context.
+#   repeated empty-response handling for the same dialog context, plus conservative retry
+#   classification for gateway-returned HTML 502 Bad Gateway pages.
 # - tool_result_max_chars: optional transport-level cap for a single tool-result text payload
 #   before Dominds projects it into the provider request. Use this when a provider/gateway enforces
 #   a stricter per-item string limit than Dominds' built-in defaults.

package/dist/llm/gen/mock.d.ts CHANGED Viewed

@@ -56,7 +56,7 @@
 import type { Team } from '../../team';
 import type { FuncTool } from '../../tool';
 import type { ChatMessage, ProviderConfig } from '../client';
-import type { LlmBatchResult, LlmGenerator, LlmRequestContext, LlmStreamReceiver, LlmStreamResult } from '../gen';
+import { type LlmBatchResult, type LlmGenerator, type LlmRequestContext, type LlmStreamReceiver, type LlmStreamResult } from '../gen';
 export declare class MockGen implements LlmGenerator {
     get apiType(): string;
     private databaseCache;

package/dist/llm/gen/mock.js CHANGED Viewed

@@ -64,6 +64,8 @@ const path_1 = __importDefault(require("path"));
 const yaml_1 = __importDefault(require("yaml"));
 const log_1 = require("../../log");
 const reply_prompt_copy_1 = require("../../runtime/reply-prompt-copy");
+const gen_1 = require("../gen");
+const stop_reason_i18n_1 = require("../stop-reason-i18n");
 const REPLY_TOOL_REMINDER_PREFIXES = [
     '[Dominds replyTellask required]',
     '[Dominds 必须调用回复工具]',
@@ -376,6 +378,16 @@ responses:
         await receiver.thinkingChunk(content.substring(0, 50) || '(empty)');
         await receiver.thinkingFinish();
         if (matched?.streamError) {
+            if (matched.emitStreamErrorBeforeThrow && receiver.streamError) {
+                await receiver.streamError(matched.streamError);
+                throw new gen_1.LlmStreamErrorEmittedError({
+                    detail: matched.streamError,
+                    i18nStopReason: (0, stop_reason_i18n_1.buildHumanSystemStopReasonTextI18n)({
+                        detail: matched.streamError,
+                        fallbackKind: 'request_failed',
+                    }),
+                });
+            }
             throw new Error(matched.streamError);
         }
         const responseText = matched?.response ?? this.makeFallbackResponse(dbPath, content, role, modelName);

package/dist/llm/gen/openai-compatible.d.ts CHANGED Viewed

@@ -14,7 +14,7 @@ import type { ChatCompletionMessageParam } from 'openai/resources/chat/completio
 import type { Team } from '../../team';
 import type { FuncTool } from '../../tool';
 import type { ChatMessage, ProviderConfig } from '../client';
-import type { LlmBatchResult, LlmFailureDisposition, LlmGenerator, LlmRequestContext, LlmStreamReceiver, LlmStreamResult } from '../gen';
+import { type LlmBatchResult, type LlmFailureDisposition, type LlmGenerator, type LlmRequestContext, type LlmStreamReceiver, type LlmStreamResult } from '../gen';
 export declare function buildOpenAiCompatibleRequestMessagesWrapper(systemPrompt: string, context: ChatMessage[], options?: {
     reasoningContentMode?: boolean;
     providerConfig?: ProviderConfig;

package/dist/llm/gen/openai-compatible.js CHANGED Viewed

@@ -21,6 +21,8 @@ const openai_1 = __importDefault(require("openai"));
 const log_1 = require("../../log");
 const i18n_text_1 = require("../../runtime/i18n-text");
 const work_language_1 = require("../../runtime/work-language");
+const gen_1 = require("../gen");
+const stop_reason_i18n_1 = require("../stop-reason-i18n");
 const artifacts_1 = require("./artifacts");
 const failure_classifier_1 = require("./failure-classifier");
 const tool_call_context_1 = require("./tool-call-context");
@@ -624,7 +626,13 @@ class OpenAiCompatibleGen {
                             if (receiver.streamError) {
                                 await receiver.streamError(detail);
                             }
-                            throw new Error(detail);
+                            throw new gen_1.LlmStreamErrorEmittedError({
+                                detail,
+                                i18nStopReason: (0, stop_reason_i18n_1.buildHumanSystemStopReasonTextI18n)({
+                                    detail,
+                                    kind: 'invalid_tool_call',
+                                }),
+                            });
                         }
                         const index = rawIndex;
                         const existing = activeCallsByIndex.get(index);

package/dist/llm/gen/openai.d.ts CHANGED Viewed

@@ -9,7 +9,7 @@ import type { ResponseInputItem } from 'openai/resources/responses/responses';
 import type { Team } from '../../team';
 import type { FuncTool } from '../../tool';
 import type { ChatMessage, ProviderConfig } from '../client';
-import type { LlmBatchResult, LlmFailureDisposition, LlmGenerator, LlmRequestContext, LlmStreamReceiver, LlmStreamResult } from '../gen';
+import { type LlmBatchResult, type LlmFailureDisposition, type LlmGenerator, type LlmRequestContext, type LlmStreamReceiver, type LlmStreamResult } from '../gen';
 export declare function buildOpenAiRequestInputWrapper(context: ChatMessage[], providerConfig?: ProviderConfig): Promise<ResponseInputItem[]>;
 export declare class OpenAiGen implements LlmGenerator {
     get apiType(): string;

package/dist/llm/gen/openai.js CHANGED Viewed

@@ -17,6 +17,8 @@ const log_1 = require("../../log");
 const i18n_text_1 = require("../../runtime/i18n-text");
 const work_language_1 = require("../../runtime/work-language");
 const api_quirks_1 = require("../api-quirks");
+const gen_1 = require("../gen");
+const stop_reason_i18n_1 = require("../stop-reason-i18n");
 const artifacts_1 = require("./artifacts");
 const failure_classifier_1 = require("./failure-classifier");
 const tool_call_context_1 = require("./tool-call-context");
@@ -502,7 +504,13 @@ async function throwOpenAiMalformedStreamEvent(receiver, eventType, detail) {
     if (receiver.streamError) {
         await receiver.streamError(message);
     }
-    throw new Error(message);
+    throw new gen_1.LlmStreamErrorEmittedError({
+        detail: message,
+        i18nStopReason: (0, stop_reason_i18n_1.buildHumanSystemStopReasonTextI18n)({
+            detail: message,
+            kind: 'malformed_stream',
+        }),
+    });
 }
 function buildOpenAiNativeToolCallFromState(state, phase) {
     if (state.itemType === 'custom_tool_call') {
@@ -1701,7 +1709,13 @@ class OpenAiGen {
                 if (receiver.streamError) {
                     await receiver.streamError(detail);
                 }
-                throw new Error(detail);
+                throw new gen_1.LlmStreamErrorEmittedError({
+                    detail,
+                    i18nStopReason: (0, stop_reason_i18n_1.buildHumanSystemStopReasonTextI18n)({
+                        detail,
+                        kind: 'incomplete_tool_call_stream',
+                    }),
+                });
             }
         }
         catch (error) {

package/dist/llm/gen.d.ts CHANGED Viewed

@@ -1,4 +1,5 @@
 import type { LlmUsageStats } from '@longrun-ai/kernel/types/context-health';
+import type { DialogDisplayTextI18n } from '@longrun-ai/kernel/types/display-state';
 import type { ReasoningPayload } from '@longrun-ai/kernel/types/storage';
 import { Team } from '../team';
 import { FuncTool } from '../tool';
@@ -7,6 +8,15 @@ export interface LlmStreamResult {
     usage: LlmUsageStats;
     llmGenModel?: string;
 }
+export declare class LlmStreamErrorEmittedError extends Error {
+    readonly detail: string;
+    readonly i18nStopReason: DialogDisplayTextI18n;
+    constructor(args: {
+        detail: string;
+        message?: string;
+        i18nStopReason: DialogDisplayTextI18n;
+    });
+}
 export type LlmBatchOutput = {
     kind: 'message';
     message: ChatMessage;

package/dist/llm/gen.js CHANGED Viewed

@@ -1,2 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.LlmStreamErrorEmittedError = void 0;
+class LlmStreamErrorEmittedError extends Error {
+    constructor(args) {
+        super(args.message ?? args.detail);
+        this.name = 'LlmStreamErrorEmittedError';
+        this.detail = args.detail;
+        this.i18nStopReason = args.i18nStopReason;
+    }
+}
+exports.LlmStreamErrorEmittedError = LlmStreamErrorEmittedError;