Add operationId to OpenAPI spec (#135)

nathanjcochran · web-flow · commit df3524b4c719 · 2026-01-20T12:11:15.000-05:00
diff --git a/internal/tiger/analytics/analytics.go b/internal/tiger/analytics/analytics.go
@@ -156,7 +156,7 @@ func (a *Analytics) Identify(options ...Option) {
 	defer cancel()
 
 	// Send the event
-	resp, err := a.client.PostAnalyticsIdentifyWithResponse(ctx, api.PostAnalyticsIdentifyJSONRequestBody{
+	resp, err := a.client.IdentifyUserWithResponse(ctx, api.IdentifyUserJSONRequestBody{
 		Properties: &properties,
 	})
 	if err != nil {
@@ -221,7 +221,7 @@ func (a *Analytics) Track(event string, options ...Option) {
 	defer cancel()
 
 	// Send the event
-	resp, err := a.client.PostAnalyticsTrackWithResponse(ctx, api.PostAnalyticsTrackJSONRequestBody{
+	resp, err := a.client.TrackEventWithResponse(ctx, api.TrackEventJSONRequestBody{
 		Event:      event,
 		Properties: &properties,
 	})
diff --git a/internal/tiger/api/client.go b/internal/tiger/api/client.go
diff --git a/internal/tiger/api/client_util_test.go b/internal/tiger/api/client_util_test.go
@@ -40,7 +40,7 @@ func TestNewTigerClientUserAgent(t *testing.T) {
 
 	// Make a request to trigger the User-Agent header
 	ctx := context.Background()
-	_, err = client.GetProjectsProjectIdServicesWithResponse(ctx, "test-project-id")
+	_, err = client.GetServicesWithResponse(ctx, "test-project-id")
 	if err != nil {
 		t.Fatalf("Request failed: %v", err)
 	}
@@ -87,7 +87,7 @@ func TestNewTigerClientAuthorizationHeader(t *testing.T) {
 
 	// Make a request to trigger the Authorization header
 	ctx := context.Background()
-	_, err = client.GetProjectsProjectIdServicesWithResponse(ctx, "test-project-id")
+	_, err = client.GetServicesWithResponse(ctx, "test-project-id")
 	if err != nil {
 		t.Fatalf("Request failed: %v", err)
 	}
diff --git a/internal/tiger/api/mocks/mock_client.go b/internal/tiger/api/mocks/mock_client.go
diff --git a/internal/tiger/api/types.go b/internal/tiger/api/types.go
diff --git a/internal/tiger/cmd/db.go b/internal/tiger/cmd/db.go
@@ -781,7 +781,7 @@ func getServiceDetails(cmd *cobra.Command, cfg *common.Config, args []string) (a
 	ctx, cancel := context.WithTimeout(cmd.Context(), 30*time.Second)
 	defer cancel()
 
-	resp, err := cfg.Client.GetProjectsProjectIdServicesServiceIdWithResponse(ctx, cfg.ProjectID, serviceID)
+	resp, err := cfg.Client.GetServiceWithResponse(ctx, cfg.ProjectID, serviceID)
 	if err != nil {
 		return api.Service{}, fmt.Errorf("failed to fetch service details: %w", err)
 	}
diff --git a/internal/tiger/cmd/password_recovery.go b/internal/tiger/cmd/password_recovery.go
@@ -244,7 +244,7 @@ func updateAndSaveServicePassword(
 ) error {
 	// Call API to update password
 	updateReq := api.UpdatePasswordInput{Password: newPassword}
-	resp, err := client.PostProjectsProjectIdServicesServiceIdUpdatePasswordWithResponse(ctx, *service.ProjectId, *service.ServiceId, updateReq)
+	resp, err := client.UpdatePasswordWithResponse(ctx, *service.ProjectId, *service.ServiceId, updateReq)
 	if err != nil {
 		return fmt.Errorf("failed to update password: %w", err)
 	}
diff --git a/internal/tiger/cmd/service.go b/internal/tiger/cmd/service.go
@@ -91,7 +91,7 @@ Examples:
 			ctx, cancel := context.WithTimeout(cmd.Context(), 30*time.Second)
 			defer cancel()
 
-			resp, err := cfg.Client.GetProjectsProjectIdServicesServiceIdWithResponse(ctx, cfg.ProjectID, serviceID)
+			resp, err := cfg.Client.GetServiceWithResponse(ctx, cfg.ProjectID, serviceID)
 			if err != nil {
 				return fmt.Errorf("failed to get service details: %w", err)
 			}
@@ -142,7 +142,7 @@ func buildServiceListCmd() *cobra.Command {
 			ctx, cancel := context.WithTimeout(cmd.Context(), 30*time.Second)
 			defer cancel()
 
-			resp, err := cfg.Client.GetProjectsProjectIdServicesWithResponse(ctx, cfg.ProjectID)
+			resp, err := cfg.Client.GetServicesWithResponse(ctx, cfg.ProjectID)
 			if err != nil {
 				return fmt.Errorf("failed to list services: %w", err)
 			}
@@ -313,7 +313,7 @@ Note: You can specify both CPU and memory together, or specify only one (the oth
 			} else {
 				fmt.Fprintf(statusOutput, "🚀 Creating service '%s' (auto-generated name)...\n", createServiceName)
 			}
-			resp, err := cfg.Client.PostProjectsProjectIdServicesWithResponse(ctx, cfg.ProjectID, serviceCreateReq)
+			resp, err := cfg.Client.CreateServiceWithResponse(ctx, cfg.ProjectID, serviceCreateReq)
 			if err != nil {
 				return fmt.Errorf("failed to create Service: %w", err)
 			}
@@ -465,7 +465,7 @@ Examples:
 			defer cancel()
 
 			// Fetch service details
-			serviceResp, err := cfg.Client.GetProjectsProjectIdServicesServiceIdWithResponse(ctx, cfg.ProjectID, serviceID)
+			serviceResp, err := cfg.Client.GetServiceWithResponse(ctx, cfg.ProjectID, serviceID)
 			if err != nil {
 				return fmt.Errorf("failed to get service details: %w", err)
 			}
@@ -848,7 +848,7 @@ Examples:
 			}
 
 			// Make the delete request
-			resp, err := cfg.Client.DeleteProjectsProjectIdServicesServiceIdWithResponse(
+			resp, err := cfg.Client.DeleteServiceWithResponse(
 				cmd.Context(),
 				api.ProjectId(cfg.ProjectID),
 				api.ServiceId(serviceID),
@@ -940,7 +940,7 @@ Examples:
 			cmd.SilenceUsage = true
 
 			// Make the start request
-			resp, err := cfg.Client.PostProjectsProjectIdServicesServiceIdStartWithResponse(
+			resp, err := cfg.Client.StartServiceWithResponse(
 				context.Background(),
 				api.ProjectId(cfg.ProjectID),
 				api.ServiceId(serviceID),
@@ -1036,7 +1036,7 @@ Examples:
 			cmd.SilenceUsage = true
 
 			// Make the stop request
-			resp, err := cfg.Client.PostProjectsProjectIdServicesServiceIdStopWithResponse(
+			resp, err := cfg.Client.StopServiceWithResponse(
 				context.Background(),
 				api.ProjectId(cfg.ProjectID),
 				api.ServiceId(serviceID),
@@ -1250,7 +1250,7 @@ Examples:
 			}
 
 			// Make API call to fork service
-			forkResp, err := cfg.Client.PostProjectsProjectIdServicesServiceIdForkServiceWithResponse(ctx, cfg.ProjectID, serviceID, forkReq)
+			forkResp, err := cfg.Client.ForkServiceWithResponse(ctx, cfg.ProjectID, serviceID, forkReq)
 			if err != nil {
 				return fmt.Errorf("failed to fork Service: %w", err)
 			}
@@ -1365,7 +1365,7 @@ func listServices(cmd *cobra.Command) ([]api.Service, error) {
 	ctx, cancel := context.WithTimeout(cmd.Context(), 30*time.Second)
 	defer cancel()
 
-	resp, err := cfg.Client.GetProjectsProjectIdServicesWithResponse(ctx, cfg.ProjectID)
+	resp, err := cfg.Client.GetServicesWithResponse(ctx, cfg.ProjectID)
 	if err != nil {
 		return nil, fmt.Errorf("failed to list services: %w", err)
 	}
diff --git a/internal/tiger/common/wait.go b/internal/tiger/common/wait.go
@@ -20,7 +20,7 @@ type WaitHandler interface {
 	// continue. It also returns an error, which is either immediately returned
 	// from WaitForService or temporarily shown next to the spinner depending
 	// on the first return value.
-	Check(resp *api.GetProjectsProjectIdServicesServiceIdResponse) (bool, error)
+	Check(resp *api.GetServiceResponse) (bool, error)
 }
 
 type WaitForServiceArgs struct {
@@ -56,7 +56,7 @@ func WaitForService(ctx context.Context, args WaitForServiceArgs) error {
 				return fmt.Errorf("error waiting - %s: %w", args.TimeoutMsg, ctx.Err())
 			}
 		case <-ticker.C:
-			resp, err := args.Client.GetProjectsProjectIdServicesServiceIdWithResponse(ctx, args.ProjectID, args.ServiceID)
+			resp, err := args.Client.GetServiceWithResponse(ctx, args.ProjectID, args.ServiceID)
 			if err != nil {
 				spinner.Update(fmt.Sprintf("Error checking service status: %s", err))
 				continue
@@ -83,7 +83,7 @@ func (h *StatusWaitHandler) Message() string {
 	return fmt.Sprintf("Service status: %s", util.DerefStr(h.Service.Status))
 }
 
-func (h *StatusWaitHandler) Check(resp *api.GetProjectsProjectIdServicesServiceIdResponse) (bool, error) {
+func (h *StatusWaitHandler) Check(resp *api.GetServiceResponse) (bool, error) {
 	switch resp.StatusCode() {
 	case 200:
 		if resp.JSON200 == nil {
@@ -122,7 +122,7 @@ func (h *DeletionWaitHandler) Message() string {
 	return fmt.Sprintf("Waiting for service '%s' to be deleted", h.ServiceID)
 }
 
-func (h *DeletionWaitHandler) Check(resp *api.GetProjectsProjectIdServicesServiceIdResponse) (bool, error) {
+func (h *DeletionWaitHandler) Check(resp *api.GetServiceResponse) (bool, error) {
 	switch resp.StatusCode() {
 	case 200:
 		return false, nil
diff --git a/internal/tiger/mcp/db_tools.go b/internal/tiger/mcp/db_tools.go
@@ -145,7 +145,7 @@ func (s *Server) handleDBExecuteQuery(ctx context.Context, req *mcp.CallToolRequ
 	)
 
 	// Get service details to construct connection string
-	serviceResp, err := cfg.Client.GetProjectsProjectIdServicesServiceIdWithResponse(ctx, cfg.ProjectID, input.ServiceID)
+	serviceResp, err := cfg.Client.GetServiceWithResponse(ctx, cfg.ProjectID, input.ServiceID)
 	if err != nil {
 		return nil, DBExecuteQueryOutput{}, fmt.Errorf("failed to get service details: %w", err)
 	}
diff --git a/internal/tiger/mcp/service_tools.go b/internal/tiger/mcp/service_tools.go
@@ -467,7 +467,7 @@ func (s *Server) handleServiceList(ctx context.Context, req *mcp.CallToolRequest
 	ctx, cancel := context.WithTimeout(ctx, 30*time.Second)
 	defer cancel()
 
-	resp, err := cfg.Client.GetProjectsProjectIdServicesWithResponse(ctx, cfg.ProjectID)
+	resp, err := cfg.Client.GetServicesWithResponse(ctx, cfg.ProjectID)
 	if err != nil {
 		return nil, ServiceListOutput{}, fmt.Errorf("failed to list services: %w", err)
 	}
@@ -509,7 +509,7 @@ func (s *Server) handleServiceGet(ctx context.Context, req *mcp.CallToolRequest,
 	ctx, cancel := context.WithTimeout(ctx, 30*time.Second)
 	defer cancel()
 
-	resp, err := cfg.Client.GetProjectsProjectIdServicesServiceIdWithResponse(ctx, cfg.ProjectID, input.ServiceID)
+	resp, err := cfg.Client.GetServiceWithResponse(ctx, cfg.ProjectID, input.ServiceID)
 	if err != nil {
 		return nil, ServiceGetOutput{}, fmt.Errorf("failed to get service details: %w", err)
 	}
@@ -577,7 +577,7 @@ func (s *Server) handleServiceCreate(ctx context.Context, req *mcp.CallToolReque
 	createCtx, cancel := context.WithTimeout(ctx, 30*time.Second)
 	defer cancel()
 
-	resp, err := cfg.Client.PostProjectsProjectIdServicesWithResponse(createCtx, cfg.ProjectID, serviceCreateReq)
+	resp, err := cfg.Client.CreateServiceWithResponse(createCtx, cfg.ProjectID, serviceCreateReq)
 	if err != nil {
 		return nil, ServiceCreateOutput{}, fmt.Errorf("failed to create service: %w", err)
 	}
@@ -699,7 +699,7 @@ func (s *Server) handleServiceFork(ctx context.Context, req *mcp.CallToolRequest
 	forkCtx, cancel := context.WithTimeout(ctx, 30*time.Second)
 	defer cancel()
 
-	resp, err := cfg.Client.PostProjectsProjectIdServicesServiceIdForkServiceWithResponse(forkCtx, cfg.ProjectID, input.ServiceID, forkReq)
+	resp, err := cfg.Client.ForkServiceWithResponse(forkCtx, cfg.ProjectID, input.ServiceID, forkReq)
 	if err != nil {
 		return nil, ServiceForkOutput{}, fmt.Errorf("failed to fork service: %w", err)
 	}
@@ -786,7 +786,7 @@ func (s *Server) handleServiceUpdatePassword(ctx context.Context, req *mcp.CallT
 	ctx, cancel := context.WithTimeout(ctx, 30*time.Second)
 	defer cancel()
 
-	resp, err := cfg.Client.PostProjectsProjectIdServicesServiceIdUpdatePasswordWithResponse(ctx, cfg.ProjectID, input.ServiceID, updateReq)
+	resp, err := cfg.Client.UpdatePasswordWithResponse(ctx, cfg.ProjectID, input.ServiceID, updateReq)
 	if err != nil {
 		return nil, ServiceUpdatePasswordOutput{}, fmt.Errorf("failed to update service password: %w", err)
 	}
@@ -798,7 +798,7 @@ func (s *Server) handleServiceUpdatePassword(ctx context.Context, req *mcp.CallT
 
 	// Get service details for password storage (similar to CLI implementation)
 	var passwordStorage *common.PasswordStorageResult
-	serviceResp, err := cfg.Client.GetProjectsProjectIdServicesServiceIdWithResponse(ctx, cfg.ProjectID, input.ServiceID)
+	serviceResp, err := cfg.Client.GetServiceWithResponse(ctx, cfg.ProjectID, input.ServiceID)
 	if err == nil && serviceResp.StatusCode() == 200 && serviceResp.JSON200 != nil {
 		// Save the new password using the shared util function
 		result, err := common.SavePasswordWithResult(api.Service(*serviceResp.JSON200), input.Password, "tsdbadmin")
@@ -834,7 +834,7 @@ func (s *Server) handleServiceStart(ctx context.Context, req *mcp.CallToolReques
 	startCtx, cancel := context.WithTimeout(ctx, 30*time.Second)
 	defer cancel()
 
-	resp, err := cfg.Client.PostProjectsProjectIdServicesServiceIdStartWithResponse(startCtx, cfg.ProjectID, input.ServiceID)
+	resp, err := cfg.Client.StartServiceWithResponse(startCtx, cfg.ProjectID, input.ServiceID)
 	if err != nil {
 		return nil, ServiceStartOutput{}, fmt.Errorf("failed to start service: %w", err)
 	}
@@ -891,7 +891,7 @@ func (s *Server) handleServiceStop(ctx context.Context, req *mcp.CallToolRequest
 	stopCtx, cancel := context.WithTimeout(ctx, 30*time.Second)
 	defer cancel()
 
-	resp, err := cfg.Client.PostProjectsProjectIdServicesServiceIdStopWithResponse(stopCtx, cfg.ProjectID, input.ServiceID)
+	resp, err := cfg.Client.StopServiceWithResponse(stopCtx, cfg.ProjectID, input.ServiceID)
 	if err != nil {
 		return nil, ServiceStopOutput{}, fmt.Errorf("failed to stop service: %w", err)
 	}
diff --git a/openapi.yaml b/openapi.yaml

Original file line number	Diff line number	Diff line change
`@@ -781,7 +781,7 @@ func getServiceDetails(cmd cobra.Command, cfg common.Config, args []string) (a`
`781`	`781`	`ctx, cancel := context.WithTimeout(cmd.Context(), 30*time.Second)`
`782`	`782`	`defer cancel()`
`783`	`783`
`784`		`- resp, err := cfg.Client.GetProjectsProjectIdServicesServiceIdWithResponse(ctx, cfg.ProjectID, serviceID)`
	`784`	`+ resp, err := cfg.Client.GetServiceWithResponse(ctx, cfg.ProjectID, serviceID)`
`785`	`785`	`if err != nil {`
`786`	`786`	`return api.Service{}, fmt.Errorf("failed to fetch service details: %w", err)`
`787`	`787`	`}`
Original file line number	Diff line number	Diff line change
`@@ -244,7 +244,7 @@ func updateAndSaveServicePassword(`
`244`	`244`	`) error {`
`245`	`245`	`// Call API to update password`
`246`	`246`	`updateReq := api.UpdatePasswordInput{Password: newPassword}`
`247`		`- resp, err := client.PostProjectsProjectIdServicesServiceIdUpdatePasswordWithResponse(ctx, service.ProjectId, service.ServiceId, updateReq)`
	`247`	`+ resp, err := client.UpdatePasswordWithResponse(ctx, service.ProjectId, service.ServiceId, updateReq)`
`248`	`248`	`if err != nil {`
`249`	`249`	`return fmt.Errorf("failed to update password: %w", err)`
`250`	`250`	`}`
Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ func (s Server) handleDBExecuteQuery(ctx context.Context, req mcp.CallToolRequ`
`145`	`145`	`)`
`146`	`146`
`147`	`147`	`// Get service details to construct connection string`
`148`		`- serviceResp, err := cfg.Client.GetProjectsProjectIdServicesServiceIdWithResponse(ctx, cfg.ProjectID, input.ServiceID)`
	`148`	`+ serviceResp, err := cfg.Client.GetServiceWithResponse(ctx, cfg.ProjectID, input.ServiceID)`
`149`	`149`	`if err != nil {`
`150`	`150`	`return nil, DBExecuteQueryOutput{}, fmt.Errorf("failed to get service details: %w", err)`
`151`	`151`	`}`