Skip to content

feat: add ListMysqlDatabases endpoint #113

Draft
VoLKyyyOG wants to merge 1 commit into
mainfrom
akiraw/ListMysqlDatabases
Draft

feat: add ListMysqlDatabases endpoint #113
VoLKyyyOG wants to merge 1 commit into
mainfrom
akiraw/ListMysqlDatabases

Conversation

@VoLKyyyOG
Copy link
Copy Markdown
Collaborator

@VoLKyyyOG VoLKyyyOG commented May 18, 2026

Create new endpoint for use by metanexus.
This lists all mysql databases.

ref: https://linear.app/squareup/issue/DFNS-445/create-schemabot-endpoint-listmysqldatabases

Copilot AI review requested due to automatic review settings May 18, 2026 02:04
@VoLKyyyOG VoLKyyyOG self-assigned this May 18, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new API endpoint for listing configured MySQL databases from SchemaBot’s server configuration.

Changes:

  • Defines response types for MySQL database inventory.
  • Registers and implements GET /api/mysql/databases with optional environment filtering.
  • Adds handler tests and updates API README route documentation.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/apitypes/apitypes.go Adds response structs for MySQL database inventory.
pkg/api/service.go Registers the new MySQL database listing route.
pkg/api/README.md Documents the new route and handler file.
pkg/api/mysql_handlers.go Implements the MySQL database inventory handler and filtering logic.
pkg/api/mysql_handlers_test.go Adds tests for listing, filtering, and empty configuration behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@VoLKyyyOG VoLKyyyOG marked this pull request as draft May 18, 2026 02:14
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