feat: sqlite implementation #438

waqarahmed6095 · 2025-04-14T14:09:58Z

Adding sqlite implementation

Fixes #406

Yuan325

Thank you for contributing! :) I left some comments for changes.

internal/sources/sqlite/sqlite.go

internal/tools/sqlitesql/sqlitesql.go

tests/sqlite_integration_test.go

docs/en/resources/tools/sqlite-sql.md

docs/en/resources/sources/sqlite.md

tests/sqlite_integration_test.go

.ci/continuous.release.cloudbuild.yaml

internal/tools/sqlitesql/sqlitesql.go

tests/sqlite_integration_test.go

Yuan325 · 2025-04-18T17:20:38Z

/gcbrun

.ci/continuous.release.cloudbuild.yaml

…al methods

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

Update SQLite source and tool configurations, add new test files, and modify .gitignore - Added `modernc.org/sqlite` as a direct dependency in `go.mod`. - Updated `go.sum` with new dependency versions. - Refactored imports in `config.go` and `mcp.go` for consistency. - Enhanced `sqlite.go` and `sqlitesql.go` with improved error handling and code formatting. - Added new test files: `test.db`, `test.ipynb`, and `tools.yaml` to `.gitignore` for better project management.

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

- Added `sqlite` to the list of tools in `.golangci.yaml`. - Downgraded Go version to 1.22 in `go.mod` and updated toolchain version accordingly. - Removed SQLite build steps from Cloud Build configuration files. - Added a new integration test step for SQLite in `integration.cloudbuild.yaml`. - Refactored `sqlitesql.go` to use `any` instead of `interface{}` for improved type handling.

n/a

Yuan325 · 2025-04-23T05:25:26Z

@waqarahmed6095 Thank you so much for your contribution!

Adding sqlite implementation --------- Co-authored-by: Yuan <45984206+Yuan325@users.noreply.github.com> Co-authored-by: Yuan Teoh <yuanteoh@google.com> fc14cbf

🤖 I have created a release *beep* *boop* --- ## [0.4.0](v0.3.0...v0.4.0) (2025-04-23) ### Features * Add `AuthRequired` to Neo4j & Dgraph Tools ([#434](#434)) ([afbf4b2](afbf4b2)) * Add `AuthRequired` to tool manifest ([#433](#433)) ([d9388ad](d9388ad)) * Add BigQuery source and tool ([#463](#463)) ([8055aa5](8055aa5)) * Add Bigtable source and tool ([#418](#418)) ([ae53b8e](ae53b8e)) * Add IAM AuthN to AlloyDB Source ([#399](#399)) ([e8ed447](e8ed447)) * Add IAM AuthN to Cloud SQL Sources ([#414](#414)) ([be85b82](be85b82)) * Add toolset feature to mcp ([#425](#425)) ([e307857](e307857)), closes [#403](#403) * Add SQLite source and tool ([#438](#438)) ([fc14cbf](fc14cbf)) * Support env replacement for tools.yaml ([#462](#462)) ([eadb678](eadb678)) ### Bug Fixes * [#419](#419) TLS https URL for SSE endpoint ([#420](#420)) ([0a7d3ff](0a7d3ff)) * **docs:** Fix link 'Edit this page' ([#454](#454)) ([969065e](969065e)), closes [#427](#427) * Update http error code from invocation ([#468](#468)) ([ff7c0ff](ff7c0ff)), closes [#465](#465) --- 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: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> Co-authored-by: Kurtis Van Gent <31518063+kurtisvg@users.noreply.github.com>

🤖 I have created a release *beep* *boop* --- ## [0.4.0](v0.3.0...v0.4.0) (2025-04-23) ### Features * Add `AuthRequired` to Neo4j & Dgraph Tools ([#434](#434)) ([afbf4b2](afbf4b2)) * Add `AuthRequired` to tool manifest ([#433](#433)) ([d9388ad](d9388ad)) * Add BigQuery source and tool ([#463](#463)) ([8055aa5](8055aa5)) * Add Bigtable source and tool ([#418](#418)) ([ae53b8e](ae53b8e)) * Add IAM AuthN to AlloyDB Source ([#399](#399)) ([e8ed447](e8ed447)) * Add IAM AuthN to Cloud SQL Sources ([#414](#414)) ([be85b82](be85b82)) * Add toolset feature to mcp ([#425](#425)) ([e307857](e307857)), closes [#403](#403) * Add SQLite source and tool ([#438](#438)) ([fc14cbf](fc14cbf)) * Support env replacement for tools.yaml ([#462](#462)) ([eadb678](eadb678)) ### Bug Fixes * [#419](#419) TLS https URL for SSE endpoint ([#420](#420)) ([0a7d3ff](0a7d3ff)) * **docs:** Fix link 'Edit this page' ([#454](#454)) ([969065e](969065e)), closes [#427](#427) * Update http error code from invocation ([#468](#468)) ([ff7c0ff](ff7c0ff)), closes [#465](#465) --- 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: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> Co-authored-by: Kurtis Van Gent <31518063+kurtisvg@users.noreply.github.com> 4ed16cc

🤖 I have created a release *beep* *boop* --- ## [0.4.0](googleapis/genai-toolbox@v0.3.0...v0.4.0) (2025-04-23) ### Features * Add `AuthRequired` to Neo4j & Dgraph Tools ([googleapis#434](googleapis#434)) ([afbf4b2](googleapis@afbf4b2)) * Add `AuthRequired` to tool manifest ([googleapis#433](googleapis#433)) ([d9388ad](googleapis@d9388ad)) * Add BigQuery source and tool ([googleapis#463](googleapis#463)) ([8055aa5](googleapis@8055aa5)) * Add Bigtable source and tool ([googleapis#418](googleapis#418)) ([ae53b8e](googleapis@ae53b8e)) * Add IAM AuthN to AlloyDB Source ([googleapis#399](googleapis#399)) ([e8ed447](googleapis@e8ed447)) * Add IAM AuthN to Cloud SQL Sources ([googleapis#414](googleapis#414)) ([be85b82](googleapis@be85b82)) * Add toolset feature to mcp ([googleapis#425](googleapis#425)) ([e307857](googleapis@e307857)), closes [googleapis#403](googleapis#403) * Add SQLite source and tool ([googleapis#438](googleapis#438)) ([fc14cbf](googleapis@fc14cbf)) * Support env replacement for tools.yaml ([googleapis#462](googleapis#462)) ([eadb678](googleapis@eadb678)) ### Bug Fixes * [googleapis#419](googleapis#419) TLS https URL for SSE endpoint ([googleapis#420](googleapis#420)) ([0a7d3ff](googleapis@0a7d3ff)) * **docs:** Fix link 'Edit this page' ([googleapis#454](googleapis#454)) ([969065e](googleapis@969065e)), closes [googleapis#427](googleapis#427) * Update http error code from invocation ([googleapis#468](googleapis#468)) ([ff7c0ff](googleapis@ff7c0ff)), closes [googleapis#465](googleapis#465) --- 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: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> Co-authored-by: Kurtis Van Gent <31518063+kurtisvg@users.noreply.github.com> 4ed16cc

Adding sqlite implementation --------- Co-authored-by: Yuan <45984206+Yuan325@users.noreply.github.com> Co-authored-by: Yuan Teoh <yuanteoh@google.com>

🤖 I have created a release *beep* *boop* --- ## [0.4.0](googleapis/genai-toolbox@v0.3.0...v0.4.0) (2025-04-23) ### Features * Add `AuthRequired` to Neo4j & Dgraph Tools ([googleapis#434](googleapis#434)) ([afbf4b2](googleapis@afbf4b2)) * Add `AuthRequired` to tool manifest ([googleapis#433](googleapis#433)) ([d9388ad](googleapis@d9388ad)) * Add BigQuery source and tool ([googleapis#463](googleapis#463)) ([8055aa5](googleapis@8055aa5)) * Add Bigtable source and tool ([googleapis#418](googleapis#418)) ([ae53b8e](googleapis@ae53b8e)) * Add IAM AuthN to AlloyDB Source ([googleapis#399](googleapis#399)) ([e8ed447](googleapis@e8ed447)) * Add IAM AuthN to Cloud SQL Sources ([googleapis#414](googleapis#414)) ([be85b82](googleapis@be85b82)) * Add toolset feature to mcp ([googleapis#425](googleapis#425)) ([e307857](googleapis@e307857)), closes [googleapis#403](googleapis#403) * Add SQLite source and tool ([googleapis#438](googleapis#438)) ([fc14cbf](googleapis@fc14cbf)) * Support env replacement for tools.yaml ([googleapis#462](googleapis#462)) ([eadb678](googleapis@eadb678)) ### Bug Fixes * [googleapis#419](googleapis#419) TLS https URL for SSE endpoint ([googleapis#420](googleapis#420)) ([0a7d3ff](googleapis@0a7d3ff)) * **docs:** Fix link 'Edit this page' ([googleapis#454](googleapis#454)) ([969065e](googleapis@969065e)), closes [googleapis#427](googleapis#427) * Update http error code from invocation ([googleapis#468](googleapis#468)) ([ff7c0ff](googleapis@ff7c0ff)), closes [googleapis#465](googleapis#465) --- 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: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> Co-authored-by: Kurtis Van Gent <31518063+kurtisvg@users.noreply.github.com>

waqarahmed6095 requested a review from a team as a code owner April 14, 2025 14:09

blunderbuss-gcf bot assigned Yuan325 Apr 14, 2025

waqarahmed6095 mentioned this pull request Apr 14, 2025

Sqlite #406

Closed

Yuan325 requested changes Apr 17, 2025

View reviewed changes

waqarahmed6095 requested a review from Yuan325 April 18, 2025 14:57

waqarahmed6095 changed the title ~~sqlite implementation~~ feat: sqlite implementation Apr 18, 2025

Yuan325 reviewed Apr 18, 2025

View reviewed changes

.ci/continuous.release.cloudbuild.yaml Outdated Show resolved Hide resolved

Yuan325 previously requested changes Apr 18, 2025

View reviewed changes

internal/tools/sqlitesql/sqlitesql.go Outdated Show resolved Hide resolved

internal/tools/sqlitesql/sqlitesql.go Outdated Show resolved Hide resolved

tests/sqlite_integration_test.go Show resolved Hide resolved

waqarahmed6095 requested a review from Yuan325 April 18, 2025 18:44

Yuan325 reviewed Apr 22, 2025

View reviewed changes

.ci/continuous.release.cloudbuild.yaml Outdated Show resolved Hide resolved

waqarahmed6095 requested a review from Yuan325 April 22, 2025 20:16

Yuan325 force-pushed the main branch from ddccc79 to c00a324 Compare April 23, 2025 00:49

Yuan325 added the tests: run Label to trigger Github Action tests. label Apr 23, 2025

github-actions bot removed the tests: run Label to trigger Github Action tests. label Apr 23, 2025

kurtisvg approved these changes Apr 23, 2025

View reviewed changes

Yuan325 force-pushed the main branch from c00a324 to 0323cbe Compare April 23, 2025 04:37

waqarahmed6095 and others added 13 commits April 22, 2025 21:41

feat: add SQLite support with configuration and SQL execution tools

05550df

feat: enhance Tool struct with parameter parsing and manifest retriev…

92da4a2

…al methods

Update internal/sources/sqlite/sqlite.go

34684fe

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

Update internal/tools/sqlitesql/sqlitesql.go

1670027

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

Update tests/sqlite_integration_test.go

301950e

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

Update docs/en/resources/tools/sqlite-sql.md

bc4f84f

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

Update docs/en/resources/tools/sqlite-sql.md

bfe4c1f

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

Update docs/en/resources/sources/sqlite.md

82b6cbe

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

Update docs/en/resources/tools/sqlite-sql.md

8bbd67f

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

Update docs/en/resources/tools/sqlite-sql.md

2a17f67

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

fix: update for sqlite

b6a90f6

Update tests/sqlite_integration_test.go

5419a6b

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

waqarahmed6095 and others added 9 commits April 22, 2025 21:43

Update docs/en/resources/tools/sqlite-sql.md

8d0204c

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

Update docs/en/resources/tools/sqlite-sql.md

163291d

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

Update docs/en/resources/tools/sqlite-sql.md

a83dbb3

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

Update .ci/continuous.release.cloudbuild.yaml

cb74a06

Update sqlitesql.go

89289bb

Update sqlite-sql.md

550f940

update integration test

58f84c8

rebase

65bf2c1

Yuan325 force-pushed the main branch from 0323cbe to 65bf2c1 Compare April 23, 2025 04:46

Yuan325 added the tests: run Label to trigger Github Action tests. label Apr 23, 2025

github-actions bot removed the tests: run Label to trigger Github Action tests. label Apr 23, 2025

Yuan325 merged commit fc14cbf into googleapis:main Apr 23, 2025
9 checks passed

release-please bot mentioned this pull request Apr 23, 2025

chore(main): release 0.4.0 #411

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: sqlite implementation #438

feat: sqlite implementation #438

Uh oh!

waqarahmed6095 commented Apr 14, 2025 •

edited by Yuan325

Loading

Uh oh!

Yuan325 left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Yuan325 commented Apr 18, 2025

Uh oh!

Uh oh!

Yuan325 commented Apr 23, 2025

Uh oh!

Uh oh!

Uh oh!

feat: sqlite implementation #438

feat: sqlite implementation #438

Uh oh!

Conversation

waqarahmed6095 commented Apr 14, 2025 • edited by Yuan325 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Yuan325 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Yuan325 commented Apr 18, 2025

Uh oh!

Uh oh!

Yuan325 commented Apr 23, 2025

Uh oh!

Uh oh!

Uh oh!

waqarahmed6095 commented Apr 14, 2025 •

edited by Yuan325

Loading