@noteplanco/noteplan-mcp 1.1.8 → 1.1.9

package/dist/tools/notes.js

@@ -69,7 +69,7 @@ function resolveNoteTarget(id, filename, space) {
         note,
     };
 }
-function resolveWritableNoteReference(input) {
+export function resolveWritableNoteReference(input) {
     if (input.id && input.id.trim().length > 0) {
         const note = store.getNote({ id: input.id.trim(), space: input.space?.trim() });
         return { note, error: note ? undefined : 'Note not found' };
@@ -124,7 +124,7 @@ function resolveWritableNoteReference(input) {
         error: 'Provide one note reference: id, filename, title, date, or query',
     };
 }
-function getWritableIdentifier(note) {
+export function getWritableIdentifier(note) {
     if (note.source === 'space') {
         return {
             identifier: note.id || note.filename,
@@ -302,7 +302,11 @@ export const createNoteSchema = z.object({
     templateTypes: z.array(z.enum(['empty-note', 'meeting-note', 'project-note', 'calendar-note'])).optional().describe('Template type tags — used when noteType="template"'),
 });
 export const updateNoteSchema = z.object({
-    filename: z.string().describe('Filename/path of the note to update'),
+    id: z.string().optional().describe('Note ID (preferred for space notes)'),
+    filename: z.string().optional().describe('Filename/path of the note to update'),
+    title: z.string().optional().describe('Note title to search for'),
+    date: z.string().optional().describe('Date for calendar notes (YYYYMMDD, YYYY-MM-DD, today, tomorrow, yesterday)'),
+    query: z.string().optional().describe('Fuzzy note query'),
     space: z.string().optional().describe('Space name or ID to search in'),
     content: z
         .string()
@@ -323,6 +327,14 @@ export const updateNoteSchema = z.object({
         .boolean()
         .optional()
         .describe('Allow replacing note content with empty/blank text (default: false)'),
+}).superRefine((input, ctx) => {
+    if (!input.id && !input.filename && !input.title && !input.date && !input.query) {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: 'Provide one note reference: id, filename, title, date, or query',
+            path: ['filename'],
+        });
+    }
 });
 export const deleteNoteSchema = z.object({
     id: z.string().optional().describe('Note ID (preferred for TeamSpace notes)'),
@@ -348,6 +360,9 @@ export const deleteNoteSchema = z.object({
 export const moveNoteSchema = z.object({
     id: z.string().optional().describe('Note ID (preferred for TeamSpace notes)'),
     filename: z.string().optional().describe('Filename/path of the note to move'),
+    title: z.string().optional().describe('Note title to search for'),
+    date: z.string().optional().describe('Date for calendar notes (YYYYMMDD, YYYY-MM-DD, today, tomorrow, yesterday)'),
+    query: z.string().optional().describe('Fuzzy note query'),
     space: z.string().optional().describe('Space name or ID to search in'),
     destinationFolder: z
         .string()
@@ -731,29 +746,33 @@ export function updateNote(params) {
                 error: 'Full note replacement is blocked for noteplan_update_note unless fullReplace=true. Prefer noteplan_search_paragraphs + noteplan_edit_line/insert_content/delete_lines for targeted edits.',
             };
         }
-        const existingNote = store.getNote({ filename: params.filename, space: params.space });
-        if (!existingNote) {
+        const noteRef = resolveWritableNoteReference(params);
+        if (!noteRef.note) {
             return {
                 success: false,
-                error: 'Note not found',
+                error: noteRef.error || 'Note not found',
+                candidates: noteRef.candidates,
             };
         }
+        const existingNote = noteRef.note;
         if (params.allowEmptyContent !== true && params.content.trim().length === 0) {
             return {
                 success: false,
                 error: 'Empty content is blocked for noteplan_update_note. Use allowEmptyContent=true to override intentionally.',
             };
         }
+        // Use the resolved filename as the confirmation token target for consistency
+        const confirmationTarget = existingNote.filename;
         if (isTrueBool(params.dryRun)) {
             const token = issueConfirmationToken({
                 tool: 'noteplan_update_note',
-                target: params.filename,
+                target: confirmationTarget,
                 action: 'full_replace',
             });
             return {
                 success: true,
                 dryRun: true,
-                message: `Dry run: note ${params.filename} would be fully replaced`,
+                message: `Dry run: note ${existingNote.filename} would be fully replaced`,
                 note: {
                     id: existingNote.id,
                     title: existingNote.title,
@@ -770,7 +789,7 @@ export function updateNote(params) {
         }
         const confirmation = validateAndConsumeConfirmationToken(params.confirmationToken, {
             tool: 'noteplan_update_note',
-            target: params.filename,
+            target: confirmationTarget,
             action: 'full_replace',
         });
         if (!confirmation.ok) {
@@ -865,14 +884,16 @@ export function deleteNote(params) {
 }
 export function moveNote(params) {
     try {
-        const target = resolveNoteTarget(params.id, params.filename, params.space);
-        if (!target.note) {
+        const noteRef = resolveWritableNoteReference(params);
+        if (!noteRef.note) {
             return {
                 success: false,
-                error: 'Note not found',
+                error: noteRef.error || 'Note not found',
+                candidates: noteRef.candidates,
             };
         }
-        const preview = store.previewMoveNote(target.identifier, params.destinationFolder);
+        const writable = getWritableIdentifier(noteRef.note);
+        const preview = store.previewMoveNote(writable.identifier, params.destinationFolder);
         const confirmationTarget = `${preview.fromFilename}=>${preview.toFilename}::${preview.destinationParentId ?? preview.destinationFolder}`;
         if (isTrueBool(params.dryRun)) {
             const token = issueConfirmationToken({
@@ -910,7 +931,7 @@ export function moveNote(params) {
                 error: confirmationFailureMessage('noteplan_move_note', confirmation.reason),
             };
         }
-        const moved = store.moveNote(target.identifier, params.destinationFolder);
+        const moved = store.moveNote(writable.identifier, params.destinationFolder);
         return {
             success: true,
             message: moved.note.source === 'space'
@@ -1068,6 +1089,21 @@ export function renameNoteFile(params) {
                 };
             }
             const renamed = store.renameSpaceNote(writeId, params.newTitle);
+            // Also update the # Title heading in the note content if it matches the old title
+            if (renamed.note.content) {
+                const lines = renamed.note.content.split('\n');
+                const titleLineIndex = lines.findIndex((l) => /^#\s+/.test(l));
+                if (titleLineIndex !== -1) {
+                    const oldHeadingTitle = lines[titleLineIndex].replace(/^#\s+/, '');
+                    if (oldHeadingTitle === renamed.fromTitle) {
+                        lines[titleLineIndex] = `# ${params.newTitle}`;
+                        const renamedWriteTarget = getWritableIdentifier(renamed.note);
+                        store.updateNote(renamedWriteTarget.identifier, lines.join('\n'), {
+                            source: renamedWriteTarget.source,
+                        });
+                    }
+                }
+            }
             return {
                 success: true,
                 message: `TeamSpace note renamed from "${renamed.fromTitle}" to "${renamed.toTitle}"`,
@@ -1175,13 +1211,25 @@ export function renameNoteFile(params) {
 }
 // Get note with line numbers
 export const getParagraphsSchema = z.object({
-    filename: z.string().describe('Filename/path of the note'),
+    id: z.string().optional().describe('Note ID (preferred for space notes)'),
+    filename: z.string().optional().describe('Filename/path of the note'),
+    title: z.string().optional().describe('Note title to search for'),
+    date: z.string().optional().describe('Date for calendar notes (YYYYMMDD, YYYY-MM-DD, today, tomorrow, yesterday)'),
+    query: z.string().optional().describe('Fuzzy note query'),
     space: z.string().optional().describe('Space name or ID to search in'),
     startLine: z.number().min(1).optional().describe('First line to include (1-indexed, inclusive)'),
     endLine: z.number().min(1).optional().describe('Last line to include (1-indexed, inclusive)'),
     limit: z.number().min(1).max(1000).optional().default(200).describe('Maximum lines to return'),
     offset: z.number().min(0).optional().default(0).describe('Pagination offset within selected range'),
     cursor: z.string().optional().describe('Cursor token from previous page (preferred over offset)'),
+}).superRefine((input, ctx) => {
+    if (!input.id && !input.filename && !input.title && !input.date && !input.query) {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: 'Provide one note reference: id, filename, title, date, or query',
+            path: ['filename'],
+        });
+    }
 });
 export const searchParagraphsSchema = z.object({
     id: z.string().optional().describe('Note ID (preferred for space notes)'),
@@ -1215,13 +1263,15 @@ export const searchParagraphsSchema = z.object({
     }
 });
 export function getParagraphs(params) {
-    const note = store.getNote({ filename: params.filename, space: params.space });
-    if (!note) {
+    const noteRef = resolveWritableNoteReference(params);
+    if (!noteRef.note) {
         return {
             success: false,
-            error: 'Note not found',
+            error: noteRef.error || 'Note not found',
+            candidates: noteRef.candidates,
         };
     }
+    const note = noteRef.note;
     const allLines = note.content.split('\n');
     const lineWindow = buildLineWindow(allLines, {
         startLine: params.startLine,
@@ -1401,15 +1451,39 @@ export function searchParagraphs(params) {
 }
 // Granular note operation schemas
 export const setPropertySchema = z.object({
-    filename: z.string().describe('Filename/path of the note'),
+    id: z.string().optional().describe('Note ID (preferred for space notes)'),
+    filename: z.string().optional().describe('Filename/path of the note'),
+    title: z.string().optional().describe('Note title to search for'),
+    date: z.string().optional().describe('Date for calendar notes (YYYYMMDD, YYYY-MM-DD, today, tomorrow, yesterday)'),
+    query: z.string().optional().describe('Fuzzy note query'),
     space: z.string().optional().describe('Space name or ID to search in'),
     key: z.string().describe('Property key (e.g., "icon", "bg-color", "status")'),
     value: z.string().describe('Property value'),
+}).superRefine((input, ctx) => {
+    if (!input.id && !input.filename && !input.title && !input.date && !input.query) {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: 'Provide one note reference: id, filename, title, date, or query',
+            path: ['filename'],
+        });
+    }
 });
 export const removePropertySchema = z.object({
-    filename: z.string().describe('Filename/path of the note'),
+    id: z.string().optional().describe('Note ID (preferred for space notes)'),
+    filename: z.string().optional().describe('Filename/path of the note'),
+    title: z.string().optional().describe('Note title to search for'),
+    date: z.string().optional().describe('Date for calendar notes (YYYYMMDD, YYYY-MM-DD, today, tomorrow, yesterday)'),
+    query: z.string().optional().describe('Fuzzy note query'),
     space: z.string().optional().describe('Space name or ID to search in'),
     key: z.string().describe('Property key to remove'),
+}).superRefine((input, ctx) => {
+    if (!input.id && !input.filename && !input.title && !input.date && !input.query) {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: 'Provide one note reference: id, filename, title, date, or query',
+            path: ['filename'],
+        });
+    }
 });
 export const insertContentSchema = z.object({
     id: z.string().optional().describe('Note ID (preferred for space notes)'),
@@ -1551,13 +1625,14 @@ export const replaceLinesSchema = z.object({
 // Granular note operation implementations
 export function setProperty(params) {
     try {
-        const note = store.getNote({ filename: params.filename, space: params.space });
-        if (!note) {
-            return { success: false, error: 'Note not found' };
+        const noteRef = resolveWritableNoteReference(params);
+        if (!noteRef.note) {
+            return { success: false, error: noteRef.error || 'Note not found', candidates: noteRef.candidates };
         }
+        const note = noteRef.note;
         const newContent = frontmatter.setFrontmatterProperty(note.content, params.key, params.value);
-        const writeIdentifier = note.source === 'space' ? (note.id || note.filename) : note.filename;
-        store.updateNote(writeIdentifier, newContent, { source: note.source });
+        const writable = getWritableIdentifier(note);
+        store.updateNote(writable.identifier, newContent, { source: writable.source });
         return {
             success: true,
             message: `Property "${params.key}" set to "${params.value}"`,
@@ -1572,13 +1647,14 @@ export function setProperty(params) {
 }
 export function removeProperty(params) {
     try {
-        const note = store.getNote({ filename: params.filename, space: params.space });
-        if (!note) {
-            return { success: false, error: 'Note not found' };
+        const noteRef = resolveWritableNoteReference(params);
+        if (!noteRef.note) {
+            return { success: false, error: noteRef.error || 'Note not found', candidates: noteRef.candidates };
         }
+        const note = noteRef.note;
         const newContent = frontmatter.removeFrontmatterProperty(note.content, params.key);
-        const writeIdentifier = note.source === 'space' ? (note.id || note.filename) : note.filename;
-        store.updateNote(writeIdentifier, newContent, { source: note.source });
+        const writable = getWritableIdentifier(note);
+        store.updateNote(writable.identifier, newContent, { source: writable.source });
         return {
             success: true,
             message: `Property "${params.key}" removed`,
@@ -1722,6 +1798,15 @@ export function appendContent(params) {
 }
 export function deleteLines(params) {
     try {
+        // Validate required line params — MCP may deliver them as undefined when omitted
+        const rawStart = params.startLine !== undefined && params.startLine !== null ? Number(params.startLine) : NaN;
+        const rawEnd = params.endLine !== undefined && params.endLine !== null ? Number(params.endLine) : NaN;
+        if (!Number.isFinite(rawStart)) {
+            return { success: false, error: 'startLine is required (1-indexed).' };
+        }
+        if (!Number.isFinite(rawEnd)) {
+            return { success: false, error: 'endLine is required (1-indexed). Pass the same value as startLine to delete a single line.' };
+        }
         const resolved = resolveWritableNoteReference(params);
         if (!resolved.note) {
             return { success: false, error: resolved.error || 'Note not found', candidates: resolved.candidates };
@@ -1881,6 +1966,15 @@ export function editLine(params) {
 }
 export function replaceLines(params) {
     try {
+        // Validate required line params — MCP may deliver them as undefined when omitted
+        const rawStart = params.startLine !== undefined && params.startLine !== null ? Number(params.startLine) : NaN;
+        const rawEnd = params.endLine !== undefined && params.endLine !== null ? Number(params.endLine) : NaN;
+        if (!Number.isFinite(rawStart)) {
+            return { success: false, error: 'startLine is required (1-indexed).' };
+        }
+        if (!Number.isFinite(rawEnd)) {
+            return { success: false, error: 'endLine is required (1-indexed). Pass the same value as startLine to replace a single line.' };
+        }
         const resolved = resolveWritableNoteReference(params);
         if (!resolved.note) {
             return { success: false, error: resolved.error || 'Note not found', candidates: resolved.candidates };