@yemi33/minions 0.1.1154 → 0.1.1156

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.1154 (2026-04-18)
+## 0.1.1156 (2026-04-18)
 
 ### Features
 - redact ADO tokens and JWTs from engine/log.json writes (#1297)
@@ -9,6 +9,8 @@
 - seed realActivityMap at spawn time, stamp pid in live-output (#1200)
 
 ### Fixes
+- scheduler substitutes {{date}} in title and description (#1275)
+-  prevent archived plans from re-triggering via orphaned .backup restore
 - pinned context optimistic save (closes #1295) (#1316)
 -  re-check throttle state per-PR iteration to avoid stale fixThrottled
 - preserve buildErrorLog through transient states, persist poll time (#1273)
@@ -27,8 +29,6 @@
 - cap temp-agent creation at maxConcurrent per tick (#1219)
 -  reassign pending items from unspawned temp agents to idle named agents (#1204) (#1212)
 - guard undefined agent in pending dispatch loop (closes #1206) (#1210)
--  improve fallback meeting conclusion
--  remove command center chevron
 
 ### Other
 - refactor: hoist fixThrottled before PR loop and drop underscore prefix

package/engine/lifecycle.js

@@ -355,8 +355,12 @@ function archivePlan(planFile, plan, projects, config) {
     // Remove .backup sidecar — if left behind, safeJson() would restore the pre-completion
     // snapshot (status: approved, no _completionNotified) on engine restart, re-triggering
     // plan completion and spawning duplicate verify tasks for already-archived plans.
+    // On Windows, the unlink can fail due to file locking; overwrite with archived status
+    // as a fallback so a restored backup is inert even if deletion fails.
     const backupPath = planPath + '.backup';
-    if (fs.existsSync(backupPath)) fs.unlinkSync(backupPath);
+    try { fs.unlinkSync(backupPath); } catch {
+      try { fs.writeFileSync(backupPath, JSON.stringify({ status: 'archived' })); } catch { }
+    }
   } catch (err) {
     log('warn', `Failed to archive PRD ${planFile}: ${err.message}`);
   }

package/engine/scheduler.js

@@ -24,10 +24,29 @@
 const fs = require('fs');
 const path = require('path');
 const shared = require('./shared');
-const { safeJson, safeWrite, mutateJsonFileLocked, ts, WI_STATUS } = shared;
+const { safeJson, safeWrite, mutateJsonFileLocked, ts, dateStamp, WI_STATUS } = shared;
 
 const SCHEDULE_RUNS_PATH = path.join(__dirname, 'schedule-runs.json');
 
+/**
+ * Substitute schedule-time template variables in a string.
+ * Currently supports:
+ *   {{date}} — today's date as YYYY-MM-DD (UTC, via dateStamp())
+ *
+ * Downstream playbook rendering (engine/playbook.js) is a single-pass replace,
+ * so any {{date}} embedded in a schedule's title/description would survive
+ * substitution of {{task_description}} and surface as an "unresolved template
+ * variables: date" warning plus a literal "{{date}}" in agent filenames.
+ * Resolve these fields at schedule time so the work item carries a concrete
+ * date string from the moment it's created.
+ *
+ * Safe on undefined/null/empty/non-string inputs — returns the input unchanged.
+ */
+function resolveScheduleTemplateVars(str) {
+  if (typeof str !== 'string' || str.length === 0) return str;
+  return str.replace(/\{\{date\}\}/g, dateStamp());
+}
+
 // Parse a single cron field into a matcher function.
 // field: e.g., "*", "5", "1,3,5", "*/15"
 // min/max: valid range (0-59 for minute, 0-23 for hour, 0-6 for dow)
@@ -141,13 +160,16 @@ function discoverScheduledWork(config) {
       const lastRun = typeof runEntry === 'string' ? runEntry : (runEntry?.lastRun || null);
       if (!shouldRunNow(sched, lastRun)) continue;
 
+      // Substitute schedule-time template vars (e.g. {{date}}) before the work
+      // item is written — single-pass playbook rendering can't reach placeholders
+      // embedded inside task_description, so they must be resolved up front.
       const workItemId = `sched-${sched.id}-${Date.now()}`;
       work.push({
         id: workItemId,
-        title: sched.title,
+        title: resolveScheduleTemplateVars(sched.title),
         type: sched.type || 'implement',
         priority: sched.priority || 'medium',
-        description: sched.description || sched.title,
+        description: resolveScheduleTemplateVars(sched.description || sched.title),
         status: WI_STATUS.PENDING,
         created: ts(),
         createdBy: 'scheduler',
@@ -170,4 +192,4 @@ function discoverScheduledWork(config) {
   return work;
 }
 
-module.exports = { parseCronExpr, parseCronField, shouldRunNow, discoverScheduledWork, SCHEDULE_RUNS_PATH };
+module.exports = { parseCronExpr, parseCronField, shouldRunNow, discoverScheduledWork, resolveScheduleTemplateVars, SCHEDULE_RUNS_PATH };

package/engine.js

@@ -3130,6 +3130,13 @@ async function discoverWork(config) {
       if (fs.existsSync(prdDir)) {
         for (const f of fs.readdirSync(prdDir).filter(f => f.endsWith('.json'))) {
           if (completedPlanCache.has(f)) continue;
+          if (fs.existsSync(path.join(prdDir, 'archive', f))) {
+            // Orphaned backup restore — plan is already archived. Purge the ghost copy.
+            try { fs.unlinkSync(path.join(prdDir, f)); } catch { }
+            try { fs.unlinkSync(path.join(prdDir, f + '.backup')); } catch { }
+            completedPlanCache.add(f);
+            continue;
+          }
           const plan = safeJson(path.join(prdDir, f));
           if (!plan?.missing_features || plan.status === 'completed') {
             if (plan?.status === 'completed') completedPlanCache.add(f);
@@ -3309,6 +3316,13 @@ async function tickInner() {
       const prdFiles = safeReadDir(PRD_DIR).filter(f => f.endsWith('.json'));
       for (const file of prdFiles) {
         if (completedPlanCache.has(file)) continue;
+        if (fs.existsSync(path.join(PRD_DIR, 'archive', file))) {
+          // Orphaned backup restore — plan is already archived. Purge the ghost copy.
+          try { fs.unlinkSync(path.join(PRD_DIR, file)); } catch { }
+          try { fs.unlinkSync(path.join(PRD_DIR, file + '.backup')); } catch { }
+          completedPlanCache.add(file);
+          continue;
+        }
         const plan = safeJson(path.join(PRD_DIR, file));
         if (plan && plan.missing_features && plan.status !== 'completed') {
           const completed = checkPlanCompletion({ item: { sourcePlan: file } }, config);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1154",
+  "version": "0.1.1156",
   "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"