feat(tool/postgres): add postgres-execute-sql-many and postgres-sql-many#3083
feat(tool/postgres): add postgres-execute-sql-many and postgres-sql-many#3083duwenxin99 wants to merge 6 commits intomainfrom
postgres-execute-sql-many and postgres-sql-many#3083Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces the postgres-execute-sql-many and postgres-sql-many tools and adds an ExecuteSql method to the Cloud SQL Admin source. Review feedback identifies several improvement opportunities, including the removal of the unused region parameter in both tools and the cleanup of the postgressqlmany configuration by removing the unsupported Parameters field and its associated runtime checks.
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces the postgres-execute-sql-many and postgres-sql-many tools, enabling SQL execution on Cloud SQL Postgres instances provided at runtime. The changes encompass tool implementations, documentation, prebuilt configuration updates, and integration tests. Review feedback suggests appending " Tool" to the documentation titles for both tools to maintain consistency.
Add 2 new tools using the Cloud SQL Admin
instances.executeSqlAPI to enable SQL execution against instances provided at runtime:Add prebuilt config for the
postgres-execute-sql-manytool tocloud-sql-postgres-admin.