fix: return success response from void operations (AI-164)

- Add SuccessResponse type for operations that don't return data
- Update all void operations to return { success: true }
- Affected operations: pauseProject, restoreProject, deleteBranch,
  mergeBranch, resetBranch, rebaseBranch, applyMigration, updateStorageConfig
- Prevents LLMs from thinking operations failed and attempting retries

Fixes AI-164

Author: Rodriguespn

packages/mcp-server-supabase/src/platform/api-platform.ts

@@ -29,6 +29,7 @@ import {
   type DebuggingOperations,
   type DeployEdgeFunctionOptions,
   type DevelopmentOperations,
+  type SuccessResponse,
   type EdgeFunction,
   type EdgeFunctionsOperations,
   type EdgeFunctionWithBody,
@@ -42,6 +43,8 @@ import {
 
 const { version } = packageJson;
 
+const SUCCESS_RESPONSE: SuccessResponse = { success: true };
+
 export type SupabaseApiPlatformOptions = {
   /**
    * The access token for the Supabase Management API.
@@ -148,6 +151,8 @@ export function createSupabaseApiPlatform(
       );
 
       assertSuccess(response, 'Failed to pause project');
+
+      return SUCCESS_RESPONSE;
     },
     async restoreProject(projectId: string) {
       const response = await managementApiClient.POST(
@@ -162,6 +167,8 @@ export function createSupabaseApiPlatform(
       );
 
       assertSuccess(response, 'Failed to restore project');
+
+      return SUCCESS_RESPONSE;
     },
   };
 
@@ -229,6 +236,7 @@ export function createSupabaseApiPlatform(
       // Intentionally don't return the result of the migration
       // to avoid prompt injection attacks. If the migration failed,
       // it will throw an error.
+      return SUCCESS_RESPONSE;
     },
   };
 
@@ -612,6 +620,8 @@ export function createSupabaseApiPlatform(
       );
 
       assertSuccess(response, 'Failed to delete branch');
+
+      return SUCCESS_RESPONSE;
     },
     async mergeBranch(branchId: string) {
       const response = await managementApiClient.POST(
@@ -627,6 +637,8 @@ export function createSupabaseApiPlatform(
       );
 
       assertSuccess(response, 'Failed to merge branch');
+
+      return SUCCESS_RESPONSE;
     },
     async resetBranch(branchId: string, options: ResetBranchOptions) {
       const { migration_version } = resetBranchOptionsSchema.parse(options);
@@ -646,6 +658,8 @@ export function createSupabaseApiPlatform(
       );
 
       assertSuccess(response, 'Failed to reset branch');
+
+      return SUCCESS_RESPONSE;
     },
     async rebaseBranch(branchId: string) {
       const response = await managementApiClient.POST(
@@ -661,6 +675,8 @@ export function createSupabaseApiPlatform(
       );
 
       assertSuccess(response, 'Failed to rebase branch');
+
+      return SUCCESS_RESPONSE;
     },
   };
 
@@ -725,7 +741,7 @@ export function createSupabaseApiPlatform(
 
       assertSuccess(response, 'Failed to update storage config');
 
-      return response.data;
+      return SUCCESS_RESPONSE;
     },
   };
 

packages/mcp-server-supabase/src/platform/types.ts

@@ -2,6 +2,10 @@ import type { InitData } from '@supabase/mcp-utils';
 import { z } from 'zod';
 import { AWS_REGION_CODES } from '../regions.js';
 
+export type SuccessResponse = {
+  success: true;
+};
+
 export const storageBucketSchema = z.object({
   id: z.string(),
   name: z.string(),
@@ -177,7 +181,7 @@ export type DatabaseOperations = {
   applyMigration(
     projectId: string,
     options: ApplyMigrationOptions
-  ): Promise<void>;
+  ): Promise<SuccessResponse>;
 };
 
 export type AccountOperations = {
@@ -186,8 +190,8 @@ export type AccountOperations = {
   listProjects(): Promise<Project[]>;
   getProject(projectId: string): Promise<Project>;
   createProject(options: CreateProjectOptions): Promise<Project>;
-  pauseProject(projectId: string): Promise<void>;
-  restoreProject(projectId: string): Promise<void>;
+  pauseProject(projectId: string): Promise<SuccessResponse>;
+  restoreProject(projectId: string): Promise<SuccessResponse>;
 };
 
 export type EdgeFunctionsOperations = {
@@ -218,7 +222,10 @@ export type DevelopmentOperations = {
 
 export type StorageOperations = {
   getStorageConfig(projectId: string): Promise<StorageConfig>;
-  updateStorageConfig(projectId: string, config: StorageConfig): Promise<void>;
+  updateStorageConfig(
+    projectId: string,
+    config: StorageConfig
+  ): Promise<SuccessResponse>;
   listAllBuckets(projectId: string): Promise<StorageBucket[]>;
 };
 
@@ -228,10 +235,13 @@ export type BranchingOperations = {
     projectId: string,
     options: CreateBranchOptions
   ): Promise<Branch>;
-  deleteBranch(branchId: string): Promise<void>;
-  mergeBranch(branchId: string): Promise<void>;
-  resetBranch(branchId: string, options: ResetBranchOptions): Promise<void>;
-  rebaseBranch(branchId: string): Promise<void>;
+  deleteBranch(branchId: string): Promise<SuccessResponse>;
+  mergeBranch(branchId: string): Promise<SuccessResponse>;
+  resetBranch(
+    branchId: string,
+    options: ResetBranchOptions
+  ): Promise<SuccessResponse>;
+  rebaseBranch(branchId: string): Promise<SuccessResponse>;
 };
 
 export type SupabasePlatform = {