-
Notifications
You must be signed in to change notification settings - Fork 212
Refactor interactive flows to use the AskUser tool
#104
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
base: feat/plan-mode
Are you sure you want to change the base?
Conversation
…-review-recommendations feat(conductor): make review recommendations more conversational
AskUser toolAskUser tool
03f0dc5 to
276994e
Compare
This change ensures users see the overview of the initialization process before the system checks for an existing state file, providing better context for the setup workflow. Resolves: #81 Change-Id: Ibb1499be24cc9cb0290eb2c5e871c9d296aeac73
Change-Id: Id1f84748d29877b8a5794bb4cd02abdbc753e11d
276994e to
57055df
Compare
commands/conductor/implement.toml
Outdated
| 1. Perform an exact, case-insensitive match for the provided name against the track descriptions you parsed. | ||
| 2. If a unique match is found, confirm the selection with the user: "I found track '<track_description>'. Is this correct?" | ||
| 3. If no match is found, or if the match is ambiguous, inform the user and ask for clarification. Suggest the next available track as below. | ||
| 2. If a unique match is found, confirm the selection with the user using the `ask_user` tool: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this is feasible, but ideally we wanted to use something like AskUser.name to avoid issues in case the AskUser name changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need to request for this capability:
This is what is supported right now as Runtime Variable Support of commands:
{{args}}: Injects the arguments provided by the user.!{command}: Executes a shell command and injects the output.@{file}: Injects the content of a file.
Could you please create a ticket for gemini cli team to add the support for __tool__{} inside command.toml files. You can reference this PR as the need.
feat(review): update review process to commit fixes and update plan
fix(conductor): move pre-initialization overview before resume check
Migrate `ask_user` calls in `implement.toml`, `newTrack.toml`, `revert.toml`, `review.toml`, and `setup.toml` to the new schema: - Use the `questions` array parameter. - Format options with `label` and `description`.
aa1f199 to
12b3b36
Compare
To improve the user experience and ensure a consistent, structured interaction model across the Conductor extension, we have migrated all legacy text-based prompts to the integrated
AskUsertool. This transition allows for more intuitive UI elements (like single/multi-choice menus and yes/no toggles) and enables better handling of human-in-the-loop interactions during automated workflows.fixes: #105
related PR: #90