@yemi33/minions 0.1.2396 → 0.1.2397

package/dashboard/js/refresh.js

@@ -141,7 +141,8 @@ const RENDER_VERSIONS = {
   projects: 9,
   notes: 1,
   prd: 3,
-  prs: 4,
+  // Bumped 4→5 for shared author/lifecycle/project/review/build filters.
+  prs: 5,
   archivedPrds: 1,
   engine: 4,
   version: 1,

package/dashboard/js/render-prs.js

@@ -1,8 +1,24 @@
 // render-prs.js — PR tracker rendering functions extracted from dashboard.html
 
 let allPrs = [];
+let filteredPrs = [];
 let prPage = 0;
+let prFilterOptionsSignature = '';
 const PR_PER_PAGE = 25;
+const PR_FILTER_CONTROL_IDS = {
+  author: 'pr-filter-author',
+  lifecycle: 'pr-filter-lifecycle',
+  project: 'pr-filter-project',
+  review: 'pr-filter-review',
+  build: 'pr-filter-build',
+};
+const PR_FILTER_ALL_LABELS = {
+  author: 'All authors',
+  lifecycle: 'All lifecycle statuses',
+  project: 'All projects',
+  review: 'All review statuses',
+  build: 'All build statuses',
+};
 
 // Fetch each project's pull-requests.json straight off disk through the
 // /state/projects/<name>/pull-requests.json static-file passthrough. This
@@ -373,32 +389,105 @@ function _restorePrScrollState(el, state) {
   tableWrap.scrollTop = state.top || 0;
 }
 
+function _syncPrFilterOptions(prs) {
+  const options = getPrFilterOptions(prs);
+  const controls = Object.keys(PR_FILTER_CONTROL_IDS).map(function(dimension) {
+    return {
+      dimension,
+      select: document.getElementById(PR_FILTER_CONTROL_IDS[dimension]),
+    };
+  }).filter(function(entry) { return entry.select; });
+  if (!controls.length) return;
+  const signature = JSON.stringify(options);
+  if (signature === prFilterOptionsSignature) return;
+  prFilterOptionsSignature = signature;
+  controls.forEach(function(entry) {
+    const dimension = entry.dimension;
+    const select = entry.select;
+    const selected = select.value;
+    const all = document.createElement('option');
+    all.value = '';
+    all.textContent = PR_FILTER_ALL_LABELS[dimension];
+    const nodes = [all].concat((options[dimension] || []).map(function(option) {
+      const node = document.createElement('option');
+      node.value = option.value;
+      node.textContent = option.label;
+      return node;
+    }));
+    select.replaceChildren(...nodes);
+    if (nodes.some(function(node) { return node.value === selected; })) select.value = selected;
+  });
+}
+
+function _readPrFilters() {
+  const filters = {};
+  Object.keys(PR_FILTER_CONTROL_IDS).forEach(function(dimension) {
+    const select = document.getElementById(PR_FILTER_CONTROL_IDS[dimension]);
+    filters[dimension] = select ? select.value : '';
+  });
+  return filters;
+}
+
+function _hasActivePrFilters(filters) {
+  return Object.keys(PR_FILTER_CONTROL_IDS).some(function(dimension) {
+    return Boolean(filters[dimension]);
+  });
+}
+
+function applyPrFilters() {
+  prPage = 0;
+  renderPrs(allPrs, { preserveScroll: false });
+}
+
+function resetPrFilters() {
+  Object.keys(PR_FILTER_CONTROL_IDS).forEach(function(dimension) {
+    const select = document.getElementById(PR_FILTER_CONTROL_IDS[dimension]);
+    if (select) select.value = '';
+  });
+  applyPrFilters();
+}
+
 function renderPrs(prs, opts) {
   opts = opts || {};
-  prs = prs.filter(p => !isDeleted('pr:' + p.id));
+  prs = Array.isArray(prs) ? prs.filter(p => p && !isDeleted('pr:' + p.id)) : [];
   allPrs = prs;
+  _syncPrFilterOptions(allPrs);
+  const filters = _readPrFilters();
+  filteredPrs = filterPullRequests(allPrs, filters);
   const el = document.getElementById('pr-content');
   const count = document.getElementById('pr-count');
-  count.textContent = prs.length;
-  if (!prs.length) {
+  const summary = document.getElementById('pr-filter-summary');
+  count.textContent = filteredPrs.length;
+  if (summary) {
+    summary.textContent = _hasActivePrFilters(filters)
+      ? 'Showing ' + filteredPrs.length + ' of ' + allPrs.length
+      : '';
+  }
+  if (!allPrs.length) {
+    prPage = 0;
     el.innerHTML = '<p class="pr-empty">No pull requests yet. PRs created by agents will appear here with review, build, and merge status.</p>';
     return;
   }
-  const totalPages = Math.ceil(prs.length / PR_PER_PAGE);
+  if (!filteredPrs.length) {
+    prPage = 0;
+    el.innerHTML = '<p class="pr-empty">No pull requests match the selected filters.</p>';
+    return;
+  }
+  const totalPages = Math.ceil(filteredPrs.length / PR_PER_PAGE);
   const previousPage = prPage;
   if (prPage >= totalPages) prPage = totalPages - 1;
   const start = prPage * PR_PER_PAGE;
-  const pagePrs = prs.slice(start, start + PR_PER_PAGE);
+  const pagePrs = filteredPrs.slice(start, start + PR_PER_PAGE);
   const rows = pagePrs.map(prRow).join('');
 
   let pager = '';
-  if (prs.length > PR_PER_PAGE) {
+  if (filteredPrs.length > PR_PER_PAGE) {
     pager = '<div class="pr-pager">' +
-      '<span class="pr-page-info">Showing ' + (start+1) + ' to ' + Math.min(start+PR_PER_PAGE, prs.length) + ' of ' + prs.length + '</span>' +
+      '<span class="pr-page-info">Showing ' + (start+1) + ' to ' + Math.min(start+PR_PER_PAGE, filteredPrs.length) + ' of ' + filteredPrs.length + '</span>' +
       '<div class="pr-pager-btns">' +
         '<button class="pr-pager-btn ' + (prPage === 0 ? 'disabled' : '') + '" onclick="prPrev()">Prev</button>' +
         '<button class="pr-pager-btn ' + (prPage >= totalPages-1 ? 'disabled' : '') + '" onclick="prNext()">Next</button>' +
-        '<button class="pr-pager-btn see-all" onclick="openAllPrs()">See all ' + prs.length + ' PRs</button>' +
+        '<button class="pr-pager-btn see-all" onclick="openAllPrs()">See all ' + filteredPrs.length + ' PRs</button>' +
       '</div>' +
     '</div>';
   }
@@ -412,14 +501,14 @@ function renderPrs(prs, opts) {
 }
 
 function prPrev() { if (prPage > 0) { prPage--; renderPrs(allPrs, { preserveScroll: false }); } }
-function prNext() { const totalPages = Math.ceil(allPrs.length / PR_PER_PAGE); if (prPage < totalPages-1) { prPage++; renderPrs(allPrs, { preserveScroll: false }); } }
+function prNext() { const totalPages = Math.ceil(filteredPrs.length / PR_PER_PAGE); if (prPage < totalPages-1) { prPage++; renderPrs(allPrs, { preserveScroll: false }); } }
 
 function openAllPrs() {
   const modalEl = document.querySelector('#modal .modal');
   if (modalEl) modalEl.classList.add('modal-wide');
-  document.getElementById('modal-title').textContent = 'All Pull Requests (' + allPrs.length + ')';
+  document.getElementById('modal-title').textContent = 'All Pull Requests (' + filteredPrs.length + ')';
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all PR user data wrapped in escapeHtml() by prRow() (fields: PR id, title, description, agent, branch, review/build/status labels)
-  document.getElementById('modal-body').innerHTML = prTableHtml(allPrs.map(prRow).join(''));
+  document.getElementById('modal-body').innerHTML = prTableHtml(filteredPrs.map(prRow).join(''));
   document.getElementById('modal-body').style.fontFamily = "'Segoe UI', system-ui, sans-serif";
   document.getElementById('modal-body').style.whiteSpace = 'normal';
   document.getElementById('modal').classList.add('open');
@@ -862,6 +951,8 @@ window.MinionsPrs = {
   prRow,
   prTableHtml,
   renderPrs,
+  applyPrFilters,
+  resetPrFilters,
   renderPrDetail: _renderPrDetail, // exported for testing
   prPrev,
   prNext,

package/dashboard/pages/prs.html

@@ -4,5 +4,34 @@
           <span style="font-size:var(--text-sm);color:var(--muted);font-weight:400;text-transform:none;letter-spacing:0">created by agents, tracked with review + build status</span>
         </h2>
         <div id="pr-toast" class="cmd-toast cmd-toast-inline" style="margin:6px 0"></div>
+        <div class="pr-filters" role="group" aria-label="Pull request filters">
+          <label class="pr-filter">Author
+            <select id="pr-filter-author" onchange="applyPrFilters()">
+              <option value="">All authors</option>
+            </select>
+          </label>
+          <label class="pr-filter">Lifecycle
+            <select id="pr-filter-lifecycle" onchange="applyPrFilters()">
+              <option value="">All lifecycle statuses</option>
+            </select>
+          </label>
+          <label class="pr-filter">Project
+            <select id="pr-filter-project" onchange="applyPrFilters()">
+              <option value="">All projects</option>
+            </select>
+          </label>
+          <label class="pr-filter">Review
+            <select id="pr-filter-review" onchange="applyPrFilters()">
+              <option value="">All review statuses</option>
+            </select>
+          </label>
+          <label class="pr-filter">Build
+            <select id="pr-filter-build" onchange="applyPrFilters()">
+              <option value="">All build statuses</option>
+            </select>
+          </label>
+          <button class="pr-filter-reset" type="button" onclick="resetPrFilters()">Reset filters</button>
+          <span class="pr-filter-summary" id="pr-filter-summary" aria-live="polite"></span>
+        </div>
         <div id="pr-content"><p class="pr-empty">No pull requests yet.</p></div>
       </section>

package/dashboard/shared/pr-filters.js

@@ -0,0 +1,166 @@
+// Shared PR filter semantics. Slim embeds the classic /prs screen, and both
+// bundles load this helper so lifecycle and legacy-field normalization stay
+// identical across surfaces.
+var PR_FILTER_DIMENSIONS = ['author', 'lifecycle', 'project', 'review', 'build'];
+var PR_LIFECYCLE_FILTER_LABELS = {
+  open: 'Open / active',
+  merged: 'Merged',
+  closed: 'Closed',
+};
+var PR_FILTER_OPTION_ORDER = {
+  lifecycle: ['open', 'merged', 'closed'],
+  review: ['approved', 'changes-requested', 'waiting', 'pending'],
+  build: ['passing', 'failing', 'running', 'none'],
+};
+
+function _prFilterText(value) {
+  return typeof value === 'string' ? value.trim() : '';
+}
+
+function _prFilterName(value) {
+  var direct = _prFilterText(value);
+  if (direct) return direct;
+  if (!value || typeof value !== 'object') return '';
+  return _prFilterText(value.login)
+    || _prFilterText(value.displayName)
+    || _prFilterText(value.uniqueName)
+    || _prFilterText(value.name);
+}
+
+function _prFilterKey(value) {
+  return _prFilterText(value).toLowerCase();
+}
+
+function normalizePrAuthor(pr) {
+  if (!pr || typeof pr !== 'object') return '';
+  return _prFilterName(pr.author)
+    || _prFilterName(pr.createdBy)
+    || _prFilterName(pr.created_by)
+    || _prFilterName(pr.agent);
+}
+
+function normalizePrLifecycleStatus(pr) {
+  if (!pr || typeof pr !== 'object') return '';
+  var status = _prFilterKey(pr.status || pr.state);
+  if (pr.merged === true || pr.isMerged === true || pr.mergedAt || pr.merged_at
+      || status === 'merged' || status === 'completed') {
+    return 'merged';
+  }
+  if (status === 'active' || status === 'open' || status === 'linked') return 'open';
+  if (status === 'closed' || status === 'abandoned' || status === 'declined') return 'closed';
+  return status;
+}
+
+function normalizePrProject(pr) {
+  if (!pr || typeof pr !== 'object') return '';
+  return _prFilterText(pr._project)
+    || _prFilterText(pr.project)
+    || _prFilterText(pr.sourceProject);
+}
+
+function normalizePrReviewStatus(pr) {
+  if (!pr || typeof pr !== 'object') return '';
+  var minionsStatus = pr.minionsReview && _prFilterText(pr.minionsReview.status);
+  var status = _prFilterKey(minionsStatus || pr.reviewStatus || pr.review_status);
+  if (!minionsStatus && status === 'waiting') {
+    if (normalizePrLifecycleStatus(pr) === 'merged') return 'approved';
+    if (_prFilterKey(pr.status) === 'abandoned') return 'pending';
+  }
+  if (status === 'approve') return 'approved';
+  if (status === 'rejected' || status === 'changes_requested'
+      || status === 'requested-changes' || status === 'changesrequired') {
+    return 'changes-requested';
+  }
+  if (status === 'reviewing') return 'waiting';
+  return status;
+}
+
+function normalizePrBuildStatus(pr) {
+  if (!pr || typeof pr !== 'object') return '';
+  var status = _prFilterKey(pr.buildStatus || pr.build_status);
+  if (status === 'success' || status === 'succeeded') return 'passing';
+  if (status === 'failed' || status === 'error') return 'failing';
+  if (status === 'in-progress' || status === 'in_progress' || status === 'queued') return 'running';
+  if (status === 'not-run' || status === 'not_run') return 'none';
+  return status;
+}
+
+function _prFilterEntry(pr, dimension) {
+  var label = '';
+  var value = '';
+  if (dimension === 'author') {
+    label = normalizePrAuthor(pr);
+    value = _prFilterKey(label);
+  } else if (dimension === 'lifecycle') {
+    value = normalizePrLifecycleStatus(pr);
+    label = PR_LIFECYCLE_FILTER_LABELS[value] || _formatPrFilterLabel(value);
+  } else if (dimension === 'project') {
+    label = normalizePrProject(pr);
+    value = _prFilterKey(label);
+  } else if (dimension === 'review') {
+    value = normalizePrReviewStatus(pr);
+    label = _formatPrFilterLabel(value);
+  } else if (dimension === 'build') {
+    value = normalizePrBuildStatus(pr);
+    label = _formatPrFilterLabel(value);
+  }
+  return value ? { value: value, label: label || value } : null;
+}
+
+function _formatPrFilterLabel(value) {
+  var text = _prFilterText(value).replace(/[-_]+/g, ' ');
+  return text ? text.charAt(0).toUpperCase() + text.slice(1) : '';
+}
+
+function getPrFilterValue(pr, dimension) {
+  var entry = _prFilterEntry(pr, dimension);
+  return entry ? entry.value : '';
+}
+
+function matchesPullRequestFilters(pr, filters) {
+  filters = filters || {};
+  for (var i = 0; i < PR_FILTER_DIMENSIONS.length; i++) {
+    var dimension = PR_FILTER_DIMENSIONS[i];
+    var selected = _prFilterKey(filters[dimension]);
+    if (selected && getPrFilterValue(pr, dimension) !== selected) return false;
+  }
+  return true;
+}
+
+function filterPullRequests(prs, filters) {
+  if (!Array.isArray(prs)) return [];
+  return prs.filter(function(pr) { return matchesPullRequestFilters(pr, filters); });
+}
+
+function getPrFilterOptions(prs) {
+  var result = {};
+  var records = Array.isArray(prs) ? prs : [];
+  PR_FILTER_DIMENSIONS.forEach(function(dimension) {
+    var seen = new Map();
+    records.forEach(function(pr) {
+      var entry = _prFilterEntry(pr, dimension);
+      if (entry && !seen.has(entry.value)) seen.set(entry.value, entry);
+    });
+    var order = PR_FILTER_OPTION_ORDER[dimension] || [];
+    result[dimension] = Array.from(seen.values()).sort(function(a, b) {
+      var ai = order.indexOf(a.value);
+      var bi = order.indexOf(b.value);
+      if (ai !== -1 || bi !== -1) {
+        if (ai === -1) return 1;
+        if (bi === -1) return -1;
+        if (ai !== bi) return ai - bi;
+      }
+      return a.label.localeCompare(b.label, undefined, { sensitivity: 'base' });
+    });
+  });
+  return result;
+}
+
+window.MinionsPrFilters = {
+  author: normalizePrAuthor,
+  normalizeLifecycle: normalizePrLifecycleStatus,
+  value: getPrFilterValue,
+  matches: matchesPullRequestFilters,
+  filter: filterPullRequests,
+  options: getPrFilterOptions,
+};

package/dashboard/styles.css

@@ -322,6 +322,34 @@
   .prd-pending { color: var(--yellow); font-style: italic; }
 
   /* PR Tracker */
+  .pr-filters {
+    display: flex; flex-wrap: wrap; align-items: end; gap: var(--space-4);
+    padding: var(--space-4) 0 var(--space-6);
+  }
+  .pr-filter {
+    display: grid; gap: var(--space-1); min-width: 140px;
+    color: var(--muted); font-size: var(--text-sm); font-weight: 600;
+  }
+  .pr-filter select {
+    min-height: 32px; padding: var(--space-2) var(--space-4);
+    color: var(--text); background: var(--surface2);
+    border: 1px solid var(--border); border-radius: var(--radius-sm);
+    font: inherit; font-weight: 400;
+  }
+  .pr-filter select:focus-visible, .pr-filter-reset:focus-visible {
+    outline: 2px solid var(--blue); outline-offset: 2px;
+  }
+  .pr-filter-reset {
+    min-height: 32px; padding: var(--space-2) var(--space-5);
+    color: var(--blue); background: transparent;
+    border: 1px solid var(--blue); border-radius: var(--radius-sm);
+    cursor: pointer; font-size: var(--text-base);
+  }
+  .pr-filter-reset:hover { background: rgba(88,166,255,0.1); }
+  .pr-filter-summary {
+    min-height: 32px; display: inline-flex; align-items: center;
+    color: var(--muted); font-size: var(--text-base);
+  }
   .pr-table-wrap { overflow-x: auto; }
   .pr-table { width: 100%; border-collapse: collapse; font-size: var(--text-md); table-layout: auto; }
   .pr-table th:last-child, .pr-table td:last-child { width: 36px; min-width: 36px; text-align: center; }
@@ -348,7 +376,7 @@
      wrap) stay visible while the user is on this page. The modal "see all"
      view uses the same colgroup but is unaffected — modal-body handles its
      own scrolling. */
-  #pr-content .pr-table-wrap--prs { max-height: calc(100vh - 200px); overflow: auto; }
+  #pr-content .pr-table-wrap--prs { max-height: calc(100vh - 270px); overflow: auto; }
   #pr-content .pr-table--prs thead th { position: sticky; top: 0; background: var(--surface); z-index: 1; }
   .pr-table th { text-align: left; color: var(--muted); font-weight: 500; font-size: var(--text-base); text-transform: uppercase; letter-spacing: 0.5px; padding: var(--space-4) var(--space-5); border-bottom: 1px solid var(--border); }
   .pr-table td { padding: var(--space-5); border-bottom: 1px solid var(--border); vertical-align: middle; white-space: nowrap; }

package/dashboard-build.js

@@ -8,7 +8,7 @@ const shared = require('./engine/shared');
 const { safeRead } = shared;
 
 const MINIONS_DIR = __dirname;
-const DASHBOARD_SHARED_JS = ['pr-merge-state'];
+const DASHBOARD_SHARED_JS = ['pr-merge-state', 'pr-filters'];
 
 function buildDashboardHtml() {
   const dashDir = path.join(MINIONS_DIR, 'dashboard');

package/dashboard.js

@@ -1796,7 +1796,7 @@ function buildDashboardHtml() {
     'confirm-dialog', 'modal', 'modal-qa', 'settings', 'qa', 'fre', 'refresh'
   ];
   let jsHtml = '';
-  for (const f of ['pr-merge-state']) {
+  for (const f of ['pr-merge-state', 'pr-filters']) {
     const content = safeRead(path.join(dashDir, 'shared', f + '.js'));
     jsHtml += `\n// ─── shared/${f}.js ────────────────────────────────────────\n${content}\n`;
   }

package/package.json

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