Skip to content

fix(looker): Convert prebuilt tool yaml to flat format#3022

Merged
drstrangelooker merged 1 commit intomainfrom
looker-migrate-toolfiles
Apr 10, 2026
Merged

fix(looker): Convert prebuilt tool yaml to flat format#3022
drstrangelooker merged 1 commit intomainfrom
looker-migrate-toolfiles

Conversation

@drstrangelooker
Copy link
Copy Markdown
Contributor

Description

Migrated the prebuilt tool files to use the flat format, then did some hand editing.
Checked using Gemini CLI that the same output is delivered to the MCP client.

Also fixed up some of the docs to use the flat format.

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:

  • Make sure you reviewed
    CONTRIBUTING.md
  • Make sure to open an issue as a
    bug/issue
    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
  • 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

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

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 refactors the Looker integration by transitioning prebuilt configuration files to a flat, multi-document YAML format and updating the associated documentation. It also introduces environment variable placeholders in examples and updates copyright headers to 2026. Feedback is provided regarding several typos and formatting errors in the looker.yaml configuration file, specifically misspelled parameter names and a stray quotation mark in a tool description.

@drstrangelooker drstrangelooker force-pushed the looker-migrate-toolfiles branch from 5f8257f to 584f512 Compare April 10, 2026 14:05
@drstrangelooker drstrangelooker force-pushed the looker-migrate-toolfiles branch from 584f512 to f5f56ea Compare April 10, 2026 18:19
@drstrangelooker drstrangelooker enabled auto-merge (squash) April 10, 2026 18:19
@drstrangelooker drstrangelooker merged commit 45c05e3 into main Apr 10, 2026
26 checks passed
@drstrangelooker drstrangelooker deleted the looker-migrate-toolfiles branch April 10, 2026 18:26
@github-actions
Copy link
Copy Markdown
Contributor

🧨 Preview deployments removed.

Cloudflare Pages environments for pr-3022 have been deleted.

github-actions bot pushed a commit that referenced this pull request Apr 10, 2026
## Description

Migrated the prebuilt tool files to use the flat format, then did some
hand editing.
Checked using Gemini CLI that the same output is delivered to the MCP
client.

Also fixed up some of the docs to use the flat format.

## 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/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
- [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 45c05e3
github-actions bot pushed a commit to renovate-bot/googleapis-_-genai-toolbox that referenced this pull request Apr 10, 2026
…eapis#3022)

## Description

Migrated the prebuilt tool files to use the flat format, then did some
hand editing.
Checked using Gemini CLI that the same output is delivered to the MCP
client.

Also fixed up some of the docs to use the flat format.

## 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/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
- [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 45c05e3
github-actions bot pushed a commit to pepe57/genai-toolbox that referenced this pull request Apr 10, 2026
…eapis#3022)

## Description

Migrated the prebuilt tool files to use the flat format, then did some
hand editing.
Checked using Gemini CLI that the same output is delivered to the MCP
client.

Also fixed up some of the docs to use the flat format.

## 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/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
- [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 45c05e3
github-actions bot pushed a commit to Jaleel-zhu/genai-toolbox that referenced this pull request Apr 10, 2026
…eapis#3022)

## Description

Migrated the prebuilt tool files to use the flat format, then did some
hand editing.
Checked using Gemini CLI that the same output is delivered to the MCP
client.

Also fixed up some of the docs to use the flat format.

## 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/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
- [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 45c05e3
github-actions bot pushed a commit to bhardwajRahul/genai-toolbox that referenced this pull request Apr 11, 2026
…eapis#3022)

## Description

Migrated the prebuilt tool files to use the flat format, then did some
hand editing.
Checked using Gemini CLI that the same output is delivered to the MCP
client.

Also fixed up some of the docs to use the flat format.

## 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/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
- [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 45c05e3
github-actions bot pushed a commit to CrazyForks/genai-toolbox that referenced this pull request Apr 11, 2026
…eapis#3022)

## Description

Migrated the prebuilt tool files to use the flat format, then did some
hand editing.
Checked using Gemini CLI that the same output is delivered to the MCP
client.

Also fixed up some of the docs to use the flat format.

## 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/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
- [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 45c05e3
github-actions bot pushed a commit to TheTechOddBug/genai-toolbox that referenced this pull request Apr 11, 2026
…eapis#3022)

## Description

Migrated the prebuilt tool files to use the flat format, then did some
hand editing.
Checked using Gemini CLI that the same output is delivered to the MCP
client.

Also fixed up some of the docs to use the flat format.

## 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/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
- [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 45c05e3
github-actions bot pushed a commit to anikasharma03/genai-toolbox that referenced this pull request Apr 13, 2026
…eapis#3022)

## Description

Migrated the prebuilt tool files to use the flat format, then did some
hand editing.
Checked using Gemini CLI that the same output is delivered to the MCP
client.

Also fixed up some of the docs to use the flat format.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants