Skip to content

Conversation

@soerenmartius
Copy link
Contributor

@soerenmartius soerenmartius commented Oct 23, 2025

Note

Adds Deployments and Previews SDK services and MCP tools (deployments listing, stack deployment details/logs, stack preview logs) with registrations, types, and comprehensive docs/tests.

  • MCP Tools:
    • Add tmc_list_deployments, tmc_get_stack_deployment, tmc_get_deployment_logs, and tmc_get_stack_preview_logs in tools/tmc/* with handlers registered in tools/handlers.go.
  • SDK (Go):
    • New services: DeploymentsService (sdk/terramate/deployments.go) and PreviewsService (sdk/terramate/previews.go), wired in client.go.
    • Extend types.go with deployment, preview, logs models and list/get options.
    • Add tests for deployments and previews endpoints (*_test.go).
  • Documentation:
    • Major README overhaul: list and document all tools, add detailed use cases, update features and coverage, project structure, and SDK section.
    • Add sdk/terramate/README.md with full SDK docs/examples.
    • Add CLAUDE.md agent guidelines.

Written by Cursor Bugbot for commit 39ce35f. This will update automatically on new commits. Configure here.

@soerenmartius soerenmartius force-pushed the soerenmartius/deploys-and-previews branch from 1bdd1d7 to 39ce35f Compare October 23, 2025 22:12
@soerenmartius soerenmartius marked this pull request as ready for review October 23, 2025 22:17
@soerenmartius soerenmartius requested review from a team as code owners October 23, 2025 22:17
addPagination(query, opts.Page, opts.PerPage)
addStringSlice(query, "repository", opts.Repository)
addStringSlice(query, "type", opts.AuthType)
addStringSlice(query, "status", opts.Status)
Copy link

Choose a reason for hiding this comment

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

Bug: Deployment Query Parameter Naming Mismatch

The DeploymentsListOptions.buildQuery() method uses "type" as the query parameter for the AuthType filter. The API expects "auth_type", which means this filter won't be applied correctly when listing deployments.

Fix in Cursor Fix in Web

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.

1 participant