apple-notes-mcp 2.6.6 → 2.6.8

package/build/index.js

@@ -39585,7 +39585,8 @@ var AppleNotesManager = class {
               set modifiedParts to ""
             end try
             try
-              set noteFolder to name of container of n
+              set noteContainer to container of n
+              set noteFolder to name of noteContainer
             on error
               set noteFolder to "Notes"
             end try
@@ -40622,12 +40623,9 @@ var AppleNotesManager = class {
       tell application "Notes"
         set theNote to note id "${safeNoteId}"
         set theAttachment to missing value
-        repeat with a in attachments of theNote
-          if (id of a as text) is "${safeAttId}" then
-            set theAttachment to a
-            exit repeat
-          end if
-        end repeat
+        try
+          set theAttachment to attachment id "${safeAttId}" of theNote
+        end try
         if theAttachment is missing value then
           return "ERR" & ${AS_FIELD_SEP} & "attachment not found"
         end if
@@ -40934,7 +40932,10 @@ var AppleNotesManager = class {
       if (parts.length >= 3) {
         attachments.push({
           id: parts[0].trim(),
-          name: parts[1].trim(),
+          // Unnamed attachments render `name` as the AppleScript sentinel; surfacing
+          // the literal "missing value" as a filename is worse than saying nothing,
+          // and `name` is a required string, so fall back to the content identifier.
+          name: normalizeAppleScriptText(parts[1]) ?? normalizeAppleScriptText(parts[2]) ?? "",
           contentType: parts[2].trim(),
           contentId: parts[2].trim() || void 0,
           url: normalizeAppleScriptText(parts[3]),
@@ -41018,7 +41019,10 @@ var AppleNotesManager = class {
       if (parts.length >= 3) {
         attachments.push({
           id: parts[0].trim(),
-          name: parts[1].trim(),
+          // Unnamed attachments render `name` as the AppleScript sentinel; surfacing
+          // the literal "missing value" as a filename is worse than saying nothing,
+          // and `name` is a required string, so fall back to the content identifier.
+          name: normalizeAppleScriptText(parts[1]) ?? normalizeAppleScriptText(parts[2]) ?? "",
           contentType: parts[2].trim(),
           contentId: parts[2].trim() || void 0,
           url: normalizeAppleScriptText(parts[3]),
@@ -41054,12 +41058,9 @@ var AppleNotesManager = class {
       tell application "Notes"
         set theNote to note id "${safeNoteId}"
         set theAttachment to missing value
-        repeat with a in attachments of theNote
-          if (id of a as text) is "${safeAttId}" then
-            set theAttachment to a
-            exit repeat
-          end if
-        end repeat
+        try
+          set theAttachment to attachment id "${safeAttId}" of theNote
+        end try
         if theAttachment is missing value then
           return "ERR" & ${AS_FIELD_SEP} & "attachment not found"
         end if
@@ -41099,8 +41100,11 @@ var AppleNotesManager = class {
     return {
       success: true,
       savedPath: abs,
-      name: parts[1]?.trim(),
-      contentType: parts[2]?.trim()
+      // Unnamed attachments render as the AppleScript sentinel here too; leaving it
+      // undefined lets save-attachment/fetch-attachment fall back to "attachment"
+      // rather than reporting `Saved "missing value"`.
+      name: normalizeAppleScriptText(parts[1]),
+      contentType: normalizeAppleScriptText(parts[2])
     };
   }
   /**

package/package.json

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