planko-mcp-server 0.4.0 → 0.4.2

package/index.js

@@ -20,6 +20,7 @@ import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'node:fs';
 import { join } from 'node:path';
 
 import { createApiClient } from './src/api.js';
+import { isBlankValue, applyDueDateFallback } from './src/sanitize.js';
 import {
   blockNoteToMarkdown,
   markdownToBlockNote,
@@ -633,9 +634,10 @@ const taskProps = {
 function buildTaskBody(params) {
   const body = {};
   for (const [key, value] of Object.entries(params)) {
-    if (value === undefined) continue;
+    if (isBlankValue(value)) continue;
     body[key] = value;
   }
+  applyDueDateFallback(body);
   if ('description' in body) {
     body.description = markdownToDescription(body.description);
   }
@@ -680,7 +682,7 @@ async function handleCreate(type, params, kindLabel) {
 
   // Resolve projectName -> projectId via the API. When omitted, OMIT the key
   // entirely so the backend applies the user's default project.
-  if (projectName) {
+  if (!isBlankValue(projectName)) {
     body.projectId = await resolveProjectId(projectName);
   }
 
@@ -885,10 +887,10 @@ async function buildListParams(type, params) {
   const { projectName, ...rest } = params;
   const out = { type };
   for (const [key, value] of Object.entries(rest)) {
-    if (value === undefined) continue;
+    if (isBlankValue(value)) continue;
     out[key] = value;
   }
-  if (projectName) {
+  if (!isBlankValue(projectName)) {
     out.projectId = await resolveProjectId(projectName);
   }
   return out;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "planko-mcp-server",
-  "version": "0.4.0",
+  "version": "0.4.2",
   "description": "MCP server for syncing Planko tasks with local Markdown files",
   "type": "module",
   "bin": {

package/src/sanitize.js

@@ -0,0 +1,37 @@
+/**
+ * Argument sanitization for MCP tool inputs.
+ *
+ * LLM tool-callers (especially GPT-class models) tend to fill EVERY schema
+ * field with placeholder/default values instead of omitting optional ones:
+ * empty strings, the all-zero ObjectId ("000000000000000000000000"), and
+ * empty arrays. Left untouched, a placeholder `projectId` makes the backend
+ * throw "Project not found", a placeholder `parentId` filters to a nonexistent
+ * parent (→ 0 results), and empty `tags`/`search` add noise. Treat all of
+ * these as "field omitted".
+ */
+
+export const ZERO_OBJECT_ID = '000000000000000000000000';
+
+export function isBlankValue(value) {
+  if (value === undefined || value === null) return true;
+  if (typeof value === 'string') {
+    const trimmed = value.trim();
+    return trimmed === '' || trimmed === ZERO_OBJECT_ID;
+  }
+  if (Array.isArray(value)) return value.length === 0;
+  return false;
+}
+
+/**
+ * LLM callers often put the date in `datePlain` and leave `dueDate` empty.
+ * The backend derives datePlain FROM dueDate but not the reverse, so a
+ * datePlain-only task ends up with dueDate=null and never appears on day/agenda
+ * views (it looks "not created"). Mirror datePlain -> dueDate in that case.
+ * Mutates and returns `body`.
+ */
+export function applyDueDateFallback(body) {
+  if (isBlankValue(body.dueDate) && !isBlankValue(body.datePlain)) {
+    body.dueDate = body.datePlain;
+  }
+  return body;
+}