fix(core): resolve TypeScript errors in multi-step workflow

- Fix parseInt type errors by casting getSetting result to string
- Add bounds checking for retry counts (1-10 range)
- Add exponential backoff for retries (1s, 2s, 4s... capped at 8s)
- Fix null check for summary.thought access

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: wtfsayo

packages/core/src/services/default-message-service.ts

@@ -1107,11 +1107,14 @@ export class DefaultMessageService implements IMessageService {
           runtime.character.templates?.multiStepDecisionTemplate || multiStepDecisionTemplate,
       });
 
-      // Retry logic for parsing failures
-      const maxParseRetries = parseInt(
-        runtime.getSetting('MULTISTEP_PARSE_RETRIES') || '5',
+      // Retry logic for parsing failures with bounds checking
+      const parseRetriesSetting = runtime.getSetting('MULTISTEP_PARSE_RETRIES');
+      const rawParseRetries = parseInt(
+        String(parseRetriesSetting ?? '5'),
         10
       );
+      // Validate retry count is within reasonable bounds (1-10)
+      const maxParseRetries = Math.max(1, Math.min(10, isNaN(rawParseRetries) ? 5 : rawParseRetries));
       let stepResultRaw: string = '';
       let parsedStep: Record<string, unknown> | null = null;
 
@@ -1148,7 +1151,9 @@ export class DefaultMessageService implements IMessageService {
             );
 
             if (parseAttempt < maxParseRetries) {
-              await new Promise((resolve) => setTimeout(resolve, 1000));
+              // Exponential backoff: 1s, 2s, 4s, etc. (capped at 8s)
+              const backoffMs = Math.min(1000 * Math.pow(2, parseAttempt - 1), 8000);
+              await new Promise((resolve) => setTimeout(resolve, backoffMs));
             }
           }
         } catch (error) {
@@ -1165,7 +1170,9 @@ export class DefaultMessageService implements IMessageService {
           if (parseAttempt >= maxParseRetries) {
             throw error;
           }
-          await new Promise((resolve) => setTimeout(resolve, 1000));
+          // Exponential backoff on error
+          const backoffMs = Math.min(1000 * Math.pow(2, parseAttempt - 1), 8000);
+          await new Promise((resolve) => setTimeout(resolve, backoffMs));
         }
       }
 
@@ -1386,11 +1393,14 @@ export class DefaultMessageService implements IMessageService {
       template: runtime.character.templates?.multiStepSummaryTemplate || multiStepSummaryTemplate,
     });
 
-    // Retry logic for summary parsing failures
-    const maxSummaryRetries = parseInt(
-      runtime.getSetting('MULTISTEP_SUMMARY_PARSE_RETRIES') || '5',
+    // Retry logic for summary parsing failures with bounds checking
+    const summaryRetriesSetting = runtime.getSetting('MULTISTEP_SUMMARY_PARSE_RETRIES');
+    const rawSummaryRetries = parseInt(
+      String(summaryRetriesSetting ?? '5'),
       10
     );
+    // Validate retry count is within reasonable bounds (1-10)
+    const maxSummaryRetries = Math.max(1, Math.min(10, isNaN(rawSummaryRetries) ? 5 : rawSummaryRetries));
     let finalOutput: string = '';
     let summary: Record<string, unknown> | null = null;
 
@@ -1427,7 +1437,9 @@ export class DefaultMessageService implements IMessageService {
           );
 
           if (summaryAttempt < maxSummaryRetries) {
-            await new Promise((resolve) => setTimeout(resolve, 1000));
+            // Exponential backoff: 1s, 2s, 4s, etc. (capped at 8s)
+            const backoffMs = Math.min(1000 * Math.pow(2, summaryAttempt - 1), 8000);
+            await new Promise((resolve) => setTimeout(resolve, backoffMs));
           }
         }
       } catch (error) {
@@ -1447,19 +1459,22 @@ export class DefaultMessageService implements IMessageService {
           );
           break;
         }
-        await new Promise((resolve) => setTimeout(resolve, 1000));
+        // Exponential backoff on error
+        const backoffMs = Math.min(1000 * Math.pow(2, summaryAttempt - 1), 8000);
+        await new Promise((resolve) => setTimeout(resolve, backoffMs));
       }
     }
 
     let responseContent: Content | null = null;
     const summaryText = summary?.text;
     if (typeof summaryText === 'string' && summaryText) {
+      const summaryThought = summary?.thought;
       responseContent = {
         actions: ['MULTI_STEP_SUMMARY'],
         text: summaryText,
         thought:
-          (typeof summary.thought === 'string'
-            ? summary.thought
+          (typeof summaryThought === 'string'
+            ? summaryThought
             : 'Final user-facing message after task completion.') ||
           'Final user-facing message after task completion.',
         simple: true,