From 2eecf9eae0d0c38770aca41f36d8998f251b2ce0 Mon Sep 17 00:00:00 2001
From: Zach Leventer <zleventer@gmail.com>
Date: Fri, 24 Apr 2026 09:35:31 -0400
Subject: [PATCH] fix: correct .default() misuse, API payload typo, and wrong
 tool name reference
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- run_apex_test.ts: change testRunId from .default() to .describe() —
  the description string was being used as the default value, meaning
  if no testRunId was provided, it would default to the literal string
  "an id of an in-progress, or completed apex test run"

- create_org_snapshot.ts: fix 'metadatadata' → 'metadata' in the
  OrgSnapshot.create() API payload (Content field)

- sfDevopsListWorkItems.ts: fix tool name reference from
  'sf-list-all-orgs' → 'list_all_orgs' to match the actual registered
  tool name, preventing agents from calling a nonexistent tool
---
 packages/mcp-provider-devops/src/tools/sfDevopsListWorkItems.ts | 2 +-
 packages/mcp-provider-dx-core/src/tools/create_org_snapshot.ts  | 2 +-
 packages/mcp-provider-dx-core/src/tools/run_apex_test.ts        | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/mcp-provider-devops/src/tools/sfDevopsListWorkItems.ts b/packages/mcp-provider-devops/src/tools/sfDevopsListWorkItems.ts
index ecf35a43..bbd38d88 100644
--- a/packages/mcp-provider-devops/src/tools/sfDevopsListWorkItems.ts
+++ b/packages/mcp-provider-devops/src/tools/sfDevopsListWorkItems.ts
@@ -47,7 +47,7 @@ export class SfDevopsListWorkItems extends McpTool<InputArgsShape, OutputArgsSha
 **Input:**
 - Either username (example devops-center@example.com) or alias (example myDevOpsOrg) is required.
       
-      **MANDATORY:** If the DevOps Center org is not given, use the 'sf-list-all-orgs' tool to list all orgs. 
+      **MANDATORY:** If the DevOps Center org is not given, use the 'list_all_orgs' tool to list all orgs. 
       The list will indicate which org is DevOps Center, or Sandbox if possible. If these details are not provided in the list, 
       ask the user to specify which org is DevOps Center org. Only proceed after the user has selected the DevOps Center org.
 
diff --git a/packages/mcp-provider-dx-core/src/tools/create_org_snapshot.ts b/packages/mcp-provider-dx-core/src/tools/create_org_snapshot.ts
index 1147fb73..f77ebeef 100644
--- a/packages/mcp-provider-dx-core/src/tools/create_org_snapshot.ts
+++ b/packages/mcp-provider-dx-core/src/tools/create_org_snapshot.ts
@@ -96,7 +96,7 @@ create a snapshot of my MyScratch in myDevHub`,
         SourceOrg: sourceOrgId,
         Description: input.description,
         SnapshotName: input.name,
-        Content: 'metadatadata',
+        Content: 'metadata',
       });
 
       if (createResponse.success === false) {
diff --git a/packages/mcp-provider-dx-core/src/tools/run_apex_test.ts b/packages/mcp-provider-dx-core/src/tools/run_apex_test.ts
index 0f429d2e..ce381738 100644
--- a/packages/mcp-provider-dx-core/src/tools/run_apex_test.ts
+++ b/packages/mcp-provider-dx-core/src/tools/run_apex_test.ts
@@ -70,7 +70,7 @@ RunSpecifiedTests="Run the Apex tests I specify, these will be specified in the
       'Weather to wait for the test to finish (false) or enque the Apex tests and return the test run id (true)',
     ),
   suiteName: z.string().describe('a suite of apex test classes to run').optional(),
-  testRunId: z.string().default('an id of an in-progress, or completed apex test run').optional(),
+  testRunId: z.string().describe('an id of an in-progress, or completed apex test run').optional(),
   verbose: z
     .boolean()
     .default(false)