apple-notes-mcp 2.8.48 → 2.8.49

package/README.md

@@ -829,8 +829,11 @@ nothing was deleted instead of claiming success.
 A note that is already in Recently Deleted is refused, because deleting it there
 removes it permanently. The folder is read live from Notes.app in the same
 AppleScript; the Recently Deleted folder is recognised by its database id (with
-Full Disk Access) or by its English name. To remove such a note for good, do it
-in Notes.app.
+Full Disk Access) or by its English name. The check fails closed: a note whose
+folder Notes.app reports as something other than a folder (as it does for a note
+trashed earlier in the same Notes session) is refused the same way, and a note
+whose folder cannot be read at all is refused with a message to retry. To remove
+such a note for good, do it in Notes.app.
 
 **⚠️ Safety:** Irreversible from the agent's side — requires explicit user confirmation before calling. Prefer `search-notes` / `list-notes` first to confirm the exact id(s) being deleted.
 

package/build/index.js

@@ -43035,29 +43035,26 @@ var AppleNotesManager = class {
       try
         set originalFolder to container of noteRef
       end try
-      if originalFolder is not missing value then
-        set __inTrash to true
-        try
-          if (class of originalFolder) is folder then set __inTrash to false
-        end try
-        try
-          if ${trashFolderIdList()} contains (id of originalFolder) then set __inTrash to true
-        end try
-        try
-          if (name of originalFolder) is "${RECENTLY_DELETED_FOLDER_NAME}" then set __inTrash to true
-        end try
-        if __inTrash then return "SAFETY_IN_RECENTLY_DELETED"
-      end if
+      if originalFolder is missing value then return "SAFETY_CONTAINER_UNKNOWN"
+      set __inTrash to true
+      try
+        if (class of originalFolder) is folder then set __inTrash to false
+      end try
+      try
+        if ${trashFolderIdList()} contains (id of originalFolder) then set __inTrash to true
+      end try
+      try
+        if (name of originalFolder) is "${RECENTLY_DELETED_FOLDER_NAME}" then set __inTrash to true
+      end try
+      if __inTrash then return "SAFETY_IN_RECENTLY_DELETED"
       set currentBody to body of noteRef
       considering case
         if currentBody is not "${safeExpectedBody}" and currentBody is not "${safeExpectedBody}" & linefeed then return "SAFETY_CONFLICT"
         delete noteRef
       end considering
-      if originalFolder is not missing value then
-        try
-          if (id of notes of originalFolder) contains "${safeId}" then return "SAFETY_NOT_DELETED"
-        end try
-      end if
+      try
+        if (id of notes of originalFolder) contains "${safeId}" then return "SAFETY_NOT_DELETED"
+      end try
       return "SAFETY_DELETED"
     `);
     const result = executeMutationAppleScript(script);
@@ -43070,6 +43067,7 @@ var AppleNotesManager = class {
     const status = result.output.trim();
     if (status === "SAFETY_CONFLICT") return { status: "conflict" };
     if (status === "SAFETY_IN_RECENTLY_DELETED") return { status: "in-recently-deleted" };
+    if (status === "SAFETY_CONTAINER_UNKNOWN") return { status: "container-unknown" };
     if (status === "SAFETY_NOT_DELETED") return { status: "not-deleted" };
     return status === "SAFETY_DELETED" ? { status: "deleted" } : { status: "failed" };
   }
@@ -51557,6 +51555,9 @@ function readExactNoteSnapshot(id2) {
   return { note, body, rich, contentHash: richContentHash(body, rich) };
 }
 var NOT_DELETED_MESSAGE = "Notes.app accepted the delete, but the note is still in its original folder, so it was not moved to Recently Deleted. Nothing was deleted; read the note again before retrying.";
+function containerUnknownMessage(title) {
+  return `Could not read which folder note "${title}" is in, so it may already be in Recently Deleted, where deleting it would remove it permanently. Nothing was deleted. Retry in a moment; if it keeps failing, quit and reopen Notes.app, then retry.`;
+}
 function inRecentlyDeletedMessage(title) {
   return `Note "${title}" is already in Recently Deleted, where deleting it would remove it permanently. Nothing was deleted. Remove it from Recently Deleted in Notes.app if that is intended.`;
 }
@@ -52829,6 +52830,9 @@ registerTool(
     if (result.status === "in-recently-deleted") {
       return errorResponse(inRecentlyDeletedMessage(snapshot.note.title));
     }
+    if (result.status === "container-unknown") {
+      return errorResponse(containerUnknownMessage(snapshot.note.title));
+    }
     if (result.status === "not-deleted") {
       return errorResponse(NOT_DELETED_MESSAGE);
     }
@@ -53509,6 +53513,9 @@ registerTool(
       if (result.status === "in-recently-deleted") {
         return { id: id2, success: false, error: inRecentlyDeletedMessage(snapshot.note.title) };
       }
+      if (result.status === "container-unknown") {
+        return { id: id2, success: false, error: containerUnknownMessage(snapshot.note.title) };
+      }
       if (result.status === "not-deleted")
         return { id: id2, success: false, error: NOT_DELETED_MESSAGE };
       return { id: id2, success: false, error: "Delete result uncertain; inspect this exact ID" };

package/package.json

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