agent-planner-mcp 1.5.1 → 1.5.4

package/package.json

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

@@ -463,8 +463,10 @@ async function listPlansHandler(args, apiClient) {
       const q = filter.query.toLowerCase();
       plans = plans.filter((p) => (p.title || '').toLowerCase().includes(q));
     }
-    plans = plans.slice(0, filter.limit || 50);
 
+    // Summarize the FULL filtered set BEFORE paginating — otherwise `total` and
+    // the status counts only reflect the truncated page, making an agent think
+    // it has seen every plan when it hasn't.
     const summary = plans.reduce(
       (acc, p) => {
         acc[p.status] = (acc[p.status] || 0) + 1;
@@ -474,10 +476,15 @@ async function listPlansHandler(args, apiClient) {
       { total: 0 },
     );
 
+    const limit = filter.limit || 50;
+    const page = plans.slice(0, limit);
+    summary.returned = page.length;
+    summary.truncated = plans.length > page.length; // true → raise `filter.limit` to see the rest
+
     return formatResponse({
       as_of: asOf(),
       summary,
-      plans: plans.map((p) => ({
+      plans: page.map((p) => ({
         id: p.id,
         title: p.title,
         status: p.status,
@@ -537,7 +544,16 @@ async function searchHandler(args, apiClient) {
       result = await apiClient.search.searchPlan(scope_id, query);
     } else {
       const global = await apiClient.search.globalSearch(query);
-      const all = Array.isArray(global?.results) ? global.results : [];
+      // The backend returns `results` as a GROUPED object
+      // ({ plans, nodes, comments, logs }), not a flat array — so the old
+      // `Array.isArray(results) ? … : []` always fell to [] and global search
+      // returned nothing. Flatten both shapes.
+      const r = global?.results;
+      const all = Array.isArray(r)
+        ? r
+        : r && typeof r === 'object'
+          ? [...(r.plans || []), ...(r.nodes || []), ...(r.comments || []), ...(r.logs || [])]
+          : [];
       const matchScope = (r) => {
         if (scope === 'global') return true;
         if (scope === 'plans') return r.type === 'plan';

package/src/tools/bdi/intentions.js

@@ -938,7 +938,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',
@@ -1133,7 +1136,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,6 +1149,7 @@ async function proposeResearchChainHandler(args, apiClient) {
     }
   }
 
+  const edgeFailures = failures.filter((f) => f.step === 'create_edge');
   return formatResponse({
     as_of: asOf(),
     plan_id,
@@ -1154,6 +1160,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.",
   });
 }