fix: use snake_case for model-facing tool inputs

AGENTS.md

@@ -22,7 +22,7 @@ These ideas should stay true as the project evolves:
 - **Host** — the MCP client presenting the agent experience and coordinating work.
 - **Server** — the local DevSpace MCP server.
 - **Workspace** — one opened directory or worktree and its accumulated instruction context.
-- **`workspaceId`** — the opaque handle returned by `open_workspace` and reused for calls in that workspace.
+- **`workspace_id`** — the opaque handle returned by `open_workspace` and reused for calls in that workspace.
 - **Allowed root** — a configured filesystem boundary within which a workspace may be opened. It is not itself necessarily a workspace.
 - **Checkout mode** — operating on an existing checkout supplied by the user.
 - **Worktree mode** — operating in an isolated Git worktree.
@@ -115,4 +115,4 @@ Start at the boundary named by the problem and follow the data. Keep policy in D
 - Preserve host and provider data unless DevSpace has a concrete reason to normalize it.
 - Add compatibility behavior only for an identified consumer with a real upgrade path.
 - Reuse glossary terms in schemas, types, documentation, and errors.
-- Keep the execution layer small, reliable, and unsurprising.
+- Keep the execution layer small, reliable, and unsurprising.

docs/artifact-exchange.md

@@ -8,19 +8,19 @@ directly into an open workspace. Enable the tool with
 
 ```text
 open_workspace
-  -> download_artifact({ file, workspaceId, path })
+  -> download_artifact({ file, workspace_id, path })
   -> { path }
 ```
 
 1. Open the project with `open_workspace`.
-2. Pass the host-provided native `file`, the returned `workspaceId`, and an
+2. Pass the host-provided native `file`, the returned `workspace_id`, and an
    unused workspace-relative `path` to `download_artifact`.
 3. Use the returned path with the ordinary DevSpace filesystem tools.
 
 ```text
 download_artifact({
   file: <native file value supplied by the MCP host>,
-  workspaceId: "ws_123",
+  workspace_id: "ws_123",
   path: "public/images/generated-image.png"
 })
 ```

docs/chatgpt-coding-workflow.md

@@ -14,18 +14,18 @@ ChatGPT should call `open_workspace` once for a project folder:
 }
 ```
 
-The result includes a `workspaceId`. All later file, search, edit, show-changes,
-and shell calls should reuse that same `workspaceId`.
+The result includes a `workspace_id`. All later file, search, edit, show-changes,
+and shell calls should reuse that same `workspace_id`.
 
 ChatGPT may support automatic checkout recovery through optional host
 conversation metadata. This is an OpenAI-host adapter detail, not a standard MCP
 conversation field. When that optional context is available, opening the same
 checkout project again in the same conversation can continue in the existing
 workspace, and the context already provided for that reused checkout is not
-repeated. The portable workflow remains the same: keep using the `workspaceId`
+repeated. The portable workflow remains the same: keep using the `workspace_id`
 returned by `open_workspace` for later operations. Hosts without supported
 conversation context receive a normal new workspace and continue with that
-explicit `workspaceId` workflow.
+explicit `workspace_id` workflow.
 The model receives actionable workspace instructions; automatic-reuse
 bookkeeping is not a model-facing choice.
 
@@ -43,7 +43,7 @@ own context.
 
 Do not call `open_workspace` again for the same checkout folder unless:
 
-- the `workspaceId` is rejected as unknown
+- the `workspace_id` is rejected as unknown
 - work moves to a different project folder
 - work switches between checkout and worktree mode
 - the user asks for a new isolated worktree
@@ -78,10 +78,10 @@ Managed worktrees are created under:
 ```
 
 Worktree mode requires a Git repository with at least one commit. It starts from
-`HEAD` unless `baseRef` is provided.
+`HEAD` unless `base_ref` is provided.
 
 Each worktree-mode call creates a new managed worktree and returns a new
-`workspaceId`. Reuse that ID for work inside that worktree; call
+`workspace_id`. Reuse that ID for work inside that worktree; call
 `open_workspace` in worktree mode again only when another isolated worktree is
 actually required.
 

docs/gotchas.md

@@ -137,19 +137,19 @@ To regenerate setup:
 npx @waishnav/devspace init --force
 ```
 
-## Unknown `workspaceId`
+## Unknown `workspace_id`
 
-`workspaceId` values are session identifiers. If the server restarts and the
+`workspace_id` values are session identifiers. If the server restarts and the
 client receives an unknown workspace error, call `open_workspace` again for that
 project.
 
 Workspace session metadata is persisted. ChatGPT may provide optional
 conversation metadata that lets DevSpace resume the same checkout workspace for
-the same project in that conversation; repeated opens reuse the `workspaceId`
+the same project in that conversation; repeated opens reuse the `workspace_id`
 and do not repeat context already provided for that reused checkout. Worktree
 mode always creates a new isolated workspace with its own complete context.
 Hosts without supported conversation metadata receive a normal new workspace.
-In all cases, continue passing the `workspaceId` returned by `open_workspace` to
+In all cases, continue passing the `workspace_id` returned by `open_workspace` to
 later tools. Other MCP hosts use this explicit workspace workflow as well.
 
 To review work, call `show_changes` once after the final related file change. It
@@ -186,7 +186,7 @@ Worktree mode requires:
 - Git installed
 - the path is inside a Git repository
 - the repository has at least one commit
-- the requested `baseRef` resolves to a commit
+- the requested `base_ref` resolves to a commit
 
 For a new repository, create the first commit or use checkout mode.
 

docs/security.md

@@ -96,7 +96,7 @@ sessions.
 
 Native file download is an opt-in, one-shot transfer into an already-open
 workspace. `download_artifact` accepts the MCP host's native file value, the
-`workspaceId` returned by `open_workspace`, and an unused relative destination
+`workspace_id` returned by `open_workspace`, and an unused relative destination
 path. It returns only the workspace-relative path and does not create a
 persistent artifact service or reusable artifact ID.
 

src/artifact-download.test.ts

@@ -74,7 +74,7 @@ function testOneToolContract(): void {
   const descriptor = registered.get("download_artifact")?.descriptor;
   assert.ok(descriptor);
   assert.deepEqual(descriptor._meta, { "openai/fileParams": ["file"] });
-  assert.deepEqual(Object.keys(descriptor.inputSchema as object).sort(), ["file", "path", "workspaceId"]);
+  assert.deepEqual(Object.keys(descriptor.inputSchema as object).sort(), ["file", "path", "workspace_id"]);
   assert.deepEqual(Object.keys(descriptor.outputSchema as object), ["path"]);
   assert.equal((descriptor.annotations as { destructiveHint?: boolean }).destructiveHint, false);
 
@@ -353,7 +353,7 @@ function testLogRedaction(): void {
       file_name: "generated.png",
       authorization: "Bearer log-secret",
     },
-    workspaceId: "ws_secret",
+    workspace_id: "ws_secret",
     path: "private/generated.png",
   });
   const serialized = JSON.stringify(fields);

src/artifact-tools.ts

@@ -103,8 +103,8 @@ export function registerArtifactTools(
         file: openAIFileReferenceInputSchema.describe(
           "Native file value authorized and supplied by the MCP host.",
         ),
-        workspaceId: z.string().min(1).describe(
-          "Workspace to use. Reuse the current project's workspaceId.",
+        workspace_id: z.string().min(1).describe(
+          "Workspace to use. Reuse the current project's workspace_id.",
         ),
         path: z.string().min(1).describe(
           "Relative destination path inside the selected workspace. The destination must not already exist.",
@@ -117,7 +117,7 @@ export function registerArtifactTools(
       annotations: ARTIFACT_WRITE_ANNOTATIONS,
     },
     async (input) => executeArtifactTool(config, input, async () => {
-      const workspace = workspaces.getWorkspace(input.workspaceId);
+      const workspace = workspaces.getWorkspace(input.workspace_id);
       const downloaded = await downloadIncomingArtifact({
         registry: incomingRegistry,
         workspaceId: workspace.id,
@@ -292,7 +292,7 @@ export function artifactToolLogFields(
     fileProvided: input.file !== undefined,
     fileReferenceShape: describeIncomingArtifactValue(input.file),
     downloadUrlHostname: incomingFileDownloadHostname(input.file),
-    workspaceId: input.workspaceId,
+    workspaceId: input.workspace_id,
     path: input.path,
   };
 }