@noteplanco/noteplan-mcp 1.1.6 → 1.1.7

package/dist/tools/notes.js

@@ -3,6 +3,7 @@ import { z } from 'zod';
 import path from 'path';
 import * as store from '../noteplan/unified-store.js';
 import * as frontmatter from '../noteplan/frontmatter-parser.js';
+import { ensureTemplateFrontmatter } from './templates.js';
 import { issueConfirmationToken, validateAndConsumeConfirmationToken, } from '../utils/confirmation-tokens.js';
 import { parseParagraphLine, buildParagraphLine } from '../noteplan/markdown-parser.js';
 function toBoundedInt(value, defaultValue, min, max) {
@@ -34,6 +35,17 @@ function toOptionalBoolean(value) {
     }
     return undefined;
 }
+/**
+ * Coerce a value to boolean — handles MCP delivering boolean params as strings.
+ * Returns true for boolean true or string "true".
+ */
+function isTrueBool(value) {
+    if (typeof value === 'boolean')
+        return value;
+    if (typeof value === 'string')
+        return value.trim().toLowerCase() === 'true';
+    return false;
+}
 function confirmationFailureMessage(toolName, reason) {
     const refreshHint = `Call ${toolName} with dryRun=true to get a new confirmationToken.`;
     if (reason === 'missing') {
@@ -286,6 +298,8 @@ export const createNoteSchema = z.object({
     folder: z.string().optional().describe('Folder to create the note in. Supports smart matching (e.g., "projects" matches "10 - Projects")'),
     create_new_folder: z.boolean().optional().describe('Set to true to create a new folder instead of matching existing ones'),
     space: z.string().optional().describe('Space name or ID to create in (e.g., "My Team" or a UUID)'),
+    noteType: z.enum(['note', 'template']).optional().default('note').describe('Type of note to create. Use "template" to create in @Templates with proper frontmatter'),
+    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'),
@@ -358,6 +372,8 @@ export const moveNoteSchema = z.object({
 export const renameNoteFileSchema = z.object({
     id: z.string().optional().describe('Note ID (preferred for TeamSpace notes)'),
     filename: z.string().optional().describe('Filename/path of the note to rename'),
+    title: z.string().optional().describe('Note title to find and rename (fuzzy matched)'),
+    query: z.string().optional().describe('Fuzzy search query to find the note'),
     space: z.string().optional().describe('Space name or ID to search in'),
     newFilename: z
         .string()
@@ -381,11 +397,11 @@ export const renameNoteFileSchema = z.object({
         .optional()
         .describe('Confirmation token issued by dryRun for rename execution'),
 }).superRefine((input, ctx) => {
-    if (!input.id && !input.filename) {
+    if (!input.id && !input.filename && !input.title && !input.query) {
         ctx.addIssue({
             code: z.ZodIssueCode.custom,
-            message: 'Provide one note reference: id or filename',
-            path: ['filename'],
+            message: 'Provide one note reference: id, filename, title, or query',
+            path: ['title'],
         });
     }
 });
@@ -670,8 +686,13 @@ export function resolveNote(params) {
 }
 export function createNote(params) {
     try {
-        const result = store.createNote(params.title, params.content, {
-            folder: params.folder,
+        const isTemplate = params.noteType === 'template';
+        const folder = isTemplate && !params.folder ? '@Templates' : params.folder;
+        const content = isTemplate
+            ? ensureTemplateFrontmatter(params.title, params.content, params.templateTypes)
+            : params.content;
+        const result = store.createNote(params.title, content, {
+            folder,
             space: params.space,
             createNewFolder: params.create_new_folder,
         });
@@ -722,7 +743,7 @@ export function updateNote(params) {
                 error: 'Empty content is blocked for noteplan_update_note. Use allowEmptyContent=true to override intentionally.',
             };
         }
-        if (params.dryRun === true) {
+        if (isTrueBool(params.dryRun)) {
             const token = issueConfirmationToken({
                 tool: 'noteplan_update_note',
                 target: params.filename,
@@ -788,7 +809,7 @@ export function deleteNote(params) {
                 error: 'Note not found',
             };
         }
-        if (params.dryRun === true) {
+        if (isTrueBool(params.dryRun)) {
             const token = issueConfirmationToken({
                 tool: 'noteplan_delete_note',
                 target: target.identifier,
@@ -852,7 +873,7 @@ export function moveNote(params) {
         }
         const preview = store.previewMoveNote(target.identifier, params.destinationFolder);
         const confirmationTarget = `${preview.fromFilename}=>${preview.toFilename}::${preview.destinationParentId ?? preview.destinationFolder}`;
-        if (params.dryRun === true) {
+        if (isTrueBool(params.dryRun)) {
             const token = issueConfirmationToken({
                 tool: 'noteplan_move_note',
                 target: confirmationTarget,
@@ -926,7 +947,7 @@ export function restoreNote(params) {
         }
         const preview = store.previewRestoreNote(target.identifier, params.destinationFolder);
         const confirmationTarget = `${preview.fromIdentifier}=>${preview.toIdentifier}`;
-        if (params.dryRun === true) {
+        if (isTrueBool(params.dryRun)) {
             const token = issueConfirmationToken({
                 tool: 'noteplan_restore_note',
                 target: confirmationTarget,
@@ -990,12 +1011,16 @@ export function restoreNote(params) {
 }
 export function renameNoteFile(params) {
     try {
-        // Resolve the note to determine if it's local or space
-        const target = resolveNoteTarget(params.id, params.filename, params.space);
-        if (!target.note) {
-            return { success: false, error: 'Note not found' };
+        // Resolve the note — supports id, filename, title, or query
+        const resolved = resolveWritableNoteReference(params);
+        if (!resolved.note) {
+            return {
+                success: false,
+                error: resolved.error || 'Note not found',
+                candidates: resolved.candidates,
+            };
         }
-        const note = target.note;
+        const note = resolved.note;
         // Space note: rename title
         if (note.source === 'space') {
             if (!params.newTitle) {
@@ -1006,7 +1031,7 @@ export function renameNoteFile(params) {
             }
             const writeId = note.id || note.filename;
             const confirmationTarget = `${note.title}=>${params.newTitle}`;
-            if (params.dryRun === true) {
+            if (isTrueBool(params.dryRun)) {
                 const token = issueConfirmationToken({
                     tool: 'noteplan_rename_note_file',
                     target: confirmationTarget,
@@ -1059,16 +1084,18 @@ export function renameNoteFile(params) {
             };
         }
         // Local note: rename file
-        if (!params.newFilename) {
+        // Accept newTitle as an alias for newFilename — "rename the note" typically means changing the title
+        const effectiveNewFilename = params.newFilename || params.newTitle;
+        if (!effectiveNewFilename) {
             return {
                 success: false,
-                error: 'newFilename is required for local notes (use newTitle for TeamSpace notes)',
+                error: 'newFilename or newTitle is required for local notes',
             };
         }
         const keepExtension = params.keepExtension ?? true;
-        const preview = store.previewRenameNoteFile(note.filename, params.newFilename, keepExtension);
+        const preview = store.previewRenameNoteFile(note.filename, effectiveNewFilename, keepExtension);
         const confirmationTarget = `${preview.fromFilename}=>${preview.toFilename}`;
-        if (params.dryRun === true) {
+        if (isTrueBool(params.dryRun)) {
             const token = issueConfirmationToken({
                 tool: 'noteplan_rename_note_file',
                 target: confirmationTarget,
@@ -1103,7 +1130,26 @@ export function renameNoteFile(params) {
                 error: confirmationFailureMessage('noteplan_rename_note_file', confirmation.reason),
             };
         }
-        const renamed = store.renameNoteFile(note.filename, params.newFilename, keepExtension);
+        const renamed = store.renameNoteFile(note.filename, effectiveNewFilename, keepExtension);
+        // Also update the # Title heading in the note content if it matches the old title
+        const newTitle = params.newTitle || params.newFilename;
+        if (newTitle && 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+/, '');
+                // Update heading if it matches the old note title (or old filename without extension)
+                const oldTitle = note.title || '';
+                const oldFilenameBase = note.filename.replace(/^.*\//, '').replace(/\.\w+$/, '');
+                if (oldHeadingTitle === oldTitle || oldHeadingTitle === oldFilenameBase) {
+                    lines[titleLineIndex] = `# ${newTitle}`;
+                    const writeTarget = getWritableIdentifier(renamed.note);
+                    store.updateNote(writeTarget.identifier, lines.join('\n'), {
+                        source: writeTarget.source,
+                    });
+                }
+            }
+        }
         return {
             success: true,
             message: `Note renamed to ${renamed.toFilename}`,
@@ -1427,6 +1473,7 @@ export const appendContentSchema = z.object({
     query: z.string().optional().describe('Resolvable note query (fuzzy note lookup before append)'),
     space: z.string().optional().describe('Space name or ID scope for title/date/query resolution'),
     content: z.string().describe('Content to append'),
+    heading: z.string().optional().describe('Heading or section marker text — when provided, appends at end of that section instead of end of note'),
     indentationStyle: z
         .enum(['tabs', 'preserve'])
         .optional()
@@ -1555,6 +1602,41 @@ export function insertContent(params) {
         const note = resolved.note;
         const indentationStyle = normalizeIndentationStyle(params.indentationStyle);
         let contentToInsert = params.content;
+        // Auto-correct position when line number is provided but position is wrong
+        // Catches LLMs sending { position: "start", line: 5 } instead of { position: "at-line", line: 5 }
+        if (params.line !== undefined && params.position !== 'at-line') {
+            params.position = 'at-line';
+        }
+        // Auto-detect raw task/checklist markdown when type is not explicitly set
+        // Catches LLMs sending "- [ ] Buy groceries", "* [x] Done", "* Buy groceries", "+ Item" without proper type
+        if (!params.type && /^[\t ]*[*+\-]\s+/.test(contentToInsert)) {
+            // Determine type from the marker character
+            const markerMatch = contentToInsert.match(/^[\t ]*([*+\-])\s+/);
+            const markerChar = markerMatch?.[1];
+            if (markerChar === '+') {
+                params.type = 'checklist';
+            }
+            else if (markerChar === '*') {
+                params.type = 'task';
+            }
+            else if (markerChar === '-' && /^[\t ]*-\s+\[[ x\->]\]\s+/.test(contentToInsert)) {
+                // Dash with checkbox is clearly a task (plain "- text" could be a bullet, so only match with checkbox)
+                params.type = 'task';
+            }
+            // Detect status from the checkbox marker if present
+            if (params.type) {
+                const statusMatch = contentToInsert.match(/\[(.)\]/);
+                if (statusMatch) {
+                    const marker = statusMatch[1];
+                    if (marker === 'x')
+                        params.taskStatus = 'done';
+                    else if (marker === '-')
+                        params.taskStatus = 'cancelled';
+                    else if (marker === '>')
+                        params.taskStatus = 'scheduled';
+                }
+            }
+        }
         if (params.type) {
             contentToInsert = contentToInsert
                 .split('\n')
@@ -1610,6 +1692,7 @@ export function appendContent(params) {
         const normalized = normalizeContentIndentation(params.content, indentationStyle);
         const newContent = frontmatter.insertContentAtPosition(note.content, normalized.content, {
             position: 'end',
+            heading: params.heading,
         });
         const writeTarget = getWritableIdentifier(note);
         store.updateNote(writeTarget.identifier, newContent, {
@@ -1642,11 +1725,14 @@ export function deleteLines(params) {
         }
         const note = resolved.note;
         const allLines = note.content.split('\n');
-        const boundedStartLine = toBoundedInt(params.startLine, 1, 1, Math.max(1, allLines.length));
-        const boundedEndLine = toBoundedInt(params.endLine, boundedStartLine, boundedStartLine, Math.max(boundedStartLine, allLines.length));
+        // Line numbers are relative to content after frontmatter
+        const fmOffset = frontmatter.getFrontmatterLineCount(note.content);
+        const contentLineCount = allLines.length - fmOffset;
+        const boundedStartLine = toBoundedInt(params.startLine, 1, 1, Math.max(1, contentLineCount));
+        const boundedEndLine = toBoundedInt(params.endLine, boundedStartLine, boundedStartLine, Math.max(boundedStartLine, contentLineCount));
         const lineCountToDelete = boundedEndLine - boundedStartLine + 1;
-        const previewStartIndex = boundedStartLine - 1;
-        const previewEndIndexExclusive = boundedEndLine;
+        const previewStartIndex = fmOffset + boundedStartLine - 1;
+        const previewEndIndexExclusive = fmOffset + boundedEndLine;
         const deletedLinesPreview = allLines
             .slice(previewStartIndex, previewEndIndexExclusive)
             .slice(0, 20)
@@ -1660,7 +1746,7 @@ export function deleteLines(params) {
             ? buildAttachmentWarningMessage(removedAttachmentReferences.length)
             : undefined;
         const confirmTarget = `${note.filename}:${boundedStartLine}-${boundedEndLine}`;
-        if (params.dryRun === true) {
+        if (isTrueBool(params.dryRun)) {
             const token = issueConfirmationToken({
                 tool: 'noteplan_delete_lines',
                 target: confirmTarget,
@@ -1724,11 +1810,13 @@ export function editLine(params) {
         const note = resolved.note;
         const lines = note.content.split('\n');
         const originalLineCount = lines.length;
-        const lineIndex = params.line - 1; // Convert to 0-indexed
-        if (lineIndex < 0 || lineIndex >= lines.length) {
+        // Offset past frontmatter so line 1 = first content line
+        const fmOffset = frontmatter.getFrontmatterLineCount(note.content);
+        const lineIndex = fmOffset + Number(params.line) - 1; // Convert to 0-indexed, skip FM
+        if (lineIndex < fmOffset || lineIndex >= lines.length) {
             return {
                 success: false,
-                error: `Line ${params.line} does not exist (note has ${lines.length} lines)`,
+                error: `Line ${params.line} does not exist (note has ${lines.length - fmOffset} content lines)`,
             };
         }
         const originalLine = lines[lineIndex];
@@ -1781,11 +1869,14 @@ export function replaceLines(params) {
         const note = resolved.note;
         const allLines = note.content.split('\n');
         const originalLineCount = allLines.length;
-        const boundedStartLine = toBoundedInt(params.startLine, 1, 1, Math.max(1, originalLineCount));
-        const boundedEndLine = toBoundedInt(params.endLine, boundedStartLine, boundedStartLine, Math.max(boundedStartLine, originalLineCount));
-        const startIndex = boundedStartLine - 1;
+        // Line numbers are relative to content after frontmatter
+        const fmOffset = frontmatter.getFrontmatterLineCount(note.content);
+        const contentLineCount = originalLineCount - fmOffset;
+        const boundedStartLine = toBoundedInt(params.startLine, 1, 1, Math.max(1, contentLineCount));
+        const boundedEndLine = toBoundedInt(params.endLine, boundedStartLine, boundedStartLine, Math.max(boundedStartLine, contentLineCount));
+        const startIndex = fmOffset + boundedStartLine - 1;
         const lineCountToReplace = boundedEndLine - boundedStartLine + 1;
-        const replacedText = allLines.slice(startIndex, boundedEndLine).join('\n');
+        const replacedText = allLines.slice(startIndex, fmOffset + boundedEndLine).join('\n');
         const indentationStyle = normalizeIndentationStyle(params.indentationStyle);
         const normalized = normalizeContentIndentation(params.content, indentationStyle);
         if (params.allowEmptyContent !== true && normalized.content.trim().length === 0) {
@@ -1806,7 +1897,7 @@ export function replaceLines(params) {
             warnings.push(`Line numbers shifted by ${lineDelta > 0 ? '+' : ''}${lineDelta} after this replacement. Re-read line numbers before the next mutation.`);
         }
         const target = `${note.filename}:${boundedStartLine}-${boundedEndLine}:${replacementLines.length}:${normalized.content.length}`;
-        if (params.dryRun === true) {
+        if (isTrueBool(params.dryRun)) {
             const token = issueConfirmationToken({
                 tool: 'noteplan_replace_lines',
                 target,