@yemi33/minions 0.1.2396 → 0.1.2398

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-prd.js

@@ -809,6 +809,13 @@ async function prdItemEdit(source, itemId) {
   if (!item) return;
 
   _prdDescRawCache = item.description || '';
+  const priorities = Array.isArray(window.MINIONS_WORK_ITEM_PRIORITIES)
+    ? window.MINIONS_WORK_ITEM_PRIORITIES
+    : [];
+  const priorityOptions = priorities.map(function(priority) {
+    const label = priority.charAt(0).toUpperCase() + priority.slice(1);
+    return '<option value="' + priority + '"' + (item.priority === priority ? ' selected' : '') + '>' + label + '</option>';
+  }).join('');
 
   // Look up work item and dispatch completion info. Issue #2949 —
   // dispatch moved off /api/status to /api/dispatch; refresh.js stashes
@@ -888,9 +895,7 @@ async function prdItemEdit(source, itemId) {
     '<div style="display:flex;gap:12px;margin-bottom:12px">' +
       '<div><label style="font-size:var(--text-base);color:var(--muted);display:block;margin-bottom:4px">Priority</label>' +
         '<select id="prd-edit-priority" style="padding:4px 8px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text)">' +
-          '<option value="high"' + (item.priority === 'high' ? ' selected' : '') + '>High</option>' +
-          '<option value="medium"' + (item.priority === 'medium' ? ' selected' : '') + '>Medium</option>' +
-          '<option value="low"' + (item.priority === 'low' ? ' selected' : '') + '>Low</option>' +
+          priorityOptions +
         '</select></div>' +
       '<div><label style="font-size:var(--text-base);color:var(--muted);display:block;margin-bottom:4px">Complexity</label>' +
         '<select id="prd-edit-complexity" style="padding:4px 8px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text)">' +

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/js/render-schedules.js

@@ -431,7 +431,9 @@ function openScheduleDetail(id) {
 
 function _scheduleFormHtml(sched, isEdit) {
   const types = ['implement', 'test', 'explore', 'ask', 'review', 'fix'];
-  const priorities = ['high', 'medium', 'low'];
+  const priorities = Array.isArray(window.MINIONS_WORK_ITEM_PRIORITIES)
+    ? window.MINIONS_WORK_ITEM_PRIORITIES
+    : [];
   const typeOpts = types.map(t => '<option value="' + t + '"' + ((sched.type || 'implement') === t ? ' selected' : '') + '>' + t + '</option>').join('');
   const priOpts = priorities.map(p => '<option value="' + p + '"' + ((sched.priority || 'medium') === p ? ' selected' : '') + '>' + p + '</option>').join('');
   const projOpts = '<option value="">Any</option>' + (cmdProjects || []).map(p => '<option value="' + escHtml(p.name) + '"' + (sched.project === p.name ? ' selected' : '') + '>' + escHtml(p.displayName || p.name) + '</option>').join('');

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

@@ -353,7 +353,9 @@ async function editWorkItem(id, source) {
     }
   }
   const types = ['implement', 'fix', 'review', 'plan', 'verify', 'decompose', 'meeting', 'investigate', 'refactor', 'test', 'explore', 'ask', 'docs', 'setup'];
-  const priorities = ['critical', 'high', 'medium', 'low'];
+  const priorities = Array.isArray(window.MINIONS_WORK_ITEM_PRIORITIES)
+    ? window.MINIONS_WORK_ITEM_PRIORITIES
+    : [];
   const agentOpts = (cmdAgents || []).map(a => '<option value="' + escapeHtml(a.id) + '"' + (item.agent === a.id ? ' selected' : '') + '>' + escapeHtml(a.name) + '</option>').join('');
   const typeOpts = types.map(t => '<option value="' + t + '"' + ((item.type || 'implement') === t ? ' selected' : '') + '>' + t + '</option>').join('');
   const priOpts = priorities.map(p => '<option value="' + p + '"' + ((item.priority || 'medium') === p ? ' selected' : '') + '>' + p + '</option>').join('');
@@ -586,7 +588,10 @@ function openCreateWorkItemModal() {
   const typeOpts = ['implement', 'fix', 'explore', 'test', 'review', 'ask', 'plan', 'verify', 'decompose', 'meeting', 'docs', 'setup'].map(t =>
     '<option value="' + t + '"' + (t === 'implement' ? ' selected' : '') + '>' + t + '</option>'
   ).join('');
-  const priOpts = ['critical', 'high', 'medium', 'low'].map(p =>
+  const priorities = Array.isArray(window.MINIONS_WORK_ITEM_PRIORITIES)
+    ? window.MINIONS_WORK_ITEM_PRIORITIES
+    : [];
+  const priOpts = priorities.map(p =>
     '<option value="' + p + '"' + (p === 'medium' ? ' selected' : '') + '>' + p + '</option>'
   ).join('');
   const agentOpts = (typeof cmdAgents !== 'undefined' ? cmdAgents : []).map(a =>

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

@@ -106,6 +106,8 @@ let PORT = REQUESTED_PORT;
 const CONFIG_PATH = path.join(MINIONS_DIR, 'config.json');
 const PINNED_PATH = path.join(MINIONS_DIR, 'pinned.md');
 const PINNED_DEFAULT_CONTENT = '# Pinned Context\n\nCritical notes visible to all agents.';
+const WORK_ITEM_PRIORITY_PARAM_HINT = `priority? (${shared.WORK_ITEM_PRIORITY_VALUES.join('|')})`;
+const PIPELINE_STAGES_PARAM_HINT = `stages[] (task/item priority: ${shared.WORK_ITEM_PRIORITY_VALUES.join('|')})`;
 const KB_PINS_PATH = shared.PINNED_ITEMS_PATH;
 const DASHBOARD_BROWSER_PRESENCE_PATH = path.join(ENGINE_DIR, 'dashboard-browser.json');
 const DASHBOARD_BROWSER_PRESENCE_MAX_AGE_MS = 45000;
@@ -1187,6 +1189,8 @@ function findWorkItemsTargetById(id, source, projects = PROJECTS) {
 
 function buildPlanWorkItem(body, projects = PROJECTS, options = {}) {
   if (!body?.title || !String(body.title).trim()) return { error: 'title is required' };
+  const priorityError = shared.validateWorkItemPriority(body.priority);
+  if (priorityError) return { error: priorityError.error, priorityError };
 
   // Multi-project: body.project may be an array. Each entry must resolve to a
   // known project; ≥2 valid entries trigger cross-repo mode (WI lands in central,
@@ -1225,7 +1229,7 @@ function buildPlanWorkItem(body, projects = PROJECTS, options = {}) {
     id: options.id || ('W-' + shared.uid()),
     title: body.title,
     type: 'plan',
-    priority: body.priority || 'high',
+    priority: body.priority ?? shared.WORK_ITEM_PRIORITY.HIGH,
     description,
     status: WI_STATUS.PENDING,
     created: now.toISOString(),
@@ -1240,6 +1244,8 @@ function buildPlanWorkItem(body, projects = PROJECTS, options = {}) {
 
 function buildManualPrdItemPlan(body, projects = PROJECTS, options = {}) {
   if (!body?.name || !String(body.name).trim()) return { error: 'name is required' };
+  const priorityError = shared.validateWorkItemPriority(body.priority);
+  if (priorityError) return { error: priorityError.error, priorityError };
   const target = resolveWorkItemsCreateTarget(body.project, projects);
   if (target.error) return { error: target.error };
 
@@ -1258,7 +1264,7 @@ function buildManualPrdItemPlan(body, projects = PROJECTS, options = {}) {
     id: body.id || ('M' + String(nowTime).slice(-4)),
     name: String(body.name).trim(),
     description: body.description || '',
-    priority: body.priority || 'medium',
+    priority: body.priority ?? shared.WORK_ITEM_PRIORITY.MEDIUM,
     estimated_complexity: body.estimated_complexity || 'medium',
     status: 'missing',
     depends_on: [],
@@ -1325,6 +1331,24 @@ function validatePipelineProjects(pipeline, projects = PROJECTS) {
   return null;
 }
 
+function validatePipelineWorkItemPriorities(pipeline) {
+  const visit = (value) => {
+    if (!value || typeof value !== 'object') return null;
+    if (Object.prototype.hasOwnProperty.call(value, 'priority')) {
+      const error = shared.validateWorkItemPriority(value.priority);
+      if (error) return error;
+    }
+    for (const key of ['stages', 'items']) {
+      for (const child of Array.isArray(value[key]) ? value[key] : []) {
+        const error = visit(child);
+        if (error) return error;
+      }
+    }
+    return null;
+  };
+  return visit(pipeline);
+}
+
 /**
  * Aggregate archived work items from central and project SQL scopes.
  *
@@ -1796,7 +1820,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`;
   }
@@ -1817,11 +1841,12 @@ function buildDashboardHtml() {
   } catch { /* registry empty or config unreadable — ship empty boot state */ }
 
   const featuresBootstrap = `window.MINIONS_FEATURES = ${JSON.stringify(featuresBoot)};\n`;
+  const workItemPrioritiesBootstrap = `window.MINIONS_WORK_ITEM_PRIORITIES = ${JSON.stringify(shared.WORK_ITEM_PRIORITY_VALUES)};\n`;
 
   let assembled = layout
     .replace('/* __CSS__ */', () => css)
     .replace('<!-- __PAGES__ -->', () => pageHtml)
-    .replace('/* __JS__ */', () => `window.__MINIONS_HOME = ${JSON.stringify(os.homedir())};\n${featuresBootstrap}${jsHtml}`)
+    .replace('/* __JS__ */', () => `window.__MINIONS_HOME = ${JSON.stringify(os.homedir())};\n${featuresBootstrap}${workItemPrioritiesBootstrap}${jsHtml}`)
     .replace(/\{\{favicon_emoji\}\}/g, FAVICON_EMOJI)
     .replace(/\{\{title_suffix\}\}/g, TITLE_SUFFIX);
 
@@ -6629,6 +6654,8 @@ const server = http.createServer(async (req, res) => {
     try {
       const body = await readBody(req);
       if (!body.title || !body.title.trim()) return jsonReply(res, 400, { error: 'title is required' });
+      const priorityError = shared.validateWorkItemPriority(body.priority);
+      if (priorityError) return jsonReply(res, 400, priorityError);
       if (body.depends_on !== undefined) {
         if (!Array.isArray(body.depends_on)) return jsonReply(res, 400, { error: 'depends_on must be an array of strings' });
         if (!body.depends_on.every(s => typeof s === 'string')) return jsonReply(res, 400, { error: 'depends_on entries must be strings' });
@@ -6686,7 +6713,7 @@ const server = http.createServer(async (req, res) => {
       const id = 'W-' + shared.uid();
       const item = {
         id, title: body.title.trim(), type: normalizedType,
-        priority: body.priority || 'medium', description: body.description || '',
+        priority: body.priority ?? shared.WORK_ITEM_PRIORITY.MEDIUM, description: body.description || '',
         status: WI_STATUS.PENDING, created: new Date().toISOString(), createdBy: 'dashboard',
       };
       if (Array.isArray(body.depends_on)) {
@@ -6835,6 +6862,8 @@ const server = http.createServer(async (req, res) => {
       const body = await readBody(req);
       const { id, source, title, description, type, priority, agent } = body;
       if (!id) return jsonReply(res, 400, { error: 'id required' });
+      const priorityError = shared.validateWorkItemPriority(priority);
+      if (priorityError) return jsonReply(res, 400, priorityError);
       if (body.depends_on !== undefined) {
         if (!Array.isArray(body.depends_on)) return jsonReply(res, 400, { error: 'depends_on must be an array of strings' });
         if (!body.depends_on.every(s => typeof s === 'string')) return jsonReply(res, 400, { error: 'depends_on entries must be strings' });
@@ -6983,7 +7012,7 @@ const server = http.createServer(async (req, res) => {
       const body = await readBody(req);
       if (!body.title || !body.title.trim()) return jsonReply(res, 400, { error: 'title is required' });
       const planWorkItem = buildPlanWorkItem(body);
-      if (planWorkItem.error) return jsonReply(res, 400, { error: planWorkItem.error });
+      if (planWorkItem.error) return jsonReply(res, 400, planWorkItem.priorityError || { error: planWorkItem.error });
       // Write as a work item with type 'plan' — user must explicitly execute plan-to-prd after reviewing
       mutateWorkItems('central', items => { items.push(planWorkItem.item); });
       recordCcTurnIfPresent(req, {
@@ -6998,7 +7027,7 @@ const server = http.createServer(async (req, res) => {
       const body = await readBody(req);
       if (!body.name || !body.name.trim()) return jsonReply(res, 400, { error: 'name is required' });
       const manualPrd = buildManualPrdItemPlan(body);
-      if (manualPrd.error) return jsonReply(res, 400, { error: manualPrd.error });
+      if (manualPrd.error) return jsonReply(res, 400, manualPrd.priorityError || { error: manualPrd.error });
 
       const planFile = 'manual-' + shared.uid() + '.json';
       prdStore.writePrd(planFile, manualPrd.plan);
@@ -7011,6 +7040,8 @@ const server = http.createServer(async (req, res) => {
     try {
       const body = await readBody(req);
       if (!body.source || !body.itemId) return jsonReply(res, 400, { error: 'source and itemId required' });
+      const priorityError = shared.validateWorkItemPriority(body.priority);
+      if (priorityError) return jsonReply(res, 400, priorityError);
       if (!body.source.endsWith('.json')) return jsonReply(res, 400, { error: 'expected a PRD JSON filename in `source` (got `' + body.source + '`). Pass prd/<plan>.json, not plans/<plan>.md.' });
       const preCheck = prdStore.readPrd(body.source);
       if (!preCheck) return jsonReply(res, 404, { error: 'plan not found' });
@@ -10671,6 +10702,8 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     const body = await readBody(req);
     let { id, cron, title, type, project, agent, description, priority, enabled } = body;
     if (!cron || !title) return jsonReply(res, 400, { error: 'cron and title are required' });
+    const priorityError = shared.validateWorkItemPriority(priority);
+    if (priorityError) return jsonReply(res, 400, priorityError);
     reloadConfig();
     const projectTarget = resolveScheduleProjectValue(project, PROJECTS);
     if (projectTarget.error) return jsonReply(res, 400, { error: projectTarget.error });
@@ -10709,6 +10742,8 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     const body = await readBody(req);
     let { id, cron, title, type, project, agent, description, priority, enabled } = body;
     if (!id) return jsonReply(res, 400, { error: 'id required' });
+    const priorityError = shared.validateWorkItemPriority(priority);
+    if (priorityError) return jsonReply(res, 400, priorityError);
     reloadConfig();
     if (project !== undefined) {
       const projectTarget = resolveScheduleProjectValue(project, PROJECTS);
@@ -10887,6 +10922,8 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     const body = await readBody(req);
     const { target, targetType, condition, interval, owner, description, project, notify, stopAfter, onNotMet, action, requires } = body;
     if (!target) return jsonReply(res, 400, { error: 'target is required' });
+    const priorityError = watchesMod.validateActionWorkItemPriority(action);
+    if (priorityError) return jsonReply(res, 400, priorityError);
     try {
       const watch = watchesMod.createWatch({ target, targetType, condition, interval, owner, description, project, notify, stopAfter, onNotMet, action, requires });
       invalidateStatusCache();
@@ -10903,6 +10940,8 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     const body = await readBody(req);
     const { id, ...updates } = body;
     if (!id) return jsonReply(res, 400, { error: 'id is required' });
+    const priorityError = watchesMod.validateActionWorkItemPriority(updates.action);
+    if (priorityError) return jsonReply(res, 400, priorityError);
     try {
       const watch = watchesMod.updateWatch(id, updates);
       if (!watch) return jsonReply(res, 404, { error: 'Watch not found' });
@@ -13391,8 +13430,8 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     { method: 'POST', path: '/api/qa/runners/reload', desc: 'Clear the in-process runner registry, re-register built-ins, and re-scan qa-runners.d/ for plugin edits.', handler: handleQaRunnersReload },
 
     // Work items
-    { method: 'POST', path: '/api/work-items', desc: 'Create a new work item', params: 'title, type?, description?, priority?, project?, agent?, agents?, scope?, references?, acceptanceCriteria?, skipPr?, oneShot?, meta?, meta.pr_followup?, meta.workdir?, X-Minions-Agent?, X-Minions-Origin-Wi?', handler: handleWorkItemsCreate },
-    { method: 'POST', path: '/api/work-items/update', desc: 'Edit a pending/failed work item', params: 'id, source?, title?, description?, type?, priority?, agent?, project?, references?, acceptanceCriteria?, depends_on?, workdir?, meta?', handler: handleWorkItemsUpdate },
+    { method: 'POST', path: '/api/work-items', desc: 'Create a new work item', params: `title, type?, description?, ${WORK_ITEM_PRIORITY_PARAM_HINT}, project?, agent?, agents?, scope?, references?, acceptanceCriteria?, skipPr?, oneShot?, meta?, meta.pr_followup?, meta.workdir?, X-Minions-Agent?, X-Minions-Origin-Wi?`, handler: handleWorkItemsCreate },
+    { method: 'POST', path: '/api/work-items/update', desc: 'Edit a pending/failed work item', params: `id, source?, title?, description?, type?, ${WORK_ITEM_PRIORITY_PARAM_HINT}, agent?, project?, references?, acceptanceCriteria?, depends_on?, workdir?, meta?`, handler: handleWorkItemsUpdate },
     { method: 'POST', path: '/api/work-items/retry', desc: 'Reset a failed/dispatched item to pending', params: 'id, source?', handler: handleWorkItemsRetry },
     { method: 'POST', path: '/api/work-items/delete', desc: 'Remove a work item, kill agent, clear dispatch', params: 'id, source?', handler: handleWorkItemsDelete },
     { method: 'POST', path: '/api/work-items/cancel', desc: 'Cancel a work item, kill agent, clear dispatch', params: 'id, source?, reason?', handler: handleWorkItemsCancel },
@@ -13496,7 +13535,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     { method: 'POST', path: '/api/notes-save', desc: 'Save edited notes.md content', params: 'content, file?', handler: handleNotesSave },
 
     // Plans
-    { method: 'POST', path: '/api/plan', desc: 'Create a plan work item that chains to PRD on completion', params: 'title, description?, priority?, project? (string OR array for cross-repo plans), agent?, branch_strategy? or branchStrategy?', handler: handlePlanCreate },
+    { method: 'POST', path: '/api/plan', desc: 'Create a plan work item that chains to PRD on completion', params: `title, description?, ${WORK_ITEM_PRIORITY_PARAM_HINT}, project? (string OR array for cross-repo plans), agent?, branch_strategy? or branchStrategy?`, handler: handlePlanCreate },
     { method: 'GET', path: '/api/plans', desc: 'List plan files (.md drafts + .json PRDs)', handler: handlePlansList },
     { method: 'POST', path: '/api/plans/trigger-verify', desc: 'Manually trigger verification for a completed plan', params: 'file', handler: handlePlansTriggerVerify },
     { method: 'POST', path: '/api/plans/approve', desc: 'Approve a plan for execution', params: 'file, approvedBy?', handler: handlePlansApprove },
@@ -13513,8 +13552,8 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     { method: 'GET', path: /^\/api\/plans\/([^?]+)$/, template: '/api/plans/:file', desc: 'Read a full plan (JSON from prd/ or markdown from plans/)', handler: handlePlansRead },
 
     // PRD items
-    { method: 'POST', path: '/api/prd-items', desc: 'Create a PRD item as a plan file in prd/ (auto-approved)', params: 'name, description?, priority?, estimated_complexity?, project?, item_project?, id?', handler: handlePrdItemsCreate },
-    { method: 'POST', path: '/api/prd-items/update', desc: 'Edit a PRD item in its source plan JSON', params: 'source, itemId, name?, description?, priority?, estimated_complexity?, status?', handler: handlePrdItemsUpdate },
+    { method: 'POST', path: '/api/prd-items', desc: 'Create a PRD item as a plan file in prd/ (auto-approved)', params: `name, description?, ${WORK_ITEM_PRIORITY_PARAM_HINT}, estimated_complexity?, project?, item_project?, id?`, handler: handlePrdItemsCreate },
+    { method: 'POST', path: '/api/prd-items/update', desc: 'Edit a PRD item in its source plan JSON', params: `source, itemId, name?, description?, ${WORK_ITEM_PRIORITY_PARAM_HINT}, estimated_complexity?, status?`, handler: handlePrdItemsUpdate },
     { method: 'POST', path: '/api/prd-items/remove', desc: 'Remove a PRD item from plan + cancel materialized work item', params: 'source, itemId', handler: handlePrdItemsRemove },
     // /api/prd/regenerate removed — use /api/plans/approve which does diff-aware update
 
@@ -14323,8 +14362,8 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
     // Schedules
     { method: 'POST', path: '/api/schedules/parse-natural', desc: 'Parse natural language schedule text into cron expression', params: 'text', handler: handleSchedulesParseNatural },
-    { method: 'POST', path: '/api/schedules', desc: 'Create a new schedule', params: 'cron, title, id?, type?, project?, agent?, description?, priority?, enabled?', handler: handleSchedulesCreate },
-    { method: 'POST', path: '/api/schedules/update', desc: 'Update an existing schedule', params: 'id, cron?, title?, type?, project?, agent?, description?, priority?, enabled?', handler: handleSchedulesUpdate },
+    { method: 'POST', path: '/api/schedules', desc: 'Create a new schedule', params: `cron, title, id?, type?, project?, agent?, description?, ${WORK_ITEM_PRIORITY_PARAM_HINT}, enabled?`, handler: handleSchedulesCreate },
+    { method: 'POST', path: '/api/schedules/update', desc: 'Update an existing schedule', params: `id, cron?, title?, type?, project?, agent?, description?, ${WORK_ITEM_PRIORITY_PARAM_HINT}, enabled?`, handler: handleSchedulesUpdate },
     { method: 'POST', path: '/api/schedules/delete', desc: 'Delete a schedule', params: 'id', handler: handleSchedulesDelete },
     { method: 'POST', path: '/api/schedules/run-now', desc: 'Manually enqueue the work item for a schedule', params: 'id', handler: handleSchedulesRunNow },
 
@@ -14345,9 +14384,11 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       const result = pipelines.map(p => ({ ...p, runs: (runs[p.id] || []).slice(-5) }));
       return jsonReply(res, 200, result);
     }},
-    { method: 'POST', path: '/api/pipelines', desc: 'Create a pipeline', params: 'id, title, stages[], trigger?, stopWhen?, monitoredResources?', handler: async (req, res) => {
+    { method: 'POST', path: '/api/pipelines', desc: 'Create a pipeline', params: `id, title, ${PIPELINE_STAGES_PARAM_HINT}, trigger?, stopWhen?, monitoredResources?`, handler: async (req, res) => {
       const body = await readBody(req);
       if (!body.id || !body.title || !body.stages) return jsonReply(res, 400, { error: 'id, title, and stages required' });
+      const priorityError = validatePipelineWorkItemPriorities({ stages: body.stages });
+      if (priorityError) return jsonReply(res, 400, priorityError);
       const { savePipeline, getPipeline } = require('./engine/pipeline');
       if (getPipeline(body.id)) return jsonReply(res, 409, { error: 'Pipeline already exists' });
       const pipeline = { id: body.id, title: body.title, stages: body.stages, trigger: body.trigger || {}, enabled: body.enabled !== false };
@@ -14361,12 +14402,14 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       invalidateStatusCache();
       return jsonReply(res, 200, { ok: true, id: pipeline.id });
     }},
-    { method: 'POST', path: '/api/pipelines/update', desc: 'Update a pipeline', params: 'id, title?, stages?, trigger?, enabled?, stopWhen?, monitoredResources?', handler: async (req, res) => {
+    { method: 'POST', path: '/api/pipelines/update', desc: 'Update a pipeline', params: `id, title?, ${PIPELINE_STAGES_PARAM_HINT.replace('stages[]', 'stages[]?')}, trigger?, enabled?, stopWhen?, monitoredResources?`, handler: async (req, res) => {
       const body = await readBody(req);
       if (!body.id) return jsonReply(res, 400, { error: 'id required' });
       const { getPipeline, savePipeline } = require('./engine/pipeline');
       const pipeline = getPipeline(body.id);
       if (!pipeline) return jsonReply(res, 404, { error: 'Pipeline not found' });
+      const priorityError = validatePipelineWorkItemPriorities({ stages: body.stages });
+      if (priorityError) return jsonReply(res, 400, priorityError);
       if (body.title !== undefined) pipeline.title = body.title;
       if (body.stages !== undefined) pipeline.stages = body.stages;
       if (body.trigger !== undefined) pipeline.trigger = body.trigger;

package/engine/shared.js

@@ -4048,6 +4048,27 @@ function backfillProjectWorkSourceDefaults(config) {
 
 // ─── Status & Type Constants ─────────────────────────────────────────────────
 
+const WORK_ITEM_PRIORITY = Object.freeze({
+  CRITICAL: 'critical',
+  HIGH: 'high',
+  MEDIUM: 'medium',
+  LOW: 'low',
+});
+const WORK_ITEM_PRIORITY_VALUES = Object.freeze(Object.values(WORK_ITEM_PRIORITY));
+const _WORK_ITEM_PRIORITY_SET = new Set(WORK_ITEM_PRIORITY_VALUES);
+const WORK_ITEM_PRIORITY_ERROR_CODE = 'invalid-work-item-priority';
+
+function validateWorkItemPriority(priority) {
+  if (priority === undefined || _WORK_ITEM_PRIORITY_SET.has(priority)) return null;
+  const rejected = JSON.stringify(priority);
+  return {
+    error: `Invalid work-item priority ${rejected}. Allowed values: ${WORK_ITEM_PRIORITY_VALUES.join(', ')}.`,
+    code: WORK_ITEM_PRIORITY_ERROR_CODE,
+    rejectedValue: priority,
+    allowedValues: [...WORK_ITEM_PRIORITY_VALUES],
+  };
+}
+
 const WI_STATUS = {
   PENDING: 'pending', DISPATCHED: 'dispatched', DONE: 'done', FAILED: 'failed',
   PAUSED: 'paused', QUEUED: 'queued',
@@ -8781,6 +8802,7 @@ module.exports = {
   runtimeConfigWarnings,
   projectWorkSourceWarnings,
   backfillProjectWorkSourceDefaults,
+  WORK_ITEM_PRIORITY, WORK_ITEM_PRIORITY_VALUES, WORK_ITEM_PRIORITY_ERROR_CODE, validateWorkItemPriority,
   WI_STATUS, DONE_STATUSES, PLAN_TERMINAL_STATUSES, isPrdArchived, isDefunctPrd, WORK_TYPE, isFixLikeWorkType, WORKTREE_REQUIRING_TYPES, LIVE_VALIDATION_WORK_TYPES, VALID_WORK_TYPES, resolveWorkItemTypeFromPrdItem, PLAN_STATUS, PRD_ITEM_STATUS, PRD_MATERIALIZABLE, PR_STATUS, PR_POLLABLE_STATUSES, PR_PENDING_REASON, BUILD_STATUS, REVIEW_STATUS, FETCH_TIMEOUT_MS, RETRY_DELAY_MS, ADO_TOKEN_REFRESH_MAX_RETRIES, DISPATCH_RESULT, mutateMetrics, mutateWatches, mutateScheduleRuns, mutatePipelineRuns, mutateManagedProcesses, mutateWorktreePool, mutateQaRuns, mutateQaSessions, trackReviewMetric, queuePlanToPrd, extractPlanDeclaredProject, extractPlanTargetProjects,
   WATCH_STATUS, WATCH_TARGET_TYPE, WATCH_CONDITION, WATCH_ABSOLUTE_CONDITIONS, WATCH_ACTION_TYPE,
   WATCH_STALLED_DEFAULT_TICKS, WATCH_STUCK_STAGE_DEFAULT_TICKS,

package/engine/watch-actions.js

@@ -397,12 +397,16 @@ registerActionType(WATCH_ACTION_TYPE.DISPATCH_WORK_ITEM, {
     { key: 'type', required: false, description: 'Work type (implement|fix|review|...). Defaults to implement.' },
     { key: 'agent', required: false, description: 'Specific agent to route to.' },
     { key: 'project', required: false, description: 'Target project (defaults to watch.project, or central).' },
-    { key: 'priority', required: false, description: 'low|medium|high. Defaults to medium.' },
+    { key: 'priority', required: false, description: `${shared.WORK_ITEM_PRIORITY_VALUES.join('|')}. Defaults to medium.` },
   ],
   handler: async (watch, ctx) => {
     const p = ctx.params || {};
     const title = String(p.title || '').trim();
     if (!title) return { ok: false, summary: 'dispatch-work-item: title is required' };
+    const priorityError = shared.validateWorkItemPriority(p.priority);
+    if (priorityError) {
+      return { ok: false, summary: priorityError.error, error: priorityError };
+    }
 
     const project = p.project || watch.project || null;
     const scope = project || 'central';
@@ -412,7 +416,7 @@ registerActionType(WATCH_ACTION_TYPE.DISPATCH_WORK_ITEM, {
       id,
       title,
       type: p.type || WORK_TYPE.IMPLEMENT,
-      priority: p.priority || 'medium',
+      priority: p.priority ?? shared.WORK_ITEM_PRIORITY.MEDIUM,
       description: p.description || '',
       status: WI_STATUS.PENDING,
       created: ts(),
@@ -473,6 +477,7 @@ registerActionType(WATCH_ACTION_TYPE.RUN_SKILL, {
     { key: 'args', required: false, description: 'Args/instructions to pass to the skill.' },
     { key: 'agent', required: false, description: 'Specific agent to route to.' },
     { key: 'project', required: false, description: 'Target project.' },
+    { key: 'priority', required: false, description: `${shared.WORK_ITEM_PRIORITY_VALUES.join('|')}. Defaults to medium.` },
   ],
   handler: async (watch, ctx) => {
     const p = ctx.params || {};
@@ -488,7 +493,7 @@ registerActionType(WATCH_ACTION_TYPE.RUN_SKILL, {
         type: WORK_TYPE.IMPLEMENT,
         agent: p.agent,
         project: p.project,
-        priority: p.priority || 'medium',
+        priority: p.priority ?? shared.WORK_ITEM_PRIORITY.MEDIUM,
       },
     });
   },
@@ -1092,6 +1097,8 @@ function validateAction(action) {
       return `action.params.${p.key} is required for action type ${type}`;
     }
   }
+  const priorityError = validateActionWorkItemPriority(action);
+  if (priorityError) return priorityError.error;
   // P-w7c5d8b3 — Phase 3.2: optional `guard` field. Shape is locked here
   // (object with a string `expr`), but the expression itself is NOT
   // pre-parsed at validation time — safe-expr.evaluate is the single
@@ -1178,6 +1185,23 @@ function validateAction(action) {
   return null;
 }
 
+function validateActionWorkItemPriority(action) {
+  if (action === null || action === undefined) return null;
+  if (Array.isArray(action)) {
+    for (const step of action) {
+      const error = validateActionWorkItemPriority(step);
+      if (error) return error;
+    }
+    return null;
+  }
+  if (typeof action !== 'object') return null;
+  const type = String(action.type || '').toLowerCase();
+  if (type !== WATCH_ACTION_TYPE.DISPATCH_WORK_ITEM && type !== WATCH_ACTION_TYPE.RUN_SKILL) return null;
+  const priority = action.params?.priority;
+  if (typeof priority === 'string' && priority.includes('{{')) return null;
+  return shared.validateWorkItemPriority(priority);
+}
+
 module.exports = {
   registerActionType,
   getActionType,
@@ -1187,6 +1211,7 @@ module.exports = {
   buildTriggerContext,
   substituteTemplate,
   validateAction,
+  validateActionWorkItemPriority,
   computeWatchDedupKey, // exported for testing (BUG-H10)
   WATCH_DEDUP_ACTIVE_STATUSES, // exported for testing (BUG-H10)
 };

package/engine/watches.js

@@ -1455,6 +1455,7 @@ module.exports = {
   runWatchAction: watchActions.runWatchAction,
   runWatchActionChain: watchActions.runWatchActionChain,
   validateAction: watchActions.validateAction,
+  validateActionWorkItemPriority: watchActions.validateActionWorkItemPriority,
   _captureState,  // exported for testing
   _runActionTask, // exported for testing — invoked by checkWatches per fired action
   _TARGET_TYPES: TARGET_TYPES, // exported for testing — direct registry access

package/package.json

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