@onereach/step-voice 7.0.26-agenttimeout.4 → 7.0.26-agenttimeout.6

package/dst/Voice Agent.js

@@ -43,7 +43,6 @@ class VoiceAgent extends voice_1.default {
         });
         this.triggers.otherwise(async () => {
             const { handlers = [], system_instruction = '', developer_messages = [], greeting_message = '', prevent_greeting_interruption = false, idle_detection_enabled = false, idle_timeout_secs = 8, idle_max_retries = 3, idle_prompts = [], idle_goodbye_message = '', custom_config_enabled = false, agent_url_overwritten = false, tool_feedback, soft_timeout, agent_url = '', mode = '', stt = {}, llm = {}, tts = {}, realtime = {} } = this.data;
-            this.log.warn('this.data updated', this.data);
             if (lodash_1.default.isEmpty(system_instruction)) {
                 this.throwError(new Error('Voice Agent: system instruction is required'));
             }
@@ -112,7 +111,7 @@ class VoiceAgent extends voice_1.default {
                         this.throwError(new Error(`Voice Agent: unknown mode "${mode}"`));
                 }
             }
-            if (this.isStepFlagEnabled(tool_feedback?.enabled)) {
+            if (this.isStepFlagEnabled(tool_feedback?.enabled) && tools.length > 0) {
                 const validatedFeedback = this.validateToolFeedback(tool_feedback);
                 if (validatedFeedback) {
                     config.tool_feedback = validatedFeedback;
@@ -143,48 +142,50 @@ class VoiceAgent extends voice_1.default {
     }
     validateToolFeedback(config) {
         if (config == null || typeof config !== 'object')
-            return undefined;
+            return null;
         const cfg = config;
         // If enabled is not boolean true, silently return undefined (disabled case)
         if (typeof cfg.enabled !== 'boolean' || cfg.enabled !== true)
-            return undefined;
+            return null;
         // Validate pre_tool_speech
         const validPreToolSpeech = ['auto', 'static', 'off'];
         if (!validPreToolSpeech.includes(cfg.pre_tool_speech)) {
             this.log.warn('Voice Agent: invalid tool_feedback.pre_tool_speech — feedback config excluded');
-            return undefined;
+            return null;
         }
         // Validate tool_call_sound
         const validToolCallSound = ['typing', 'none'];
         if (!validToolCallSound.includes(cfg.tool_call_sound)) {
             this.log.warn('Voice Agent: invalid tool_feedback.tool_call_sound — feedback config excluded');
-            return undefined;
+            return null;
         }
         // Validate tool_call_sound_behavior
         const validToolCallSoundBehavior = ['always', 'with_pre_speech'];
         if (!validToolCallSoundBehavior.includes(cfg.tool_call_sound_behavior)) {
             this.log.warn('Voice Agent: invalid tool_feedback.tool_call_sound_behavior — feedback config excluded');
-            return undefined;
+            return null;
         }
         // Validate delay_seconds
-        const delay = cfg.delay_seconds;
-        if (typeof delay !== 'number' || !Number.isFinite(delay) || delay <= 0 || delay > 30) {
+        const delay = Number(cfg.delay_seconds);
+        if (delay <= 0 || delay > 30) {
             this.log.warn('Voice Agent: invalid tool_feedback.delay_seconds — feedback config excluded');
-            return undefined;
+            return null;
         }
-        // Validate pre_tool_speech_phrases
+        // Validate pre_tool_speech_phrases (only required when mode is 'static')
         const phrases = cfg.pre_tool_speech_phrases;
-        if (!Array.isArray(phrases) ||
-            phrases.length < 1 ||
-            phrases.length > 20 ||
-            !phrases.every((p) => typeof p === 'string' && p.trim().length > 0)) {
-            this.log.warn('Voice Agent: invalid tool_feedback.pre_tool_speech_phrases — feedback config excluded');
-            return undefined;
+        if (cfg.pre_tool_speech === 'static') {
+            if (!Array.isArray(phrases) ||
+                phrases.length < 1 ||
+                phrases.length > 20 ||
+                !phrases.every((p) => typeof p === 'string' && p.trim().length > 0)) {
+                this.log.warn('Voice Agent: invalid tool_feedback.pre_tool_speech_phrases — feedback config excluded');
+                return null;
+            }
         }
         return {
             enabled: true,
             pre_tool_speech: cfg.pre_tool_speech,
-            pre_tool_speech_phrases: phrases,
+            pre_tool_speech_phrases: (Array.isArray(phrases) ? phrases : []),
             tool_call_sound: cfg.tool_call_sound,
             tool_call_sound_behavior: cfg.tool_call_sound_behavior,
             delay_seconds: delay
@@ -217,7 +218,7 @@ class VoiceAgent extends voice_1.default {
         const result = {};
         if ('delay_seconds' in raw && raw.delay_seconds != null) {
             const delay = Number(raw.delay_seconds);
-            if (!Number.isFinite(delay) || delay <= 0 || delay > 30) {
+            if (delay <= 0 || delay > 30) {
                 this.log.warn(`Voice Agent: invalid feedback.delay_seconds for tool "${toolName}" — per-tool feedback excluded`);
                 return null;
             }
@@ -253,8 +254,8 @@ class VoiceAgent extends voice_1.default {
         if (config == null || typeof config !== 'object' || Array.isArray(config))
             return null;
         const cfg = config;
-        const timeoutSeconds = cfg.timeout_seconds;
-        if (typeof timeoutSeconds !== 'number' || !Number.isFinite(timeoutSeconds) || timeoutSeconds <= 0 || timeoutSeconds > 30) {
+        const timeoutSeconds = Number(cfg.timeout_seconds);
+        if (timeoutSeconds <= 0 || timeoutSeconds > 30) {
             this.log.warn('Voice Agent: invalid soft_timeout.timeout_seconds — soft timeout config excluded');
             return null;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onereach/step-voice",
-  "version": "7.0.26-agenttimeout.4",
+  "version": "7.0.26-agenttimeout.6",
   "author": "Roman Zolotarov <roman.zolotarov@onereach.com>",
   "contributors": [
     "Roman Zolotarov",