refactor(filesystem): rename write_or_update_file to create_or_append_file

Renames the tool, schema, and underlying function to accurately describe
the behavior (create-or-append, not update/overwrite) per review feedback.

- Tool: write_or_update_file -> create_or_append_file
- Schema: WriteOrUpdateFileArgsSchema -> CreateOrAppendFileArgsSchema
- Function: writeOrUpdateFileContent -> createOrAppendFileContent
- README updated; tests updated; append_file description updated.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: agn-7

src/filesystem/README.md

@@ -102,7 +102,7 @@ The server's directory access control follows this flow:
   - File must already exist - use `write_file` to create new files
   - Preserves existing content, adds new content at the end
 
-- **write_or_update_file**
+- **create_or_append_file**
   - Create new file or append to existing file
   - Inputs:
     - `path` (string): File location
@@ -219,7 +219,7 @@ The mapping for filesystem tools is:
 | `create_directory`          | `false`      | `true`         | `false`         | Re‑creating the same dir is a no‑op             |
 | `write_file`                | `false`      | `true`         | `true`          | Overwrites existing files                       |
 | `append_file`               | `false`      | `false`        | `false`         | Appends to existing files; not idempotent       |
-| `write_or_update_file`      | `false`      | `false`        | `false`         | Creates or appends; behavior depends on state   |
+| `create_or_append_file`     | `false`      | `false`        | `false`         | Creates or appends; behavior depends on state   |
 | `edit_file`                 | `false`      | `false`        | `true`          | Re‑applying edits can fail or double‑apply      |
 | `move_file`                 | `false`      | `false`        | `true`          | Deletes source file                             |
 

src/filesystem/__tests__/lib.test.ts

@@ -15,7 +15,7 @@ import {
   readFileContent,
   writeFileContent,
   appendFileContent,
-  writeOrUpdateFileContent,
+  createOrAppendFileContent,
   // Search & filtering functions
   searchFilesWithValidation,
   // File editing functions
@@ -344,14 +344,14 @@ describe('Lib Functions', () => {
       });
     });
 
-    describe('writeOrUpdateFileContent', () => {
+    describe('createOrAppendFileContent', () => {
       it('creates new file if it does not exist', async () => {
         const error = new Error('ENOENT');
         (error as any).code = 'ENOENT';
         mockFs.appendFile.mockRejectedValue(error);
         mockFs.writeFile.mockResolvedValue(undefined);
 
-        await writeOrUpdateFileContent('/test/newfile.txt', 'initial content');
+        await createOrAppendFileContent('/test/newfile.txt', 'initial content');
 
         expect(mockFs.writeFile).toHaveBeenCalledWith(
           '/test/newfile.txt',
@@ -363,7 +363,7 @@ describe('Lib Functions', () => {
       it('appends to existing file', async () => {
         mockFs.appendFile.mockResolvedValue(undefined);
 
-        await writeOrUpdateFileContent('/test/file.txt', '\nappended content');
+        await createOrAppendFileContent('/test/file.txt', '\nappended content');
 
         expect(mockFs.appendFile).toHaveBeenCalledWith(
           '/test/file.txt',
@@ -385,7 +385,7 @@ describe('Lib Functions', () => {
           .mockResolvedValueOnce(undefined);
         mockFs.rename.mockResolvedValue(undefined);
 
-        await writeOrUpdateFileContent('/test/file.txt', 'content');
+        await createOrAppendFileContent('/test/file.txt', 'content');
 
         expect(mockFs.writeFile).toHaveBeenCalledTimes(2);
         expect(mockFs.rename).toHaveBeenCalled();
@@ -396,7 +396,7 @@ describe('Lib Functions', () => {
         (error as any).code = 'EACCES';
         mockFs.appendFile.mockRejectedValue(error);
 
-        await expect(writeOrUpdateFileContent('/test/file.txt', 'content'))
+        await expect(createOrAppendFileContent('/test/file.txt', 'content'))
           .rejects.toThrow('Permission denied');
       });
     });

src/filesystem/index.ts

@@ -22,7 +22,7 @@ import {
   readFileContent,
   writeFileContent,
   appendFileContent,
-  writeOrUpdateFileContent,
+  createOrAppendFileContent,
   searchFilesWithValidation,
   applyFileEdits,
   tailFile,
@@ -118,7 +118,7 @@ const PathContentArgsSchema = z.object({
 });
 const WriteFileArgsSchema = PathContentArgsSchema;
 const AppendFileArgsSchema = PathContentArgsSchema;
-const WriteOrUpdateFileArgsSchema = PathContentArgsSchema;
+const CreateOrAppendFileArgsSchema = PathContentArgsSchema;
 
 const EditOperation = z.object({
   oldText: z.string().describe('Text to search for - must match exactly'),
@@ -374,7 +374,7 @@ server.registerTool(
     description:
       "Append content to the end of an existing file. This operation adds new content " +
       "to the file without modifying existing content. The file must already exist - " +
-      "use write_file to create new files or write_or_update_file to create or append. " +
+      "use write_file to create new files or create_or_append_file to create or append. " +
       "Only works within allowed directories.",
     inputSchema: {
       path: z.string(),
@@ -395,9 +395,9 @@ server.registerTool(
 );
 
 server.registerTool(
-  "write_or_update_file",
+  "create_or_append_file",
   {
-    title: "Write or Update File",
+    title: "Create or Append File",
     description:
       "Create a new file with content, or append to an existing file. If the file " +
       "does not exist, it will be created with the provided content. If the file " +
@@ -411,10 +411,10 @@ server.registerTool(
     outputSchema: { content: z.string() },
     annotations: { readOnlyHint: false, idempotentHint: false, destructiveHint: false }
   },
-  async (args: z.infer<typeof WriteOrUpdateFileArgsSchema>) => {
+  async (args: z.infer<typeof CreateOrAppendFileArgsSchema>) => {
     const validPath = await validatePath(args.path);
     const existed = await fs.access(validPath).then(() => true).catch(() => false);
-    await writeOrUpdateFileContent(validPath, args.content);
+    await createOrAppendFileContent(validPath, args.content);
 
     const message = existed
       ? `Successfully appended content to ${args.path}`

src/filesystem/lib.ts

@@ -195,7 +195,7 @@ export async function appendFileContent(filePath: string, content: string): Prom
   }
 }
 
-export async function writeOrUpdateFileContent(filePath: string, content: string): Promise<void> {
+export async function createOrAppendFileContent(filePath: string, content: string): Promise<void> {
   try {
     await appendFileContent(filePath, content);
   } catch (error) {