wolfpack-mcp 1.0.91 → 1.0.93

package/dist/agentBuilderTools.js

@@ -113,10 +113,6 @@ export const AGENT_BUILDER_TOOLS = [
                     items: { type: 'string' },
                     description: 'Replace the full set of MCP permissions granted to the agent\'s session API key (e.g. ["mcp:work_items:read", "mcp:comments:create"]). Grant only what the agent\'s tasks need. An empty array falls back to a read-only default.',
                 },
-                scheduling_enabled: {
-                    type: 'boolean',
-                    description: "The agent-level scheduling switch. No schedule fires while it is off, whatever the schedule's own enabled flag says.",
-                },
                 ...ORG_SLUG_PROP,
             },
             required: ['agent_id'],
@@ -354,10 +350,6 @@ export const AGENT_BUILDER_TOOLS = [
                 agent_id: { type: 'string', description: 'Agent profile ID' },
                 name: { type: 'string', description: 'Task name' },
                 prompt: { type: 'string', description: 'What the agent should do when this task runs' },
-                scheduled_enabled: {
-                    type: 'boolean',
-                    description: 'Whether this task is included in scheduled runs (default false)',
-                },
                 sort_order: {
                     type: 'number',
                     description: 'Order within scheduled batch (lower = first)',
@@ -388,7 +380,7 @@ export const AGENT_BUILDER_TOOLS = [
     },
     {
         name: 'update_agent_task',
-        description: "Update a task's prompt, schedule settings, model pin, or disabled tools/MCP servers.",
+        description: "Update a task's prompt, sort order, model pin, or disabled tools/MCP servers.",
         inputSchema: {
             type: 'object',
             properties: {
@@ -396,7 +388,6 @@ export const AGENT_BUILDER_TOOLS = [
                 task_id: { type: 'string', description: 'Task ID' },
                 name: { type: 'string', description: 'Updated task name' },
                 prompt: { type: 'string', description: 'Updated prompt' },
-                scheduled_enabled: { type: 'boolean', description: 'Include in scheduled runs' },
                 sort_order: { type: 'number', description: 'Order within scheduled batch' },
                 is_active: { type: 'boolean', description: 'Whether this task is active' },
                 ...TASK_MODEL_PROPS,
@@ -836,19 +827,6 @@ const AgentTaskSchema = z.object({
     org_slug: orgSlugField,
 });
 const SkillIdSchema = z.object({ skill_id: z.string(), org_slug: orgSlugField });
-/**
- * A schedule enabled on an agent whose own scheduling switch is off never
- * fires (#2083) — say so in the result rather than letting the caller find
- * out from a schedule that stays silent.
- */
-async function schedulingOffWarning(client, agentId, scheduleEnabled, orgSlug) {
-    if (!scheduleEnabled)
-        return '';
-    const agent = await client.getAgent(agentId, orgSlug);
-    return !agent || agent.schedulingEnabled
-        ? ''
-        : "\n\nWARNING: this agent's scheduling is switched off, so the schedule will not fire until update_agent sets scheduling_enabled: true.";
-}
 export async function handleAgentBuilderTool(name, args, client) {
     const text = (t) => JSON.stringify(t, null, 2);
     switch (name) {
@@ -906,7 +884,6 @@ export async function handleAgentBuilderTool(name, args, client) {
                 llm_model: z.string().nullable().optional(),
                 llm_family: z.string().nullable().optional(),
                 api_key_permissions: z.array(z.string()).optional(),
-                scheduling_enabled: z.boolean().optional(),
                 org_slug: orgSlugField,
             })
                 .parse(args);
@@ -930,8 +907,6 @@ export async function handleAgentBuilderTool(name, args, client) {
                 fields.llmFamily = rest.llm_family;
             if (rest.api_key_permissions !== undefined)
                 fields.apiKeyPermissions = rest.api_key_permissions;
-            if (rest.scheduling_enabled !== undefined)
-                fields.schedulingEnabled = rest.scheduling_enabled;
             const agent = await client.updateAgent(agent_id, fields, resolveOrg(parsed));
             return { content: [{ type: 'text', text: `Updated agent\n\n${text(agent)}` }] };
         }
@@ -1095,7 +1070,6 @@ export async function handleAgentBuilderTool(name, args, client) {
                 agent_id: z.string(),
                 name: z.string(),
                 prompt: z.string(),
-                scheduled_enabled: z.boolean().optional(),
                 sort_order: z.number().optional(),
                 llm_provider: z.string().nullable().optional(),
                 llm_model: z.string().nullable().optional(),
@@ -1109,7 +1083,6 @@ export async function handleAgentBuilderTool(name, args, client) {
             const task = await client.createAgentTask(parsed.agent_id, {
                 name: parsed.name,
                 prompt: parsed.prompt,
-                scheduledEnabled: parsed.scheduled_enabled,
                 sortOrder: parsed.sort_order,
                 llmProvider: parsed.llm_provider,
                 llmModel: parsed.llm_model,
@@ -1127,7 +1100,6 @@ export async function handleAgentBuilderTool(name, args, client) {
                 task_id: z.string(),
                 name: z.string().optional(),
                 prompt: z.string().optional(),
-                scheduled_enabled: z.boolean().optional(),
                 sort_order: z.number().optional(),
                 is_active: z.boolean().optional(),
                 llm_provider: z.string().nullable().optional(),
@@ -1144,8 +1116,6 @@ export async function handleAgentBuilderTool(name, args, client) {
                 fields.name = parsed.name;
             if (parsed.prompt !== undefined)
                 fields.prompt = parsed.prompt;
-            if (parsed.scheduled_enabled !== undefined)
-                fields.scheduledEnabled = parsed.scheduled_enabled;
             if (parsed.sort_order !== undefined)
                 fields.sortOrder = parsed.sort_order;
             if (parsed.is_active !== undefined)
@@ -1256,12 +1226,11 @@ export async function handleAgentBuilderTool(name, args, client) {
             if (rest.task_ids !== undefined)
                 body.taskIds = rest.task_ids;
             const schedule = await client.createAgentSchedule(agent_id, body, resolveOrg(parsed));
-            const warning = await schedulingOffWarning(client, agent_id, schedule.enabled, resolveOrg(parsed));
             return {
                 content: [
                     {
                         type: 'text',
-                        text: `Created schedule "${schedule.name}"${warning}\n\n${text(schedule)}`,
+                        text: `Created schedule "${schedule.name}"\n\n${text(schedule)}`,
                     },
                 ],
             };
@@ -1316,9 +1285,8 @@ export async function handleAgentBuilderTool(name, args, client) {
             if (rest.task_ids !== undefined)
                 body.taskIds = rest.task_ids;
             const schedule = await client.updateAgentSchedule(agent_id, schedule_id, body, resolveOrg(parsed));
-            const warning = await schedulingOffWarning(client, agent_id, schedule.enabled, resolveOrg(parsed));
             return {
-                content: [{ type: 'text', text: `Updated schedule${warning}\n\n${text(schedule)}` }],
+                content: [{ type: 'text', text: `Updated schedule\n\n${text(schedule)}` }],
             };
         }
         // ─── Skills ─────────────────────────────────────────────────────────────

package/dist/agentBuilderTools.test.js

@@ -98,38 +98,28 @@ describe('extra instructions on a task run over MCP (#2048)', () => {
         expect(runAgentTask.mock.calls[0][6]).toBeUndefined();
     });
 });
-// #2083: everything the schedule editor can set is settable over MCP, and a
-// schedule enabled on an agent whose scheduling switch is off says so.
-describe('schedule pre-flight and scheduling switch over MCP (#2083)', () => {
+// #2083: everything the schedule editor can set is settable over MCP.
+describe('schedule pre-flight over MCP (#2083)', () => {
     const SCHEDULE_FIELDS = ['preflight_check', 'check_kinds', 'skip_while_active'];
     it.each(['create_agent_schedule', 'update_agent_schedule'])('%s advertises the pre-flight fields', (name) => {
         const properties = toolNamed(name).inputSchema.properties;
         expect(Object.keys(properties)).toEqual(expect.arrayContaining(SCHEDULE_FIELDS));
     });
-    it('update_agent advertises the scheduling switch', () => {
-        const properties = toolNamed('update_agent').inputSchema.properties;
-        expect(Object.keys(properties)).toContain('scheduling_enabled');
-    });
-    function scheduleClient(schedulingEnabled) {
+    function scheduleClient() {
         const schedule = { id: 'sched-1', name: 'Reviews', enabled: true };
         const createAgentSchedule = vi.fn().mockResolvedValue(schedule);
         const updateAgentSchedule = vi.fn().mockResolvedValue(schedule);
-        const updateAgent = vi.fn().mockResolvedValue({ id: 'agent-1', schedulingEnabled });
-        const getAgent = vi.fn().mockResolvedValue({ id: 'agent-1', schedulingEnabled });
         return {
             client: {
                 createAgentSchedule,
                 updateAgentSchedule,
-                updateAgent,
-                getAgent,
             },
             createAgentSchedule,
             updateAgentSchedule,
-            updateAgent,
         };
     }
     it('sends the pre-flight, kinds and skip-while-active when creating a schedule', async () => {
-        const { client, createAgentSchedule } = scheduleClient(true);
+        const { client, createAgentSchedule } = scheduleClient();
         await handleAgentBuilderTool('create_agent_schedule', {
             agent_id: 'agent-1',
             preflight_check: 'claimable-check',
@@ -143,23 +133,14 @@ describe('schedule pre-flight and scheduling switch over MCP (#2083)', () => {
         });
     });
     it('sends the same fields when updating a schedule', async () => {
-        const { client, updateAgentSchedule } = scheduleClient(true);
+        const { client, updateAgentSchedule } = scheduleClient();
         await handleAgentBuilderTool('update_agent_schedule', { agent_id: 'agent-1', schedule_id: 'sched-1', preflight_check: null, check_kinds: [] }, client);
         expect(updateAgentSchedule.mock.calls[0][2]).toEqual({ preflightCheck: null, checkKinds: [] });
     });
-    it('sends the scheduling switch when updating an agent', async () => {
-        const { client, updateAgent } = scheduleClient(true);
-        await handleAgentBuilderTool('update_agent', { agent_id: 'agent-1', scheduling_enabled: true }, client);
-        expect(updateAgent.mock.calls[0][1]).toEqual({ schedulingEnabled: true });
-    });
-    it('warns when an enabled schedule lands on an agent whose scheduling is off', async () => {
-        const { client } = scheduleClient(false);
-        const result = await handleAgentBuilderTool('create_agent_schedule', { agent_id: 'agent-1', enabled: true }, client);
-        expect(result.content[0].text).toContain('scheduling is switched off');
-    });
-    it('stays quiet when the agent is scheduling', async () => {
-        const { client } = scheduleClient(true);
-        const result = await handleAgentBuilderTool('create_agent_schedule', { agent_id: 'agent-1', enabled: true }, client);
-        expect(result.content[0].text).not.toContain('WARNING');
+    // #2067 retired the profile-level scheduling switch: a schedule's own
+    // enabled flag is the only switch, so update_agent must not offer another.
+    it('update_agent no longer advertises a scheduling switch', () => {
+        const properties = toolNamed('update_agent').inputSchema.properties;
+        expect(Object.keys(properties)).not.toContain('scheduling_enabled');
     });
 });

package/dist/client.js

@@ -252,6 +252,9 @@ export class WolfpackClient {
         const query = params.toString();
         return this.api.get(this.withTeamSlug(`/work-item-checks${query ? `?${query}` : ''}`, teamSlug));
     }
+    async addWorkItemCheck(workItemId, body, teamSlug) {
+        return this.api.post(this.withTeamSlug(`/work-items/${workItemId}/checks`, teamSlug), body);
+    }
     async claimWorkItemCheck(workItemId, checkId, teamSlug) {
         return this.api.post(this.withTeamSlug(`/work-items/${workItemId}/checks/${checkId}/claim`, teamSlug), {});
     }

package/dist/index.js

@@ -234,6 +234,18 @@ const ListWorkItemChecksSchema = z.object({
         .optional()
         .describe('Project slug (required for multi-project users, use list_projects to get slugs)'),
 });
+const AddWorkItemCheckSchema = z.object({
+    work_item_id: refIdString().describe('The refId of the work item'),
+    kind: z.string().describe('The check kind to add, e.g. security-review'),
+    comment: z
+        .string()
+        .optional()
+        .describe('Why the check is being requested — posted as a comment on the item'),
+    project_slug: z
+        .string()
+        .optional()
+        .describe('Project slug (required for multi-project users, use list_projects to get slugs)'),
+});
 const ClaimWorkItemCheckSchema = z.object({
     work_item_id: refIdString().describe('The refId of the work item'),
     check_id: z.string().describe('The check ID (from list_work_item_checks)'),
@@ -1096,7 +1108,8 @@ class WolfpackMCPServer {
                             'Checks on blocked items are left out — blocked work is not picked up, and claiming one is refused. ' +
                             'REVIEW WORKFLOW: claim a check with claim_work_item_check, get_work_item to read the work, review it, ' +
                             'then complete_work_item_check with "passed" or "failed" and a short summary. ' +
-                            'A failed check sends the item back to its developer with your summary as feedback; do not change the item status yourself.',
+                            'A failed check sends the item back to its developer with your summary as feedback; do not change the item status yourself. ' +
+                            'To request a further review of another kind on an item (e.g. security-review), use add_work_item_check.',
                         inputSchema: {
                             type: 'object',
                             properties: {
@@ -1120,6 +1133,31 @@ class WolfpackMCPServer {
                             },
                         },
                     },
+                    {
+                        name: 'add_work_item_check',
+                        description: 'Add a pending review check of a kind (e.g. security-review) to a work item you do not lead, so the reviewer that answers that kind picks it up from its inbox. ' +
+                            "Use it when a change touches a sensitive path and needs a second, specialised review. A case waiting on the item's checks keeps waiting until the new check is resolved. " +
+                            'Refused for a kind already pending on the item, for person-only kinds, and on your own work.',
+                        inputSchema: {
+                            type: 'object',
+                            properties: {
+                                work_item_id: { type: 'string', description: 'The refId of the work item' },
+                                kind: {
+                                    type: 'string',
+                                    description: 'The check kind to add, e.g. security-review',
+                                },
+                                comment: {
+                                    type: 'string',
+                                    description: 'Why the check is being requested — posted as a comment on the item',
+                                },
+                                project_slug: {
+                                    type: 'string',
+                                    description: 'Project slug (required for multi-project users, use list_projects to get slugs)',
+                                },
+                            },
+                            required: ['work_item_id', 'kind'],
+                        },
+                    },
                     {
                         name: 'claim_work_item_check',
                         description: 'Claim a pending, unclaimed review check on a work item so other reviewers know it is being handled. ' +
@@ -2506,6 +2544,19 @@ class WolfpackMCPServer {
                             : JSON.stringify(checks.map(formatWorkItemCheck), null, 2);
                         return { content: [{ type: 'text', text }] };
                     }
+                    case 'add_work_item_check': {
+                        const parsed = AddWorkItemCheckSchema.parse(args);
+                        const teamSlug = parsed.project_slug || this.client.getProjectSlug() || undefined;
+                        const check = await this.client.addWorkItemCheck(parsed.work_item_id, { kind: parsed.kind, comment: parsed.comment }, teamSlug);
+                        return {
+                            content: [
+                                {
+                                    type: 'text',
+                                    text: `Added ${check.kind} check to #${check.workItemRefId} "${check.workItemTitle}"\n\n${JSON.stringify(formatWorkItemCheck(check), null, 2)}`,
+                                },
+                            ],
+                        };
+                    }
                     case 'claim_work_item_check': {
                         const parsed = ClaimWorkItemCheckSchema.parse(args);
                         const teamSlug = parsed.project_slug || this.client.getProjectSlug() || undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wolfpack-mcp",
-  "version": "1.0.91",
+  "version": "1.0.93",
   "description": "MCP server for Wolfpack AI-enhanced software delivery tools",
   "main": "dist/index.js",
   "type": "module",