feat: support for annotations#2007
Conversation
Summary of ChangesHello @drstrangelooker, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request enhances the Model Context Protocol (MCP) implementation by adding support for tool annotations. This feature allows tools to declare specific operational characteristics, such as whether they are read-only or destructive, directly within their manifest. The changes involve introducing a new Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
29df8e5 to
9fcd0c8
Compare
There was a problem hiding this comment.
Code Review
This pull request adds support for tool annotations as defined in the MCP specification, which is a valuable feature. The implementation correctly introduces a ToolAnnotations struct and updates the GetMcpManifest function. However, I've found a few critical issues: the ToolAnnotations struct is missing json tags, which will lead to incorrect serialization, and the logic for excluding annotations in older API versions is incorrect because it modifies copies of slice elements. I've also suggested adding test coverage for the new annotations feature. Addressing these points will ensure the feature is robust and compliant with the spec.
10a3797 to
980d771
Compare
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request successfully adds support for MCP tool annotations, introducing the ToolAnnotations struct and integrating it into the McpManifest. The widespread updates to GetMcpManifest calls are consistent and correct. I've provided a couple of suggestions for the internal/server/mcp/*/method.go files to enhance safety by avoiding in-place modification of manifest slices, which will make the code more robust. The addition of specific annotations to the Looker tools is a valuable improvement.
bb350cf to
195089c
Compare
8703b2c to
0ac456f
Compare
averikitsch
left a comment
There was a problem hiding this comment.
What do we think about expose this in the custom tools.yaml?
I think that is a great idea. |
c448a81 to
5be1e7d
Compare
If I understand how yaml serialization is done it should be pretty trivial. |
@averikitsch I moved it into the tools config. Take a look at the prebuilt looker.yaml for examples. |
645a2d9 to
d6a4400
Compare
The MCP spec supports tool annotations like the below structure in the 2025-06-18 version of the spec. https://modelcontextprotocol.io/specification/2025-06-18/schema#toolannotations ``` { destructiveHint?: boolean; idempotentHint?: boolean; openWorldHint?: boolean; readOnlyHint?: boolean; } ``` Added a ToolAnnotations structure, an Annotations member to the McpManifest structure, and a nil initializer for the Annotations member to all calls to GetMcpManifest. The ToolAnnotations structure and the member annotations are all defined as pointers so that they are omited when not set. There are times when the zero value is meaningful so this was the only way to make sure that we distinguish between not setting the annotation and setting it with a zero value.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
d6a4400 to
b80efcd
Compare
## Description The MCP spec supports tool annotations like the below structure in the 2025-06-18 version of the spec. https://modelcontextprotocol.io/specification/2025-06-18/schema#toolannotations ``` { destructiveHint?: boolean; idempotentHint?: boolean; openWorldHint?: boolean; readOnlyHint?: boolean; } ``` Added a ToolAnnotations structure, an Annotations member to the McpManifest structure, and a nil initializer for the Annotations member to all calls to GetMcpManifest. The ToolAnnotations structure and the member annotations are all defined as pointers so that they are omited when not set. There are times when the zero value is meaningful so this was the only way to make sure that we distinguish between not setting the annotation and setting it with a zero value. ## PR Checklist > Thank you for opening a Pull Request! Before submitting your PR, there are a > few things you can do to make sure it goes smoothly: - [x] Make sure you reviewed [CONTRIBUTING.md](https://github.com/googleapis/genai-toolbox/blob/main/CONTRIBUTING.md) - [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/genai-toolbox/issues/new/choose) before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea - [x] Ensure the tests and linter pass - [x] Code coverage does not decrease (if any source code was changed) - [x] Appropriate docs were updated (if necessary) - [x] Make sure to add `!` if this involve a breaking change 🛠️ Fixes googleapis#927 --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> ac21335
## Description The MCP spec supports tool annotations like the below structure in the 2025-06-18 version of the spec. https://modelcontextprotocol.io/specification/2025-06-18/schema#toolannotations ``` { destructiveHint?: boolean; idempotentHint?: boolean; openWorldHint?: boolean; readOnlyHint?: boolean; } ``` Added a ToolAnnotations structure, an Annotations member to the McpManifest structure, and a nil initializer for the Annotations member to all calls to GetMcpManifest. The ToolAnnotations structure and the member annotations are all defined as pointers so that they are omited when not set. There are times when the zero value is meaningful so this was the only way to make sure that we distinguish between not setting the annotation and setting it with a zero value. ## PR Checklist > Thank you for opening a Pull Request! Before submitting your PR, there are a > few things you can do to make sure it goes smoothly: - [x] Make sure you reviewed [CONTRIBUTING.md](https://github.com/googleapis/genai-toolbox/blob/main/CONTRIBUTING.md) - [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/genai-toolbox/issues/new/choose) before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea - [x] Ensure the tests and linter pass - [x] Code coverage does not decrease (if any source code was changed) - [x] Appropriate docs were updated (if necessary) - [x] Make sure to add `!` if this involve a breaking change 🛠️ Fixes googleapis#927 --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> ac21335
🤖 I have created a release *beep* *boop* --- ## [0.22.0](v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([#1984](#1984)) ([862868f](862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([#1976](#1976)) ([9f76026](9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([#2056](#2056)) ([0e7fbf4](0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([#1845](#1845)) ([5e43630](5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](17a9792)) * Support alternate accessToken header name ([#1968](#1968)) ([18017d6](18017d6)) * Support for annotations ([#2007](#2007)) ([ac21335](ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([#1943](#1943)) ([7a9cc63](7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([#1722](#1722)) ([8752e05](8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([#1919](#1919)) ([f4b1f0a](f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([#1934](#1934)) ([5ad7c61](5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([#1923](#1923)) ([a0f44d3](a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([#2045](#2045)) ([fb7aae9](fb7aae9)) * Correct FAQ to mention HTTP tools ([#2036](#2036)) ([7b44237](7b44237)) * Format BigQuery numeric output as decimal strings ([#2084](#2084)) ([155bff8](155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([#2049](#2049)) ([565460c](565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([#2046](#2046)) ([a03984c](a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([#2028](#2028)) ([730ac6d](730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([#2099](#2099)) ([744214e](744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.22.0](v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([#1984](#1984)) ([862868f](862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([#1976](#1976)) ([9f76026](9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([#2056](#2056)) ([0e7fbf4](0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([#1845](#1845)) ([5e43630](5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](17a9792)) * Support alternate accessToken header name ([#1968](#1968)) ([18017d6](18017d6)) * Support for annotations ([#2007](#2007)) ([ac21335](ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([#1943](#1943)) ([7a9cc63](7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([#1722](#1722)) ([8752e05](8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([#1919](#1919)) ([f4b1f0a](f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([#1934](#1934)) ([5ad7c61](5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([#1923](#1923)) ([a0f44d3](a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([#2045](#2045)) ([fb7aae9](fb7aae9)) * Correct FAQ to mention HTTP tools ([#2036](#2036)) ([7b44237](7b44237)) * Format BigQuery numeric output as decimal strings ([#2084](#2084)) ([155bff8](155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([#2049](#2049)) ([565460c](565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([#2046](#2046)) ([a03984c](a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([#2028](#2028)) ([730ac6d](730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([#2099](#2099)) ([744214e](744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> cb4529c
🤖 I have created a release *beep* *boop* --- ## [0.22.0](googleapis/mcp-toolbox@v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([googleapis#1984](googleapis#1984)) ([862868f](googleapis@862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([googleapis#1976](googleapis#1976)) ([9f76026](googleapis@9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([googleapis#2056](googleapis#2056)) ([0e7fbf4](googleapis@0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([googleapis#1845](googleapis#1845)) ([5e43630](googleapis@5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](googleapis@1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](googleapis@17a9792)) * Support alternate accessToken header name ([googleapis#1968](googleapis#1968)) ([18017d6](googleapis@18017d6)) * Support for annotations ([googleapis#2007](googleapis#2007)) ([ac21335](googleapis@ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([googleapis#1943](googleapis#1943)) ([7a9cc63](googleapis@7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([googleapis#1722](googleapis#1722)) ([8752e05](googleapis@8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([googleapis#1919](googleapis#1919)) ([f4b1f0a](googleapis@f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([googleapis#1934](googleapis#1934)) ([5ad7c61](googleapis@5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([googleapis#1923](googleapis#1923)) ([a0f44d3](googleapis@a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([googleapis#2045](googleapis#2045)) ([fb7aae9](googleapis@fb7aae9)) * Correct FAQ to mention HTTP tools ([googleapis#2036](googleapis#2036)) ([7b44237](googleapis@7b44237)) * Format BigQuery numeric output as decimal strings ([googleapis#2084](googleapis#2084)) ([155bff8](googleapis@155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([googleapis#2049](googleapis#2049)) ([565460c](googleapis@565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([googleapis#2046](googleapis#2046)) ([a03984c](googleapis@a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([googleapis#2028](googleapis#2028)) ([730ac6d](googleapis@730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([googleapis#2099](googleapis#2099)) ([744214e](googleapis@744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> cb4529c
🤖 I have created a release *beep* *boop* --- ## [0.22.0](googleapis/mcp-toolbox@v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([googleapis#1984](googleapis#1984)) ([862868f](googleapis@862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([googleapis#1976](googleapis#1976)) ([9f76026](googleapis@9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([googleapis#2056](googleapis#2056)) ([0e7fbf4](googleapis@0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([googleapis#1845](googleapis#1845)) ([5e43630](googleapis@5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](googleapis@1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](googleapis@17a9792)) * Support alternate accessToken header name ([googleapis#1968](googleapis#1968)) ([18017d6](googleapis@18017d6)) * Support for annotations ([googleapis#2007](googleapis#2007)) ([ac21335](googleapis@ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([googleapis#1943](googleapis#1943)) ([7a9cc63](googleapis@7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([googleapis#1722](googleapis#1722)) ([8752e05](googleapis@8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([googleapis#1919](googleapis#1919)) ([f4b1f0a](googleapis@f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([googleapis#1934](googleapis#1934)) ([5ad7c61](googleapis@5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([googleapis#1923](googleapis#1923)) ([a0f44d3](googleapis@a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([googleapis#2045](googleapis#2045)) ([fb7aae9](googleapis@fb7aae9)) * Correct FAQ to mention HTTP tools ([googleapis#2036](googleapis#2036)) ([7b44237](googleapis@7b44237)) * Format BigQuery numeric output as decimal strings ([googleapis#2084](googleapis#2084)) ([155bff8](googleapis@155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([googleapis#2049](googleapis#2049)) ([565460c](googleapis@565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([googleapis#2046](googleapis#2046)) ([a03984c](googleapis@a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([googleapis#2028](googleapis#2028)) ([730ac6d](googleapis@730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([googleapis#2099](googleapis#2099)) ([744214e](googleapis@744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> cb4529c
🤖 I have created a release *beep* *boop* --- ## [0.22.0](googleapis/mcp-toolbox@v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([googleapis#1984](googleapis#1984)) ([862868f](googleapis@862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([googleapis#1976](googleapis#1976)) ([9f76026](googleapis@9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([googleapis#2056](googleapis#2056)) ([0e7fbf4](googleapis@0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([googleapis#1845](googleapis#1845)) ([5e43630](googleapis@5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](googleapis@1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](googleapis@17a9792)) * Support alternate accessToken header name ([googleapis#1968](googleapis#1968)) ([18017d6](googleapis@18017d6)) * Support for annotations ([googleapis#2007](googleapis#2007)) ([ac21335](googleapis@ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([googleapis#1943](googleapis#1943)) ([7a9cc63](googleapis@7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([googleapis#1722](googleapis#1722)) ([8752e05](googleapis@8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([googleapis#1919](googleapis#1919)) ([f4b1f0a](googleapis@f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([googleapis#1934](googleapis#1934)) ([5ad7c61](googleapis@5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([googleapis#1923](googleapis#1923)) ([a0f44d3](googleapis@a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([googleapis#2045](googleapis#2045)) ([fb7aae9](googleapis@fb7aae9)) * Correct FAQ to mention HTTP tools ([googleapis#2036](googleapis#2036)) ([7b44237](googleapis@7b44237)) * Format BigQuery numeric output as decimal strings ([googleapis#2084](googleapis#2084)) ([155bff8](googleapis@155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([googleapis#2049](googleapis#2049)) ([565460c](googleapis@565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([googleapis#2046](googleapis#2046)) ([a03984c](googleapis@a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([googleapis#2028](googleapis#2028)) ([730ac6d](googleapis@730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([googleapis#2099](googleapis#2099)) ([744214e](googleapis@744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> cb4529c
🤖 I have created a release *beep* *boop* --- ## [0.22.0](googleapis/mcp-toolbox@v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([googleapis#1984](googleapis#1984)) ([862868f](googleapis@862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([googleapis#1976](googleapis#1976)) ([9f76026](googleapis@9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([googleapis#2056](googleapis#2056)) ([0e7fbf4](googleapis@0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([googleapis#1845](googleapis#1845)) ([5e43630](googleapis@5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](googleapis@1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](googleapis@17a9792)) * Support alternate accessToken header name ([googleapis#1968](googleapis#1968)) ([18017d6](googleapis@18017d6)) * Support for annotations ([googleapis#2007](googleapis#2007)) ([ac21335](googleapis@ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([googleapis#1943](googleapis#1943)) ([7a9cc63](googleapis@7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([googleapis#1722](googleapis#1722)) ([8752e05](googleapis@8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([googleapis#1919](googleapis#1919)) ([f4b1f0a](googleapis@f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([googleapis#1934](googleapis#1934)) ([5ad7c61](googleapis@5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([googleapis#1923](googleapis#1923)) ([a0f44d3](googleapis@a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([googleapis#2045](googleapis#2045)) ([fb7aae9](googleapis@fb7aae9)) * Correct FAQ to mention HTTP tools ([googleapis#2036](googleapis#2036)) ([7b44237](googleapis@7b44237)) * Format BigQuery numeric output as decimal strings ([googleapis#2084](googleapis#2084)) ([155bff8](googleapis@155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([googleapis#2049](googleapis#2049)) ([565460c](googleapis@565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([googleapis#2046](googleapis#2046)) ([a03984c](googleapis@a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([googleapis#2028](googleapis#2028)) ([730ac6d](googleapis@730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([googleapis#2099](googleapis#2099)) ([744214e](googleapis@744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> cb4529c
🤖 I have created a release *beep* *boop* --- ## [0.22.0](googleapis/mcp-toolbox@v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([googleapis#1984](googleapis#1984)) ([862868f](googleapis@862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([googleapis#1976](googleapis#1976)) ([9f76026](googleapis@9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([googleapis#2056](googleapis#2056)) ([0e7fbf4](googleapis@0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([googleapis#1845](googleapis#1845)) ([5e43630](googleapis@5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](googleapis@1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](googleapis@17a9792)) * Support alternate accessToken header name ([googleapis#1968](googleapis#1968)) ([18017d6](googleapis@18017d6)) * Support for annotations ([googleapis#2007](googleapis#2007)) ([ac21335](googleapis@ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([googleapis#1943](googleapis#1943)) ([7a9cc63](googleapis@7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([googleapis#1722](googleapis#1722)) ([8752e05](googleapis@8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([googleapis#1919](googleapis#1919)) ([f4b1f0a](googleapis@f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([googleapis#1934](googleapis#1934)) ([5ad7c61](googleapis@5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([googleapis#1923](googleapis#1923)) ([a0f44d3](googleapis@a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([googleapis#2045](googleapis#2045)) ([fb7aae9](googleapis@fb7aae9)) * Correct FAQ to mention HTTP tools ([googleapis#2036](googleapis#2036)) ([7b44237](googleapis@7b44237)) * Format BigQuery numeric output as decimal strings ([googleapis#2084](googleapis#2084)) ([155bff8](googleapis@155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([googleapis#2049](googleapis#2049)) ([565460c](googleapis@565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([googleapis#2046](googleapis#2046)) ([a03984c](googleapis@a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([googleapis#2028](googleapis#2028)) ([730ac6d](googleapis@730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([googleapis#2099](googleapis#2099)) ([744214e](googleapis@744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> cb4529c
🤖 I have created a release *beep* *boop* --- ## [0.22.0](v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([#1984](#1984)) ([862868f](862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([#1976](#1976)) ([9f76026](9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([#2056](#2056)) ([0e7fbf4](0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([#1845](#1845)) ([5e43630](5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](17a9792)) * Support alternate accessToken header name ([#1968](#1968)) ([18017d6](18017d6)) * Support for annotations ([#2007](#2007)) ([ac21335](ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([#1943](#1943)) ([7a9cc63](7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([#1722](#1722)) ([8752e05](8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([#1919](#1919)) ([f4b1f0a](f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([#1934](#1934)) ([5ad7c61](5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([#1923](#1923)) ([a0f44d3](a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([#2045](#2045)) ([fb7aae9](fb7aae9)) * Correct FAQ to mention HTTP tools ([#2036](#2036)) ([7b44237](7b44237)) * Format BigQuery numeric output as decimal strings ([#2084](#2084)) ([155bff8](155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([#2049](#2049)) ([565460c](565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([#2046](#2046)) ([a03984c](a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([#2028](#2028)) ([730ac6d](730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([#2099](#2099)) ([744214e](744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>
## Description The MCP spec supports tool annotations like the below structure in the 2025-06-18 version of the spec. https://modelcontextprotocol.io/specification/2025-06-18/schema#toolannotations ``` { destructiveHint?: boolean; idempotentHint?: boolean; openWorldHint?: boolean; readOnlyHint?: boolean; } ``` Added a ToolAnnotations structure, an Annotations member to the McpManifest structure, and a nil initializer for the Annotations member to all calls to GetMcpManifest. The ToolAnnotations structure and the member annotations are all defined as pointers so that they are omited when not set. There are times when the zero value is meaningful so this was the only way to make sure that we distinguish between not setting the annotation and setting it with a zero value. ## PR Checklist > Thank you for opening a Pull Request! Before submitting your PR, there are a > few things you can do to make sure it goes smoothly: - [x] Make sure you reviewed [CONTRIBUTING.md](https://github.com/googleapis/genai-toolbox/blob/main/CONTRIBUTING.md) - [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/genai-toolbox/issues/new/choose) before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea - [x] Ensure the tests and linter pass - [x] Code coverage does not decrease (if any source code was changed) - [x] Appropriate docs were updated (if necessary) - [x] Make sure to add `!` if this involve a breaking change 🛠️ Fixes googleapis#927 --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.22.0](googleapis/mcp-toolbox@v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([googleapis#1984](googleapis#1984)) ([862868f](googleapis@862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([googleapis#1976](googleapis#1976)) ([9f76026](googleapis@9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([googleapis#2056](googleapis#2056)) ([0e7fbf4](googleapis@0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([googleapis#1845](googleapis#1845)) ([5e43630](googleapis@5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](googleapis@1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](googleapis@17a9792)) * Support alternate accessToken header name ([googleapis#1968](googleapis#1968)) ([18017d6](googleapis@18017d6)) * Support for annotations ([googleapis#2007](googleapis#2007)) ([ac21335](googleapis@ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([googleapis#1943](googleapis#1943)) ([7a9cc63](googleapis@7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([googleapis#1722](googleapis#1722)) ([8752e05](googleapis@8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([googleapis#1919](googleapis#1919)) ([f4b1f0a](googleapis@f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([googleapis#1934](googleapis#1934)) ([5ad7c61](googleapis@5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([googleapis#1923](googleapis#1923)) ([a0f44d3](googleapis@a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([googleapis#2045](googleapis#2045)) ([fb7aae9](googleapis@fb7aae9)) * Correct FAQ to mention HTTP tools ([googleapis#2036](googleapis#2036)) ([7b44237](googleapis@7b44237)) * Format BigQuery numeric output as decimal strings ([googleapis#2084](googleapis#2084)) ([155bff8](googleapis@155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([googleapis#2049](googleapis#2049)) ([565460c](googleapis@565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([googleapis#2046](googleapis#2046)) ([a03984c](googleapis@a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([googleapis#2028](googleapis#2028)) ([730ac6d](googleapis@730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([googleapis#2099](googleapis#2099)) ([744214e](googleapis@744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.22.0](googleapis/mcp-toolbox@v0.21.0...v0.22.0) (2025-12-04) ### Features * Add allowed-origins flag ([#1984](googleapis/mcp-toolbox#1984)) ([862868f](googleapis/mcp-toolbox@862868f)) * **tools/postgres:** Add list-query-stats and get-column-cardinality functions ([#1976](googleapis/mcp-toolbox#1976)) ([9f76026](googleapis/mcp-toolbox@9f76026)) * **tools/spanner:** Add spanner list graphs to prebuiltconfigs ([#2056](googleapis/mcp-toolbox#2056)) ([0e7fbf4](googleapis/mcp-toolbox@0e7fbf4)) * **prebuilt/cloud-sql:** Add clone instance tool for cloud sql ([#1845](googleapis/mcp-toolbox#1845)) ([5e43630](googleapis/mcp-toolbox@5e43630)) * **serverless-spark:** Add create_pyspark_batch tool ([1bf0b51](googleapis/mcp-toolbox@1bf0b51)) * **serverless-spark:** Add create_spark_batch tool ([17a9792](googleapis/mcp-toolbox@17a9792)) * Support alternate accessToken header name ([#1968](googleapis/mcp-toolbox#1968)) ([18017d6](googleapis/mcp-toolbox@18017d6)) * Support for annotations ([#2007](googleapis/mcp-toolbox#2007)) ([ac21335](googleapis/mcp-toolbox@ac21335)) * **tool/mssql:** Set default host and port for MSSQL source ([#1943](googleapis/mcp-toolbox#1943)) ([7a9cc63](googleapis/mcp-toolbox@7a9cc63)) * **tools/cloudsqlpg:** Add CloudSQL PostgreSQL pre-check tool ([#1722](googleapis/mcp-toolbox#1722)) ([8752e05](googleapis/mcp-toolbox@8752e05)) * **tools/postgres-list-publication-tables:** Add new postgres-list-publication-tables tool ([#1919](googleapis/mcp-toolbox#1919)) ([f4b1f0a](googleapis/mcp-toolbox@f4b1f0a)) * **tools/postgres-list-tablespaces:** Add new postgres-list-tablespaces tool ([#1934](googleapis/mcp-toolbox#1934)) ([5ad7c61](googleapis/mcp-toolbox@5ad7c61)) * **tools/spanner-list-graph:** Tool impl + docs + tests ([#1923](googleapis/mcp-toolbox#1923)) ([a0f44d3](googleapis/mcp-toolbox@a0f44d3)) ### Bug Fixes * Add import for firebirdsql ([#2045](googleapis/mcp-toolbox#2045)) ([fb7aae9](googleapis/mcp-toolbox@fb7aae9)) * Correct FAQ to mention HTTP tools ([#2036](googleapis/mcp-toolbox#2036)) ([7b44237](googleapis/mcp-toolbox@7b44237)) * Format BigQuery numeric output as decimal strings ([#2084](googleapis/mcp-toolbox#2084)) ([155bff8](googleapis/mcp-toolbox@155bff8)) * Set default annotations for tools in code if annotation not provided in yaml ([#2049](googleapis/mcp-toolbox#2049)) ([565460c](googleapis/mcp-toolbox@565460c)) * **tools/alloydb-postgres-list-tables:** Exclude google_ml schema from list_tables ([#2046](googleapis/mcp-toolbox#2046)) ([a03984c](googleapis/mcp-toolbox@a03984c)) * **tools/alloydbcreateuser:** Remove duplication of project praram ([#2028](googleapis/mcp-toolbox#2028)) ([730ac6d](googleapis/mcp-toolbox@730ac6d)) * **tools/mongodb:** Remove `required` tag from the `canonical` field ([#2099](googleapis/mcp-toolbox#2099)) ([744214e](googleapis/mcp-toolbox@744214e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>
Description
The MCP spec supports tool annotations like the below structure in the 2025-06-18 version of the spec.
https://modelcontextprotocol.io/specification/2025-06-18/schema#toolannotations
Added a ToolAnnotations structure, an Annotations member to the McpManifest structure, and a nil initializer for the Annotations member to all calls to GetMcpManifest.
The ToolAnnotations structure and the member annotations are all defined as pointers so that they are omited when not set. There are times when the zero value is meaningful so this was the only way to make sure that we distinguish between not setting the annotation and setting it with a zero value.
PR Checklist
CONTRIBUTING.md
bug/issue
before writing your code! That way we can discuss the change, evaluate
designs, and agree on the general idea
!if this involve a breaking change🛠️ Fixes #927