-
-
Notifications
You must be signed in to change notification settings - Fork 2
feat: support for intent, events and buttons requests #67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Caution Review failedThe pull request is closed. WalkthroughThis PR extends the CLI's interaction test framework with support for intents, entities, events, and button interactions. It introduces a new Changes
Sequence DiagramsequenceDiagram
participant TestRunner as Test Runner
participant Dialog as Dialog Manager
participant Voiceflow as Voiceflow API
participant TestTypes as Test Types
TestRunner->>Dialog: Interaction (text/event/intent/button)<br/>+ availableButtons
activate Dialog
alt Interaction Type
rect rgb(200, 240, 200)
Dialog->>TestTypes: Extract button from<br/>previous response
TestTypes-->>Dialog: availableButtons []Button
Dialog->>Dialog: Accumulate buttons
end
rect rgb(220, 220, 240)
Dialog->>Dialog: Match button by label<br/>in availableButtons
Dialog->>Dialog: Build ButtonRequest<br/>(type + payload)
end
rect rgb(240, 220, 220)
Dialog->>Dialog: Build EventPayload<br/>or IntentPayload<br/>(with entities)
end
end
Dialog->>Voiceflow: InteractionRequest<br/>(Action with event/intent/button payload)
activate Voiceflow
Voiceflow-->>Dialog: InteractionResponse[]
deactivate Voiceflow
rect rgb(200, 240, 200)
Dialog->>Dialog: Extract Choice response<br/>buttons for next iteration
end
Dialog-->>TestRunner: Response[] + buttons
deactivate Dialog
TestRunner->>TestRunner: Log interaction details<br/>Propagate remainingValidations<br/>Pass buttons to next interaction
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Areas requiring extra attention:
Possibly related PRs
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (43)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
added support for the following requests:
Summary by CodeRabbit
New Features
--show-tester-messagesCLI flag (default: enabled) to control visibility of tester agent messages during agent-to-agent testing scenarios.Documentation
Chores
✏️ Tip: You can customize this high-level summary in your review settings.