From 5a3987bd9f2120e04095067ffc825d36dec698ca Mon Sep 17 00:00:00 2001 From: saravanakumardb1 Date: Thu, 5 Mar 2026 14:06:19 -0800 Subject: [PATCH] fix(mcp-server): jarvis.marketplace.certify routes to wrong endpoint on rejection MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The backend has two separate endpoints: POST /marketplace/admin/:id/approve POST /marketplace/admin/:id/reject Previously jarvisMarketplaceCertify always called /approve regardless of the decision field — rejections would silently approve listings. Fix: derive action from decision.decision ('approved' → 'approve', else 'reject') and use the correct endpoint path. --- services/mcp-server/src/lib/jarvis-client.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/services/mcp-server/src/lib/jarvis-client.ts b/services/mcp-server/src/lib/jarvis-client.ts index 51fbb84a..5f9cbe5e 100644 --- a/services/mcp-server/src/lib/jarvis-client.ts +++ b/services/mcp-server/src/lib/jarvis-client.ts @@ -194,8 +194,9 @@ export function jarvisMarketplaceCertify( decision: { decision: 'approved' | 'rejected'; notes?: string }, opts: JarvisClientOptions ): Promise { + const action = decision.decision === 'approved' ? 'approve' : 'reject'; return jarvisFetch( - `/marketplace/admin/${listingId}/approve`, + `/marketplace/admin/${listingId}/${action}`, { method: 'POST', body: JSON.stringify(decision) }, opts );