@projectsolo/solo-mission-mcp 0.22.1 → 0.23.0

package/README.md

@@ -160,7 +160,7 @@ Free off-chain missions need no wallet. To fund or refund a paid mission, point
 
 ## Tools Reference
 
-34 tools in total.
+33 tools in total.
 
 ### Agent (1 tool)
 
@@ -219,14 +219,15 @@ Free off-chain missions need no wallet. To fund or refund a paid mission, point
 
 **Limitation:** The buffers are in-memory only. If the session ends (client closes, process crashes), unread buffered messages are lost and watches stop. To recover, call `get_messages` with a manual `since` timestamp and re-watch.
 
-### Media tracks (4 tools, `media_review` missions)
+### Media tracks (3 tools, `media_review` missions)
 
 | Tool | Description |
 |---|---|
 | `add_mission_track` | Upload an audio, image or video track, via `file_path` (use this for audio/video) or `file_base64` |
 | `list_mission_tracks` | List a mission's tracks with `vote_counts` and `total_listen_seconds` |
 | `get_track_ratings` | Per-participant star ratings and comments for one track |
-| `delete_mission_track` | Delete a track that has no ratings yet |
+
+Tracks can't be deleted or replaced. To change the media, cancel the mission and create a new one: `cancel_mission` for an off-chain or unfunded (`pending_funding`) mission, or `refund_solana_mission` with `action: 'cancel'` for a funded one before its `qualify_deadline`. If an upload fails partway, retry `add_mission_track`: an unconfirmed track is ignored (not listed, not counted toward the 20-track cap or completion).
 
 ### Solana (4 tools)
 

package/dist/index.js

@@ -1,6 +1,5 @@
 #!/usr/bin/env node
 import {
-  apiDelete,
   apiGet,
   apiPatch,
   apiPost,
@@ -789,7 +788,7 @@ import { readFile } from "fs/promises";
 var trackTools = [
   {
     name: "add_mission_track",
-    description: "Upload a media item (audio, image, or video) to a media_review mission. Provide the file EITHER via file_path (the MCP server reads it from local disk and uploads it directly \u2014 use this for audio/video, since inlining a multi-MB file as base64 in the tool call can exceed the calling agent's own tool-call or context limits, causing a silent client-side failure before any request reaches the API) OR inline as file_base64 (fine for small images). Exactly one of the two is required. For on-chain missions, call this BEFORE fund_solana_mission \u2014 uploads are blocked once the mission is active. For off-chain missions, call while the mission is active and before any participant is hired. The item becomes visible to hired participants once confirmed.",
+    description: "Upload a media item (audio, image, or video) to a media_review mission. Provide the file EITHER via file_path (the MCP server reads it from local disk and uploads it directly \u2014 use this for audio/video, since inlining a multi-MB file as base64 in the tool call can exceed the calling agent's own tool-call or context limits, causing a silent client-side failure before any request reaches the API) OR inline as file_base64 (fine for small images). Exactly one of the two is required. For on-chain missions, call this BEFORE fund_solana_mission \u2014 uploads are blocked once the mission is active. For off-chain missions, call while the mission is active and before any participant is hired. The item becomes visible to hired participants once confirmed. Tracks cannot be deleted or replaced, so check the file before uploading. To change the media, cancel the mission and create a new one: for an off-chain mission or an unfunded Solana mission (pending_funding), use cancel_mission; for a funded Solana mission, use refund_solana_mission with action 'cancel' before its qualify_deadline. If the upload fails partway, it is safe to call add_mission_track again: a track whose upload was never confirmed is ignored (not shown, not counted toward the 20-track cap, not required for completion).",
     inputSchema: {
       type: "object",
       properties: {
@@ -836,18 +835,6 @@ var trackTools = [
       },
       required: ["mission_id", "track_id"]
     }
-  },
-  {
-    name: "delete_mission_track",
-    description: "Delete a track from a mission. Only allowed if the track has received no ratings yet.",
-    inputSchema: {
-      type: "object",
-      properties: {
-        mission_id: { type: "string", description: "ID of the mission" },
-        track_id: { type: "string", description: "ID of the track to delete" }
-      },
-      required: ["mission_id", "track_id"]
-    }
   }
 ];
 async function handleTrackTool(name, args) {
@@ -881,9 +868,9 @@ async function handleTrackTool(name, args) {
       });
       if (!uploadRes.ok) {
         const errText = await uploadRes.text().catch(() => "");
-        await apiDelete(`/agent/missions/${mission_id}/tracks/${urlRes.track_id}`).catch(() => {
-        });
-        throw new Error(`Media upload failed: ${uploadRes.status} ${errText}`);
+        throw new Error(
+          `Media upload failed: ${uploadRes.status} ${errText}. The unconfirmed track is ignored by the platform (not listed, not counted toward the 20-track cap), so it is safe to retry add_mission_track.`
+        );
       }
       const confirmed = await apiPost(
         `/agent/missions/${mission_id}/tracks/${urlRes.track_id}/confirm`,
@@ -899,10 +886,6 @@ async function handleTrackTool(name, args) {
       const { mission_id, track_id } = args;
       return apiGet(`/agent/missions/${mission_id}/tracks/${track_id}/ratings`);
     }
-    case "delete_mission_track": {
-      const { mission_id, track_id } = args;
-      return apiDelete(`/agent/missions/${mission_id}/tracks/${track_id}`);
-    }
     default:
       throw new Error(`Unknown track tool: ${name}`);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@projectsolo/solo-mission-mcp",
-  "version": "0.22.1",
+  "version": "0.23.0",
   "description": "MCP server for SoloMission Platform — lets AI agents create missions, browse humans, and chat.",
   "type": "module",
   "main": "dist/index.js",

package/src/tools/tracks.test.ts

@@ -22,7 +22,7 @@ vi.mock('../api/client.js', () => ({
   apiDelete: (...args: unknown[]) => apiDelete(...args),
 }));
 
-import { handleTrackTool } from './tracks.js';
+import { handleTrackTool, trackTools } from './tracks.js';
 
 describe('add_mission_track', () => {
   let tmpDir: string;
@@ -131,9 +131,8 @@ describe('add_mission_track', () => {
     expect(fetchMock).not.toHaveBeenCalled();
   });
 
-  it('deletes the pending track doc when the signed-URL PUT fails', async () => {
+  it('does not try to delete the pending track when the signed-URL PUT fails, and says a retry is safe', async () => {
     mockHappyPathApi();
-    apiDelete.mockResolvedValue(undefined);
     fetchMock.mockResolvedValue({ ok: false, status: 500, text: async () => 'boom' });
     const filePath = join(tmpDir, 'song.mp3');
     writeFileSync(filePath, 'bytes');
@@ -145,7 +144,19 @@ describe('add_mission_track', () => {
         content_type: 'audio/mpeg',
         file_path: filePath,
       }),
-    ).rejects.toThrow(/Media upload failed/);
-    expect(apiDelete).toHaveBeenCalledWith('/agent/missions/m1/tracks/trk_1');
+    ).rejects.toThrow(/Media upload failed: 500 boom.*safe to retry add_mission_track/);
+    // The backend's DELETE track route is gone; an unconfirmed track is ignored server-side.
+    expect(apiDelete).not.toHaveBeenCalled();
+    // No confirm call for a failed upload.
+    expect(apiPost).toHaveBeenCalledTimes(1);
+  });
+});
+
+describe('track tool set', () => {
+  it('no longer exposes delete_mission_track', async () => {
+    expect(trackTools.map((t) => t.name)).not.toContain('delete_mission_track');
+    await expect(handleTrackTool('delete_mission_track', { mission_id: 'm1', track_id: 't1' })).rejects.toThrow(
+      /Unknown track tool/,
+    );
   });
 });

package/src/tools/tracks.ts

@@ -1,6 +1,6 @@
 import { readFile } from 'node:fs/promises';
 import { Tool } from '@modelcontextprotocol/sdk/types.js';
-import { apiGet, apiPost, apiDelete } from '../api/client.js';
+import { apiGet, apiPost } from '../api/client.js';
 
 export const trackTools: Tool[] = [
   {
@@ -13,7 +13,13 @@ export const trackTools: Tool[] = [
       'small images). Exactly one of the two is required. ' +
       'For on-chain missions, call this BEFORE fund_solana_mission — uploads are blocked once the mission is active. ' +
       'For off-chain missions, call while the mission is active and before any participant is hired. ' +
-      'The item becomes visible to hired participants once confirmed.',
+      'The item becomes visible to hired participants once confirmed. ' +
+      'Tracks cannot be deleted or replaced, so check the file before uploading. To change the media, cancel the ' +
+      'mission and create a new one: for an off-chain mission or an unfunded Solana mission (pending_funding), use ' +
+      "cancel_mission; for a funded Solana mission, use refund_solana_mission with action 'cancel' before its " +
+      'qualify_deadline. ' +
+      'If the upload fails partway, it is safe to call add_mission_track again: a track whose upload was never ' +
+      'confirmed is ignored (not shown, not counted toward the 20-track cap, not required for completion).',
     inputSchema: {
       type: 'object',
       properties: {
@@ -78,18 +84,6 @@ export const trackTools: Tool[] = [
       required: ['mission_id', 'track_id'],
     },
   },
-  {
-    name: 'delete_mission_track',
-    description: 'Delete a track from a mission. Only allowed if the track has received no ratings yet.',
-    inputSchema: {
-      type: 'object',
-      properties: {
-        mission_id: { type: 'string', description: 'ID of the mission' },
-        track_id: { type: 'string', description: 'ID of the track to delete' },
-      },
-      required: ['mission_id', 'track_id'],
-    },
-  },
 ];
 
 export async function handleTrackTool(name: string, args: Record<string, any>): Promise<unknown> {
@@ -125,8 +119,9 @@ export async function handleTrackTool(name: string, args: Record<string, any>):
       );
 
       // Step 2: Upload binary to the signed URL.
-      // On failure, delete the pending track doc so it doesn't consume the 20-track
-      // slot or appear as a ghost in list results.
+      // A failed upload leaves the track doc pending (unconfirmed). The backend ignores
+      // pending tracks: they are not listed, not counted toward the 20-track cap and not
+      // required for completion. Tracks cannot be deleted, so there is nothing to clean up.
       const uploadRes = await fetch(urlRes.upload_url, {
         method: 'PUT',
         headers: { 'Content-Type': content_type },
@@ -134,9 +129,11 @@ export async function handleTrackTool(name: string, args: Record<string, any>):
       });
       if (!uploadRes.ok) {
         const errText = await uploadRes.text().catch(() => '');
-        // Best-effort cleanup — ignore errors so the original failure is surfaced
-        await apiDelete(`/agent/missions/${mission_id}/tracks/${urlRes.track_id}`).catch(() => {});
-        throw new Error(`Media upload failed: ${uploadRes.status} ${errText}`);
+        throw new Error(
+          `Media upload failed: ${uploadRes.status} ${errText}. ` +
+            'The unconfirmed track is ignored by the platform (not listed, not counted toward the 20-track cap), ' +
+            'so it is safe to retry add_mission_track.',
+        );
       }
 
       // Step 3: Confirm upload and optionally set duration
@@ -158,11 +155,6 @@ export async function handleTrackTool(name: string, args: Record<string, any>):
       return apiGet(`/agent/missions/${mission_id}/tracks/${track_id}/ratings`);
     }
 
-    case 'delete_mission_track': {
-      const { mission_id, track_id } = args;
-      return apiDelete(`/agent/missions/${mission_id}/tracks/${track_id}`);
-    }
-
     default:
       throw new Error(`Unknown track tool: ${name}`);
   }