@zoobbe/cli 1.2.3 → 1.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zoobbe/cli",
-  "version": "1.2.3",
+  "version": "1.3.0",
   "description": "Zoobbe CLI - Manage boards, cards, and projects from the terminal",
   "main": "src/index.js",
   "bin": {

package/src/commands/card.js

@@ -23,30 +23,21 @@ card
       let cards = [];
 
       if (options.board) {
-        // Get lists with populated cards for this board
+        // Get lists with populated cards for this board (single API call)
         const listsData = await withSpinner('Fetching cards...', () =>
           client.get(`/boards/${options.board}/lists`)
         );
         const lists = listsData.lists || listsData.data || listsData;
         const listArr = Array.isArray(lists) ? lists : Object.values(lists);
 
-        // Collect card shortIds per list
-        const cardRefs = [];
+        // Extract cards directly from list data (no extra API calls)
         for (const list of listArr) {
           if (list.cards && Array.isArray(list.cards)) {
             for (const c of list.cards) {
-              cardRefs.push({ id: c.shortId || c._id, listName: list.title || '' });
+              cards.push({ ...c, _listName: list.title || '' });
             }
           }
         }
-
-        // Fetch full card details in parallel
-        const results = await Promise.allSettled(
-          cardRefs.map(ref => client.get(`/cards/${ref.id}`).then(d => ({ ...d, _listName: ref.listName })))
-        );
-        for (const r of results) {
-          if (r.status === 'fulfilled') cards.push(r.value);
-        }
       } else {
         // Get cards for current user
         const userId = config.get('userId');
@@ -333,7 +324,7 @@ card
   .action(async (cardId) => {
     try {
       await withSpinner('Updating card...', () =>
-        client.post(`/cards/update/${cardId}`, { isComplete: true })
+        client.post(`/cards/update/${cardId}`, { completed: true })
       );
 
       success(`Card ${cardId} marked as complete`);
@@ -658,7 +649,7 @@ card
   .action(async (cardId) => {
     try {
       await withSpinner('Updating card...', () =>
-        client.post(`/cards/update/${cardId}`, { isComplete: false })
+        client.post(`/cards/update/${cardId}`, { completed: false })
       );
       success(`Card ${cardId} marked as incomplete`);
     } catch (err) {

package/src/mcp-server.js

@@ -19,6 +19,25 @@ const config = require('./lib/config');
 // Mark all requests from MCP server as AI agent actions
 client.setSource('mcp');
 
+// ── Priority Normalization ──
+// Backend expects: Low, Normal, High, Urgent
+// MCP accepts both formats for convenience
+const PRIORITY_MAP = {
+  low: 'Low',
+  normal: 'Normal',
+  medium: 'Normal',
+  high: 'High',
+  urgent: 'Urgent',
+  critical: 'Urgent',
+};
+
+function normalizePriority(value) {
+  if (!value) return null;
+  const mapped = PRIORITY_MAP[value.toLowerCase()];
+  if (!mapped) throw new Error(`Invalid priority "${value}". Valid: Low, Normal, High, Urgent (or: low, medium, high, critical)`);
+  return mapped;
+}
+
 // ── Tool Definitions ──
 
 const TOOLS = [
@@ -201,7 +220,7 @@ const TOOLS = [
         list_id: { type: 'string', description: 'List ID (uses first list if omitted)' },
         description: { type: 'string', description: 'Card description' },
         due_date: { type: 'string', description: 'Due date (YYYY-MM-DD)' },
-        priority: { type: 'string', description: 'Priority: low, medium, high, critical' },
+        priority: { type: 'string', description: 'Priority: Low, Normal, High, Urgent (also accepts: low, medium, high, critical)' },
       },
       required: ['title', 'board_id'],
     },
@@ -297,7 +316,7 @@ const TOOLS = [
       type: 'object',
       properties: {
         card_id: { type: 'string', description: 'Card short ID' },
-        priority: { type: 'string', description: 'Priority: low, medium, high, critical' },
+        priority: { type: 'string', description: 'Priority: Low, Normal, High, Urgent (also accepts: low, medium, high, critical)' },
       },
       required: ['card_id', 'priority'],
     },
@@ -383,7 +402,7 @@ async function executeTool(name, args = {}) {
     }
 
     case 'list_lists': {
-      const data = await client.get(`/${args.board_id}/actionLists/`);
+      const data = await client.get(`/boards/${args.board_id}/lists`);
       let lists = data.lists || data.data || data;
       lists = Array.isArray(lists) ? lists : Object.values(lists);
       return lists.filter(l => !l.archived).map(l => ({
@@ -427,21 +446,16 @@ async function executeTool(name, args = {}) {
       const lists = listsData.lists || listsData.data || listsData;
       const listArr = Array.isArray(lists) ? lists : Object.values(lists);
 
-      const cardRefs = [];
+      // Extract cards directly from list data (single API call, no N+1)
+      let cards = [];
       for (const list of listArr) {
         if (list.cards && Array.isArray(list.cards)) {
           for (const c of list.cards) {
-            cardRefs.push({ id: c.shortId || c._id, listName: list.title || '' });
+            cards.push({ ...c, _listName: list.title || '' });
           }
         }
       }
 
-      const results = await Promise.allSettled(
-        cardRefs.map(ref => client.get(`/cards/${ref.id}`).then(d => ({ ...d, _listName: ref.listName })))
-      );
-
-      let cards = results.filter(r => r.status === 'fulfilled').map(r => r.value);
-
       if (args.due) {
         const now = new Date();
         cards = cards.filter(c => {
@@ -457,6 +471,7 @@ async function executeTool(name, args = {}) {
       return cards.map(c => ({
         title: c.title || c.name,
         id: c.shortId || c._id,
+        cardNumber: c.cardNumber || null,
         list: c._listName || c.actionListTitle || '',
         due: c.dueDate?.date || c.dueDate || null,
         members: c.members?.length || 0,
@@ -471,6 +486,7 @@ async function executeTool(name, args = {}) {
       return {
         title: c.title || c.name,
         id: c.shortId || c._id,
+        cardNumber: c.cardNumber || null,
         board: c.boardTitle || c.board,
         list: c.actionListTitle || c.actionList?.name,
         description: c.description || '',
@@ -525,6 +541,7 @@ async function executeTool(name, args = {}) {
       return (Array.isArray(cards) ? cards : []).map(c => ({
         title: c.title || c.name,
         id: c.shortId || c._id,
+        cardNumber: c.cardNumber || null,
         board: c.boardTitle || '',
         list: c.actionListTitle || '',
         due: c.dueDate?.date || c.dueDate || null,
@@ -534,8 +551,13 @@ async function executeTool(name, args = {}) {
     }
 
     case 'search': {
-      const data = await client.get(`/search?q=${encodeURIComponent(args.query)}&workspaceId=${encodeURIComponent(workspaceId)}`);
-      return data.data || data;
+      const data = await client.post(`/global/search?query=${encodeURIComponent(args.query)}`, {});
+      const results = data.data || data;
+      const out = [];
+      if (results.boards) out.push(...results.boards.map(b => ({ type: 'board', title: b.title || b.name, id: b.shortId || b._id })));
+      if (results.cards) out.push(...results.cards.map(c => ({ type: 'card', title: c.title, id: c.shortId || c._id, board: c.boardTitle || '' })));
+      if (results.pages) out.push(...results.pages.map(p => ({ type: 'page', title: p.title, id: p.shortId || p._id })));
+      return out.length ? out : results;
     }
 
     case 'list_notifications': {
@@ -580,10 +602,10 @@ async function executeTool(name, args = {}) {
         await client.post(`/cards/${cardId}/duedate`, { dueDate: args.due_date });
       }
       if (args.priority) {
-        await client.post(`/cards/${cardId}/priority`, { priority: args.priority });
+        await client.post(`/cards/${cardId}/priority`, { priority: normalizePriority(args.priority) });
       }
 
-      return { created: true, title: args.title, id: cardId, changed_by: 'AI Agent (via MCP)' };
+      return { created: true, title: args.title, id: cardId, cardNumber: c.cardNumber || null, changed_by: 'AI Agent (via MCP)' };
     }
 
     case 'update_card': {
@@ -637,12 +659,12 @@ async function executeTool(name, args = {}) {
     }
 
     case 'mark_card_done': {
-      await client.post(`/cards/update/${args.card_id}`, { isComplete: true });
+      await client.post(`/cards/update/${args.card_id}`, { completed: true });
       return { done: true, card_id: args.card_id, changed_by: 'AI Agent (via MCP)' };
     }
 
     case 'mark_card_undone': {
-      await client.post(`/cards/update/${args.card_id}`, { isComplete: false });
+      await client.post(`/cards/update/${args.card_id}`, { completed: false });
       return { undone: true, card_id: args.card_id, changed_by: 'AI Agent (via MCP)' };
     }
 
@@ -656,8 +678,9 @@ async function executeTool(name, args = {}) {
     }
 
     case 'set_priority': {
-      await client.post(`/cards/${args.card_id}/priority`, { priority: args.priority });
-      return { set: true, card_id: args.card_id, priority: args.priority, changed_by: 'AI Agent (via MCP)' };
+      const normalized = normalizePriority(args.priority);
+      await client.post(`/cards/${args.card_id}/priority`, { priority: normalized });
+      return { set: true, card_id: args.card_id, priority: normalized, changed_by: 'AI Agent (via MCP)' };
     }
 
     case 'archive_card': {
@@ -701,7 +724,7 @@ async function executeTool(name, args = {}) {
 
 const SERVER_INFO = {
   name: 'zoobbe',
-  version: '1.2.2',
+  version: '1.3.0',
 };
 
 function makeResponse(id, result) {