agent-planner-mcp 1.5.3 → 1.5.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-planner-mcp",
-  "version": "1.5.3",
+  "version": "1.5.5",
   "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.
@@ -486,6 +486,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.
@@ -938,7 +940,10 @@ async function formIntentionHandler(args, apiClient) {
         continue;
       }
       try {
-        await apiClient.axiosInstance.post('/dependencies', {
+        // Plan-scoped route — the bare /dependencies path is unmounted (only
+        // /dependencies/cross-plan + /external exist), so it 404s and silently
+        // dropped every inline edge. POST /plans/:id/dependencies is canonical.
+        await apiClient.axiosInstance.post(`/plans/${plan.id}/dependencies`, {
           source_node_id: sourceId,
           target_node_id: intent.targetId,
           dependency_type: 'blocks',
@@ -963,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',
@@ -972,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 =
@@ -1133,7 +1139,9 @@ async function proposeResearchChainHandler(args, apiClient) {
     [created.plan.id, created.implement.id],
   ]) {
     try {
-      await apiClient.axiosInstance.post('/dependencies', {
+      // Plan-scoped route — bare /dependencies 404s (unmounted), which silently
+      // dropped both chain edges and left 3 orphan tasks.
+      await apiClient.axiosInstance.post(`/plans/${plan_id}/dependencies`, {
         source_node_id: from,
         target_node_id: to,
         dependency_type: 'blocks',
@@ -1144,9 +1152,11 @@ async function proposeResearchChainHandler(args, apiClient) {
     }
   }
 
+  const edgeFailures = failures.filter((f) => f.step === 'create_edge');
   return formatResponse({
     as_of: asOf(),
     plan_id,
+    url: planUrl(plan_id),
     parent_id,
     rationale,
     research: { id: created.research.id, title: created.research.title },
@@ -1154,6 +1164,11 @@ async function proposeResearchChainHandler(args, apiClient) {
     implement: { id: created.implement.id, title: created.implement.title },
     edges,
     failures,
+    // Fail loudly: don't let the caller believe the chain is wired when the
+    // blocking edges didn't get created.
+    ...(edgeFailures.length
+      ? { warning: `Chain tasks created but ${edgeFailures.length} blocking edge(s) FAILED — the Research→Plan→Implement ordering is NOT wired: ${edgeFailures.map((f) => f.error).join('; ')}` }
+      : {}),
     next_step: "Claim the Research task with claim_next_task({plan_id}) to begin investigation.",
   });
 }