Fix nested folder creation and note creation in deep folders#9
Closed
robschmitt wants to merge 1 commit intosweetrb:mainfrom
Closed
Fix nested folder creation and note creation in deep folders#9robschmitt wants to merge 1 commit intosweetrb:mainfrom
robschmitt wants to merge 1 commit intosweetrb:mainfrom
Conversation
…ders createFolder now supports slash-separated paths (e.g. "Retro Tech/PC/CPUs"), creating each intermediate folder as needed. It checks existence before creating to prevent duplicate ghost folders in Apple Notes' CoreData store. Also fixes note creation in deeply nested folders by using implicit AppleScript return instead of set/return pattern which fails with nested folder refs (-1728).
sweetrb
added a commit
that referenced
this pull request
Apr 6, 2026
…ders - createFolder now iterates path segments with existence checks to prevent duplicate ghost folders in CoreData - createNote uses implicit return for folder-based creation to avoid AppleScript -1728 error in deeply nested contexts - Updated create-folder tool schema to document nested path support - Bumped to v1.4.1 Based on PR #9 by @robschmitt Co-Authored-By: Rob Schmitt <robschmitt@users.noreply.github.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Owner
|
Nice catch on all three of these, Rob — the ghost folder issue from missing existence checks was a sneaky one, and the -1728 error with |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
"Retro Tech/PC/CPUs"creates all intermediate folders, matching the/-separated convention used elsewhere in the codebase (create-note, move-note, delete-folder)set/return idpattern fails with nestedoffolder references (-1728); switched to implicit return which works correctlyTest plan
create-folderwith nested path (e.g."A/B/C") creates proper hierarchycreate-folderwith already-existing path succeeds without creating duplicatescreate-notein a deeply nested folder places the note in the correct foldercreate-folderwith single segment still works as before