rasa-pro 3.14.0.dev6__py3-none-any.whl → 3.14.0.dev8__py3-none-any.whl

rasa/dialogue_understanding/commands/cancel_flow_command.py

@@ -2,7 +2,7 @@ from __future__ import annotations
 
 import re
 from dataclasses import dataclass
-from typing import Any, Dict, List, Optional, Tuple
+from typing import Any, Dict, List
 
 import structlog
 
@@ -16,7 +16,6 @@ from rasa.dialogue_understanding.commands.command_syntax_manager import (
 )
 from rasa.dialogue_understanding.patterns.cancel import CancelPatternFlowStackFrame
 from rasa.dialogue_understanding.stack.dialogue_stack import DialogueStack
-from rasa.dialogue_understanding.stack.frames import DialogueStackFrame
 from rasa.dialogue_understanding.stack.frames.flow_stack_frame import (
     FlowStackFrameType,
     UserFlowStackFrame,
@@ -74,8 +73,7 @@ class CancelFlowCommand(Command):
             # we should never get here as we should always find the user flow
             # that was canceled.
             raise ValueError(
-                f"Could not find a user flow frame to cancel. "
-                f"Current stack: {stack}."
+                f"Could not find a user flow frame to cancel. Current stack: {stack}."
             )
 
     def run_command_on_tracker(
@@ -175,80 +173,3 @@ class CancelFlowCommand(Command):
             CommandSyntaxManager.get_syntax_version(),
             mapper[CommandSyntaxManager.get_default_syntax_version()],
         )
-
-    def cancel_flow(
-        self,
-        tracker: DialogueStateTracker,
-        stack: DialogueStack,
-        flow_id: str,
-    ) -> List[Event]:
-        """Cancels a flow by flow id."""
-        applied_events: List[Event] = []
-
-        frames_to_cancel, user_frame_to_cancel = self._collect_frames_to_cancel(
-            stack, flow_id
-        )
-
-        # if the flow is not on the stack, do nothing
-        if user_frame_to_cancel is None:
-            structlogger.error(
-                "cancel_flow_command.cancel_flow.no_user_frame_to_cancel",
-                command=self,
-            )
-            return []
-
-        frames_ids_to_cancel = [frame.frame_id for frame in frames_to_cancel]
-
-        stack.push(
-            CancelPatternFlowStackFrame(
-                canceled_name=flow_id,
-                canceled_frames=frames_ids_to_cancel,
-            )
-        )
-
-        # create flow cancelled event
-        applied_events.extend(
-            [
-                FlowCancelled(
-                    user_frame_to_cancel.flow_id, user_frame_to_cancel.step_id
-                ),
-            ]
-        )
-
-        update_stack_events = tracker.create_stack_updated_events(stack)
-
-        return applied_events + update_stack_events
-
-    def _collect_frames_to_cancel(
-        self, stack: DialogueStack, target_flow_id: str
-    ) -> Tuple[List[DialogueStackFrame], Optional[UserFlowStackFrame]]:
-        """Collect frames that need to be cancelled.
-
-        Args:
-            stack: The stack to collect frames from.
-            target_flow_id: The ID of the flow to cancel.
-
-        Returns:
-            A tuple containing (frames_to_cancel, frame_to_cancel).
-        """
-        frames_to_cancel: List[DialogueStackFrame] = []
-        frame_found = False
-        frame_to_cancel = None
-
-        for frame in stack.frames:
-            if isinstance(frame, UserFlowStackFrame) and (
-                frame.frame_type == FlowStackFrameType.REGULAR
-                or frame.frame_type == FlowStackFrameType.INTERRUPT
-            ):
-                if frame.flow_id == target_flow_id:
-                    frames_to_cancel.append(frame)
-                    frame_to_cancel = frame
-                    frame_found = True
-                    continue
-                elif frame_found:
-                    break
-
-            if frame_found:
-                frames_to_cancel.append(frame)
-
-        return list(frames_to_cancel), frame_to_cancel

rasa/dialogue_understanding/commands/start_flow_command.py

@@ -2,7 +2,7 @@ from __future__ import annotations
 
 import re
 from dataclasses import dataclass
-from typing import Any, Dict, List, Optional, Tuple
+from typing import Any, Dict, List, Optional
 
 import structlog
 
@@ -11,17 +11,15 @@ from rasa.dialogue_understanding.commands.command_syntax_manager import (
     CommandSyntaxManager,
     CommandSyntaxVersion,
 )
-from rasa.dialogue_understanding.stack.dialogue_stack import DialogueStack
-from rasa.dialogue_understanding.stack.frames.dialogue_stack_frame import (
-    DialogueStackFrame,
+from rasa.dialogue_understanding.commands.utils import (
+    remove_pattern_continue_interrupted_frames,
+    resume_flow,
 )
 from rasa.dialogue_understanding.stack.frames.flow_stack_frame import (
-    AgentStackFrame,
     AgentState,
     FlowStackFrameType,
     UserFlowStackFrame,
 )
-from rasa.dialogue_understanding.stack.frames.pattern_frame import PatternFlowStackFrame
 from rasa.dialogue_understanding.stack.utils import (
     is_continue_interrupted_flow_active,
     top_user_flow_frame,
@@ -29,10 +27,8 @@ from rasa.dialogue_understanding.stack.utils import (
 )
 from rasa.shared.core.events import (
     AgentInterrupted,
-    AgentResumed,
     Event,
     FlowInterrupted,
-    FlowResumed,
 )
 from rasa.shared.core.flows import FlowsList
 from rasa.shared.core.trackers import DialogueStateTracker
@@ -108,7 +104,7 @@ class StartFlowCommand(Command):
             # predicted a start flow command for the flow which is on top of the stack,
             # we just need to remove the pattern_continue_interrupted frame(s) from the
             # stack
-            stack = _remove_pattern_continue_interrupted_frames(stack)
+            stack = remove_pattern_continue_interrupted_frames(stack)
             return applied_events + tracker.create_stack_updated_events(stack)
 
         # if the flow is already on the stack, resume it
@@ -118,11 +114,22 @@ class StartFlowCommand(Command):
         ):
             # if pattern_continue_interrupted is active, we need to remove it
             # from the stack before resuming the flow
-            stack = _remove_pattern_continue_interrupted_frames(stack)
-            return self.resume_flow(tracker, stack, original_user_frame)
+            stack = remove_pattern_continue_interrupted_frames(stack)
+            applied_events.extend(resume_flow(self.flow, tracker, stack))
+            # the current active flow is interrupted
+            applied_events.append(
+                FlowInterrupted(
+                    original_user_frame.flow_id, original_user_frame.step_id
+                )
+            )
+            return applied_events
 
         frame_type = FlowStackFrameType.REGULAR
 
+        # remove the pattern_continue_interrupted frames from the stack
+        # if it is currently active but the user digressed from the pattern
+        stack = remove_pattern_continue_interrupted_frames(stack)
+
         if original_top_flow:
             # if the original top flow is not the same as the flow to start,
             # interrupt the current active flow
@@ -198,105 +205,3 @@ class StartFlowCommand(Command):
             CommandSyntaxManager.get_syntax_version(),
             mapper[CommandSyntaxManager.get_default_syntax_version()],
         )
-
-    def resume_flow(
-        self,
-        tracker: DialogueStateTracker,
-        stack: DialogueStack,
-        original_user_frame: UserFlowStackFrame,
-    ) -> List[Event]:
-        """Resumes a flow by reordering frames."""
-        applied_events: List[Event] = []
-
-        # Resume existing flow by reordering frames
-        frames_to_resume, user_frame_to_resume = self._collect_frames_to_resume(
-            stack, self.flow
-        )
-
-        # if the flow is not on the stack, do nothing
-        # this should not happen, but just in case
-        if user_frame_to_resume is None:
-            structlogger.error(
-                "start_flow_command.resume_flow.no_user_frame_to_resume",
-                command=self,
-            )
-            return []
-
-        # move the frames to the top of the stack, e.g. reorder the frames
-        # on the stack
-        stack.move_frames_to_top(frames_to_resume)
-        agent_stack_frame = next(
-            (frame for frame in frames_to_resume if isinstance(frame, AgentStackFrame)),
-            None,
-        )
-        if agent_stack_frame:
-            agent_id = agent_stack_frame.agent_id
-            applied_events.append(AgentResumed(agent_id, agent_stack_frame.flow_id))
-
-        # Create flow interruption and resumption events
-        applied_events.extend(
-            [
-                # the current active flow is interrupted
-                FlowInterrupted(
-                    original_user_frame.flow_id, original_user_frame.step_id
-                ),
-                # the flow, which was on the stack, is resumed
-                FlowResumed(user_frame_to_resume.flow_id, user_frame_to_resume.step_id),
-            ]
-        )
-
-        return applied_events + tracker.create_stack_updated_events(stack)
-
-    def _collect_frames_to_resume(
-        self, stack: DialogueStack, target_flow_id: str
-    ) -> Tuple[List[DialogueStackFrame], Optional[UserFlowStackFrame]]:
-        """Collect frames that need to be resumed for the target flow.
-
-        Args:
-            stack: The stack to collect frames from.
-            target_flow_id: The ID of the flow to resume.
-
-        Returns:
-            A tuple containing (frames_to_resume, frame_to_resume).
-        """
-        frames_to_resume: List[DialogueStackFrame] = []
-        frame_found = False
-        frame_to_resume = None
-
-        for frame in stack.frames:
-            if isinstance(frame, UserFlowStackFrame) and (
-                frame.frame_type == FlowStackFrameType.REGULAR
-                or frame.frame_type == FlowStackFrameType.INTERRUPT
-            ):
-                if frame.flow_id == target_flow_id:
-                    frames_to_resume.append(frame)
-                    frame_to_resume = frame
-                    frame_found = True
-                    continue
-                elif frame_found:
-                    break
-
-            if frame_found:
-                frames_to_resume.append(frame)
-
-        return list(frames_to_resume), frame_to_resume
-
-
-def _remove_pattern_continue_interrupted_frames(stack: DialogueStack) -> DialogueStack:
-    """Remove pattern_continue_interrupted frames from the stack."""
-    if not is_continue_interrupted_flow_active(stack):
-        return stack
-
-    # remove pattern_continue_interrupted from the stack
-    top_frame = stack.top()
-    while isinstance(top_frame, PatternFlowStackFrame):
-        # If the top frame is a pattern frame, we need to remove it
-        # before continuing with the active user flow frame.
-        # This prevents the pattern frame
-        # from being left on the stack when the flow is started
-        # which would prevent pattern_completed to be triggered
-        # once the user flow is completed.
-        stack.pop()
-        top_frame = stack.top()
-
-    return stack

rasa/dialogue_understanding/commands/utils.py

@@ -5,9 +5,21 @@ import structlog
 from rasa.dialogue_understanding.patterns.validate_slot import (
     ValidateSlotPatternFlowStackFrame,
 )
+from rasa.dialogue_understanding.stack.dialogue_stack import DialogueStack
+from rasa.dialogue_understanding.stack.frames.dialogue_stack_frame import (
+    DialogueStackFrame,
+)
+from rasa.dialogue_understanding.stack.frames.flow_stack_frame import (
+    AgentStackFrame,
+    FlowStackFrameType,
+    UserFlowStackFrame,
+)
+from rasa.dialogue_understanding.stack.frames.pattern_frame import PatternFlowStackFrame
 from rasa.shared.constants import ACTION_ASK_PREFIX, UTTER_ASK_PREFIX
 from rasa.shared.core.events import (
+    AgentResumed,
     Event,
+    FlowResumed,
     SlotSet,
 )
 from rasa.shared.core.flows import FlowsList
@@ -154,3 +166,109 @@ def find_default_flows_collecting_slot(
             for step in flow.get_collect_steps()
         )
     ]
+
+
+def resume_flow(
+    flow_to_resume: str,
+    tracker: DialogueStateTracker,
+    stack: DialogueStack,
+) -> List[Event]:
+    """Resumes a flow by reordering frames."""
+    applied_events: List[Event] = []
+
+    # Resume existing flow by reordering frames
+    frames_to_resume, user_frame_to_resume = collect_frames_to_resume(
+        stack, flow_to_resume
+    )
+
+    # if the flow is not on the stack, do nothing
+    # this should not happen, but just in case
+    if user_frame_to_resume is None:
+        structlogger.error(
+            "resume_flow.no_user_frame_to_resume",
+            flow_to_resume=flow_to_resume,
+        )
+        return []
+
+    # move the frames to the top of the stack, e.g. reorder the frames
+    # on the stack
+    stack.move_frames_to_top(frames_to_resume)
+
+    # create agent resumed events if the agent frame is now on top of the stack
+    agent_stack_frame = next(
+        (frame for frame in frames_to_resume if isinstance(frame, AgentStackFrame)),
+        None,
+    )
+    if agent_stack_frame:
+        agent_id = agent_stack_frame.agent_id
+        applied_events.append(AgentResumed(agent_id, agent_stack_frame.flow_id))
+
+    # Create flow interruption and resumption events
+    applied_events.extend(
+        [
+            # the flow, which was on the stack, is resumed
+            FlowResumed(user_frame_to_resume.flow_id, user_frame_to_resume.step_id),
+        ]
+    )
+
+    return applied_events + tracker.create_stack_updated_events(stack)
+
+
+def collect_frames_to_resume(
+    stack: DialogueStack,
+    target_flow_id: str,  # pyright: ignore[reportUndefinedVariable]
+) -> Tuple[List[DialogueStackFrame], Optional[UserFlowStackFrame]]:
+    """Collect frames that need to be resumed for the target flow.
+
+    Args:
+        stack: The stack to collect frames from.
+        target_flow_id: The ID of the flow to resume.
+
+    Returns:
+        A tuple containing (frames_to_resume, frame_to_resume).
+    """
+    frames_to_resume: List[DialogueStackFrame] = []
+    frame_found = False
+    frame_to_resume = None
+
+    for frame in stack.frames:
+        if isinstance(frame, UserFlowStackFrame) and (
+            frame.frame_type == FlowStackFrameType.REGULAR
+            or frame.frame_type == FlowStackFrameType.INTERRUPT
+        ):
+            if frame.flow_id == target_flow_id:
+                frames_to_resume.append(frame)
+                frame_to_resume = frame
+                frame_found = True
+                continue
+            elif frame_found:
+                break
+
+        if frame_found:
+            frames_to_resume.append(frame)
+
+    return list(frames_to_resume), frame_to_resume
+
+
+def remove_pattern_continue_interrupted_frames(stack: DialogueStack) -> DialogueStack:
+    """Remove pattern_continue_interrupted frames from the stack."""
+    from rasa.dialogue_understanding.stack.utils import (
+        is_continue_interrupted_flow_active,
+    )
+
+    if not is_continue_interrupted_flow_active(stack):
+        return stack
+
+    # remove pattern_continue_interrupted from the stack
+    top_frame = stack.top()
+    while isinstance(top_frame, PatternFlowStackFrame):
+        # If the top frame is a pattern frame, we need to remove it
+        # before continuing with the active user flow frame.
+        # This prevents the pattern frame
+        # from being left on the stack when the flow is started
+        # which would prevent pattern_completed to be triggered
+        # once the user flow is completed.
+        stack.pop()
+        top_frame = stack.top()
+
+    return stack

rasa/dialogue_understanding/generator/flow_retrieval.py

@@ -219,11 +219,6 @@ class FlowRetrieval(EmbeddingsHealthCheckMixin):
         if self.vector_store is not None:
             with self._model_storage.write_to(self._resource) as model_path:
                 self.vector_store.save_local(model_path)
-        else:
-            structlogger.warning(
-                "flow_retrieval.persist_vector_store.not_initialized",
-                event_info="Vector store is None, not persisted.",
-            )
 
     def _persist_config(self) -> None:
         with self._model_storage.write_to(self._resource) as path:
@@ -249,6 +244,16 @@ class FlowRetrieval(EmbeddingsHealthCheckMixin):
         )
 
         flows_to_embedd = flows.exclude_link_only_flows()
+
+        if not flows_to_embedd:
+            structlogger.debug(
+                "flow_retrieval.populate_vector_store.no_flows_to_embed",
+                event_info=(
+                    "No flows to embed in the vector store, skipping population."
+                ),
+            )
+            return
+
         embeddings = self._create_embedder(self.config)
         documents = self._generate_flow_documents(flows_to_embedd, domain)
         try:
@@ -420,10 +425,6 @@ class FlowRetrieval(EmbeddingsHealthCheckMixin):
             The top k documents with similarity scores.
         """
         if self.vector_store is None:
-            structlogger.error(
-                "flow_retrieval.query_vector_store.vector_store_not_configured",
-                event_info="Vector store is not configured",
-            )
             return []
         try:
             documents_with_scores = (

rasa/dialogue_understanding/generator/prompt_templates/agent_command_prompt_v2_claude_3_5_sonnet_20240620_template.jinja2

@@ -5,15 +5,15 @@ Your task is to analyze the current conversation context and generate a list of
 
 ## Available Actions:
 * `start flow flow_name`: Starting a flow. For example, `start flow transfer_money` or `start flow list_contacts`.
-* `set slot slot_name slot_value`: Slot setting. For example, `set slot transfer_money_recipient Freddy`. Can be used to correct and change previously set values.
+* `set slot slot_name slot_value`: Slot setting. For example, `set slot transfer_money_recipient Freddy`. Can be used to correct and change previously set values. ONLY use slots that are explicitly defined in the flow's slot list.
 * `cancel flow`: Cancelling the current flow.
 * `disambiguate flows flow_name1 flow_name2 ... flow_name_n`: Disambiguate which flow should be started when user input is ambiguous by listing the potential flows as options. For example, `disambiguate flows list_contacts add_contact remove_contact ...` if the user just wrote "contacts".
 * `provide info`: Responding to the user's questions by supplying relevant information, such as answering FAQs or explaining services.
 * `offtopic reply`: Responding to casual or social user messages that are unrelated to any flows, engaging in friendly conversation and addressing off-topic remarks.
 * `hand over`: Handing over to a human, in case the user seems frustrated or explicitly asks to speak to one.
 * `repeat message`: Repeating the last bot message.
-{% if active_agent %} * `continue agent`: Continue the currently active agent {{ active_agent.name }}.{% endif %}
-{% if completed_agents %}* `restart agent agent_name`: Restart the agent with the given name, in case the user wants to change some answer to a previous question asked by the agent. For example, `restart agent car_research_agent` if the user chnaged his mind about the car he wants to buy.{% endif %}
+{% if active_agent %} * `continue agent`: Continue the currently active agent {{ active_agent.name }}. This has HIGHEST PRIORITY when an agent is active and the user is responding to agent questions.{% endif %}
+{% if completed_agents %}* `restart agent agent_name`: Restart the agent with the given name, in case the user wants to change some answer to a previous question asked by the agent. For example, `restart agent car_research_agent` if the user changed his mind about the car he wants to buy. ONLY use agents that are listed in the `completed_agents` section.{% endif %}
 
 --
 
@@ -23,13 +23,18 @@ Your task is to analyze the current conversation context and generate a list of
 * Set the boolean slots based on the user response. Map positive responses to `True`, and negative to `False`.
 * Always refer to the slot description to determine what information should be extracted and how it should be formatted.
 * For text slots, extract values exactly as provided by the user unless the slot description specifies otherwise. Preserve formatting and avoid rewording, truncation, or making assumptions.
+* ONLY use `set slot` with slots that are explicitly defined in the current flow's slot list. Do NOT create or assume slots that don't exist.
 * Only use information provided by the user.
 * Use clarification in ambiguous cases.
 * Multiple flows can be started. If a user wants to digress into a second flow, you do not need to cancel the current flow.
 * Do not cancel the flow unless the user explicitly requests it.
 * Strictly adhere to the provided action format.
+* ONLY use the exact actions listed above. Do NOT invent new actions like "respond <message>" or any other variations.
 * Focus on the last message and take it one step at a time.
-* Use the previous conversation steps only to aid understanding.
+* Use the previous conversation steps only to aid understanding.{% if active_agent %}
+* When an agent is active, ALWAYS prioritize `continue agent` over `provide info` or `offtopic reply` unless the user is clearly asking something unrelated to the agent's task.{% endif %}{% if completed_agents %}
+* ONLY use `restart agent` with agents that are listed in the `completed_agents` section. Do NOT restart non-existent agents.{% endif %}{% if active_agent or completed_agents %}
+* If you're unsure about agent names, refer to the structured data provided in the `Current State` section.{% endif %}
 
 --
 
@@ -44,7 +49,7 @@ Use the following structured data:
 ## Current State
 {% if current_flow != None %}Use the following structured data:
 ```json
-{"active_flow": "{{ current_flow }}","current_step": {"requested_slot": "{{ current_slot }}","requested_slot_description": {{ current_slot_description | to_json_escaped_string }}},"slots": [{% for slot in flow_slots %}{"name": "{{ slot.name }}","value": "{{ slot.value }}","type": "{{ slot.type }}"{{ "," if slot.description or slot.allowed_values else "" }}{% if slot.description %}"description": {{ slot.description | to_json_escaped_string }}{{ "," if slot.allowed_values else "" }}{% endif %}{% if slot.allowed_values %}"allowed_values": "{{ slot.allowed_values }}"{% endif %}}{% if not loop.last %},{% endif %}{% endfor %}]{% if active_agent or completed_agents %},{% endif %}{% if active_agent %}"active_agent": {"name": "{{ active_agent.name }}","description": "{{ active_agent.description }}"}{% if completed_agents %},{% endif %}{% endif %}{% if completed_agents %}"completed_agents": [{% for agent in completed_agents %}{"name": "{{ agent.name }}","description": "{{ agent.description }}"}{% if not loop.last %},{% endif %}{% endfor %}]{% endif %}}
+{"active_flow":{"name":"{{ current_flow }}","current_step":{"requested_slot":"{{ current_slot }}","requested_slot_description":{{ current_slot_description | to_json_escaped_string }}},"slots":[{% for slot in flow_slots %}{"name":"{{ slot.name }}","value":"{{ slot.value }}","type":"{{ slot.type }}"{% if slot.description %},"description":{{ slot.description | to_json_escaped_string }}{% endif %}{% if slot.allowed_values %},"allowed_values":"{{ slot.allowed_values }}"{% endif %}}{% if not loop.last %},{% endif %}{% endfor %}]}{% if active_agent %},"active_agent":{"name":"{{ active_agent.name }}","description":{{ active_agent.description | to_json_escaped_string }}}{% endif %}{% if completed_agents %},"completed_agents":[{% for agent in completed_agents %}{"name":"{{ agent.name }}","description":{{ agent.description | to_json_escaped_string }}}{% if not loop.last %},{% endif %}{% endfor %}]{% endif %}}
 ```{% else %}
 You are currently not inside any flow.{% endif %}
 

rasa/dialogue_understanding/generator/prompt_templates/agent_command_prompt_v2_gpt_4o_2024_11_20_template.jinja2

@@ -13,15 +13,15 @@ Use the following structured data:
 
 ## Available Actions:
 * `start flow flow_name`: Starting a flow. For example, `start flow transfer_money` or `start flow list_contacts`.
-* `set slot slot_name slot_value`: Slot setting. For example, `set slot transfer_money_recipient Freddy`. Can be used to correct and change previously set values.
+* `set slot slot_name slot_value`: Slot setting. For example, `set slot transfer_money_recipient Freddy`. Can be used to correct and change previously set values. ONLY use slots that are explicitly defined in the flow's slot list.
 * `cancel flow`: Cancelling the current flow.
 * `disambiguate flows flow_name1 flow_name2 ... flow_name_n`: Disambiguate which flow should be started when user input is ambiguous by listing the potential flows as options. For example, `disambiguate flows list_contacts add_contact remove_contact ...` if the user just wrote "contacts".
 * `provide info`: Responding to the user's questions by supplying relevant information, such as answering FAQs or explaining services.
 * `offtopic reply`: Responding to casual or social user messages that are unrelated to any flows, engaging in friendly conversation and addressing off-topic remarks.
 * `hand over`: Handing over to a human, in case the user seems frustrated or explicitly asks to speak to one.
 * `repeat message`: Repeating the last bot message.
-{% if active_agent %} * `continue agent`: Continue the currently active agent {{ active_agent.name }}.{% endif %}
-{% if completed_agents %}* `restart agent agent_name`: Restart the agent with the given name, in case the user wants to change some answer to a previous question asked by the agent. For example, `restart agent car_research_agent` if the user chnaged his mind about the car he wants to buy.{% endif %}
+{% if active_agent %} * `continue agent`: Continue the currently active agent {{ active_agent.name }}. This has HIGHEST PRIORITY when an agent is active and the user is responding to agent questions.{% endif %}
+{% if completed_agents %}* `restart agent agent_name`: Restart the agent with the given name, in case the user wants to change some answer to a previous question asked by the agent. For example, `restart agent car_research_agent` if the user changed his mind about the car he wants to buy. ONLY use agents that are listed in the `completed_agents` section.{% endif %}
 
 ---
 
@@ -30,13 +30,18 @@ Use the following structured data:
 * For categorical slots try to match the user message with allowed slot values. Use "other" if you cannot match it.
 * Set the boolean slots based on the user response. Map positive responses to `True`, and negative to `False`.
 * Extract text slot values exactly as provided by the user. Avoid assumptions, format changes, or partial extractions.
+* ONLY use `set slot` with slots that are explicitly defined in the current flow's slot list. Do NOT create or assume slots that don't exist.
 * Only use information provided by the user.
 * Use clarification in ambiguous cases.
 * Multiple flows can be started. If a user wants to digress into a second flow, you do not need to cancel the current flow.
 * Do not cancel the flow unless the user explicitly requests it.
 * Strictly adhere to the provided action format.
+* ONLY use the exact actions listed above. Do NOT invent new actions like "respond <message>" or any other variations.
 * Focus on the last message and take it one step at a time.
-* Use the previous conversation steps only to aid understanding.
+* Use the previous conversation steps only to aid understanding.{% if active_agent %}
+* When an agent is active, ALWAYS prioritize `continue agent` over `provide info` or `offtopic reply` unless the user is clearly asking something unrelated to the agent's task.{% endif %}{% if completed_agents %}
+* ONLY use `restart agent` with agents that are listed in the `completed_agents` section. Do NOT restart non-existent agents.{% endif %}{% if active_agent or completed_agents %}
+* If you're unsure about agent names, refer to the structured data provided in the `Current State` section.{% endif %}
 
 ---
 
@@ -44,7 +49,7 @@ Use the following structured data:
 {% if current_flow != None %}
 Use the following structured data:
 ```json
-{"active_flow": "{{ current_flow }}","current_step": {"requested_slot": "{{ current_slot }}","requested_slot_description": {{ current_slot_description | to_json_escaped_string }}},"slots": [{% for slot in flow_slots %}{"name": "{{ slot.name }}","value": "{{ slot.value }}","type": "{{ slot.type }}"{{ "," if slot.description or slot.allowed_values else "" }}{% if slot.description %}"description": {{ slot.description | to_json_escaped_string }}{{ "," if slot.allowed_values else "" }}{% endif %}{% if slot.allowed_values %}"allowed_values": "{{ slot.allowed_values }}"{% endif %}}{% if not loop.last %},{% endif %}{% endfor %}]{% if active_agent or completed_agents %},{% endif %}{% if active_agent %}"active_agent": {"name": "{{ active_agent.name }}","description": "{{ active_agent.description }}"}{% if completed_agents %},{% endif %}{% endif %}{% if completed_agents %}"completed_agents": [{% for agent in completed_agents %}{"name": "{{ agent.name }}","description": "{{ agent.description }}"}{% if not loop.last %},{% endif %}{% endfor %}]{% endif %}}
+{"active_flow":{"name":"{{ current_flow }}","current_step":{"requested_slot":"{{ current_slot }}","requested_slot_description":{{ current_slot_description | to_json_escaped_string }}},"slots":[{% for slot in flow_slots %}{"name":"{{ slot.name }}","value":"{{ slot.value }}","type":"{{ slot.type }}"{% if slot.description %},"description":{{ slot.description | to_json_escaped_string }}{% endif %}{% if slot.allowed_values %},"allowed_values":"{{ slot.allowed_values }}"{% endif %}}{% if not loop.last %},{% endif %}{% endfor %}]}{% if active_agent %},"active_agent":{"name":"{{ active_agent.name }}","description":{{ active_agent.description | to_json_escaped_string }}}{% endif %}{% if completed_agents %},"completed_agents":[{% for agent in completed_agents %}{"name":"{{ agent.name }}","description":{{ agent.description | to_json_escaped_string }}}{% if not loop.last %},{% endif %}{% endfor %}]{% endif %}}
 ```{% else %}
 You are currently not inside any flow.{% endif %}
 

rasa/dialogue_understanding/generator/prompt_templates/agent_command_prompt_v3_claude_3_5_sonnet_20240620_template.jinja2

@@ -5,13 +5,13 @@ Your task is to analyze the current conversation context and generate a list of
 
 ## Available Actions:
 * `start flow flow_name`: Start a flow. For example, `start flow transfer_money` or `start flow list_contacts`.
-* `set slot slot_name slot_value`: Set a slot for the active flow. For example, `set slot transfer_money_recipient Freddy`. Can be used to correct and change previously set values.
+* `set slot slot_name slot_value`: Set a slot for the active flow. For example, `set slot transfer_money_recipient Freddy`. Can be used to correct and change previously set values. ONLY use slots that are explicitly defined in the flow's slot list.
 * `disambiguate flows flow_name1 flow_name2 ... flow_name_n`: When a message could refer to multiple flows, list the possible flows as options to clarify. Example: `disambiguate flows list_contacts add_contact remove_contact`.
 * `search and reply`: Provide a response from the knowledge base to address the user's inquiry when no flows fit, including domain knowledge, FAQs, and all off-topic or social messages.
 * `cancel flow`: Cancel the current flow if the user requests it.
 * `repeat message`: Repeat the last bot message.
-{% if active_agent %} * `continue agent`: Continue the currently active agent {{ active_agent.name }}.{% endif %}
-{% if completed_agents %}* `restart agent agent_name`: Restart the agent with the given name, in case the user wants to change some answer to a previous question asked by the agent. For example, `restart agent car_research_agent` if the user chnaged his mind about the car he wants to buy.{% endif %}
+{% if active_agent %} * `continue agent`: Continue the currently active agent {{ active_agent.name }}. This has HIGHEST PRIORITY when an agent is active and the user is responding to agent questions.{% endif %}
+{% if completed_agents %}* `restart agent agent_name`: Restart the agent with the given name, in case the user wants to change some answer to a previous question asked by the agent. For example, `restart agent car_research_agent` if the user changed his mind about the car he wants to buy. ONLY use agents that are listed in the `completed_agents` section.{% endif %}
 
 ---
 
@@ -24,6 +24,7 @@ Your task is to analyze the current conversation context and generate a list of
 * For categorical slots, try to match the user message with allowed slot values. Use "other" if you cannot match it.
 * Set the boolean slots based on the user's response. Map positive responses to `True`, and negative to `False`.
 * Extract text slot values exactly as provided by the user. Avoid assumptions, format changes, or partial extractions.
+* ONLY use `set slot` with slots that are explicitly defined in the current flow's slot list. Do NOT create or assume slots that don't exist.
 ### Disambiguate Flows
 * Use `disambiguate flows` when the user's message matches multiple flows and you cannot decide which flow is most appropriate.
 * If the user message is short and not precise enough to start a flow or `search and reply`, disambiguate.
@@ -34,22 +35,29 @@ Your task is to analyze the current conversation context and generate a list of
 * Flow Priority: If you are unsure between starting a flow or `search and reply`, always prioritize starting a flow.
 ### Cancel Flow
 * Do not cancel any flow unless the user explicitly requests it.
-* Multiple flows can be started without cancelling the previous, if the user wants to pursue multiple processes.
-### General Tips
+* Multiple flows can be started without cancelling the previous, if the user wants to pursue multiple processes.{% if active_agent or completed_agents %}
+### Agents{% if active_agent %}
+* When an agent is active, ALWAYS prioritize `continue agent` over `search and reply` unless the user is clearly asking something unrelated to the agent's task.{% endif %}{% if completed_agents %}
+* ONLY use `restart agent` with agents that are listed in the `completed_agents` section. Do NOT restart non-existent agents.{% endif %}
+* If you're unsure about agent names, refer to the structured data provided in the `Current State` section.
+{% endif %}### General Tips
 * Only use information provided by the user.
 * Strictly adhere to the provided action format.
+* ONLY use the exact actions listed above. Do NOT invent new actions like "respond <message>" or any other variations.
 * Focus on the last message and take it one step at a time.
 * Use the previous conversation steps only to aid understanding.
 
+
 ---
 
 ## Decision Rule Table
-| Condition                                             | Action             |
-|-------------------------------------------------------|--------------------|
-| Flow perfectly matches user's message                 | start flow         |
-| Multiple flows are equally strong, relevant matches   | disambiguate flows |
-| User's message is unclear or imprecise                | disambiguate flows |
-| No flow fits at all, but knowledge base may help      | search and reply   |
+| Condition                                                     | Action             |
+|---------------------------------------------------------------|--------------------|{% if active_agent %}
+| Agent is active and the user is responding to agent questions | continue agent     |{% endif %}
+| Flow perfectly matches user's message                         | start flow         |
+| Multiple flows are equally strong, relevant matches           | disambiguate flows |
+| User's message is unclear or imprecise                        | disambiguate flows |
+| No flow fits at all, but knowledge base may help              | search and reply   |
 
 ---
 
@@ -64,7 +72,7 @@ Use the following structured data:
 ## Current State
 {% if current_flow != None %}Use the following structured data:
 ```json
-{"active_flow": "{{ current_flow }}","current_step": {"requested_slot": "{{ current_slot }}","requested_slot_description": {{ current_slot_description | to_json_escaped_string }}},"slots": [{% for slot in flow_slots %}{"name": "{{ slot.name }}","value": "{{ slot.value }}","type": "{{ slot.type }}"{{ "," if slot.description or slot.allowed_values else "" }}{% if slot.description %}"description": {{ slot.description | to_json_escaped_string }}{{ "," if slot.allowed_values else "" }}{% endif %}{% if slot.allowed_values %}"allowed_values": "{{ slot.allowed_values }}"{% endif %}}{% if not loop.last %},{% endif %}{% endfor %}]{% if active_agent or completed_agents %},{% endif %}{% if active_agent %}"active_agent": {"name": "{{ active_agent.name }}","description": "{{ active_agent.description }}"}{% if completed_agents %},{% endif %}{% endif %}{% if completed_agents %}"completed_agents": [{% for agent in completed_agents %}{"name": "{{ agent.name }}","description": "{{ agent.description }}"}{% if not loop.last %},{% endif %}{% endfor %}]{% endif %}}
+{"active_flow":{"name":"{{ current_flow }}","current_step":{"requested_slot":"{{ current_slot }}","requested_slot_description":{{ current_slot_description | to_json_escaped_string }}},"slots":[{% for slot in flow_slots %}{"name":"{{ slot.name }}","value":"{{ slot.value }}","type":"{{ slot.type }}"{% if slot.description %},"description":{{ slot.description | to_json_escaped_string }}{% endif %}{% if slot.allowed_values %},"allowed_values":"{{ slot.allowed_values }}"{% endif %}}{% if not loop.last %},{% endif %}{% endfor %}]}{% if active_agent %},"active_agent":{"name":"{{ active_agent.name }}","description":{{ active_agent.description | to_json_escaped_string }}}{% endif %}{% if completed_agents %},"completed_agents":[{% for agent in completed_agents %}{"name":"{{ agent.name }}","description":{{ agent.description | to_json_escaped_string }}}{% if not loop.last %},{% endif %}{% endfor %}]{% endif %}}
 ```{% else %}
 You are currently not inside any flow.{% endif %}