fix(tools/alloydbainl): only add psv when NL Config Param is defined#2265
Conversation
Summary of ChangesHello @Yuan325, 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 resolves an issue where the AlloyDB AI Natural Language (NL) tool would always require Parameter-Sensitive Values (PSV), even when no NL Configuration Parameters were specified. The change optimizes the SQL query generation by conditionally including the Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. 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
|
There was a problem hiding this comment.
Code Review
This pull request correctly modifies the SQL statement generation to only include param_names and param_values when parameters are present, which fixes the described issue. I've included one suggestion to refactor the new code block for improved conciseness and to remove a redundant function call.
e9d2d62 to
8905986
Compare
8905986 to
710ae06
Compare
🤖 I have created a release *beep* *boop* --- ## [0.25.0](v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([#2121](#2121)) ([9c62f31](9c62f31)) * Add `allowed-hosts` flag ([#2254](#2254)) ([17b41f6](17b41f6)) * Add parameter default value to manifest ([#2264](#2264)) ([9d1feca](9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([#858](#858)) ([b706b5b](b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([#2202](#2202)) ([731a32e](731a32e)) * **sources/bigquery:** Make credentials scope configurable ([#2210](#2210)) ([a450600](a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([#2155](#2155)) ([4a4cf1e](4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([#2245](#2245)) ([eb79339](eb79339)) * **tools/postgressql:** Add tool to list store procedure ([#2156](#2156)) ([cf0fc51](cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([#2151](#2151)) ([17b70cc](17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([#2281](#2281)) ([a779975](a779975)) * **sources/cloudgda:** Add import for cloudgda source ([#2217](#2217)) ([7daa411](7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([#2228](#2228)) ([7053fbb](7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([#2265](#2265)) ([ef8f3b0](ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([#2231](#2231)) ([268700b](268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.25.0](v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([#2121](#2121)) ([9c62f31](9c62f31)) * Add `allowed-hosts` flag ([#2254](#2254)) ([17b41f6](17b41f6)) * Add parameter default value to manifest ([#2264](#2264)) ([9d1feca](9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([#858](#858)) ([b706b5b](b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([#2202](#2202)) ([731a32e](731a32e)) * **sources/bigquery:** Make credentials scope configurable ([#2210](#2210)) ([a450600](a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([#2155](#2155)) ([4a4cf1e](4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([#2245](#2245)) ([eb79339](eb79339)) * **tools/postgressql:** Add tool to list store procedure ([#2156](#2156)) ([cf0fc51](cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([#2151](#2151)) ([17b70cc](17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([#2281](#2281)) ([a779975](a779975)) * **sources/cloudgda:** Add import for cloudgda source ([#2217](#2217)) ([7daa411](7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([#2228](#2228)) ([7053fbb](7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([#2265](#2265)) ([ef8f3b0](ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([#2231](#2231)) ([268700b](268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 41b518b
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 41b518b
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 41b518b
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 41b518b
…2265) ## Description PSV should only be required when when it is needed. Currently, we require psv even whenever user uses AlloyDB AI NL tool. This is due to the statement that we use to execute nl query. This PR modified the statement query to only utilize `param_names` and `param_values` when needed. Manually tested with a db that does not have psv installed. 🛠️ Fixes #1970
🤖 I have created a release *beep* *boop* --- ## [0.25.0](v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([#2121](#2121)) ([9c62f31](9c62f31)) * Add `allowed-hosts` flag ([#2254](#2254)) ([17b41f6](17b41f6)) * Add parameter default value to manifest ([#2264](#2264)) ([9d1feca](9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([#858](#858)) ([b706b5b](b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([#2202](#2202)) ([731a32e](731a32e)) * **sources/bigquery:** Make credentials scope configurable ([#2210](#2210)) ([a450600](a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([#2155](#2155)) ([4a4cf1e](4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([#2245](#2245)) ([eb79339](eb79339)) * **tools/postgressql:** Add tool to list store procedure ([#2156](#2156)) ([cf0fc51](cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([#2151](#2151)) ([17b70cc](17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([#2281](#2281)) ([a779975](a779975)) * **sources/cloudgda:** Add import for cloudgda source ([#2217](#2217)) ([7daa411](7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([#2228](#2228)) ([7053fbb](7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([#2265](#2265)) ([ef8f3b0](ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([#2231](#2231)) ([268700b](268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 41b518b
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 41b518b
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 41b518b
…oogleapis#2265) ## Description PSV should only be required when when it is needed. Currently, we require psv even whenever user uses AlloyDB AI NL tool. This is due to the statement that we use to execute nl query. This PR modified the statement query to only utilize `param_names` and `param_values` when needed. Manually tested with a db that does not have psv installed. 🛠️ Fixes googleapis#1970
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
…oogleapis#2265) ## Description PSV should only be required when when it is needed. Currently, we require psv even whenever user uses AlloyDB AI NL tool. This is due to the statement that we use to execute nl query. This PR modified the statement query to only utilize `param_names` and `param_values` when needed. Manually tested with a db that does not have psv installed. 🛠️ Fixes googleapis#1970
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 41b518b
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 41b518b
…oogleapis#2265) ## Description PSV should only be required when when it is needed. Currently, we require psv even whenever user uses AlloyDB AI NL tool. This is due to the statement that we use to execute nl query. This PR modified the statement query to only utilize `param_names` and `param_values` when needed. Manually tested with a db that does not have psv installed. 🛠️ Fixes googleapis#1970
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
…oogleapis#2265) ## Description PSV should only be required when when it is needed. Currently, we require psv even whenever user uses AlloyDB AI NL tool. This is due to the statement that we use to execute nl query. This PR modified the statement query to only utilize `param_names` and `param_values` when needed. Manually tested with a db that does not have psv installed. 🛠️ Fixes googleapis#1970
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([googleapis#2121](googleapis#2121)) ([9c62f31](googleapis@9c62f31)) * Add `allowed-hosts` flag ([googleapis#2254](googleapis#2254)) ([17b41f6](googleapis@17b41f6)) * Add parameter default value to manifest ([googleapis#2264](googleapis#2264)) ([9d1feca](googleapis@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([googleapis#858](googleapis#858)) ([b706b5b](googleapis@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([googleapis#2202](googleapis#2202)) ([731a32e](googleapis@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([googleapis#2210](googleapis#2210)) ([a450600](googleapis@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([googleapis#2155](googleapis#2155)) ([4a4cf1e](googleapis@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([googleapis#2245](googleapis#2245)) ([eb79339](googleapis@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([googleapis#2156](googleapis#2156)) ([cf0fc51](googleapis@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([googleapis#2151](googleapis#2151)) ([17b70cc](googleapis@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([googleapis#2281](googleapis#2281)) ([a779975](googleapis@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([googleapis#2217](googleapis#2217)) ([7daa411](googleapis@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([googleapis#2228](googleapis#2228)) ([7053fbb](googleapis@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([googleapis#2265](googleapis#2265)) ([ef8f3b0](googleapis@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([googleapis#2231](googleapis#2231)) ([268700b](googleapis@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.25.0](googleapis/mcp-toolbox@v0.24.0...v0.25.0) (2026-01-08) ### Features * Add `embeddingModel` support ([#2121](googleapis/mcp-toolbox#2121)) ([9c62f31](googleapis/mcp-toolbox@9c62f31)) * Add `allowed-hosts` flag ([#2254](googleapis/mcp-toolbox#2254)) ([17b41f6](googleapis/mcp-toolbox@17b41f6)) * Add parameter default value to manifest ([#2264](googleapis/mcp-toolbox#2264)) ([9d1feca](googleapis/mcp-toolbox@9d1feca)) * **snowflake:** Add Snowflake Source and Tools ([#858](googleapis/mcp-toolbox#858)) ([b706b5b](googleapis/mcp-toolbox@b706b5b)) * **prebuilt/cloud-sql-mysql:** Update CSQL MySQL prebuilt tools to use IAM ([#2202](googleapis/mcp-toolbox#2202)) ([731a32e](googleapis/mcp-toolbox@731a32e)) * **sources/bigquery:** Make credentials scope configurable ([#2210](googleapis/mcp-toolbox#2210)) ([a450600](googleapis/mcp-toolbox@a450600)) * **sources/trino:** Add ssl verification options and fix docs example ([#2155](googleapis/mcp-toolbox#2155)) ([4a4cf1e](googleapis/mcp-toolbox@4a4cf1e)) * **tools/looker:** Add ability to set destination folder with `make_look` and `make_dashboard`. ([#2245](googleapis/mcp-toolbox#2245)) ([eb79339](googleapis/mcp-toolbox@eb79339)) * **tools/postgressql:** Add tool to list store procedure ([#2156](googleapis/mcp-toolbox#2156)) ([cf0fc51](googleapis/mcp-toolbox@cf0fc51)) * **tools/postgressql:** Add Parameter `embeddedBy` config support ([#2151](googleapis/mcp-toolbox#2151)) ([17b70cc](googleapis/mcp-toolbox@17b70cc)) ### Bug Fixes * **server:** Add `embeddingModel` config initialization ([#2281](googleapis/mcp-toolbox#2281)) ([a779975](googleapis/mcp-toolbox@a779975)) * **sources/cloudgda:** Add import for cloudgda source ([#2217](googleapis/mcp-toolbox#2217)) ([7daa411](googleapis/mcp-toolbox@7daa411)) * **tools/alloydb-wait-for-operation:** Fix connection message generation ([#2228](googleapis/mcp-toolbox#2228)) ([7053fbb](googleapis/mcp-toolbox@7053fbb)) * **tools/alloydbainl:** Only add psv when NL Config Param is defined ([#2265](googleapis/mcp-toolbox#2265)) ([ef8f3b0](googleapis/mcp-toolbox@ef8f3b0)) * **tools/looker:** Looker client OAuth nil pointer error ([#2231](googleapis/mcp-toolbox#2231)) ([268700b](googleapis/mcp-toolbox@268700b)) --- 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: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
Description
PSV should only be required when when it is needed. Currently, we require psv even whenever user uses AlloyDB AI NL tool. This is due to the statement that we use to execute nl query.
This PR modified the statement query to only utilize
param_namesandparam_valueswhen needed.Manually tested with a db that does not have psv installed.
🛠️ Fixes #1970