Skip to content

refactor(ai): update Tool to GenerativeModel.Tool - #1873

Merged
andrewheard merged 3 commits into
mainfrom
ah/ai-tool-name
Sep 25, 2026
Merged

andrewheard merged 3 commits into
mainfrom
ah/ai-tool-name

Conversation

@andrewheard

@andrewheard andrewheard commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Update explicit Tool type annotations in Sample to GenerativeModel.Tool to prepare for the SDK refactoring in firebase/firebase-ios-sdk#16700, which nests Tool inside GenerativeModel and removes the top-level declaration.

Update explicit `Tool` type annotations in `Sample` to
`GenerativeModel.Tool` to prepare for the SDK refactoring in
firebase/firebase-ios-sdk#16700, which nests `Tool` inside
`GenerativeModel` and removes the top-level declaration.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Sample struct in Sample.swift to use GenerativeModel.Tool instead of Tool for its tools property and initializer argument. There are no review comments, and I have no feedback to provide.

Update the CI matrix to test on Xcode 26.2 for macOS 15 and Xcode 26.6
with iOS 26.5 for macOS 26.

This provides a more relevant test range since Firebase requires Xcode
26.2+ and Xcode 26.6 is the newest available on the macOS 26 runners.
Temporarily update the firebase-ios-sdk Swift package dependency to the
CocoaPods-13.0.0 branch to test against the upcoming breaking change
release.
@andrewheard
andrewheard marked this pull request as ready for review September 25, 2026 18:04
@andrewheard
andrewheard merged commit 087e4f4 into main Sep 25, 2026
18 checks passed
@andrewheard
andrewheard deleted the ah/ai-tool-name branch September 25, 2026 18:05
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.

2 participants