n8n 2.32.3 → 2.32.5

package/dist/modules/instance-ai/instance-ai.service.js

@@ -177,6 +177,7 @@ function getAbortReason(signal) {
     return typeof reason === 'string' ? reason : 'user_cancelled';
 }
 const MAX_CONCURRENT_BACKGROUND_TASKS_PER_THREAD = 5;
+const MAX_CONSECUTIVE_FAILED_INTERNAL_FOLLOW_UPS = 3;
 const TITLE_REFINE_HISTORY_LIMIT = 50;
 function toConfirmationData(request) {
     switch (request.kind) {
@@ -256,6 +257,7 @@ let InstanceAiService = class InstanceAiService {
         this.threadPushRef = new Map();
         this.planRequestsByThread = new Map();
         this.schedulerLocks = new Map();
+        this.failedInternalFollowUpStreaks = new Map();
         this.pendingCheckpointReentries = new Map();
         this.runDebugBuffer = new instance_ai_1.RunDebugBuffer();
         this.checkpointPruningStopped = true;
@@ -842,6 +844,7 @@ let InstanceAiService = class InstanceAiService {
             metadata: { completion_source: 'service_cleanup' },
         });
         this.schedulerLocks.delete(threadId);
+        this.failedInternalFollowUpStreaks.delete(threadId);
         this.domainAccessTrackersByThread.delete(threadId);
         this.evalCredentialAllowlists.clearThread(threadId);
         this.threadPushRef.delete(threadId);
@@ -1222,6 +1225,9 @@ let InstanceAiService = class InstanceAiService {
         const localGatewayDisabledForUser = await this.settingsService.isLocalGatewayDisabledForUser(user.id);
         const userGateway = this.gatewayService.findGateway(user.id);
         const { searchProxyConfig, tracingProxyConfig, tokenManager, proxyBaseUrl } = proxyRunConfig ?? (await this.createProxyRunConfig(user));
+        const modelId = proxyBaseUrl && tokenManager
+            ? await this.modelService.resolveProxyModel(user, proxyBaseUrl, tokenManager)
+            : await this.modelService.resolveAgentModelConfig(user);
         const configEvalsEnabled = await this.adapterService.isConfigEvalsEnabled(user);
         const context = this.adapterService.createContext(user, {
             searchProxyConfig,
@@ -1230,6 +1236,7 @@ let InstanceAiService = class InstanceAiService {
             projectId: boundProjectId,
             credentialIdAllowlist: this.evalCredentialAllowlists.get(threadId),
             configEvalsEnabled,
+            modelId,
         });
         this.gatewayService.applyToolPolicy(user.id);
         const gatewayMcpServer = !localGatewayDisabledForUser && userGateway?.isConnected ? userGateway : undefined;
@@ -1295,9 +1302,6 @@ let InstanceAiService = class InstanceAiService {
                 status: localGatewayDisabledForUser ? 'disabled' : 'disconnected',
             };
         }
-        const modelId = proxyBaseUrl && tokenManager
-            ? await this.modelService.resolveProxyModel(user, proxyBaseUrl, tokenManager)
-            : await this.modelService.resolveAgentModelConfig(user);
         const taskStorage = new instance_ai_1.ThreadTaskStorage(memory);
         const iterationLog = this.dbIterationLogStorage;
         const snapshotStorage = this.dbSnapshotStorage;
@@ -1778,11 +1782,29 @@ let InstanceAiService = class InstanceAiService {
     async releaseWorkItemSetupRoutingClaim(threadId, workItemId, claimId) {
         await new instance_ai_1.WorkflowLoopStorage(this.agentMemory).releaseSetupRoutingClaim(threadId, workItemId, claimId);
     }
+    updateInternalFollowUpFailureStreak(threadId, status, isInternalFollowUp) {
+        if (status === 'completed' || status === 'suspended') {
+            this.failedInternalFollowUpStreaks.delete(threadId);
+            return;
+        }
+        if (status === 'error' && isInternalFollowUp) {
+            this.failedInternalFollowUpStreaks.set(threadId, (this.failedInternalFollowUpStreaks.get(threadId) ?? 0) + 1);
+        }
+    }
     async startInternalFollowUpRun(user, threadId, message, messageGroupId, isReplanFollowUp = false, checkpoint, resumeReasonOverride, plannedBuild) {
         if (this.runState.hasLiveRun(threadId)) {
             this.logger.warn('Skipping internal follow-up: active run exists', { threadId });
             return '';
         }
+        const failedStreak = this.failedInternalFollowUpStreaks.get(threadId) ?? 0;
+        if (failedStreak >= MAX_CONSECUTIVE_FAILED_INTERNAL_FOLLOW_UPS) {
+            this.logger.warn('Skipping internal follow-up: consecutive follow-up runs keep failing', {
+                threadId,
+                failedStreak,
+                resumeReason: resumeReasonOverride,
+            });
+            return '';
+        }
         const { runId, abortController } = this.runState.startRun({
             threadId,
             user,
@@ -2538,6 +2560,7 @@ let InstanceAiService = class InstanceAiService {
                     this.liveness.consumeRunTimeout(runId);
                 }
             }
+            this.updateInternalFollowUpFailureStreak(threadId, messageTraceFinalization?.status, resumeReason !== undefined);
             if (!this.runState.hasSuspendedRun(threadId)) {
                 if (checkpoint?.isCheckpointFollowUp) {
                     await this.finalizeCheckpointFollowUp(user, threadId, checkpoint.checkpointTaskId);
@@ -3281,6 +3304,7 @@ let InstanceAiService = class InstanceAiService {
                     this.liveness.consumeRunTimeout(opts.runId);
                 }
             }
+            this.updateInternalFollowUpFailureStreak(opts.threadId, messageTraceFinalization?.status, false);
             if (!this.runState.hasSuspendedRun(opts.threadId)) {
                 if (opts.checkpoint?.isCheckpointFollowUp) {
                     await this.finalizeCheckpointFollowUp(opts.user, opts.threadId, opts.checkpoint.checkpointTaskId);