Jack Larch 7ed92c8023 fix(tools/bigquery-execute-sql): avoid surfacing invalid queries as MCP 500s (#3056) ...

## Description

Route `bigquery-execute-sql` dry-run validation failures through the
shared GCP error processor instead of hardcoding them as internal server
errors.

This keeps normal BigQuery query mistakes in the tool-error path rather
than escalating them into MCP transport failures.

## PR Checklist

- [x] Make sure you reviewed
[CONTRIBUTING.md](https://github.com/googleapis/mcp-toolbox/blob/main/CONTRIBUTING.md)
- [x] Make sure to open an issue as a
[bug/issue](https://github.com/googleapis/mcp-toolbox/issues/new/choose)
before writing your code! That way we can discuss the change, evaluate
designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [x] 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

Notes:
- I could not run `go test` locally in this environment because the Go
toolchain is not installed on this machine.
- I added a focused regression test around `ProcessGcpError` so the
intended classification is explicit in CI.

🛠️ Fixes #3055

Made with [Cursor](https://cursor.com)

Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>

2026-04-20 17:31:35 +00:00

..

bigquery_integration_test.go

fix(tools/bigquery-execute-sql): avoid surfacing invalid queries as MCP 500s (#3056)

2026-04-20 17:31:35 +00:00