make file write labeling consistent

Author: courtney-sims

apps/sandbox-container/README.md

@@ -48,9 +48,10 @@ npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14387504
 - `container_initialize`: (Re)start a container. Containers are intended to be ephemeral and don't save any state. Containers are only guaranteed to last 10m (this is just because I have a max of like ~5 containers per account).
 - `container_ping`: Ping a container for connectivity
 - `container_exec`: Run a command in the shell
-- `container_files_write`: Write to a file
+- `container_file_write`: Write to a file
 - `container_files_list`: List all files in the work directory
 - `container_file_read`: Read the contents of a single file or directory
+- `container_file_delete`: Delete a single file or directory
 
 ## Resources
 

apps/sandbox-container/container/index.ts

@@ -1,13 +1,13 @@
-import { exec } from 'node:child_process'
-import * as fs from 'node:fs/promises'
-import path from 'node:path'
 import { serve } from '@hono/node-server'
 import { zValidator } from '@hono/zod-validator'
 import { Hono } from 'hono'
 import { streamText } from 'hono/streaming'
 import mime from 'mime'
+import { exec } from 'node:child_process'
+import * as fs from 'node:fs/promises'
+import path from 'node:path'
 
-import { ExecParams, FilesWrite } from '../shared/schema.ts'
+import { ExecParams, FileWrite } from '../shared/schema.ts'
 import {
 	DIRECTORY_CONTENT_TYPE,
 	get_file_name_from_path,
@@ -96,7 +96,7 @@ app.get('/files/contents/*', async (c) => {
  *
  * Create or update file contents
  */
-app.post('/files/contents', zValidator('json', FilesWrite), async (c) => {
+app.post('/files/contents', zValidator('json', FileWrite), async (c) => {
 	const file = c.req.valid('json')
 	const reqPath = await get_file_name_from_path(file.path)
 

apps/sandbox-container/server/containerMcp.ts

@@ -2,14 +2,14 @@ import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js'
 import { McpAgent } from 'agents/mcp'
 
 import { OPEN_CONTAINER_PORT } from '../shared/consts'
-import { ExecParams, FilePathParam, FilesWrite } from '../shared/schema'
+import { ExecParams, FilePathParam, FileWrite } from '../shared/schema'
 import { MAX_CONTAINERS, proxyFetch, startAndWaitForPort } from './containerHelpers'
 import { getContainerManager } from './containerManager'
 import { BASE_INSTRUCTIONS } from './prompts'
 import { fileToBase64, stripProtocolFromFilePath } from './utils'
 
-import type { FileList } from '../shared/schema'
 import type { Env, Props } from '.'
+import type { FileList } from '../shared/schema'
 
 export class ContainerMcpAgent extends McpAgent<Env, Props> {
 	server = new McpServer(
@@ -84,13 +84,13 @@ export class ContainerMcpAgent extends McpAgent<Env, Props> {
 			}
 		)
 		this.server.tool(
-			'container_files_write',
+			'container_file_write',
 			'Write file contents',
-			{ args: FilesWrite },
+			{ args: FileWrite },
 			async ({ args }) => {
 				args.path = await stripProtocolFromFilePath(args.path)
 				return {
-					content: [{ type: 'text', text: await this.container_files_write(args) }],
+					content: [{ type: 'text', text: await this.container_file_write(args) }],
 				}
 			}
 		)
@@ -266,7 +266,7 @@ export class ContainerMcpAgent extends McpAgent<Env, Props> {
 		}
 	}
 
-	async container_files_write(file: FilesWrite): Promise<string> {
+	async container_file_write(file: FileWrite): Promise<string> {
 		const res = await proxyFetch(
 			this.env.ENVIRONMENT,
 			this.ctx.container,

apps/sandbox-container/server/prompts.ts

@@ -48,7 +48,7 @@ AVOID manually reading or writing files using the \`container_exec\` tool. You s
 To manage container lifecycle, use the \`container_start\` and \`container_kill\` tools. If you run into errors where you can't connect to the container, attempt to kill and restart the container. If that doesn't work, the system is probably overloaded.
 
 You can execute actions in the container using the \`container_exec\` tool. By default, stdout is returned back as a string.
-To write a file, use the \`container_files_write\` tool.
+To write a file, use the \`container_file_write\` tool. To delete a file, use the \`container_file_delete\` tool.
 
 The \`container_files_list\` allows you to list file resources. Content is omitted from the response of this tool and all mimeTypes are \`text/plain\` even if the file ending suggests otherwise.
 If you want to get the file contents of a file resource, use \`container_file_read\`, which will return the file contents.

apps/sandbox-container/shared/schema.ts

@@ -7,8 +7,8 @@ export const ExecParams = z.object({
 	streamStderr: z.boolean().default(true),
 })
 
-export type FilesWrite = z.infer<typeof FilesWrite>
-export const FilesWrite = z.object({
+export type FileWrite = z.infer<typeof FileWrite>
+export const FileWrite = z.object({
 	path: z.string(),
 	text: z.string().describe('Full text content of the file you want to write.'),
 })