apple-notes-mcp 2.5.6 → 2.5.7

package/build/index.js

@@ -1136,10 +1136,7 @@ server.registerTool("batch-delete-notes", {
 server.registerTool("batch-move-notes", {
     description: "Use when: moving multiple notes by id into one destination folder.\nReturns: per-id success/failure counts.\nDo not use when: moving a single note (move-note).\nNote: the destination folder must already exist (create-folder).",
     inputSchema: {
-        ids: z
-            .array(z.string().max(MAX.ID))
-            .max(MAX.BATCH_IDS)
-            .describe("Array of note IDs to move"),
+        ids: z.array(z.string().max(MAX.ID)).max(MAX.BATCH_IDS).describe("Array of note IDs to move"),
         folder: z.string().max(MAX.FOLDER).describe("Destination folder name"),
         account: z
             .string()

package/build/services/appleNotesManager.js

@@ -585,8 +585,13 @@ export class AppleNotesManager {
             console.error(`Failed to create note "${title}":`, result.error);
             return null;
         }
-        // Extract the CoreData ID from the response
-        const noteId = result.output.trim();
+        // Extract the CoreData ID from the response.
+        // AppleScript's `id of newNote` yields an object specifier of the form
+        // "note id x-coredata://<uuid>/ICNote/pN" — with a literal "note id " prefix.
+        // Strip that prefix so we return the bare x-coredata:// URL that the id
+        // validator and downstream tools (get-note-content, update-note) accept.
+        const rawOutput = result.output.trim();
+        const noteId = extractCoreDataId(rawOutput, "note") || rawOutput;
         // Return a Note object representing the created note with real ID
         const now = new Date();
         return {

package/build/services/appleNotesManager.test.js

@@ -341,6 +341,33 @@ describe("AppleNotesManager", () => {
             expect(result?.content).toBe("Eggs, Milk, Bread");
             expect(result?.account).toBe("iCloud"); // Default account
         });
+        it("strips the 'note id ' prefix from the returned id (#create-note-id)", () => {
+            // AppleScript's `id of newNote` yields an object specifier with a literal
+            // "note id " prefix. The returned id must be the bare x-coredata:// URL so
+            // downstream tools (get-note-content, update-note) accept it.
+            mockExecuteAppleScript.mockReturnValue({
+                success: true,
+                output: "note id x-coredata://ABC-DEF/ICNote/p42",
+            });
+            const result = manager.createNote("Prefixed", "Body");
+            expect(result?.id).toBe("x-coredata://ABC-DEF/ICNote/p42");
+            expect(result?.id).not.toMatch(/^note id /);
+        });
+        it("returned id round-trips through get-note-content and update-note (#create-note-id)", () => {
+            mockExecuteAppleScript.mockReturnValue({
+                success: true,
+                output: "note id x-coredata://ABC-DEF/ICNote/p99",
+            });
+            const created = manager.createNote("Roundtrip", "Body");
+            const id = created?.id;
+            expect(id).toBe("x-coredata://ABC-DEF/ICNote/p99");
+            // Reading back by the returned id must not throw "Invalid note ID format".
+            mockExecuteAppleScript.mockReturnValue({ success: true, output: "Body text" });
+            expect(() => manager.getNoteContentById(id)).not.toThrow();
+            // Updating by the returned id must not throw either.
+            mockExecuteAppleScript.mockReturnValue({ success: true, output: "" });
+            expect(() => manager.updateNoteById(id, undefined, "New body")).not.toThrow();
+        });
         it("returns null when AppleScript fails", () => {
             mockExecuteAppleScript.mockReturnValue({
                 success: false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "apple-notes-mcp",
-  "version": "2.5.6",
+  "version": "2.5.7",
   "description": "MCP server for Apple Notes - create, search, update, and manage notes via Claude and other AI assistants",
   "type": "module",
   "main": "build/index.js",