@yemi33/minions 0.1.1730 → 0.1.1732

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.1732 (2026-05-05)
+
+### Features
+- fix pipeline cron display (#2087)
+
 ## 0.1.1730 (2026-05-05)
 
 ### Features

package/dashboard/js/render-pipelines.js

@@ -268,6 +268,13 @@ function _buildNodeChain(stages, run, options) {
   return html;
 }
 
+function _renderPipelineTriggerLabel(cron) {
+  if (!cron) return 'Manual trigger';
+  var human = _cronToHuman(cron);
+  var tz = Intl.DateTimeFormat().resolvedOptions().timeZone;
+  return escHtml(human) + ' <span style="opacity:0.6">(' + escHtml(tz) + ')</span>';
+}
+
 function renderPipelines(pipelines) {
   pipelines = (pipelines || []).filter(function(p) { return !isDeleted('pipeline:' + p.id); });
   _pipelinesData = pipelines;
@@ -331,7 +338,7 @@ function openPipelineDetail(id) {
   // Status + actions
   var activeRun = _getPipelineActiveRun(p);
   html += '<div style="display:flex;justify-content:space-between;align-items:center">' +
-    '<span style="font-size:10px;color:var(--muted)">' + (p.trigger?.cron ? escHtml(_cronToHuman(p.trigger.cron)) + ' <span style="opacity:0.6">(' + escHtml(p.trigger.cron) + ', ' + escHtml(Intl.DateTimeFormat().resolvedOptions().timeZone) + ')</span>' : 'Manual trigger') + ' · ' + escHtml(_getPipelineStageLabel(p)) + '</span>' +
+    '<span style="font-size:10px;color:var(--muted)">' + _renderPipelineTriggerLabel(p.trigger?.cron) + ' · ' + escHtml(_getPipelineStageLabel(p)) + '</span>' +
     '<div style="display:flex;gap:6px">' +
       (activeRun
         ? '<button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_abortPipeline(\'' + escHtml(id) + '\',this)">Abort</button>' +

package/dashboard/js/render-schedules.js

@@ -5,14 +5,50 @@
 const _DAYS = ['Sun', 'Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat'];
 const _DAY_NAMES = ['Sundays', 'Mondays', 'Tuesdays', 'Wednesdays', 'Thursdays', 'Fridays', 'Saturdays'];
 
-/** Convert 3-field cron (minute hour dayOfWeek) to human-readable text */
+function _formatCronDowList(days, timeStr, fallback) {
+  const normalized = [...new Set(days)].sort((a, b) => a - b);
+  const key = normalized.join(',');
+  if (key === '0,1,2,3,4,5,6') return 'Daily at ' + timeStr;
+  if (key === '1,2,3,4,5') return 'Weekdays at ' + timeStr;
+  if (key === '0,6') return 'Weekends at ' + timeStr;
+  if (normalized.length === 1) return _DAY_NAMES[normalized[0]] + ' at ' + timeStr;
+  if (normalized.length > 1) return normalized.map(d => _DAY_NAMES[d]).join(', ') + ' at ' + timeStr;
+  return fallback;
+}
+
+function _parseCronDowList(dow, allowSevenAsSunday) {
+  if (!dow || typeof dow !== 'string') return null;
+  const values = [];
+  const parts = dow.split(',');
+  for (const part of parts) {
+    const token = part.trim();
+    if (!/^\d+$/.test(token)) return null;
+    let day = parseInt(token, 10);
+    if (allowSevenAsSunday && day === 7) day = 0;
+    if (day < 0 || day > 6) return null;
+    values.push(day);
+  }
+  return values;
+}
+
+function _cronTimeString(minute, hour) {
+  const h = parseInt(hour, 10);
+  const m = parseInt(minute, 10);
+  if (isNaN(h) || isNaN(m)) return null;
+  return String(h).padStart(2, '0') + ':' + String(m).padStart(2, '0');
+}
+
+/** Convert 3-field Minions cron or common 5-field cron to human-readable text */
 function _cronToHuman(cron) {
   if (!cron || typeof cron !== 'string') return cron || '';
   const parts = cron.trim().split(/\s+/);
-  if (parts.length !== 3) return cron;
-  const [minute, hour, dow] = parts;
+  if (parts.length !== 3 && parts.length !== 5) return cron;
+  const minute = parts[0];
+  const hour = parts[1];
+  const dow = parts.length === 3 ? parts[2] : parts[4];
 
-  if (minute === '*' && hour === '*' && dow === '*') return 'Every minute';
+  if (parts.length === 5 && (parts[2] !== '*' || parts[3] !== '*')) return cron;
+  if (parts.length === 3 && minute === '*' && hour === '*' && dow === '*') return 'Every minute';
 
   const h = parseInt(hour, 10);
   const m = parseInt(minute, 10);
@@ -26,13 +62,20 @@ function _cronToHuman(cron) {
   const normalized = dow.split(',').map(d => d.trim()).sort().join(',');
   if (dow === '1-5' || normalized === '1,2,3,4,5') return 'Weekdays at ' + timeStr;
   if (normalized === '0,6' || normalized === '6,0') return 'Weekends at ' + timeStr;
+  if (parts.length === 5 && (dow === '0-6' || dow === '0-7')) return 'Daily at ' + timeStr;
 
   // Single day
-  const dayNum = parseInt(dow, 10);
-  if (!isNaN(dayNum) && dayNum >= 0 && dayNum <= 6 && String(dayNum) === dow) {
+  let dayNum = parseInt(dow, 10);
+  if (parts.length === 5 && dayNum === 7 && dow === '7') dayNum = 0;
+  if (!isNaN(dayNum) && dayNum >= 0 && dayNum <= 6 && String(parseInt(dow, 10)) === dow) {
     return _DAY_NAMES[dayNum] + ' at ' + timeStr;
   }
 
+  if (parts.length === 5) {
+    const days = _parseCronDowList(dow, true);
+    if (days) return _formatCronDowList(days, timeStr, cron);
+  }
+
   return cron;
 }
 

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-05T09:50:02.177Z"
+  "cachedAt": "2026-05-05T09:53:42.976Z"
 }

package/engine/project-discovery.js

@@ -245,23 +245,33 @@ function execGit(execFileSync, targetDir, args, timeout = 5000) {
   })).trim();
 }
 
+function parseOriginHeadBranch(headRef) {
+  const branch = String(headRef || '').trim().replace(/^refs\/remotes\/origin\//, '');
+  return branch && branch !== 'HEAD' ? branch : '';
+}
+
+function discoverMainBranch(execFileSync, targetDir) {
+  try {
+    return parseOriginHeadBranch(execGit(execFileSync, targetDir, ['symbolic-ref', 'refs/remotes/origin/HEAD']));
+  } catch {}
+
+  try {
+    execGit(execFileSync, targetDir, ['remote', 'set-head', 'origin', '-a'], 10000);
+    return parseOriginHeadBranch(execGit(execFileSync, targetDir, ['symbolic-ref', 'refs/remotes/origin/HEAD']));
+  } catch {}
+
+  return '';
+}
+
 function discoverProjectMetadata(targetDir, options = {}) {
   const execFileSync = options.execFileSync || defaultExecFileSync;
   const result = { _found: [] };
 
-  try {
-    let head = '';
-    try {
-      head = execGit(execFileSync, targetDir, ['symbolic-ref', 'refs/remotes/origin/HEAD']);
-    } catch {
-      head = execGit(execFileSync, targetDir, ['symbolic-ref', 'HEAD']);
-    }
-    const branch = head.replace('refs/remotes/origin/', '').replace('refs/heads/', '');
-    if (branch) {
-      result.mainBranch = branch;
-      result._found.push('main branch');
-    }
-  } catch {}
+  const branch = discoverMainBranch(execFileSync, targetDir);
+  if (branch) {
+    result.mainBranch = branch;
+    result._found.push('main branch');
+  }
 
   try {
     const remoteUrl = execGit(execFileSync, targetDir, ['remote', 'get-url', 'origin']);

package/package.json

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