Skip to content
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

WIP move tool calling to boxel message events and add support for multiple tool calls/commands #2145

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

lukemelia
Copy link
Contributor

@lukemelia lukemelia commented Feb 13, 2025

  • add host tests for multiple tool calls
  • add aibot tests for multiple tool calls with results
  • Ensure code TODOs are addressed

Copy link

github-actions bot commented Feb 13, 2025

Host Test Results

    1 files  ±0      1 suites  ±0   22m 36s ⏱️ -50s
764 tests +1  762 ✔️ +3  2 💤  - 1  0 ±0 
769 runs  +1  767 ✔️ +4  2 💤  - 1  0  - 1 

Results for commit 899da9b. ± Comparison against base commit 40442c9.

This pull request removes 6 and adds 7 tests. Note that renamed tests count towards both.
Chrome 133.0 ‑ Spec preview: have ability to create new spec instances
Chrome 133.0 ‑ Spec preview: title does not default to "default"
Chrome 133.0 ‑ Spec preview: view when there are multiple spec instances
Chrome 133.0 ‑ Spec preview: view when there are no spec instances
Chrome 133.0 ‑ Spec preview: view when there is a single spec instance
Chrome 133.0 ‑ Spec preview: view when users cannot write
Chrome 133.0 ‑ Acceptance | Spec preview: have ability to create new spec instances
Chrome 133.0 ‑ Acceptance | Spec preview: title does not default to "default"
Chrome 133.0 ‑ Acceptance | Spec preview: view when there are multiple spec instances
Chrome 133.0 ‑ Acceptance | Spec preview: view when there are no spec instances
Chrome 133.0 ‑ Acceptance | Spec preview: view when there is a single spec instance
Chrome 133.0 ‑ Acceptance | Spec preview: view when users cannot write
Chrome 133.0 ‑ Acceptance | code-submode | playground panel: can choose another instance to be opened in playground panel

♻️ This comment has been updated with latest results.

@lukemelia lukemelia force-pushed the cs-7993-support-multiple-tool-calls-per-ai-response branch 2 times, most recently from b762a09 to 8d52ce0 Compare February 14, 2025 04:07
@lukemelia lukemelia force-pushed the cs-7993-support-multiple-tool-calls-per-ai-response branch from 8d52ce0 to c7a2bd6 Compare February 14, 2025 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant