pkm-mcp-server 1.1.0 → 1.1.1

package/helpers.js

@@ -12,6 +12,39 @@ const PRIORITY_RANKS = { urgent: 3, high: 2, normal: 1, low: 0 };
 const DATE_PATTERN = /^\d{4}-\d{2}-\d{2}$/;
 const DANGEROUS_KEYS = new Set(["__proto__", "constructor", "prototype"]);
 
+// Canonical frontmatter field constraints, keyed by note type.
+// Used by vault_write (template-based) and vault_update_frontmatter (existing files).
+const FIELD_ENUMS = {
+  status: {
+    task: ["pending", "active", "done", "cancelled"],
+  },
+  priority: {
+    task: Object.keys(PRIORITY_RANKS),  // low, normal, high, urgent
+  },
+};
+
+/**
+ * Validate a frontmatter field value against FIELD_ENUMS for a given note type.
+ * Throws if the value is not in the allowed set.
+ *
+ * @param {string} fieldName - the frontmatter field (e.g. "status")
+ * @param {*} value - the value being set
+ * @param {string} noteType - the note's type field (e.g. "task")
+ */
+export function validateFieldEnum(fieldName, value, noteType) {
+  if (value === null || value === undefined) return;
+  const enumsByType = FIELD_ENUMS[fieldName];
+  if (!enumsByType) return;
+  const allowed = enumsByType[noteType];
+  if (!allowed) return;
+  const strValue = String(value);
+  if (!allowed.includes(strValue)) {
+    throw new Error(
+      `Invalid ${fieldName} "${strValue}" for type "${noteType}". Allowed values: ${allowed.join(", ")}`
+    );
+  }
+}
+
 /**
  * Compare two frontmatter values for sorting.
  * Smart ordering: priority uses custom ranks, dates sort chronologically, strings use localeCompare.
@@ -314,11 +347,18 @@ export function substituteTemplateVariables(content, vars) {
         }
       }
 
+      // Extract note type from template for enum validation
+      const typeMatch = frontmatterSection.match(/^type:\s*(.+)$/m);
+      const noteType = typeMatch ? typeMatch[1].trim() : null;
+
       for (const [key, value] of Object.entries(vars.frontmatter)) {
         if (key === "tags") continue;
         if (DANGEROUS_KEYS.has(key)) {
           throw new Error(`Disallowed frontmatter key: "${key}"`);
         }
+        if (noteType && typeof value === "string") {
+          validateFieldEnum(key, value, noteType);
+        }
         if (typeof value === "string") {
           if (!/^[a-zA-Z][a-zA-Z0-9_-]*$/.test(key)) {
             throw new Error(`Invalid frontmatter key: "${key}". Keys must start with a letter and contain only letters, digits, hyphens, or underscores.`);
@@ -843,6 +883,11 @@ export function updateFrontmatter(content, fields) {
           throw new Error("tags must be a non-empty array");
         }
       }
+      // Use the file's existing type (or the new type if being changed) for enum validation
+      const effectiveType = key === "type" ? undefined : (fields.type || parsed.type);
+      if (effectiveType) {
+        validateFieldEnum(key, value, String(effectiveType));
+      }
       parsed[key] = value;
     }
   }

package/index.js

@@ -115,8 +115,8 @@ Pass custom <%...%> variables via the 'variables' parameter.`,
             description: "Frontmatter fields to set (e.g., {tags: ['tag1', 'tag2'], status: 'active'})",
             properties: {
               tags: { type: "array", items: { type: "string" }, description: "Tags for the note (required)" },
-              status: { type: "string", description: "Note status" },
-              priority: { type: "string", description: "Priority level (for tasks)" },
+              status: { type: "string", description: "Note status. For tasks: pending, active, done, cancelled" },
+              priority: { type: "string", description: "Priority level. For tasks: low, normal, high, urgent" },
               project: { type: "string", description: "Project name (for devlogs)" },
               deciders: { type: "string", description: "Decision makers (for ADRs)" },
               due: { type: "string", description: "Due date (for tasks)" },
@@ -157,7 +157,7 @@ Pass custom <%...%> variables via the 'variables' parameter.`,
     },
     {
       name: "vault_update_frontmatter",
-      description: "Update YAML frontmatter fields in an existing note. Parses existing frontmatter, updates specified fields, preserves everything else. Set a field to null to remove it. Protected fields (type, created, tags) cannot be removed.",
+      description: "Update YAML frontmatter fields in an existing note. Parses existing frontmatter, updates specified fields, preserves everything else. Set a field to null to remove it. Protected fields (type, created, tags) cannot be removed. Field values are validated against the note's type (e.g. task status must be: pending, active, done, cancelled; task priority must be: low, normal, high, urgent).",
       inputSchema: {
         type: "object",
         properties: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pkm-mcp-server",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "MCP server for Obsidian vault integration with Claude Code — 18 tools for notes, search, and graph traversal",
   "main": "index.js",
   "exports": {