fix(database): improve error handling, password generation, and clean up unused tool options#57
Open
martin-helmich wants to merge 10 commits into
Open
fix(database): improve error handling, password generation, and clean up unused tool options#57martin-helmich wants to merge 10 commits into
martin-helmich wants to merge 10 commits into
Conversation
Previously, database operations returned generic "Request failed with status code 400" errors without the actual validation messages from the API. Now the error handling extracts structured error details including individual field validation errors. Example output: "Validation error: version: must be one of '8.0', '8.4'" Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
martin-helmich
force-pushed
the
fix/database-api-error-details
branch
from
cbda0c5 to
b609629
Compare
…hars The auto-generated password for MySQL database creation was using hex-only characters, which failed API validation requiring at least one special character. Now generates passwords using full alphanumeric + special char set (~128 bits entropy) that meet all API requirements, and documents the password constraints in the tool description. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Document that the version should be in '<major>.<minor>' format (e.g. '8.0') instead of referencing a CLI command that isn't available via MCP. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
MySQL and Redis version list tools always return JSON regardless of output format options. Remove unused parameters (output/outputFormat, extended, noHeader, noTruncate, noRelativeDates, csvSeparator) since they have no effect. Keep projectId for Redis as different projects may have different versions available. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…h 1) Remove unused output/outputFormat, extended, noHeader, noTruncate, noRelativeDates, and csvSeparator options from tool definitions and handlers. These options had no effect - all tools return structured JSON regardless of settings. Cleaned up: app/list, project/list, database tools (mysql/list, redis/list, list, charsets, user-list), domain tools (list, dnszone/list, virtualhost-list), backup tools (list, schedule-list), and version tools (mysql/versions, redis/versions). Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…h 2) Cleaned up: cronjob/list, cronjob/execution-list, container/list, server/list. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…h 3) Remove unused output/outputFormat, extended, noHeader, noTruncate, noRelativeDates, and csvSeparator options from remaining 24 tool definitions and handlers. Cleaned up: user/ssh-key/list, volume/list, stack/ps, user/api-token/list, user/session/list, ssh/user-list, stack/list, sftp/user-list, project/membership-list, project/membership-list-own, registry/list, project/invite-list, project/invite-list-own, mail/address/list, mail/deliverybox/list, org/invite-list, org/invite-list-own, extension/list, extension/list-installed, database/redis/get, database/mysql/user-get, conversation/list, conversation/categories, app/list-upgrade-candidates. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
martin-helmich
changed the title
…deploy - Add compose-to-mittwald.ts converter that parses docker-compose YAML and converts to Mittwald declarative stack API format - Add declareStack library function using @mittwald/api-client types (ContainerServiceDeclareRequest, ContainerVolumeDeclareRequest) - Update mittwald_stack_deploy tool to accept composeYaml string content instead of file path (works for remote MCP server scenarios) - Converter supports: services with image, ports, environment, volumes, command, entrypoint; warns about unsupported features (networks, deploy, healthcheck, depends_on) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add explicit instruction in mittwald_stack_deploy description to first read the existing stack configuration before updating, to prevent accidental deletion of existing services/volumes. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
API Error Handling
Validation error: version: must be one of '8.0', '8.4'; userPassword: must be at least 16 charactersMySQL Password Generation
-_;)Tool Parameter Cleanup
<major>.<minor>, e.g.8.0)output/outputFormat,extended,noHeader,noTruncate,noRelativeDates,csvSeparatorFiles Changed
🤖 Generated with Claude Code