@yemi33/minions 0.1.2448 → 0.1.2449

package/dashboard/js/render-schedules.js

@@ -430,7 +430,11 @@ function openScheduleDetail(id) {
 // ─── Form ───────────────────────────────────────────────────────────────────
 
 function _scheduleFormHtml(sched, isEdit) {
-  const types = ['implement', 'test', 'explore', 'ask', 'review', 'fix'];
+  const types = ['implement', 'test', 'explore', 'ask', 'review', 'fix', 'setup'];
+  // W-ms5tb6ha016fda10 — only these types may run without a repository
+  // checkout. Mirrors shared.CONTROL_PLANE_ELIGIBLE_TYPES; the server
+  // re-validates fail-closed, so this is a UX affordance, not the gate.
+  const controlPlaneTypes = ['setup', 'ask', 'explore', 'meeting'];
   const priorities = Array.isArray(window.MINIONS_WORK_ITEM_PRIORITIES)
     ? window.MINIONS_WORK_ITEM_PRIORITIES
     : [];
@@ -510,12 +514,21 @@ function _scheduleFormHtml(sched, isEdit) {
     '</div>' +
     '<div style="display:flex;gap:12px">' +
       '<label style="color:var(--text);font-size:var(--text-md);flex:1">Type' +
-        '<select id="sched-edit-type" style="' + inputStyle + '">' + typeOpts + '</select>' +
+        '<select id="sched-edit-type" style="' + inputStyle + '" onchange="_syncScheduleSurfaceAvailability()">' + typeOpts + '</select>' +
       '</label>' +
       '<label style="color:var(--text);font-size:var(--text-md);flex:1">Priority' +
         '<select id="sched-edit-priority" style="' + inputStyle + '">' + priOpts + '</select>' +
       '</label>' +
     '</div>' +
+    '<label id="sched-surface-row" style="color:var(--text);font-size:var(--text-md)"' +
+      ' title="Control-plane tasks call the Minions API and gh/az only. The engine skips git worktree creation entirely, so they cost no checkout and never race shared remote refs. Only setup / ask / explore / meeting may use it.">' +
+      'Execution surface' +
+      '<select id="sched-edit-surface" style="' + inputStyle + '">' +
+        '<option value="repo"' + (sched.executionSurface === 'control-plane' ? '' : ' selected') + '>repo — allocate a git worktree (default)</option>' +
+        '<option value="control-plane"' + (sched.executionSurface === 'control-plane' ? ' selected' : '') + '>control-plane — API/CLI only, no checkout</option>' +
+      '</select>' +
+      '<span id="sched-surface-hint" style="display:block;margin-top:4px;font-size:var(--text-base);color:var(--muted)"></span>' +
+    '</label>' +
     '<div style="display:flex;gap:12px">' +
       '<label style="color:var(--text);font-size:var(--text-md);flex:1">Project' +
         '<select id="sched-edit-project" style="' + inputStyle + '">' + projOpts + '</select>' +
@@ -534,6 +547,33 @@ function _scheduleFormHtml(sched, isEdit) {
   '</div>';
 }
 
+function _scheduleControlPlaneTypes() {
+  return ['setup', 'ask', 'explore', 'meeting'];
+}
+
+/**
+ * W-ms5tb6ha016fda10 — keep the Execution-surface selector honest as the Type
+ * changes. Repo-reading/mutating types cannot run without a checkout, so the
+ * control-plane option is disabled (and any stale selection reset) rather than
+ * letting the operator submit something the server will reject.
+ */
+function _syncScheduleSurfaceAvailability() {
+  const typeEl = document.getElementById('sched-edit-type');
+  const surfaceEl = document.getElementById('sched-edit-surface');
+  const hintEl = document.getElementById('sched-surface-hint');
+  if (!typeEl || !surfaceEl) return;
+  const eligible = _scheduleControlPlaneTypes().includes(typeEl.value);
+  const cpOption = surfaceEl.querySelector('option[value="control-plane"]');
+  if (cpOption) cpOption.disabled = !eligible;
+  if (!eligible && surfaceEl.value === 'control-plane') surfaceEl.value = 'repo';
+  surfaceEl.disabled = !eligible;
+  if (hintEl) {
+    hintEl.textContent = eligible
+      ? 'Control-plane skips git worktree creation — use it for API/CLI-only maintenance jobs.'
+      : 'Type "' + typeEl.value + '" reads or changes repository files, so it always gets a worktree.';
+  }
+}
+
 function openCreateScheduleModal() {
   window._schedComputedCron = '';
   document.getElementById('modal-title').textContent = 'New Scheduled Task';
@@ -542,6 +582,7 @@ function openCreateScheduleModal() {
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() by _scheduleFormHtml() (fields: project names, agent ids/names)
   document.getElementById('modal-body').innerHTML = _scheduleFormHtml({}, false);
   document.getElementById('modal').classList.add('open');
+  _syncScheduleSurfaceAvailability();
   _updateCronPreview();
 }
 
@@ -556,6 +597,7 @@ function openEditScheduleModal(id) {
   document.getElementById('modal-body').innerHTML = _scheduleFormHtml(sched, true);
   window._editScheduleId = id;
   document.getElementById('modal').classList.add('open');
+  _syncScheduleSurfaceAvailability();
   _updateCronPreview();
 }
 
@@ -569,6 +611,8 @@ async function submitSchedule(isEdit, e) {
   const project = document.getElementById('sched-edit-project').value;
   const agent = document.getElementById('sched-edit-agent').value;
   const description = document.getElementById('sched-edit-desc').value;
+  const surfaceEl = document.getElementById('sched-edit-surface');
+  const executionSurface = surfaceEl && !surfaceEl.disabled ? surfaceEl.value : 'repo';
 
   let id;
   if (isEdit) {
@@ -581,7 +625,7 @@ async function submitSchedule(isEdit, e) {
   if (!title) { _resetSchedBtn(); _showScheduleError('Title is required'); return; }
   if (!cron) { _resetSchedBtn(); _showScheduleError('Schedule is required \u2014 select days and time, or use natural language'); return; }
 
-  const payload = { id, title, cron, type, priority, project: project || undefined, agent: agent || undefined, description: description || undefined, enabled: true };
+  const payload = { id, title, cron, type, priority, project: project || undefined, agent: agent || undefined, description: description || undefined, enabled: true, executionSurface };
   const url = isEdit ? '/api/schedules/update' : '/api/schedules';
   try {
     const res = await fetch(url, {
@@ -689,3 +733,19 @@ async function deleteSchedule(id) {
 window._generateScheduleId = _generateScheduleId;
 
 window.MinionsSchedules = { renderSchedules, openCreateScheduleModal, openEditScheduleModal, openScheduleDetail, submitSchedule, toggleScheduleEnabled, runScheduleNow, deleteSchedule, _cronToHuman, _parseNaturalCron, _toggleCronMode, _quickSelectDays, _toggleDayPill, _updateCronPreview, _schedPrev, _schedNext, _schedSetView };
+
+// ── Slim "+ Add" deep-link (W-ms258yw0000ffd8a) ────────────────────────
+// The Slim Automation tile's "+ Add" picker embeds this classic screen at
+// /schedule?embed=1#create and relies on this hook to auto-open the Create
+// Scheduled Task dialog — the classic create flow is reused, not forked (CLAUDE
+// .md slim-vs-classic). Fires only on the /schedule path with a #create hash;
+// the standalone classic screen and every other embed is otherwise untouched.
+// isReady is load-bearing: _scheduleFormHtml builds its Project/Agent selects
+// from cmdProjects/cmdAgents (see above), which refresh.js only fills after the
+// first data tick — opening earlier yields a form with empty dropdowns that
+// never self-heals. Helper lives in utils.js#bootCreateDeepLink.
+typeof bootCreateDeepLink === 'function' && bootCreateDeepLink({
+  route: '/schedule',
+  resolve: function() { return typeof openCreateScheduleModal === 'function' ? openCreateScheduleModal : null; },
+  isReady: classicCreateFormDataReady,
+});

package/dashboard/js/render-utils.js

@@ -82,7 +82,7 @@ function formatToolSummary(name, input) {
 // record, rendered in sequence so prose and tool activity stay interleaved.
 //
 // Text-segment boundaries come from the server `segmentId` on chunk events
-// (engine/llm.js increments it on a tool boundary or a non-extending text
+// (engine/agents/llm.js increments it on a tool boundary or a non-extending text
 // push). When segmentId is absent (Copilot pool path, live-stream replay,
 // older engine) ccSegmentsApplyChunk falls back to a heuristic: a chunk that
 // arrives after a tool starts a new text segment; a chunk that extends the
@@ -343,7 +343,7 @@ function _renderJsonObj(obj, state) {
     // W-mpob4nyk0006580e — the terminal banner is no longer derived here.
     // The engine's dispatch.result is the single source of truth and is
     // pushed to the live view via the SSE `dispatch.terminal` channel
-    // (engine/dispatch-events.js → dashboard.js handleAgentLiveStream).
+    // (engine/orchestration/dispatch-events.js → dashboard.js handleAgentLiveStream).
     // `result` JSONL lines and the `[process-exit]` sentinel are still
     // parsed for other purposes (e.g. tool deltas above), but they must
     // NOT trigger any banner here — log heuristics can disagree with the
@@ -466,7 +466,7 @@ function renderAgentOutput(text) {
 /**
  * W-mpob4nyk0006580e — render the terminal banner from an authoritative
  * `dispatch.terminal` SSE event. The shape mirrors
- * engine/dispatch-events.js buildTerminalEvent():
+ * engine/orchestration/dispatch-events.js buildTerminalEvent():
  *   { type, ts, dispatchId, agentId, result, reason, failureClass, prUrl, summary }
  *
  * The banner state mirrors `dispatch.result` exactly — green for SUCCESS,

package/dashboard/js/render-watches.js

@@ -93,8 +93,8 @@ function _targetTypeLabel(type) {
 // renderArtifactLink / openArtifact (defined in render-utils.js). Returns null
 // for targetTypes that don't correspond to a navigable artifact (teams-channel,
 // future plugin types) so callers fall back to plain text rendering. The
-// mapping covers the 8 first-class TARGET_TYPES registered in engine/watches.js
-// (engine/shared.js#WATCH_TARGET_TYPE) — dispatch ids are WI ids in practice
+// mapping covers the 8 first-class TARGET_TYPES in engine/watches.js
+// (engine/core/shared.js#WATCH_TARGET_TYPE) — dispatch ids are WI ids in practice
 // so they share the wi chip target.
 var _WATCH_TARGET_TO_ART_TYPE = {
   'pr': 'pr',
@@ -754,7 +754,7 @@ function submitWatch() {
   // P-w14e7a8c — Phase 7.1: harvest action steps from the chain editor.
   // 0 steps -> action: null (notify-only legacy behavior).
   // 1 step  -> action: { type, params, guard? } (single-object form).
-  // 2+ steps -> action: [ {...}, ... ] (array-form chain — engine/watch-actions.js
+  // 2+ steps -> action: [ {...}, ... ] (array-form chain — engine/watches/actions.js
   //             runWatchActionChain handles it).
   var stepRows = document.querySelectorAll('.watch-step-row');
   var steps = [];
@@ -867,3 +867,19 @@ window.MinionsWatches = {
   _watchTargetToArtType: _watchTargetToArtType,
   _renderWatchTargetChip: _renderWatchTargetChip,
 };
+
+// ── Slim "+ Add" deep-link (W-ms258yw0000ffd8a) ────────────────────────
+// The Slim Automation tile's "+ Add" picker embeds this classic screen at
+// /watches?embed=1#create and relies on this hook to auto-open the Create Watch
+// dialog — the classic create flow is reused, not forked (CLAUDE.md slim-vs-
+// classic). Fires only on the /watches path with a #create hash; the standalone
+// classic screen and every other embed is otherwise untouched.
+// isReady is load-bearing: _renderCreateWatchModal builds its Owner/Project
+// selects from cmdAgents/cmdProjects (see above), which refresh.js only fills
+// after the first data tick — opening earlier yields a form with empty dropdowns
+// that never self-heals. Helper lives in utils.js#bootCreateDeepLink.
+typeof bootCreateDeepLink === 'function' && bootCreateDeepLink({
+  route: '/watches',
+  resolve: function() { return typeof openCreateWatchModal === 'function' ? openCreateWatchModal : null; },
+  isReady: classicCreateFormDataReady,
+});

package/dashboard/js/render-work-items.js

@@ -1,9 +1,41 @@
 // dashboard/js/render-work-items.js — Work item rendering and management extracted from dashboard.html
 
 let allWorkItems = [];
+let filteredWorkItems = [];
 let wiPage = 0;
+let wiFilterOptionsSignature = '';
 const WI_PER_PAGE = 20;
 
+// Filter + sort controls modeled on the PR table (dashboard/js/render-prs.js).
+// Each dimension maps to a <select> in dashboard/pages/work.html; options are
+// populated dynamically from the current data set with a signature guard so the
+// user's selection survives the 4s refresh poll. (W-ms7kutzv003q17ad)
+const WI_FILTER_CONTROL_IDS = {
+  status: 'wi-filter-status',
+  type: 'wi-filter-type',
+  project: 'wi-filter-project',
+  agent: 'wi-filter-agent',
+  priority: 'wi-filter-priority',
+};
+const WI_FILTER_ALL_LABELS = {
+  status: 'All statuses',
+  type: 'All types',
+  project: 'All projects',
+  agent: 'All agents',
+  priority: 'All priorities',
+};
+const WI_SORT_LABELS = [
+  { value: 'lifecycle', label: 'Lifecycle (active first)' },
+  { value: 'updated', label: 'Last updated' },
+  { value: 'created', label: 'Created' },
+  { value: 'priority', label: 'Priority' },
+  { value: 'status', label: 'Status' },
+  { value: 'type', label: 'Type' },
+];
+const WI_SORT_CONTROL_ID = 'wi-sort-by';
+const WI_SORT_DIR_CONTROL_ID = 'wi-sort-dir';
+
+
 // W-mq08kuog001110a6 — Needs-Attention badge for stuck pending work items.
 //
 // A pending WI is "needs attention" when the engine is intentionally refusing
@@ -26,6 +58,7 @@ const WI_PER_PAGE = 20;
 //   * branch_locked — wait for the holding dispatch
 //   * worktree_branch_held — configured checkout is re-checked each tick
 //   * dependency_unmet — auto-clears when upstream dep finishes (engine discovery loop + DEPENDENCY_MET watch)
+//   * dependency_conflict — dep-branch merge conflict repair in progress; auto-clears when the conflict-repair WI completes (W-mrzqfozm00035365)
 //   * live_checkout_busy — one live-mode dispatch at a time; clears when running dispatch finishes
 //
 // no_agent (kept as a trigger) now means the engine found NO eligible agent
@@ -73,6 +106,22 @@ function needsAttentionInfo(item) {
     var budgetMsg = 'Assigned agent ' + agent + ' is over its maxBudgetUsd cap.';
     return { kind: 'budget_exceeded', short: budgetMsg, full: budgetMsg };
   }
+  // W-mrzqfozm00035365 — a downstream item whose dependency-branch merge
+  // conflict could NOT be repaired (repair WI failed/cancelled, or completed
+  // but the branches still conflict) is an actionable block: it will not
+  // dispatch until a human resolves the conflict on the dep branch. The
+  // in-progress `dependency_conflict` state is deliberately NOT surfaced here —
+  // it auto-clears when the repair completes.
+  if (reason === 'dependency_conflict_unresolved') {
+    var dc = item._dependencyConflict || {};
+    var cb = dc.conflictBranch || '(dependency branch)';
+    var cw = dc.conflictsWith || 'the base branch';
+    var rw = dc.repairWiId || '(conflict-fix work item)';
+    var dcMsg = 'Dependency branch ' + cb + ' conflicts with ' + cw +
+      ' and the conflict-repair work item ' + rw + ' did not resolve it. ' +
+      'Resolve the conflict on ' + cb + ' (or re-run ' + rw + '); this item re-runs dependency integration automatically once the repair succeeds.';
+    return { kind: 'dependency_conflict_unresolved', short: dcMsg, full: dcMsg };
+  }
   return null;
 }
 
@@ -80,6 +129,20 @@ function formatPendingReason(item, detailed) {
   var reason = item && item._pendingReason ? String(item._pendingReason) : '';
   if (!reason) return '';
   if (reason === 'already_dispatched') return 'Queued — waiting for available agent slot';
+  if (reason === 'dependency_conflict') {
+    if (!detailed) return 'dependency conflict — repair in progress';
+    var dcp = item._dependencyConflict || {};
+    return 'Dependency branch ' + (dcp.conflictBranch || '(unknown)') + ' conflicts with ' +
+      (dcp.conflictsWith || 'the base branch') + '. Held pending behind conflict-repair ' +
+      (dcp.repairWiId || '(repair WI)') + '; auto-resumes when the repair completes.';
+  }
+  if (reason === 'dependency_conflict_unresolved') {
+    if (!detailed) return 'dependency conflict — needs attention';
+    var dcu = item._dependencyConflict || {};
+    return 'Dependency branch ' + (dcu.conflictBranch || '(unknown)') + ' still conflicts with ' +
+      (dcu.conflictsWith || 'the base branch') + ' after conflict-repair ' +
+      (dcu.repairWiId || '(repair WI)') + '. Needs manual resolution.';
+  }
   if (reason === 'agents_busy') return 'Waiting for an available agent — all eligible agents are busy';
   if (reason !== 'worktree_branch_held') return reason.replace(/_/g, ' ');
   if (!detailed) return 'operator checkout holds branch';
@@ -187,6 +250,7 @@ function wiRow(item) {
     '</td>' +
     '<td>' + prLink + (failSnippet ? failSnippet : '') + '</td>' +
     '<td><span class="pr-date">' + escapeHtml((item.created || '').slice(0, 16).replace('T', ' ')) + '</span></td>' +
+    '<td><span class="pr-date" title="' + escapeHtml(_wiUpdatedTitle(item)) + '">' + escapeHtml(_wiUpdatedLabel(item)) + '</span></td>' +
     '<td style="white-space:nowrap;font-size:var(--text-sm);color:var(--muted)">' +
       (function() {
         // /api/status slims the full arrays into integer counts
@@ -217,44 +281,188 @@ function wiRow(item) {
   '</tr>';
 }
 
+function _wiUpdatedLabel(item) {
+  var ms = window.MinionsWorkItemFilters && window.MinionsWorkItemFilters.lastUpdatedMs
+    ? window.MinionsWorkItemFilters.lastUpdatedMs(item)
+    : null;
+  if (ms == null) return '—';
+  return typeof timeAgo === 'function' ? timeAgo(new Date(ms).toISOString()) : new Date(ms).toISOString().slice(0, 16).replace('T', ' ');
+}
+
+function _wiUpdatedTitle(item) {
+  var ms = window.MinionsWorkItemFilters && window.MinionsWorkItemFilters.lastUpdatedMs
+    ? window.MinionsWorkItemFilters.lastUpdatedMs(item)
+    : null;
+  if (ms == null) return 'No update timestamp';
+  return new Date(ms).toISOString().slice(0, 16).replace('T', ' ') + ' UTC';
+}
+
+// Populate the filter <select>s from the current data set. Guarded by a
+// signature so options only re-render when the derivable option set changes —
+// this preserves the user's current selection across the periodic poll. Mirrors
+// _syncPrFilterOptions in render-prs.js. (W-ms7kutzv003q17ad)
+function _syncWiFilterOptions(items) {
+  if (!window.MinionsWorkItemFilters) return;
+  const options = window.MinionsWorkItemFilters.options(items);
+  const controls = Object.keys(WI_FILTER_CONTROL_IDS).map(function(dimension) {
+    return {
+      dimension: dimension,
+      select: document.getElementById(WI_FILTER_CONTROL_IDS[dimension]),
+    };
+  }).filter(function(entry) { return entry.select; });
+  if (!controls.length) return;
+  const signature = JSON.stringify(options);
+  if (signature === wiFilterOptionsSignature) return;
+  wiFilterOptionsSignature = 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 = WI_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 _readWiFilters() {
+  const filters = {};
+  Object.keys(WI_FILTER_CONTROL_IDS).forEach(function(dimension) {
+    const select = document.getElementById(WI_FILTER_CONTROL_IDS[dimension]);
+    filters[dimension] = select ? select.value : '';
+  });
+  return filters;
+}
+
+function _hasActiveWiFilters(filters) {
+  return Object.keys(WI_FILTER_CONTROL_IDS).some(function(dimension) {
+    return Boolean(filters[dimension]);
+  });
+}
+
+// Single source for the sort default in this module: the shared helper's
+// WI_SORT_DEFAULT (dashboard/shared/wi-filters.js), with one literal fallback
+// for the no-module case. Kept in sync with the shared default by
+// test/unit/wi-filters.test.js.
+function _wiSortDefault() {
+  return (window.MinionsWorkItemFilters && window.MinionsWorkItemFilters.sortDefault)
+    || { sortBy: 'lifecycle', dir: 'desc' };
+}
+
+function _readWiSort() {
+  const byEl = document.getElementById(WI_SORT_CONTROL_ID);
+  const dirEl = document.getElementById(WI_SORT_DIR_CONTROL_ID);
+  const def = _wiSortDefault();
+  return {
+    sortBy: byEl && byEl.value ? byEl.value : def.sortBy,
+    dir: dirEl && dirEl.value ? dirEl.value : def.dir,
+  };
+}
+
+function applyWiFilters() {
+  wiPage = 0;
+  renderWorkItems(allWorkItems, { preserveScroll: false });
+}
+
+function resetWiFilters() {
+  Object.keys(WI_FILTER_CONTROL_IDS).forEach(function(dimension) {
+    const select = document.getElementById(WI_FILTER_CONTROL_IDS[dimension]);
+    if (select) select.value = '';
+  });
+  const byEl = document.getElementById(WI_SORT_CONTROL_ID);
+  const dirEl = document.getElementById(WI_SORT_DIR_CONTROL_ID);
+  const def = _wiSortDefault();
+  if (byEl) byEl.value = def.sortBy;
+  if (dirEl) {
+    dirEl.value = def.dir;
+    _syncWiSortDirLabel(dirEl);
+  }
+  applyWiFilters();
+}
+
+// Toggle the asc/desc direction button (value + arrow glyph) and re-render.
+function toggleWiSortDir() {
+  const dirEl = document.getElementById(WI_SORT_DIR_CONTROL_ID);
+  if (dirEl) {
+    dirEl.value = dirEl.value === 'asc' ? 'desc' : 'asc';
+    _syncWiSortDirLabel(dirEl);
+  }
+  applyWiFilters();
+}
+
+function _syncWiSortDirLabel(dirEl) {
+  if (!dirEl) return;
+  const asc = dirEl.value === 'asc';
+  dirEl.setAttribute('aria-label', asc ? 'Sort ascending (oldest first)' : 'Sort descending (newest first)');
+  dirEl.title = asc ? 'Ascending — click for descending' : 'Descending — click for ascending';
+  dirEl.textContent = asc ? '\u2191 Asc' : '\u2193 Desc';
+}
+
+// Apply the active filters + sort to the raw items. dashboard-build.js always
+// assembles dashboard/shared/wi-filters.js before this module (enforced by
+// test/unit/wi-filters.test.js), so the shared helper is the ONE definition of
+// filter and lifecycle-sort semantics. A stripped harness without it renders
+// the raw list rather than carrying a second, drift-prone status order.
+function _computeWiView(items) {
+  const filters = _readWiFilters();
+  const sort = _readWiSort();
+  const wf = window.MinionsWorkItemFilters;
+  if (!wf) return { view: items.slice(), filters: filters };
+  return { view: wf.sort(wf.filter(items, filters), sort), filters: filters };
+}
+
 function renderWorkItems(items, opts) {
   opts = opts || {};
   items = items.filter(function(w) { return !isDeleted('wi:' + w.id); });
-  // Sort: active/dispatched first, then by most recent activity
-  const statusOrder = { dispatched: 0, pending: 1, queued: 1, failed: 2, done: 3 };
-  items.sort((a, b) => {
-    const sa = statusOrder[a.status] ?? 2, sb = statusOrder[b.status] ?? 2;
-    if (sa !== sb) return sa - sb;
-    const ta = a.completedAt || a.dispatched_at || a.created || '';
-    const tb = b.completedAt || b.dispatched_at || b.created || '';
-    return tb.localeCompare(ta); // most recent first
-  });
   allWorkItems = items;
+  _syncWiFilterOptions(allWorkItems);
+  const computed = _computeWiView(allWorkItems);
+  const view = computed.view;
+  filteredWorkItems = view;
   const el = document.getElementById('work-items-content');
   const countEl = document.getElementById('wi-count');
-  if (countEl) countEl.textContent = items.length;
-  if (!items.length) {
+  const summaryEl = document.getElementById('wi-filter-summary');
+  if (countEl) countEl.textContent = view.length;
+  if (summaryEl) {
+    summaryEl.textContent = _hasActiveWiFilters(computed.filters)
+      ? 'Showing ' + view.length + ' of ' + allWorkItems.length
+      : '';
+  }
+  if (!allWorkItems.length) {
+    wiPage = 0;
     el.innerHTML = '<p class="empty">No work items. Add tasks via Command Center above.</p>';
     return;
   }
+  if (!view.length) {
+    wiPage = 0;
+    el.innerHTML = '<p class="empty">No work items match the selected filters.</p>';
+    return;
+  }
 
-  const totalPages = Math.ceil(items.length / WI_PER_PAGE);
+  const totalPages = Math.ceil(view.length / WI_PER_PAGE);
   const previousPage = wiPage;
   if (wiPage >= totalPages) wiPage = totalPages - 1;
   const start = wiPage * WI_PER_PAGE;
-  const pageItems = items.slice(start, start + WI_PER_PAGE);
+  const pageItems = view.slice(start, start + WI_PER_PAGE);
 
-  let html = '<div class="pr-table-wrap"><table class="pr-table"><thead><tr><th>Title</th><th>Project</th><th>Type</th><th>Priority</th><th>Status</th><th>Agent</th><th>PR</th><th>Created</th><th></th><th></th></tr></thead><tbody>';
+  let html = '<div class="pr-table-wrap"><table class="pr-table"><thead><tr><th>Title</th><th>Project</th><th>Type</th><th>Priority</th><th>Status</th><th>Agent</th><th>PR</th><th>Created</th><th>Updated</th><th></th><th></th></tr></thead><tbody>';
   html += pageItems.map(wiRow).join('');
   html += '</tbody></table></div>';
 
-  if (items.length > WI_PER_PAGE) {
+  if (view.length > WI_PER_PAGE) {
     html += '<div class="pr-pager">' +
-      '<span class="pr-page-info">Showing ' + (start+1) + ' to ' + Math.min(start+WI_PER_PAGE, items.length) + ' of ' + items.length + '</span>' +
+      '<span class="pr-page-info">Showing ' + (start+1) + ' to ' + Math.min(start+WI_PER_PAGE, view.length) + ' of ' + view.length + '</span>' +
       '<div class="pr-pager-btns">' +
         '<button class="pr-pager-btn ' + (wiPage === 0 ? 'disabled' : '') + '" onclick="wiPrev()">Prev</button>' +
         '<button class="pr-pager-btn ' + (wiPage >= totalPages-1 ? 'disabled' : '') + '" onclick="wiNext()">Next</button>' +
-        '<button class="pr-pager-btn see-all" onclick="openAllWorkItems()">See all ' + items.length + '</button>' +
+        '<button class="pr-pager-btn see-all" onclick="openAllWorkItems()">See all ' + view.length + '</button>' +
       '</div>' +
     '</div>';
   }
@@ -517,7 +725,7 @@ async function retryWorkItem(id, source) {
 }
 
 function wiPrev() { if (wiPage > 0) { wiPage--; renderWorkItems(allWorkItems, { preserveScroll: false }); } }
-function wiNext() { const tp = Math.ceil(allWorkItems.length / WI_PER_PAGE); if (wiPage < tp-1) { wiPage++; renderWorkItems(allWorkItems, { preserveScroll: false }); } }
+function wiNext() { const tp = Math.ceil(filteredWorkItems.length / WI_PER_PAGE); if (wiPage < tp-1) { wiPage++; renderWorkItems(allWorkItems, { preserveScroll: false }); } }
 
 let _feedbackRating = null;
 function feedbackWorkItem(id, source) {
@@ -676,7 +884,7 @@ async function _submitCreateWorkItem(e) {
 }
 
 // P-79b47b0c — Derive a canonical PR id (`<host>:<slug>#<number>`) from a
-// GitHub or ADO PR URL. Matches the format produced by engine/queries.js
+// GitHub or ADO PR URL. Matches the format produced by engine/core/queries.js
 // when stamping pr.id. Returns null when the URL doesn't match a known
 // host pattern.
 function _wiDeriveCanonicalPrIdFromUrl(url) {
@@ -691,7 +899,7 @@ function _wiDeriveCanonicalPrIdFromUrl(url) {
 // W-ms0ytfdd000d5a26 — provenance hint for the WI detail modal's Model field.
 // Explains WHERE the shown model came from so an operator can tell a recorded
 // fact from a projection. Mirrors the `source` values produced by
-// engine/execution-model.js#resolveWorkItemModel.
+// engine/core/execution-model.js#resolveWorkItemModel.
 function _wiModelSourceHint(m) {
   var runtime = m.runtime || 'runtime';
   var hint;
@@ -766,7 +974,7 @@ function _wiRenderDetail(item) {
   html += field('Description', '<div id="wi-detail-desc" style="font-size:var(--text-md);max-height:320px;overflow-y:auto;padding:8px 10px;background:var(--surface2);border:1px solid var(--border);border-radius:var(--radius-sm)">' + _descHtml + '</div>');
   html += field('Agent', escapeHtml(item.dispatched_to || item.agent || 'Auto'));
   // W-ms0ytfdd000d5a26 — the model this work item actually ran on. Resolved
-  // server-side (engine/queries.js#resolveWorkItemModelInfo) so persisted
+  // server-side (engine/core/queries.js#resolveWorkItemModelInfo) so persisted
   // dispatch evidence — what the runtime reported, else what Minions requested
   // — always beats current config. Items with no dispatch record show the
   // projection from current config, marked as such. When nothing concrete is
@@ -795,8 +1003,8 @@ function _wiRenderDetail(item) {
   if (item.failReason) html += field('Failure Reason', '<span style="color:var(--red)">' + escapeHtml(item.failReason) + '</span>');
   // W-mrtdet41000h935a — surface the cancellation reason for CANCELLED items.
   // Cancellers stamp the reason on the record: POST /api/work-items/cancel and
-  // engine/dispatch.js/watch-actions.js write `_cancelledBy`; the goal-invalidation
-  // path (engine/lifecycle.js) writes `cancellationReason`. Prefer _cancelledBy,
+  // engine/orchestration/dispatch.js and engine/watches/actions.js write `_cancelledBy`; the goal-invalidation
+  // path (engine/orchestration/lifecycle.js) writes `cancellationReason`. Prefer _cancelledBy,
   // fall back to cancellationReason, then a no-reason marker. cancelledAt (ISO) is
   // rendered like the other timestamps when present. Reason text can carry
   // untrusted PR/branch content, so it is escaped.
@@ -885,7 +1093,7 @@ function _wiRenderDetail(item) {
   // P-c549d07e — "Spawned by" back-link chip. The schedule/pipeline that
   // dispatched this WI is the natural Back target from the WI modal. The
   // primary source is meta.spawnedBy ('schedule:<id>' | 'pipeline:<id>'),
-  // stamped by engine/scheduler.js + engine/pipeline.js on new dispatches.
+  // stamped by engine/orchestration/scheduler.js + engine/orchestration/pipeline.js on new dispatches.
   // For older WIs already in state we fall back to (a) item._scheduleId
   // and (b) item.createdBy when it starts with 'pipeline:' so historical
   // items still light up the chip — purely additive, no behavior change.
@@ -925,7 +1133,7 @@ function _wiRenderDetail(item) {
   // named e.g. "src"). The genuine source-plan/PRD case is already fully
   // covered by arts.plan / arts.prd / arts.sourcePlan below, which are
   // populated server-side from the WI's real sourcePlan/_planFileName/
-  // planFile fields (engine/queries.js) — no separate item._source-based
+  // planFile fields (engine/core/queries.js) — no separate item._source-based
   // chip is needed.
   var arts = item._artifacts || {};
   var artPills = '';
@@ -963,7 +1171,7 @@ function _wiRenderDetail(item) {
 
   // P-34fa5d79 — Mentions: notes whose YAML frontmatter cites this WI
   // (work_item: / wi: / sourceItem:). Slim string[] field set by
-  // engine/queries.js#getWorkItems from _buildNotesByWiMap. Each entry is
+  // engine/core/queries.js#getWorkItems from _buildNotesByWiMap. Each entry is
   // either a bare inbox filename or `archive:<filename>` for archive notes.
   //
   // #308 — repeated attempts for one WI used to render a flat pile of
@@ -1077,9 +1285,9 @@ function openWorkItemDetail(id, source) {
 }
 
 function openAllWorkItems() {
-  document.getElementById('modal-title').textContent = 'All Work Items (' + allWorkItems.length + ')';
-  const html = '<div class="pr-table-wrap"><table class="pr-table"><thead><tr><th>Title</th><th>Project</th><th>Type</th><th>Priority</th><th>Status</th><th>Agent</th><th>PR</th><th>Created</th><th></th><th></th></tr></thead><tbody>' +
-    allWorkItems.map(wiRow).join('') + '</tbody></table></div>';
+  document.getElementById('modal-title').textContent = 'All Work Items (' + filteredWorkItems.length + ')';
+  const html = '<div class="pr-table-wrap"><table class="pr-table"><thead><tr><th>Title</th><th>Project</th><th>Type</th><th>Priority</th><th>Status</th><th>Agent</th><th>PR</th><th>Created</th><th>Updated</th><th></th><th></th></tr></thead><tbody>' +
+    filteredWorkItems.map(wiRow).join('') + '</tbody></table></div>';
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escapeHtml() by wiRow() (fields: work item id/title/source/status/agent/PR links/dates/follow-up metadata)
   document.getElementById('modal-body').innerHTML = html;
   document.getElementById('modal-body').style.fontFamily = "'Segoe UI', system-ui, sans-serif";
@@ -1206,7 +1414,7 @@ function _wiInjectSourceChipFromFrontmatter(content, containerId) {
 }
 
 if (typeof window !== 'undefined') {
-  window.MinionsWork = { wiRow, renderWorkItems, editWorkItem, submitWorkItemEdit, deleteWorkItem, archiveWorkItem, toggleWorkItemArchive, retryWorkItem, wiPrev, wiNext, feedbackWorkItem, openCreateWorkItemModal, openWorkItemDetail, openAllWorkItems, viewAgentOutput, openInboxNote, needsAttentionInfo };
+  window.MinionsWork = { wiRow, renderWorkItems, editWorkItem, submitWorkItemEdit, deleteWorkItem, archiveWorkItem, toggleWorkItemArchive, retryWorkItem, wiPrev, wiNext, feedbackWorkItem, openCreateWorkItemModal, openWorkItemDetail, openAllWorkItems, viewAgentOutput, openInboxNote, needsAttentionInfo, applyWiFilters, resetWiFilters, toggleWiSortDir };
 }
 // exported for testing — pure helpers safe to require under Node (no DOM deps).
 if (typeof module !== 'undefined' && module.exports) {