@yemi33/minions 0.1.2448 → 0.1.2449

package/dashboard/js/render-prs.js

@@ -3,7 +3,6 @@
 let allPrs = [];
 let filteredPrs = [];
 let prPage = 0;
-let prFilterOptionsSignature = '';
 const PR_PER_PAGE = 25;
 const PR_FILTER_CONTROL_IDS = {
   author: 'pr-filter-author',
@@ -12,13 +11,32 @@ const PR_FILTER_CONTROL_IDS = {
   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',
-};
+
+// DOM half of the filter bar — the generic controller from
+// shared/record-filters.js, shared with the classic Work Items bar. It owns the
+// option-signature gate that keeps the 4 s poll from rebuilding <option> nodes
+// under an open dropdown.
+//
+// Built lazily rather than at module scope: this file is also require()d
+// directly under Node by tests that exercise its pure helpers, where `window`
+// does not exist.
+let _prFilterBarInstance = null;
+function _prFilterBar() {
+  if (!_prFilterBarInstance) {
+    _prFilterBarInstance = window.MinionsRecordFilters.createBar({
+      controlIds: PR_FILTER_CONTROL_IDS,
+      allLabels: {
+        author: 'All authors',
+        lifecycle: 'All lifecycle statuses',
+        project: 'All projects',
+        review: 'All review statuses',
+        build: 'All build statuses',
+      },
+      getOptions: getPrFilterOptions,
+    });
+  }
+  return _prFilterBarInstance;
+}
 
 // Pull requests are served fresh by GET /api/pull-requests (see refresh.js),
 // which runs queries.getPullRequests() server-side. The per-project
@@ -103,6 +121,60 @@ function _renderPrMergeConflictBadge(pr) {
     '" aria-label="' + escapeHtml(PR_MERGE_CONFLICT_LABEL) + '">conflict</span>';
 }
 
+function prAutomationToggle(idMatch, automation, label, enabled) {
+  if (!idMatch) return '';
+  var stateClass = enabled ? 'pr-automation-on' : 'pr-automation-off';
+  var stateLabel = enabled ? 'enabled' : 'disabled';
+  var title = label + ' ' + stateLabel + ' for this PR. Click to ' + (enabled ? 'disable.' : 'enable.');
+  return '<label class="pr-automation-switch ' + stateClass + '" '
+    + 'title="' + escapeHtml(title) + '" '
+    + 'onclick="event.stopPropagation()">'
+    + '<input type="checkbox" class="pr-automation-switch__input"'
+    + (enabled ? ' checked' : '') + ' '
+    + 'aria-label="' + escapeHtml(label + ': ' + stateLabel) + '" '
+    + 'data-pr-host="' + escapeHtml(idMatch[1]) + '" '
+    + 'data-pr-slug="' + escapeHtml(idMatch[2]) + '" '
+    + 'data-pr-number="' + escapeHtml(idMatch[3]) + '" '
+    + 'data-pr-automation="' + escapeHtml(automation) + '" '
+    + 'data-pr-enabled="' + (enabled ? '1' : '0') + '" '
+    + 'onchange="togglePrAutomation(this)">'
+    + '<span class="pr-automation-switch__track" aria-hidden="true">'
+    + '<span class="pr-automation-switch__knob"></span>'
+    + '</span>'
+    + '</label>';
+}
+
+// PR author cell (W-mscibegy005e8b74). Renders the real individual the PR is
+// for — display name plus @login when both are known — separate from the
+// Minions agent (still shown in the PR detail panel). Missing/legacy author
+// renders "Unknown" rather than borrowing the agent name. Links to the provider
+// profile only when a validated http(s) URL is present.
+function _prAuthorCellHtml(pr) {
+  var a = pr ? pr.author : null;
+  var login = '';
+  var displayName = '';
+  var url = '';
+  if (a && typeof a === 'object') {
+    login = a.login || '';
+    displayName = a.displayName || '';
+    url = a.url || '';
+  } else if (typeof a === 'string') {
+    login = a;
+  }
+  var primary = displayName || login;
+  if (!primary) {
+    return '<span style="color:var(--muted);font-size:var(--text-base)" title="Author unavailable">Unknown</span>';
+  }
+  var secondary = (displayName && login && login !== displayName) ? login : '';
+  var inner = escapeHtml(primary)
+    + (secondary ? ' <span style="color:var(--muted);font-size:var(--text-base)">@' + escapeHtml(secondary) + '</span>' : '');
+  var titleText = secondary ? primary + ' (' + secondary + ')' : primary;
+  if (url && /^https?:\/\//i.test(url)) {
+    return '<a class="pr-agent" title="' + escapeHtml(titleText) + '" href="' + escapeHtml(safeUrl(url)) + '" target="_blank" rel="noopener">' + inner + '</a>';
+  }
+  return '<span class="pr-agent" title="' + escapeHtml(titleText) + '">' + inner + '</span>';
+}
+
 function prRow(pr) {
   // Minions review (agent) state — separate from ADO human review
   const sq = pr.minionsReview || {};
@@ -118,10 +190,24 @@ function prRow(pr) {
     ? 'ADO has builds on this PR\'s merge ref, but none target the current merge commit (typically after a rebase or target-branch advance). Engine classified from the pre-rebase build set and queued a fresh build (rate limited 30m). See Issue #2747.'
     : (pr._buildStatusDetail || '');
   const mergeConflictBadge = _renderPrMergeConflictBadge(pr);
-  const mergeStateHtml = mergeConflictBadge ||
-    '<span style="color:var(--muted);font-size:var(--text-base)" aria-label="No merge conflict reported">—</span>';
-  const statusClass = pr.status === 'merged' ? 'merged' : pr.status === 'abandoned' ? 'rejected' : pr.status === 'active' ? 'active' : 'draft';
-  const statusLabel = pr.status || 'active';
+  // W-ms8z6l8z — the lifecycle badge is a single, mutually-exclusive state:
+  // an open draft PR reads "draft"; an open non-draft PR reads "active";
+  // terminal PRs read merged/abandoned. The canonical draft signal is
+  // pr.isDraft (plumbed by engine/providers/github.js + engine/ado/index.js,
+  // exposed via engine/api/bridge.js), never title text. There is no separate
+  // "ready" pill — "active" already communicates open + non-draft, so
+  // rendering both "active" and "ready" together was redundant. The
+  // merge-conflict badge (formerly its own never-populated "Merge" column) is
+  // folded in beside the lifecycle badge because it is a genuinely distinct
+  // state that "active"/"draft" does not convey.
+  const isDraftPr = pr.isDraft === true;
+  const isOpenPr = !pr.status || pr.status === 'active';
+  const statusClass = pr.status === 'merged' ? 'merged'
+    : pr.status === 'abandoned' ? 'rejected'
+    : (isDraftPr && isOpenPr) ? 'draft'
+    : pr.status === 'active' ? 'active'
+    : 'draft';
+  const statusLabel = (isDraftPr && isOpenPr) ? 'draft' : (pr.status || 'active');
   const branchError = pr._branchResolutionError?.reason || '';
   const branchLabel = pr.branch || (branchError ? 'missing branch' : '—');
   const branchClass = 'pr-branch' + (branchError ? ' branch-missing' : '');
@@ -166,7 +252,7 @@ function prRow(pr) {
   // no-op pausedCauses above: pr._buildFixIneffective is set when a
   // BUILD_FAILURE fix reported "no branch change" AND a live CI re-poll
   // confirmed the build is STILL failing on that same head (see
-  // engine/lifecycle.js recordBuildFixIneffective). This is a separate,
+  // engine/orchestration/lifecycle.js recordBuildFixIneffective). This is a separate,
   // headSha-keyed pause distinct from _noOpFixes, so it needs its own,
   // clearly-labeled indicator — a still-red build after an unchanged fix
   // attempt usually means the failure isn't fixable by another agent pass
@@ -197,7 +283,7 @@ function prRow(pr) {
   // Issue #764 — worktree-held-externally chip. pr._worktreeHeldPaused is set
   // when a PR-driven review/fix dispatch hit WORKTREE_PREFLIGHT because this
   // PR's branch is checked out at the project root or in an external
-  // worktree the engine can't reach (see engine/lifecycle.js
+  // worktree the engine can't reach (see engine/orchestration/lifecycle.js
   // recordWorktreeHeldPause). engine.js auto-clears this pause once it
   // detects the holder no longer holds the branch; this chip lets an
   // operator force-clear it immediately via
@@ -218,7 +304,7 @@ function prRow(pr) {
   // Polish (W-mqv5ccn7): flatten Markdown in the body so the preview shows
   // plain text instead of raw `##` / `**…**` / backtick source.
   const descPreview = stripMarkdown(pr.description || '');
-  const agentText = pr.agent || '—';
+  const authorCellHtml = _prAuthorCellHtml(pr);
   const reviewerCell = sq.reviewer && sq.status !== 'waiting'
     ? '<span class="pr-agent" title="' + escapeHtml(sq.note || sq.reviewer) + '">' + escapeHtml(sq.reviewer) + '</span>'
     : sq.reviewer && sq.status === 'waiting'
@@ -227,96 +313,72 @@ function prRow(pr) {
         ? '<span class="pr-agent" title="' + escapeHtml(pr.reviewedBy.join(', ')) + '">' + escapeHtml(pr.reviewedBy.join(', ')) + '</span>'
         : '<span style="color:var(--muted);font-size:var(--text-base)">—</span>';
   const createdLabel = (pr.created || '—').slice(0, 16).replace('T', ' ');
-  // Per-row auto-observe toggle (W-mpmwxkzm0009ba0b; canonicalized in
-  // W-mq5s5ttx000j7ab8-c). pr.contextOnly === true means the engine polls
-  // status/comments but does NOT dispatch review/fix agents against this
-  // PR. Default (undefined / false) = observed.
-  var observe = pr.contextOnly !== true;
-  var observeClass = observe ? 'pr-observe-on' : 'pr-observe-off';
-  var observeLabel = observe ? 'observed' : 'context';
-  var observeTitle = observe
-    ? 'Auto-observe ON — engine reviews this PR and dispatches fix agents on review/build feedback. Click to switch to context-only.'
-    : 'Context only — engine polls status and comments but will not dispatch review/fix agents. Click to enable auto-observe.';
-  // pr.id is canonical: "<host>:<slug>#<number>" (engine/shared.js getCanonicalPrId).
-  // Parse here so the toggle handler can post {host, slug, number} without
-  // shipping additional fields per row.
-  var observeBtn = '';
+  // pr.id is canonical: "<host>:<slug>#<number>" (engine/core/shared.js getCanonicalPrId).
+  // Existing records inherit both controls from contextOnly until an operator
+  // changes either switch, at which point the server persists both explicitly.
   var idMatch = typeof pr.id === 'string' ? pr.id.match(/^([^:]+):(.+)#(\d+)$/) : null;
-  if (idMatch) {
-    // Real toggle-switch (W-mpof8qam000h62fb). Semantic <input type=checkbox>
-    // inside a <label> gives free keyboard + screen-reader support; clicks on
-    // the track or knob flip the state via the wrapping label. Data-attrs stay
-    // on the input so togglePrObserve(this) keeps working unchanged.
-    observeBtn = '<label class="pr-observe-switch ' + observeClass + '" '
-      + 'title="' + escapeHtml(observeTitle) + '" '
-      + 'onclick="event.stopPropagation()">'
-      + '<input type="checkbox" class="pr-observe-switch__input"'
-      + (observe ? ' checked' : '') + ' '
-      + 'aria-label="' + escapeHtml('Auto-observe: ' + observeLabel) + '" '
-      + 'data-pr-host="' + escapeHtml(idMatch[1]) + '" '
-      + 'data-pr-slug="' + escapeHtml(idMatch[2]) + '" '
-      + 'data-pr-number="' + escapeHtml(idMatch[3]) + '" '
-      + 'data-pr-observe="' + (observe ? '1' : '0') + '" '
-      + 'onchange="togglePrObserve(this)">'
-      + '<span class="pr-observe-switch__track" aria-hidden="true">'
-      + '<span class="pr-observe-switch__knob"></span>'
-      + '</span>'
-      + '</label> ';
-  }
+  var autoFixEnabled = typeof pr.autoFixEnabled === 'boolean'
+    ? pr.autoFixEnabled
+    : pr.contextOnly !== true;
+  var autoReviewEnabled = typeof pr.autoReviewEnabled === 'boolean'
+    ? pr.autoReviewEnabled
+    : pr.contextOnly !== true;
+  var autoFixBtn = prAutomationToggle(idMatch, 'fix', 'Auto Fix', autoFixEnabled);
+  var autoReviewBtn = prAutomationToggle(idMatch, 'review', 'Auto Review', autoReviewEnabled);
   // Title attrs live on the inner element (link/span/badge) so hovering the
   // ellipsis-truncated content reveals the full text. Cell tags stay bare so
   // the header-to-cell count assertion in test/unit.test.js continues to
   // balance.
-  // First-column secondary project label (W-mr3mowg9) — mirrors the pr-desc
-  // secondary-text pattern used in the Title cell so reviewers can see at a
-  // glance which project a PR belongs to. `_project` is stamped by the engine
-  // enrichment / getAllPrs stamping (see top-of-file note + line ~33).
+  // Dedicated Project column (W-ms90po5j; was a secondary label folded into
+  // the PR-id cell under W-mr3mowg9). `_project` is stamped by the engine
+  // enrichment / getAllPrs stamping (see top-of-file note + line ~33). Empty or
+  // unknown project values render the shared muted em-dash placeholder so the
+  // cell matches the table's other empty-value conventions (e.g. Observe).
   const projectName = pr._project ? String(pr._project) : '';
-  const projectLabelHtml = projectName
-    ? '<div class="pr-desc pr-id-project" title="' + escapeHtml(projectName) + '">' + escapeHtml(projectName) + '</div>'
-    : '';
+  const projectCellHtml = projectName
+    ? '<span class="pr-project" title="' + escapeHtml(projectName) + '">' + escapeHtml(projectName) + '</span>'
+    : '<span style="color:var(--muted);font-size:var(--text-base)" aria-label="No project">—</span>';
   return '<tr>' +
-    '<td><span class="pr-id" title="' + escapeHtml(String(prId)) + '">' + escapeHtml(prShortId(String(prId))) + '</span>' + projectLabelHtml + '</td>' +
+    '<td><span class="pr-id" title="' + escapeHtml(String(prId)) + '">' + escapeHtml(prShortId(String(prId))) + '</span></td>' +
+    '<td>' + projectCellHtml + '</td>' +
     '<td><a class="pr-title" title="' + escapeHtml(titleText) + '" href="' + escapeHtml(safeUrl(url)) + '" target="_blank" rel="noopener">' + escapeHtml(titleText) + '</a>' + followupChip + pausedChips + (descPreview ? '<div class="pr-desc" title="' + escapeHtml(descPreview) + '">' + escapeHtml(descPreview.length > 120 ? descPreview.slice(0, 120) + '...' : descPreview) + '</div>' : '') + '</td>' +
-    '<td><span class="pr-badge ' + statusClass + '" title="' + escapeHtml(statusLabel) + '">' + escapeHtml(statusLabel) + '</span></td>' +
+    '<td><span class="pr-status-badges"><span class="pr-badge ' + statusClass + '" title="' + escapeHtml(statusLabel) + '">' + escapeHtml(statusLabel) + '</span>' + mergeConflictBadge + '</span></td>' +
     '<td><span class="pr-badge ' + reviewClass + '" title="' + escapeHtml(reviewTitle || reviewLabel) + '">' + escapeHtml(reviewLabel) + '</span></td>' +
     '<td><span class="pr-badge ' + buildClass + '" title="' + escapeHtml(buildTitle || buildLabel) + '">' + escapeHtml(buildLabel) + '</span></td>' +
-    '<td>' + mergeStateHtml + '</td>' +
     '<td><span class="' + branchClass + '" title="' + escapeHtml(branchError || branchLabel) + '">' + escapeHtml(branchLabel) + '</span>' + pendingReasonHtml + '</td>' +
-    '<td><span class="pr-agent" title="' + escapeHtml(agentText) + '">' + escapeHtml(agentText) + '</span></td>' +
+    '<td>' + authorCellHtml + '</td>' +
     '<td>' + reviewerCell + '</td>' +
-    '<td>' + (observeBtn || '<span style="color:var(--muted);font-size:var(--text-base)">—</span>') + '</td>' +
+    '<td>' + (autoFixBtn || '<span style="color:var(--muted);font-size:var(--text-base)">—</span>') + '</td>' +
+    '<td>' + (autoReviewBtn || '<span style="color:var(--muted);font-size:var(--text-base)">—</span>') + '</td>' +
     '<td><span class="pr-date" title="' + escapeHtml(createdLabel) + '">' + escapeHtml(createdLabel) + '</span></td>' +
-    '<td><button class="pr-info-pill" data-pr-id="' + escapeHtml(String(prId)) + '" onclick="event.stopPropagation();openPrDetail(this.dataset.prId)" title="Open PR detail (in-stack modal)">&#x2197;</button>' +
-      '<button class="btn-destructive" style="padding:1px 5px;margin-left:4px" data-pr-id="' + escapeHtml(String(prId)) + '" onclick="event.stopPropagation();unlinkPr(this.dataset.prId)" title="Remove from tracking">x</button></td>' +
+    '<td><button class="btn-destructive" style="padding:1px 5px" data-pr-id="' + escapeHtml(String(prId)) + '" onclick="event.stopPropagation();unlinkPr(this.dataset.prId)" title="Remove from tracking">x</button></td>' +
     '</tr>';
 }
 
-// Explicit per-column widths keep the PR table from ballooning when titles or
-// branches are long. Total 1600px -> table grows past viewport on narrow
-// windows and the .pr-table-wrap--prs container scrolls horizontally inside
-// the viewport (sticky scrollbar — see styles.css).
+// Explicit widths keep long titles/branches bounded; narrow viewports scroll
+// inside .pr-table-wrap--prs (sticky scrollbar — see styles.css).
 const PRS_COLGROUP =
   '<colgroup>' +
-    '<col style="width:130px">' +   // PR id (short label: `gh #481` / `ado #5383607`) + project label
+    '<col style="width:130px">' +   // PR id (short label: `gh #481` / `ado #5383607`)
+    '<col style="width:130px">' +   // Project
     '<col style="width:320px">' +   // Title
-    '<col style="width:110px">' +   // Status
+    '<col style="width:150px">' +   // Status (lifecycle badge [draft/active/merged/abandoned] + merge-conflict badge)
     '<col style="width:130px">' +   // Review
     '<col style="width:130px">' +   // Build
-    '<col style="width:120px">' +   // Merge conflict
     '<col style="width:200px">' +   // Branch
-    '<col style="width:140px">' +   // Agent
+    '<col style="width:160px">' +   // Author
     '<col style="width:140px">' +   // Signed Off By
-    '<col style="width:100px">' +   // Observe
+    '<col style="width:100px">' +   // Auto Fix
+    '<col style="width:100px">' +   // Auto Review
     '<col style="width:130px">' +   // Created
-    '<col style="width:80px">' +    // Actions (info-pill + remove)
+    '<col style="width:80px">' +    // Actions (remove)
   '</colgroup>';
 
 function prTableHtml(rows) {
   return '<div class="pr-table-wrap pr-table-wrap--prs"><table class="pr-table pr-table--prs">' +
     PRS_COLGROUP +
     '<thead><tr>' +
-    '<th>PR</th><th>Title</th><th>Status</th><th>Review</th><th>Build</th><th>Merge</th><th>Branch</th><th>Agent</th><th>Signed Off By</th><th>Observe</th><th>Created</th><th></th>' +
+    '<th>PR</th><th>Project</th><th>Title</th><th>Status</th><th>Review</th><th>Build</th><th>Branch</th><th>Author</th><th>Signed Off By</th><th>Auto Fix</th><th>Auto Review</th><th>Created</th><th></th>' +
     '</tr></thead><tbody>' + rows + '</tbody></table></div>';
 }
 
@@ -333,61 +395,13 @@ 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 = '';
-  });
+  _prFilterBar().clear();
   applyPrFilters();
 }
 
@@ -395,21 +409,22 @@ function renderPrs(prs, opts) {
   opts = opts || {};
   prs = Array.isArray(prs) ? prs.filter(p => p && !isDeleted('pr:' + p.id)) : [];
   allPrs = prs;
-  _syncPrFilterOptions(allPrs);
-  const filters = _readPrFilters();
+  const _bar = _prFilterBar();
+  _bar.syncOptions(allPrs);
+  const filters = _bar.read();
   filteredPrs = filterPullRequests(allPrs, filters);
   const el = document.getElementById('pr-content');
   const count = document.getElementById('pr-count');
   const summary = document.getElementById('pr-filter-summary');
   count.textContent = filteredPrs.length;
   if (summary) {
-    summary.textContent = _hasActivePrFilters(filters)
+    summary.textContent = _bar.hasActive(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>';
+    el.innerHTML = '<p class="pr-empty">No pull requests yet. PRs created by agents will appear here with review and build status.</p>';
     return;
   }
   if (!filteredPrs.length) {
@@ -521,12 +536,14 @@ function _renderPrDetail(pr) {
   const mergeConflictBadge = _renderPrMergeConflictBadge(pr);
 
   const badges =
-    '<span class="pr-badge ' + statusClass + '" title="status">' + escapeHtml(String(statusLabel)) + '</span> ' +
-    '<span class="pr-badge ' + reviewClass + '" title="review">' + escapeHtml(String(reviewLabel)) + '</span> ' +
+    '<span class="pr-status-badges">' +
+    '<span class="pr-badge ' + statusClass + '" title="status">' + escapeHtml(String(statusLabel)) + '</span>' +
+    '<span class="pr-badge ' + reviewClass + '" title="review">' + escapeHtml(String(reviewLabel)) + '</span>' +
     '<span class="pr-badge ' + buildClass + '" title="build">' + escapeHtml(String(buildLabel)) + '</span>' +
-    (mergeConflictBadge ? ' ' + mergeConflictBadge : '');
+    mergeConflictBadge +
+    '</span>';
 
-  // Linked WIs — join via item._pr === pr.id (stamped by engine/queries.js).
+  // Linked WIs — join via item._pr === pr.id (stamped by engine/core/queries.js).
   const wis = (window._lastWorkItems) || [];
   const linkedWis = wis.filter(w => w && w._pr === pr.id);
   let linkedChips = '';
@@ -601,7 +618,26 @@ function _renderPrDetail(pr) {
       '</div>';
   }
 
-  const agent = pr.agent ? '<div><strong style="color:var(--muted);font-size:var(--text-base)">Agent:</strong> ' + escapeHtml(String(pr.agent)) + '</div>' : '';
+  // Author (real individual the PR is for) — shown alongside the Minions agent
+  // so operators can answer both "who is this for?" and "which agent handled it?".
+  let author = '';
+  {
+    const a = pr.author;
+    let login = '', displayName = '', url = '';
+    if (a && typeof a === 'object') { login = a.login || ''; displayName = a.displayName || ''; url = a.url || ''; }
+    else if (typeof a === 'string') { login = a; }
+    const primary = displayName || login;
+    if (primary) {
+      const secondary = (displayName && login && login !== displayName) ? ' <span style="color:var(--muted)">@' + escapeHtml(login) + '</span>' : '';
+      const valHtml = (url && /^https?:\/\//i.test(url))
+        ? '<a href="' + escapeHtml(safeUrl(url)) + '" target="_blank" rel="noopener">' + escapeHtml(primary) + '</a>' + secondary
+        : escapeHtml(primary) + secondary;
+      author = '<div><strong style="color:var(--muted);font-size:var(--text-base)">Author:</strong> ' + valHtml + '</div>';
+    } else {
+      author = '<div><strong style="color:var(--muted);font-size:var(--text-base)">Author:</strong> <span style="color:var(--muted)">Unknown</span></div>';
+    }
+  }
+  const agent = pr.agent ? '<div><strong style="color:var(--muted);font-size:var(--text-base)">Minions agent:</strong> ' + escapeHtml(String(pr.agent)) + '</div>' : '';
   const branch = pr.branch ? '<div><strong style="color:var(--muted);font-size:var(--text-base)">Branch:</strong> <code>' + escapeHtml(String(pr.branch)) + '</code></div>' : '';
   const created = pr.created ? '<div><strong style="color:var(--muted);font-size:var(--text-base)">Created:</strong> ' + escapeHtml(String(pr.created)) + '</div>' : '';
   const project = pr._project ? '<div><strong style="color:var(--muted);font-size:var(--text-base)">Project:</strong> ' + escapeHtml(String(pr._project)) + '</div>' : '';
@@ -620,7 +656,7 @@ function _renderPrDetail(pr) {
       '<code style="color:var(--muted);font-size:var(--text-base)">' + escapeHtml(String(pr.id || '')) + '</code><br>' +
       badges +
     '</div>' +
-    agent + branch + project + created +
+    author + agent + branch + project + created +
     reviewers +
     linkedChips +
     parentChip +
@@ -733,57 +769,50 @@ async function unlinkPr(id) {
 // consumes canonical `contextOnly` to gate review/fix dispatch (engine/
 // shared.js isAutoManagedPrRecord + discoverFromPrs).
 //
-// `btn` is the <input type=checkbox> inside .pr-observe-switch. Data-attrs
-// live on the input so this handler keeps reading them directly; visual
-// state is mirrored onto the wrapping <label> via the pr-observe-on/off
-// class so CSS can paint the track and slide the knob.
-async function togglePrObserve(btn) {
+async function togglePrAutomation(btn) {
   if (!btn || btn.disabled) return;
   const host = btn.dataset.prHost;
   const slug = btn.dataset.prSlug;
   const number = parseInt(btn.dataset.prNumber, 10);
-  if (!host || !slug || !Number.isFinite(number)) return;
-  const wasObserve = btn.dataset.prObserve === '1';
-  const next = !wasObserve;
-  const wrapper = btn.closest('.pr-observe-switch');
-  // Optimistic flip
-  btn.dataset.prObserve = next ? '1' : '0';
+  const automation = btn.dataset.prAutomation;
+  if (!host || !slug || !Number.isFinite(number) || !['fix', 'review'].includes(automation)) return;
+  const wasEnabled = btn.dataset.prEnabled === '1';
+  const next = !wasEnabled;
+  const label = automation === 'fix' ? 'Auto Fix' : 'Auto Review';
+  const wrapper = btn.closest('.pr-automation-switch');
+  btn.dataset.prEnabled = next ? '1' : '0';
   btn.checked = next;
-  btn.setAttribute('aria-label', 'Auto-observe: ' + (next ? 'observed' : 'context'));
+  btn.setAttribute('aria-label', label + ': ' + (next ? 'enabled' : 'disabled'));
   if (wrapper) {
-    wrapper.classList.toggle('pr-observe-on', next);
-    wrapper.classList.toggle('pr-observe-off', !next);
+    wrapper.classList.toggle('pr-automation-on', next);
+    wrapper.classList.toggle('pr-automation-off', !next);
   }
   btn.disabled = true;
   try {
-    const res = await fetch('/api/pull-requests/observe', {
+    const res = await fetch('/api/pull-requests/automation', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
-      // Canonical contextOnly is the primary payload (P-2b6e4d81); the
-      // legacy `observe` body param remains a server-side back-compat
-      // fallback for any other caller, so the client no longer sends it.
-      body: JSON.stringify({ host, slug, number, contextOnly: !next })
+      body: JSON.stringify({ host, slug, number, automation, enabled: next })
     });
     if (!res.ok) {
       const d = await res.json().catch(() => ({}));
-      // Revert
-      btn.dataset.prObserve = wasObserve ? '1' : '0';
-      btn.checked = wasObserve;
-      btn.setAttribute('aria-label', 'Auto-observe: ' + (wasObserve ? 'observed' : 'context'));
+      btn.dataset.prEnabled = wasEnabled ? '1' : '0';
+      btn.checked = wasEnabled;
+      btn.setAttribute('aria-label', label + ': ' + (wasEnabled ? 'enabled' : 'disabled'));
       if (wrapper) {
-        wrapper.classList.toggle('pr-observe-on', wasObserve);
-        wrapper.classList.toggle('pr-observe-off', !wasObserve);
+        wrapper.classList.toggle('pr-automation-on', wasEnabled);
+        wrapper.classList.toggle('pr-automation-off', !wasEnabled);
       }
       showToast('pr-toast', 'Failed: ' + (d.error || 'unknown'), false);
       return;
     }
-    showToast('pr-toast', next ? 'Auto-observe enabled' : 'Switched to context only', true);
+    showToast('pr-toast', label + (next ? ' enabled' : ' disabled'), true);
   } catch (e) {
-    btn.dataset.prObserve = wasObserve ? '1' : '0';
-    btn.checked = wasObserve;
-    btn.setAttribute('aria-label', 'Auto-observe: ' + (wasObserve ? 'observed' : 'context'));
+    btn.dataset.prEnabled = wasEnabled ? '1' : '0';
+    btn.checked = wasEnabled;
+    btn.setAttribute('aria-label', label + ': ' + (wasEnabled ? 'enabled' : 'disabled'));
     if (wrapper) {
-      wrapper.classList.toggle('pr-observe-on', wasObserve);
-      wrapper.classList.toggle('pr-observe-off', !wasObserve);
+      wrapper.classList.toggle('pr-automation-on', wasEnabled);
+      wrapper.classList.toggle('pr-automation-off', !wasEnabled);
     }
     showToast('pr-toast', 'Error: ' + e.message, false);
   } finally {
@@ -905,7 +934,7 @@ window.MinionsPrs = {
   openAddPrModal,
   openPrDetail,
   unlinkPr,
-  togglePrObserve,
+  togglePrAutomation,
   resumePausedCause,
   resumeBuildFixIneffective,
   resumeWorktreeHeldPause