agent-planner-mcp 1.5.12 → 1.5.14

package/SKILL.md

@@ -83,6 +83,7 @@ A Workspace is a folder under an Organization that owns goals + plans — a grou
 - `list_blueprints` — list blueprints visible to user (owned + public/unlisted), filterable by scope
 - `fork_blueprint` — instantiate a plan-scope blueprint as a new plan in a target workspace
 - `save_as_blueprint` — snapshot a live plan as a reusable blueprint. Captures structure, agent_instructions, and dependencies; excludes statuses, claims, knowledge episodes, logs, decisions, and agent assignments
+- `delete_blueprint` — delete a blueprint you own (hard delete; already-forked plans are unaffected)
 
 ### Utility
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-planner-mcp",
-  "version": "1.5.12",
+  "version": "1.5.14",
   "description": "MCP server for AgentPlanner — AI agent orchestration with planning, dependencies, knowledge graphs, and human oversight",
   "main": "src/index.js",
   "bin": {

package/src/tools/bdi/desires.js

@@ -114,15 +114,15 @@ async function updateGoalHandler(args, apiClient) {
   // with the backend, but success_criteria must be camelCased to successCriteria
   // — the goal update schema is .strict(), so the snake_case key was rejected
   // with a 400 (the one multi-word field, hence "description writes but
-  // success_criteria fails"). Match the create shape: wrap a bare array as
-  // { criteria: [...] }.
+  // success_criteria fails"). Send the array shape directly: it is the backend's
+  // preferred form. (The old { criteria: [...] } wrap made the backend count
+  // Object.keys()==1 and skip per-criterion knowledge grounding.)
   const directFields = {};
   for (const k of ['title', 'description', 'priority', 'status']) {
     if (changes[k] !== undefined) directFields[k] = changes[k];
   }
   if (changes.success_criteria !== undefined) {
-    const sc = changes.success_criteria;
-    directFields.successCriteria = Array.isArray(sc) ? { criteria: sc } : sc;
+    directFields.successCriteria = changes.success_criteria;
   }
   // Map the public `committed` boolean onto the backend's commitment write
   // (the API still accepts the legacy goalType field and translates it to
@@ -245,7 +245,7 @@ async function deriveSubgoalHandler(args, apiClient) {
     parentGoalId: parent_goal_id,
     organizationId: parent.organization_id || parent.organizationId || undefined,
   };
-  if (success_criteria) payload.successCriteria = { criteria: success_criteria };
+  if (success_criteria) payload.successCriteria = success_criteria;
   if (typeof priority === 'number') payload.priority = priority;
 
   let goal;
@@ -318,7 +318,7 @@ async function createGoalHandler(args, apiClient) {
 
   const payload = { title, type, status };
   if (description) payload.description = description;
-  if (success_criteria) payload.successCriteria = { criteria: success_criteria };
+  if (success_criteria) payload.successCriteria = success_criteria;
   if (typeof priority === 'number') payload.priority = priority;
   if (workspace_id) payload.workspaceId = workspace_id;
 

package/src/tools/bdi/workspaces.js

@@ -8,7 +8,7 @@
  * agent-planner/docs/WORKSPACE_BLUEPRINT_SKETCH.md for the design.
  */
 
-const { asOf, formatResponse, errorResponse, safeArray } = require('./_shared');
+const { asOf, formatResponse, errorResponse, safeArray, apiErrorMessage } = require('./_shared');
 
 // ─── Workspaces ──────────────────────────────────────────────────
 
@@ -197,6 +197,37 @@ async function saveAsBlueprintHandler(args, apiClient) {
   }
 }
 
+const deleteBlueprintDefinition = {
+  name: 'delete_blueprint',
+  description:
+    "Delete a blueprint you own. Hard delete (the snapshot is removed); plans " +
+    "already forked from it are unaffected. Owner-only. Completes the blueprint " +
+    "lifecycle alongside save_as_blueprint and fork_blueprint.",
+  inputSchema: {
+    type: 'object',
+    properties: {
+      blueprint_id: { type: 'string' },
+    },
+    required: ['blueprint_id'],
+  },
+};
+
+async function deleteBlueprintHandler(args, apiClient) {
+  const { blueprint_id } = args;
+  if (!blueprint_id) {
+    return errorResponse('invalid_arg', 'delete_blueprint requires blueprint_id');
+  }
+  try {
+    await apiClient.blueprints.delete(blueprint_id);
+    return formatResponse({ as_of: asOf(), blueprint_id, deleted: true });
+  } catch (err) {
+    const status = err.response?.status;
+    if (status === 404) return errorResponse('not_found', `Blueprint ${blueprint_id} not found`);
+    if (status === 403) return errorResponse('forbidden', 'Only the owner can delete this blueprint');
+    return errorResponse('upstream_unavailable', `delete_blueprint failed: ${apiErrorMessage(err)}`);
+  }
+}
+
 module.exports = {
   definitions: [
     listWorkspacesDefinition,
@@ -204,6 +235,7 @@ module.exports = {
     listBlueprintsDefinition,
     forkBlueprintDefinition,
     saveAsBlueprintDefinition,
+    deleteBlueprintDefinition,
   ],
   handlers: {
     list_workspaces: listWorkspacesHandler,
@@ -211,5 +243,6 @@ module.exports = {
     list_blueprints: listBlueprintsHandler,
     fork_blueprint: forkBlueprintHandler,
     save_as_blueprint: saveAsBlueprintHandler,
+    delete_blueprint: deleteBlueprintHandler,
   },
 };