fix(tool/bigquery): handle omitted optional parameters with typed NULLs

[Deeven-Seru]

Description

This PR fixes a runtime bigquery: nil parameter error that occurred when optional parameters were omitted in BigQuery tool calls. The issue was caused by the tool sending untyped nil values to the BigQuery API, which the SDK does not accept for parameters.

Impact:

• Resolves crashes when executing queries with omitted optional parameters.
• Enables the use of IS NULL logic and optional filters in BigQuery SQL statements.

Summary of Solution:

• Typed NULL Mapping: Refactored parameter processing to map tool-level nil values to specific BigQuery SDK types (NullString, NullInt64, NullFloat64, NullBool).
• REST API Support: Implemented NullFields handling for the low-level REST client to ensure NULL values are correctly marshaled during dry runs.
• Array Support: Added handling for typed nil slices for array parameters to ensure alignment with standard SQL expectations.
• Verification: Added comprehensive unit tests in bigquerysql_invoke_test.go to validate mapping for all supported scalar and collection types.

PR Checklist

• Make sure you reviewed CONTRIBUTING.md
• Make sure to open an issue as a bug/issue before writing your code!
• Ensure the tests and linter pass
• Code coverage does not decrease (if any source code was changed)
• Appropriate docs were updated (if necessary)
• Make sure to add ! if this involve a breaking change

🛠️ Fixes #3033

[gemini-code-assist]

Code Review

This pull request refactors the BigQuery SQL tool to better handle optional parameters and NULL values by introducing a dedicated buildQueryParameters function. It also adds comprehensive unit tests for parameter building. The review feedback correctly identifies a potential issue with named parameter detection using simple string containment, which could lead to false positives, and suggests handling the TypeMap parameter type to prevent potential nil pointer issues.

[duwenxin99]

Hi @Deeven-Seru, thanks for the PR! We hope to include this fix in the release tomorrow. Let me know if you have any issues fixing the test failures. Otherwise LGTM.

[Deeven-Seru]

hi @duwenxin99 thanks for reviewing. done with final touches, please re initiate the checks

[duwenxin99]

/gcbrun

[duwenxin99]

/gcbrun

[duwenxin99]

/gcbrun

[Deeven-Seru]

🎉