@yemi33/minions 0.1.2397 → 0.1.2398

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-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.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.
  *
@@ -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.2397",
+  "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"