Skip to content

fix(parameters): report the offending value in array/map type errors#3512

Merged
Yuan325 merged 4 commits into
googleapis:mainfrom
he-yufeng:fix/array-map-parse-error-value
Jul 7, 2026
Merged

fix(parameters): report the offending value in array/map type errors#3512
Yuan325 merged 4 commits into
googleapis:mainfrom
he-yufeng:fix/array-map-parse-error-value

Conversation

@he-yufeng

@he-yufeng he-yufeng commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Description

ArrayParameter.Parse and MapParameter.Parse construct their ParseTypeError from the variable produced by the type assertion (arrVal / m). But that branch is only reached after the assertion has failed, so the variable is always the nil zero value. The resulting error drops the value the caller actually passed:

  • an array parameter given a string reports [] not type "array"
  • a map parameter given a string reports map[] not type "map"

Every other Parse implementation (StringParameter, IntParameter, FloatParameter, BooleanParameter, …) passes the original v to ParseTypeError; these two had just drifted out of line. Passing v in both restores the actual input in the message:

  • "not-an-array" not type "array"
  • "not-a-map" not type "map"

Added TestParseTypeErrorReportsValue, which asserts the array/map type errors name the offending value. It fails on main and passes with this change.

PR Checklist

  • Reviewed CONTRIBUTING.md
  • Tests and linter pass locally (go test ./internal/util/parameters/, go vet, gofmt -l)
  • Code coverage does not decrease (adds a test)
  • Docs — not applicable
  • Breaking change — no

Didn't file a separate issue first since this is a small, self-contained consistency fix; happy to open one if you'd prefer to track it.

@he-yufeng
he-yufeng requested a review from a team as a code owner June 24, 2026 15:30
gemini-code-assist[bot]
gemini-code-assist Bot reviewed Jun 24, 2026
View reviewed changes

@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 fixes an issue in the array and map parameter parsers where type assertion failures reported the nil zero-value of the target type instead of the actual offending input value. A new unit test, TestParseTypeErrorReportsValue, has been added to verify that the correct offending value is reported in the error message. There are no review comments to address.

This was referenced Jun 24, 2026
Closed
Merged
@he-yufeng
fix(parameters): report the offending value in array/map type errors
0982761 
ArrayParameter.Parse and MapParameter.Parse passed their asserted
variable (arrVal / m) to ParseTypeError, but that assertion had already
failed, so the value was always the nil zero value. The errors read
`[] not type "array"` and `map[] not type "map"` instead of naming the
actual input. Every other Parse implementation passes the original value
v; align these two with it.
@he-yufeng
he-yufeng force-pushed the fix/array-map-parse-error-value branch from 150b734 to 0982761 Compare June 25, 2026 04:51
@averikitsch averikitsch added the blunderbuss: assign Instruct blunderbuss to assign someone label Jun 25, 2026
@blunderbuss-gcf blunderbuss-gcf Bot removed the blunderbuss: assign Instruct blunderbuss to assign someone label Jun 25, 2026
@Yuan325 Yuan325 assigned Yuan325 and unassigned duwenxin99 Jul 7, 2026
@Yuan325 Yuan325 added the priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. label Jul 7, 2026
Yuan325 added a commit that referenced this pull request Jul 7, 2026
@he-yufeng @Yuan325
fix(parameters): return an error instead of panicking on a non-string…
66a0d53 
… type field (#3516)

## Description

`parseParamFromDelayedUnmarshaler` reads the `type` field from a
parameter map and passes it straight to `ParseParameter` with an
unchecked type assertion:

```go
return ParseParameter(ctx, p, t.(string))
```

If a tools file declares a parameter whose `type` is not a string (for
example `type: 123` from a YAML typo, or `type:` left as a mapping),
`t.(string)` panics with `interface conversion: interface {} is int, not
string` instead of surfacing a config error. The panic propagates out of
`UnmarshalYAML`, so a single malformed parameter takes down config
loading with a stack trace rather than a readable message.

This is inconsistent with the rest of the function and with
`ParseParameter` itself: the missing-`type` case right above already
returns a clean error, and `ParseParameter`'s switch has a default
branch that reports unknown types as errors. Only the string assertion
was unguarded.

The fix uses the comma-ok form and returns the same style of error the
surrounding code already uses:

```go
typeStr, ok := t.(string)
if !ok {
    return nil, fmt.Errorf("parameter 'type' field must be a string, got %T", t)
}
return ParseParameter(ctx, p, typeStr)
```

Added a `TestFailParametersUnmarshal` case ("common parameter with
non-string type") that feeds an integer `type` and expects the error. I
confirmed it panics on `main` before the change and passes after; the
full `internal/util/parameters` package still passes and `gofmt`/`go
vet` are clean.

Note this is a separate concern from #3512, which also touches
`parameters.go` but in a different function (the array/map element
parsing paths). No overlap with that change.

## PR Checklist

- [x] Make sure you reviewed
[CONTRIBUTING.md](https://github.com/googleapis/mcp-toolbox/blob/main/CONTRIBUTING.md)
- [x] Ensure the tests and linter pass
- [x] Code coverage does not decrease (added a test covering the new
branch)
- [ ] Appropriate docs were updated (no behavior/doc change, internal
robustness fix)

Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
@Yuan325

Yuan325 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

/gcbrun

@Yuan325
Yuan325 enabled auto-merge (squash) July 7, 2026 03:42
@Yuan325
Yuan325 merged commit 4034d6f into googleapis:main Jul 7, 2026
20 checks passed
@release-please release-please Bot mentioned this pull request Jul 7, 2026
Merged
github-actions Bot pushed a commit to rodineyw/mcp-toolbox that referenced this pull request Jul 7, 2026
@pull
deploy: fix(parameters): report the offending value in array/map type…
190d679 
… errors (googleapis#3512)

## Description

`ArrayParameter.Parse` and `MapParameter.Parse` construct their
`ParseTypeError` from the variable produced by the type assertion
(`arrVal` / `m`). But that branch is only reached *after* the assertion
has failed, so the variable is always the nil zero value. The resulting
error drops the value the caller actually passed:

- an array parameter given a string reports `[] not type "array"`
- a map parameter given a string reports `map[] not type "map"`

Every other `Parse` implementation (`StringParameter`, `IntParameter`,
`FloatParameter`, `BooleanParameter`, …) passes the original `v` to
`ParseTypeError`; these two had just drifted out of line. Passing `v` in
both restores the actual input in the message:

- `"not-an-array" not type "array"`
- `"not-a-map" not type "map"`

Added `TestParseTypeErrorReportsValue`, which asserts the array/map type
errors name the offending value. It fails on `main` and passes with this
change.

## PR Checklist

- [x] Reviewed CONTRIBUTING.md
- [x] Tests and linter pass locally (`go test
./internal/util/parameters/`, `go vet`, `gofmt -l`)
- [x] Code coverage does not decrease (adds a test)
- [ ] Docs — not applicable
- [ ] Breaking change — no

Didn't file a separate issue first since this is a small, self-contained
consistency fix; happy to open one if you'd prefer to track it.

Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 4034d6f
github-actions Bot pushed a commit to Jaleel-zhu/genai-toolbox that referenced this pull request Jul 7, 2026
@pull
deploy: fix(parameters): report the offending value in array/map type…
640ba5e 
… errors (googleapis#3512)

## Description

`ArrayParameter.Parse` and `MapParameter.Parse` construct their
`ParseTypeError` from the variable produced by the type assertion
(`arrVal` / `m`). But that branch is only reached *after* the assertion
has failed, so the variable is always the nil zero value. The resulting
error drops the value the caller actually passed:

- an array parameter given a string reports `[] not type "array"`
- a map parameter given a string reports `map[] not type "map"`

Every other `Parse` implementation (`StringParameter`, `IntParameter`,
`FloatParameter`, `BooleanParameter`, …) passes the original `v` to
`ParseTypeError`; these two had just drifted out of line. Passing `v` in
both restores the actual input in the message:

- `"not-an-array" not type "array"`
- `"not-a-map" not type "map"`

Added `TestParseTypeErrorReportsValue`, which asserts the array/map type
errors name the offending value. It fails on `main` and passes with this
change.

## PR Checklist

- [x] Reviewed CONTRIBUTING.md
- [x] Tests and linter pass locally (`go test
./internal/util/parameters/`, `go vet`, `gofmt -l`)
- [x] Code coverage does not decrease (adds a test)
- [ ] Docs — not applicable
- [ ] Breaking change — no

Didn't file a separate issue first since this is a small, self-contained
consistency fix; happy to open one if you'd prefer to track it.

Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 4034d6f
github-actions Bot pushed a commit to pepe57/genai-toolbox that referenced this pull request Jul 7, 2026
@pull
deploy: fix(parameters): report the offending value in array/map type…
bf6787f 
… errors (googleapis#3512)

## Description

`ArrayParameter.Parse` and `MapParameter.Parse` construct their
`ParseTypeError` from the variable produced by the type assertion
(`arrVal` / `m`). But that branch is only reached *after* the assertion
has failed, so the variable is always the nil zero value. The resulting
error drops the value the caller actually passed:

- an array parameter given a string reports `[] not type "array"`
- a map parameter given a string reports `map[] not type "map"`

Every other `Parse` implementation (`StringParameter`, `IntParameter`,
`FloatParameter`, `BooleanParameter`, …) passes the original `v` to
`ParseTypeError`; these two had just drifted out of line. Passing `v` in
both restores the actual input in the message:

- `"not-an-array" not type "array"`
- `"not-a-map" not type "map"`

Added `TestParseTypeErrorReportsValue`, which asserts the array/map type
errors name the offending value. It fails on `main` and passes with this
change.

## PR Checklist

- [x] Reviewed CONTRIBUTING.md
- [x] Tests and linter pass locally (`go test
./internal/util/parameters/`, `go vet`, `gofmt -l`)
- [x] Code coverage does not decrease (adds a test)
- [ ] Docs — not applicable
- [ ] Breaking change — no

Didn't file a separate issue first since this is a small, self-contained
consistency fix; happy to open one if you'd prefer to track it.

Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 4034d6f
Yuan325 added a commit that referenced this pull request Jul 16, 2026
@release-please @Yuan325
chore(main): release 1.7.0 (#3568)
fbe2b21 
🤖 I have created a release *beep* *boop*
---


##
[1.7.0](v1.6.0...v1.7.0)
(2026-07-16)


#### Features

* Add `quotaProject` support for BigQuery and Looker conversational
analytics
([#2610](#2610))
([f3e7ca9](f3e7ca9))
* **arcadedb:** Add arcadedb source and tools
([#2961](#2961))
([351de00](351de00))
* **cmd/internal,docs:** Add warning log that prebuilt tools are for
developer use
([#3451](#3451))
([8cffcef](8cffcef))
* **source/postgres:** Add optional connectTimeout
([#3620](#3620))
([b574b07](b574b07))
* **tool/clickhouse-sql:** Add native vector embedding support
([#3229](#3229))
([6cbe1c2](6cbe1c2))
*
**tool/dataplex-update-data-product,tool/dataplex-create-data-asset,tool/dataplex-update-data-asset:**
Add update data product, create data asset, and update data asset tools
for knowledge catalog source
([#3574](#3574))
([721c204](721c204))
* **tools/dataplex-create-data-product:** Add
dataplex-create-data-product tool
([#3504](#3504))
([5cee0d2](5cee0d2))
* **tools/dataplex-get-data-asset:** Add dataplex-get-data-asset tool
([#3503](#3503))
([1ddfbe9](1ddfbe9))


### Bug Fixes

* **parameters:** Report the offending value in array/map type errors
([#3512](#3512))
([4034d6f](4034d6f))
* **parameters:** Return an error instead of panicking on a non-string
type field
([#3516](#3516))
([66a0d53](66a0d53))
* **source/looker:** Dynamically resolve public host URL
([#3603](#3603))
([0428afd](0428afd))
* **tool/looker-run-dashboard:** Add support for SQL Runner query tiles
([#3594](#3594))
([0975d0a](0975d0a))

---
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>
github-actions Bot pushed a commit that referenced this pull request Jul 16, 2026
@Yuan325 @release-please
deploy: chore(main): release 1.7.0 (#3568)
493e0ee 
🤖 I have created a release *beep* *boop*
---

##
[1.7.0](v1.6.0...v1.7.0)
(2026-07-16)

#### Features

* Add `quotaProject` support for BigQuery and Looker conversational
analytics
([#2610](#2610))
([f3e7ca9](f3e7ca9))
* **arcadedb:** Add arcadedb source and tools
([#2961](#2961))
([351de00](351de00))
* **cmd/internal,docs:** Add warning log that prebuilt tools are for
developer use
([#3451](#3451))
([8cffcef](8cffcef))
* **source/postgres:** Add optional connectTimeout
([#3620](#3620))
([b574b07](b574b07))
* **tool/clickhouse-sql:** Add native vector embedding support
([#3229](#3229))
([6cbe1c2](6cbe1c2))
*
**tool/dataplex-update-data-product,tool/dataplex-create-data-asset,tool/dataplex-update-data-asset:**
Add update data product, create data asset, and update data asset tools
for knowledge catalog source
([#3574](#3574))
([721c204](721c204))
* **tools/dataplex-create-data-product:** Add
dataplex-create-data-product tool
([#3504](#3504))
([5cee0d2](5cee0d2))
* **tools/dataplex-get-data-asset:** Add dataplex-get-data-asset tool
([#3503](#3503))
([1ddfbe9](1ddfbe9))

### Bug Fixes

* **parameters:** Report the offending value in array/map type errors
([#3512](#3512))
([4034d6f](4034d6f))
* **parameters:** Return an error instead of panicking on a non-string
type field
([#3516](#3516))
([66a0d53](66a0d53))
* **source/looker:** Dynamically resolve public host URL
([#3603](#3603))
([0428afd](0428afd))
* **tool/looker-run-dashboard:** Add support for SQL Runner query tiles
([#3594](#3594))
([0975d0a](0975d0a))

---
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> fbe2b21
@BrewTestBot BrewTestBot mentioned this pull request Jul 16, 2026
Merged
github-actions Bot pushed a commit to rodineyw/mcp-toolbox that referenced this pull request Jul 17, 2026
@pull @release-please
deploy: chore(main): release 1.7.0 (googleapis#3568)
e417c38 
🤖 I have created a release *beep* *boop*
---

##
[1.7.0](googleapis/mcp-toolbox@v1.6.0...v1.7.0)
(2026-07-16)

#### Features

* Add `quotaProject` support for BigQuery and Looker conversational
analytics
([googleapis#2610](googleapis#2610))
([f3e7ca9](googleapis@f3e7ca9))
* **arcadedb:** Add arcadedb source and tools
([googleapis#2961](googleapis#2961))
([351de00](googleapis@351de00))
* **cmd/internal,docs:** Add warning log that prebuilt tools are for
developer use
([googleapis#3451](googleapis#3451))
([8cffcef](googleapis@8cffcef))
* **source/postgres:** Add optional connectTimeout
([googleapis#3620](googleapis#3620))
([b574b07](googleapis@b574b07))
* **tool/clickhouse-sql:** Add native vector embedding support
([googleapis#3229](googleapis#3229))
([6cbe1c2](googleapis@6cbe1c2))
*
**tool/dataplex-update-data-product,tool/dataplex-create-data-asset,tool/dataplex-update-data-asset:**
Add update data product, create data asset, and update data asset tools
for knowledge catalog source
([googleapis#3574](googleapis#3574))
([721c204](googleapis@721c204))
* **tools/dataplex-create-data-product:** Add
dataplex-create-data-product tool
([googleapis#3504](googleapis#3504))
([5cee0d2](googleapis@5cee0d2))
* **tools/dataplex-get-data-asset:** Add dataplex-get-data-asset tool
([googleapis#3503](googleapis#3503))
([1ddfbe9](googleapis@1ddfbe9))

### Bug Fixes

* **parameters:** Report the offending value in array/map type errors
([googleapis#3512](googleapis#3512))
([4034d6f](googleapis@4034d6f))
* **parameters:** Return an error instead of panicking on a non-string
type field
([googleapis#3516](googleapis#3516))
([66a0d53](googleapis@66a0d53))
* **source/looker:** Dynamically resolve public host URL
([googleapis#3603](googleapis#3603))
([0428afd](googleapis@0428afd))
* **tool/looker-run-dashboard:** Add support for SQL Runner query tiles
([googleapis#3594](googleapis#3594))
([0975d0a](googleapis@0975d0a))

---
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> fbe2b21
github-actions Bot pushed a commit to Jaleel-zhu/genai-toolbox that referenced this pull request Jul 17, 2026
@pull @release-please
deploy: chore(main): release 1.7.0 (googleapis#3568)
bf47240 
🤖 I have created a release *beep* *boop*
---

##
[1.7.0](googleapis/mcp-toolbox@v1.6.0...v1.7.0)
(2026-07-16)

#### Features

* Add `quotaProject` support for BigQuery and Looker conversational
analytics
([googleapis#2610](googleapis#2610))
([f3e7ca9](googleapis@f3e7ca9))
* **arcadedb:** Add arcadedb source and tools
([googleapis#2961](googleapis#2961))
([351de00](googleapis@351de00))
* **cmd/internal,docs:** Add warning log that prebuilt tools are for
developer use
([googleapis#3451](googleapis#3451))
([8cffcef](googleapis@8cffcef))
* **source/postgres:** Add optional connectTimeout
([googleapis#3620](googleapis#3620))
([b574b07](googleapis@b574b07))
* **tool/clickhouse-sql:** Add native vector embedding support
([googleapis#3229](googleapis#3229))
([6cbe1c2](googleapis@6cbe1c2))
*
**tool/dataplex-update-data-product,tool/dataplex-create-data-asset,tool/dataplex-update-data-asset:**
Add update data product, create data asset, and update data asset tools
for knowledge catalog source
([googleapis#3574](googleapis#3574))
([721c204](googleapis@721c204))
* **tools/dataplex-create-data-product:** Add
dataplex-create-data-product tool
([googleapis#3504](googleapis#3504))
([5cee0d2](googleapis@5cee0d2))
* **tools/dataplex-get-data-asset:** Add dataplex-get-data-asset tool
([googleapis#3503](googleapis#3503))
([1ddfbe9](googleapis@1ddfbe9))

### Bug Fixes

* **parameters:** Report the offending value in array/map type errors
([googleapis#3512](googleapis#3512))
([4034d6f](googleapis@4034d6f))
* **parameters:** Return an error instead of panicking on a non-string
type field
([googleapis#3516](googleapis#3516))
([66a0d53](googleapis@66a0d53))
* **source/looker:** Dynamically resolve public host URL
([googleapis#3603](googleapis#3603))
([0428afd](googleapis@0428afd))
* **tool/looker-run-dashboard:** Add support for SQL Runner query tiles
([googleapis#3594](googleapis#3594))
([0975d0a](googleapis@0975d0a))

---
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> fbe2b21
github-actions Bot pushed a commit to pepe57/genai-toolbox that referenced this pull request Jul 17, 2026
@pull @release-please
deploy: chore(main): release 1.7.0 (googleapis#3568)
b1a1118 
🤖 I have created a release *beep* *boop*
---

##
[1.7.0](googleapis/mcp-toolbox@v1.6.0...v1.7.0)
(2026-07-16)

#### Features

* Add `quotaProject` support for BigQuery and Looker conversational
analytics
([googleapis#2610](googleapis#2610))
([f3e7ca9](googleapis@f3e7ca9))
* **arcadedb:** Add arcadedb source and tools
([googleapis#2961](googleapis#2961))
([351de00](googleapis@351de00))
* **cmd/internal,docs:** Add warning log that prebuilt tools are for
developer use
([googleapis#3451](googleapis#3451))
([8cffcef](googleapis@8cffcef))
* **source/postgres:** Add optional connectTimeout
([googleapis#3620](googleapis#3620))
([b574b07](googleapis@b574b07))
* **tool/clickhouse-sql:** Add native vector embedding support
([googleapis#3229](googleapis#3229))
([6cbe1c2](googleapis@6cbe1c2))
*
**tool/dataplex-update-data-product,tool/dataplex-create-data-asset,tool/dataplex-update-data-asset:**
Add update data product, create data asset, and update data asset tools
for knowledge catalog source
([googleapis#3574](googleapis#3574))
([721c204](googleapis@721c204))
* **tools/dataplex-create-data-product:** Add
dataplex-create-data-product tool
([googleapis#3504](googleapis#3504))
([5cee0d2](googleapis@5cee0d2))
* **tools/dataplex-get-data-asset:** Add dataplex-get-data-asset tool
([googleapis#3503](googleapis#3503))
([1ddfbe9](googleapis@1ddfbe9))

### Bug Fixes

* **parameters:** Report the offending value in array/map type errors
([googleapis#3512](googleapis#3512))
([4034d6f](googleapis@4034d6f))
* **parameters:** Return an error instead of panicking on a non-string
type field
([googleapis#3516](googleapis#3516))
([66a0d53](googleapis@66a0d53))
* **source/looker:** Dynamically resolve public host URL
([googleapis#3603](googleapis#3603))
([0428afd](googleapis@0428afd))
* **tool/looker-run-dashboard:** Add support for SQL Runner query tiles
([googleapis#3594](googleapis#3594))
([0975d0a](googleapis@0975d0a))

---
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> fbe2b21
github-actions Bot pushed a commit to CaffeeLake/genai-toolbox that referenced this pull request Jul 17, 2026
@CaffeeLake @release-please
deploy: chore(main): release 1.7.0 (googleapis#3568)
3ecb9ec 
🤖 I have created a release *beep* *boop*
---

##
[1.7.0](googleapis/mcp-toolbox@v1.6.0...v1.7.0)
(2026-07-16)

#### Features

* Add `quotaProject` support for BigQuery and Looker conversational
analytics
([googleapis#2610](googleapis#2610))
([f3e7ca9](googleapis@f3e7ca9))
* **arcadedb:** Add arcadedb source and tools
([googleapis#2961](googleapis#2961))
([351de00](googleapis@351de00))
* **cmd/internal,docs:** Add warning log that prebuilt tools are for
developer use
([googleapis#3451](googleapis#3451))
([8cffcef](googleapis@8cffcef))
* **source/postgres:** Add optional connectTimeout
([googleapis#3620](googleapis#3620))
([b574b07](googleapis@b574b07))
* **tool/clickhouse-sql:** Add native vector embedding support
([googleapis#3229](googleapis#3229))
([6cbe1c2](googleapis@6cbe1c2))
*
**tool/dataplex-update-data-product,tool/dataplex-create-data-asset,tool/dataplex-update-data-asset:**
Add update data product, create data asset, and update data asset tools
for knowledge catalog source
([googleapis#3574](googleapis#3574))
([721c204](googleapis@721c204))
* **tools/dataplex-create-data-product:** Add
dataplex-create-data-product tool
([googleapis#3504](googleapis#3504))
([5cee0d2](googleapis@5cee0d2))
* **tools/dataplex-get-data-asset:** Add dataplex-get-data-asset tool
([googleapis#3503](googleapis#3503))
([1ddfbe9](googleapis@1ddfbe9))

### Bug Fixes

* **parameters:** Report the offending value in array/map type errors
([googleapis#3512](googleapis#3512))
([4034d6f](googleapis@4034d6f))
* **parameters:** Return an error instead of panicking on a non-string
type field
([googleapis#3516](googleapis#3516))
([66a0d53](googleapis@66a0d53))
* **source/looker:** Dynamically resolve public host URL
([googleapis#3603](googleapis#3603))
([0428afd](googleapis@0428afd))
* **tool/looker-run-dashboard:** Add support for SQL Runner query tiles
([googleapis#3594](googleapis#3594))
([0975d0a](googleapis@0975d0a))

---
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> fbe2b21
github-actions Bot pushed a commit to bhardwajRahul/genai-toolbox that referenced this pull request Jul 17, 2026
@pull @release-please
deploy: chore(main): release 1.7.0 (googleapis#3568)
30292f6 
🤖 I have created a release *beep* *boop*
---

##
[1.7.0](googleapis/mcp-toolbox@v1.6.0...v1.7.0)
(2026-07-16)

#### Features

* Add `quotaProject` support for BigQuery and Looker conversational
analytics
([googleapis#2610](googleapis#2610))
([f3e7ca9](googleapis@f3e7ca9))
* **arcadedb:** Add arcadedb source and tools
([googleapis#2961](googleapis#2961))
([351de00](googleapis@351de00))
* **cmd/internal,docs:** Add warning log that prebuilt tools are for
developer use
([googleapis#3451](googleapis#3451))
([8cffcef](googleapis@8cffcef))
* **source/postgres:** Add optional connectTimeout
([googleapis#3620](googleapis#3620))
([b574b07](googleapis@b574b07))
* **tool/clickhouse-sql:** Add native vector embedding support
([googleapis#3229](googleapis#3229))
([6cbe1c2](googleapis@6cbe1c2))
*
**tool/dataplex-update-data-product,tool/dataplex-create-data-asset,tool/dataplex-update-data-asset:**
Add update data product, create data asset, and update data asset tools
for knowledge catalog source
([googleapis#3574](googleapis#3574))
([721c204](googleapis@721c204))
* **tools/dataplex-create-data-product:** Add
dataplex-create-data-product tool
([googleapis#3504](googleapis#3504))
([5cee0d2](googleapis@5cee0d2))
* **tools/dataplex-get-data-asset:** Add dataplex-get-data-asset tool
([googleapis#3503](googleapis#3503))
([1ddfbe9](googleapis@1ddfbe9))

### Bug Fixes

* **parameters:** Report the offending value in array/map type errors
([googleapis#3512](googleapis#3512))
([4034d6f](googleapis@4034d6f))
* **parameters:** Return an error instead of panicking on a non-string
type field
([googleapis#3516](googleapis#3516))
([66a0d53](googleapis@66a0d53))
* **source/looker:** Dynamically resolve public host URL
([googleapis#3603](googleapis#3603))
([0428afd](googleapis@0428afd))
* **tool/looker-run-dashboard:** Add support for SQL Runner query tiles
([googleapis#3594](googleapis#3594))
([0975d0a](googleapis@0975d0a))

---
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> fbe2b21
github-actions Bot pushed a commit to CrazyForks/genai-toolbox that referenced this pull request Jul 17, 2026
@pull @release-please
deploy: chore(main): release 1.7.0 (googleapis#3568)
dca249b 
🤖 I have created a release *beep* *boop*
---

##
[1.7.0](googleapis/mcp-toolbox@v1.6.0...v1.7.0)
(2026-07-16)

#### Features

* Add `quotaProject` support for BigQuery and Looker conversational
analytics
([googleapis#2610](googleapis#2610))
([f3e7ca9](googleapis@f3e7ca9))
* **arcadedb:** Add arcadedb source and tools
([googleapis#2961](googleapis#2961))
([351de00](googleapis@351de00))
* **cmd/internal,docs:** Add warning log that prebuilt tools are for
developer use
([googleapis#3451](googleapis#3451))
([8cffcef](googleapis@8cffcef))
* **source/postgres:** Add optional connectTimeout
([googleapis#3620](googleapis#3620))
([b574b07](googleapis@b574b07))
* **tool/clickhouse-sql:** Add native vector embedding support
([googleapis#3229](googleapis#3229))
([6cbe1c2](googleapis@6cbe1c2))
*
**tool/dataplex-update-data-product,tool/dataplex-create-data-asset,tool/dataplex-update-data-asset:**
Add update data product, create data asset, and update data asset tools
for knowledge catalog source
([googleapis#3574](googleapis#3574))
([721c204](googleapis@721c204))
* **tools/dataplex-create-data-product:** Add
dataplex-create-data-product tool
([googleapis#3504](googleapis#3504))
([5cee0d2](googleapis@5cee0d2))
* **tools/dataplex-get-data-asset:** Add dataplex-get-data-asset tool
([googleapis#3503](googleapis#3503))
([1ddfbe9](googleapis@1ddfbe9))

### Bug Fixes

* **parameters:** Report the offending value in array/map type errors
([googleapis#3512](googleapis#3512))
([4034d6f](googleapis@4034d6f))
* **parameters:** Return an error instead of panicking on a non-string
type field
([googleapis#3516](googleapis#3516))
([66a0d53](googleapis@66a0d53))
* **source/looker:** Dynamically resolve public host URL
([googleapis#3603](googleapis#3603))
([0428afd](googleapis@0428afd))
* **tool/looker-run-dashboard:** Add support for SQL Runner query tiles
([googleapis#3594](googleapis#3594))
([0975d0a](googleapis@0975d0a))

---
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> fbe2b21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Yuan325 Yuan325 Yuan325 approved these changes
+1 more reviewer
@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@Yuan325 Yuan325

Labels

priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@he-yufeng @Yuan325 @averikitsch @duwenxin99