@yemi33/minions 0.1.1878 → 0.1.1880

package/dashboard/js/render-plans.js

@@ -61,12 +61,60 @@ async function refreshPlans() {
   } catch (e) { console.error('plans refresh:', e.message); }
 }
 
+/**
+ * Returns true when a failed plan follow-up work item's intended outcome has
+ * been satisfied by a later/alternate path — so the stale failure should not
+ * drive the plan's top-level "has failures" badge.
+ *
+ * Reconciliation rules (issue #2366):
+ * - Failed `itemType: 'pr'` (Create PR for plan): satisfied when a PR linked
+ *   to the same plan exists in any non-terminated state (active/merged/linked).
+ *   The original failure is preserved on the WI for audit; it just doesn't
+ *   count toward the rollup once a PR actually got created.
+ * - Failed `itemType: 'verify'`: satisfied when a newer verify WI for the
+ *   same plan + project exists and is done OR in-flight (pending/dispatched).
+ *
+ * Returns false when the outcome cannot be confirmed satisfied — failure
+ * remains live and drives the badge.
+ */
+function isFollowUpOutcomeSatisfied(wi, allWorkItems, allPrs) {
+  if (!wi || wi.status !== 'failed') return false;
+  if (wi.itemType === 'pr') {
+    // PR for this plan was created on any non-terminated path.
+    // "abandoned" / "closed" are terminal failures; everything else counts as
+    // the desired outcome being reached.
+    return (allPrs || []).some(pr =>
+      pr && pr.sourcePlan === wi.sourcePlan &&
+      pr.status !== 'abandoned' && pr.status !== 'closed'
+    );
+  }
+  if (wi.itemType === 'verify') {
+    // A newer verify WI for the same plan + project that is in flight or done.
+    const wiCreated = wi.created || '';
+    return (allWorkItems || []).some(other =>
+      other && other !== wi && other.itemType === 'verify' &&
+      other.sourcePlan === wi.sourcePlan &&
+      (other.project || '') === (wi.project || '') &&
+      (other.status === 'done' || other.status === 'pending' || other.status === 'dispatched') &&
+      (other.created || '') > wiCreated
+    );
+  }
+  return false;
+}
+
 /**
  * Derive effective plan/PRD status from work items (single source of truth).
  * PRD JSON status is treated as user intent (approved, paused, rejected),
  * but completion/progress is always derived from actual work item state.
+ *
+ * opts.allPrs — optional PR records used to reconcile stale failed follow-up
+ * tasks (issue #2366). Falls back to window._lastStatus.pullRequests in the
+ * browser. Pass `[]` to disable reconciliation.
  */
-function derivePlanStatus(prdFile, mdFile, prdJsonStatus, workItems) {
+function derivePlanStatus(prdFile, mdFile, prdJsonStatus, workItems, opts) {
+  const allPrs = (opts && opts.allPrs)
+    || (typeof window !== 'undefined' && window._lastStatus && window._lastStatus.pullRequests)
+    || [];
   const wi = workItems.filter(w =>
     w.sourcePlan === prdFile || w.sourcePlan === mdFile ||
     (w.type === 'plan-to-prd' && (w.planFile === prdFile || w.planFile === mdFile))
@@ -74,8 +122,18 @@ function derivePlanStatus(prdFile, mdFile, prdJsonStatus, workItems) {
   const implementWi = wi.filter(w => w.type !== 'plan-to-prd' && w.type !== 'verify');
   const hasPendingPrd = wi.some(w => w.type === 'plan-to-prd' && (w.status === 'pending' || w.status === 'dispatched'));
   const hasActiveWork = implementWi.some(w => w.status === 'pending' || w.status === 'dispatched');
-  const allDone = implementWi.length > 0 && implementWi.every(w => w.status === 'done' || w.status === 'decomposed');
-  const hasFailed = implementWi.some(w => w.status === 'failed');
+  // A WI is "effectively done" when it succeeded normally OR when it is a
+  // reconciled follow-up failure (intended outcome satisfied elsewhere — #2366).
+  // The original failure record is preserved on the WI for audit; it just
+  // stops blocking plan completion and stops driving the red badge.
+  const isReconciledFailure = (w) =>
+    w.status === 'failed' &&
+    (w.itemType === 'pr' || w.itemType === 'verify') &&
+    isFollowUpOutcomeSatisfied(w, workItems, allPrs);
+  const isEffectivelyDone = (w) =>
+    w.status === 'done' || w.status === 'decomposed' || isReconciledFailure(w);
+  const allDone = implementWi.length > 0 && implementWi.every(isEffectivelyDone);
+  const hasFailed = implementWi.some(w => w.status === 'failed' && !isReconciledFailure(w));
 
   // User-set statuses take priority when no work has started
   if (prdJsonStatus === 'rejected') return 'rejected';
@@ -897,4 +955,4 @@ async function planUnarchive(file, btn) {
   } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
-window.MinionsPlans = { openCreatePlanModal, refreshPlans, derivePlanStatus, renderPlans, openArchivedPlansModal, planExecute, planReexecuteModal, planReexecuteSubmit, planSubmitRevise, planShowRevise, planHideRevise, planView, planApprove, planArchive, planUnarchive, planDelete, planPause, planReject, planDiscuss, planOpenInDocChat, planRegeneratePRD, openVerifyGuide, triggerVerify };
+window.MinionsPlans = { openCreatePlanModal, refreshPlans, derivePlanStatus, isFollowUpOutcomeSatisfied, renderPlans, openArchivedPlansModal, planExecute, planReexecuteModal, planReexecuteSubmit, planSubmitRevise, planShowRevise, planHideRevise, planView, planApprove, planArchive, planUnarchive, planDelete, planPause, planReject, planDiscuss, planOpenInDocChat, planRegeneratePRD, openVerifyGuide, triggerVerify };

package/knowledge/agents/README.md

@@ -0,0 +1,34 @@
+# Per-Agent Memory Files
+
+This directory holds **per-agent personal memory files** — one markdown file per
+named agent (e.g. `dallas.md`, `rebecca.md`). Each file is a personal notebook
+for that agent: the consolidation pipeline appends inbox findings authored by
+the agent to their own file, and the engine injects the agent's file into their
+prompt context on every dispatch (alongside the broadcast `notes.md`).
+
+## How it works
+
+1. **Authoring**: When an agent writes a learnings file under
+   `notes/inbox/<agent>-<task>-<date>.md`, the YAML frontmatter `agent:` field
+   identifies the author. Filename prefix (`<agent>-...`) is used as fallback.
+2. **Consolidation**: `engine/consolidation.js` runs on every tick. After the
+   shared `notes.md` digest is written, `classifyToKnowledgeBase` also
+   appends each item to `knowledge/agents/<agent>.md` when the agent is a
+   known team member (matches `config.agents`). This is **strict superset**
+   semantics — broadcast consolidation still happens; per-agent routing is in
+   addition.
+3. **Prompt injection**: `engine/playbook.js` injects, in order, `pinned.md`,
+   `notes.md`, then `knowledge/agents/<agentId>.md` (when present) into the
+   agent's prompt. Missing files are silently skipped.
+
+## Conventions
+
+- One `.md` file per agent, named `<agent-id>.md` (lowercase).
+- Append-only from the consolidation engine — agents should not edit their
+  own memory files directly. To record a learning, write to
+  `notes/inbox/` as usual; the sweep will route it.
+- Capped at ~25 KB per file; oldest sections are pruned at section
+  boundaries to stay under budget.
+- Frontmatter `agent:` in inbox notes is the canonical author field.
+- Temp agents (e.g. `temp-mp1fnkze...`) are skipped — only configured
+  team members get a memory file.

package/knowledge/agents/dallas.md

@@ -0,0 +1,9 @@
+# Dallas — Personal Memory
+
+This is Dallas's personal memory file, curated by the consolidation pipeline
+from inbox findings tagged `agent: dallas`. The engine injects this file into
+Dallas's prompt context on every dispatch (alongside `notes.md`).
+
+See `knowledge/agents/README.md` for the convention.
+
+<!-- Append-only; consolidation engine writes new sections below. -->

package/knowledge/agents/lambert.md

@@ -0,0 +1,9 @@
+# Lambert — Personal Memory
+
+This is Lambert's personal memory file, curated by the consolidation pipeline
+from inbox findings tagged `agent: lambert`. The engine injects this file into
+Lambert's prompt context on every dispatch (alongside `notes.md`).
+
+See `knowledge/agents/README.md` for the convention.
+
+<!-- Append-only; consolidation engine writes new sections below. -->

package/knowledge/agents/ralph.md

@@ -0,0 +1,9 @@
+# Ralph — Personal Memory
+
+This is Ralph's personal memory file, curated by the consolidation pipeline
+from inbox findings tagged `agent: ralph`. The engine injects this file into
+Ralph's prompt context on every dispatch (alongside `notes.md`).
+
+See `knowledge/agents/README.md` for the convention.
+
+<!-- Append-only; consolidation engine writes new sections below. -->

package/knowledge/agents/rebecca.md

@@ -0,0 +1,9 @@
+# Rebecca — Personal Memory
+
+This is Rebecca's personal memory file, curated by the consolidation pipeline
+from inbox findings tagged `agent: rebecca`. The engine injects this file into
+Rebecca's prompt context on every dispatch (alongside `notes.md`).
+
+See `knowledge/agents/README.md` for the convention.
+
+<!-- Append-only; consolidation engine writes new sections below. -->

package/knowledge/agents/ripley.md

@@ -0,0 +1,9 @@
+# Ripley — Personal Memory
+
+This is Ripley's personal memory file, curated by the consolidation pipeline
+from inbox findings tagged `agent: ripley`. The engine injects this file into
+Ripley's prompt context on every dispatch (alongside `notes.md`).
+
+See `knowledge/agents/README.md` for the convention.
+
+<!-- Append-only; consolidation engine writes new sections below. -->

package/package.json

@@ -1,11 +1,14 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1878",
+  "version": "0.1.1880",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"
   },
   "scripts": {
+    "package:check": "node tools/check-package-boundary.js public",
+    "package:prepare:public": "node tools/prepare-package.js public",
+    "package:prepare:internal": "node tools/prepare-package.js internal",
     "test": "node test/run-parallel.js",
     "test:sequential": "node test/unit.test.js",
     "test:unit": "node test/run-parallel.js",
@@ -32,11 +35,28 @@
   ],
   "author": "yemi33",
   "license": "MIT",
+  "files": [
+    "agents/*/charter.md",
+    "bin/",
+    "config.template.json",
+    "dashboard/",
+    "dashboard-build.js",
+    "dashboard.js",
+    "docs/",
+    "engine/",
+    "engine.js",
+    "knowledge/agents/*.md",
+    "minions.js",
+    "playbooks/",
+    "prompts/",
+    "routing.md",
+    "skills/"
+  ],
   "repository": {
     "type": "git",
     "url": "https://github.com/yemi33/minions.git"
   },
-  "homepage": "https://github.com/yemi33/minions#readme",
+  "homepage": "https://yemi33.github.io/minions/",
   "engines": {
     "node": ">=18"
   },

package/.claudeignore

@@ -1,5 +0,0 @@
-agents/
-engine/tmp/
-engine/contexts/
-prd/*.backup
-plans/archive/

package/.gitattributes

@@ -1,2 +0,0 @@
-* text=auto eol=lf
-*.ps1 text eol=crlf