surecore-studio-mcp 0.1.0 → 0.1.1

package/dist/index.js

@@ -5,6 +5,34 @@ import { z } from 'zod';
 import { loadConfig } from './config.js';
 import { createApiClient } from './apiClient.js';
 import { CREATE_TASK_TYPES, listMyTasks, getTaskTool, searchTasks, updateTaskStatus, createTaskTool, updateTask, listComments, addComment, getAttachment, addAttachment, listMembersTool, } from './tools.js';
+// ---------------------------------------------------------------------------
+// Workspace cache — statuses and tags are user-definable and change at any
+// time. Re-fetch at most once per TTL so every tool call uses a fresh list
+// without hammering the API on every keystroke.
+// ---------------------------------------------------------------------------
+const WS_TTL_MS = 60_000;
+class WorkspaceCache {
+    api;
+    ws = null;
+    fetchedAt = 0;
+    constructor(api) {
+        this.api = api;
+    }
+    async get() {
+        if (this.ws && Date.now() - this.fetchedAt < WS_TTL_MS)
+            return this.ws;
+        const [statuses, tags, members] = await Promise.all([
+            this.api.listStatuses(),
+            this.api.listTags().catch(() => []),
+            this.api.listMembers().catch(() => []),
+        ]);
+        this.ws = { statuses, tags, members };
+        this.fetchedAt = Date.now();
+        return this.ws;
+    }
+    /** Call after a local mutation so the next read is always fresh. */
+    invalidate() { this.fetchedAt = 0; }
+}
 async function main() {
     const cfg = loadConfig();
     const api = createApiClient(cfg);
@@ -17,28 +45,20 @@ async function main() {
         process.stderr.write('surecore-tasks: could not authenticate — check SURECORE_PAT (generate one in Settings › Developer).\n');
         process.exit(1);
     }
-    // Read the workspace's OWN vocabulary. Statuses and tags are user-definable,
-    // so shipping a fixed list here would make anything created later invisible.
-    // Tags and members are optional conveniences — a failure there must not stop
-    // the server from serving tasks.
-    const statuses = await api.listStatuses();
-    const [tags, members] = await Promise.all([
-        api.listTags().catch(() => []),
-        api.listMembers().catch(() => []),
-    ]);
-    const ws = { statuses, tags, members };
-    const statusList = statuses.map((s) => s.value);
-    const tagList = tags.map((t) => t.value);
-    // z.enum needs a non-empty tuple; statuses always have at least todo/done,
-    // but a workspace can legitimately have zero tags.
-    const statusSchema = z.enum(statusList);
-    const tagSchema = tagList.length
-        ? z.enum(tagList)
-        : z.string().describe('This workspace has no tags defined yet.');
-    const server = new McpServer({ name: 'surecore-tasks', version: '0.1.6' });
-    server.tool('list_my_tasks', 'List the tasks assigned to you, optionally filtered by status, project or tag.', { status: statusSchema.optional(), projectId: z.string().optional(), tag: tagSchema.optional() }, (args) => listMyTasks(api, me, args));
-    server.tool('get_task', 'Get one task in full: its fields, description, assignee names, comment thread and the list of attached files. Use this for "read task N" — a second call is not needed for comments.', { task: z.string().describe('Task number or appId') }, (args) => getTaskTool(api, args, ws, me));
-    server.tool('search_tasks', `Search tasks by status, project, tag and/or free text. Statuses in this workspace: ${statusList.join(', ')}.${tagList.length ? ` Tags: ${tagList.join(', ')}.` : ''}`, { status: statusSchema.optional(), projectId: z.string().optional(), text: z.string().optional(), tag: tagSchema.optional() }, (args) => searchTasks(api, args, ws));
+    const wsCache = new WorkspaceCache(api);
+    // -------------------------------------------------------------------------
+    // Status and tag params use z.string() rather than z.enum(). The enum was
+    // baked in at startup and silently rejected any value added after the
+    // process started. The server validates the value anyway (and returns a
+    // clear error listing current valid values), so the enum gate is redundant
+    // and harmful.
+    // -------------------------------------------------------------------------
+    const statusParam = z.string().describe('A status value (e.g. "todo", "in-progress"). The server validates against the workspace\'s current list and returns valid options on a mismatch.');
+    const tagParam = z.string().describe('A tag value defined in this workspace.');
+    const server = new McpServer({ name: 'surecore-tasks', version: '0.1.7' });
+    server.tool('list_my_tasks', 'List the tasks assigned to you, optionally filtered by status, project or tag.', { status: statusParam.optional(), projectId: z.string().optional(), tag: tagParam.optional() }, async (args) => listMyTasks(api, me, args));
+    server.tool('get_task', 'Get one task in full: its fields, description, assignee names, comment thread and the list of attached files. Use this for "read task N" — a second call is not needed for comments.', { task: z.string().describe('Task number or appId') }, async (args) => getTaskTool(api, args, await wsCache.get(), me));
+    server.tool('search_tasks', 'Search tasks by status, project, tag and/or free text. Statuses and tags are fetched fresh on each call so newly created values are always recognised.', { status: statusParam.optional(), projectId: z.string().optional(), text: z.string().optional(), tag: tagParam.optional() }, async (args) => searchTasks(api, args, await wsCache.get()));
     server.tool('create_task', 'Create a new task or issue under a project (NOT a ticket — tickets come from support intake). Confirm the title, project, and type with the user before calling. `project` accepts a project name or id.', {
         title: z.string().describe('The task title'),
         project: z.string().describe('Project name (e.g. "AAIB") or project id'),
@@ -51,14 +71,14 @@ async function main() {
         title: z.string().optional(),
         description: z.string().optional(),
         priority: z.enum(['low', 'medium', 'high']).optional(),
-        status: statusSchema.optional(),
+        status: statusParam.optional(),
         dueDate: z.string().optional().describe('YYYY-MM-DD'),
-        tags: z.array(tagSchema).optional().describe('Replaces the task\'s tags'),
+        tags: z.array(tagParam).optional().describe('Replaces the task\'s tags'),
         assignees: z.array(z.string()).optional().describe('Member names or ids; replaces the current assignees'),
-    }, (args) => updateTask(api, args, ws));
-    server.tool('update_task_status', 'Move a task to a new status. Confirm the task and the target status with the user before calling. For customer-linked tickets this notifies the customer, exactly like the web app.', { task: z.string().describe('Task number or appId'), status: statusSchema }, (args) => updateTaskStatus(api, args, ws));
+    }, async (args) => { const ws = await wsCache.get(); return updateTask(api, args, ws); });
+    server.tool('update_task_status', 'Move a task to a new status. Confirm the task and the target status with the user before calling. For customer-linked tickets this notifies the customer, exactly like the web app.', { task: z.string().describe('Task number or appId'), status: statusParam }, async (args) => { const ws = await wsCache.get(); return updateTaskStatus(api, args, ws); });
     server.tool('get_attachment', 'Pull one file attached to a task into the conversation. Images are shown inline; other types are reported by name and type. `file` accepts the file name or its id.', { task: z.string().describe('Task number or appId'), file: z.string().describe('Attachment file name or id') }, (args) => getAttachment(api, args));
-    server.tool('list_members', 'List the workspace members with their ids. Use this to turn a member id seen on a task into a person, or to find the id for an assignee.', {}, () => listMembersTool(ws));
+    server.tool('list_members', 'List the workspace members with their ids. Use this to turn a member id seen on a task into a person, or to find the id for an assignee.', {}, async () => listMembersTool(await wsCache.get()));
     server.tool('add_attachment', 'Attach a file from this machine to a task. Give the full path. Used to attach a work report to a task when the work is finished.', { task: z.string().describe('Task number or appId'), path: z.string().describe('Full path to the file on this machine') }, (args) => addAttachment(api, args));
     server.tool('list_comments', 'Read the comment thread on a task, oldest first.', { task: z.string().describe('Task number or appId') }, (args) => listComments(api, args));
     server.tool('add_comment', 'Post a comment on a task. It is always attributed to you (the token holder). Confirm the wording with the user before calling.', { task: z.string().describe('Task number or appId'), text: z.string().describe('The comment body') }, (args) => addComment(api, me, args));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "surecore-studio-mcp",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "MCP server for SurecoreStudio: read and update tasks, comments and attachments from Claude.",
   "license": "MIT",
   "author": "Surecore",
@@ -46,4 +46,4 @@
   "publishConfig": {
     "access": "public"
   }
-}
+}