Skip to content

Conversation

@michaelneale
Copy link
Collaborator

cover gemini 3 flash (preview)

add gemini 3 flash preview to provider e2e test coverage
Copilot AI review requested due to automatic review settings December 22, 2025 23:39
@michaelneale michaelneale requested review from DOsinga and removed request for Copilot December 22, 2025 23:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds coverage for the new Gemini 3 Flash preview model in the provider test script.

  • Adds gemini-3-flash-preview to the list of Google models tested

"openai:gpt-4o:gpt-4o-mini:gpt-3.5-turbo:gpt-5"
"anthropic:claude-sonnet-4-5-20250929:claude-opus-4-1-20250805"
"google:gemini-2.5-pro:gemini-2.5-flash:gemini-3-pro-preview"
"google:gemini-2.5-pro:gemini-2.5-flash:gemini-3-pro-preview:gemini-3-flash-preview"
Copy link

Copilot AI Dec 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The model gemini-3-flash-preview is being added to the test script but is missing from the GOOGLE_KNOWN_MODELS constant in crates/goose/src/providers/google.rs. The known models list should include this model for consistency with the test coverage.

Suggested change
"google:gemini-2.5-pro:gemini-2.5-flash:gemini-3-pro-preview:gemini-3-flash-preview"
"google:gemini-2.5-pro:gemini-2.5-flash:gemini-3-pro-preview"

Copilot uses AI. Check for mistakes.
Copy link
Collaborator

@codefromthecrypt codefromthecrypt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

assuming we can ignore the copilot thing

@michaelneale
Copy link
Collaborator Author

yeah - eventually we drop the -preview.

@michaelneale michaelneale merged commit 5ca7eb2 into main Dec 23, 2025
29 of 30 checks passed
@michaelneale michaelneale deleted the micn/add-gemini3-flash branch December 23, 2025 00:12
wpfleger96 added a commit that referenced this pull request Dec 23, 2025
* main: (155 commits)
  remove Tool Selection Strategy preview (#6250)
  fix(cli): correct bash syntax in terminal integration functions (#6181)
  fix : opening a session to view it modifies session history order in desktop (#6156)
  test: fix recipe and audio tests to avoid side effects (#6231)
  chore: Update gemini versions in test_providers.sh (#6246)
  feat: option to stream json - jsonl really (#6228)
  feat: add mcp app renderer (#6095)
  docs: update skills extension to support .agents/skills directories (#6199)
  Add YouTube short to Chrome DevTools MCP tutorial (#6244)
  docs: Caveats for privacy information in logs documentation (#6218)
  move goose issue solver to opus (#6233)
  feat: improved UX for tool calls via execute_code (#6205)
  Blog: Code Mode Doesn't Replace MCP (#6227)
  fix: prevent keychain requests during cargo test (#6219)
  test: fix test_max_turns_limit slow execution and wrong message type (#6221)
  Skills vs MCP blog (#6220)
  Add blog post: Does Your AI Agent Need a Plan? (#6209)
  fix(ui): enable MCP UI to send a prompt message when an element is clicked (#6207)
  docs: param option for recipe deeplink/open (#6206)
  docs: edit in place or fork session (#6203)
  ...
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.

3 participants