agent-planner-mcp 1.5.4 → 1.5.6

package/SKILL.md

@@ -123,7 +123,7 @@ The `update_task` call is atomic — status change, log entry, claim release, an
 
 ```
 1. claim_next_task(scope={ plan_id }, ttl_minutes=30) → exclusive ownership
-2. task_context(task_id, depth=4) periodically to refresh as work progresses
+2. task_context(node_id, depth=4) periodically to refresh as work progresses
 3. update_task(...) for state transitions
 4. release_task(task_id, message='handoff to teammate') for explicit handoff
 ```

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-planner-mcp",
-  "version": "1.5.4",
+  "version": "1.5.6",
   "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/_shared.js

@@ -26,6 +26,24 @@ function safeArray(value) {
   return Array.isArray(value) ? value : [];
 }
 
+/**
+ * The web app origin (where /app/plans/:id lives), for building shareable plan
+ * links agents can post (e.g. to Slack). Derived from API_URL — the web app
+ * shares the origin and the API sits under /api behind nginx — with an explicit
+ * AGENTPLANNER_WEB_URL override for local/self-hosted setups where the UI is on
+ * a different host/port.
+ */
+function webOrigin() {
+  if (process.env.AGENTPLANNER_WEB_URL) return process.env.AGENTPLANNER_WEB_URL.replace(/\/+$/, '');
+  const api = process.env.API_URL || 'https://agentplanner.io/api';
+  return api.replace(/\/+$/, '').replace(/\/api$/, '') || 'https://agentplanner.io';
+}
+
+/** Shareable web link to a plan (set the plan's visibility to unlisted/public for a rich unfurl). */
+function planUrl(planId) {
+  return planId ? `${webOrigin()}/app/plans/${planId}` : null;
+}
+
 /**
  * True when an error means the backend has no /v1 surface (pre-consolidation
  * self-hosted API). Express returns a default 404 with no structured body for
@@ -38,4 +56,4 @@ function isV1Unavailable(err) {
   return !(body && typeof body === 'object' && body.error);
 }
 
-module.exports = { asOf, formatResponse, errorResponse, safeArray, isV1Unavailable };
+module.exports = { asOf, formatResponse, errorResponse, safeArray, isV1Unavailable, webOrigin, planUrl };

package/src/tools/bdi/beliefs.js

@@ -5,7 +5,7 @@
  * plan_analysis. Each answers one whole agentic question and returns `as_of`.
  */
 
-const { asOf, formatResponse, errorResponse, safeArray, isV1Unavailable } = require('./_shared');
+const { asOf, formatResponse, errorResponse, safeArray, isV1Unavailable, planUrl } = require('./_shared');
 
 // ─────────────────────────────────────────────────────────────────────────
 // briefing — bundled mission control state. Replaces 4 round trips.
@@ -197,18 +197,26 @@ const taskContextDefinition = {
   inputSchema: {
     type: 'object',
     properties: {
-      task_id: { type: 'string' },
+      // `node_id` is the canonical name (matches every other tool + the skill
+      // docs). `task_id` is kept as an accepted alias for back-compat.
+      node_id: { type: 'string', description: 'The task/node id to load context for.' },
+      task_id: { type: 'string', description: 'Alias for node_id (back-compat).' },
       depth: { type: 'integer', enum: [1, 2, 3, 4], default: 2 },
       token_budget: { type: 'integer', default: 0 },
     },
-    required: ['task_id'],
+    // Neither is strictly required at the schema level because either name is
+    // accepted; the handler validates that one was supplied with a clear error.
   },
 };
 
 async function taskContextHandler(args, apiClient) {
-  const { task_id, depth = 2, token_budget = 0 } = args;
+  const { node_id, task_id, depth = 2, token_budget = 0 } = args;
+  const nodeId = node_id || task_id;
+  if (!nodeId) {
+    return errorResponse('invalid_arg', 'task_context requires node_id (the task/node id).');
+  }
   const params = new URLSearchParams({
-    node_id: task_id,
+    node_id: nodeId,
     depth: String(depth),
     token_budget: String(token_budget),
     log_limit: '10',
@@ -486,6 +494,7 @@ async function listPlansHandler(args, apiClient) {
       summary,
       plans: page.map((p) => ({
         id: p.id,
+        url: planUrl(p.id),
         title: p.title,
         status: p.status,
         visibility: p.visibility,

package/src/tools/bdi/intentions.js

@@ -13,7 +13,7 @@
  * See ../../../docs/MCP_v1.0_FULL_SURFACE.md for design rationale.
  */
 
-const { asOf, formatResponse, errorResponse, isV1Unavailable } = require('./_shared');
+const { asOf, formatResponse, errorResponse, isV1Unavailable, planUrl } = require('./_shared');
 const { version: PKG_VERSION } = require('../../../package.json');
 
 // Provenance tag stamped onto every plan this server creates, so a plan stays
@@ -854,16 +854,18 @@ async function formIntentionHandler(args, apiClient) {
         tree,
         client_version: CLIENT_TAG,
       });
+      const facadePlanId = result.plan?.id || result.plan_id;
       return formatResponse({
         ...result,
-        plan_id: result.plan?.id || result.plan_id,
+        plan_id: facadePlanId,
+        url: planUrl(facadePlanId),
         goal_id,
         status: result.plan?.status || status,
         is_draft: (result.plan?.status || status) === 'draft',
         nodes_created: Array.isArray(result.tree) ? result.tree.length : undefined,
         next_step: (result.plan?.status || status) === 'draft'
           ? "Plan created as draft. Will surface in dashboard pending for human review. Auto-promotes to active when first task moves to in_progress."
-          : "Plan active. Claim a task with claim_next_task({plan_id}) to begin work.",
+          : `Plan active. Claim a task with claim_next_task({plan_id}) to begin work. Shareable link: ${planUrl(facadePlanId)} (set visibility:'unlisted' for a rich Slack/social preview).`,
       });
     } catch {
       // Fall through to the legacy multi-call path for older/self-hosted APIs.
@@ -966,6 +968,7 @@ async function formIntentionHandler(args, apiClient) {
   const response = {
     as_of: asOf(),
     plan_id: plan.id,
+    url: planUrl(plan.id),
     goal_id,
     status: plan.status,
     is_draft: plan.status === 'draft',
@@ -975,7 +978,7 @@ async function formIntentionHandler(args, apiClient) {
     structure,
     next_step: plan.status === 'draft'
       ? "Plan created as draft. Will surface in dashboard pending for human review. Auto-promotes to active when first task moves to in_progress."
-      : "Plan active. Claim a task with claim_next_task({plan_id}) to begin work.",
+      : `Plan active. Claim a task with claim_next_task({plan_id}) to begin work. Shareable link: ${planUrl(plan.id)} (set visibility:'unlisted' for a rich Slack/social preview).`,
   };
   if (createdWithoutDependencies) {
     response.warning =
@@ -1153,6 +1156,7 @@ async function proposeResearchChainHandler(args, apiClient) {
   return formatResponse({
     as_of: asOf(),
     plan_id,
+    url: planUrl(plan_id),
     parent_id,
     rationale,
     research: { id: created.research.id, title: created.research.title },