fraim-hub 2.0.340 → 2.0.341

package/public/ai-hub/script.js

@@ -2234,6 +2234,16 @@ function composerIsArmed() {
 function syncComposerPlaceholder(conv = activeConversation()) {
   const textarea = els['coach-text'];
   if (!textarea || textarea.disabled) return;
+  // Issue #1911 R6: a pane surface that claims the composer states what to type
+  // for its own mode, and outranks every case below it.
+  const surfaceComposer = paneSurfaceComposerContract();
+  if (surfaceComposer && typeof surfaceComposer.placeholder === 'function') {
+    const surfacePlaceholder = surfaceComposer.placeholder(PaneSurfaces.activeContext());
+    if (surfacePlaceholder) {
+      textarea.placeholder = surfacePlaceholder;
+      return;
+    }
+  }
   if (state.cpSelectedJob) {
     textarea.placeholder = textarea.dataset.placeholderArmed || 'Now type what you want done...';
   } else if (convAwaitingReview(conv)) {
@@ -3954,11 +3964,15 @@ function formatMicroLogLine(event, mode = 'events') {
 
 function renderActive() {
   const conv = activeConversation();
+  // Issue #1911 R7: a focused conversation owns the pane. Evicting here (rather
+  // than leaving a surface rendering behind the thread) is what makes the slot
+  // safe to reuse - every surface gets the same eviction rule for free.
+  if (conv && paneSurfaceOpen() && (typeof PaneSurfaces.isSubmitting !== 'function' || !PaneSurfaces.isSubmitting())) PaneSurfaces.close({ restoreFocus: false });
   if (typeof syncTaskPaneSurfaceMode === 'function') syncTaskPaneSurfaceMode();
   if (typeof tfApplyWorkspaceMode === 'function') tfApplyWorkspaceMode();
   if (!conv) {
     syncMicroArchive(null);
-    els['empty'].hidden = false;
+    els['empty'].hidden = paneSurfaceOpen();
     els['active-conv'].hidden = true;
     renderHubAgentSetupPanel();
     const _c = document.getElementById('conversation');
@@ -5664,23 +5678,16 @@ function latestEmployeeSurfaceText(conv) {
 }
 
 /**
- * Issue #1755: a sleep-on-learnings message carries its proposal set as a
- * "## Pending Review" markdown block. That block is the Hub's input, not prose
- * the manager should read: it renders below as cards they can act on. Printing
- * both puts every lesson title on the screen twice.
+ * Issue #1857: the strip step is gone along with the block it stripped.
  *
- * Only the block is dropped, and only when the cards are actually rendering.
- * Whatever the job wrote above it (its analysis summary) is what the manager
- * came to read, so that stays. When no cards render, nothing is stripped, so a
- * proposal set can never go missing.
+ * It existed because the job printed its proposal set as markdown into the turn
+ * and the Hub rendered cards from the same text, so both appeared. The set now
+ * travels as structured evidence and never appears in the prose, so there is
+ * nothing to remove. Kept as an identity function only so the single call site
+ * reads unchanged; removing it there is a separate, noisier diff.
  */
-function tfStripRenderedProposalBlock(text, conv) {
-  const raw = String(text || '');
-  if (!conv || conv.jobId !== 'sleep-on-learnings') return raw;
-  const marker = raw.search(/^##\s*Pending Review\s*$/im);
-  if (marker === -1) return raw;
-  if (typeof tfConvHasProposals !== 'function' || !tfConvHasProposals(conv)) return raw;
-  return raw.slice(0, marker).trimEnd();
+function tfStripRenderedProposalBlock(text) {
+  return String(text || '');
 }
 
 function appendMessageDom(role, text, conv, message) {
@@ -5746,7 +5753,7 @@ function appendMessageDom(role, text, conv, message) {
     // surfaceText collapses whitespace which destroys markdown. Only use it
     // when it returned a transformed value (Started/Resumed prefix overrides).
     const useSurfaced = surfaced && surfaced !== raw.replace(/\s+/g, ' ').trim();
-    const content = tfStripRenderedProposalBlock(useSurfaced ? surfaced : raw, conv);
+    const content = tfStripRenderedProposalBlock(useSurfaced ? surfaced : raw);
     bubble.innerHTML = formatEmployeeText(content || (conv.status === 'completed' ? 'Done, please review.' : 'Working on it…'));
   } else {
     bubble.textContent = role === 'manager' ? managerBubbleText(text) : (surfaceText(role, text, conv) || text);
@@ -6267,6 +6274,14 @@ function syncSendButton() {
     syncComposerStopButton(conv);
     return;
   }
+  // Issue #1911 R10: while a pane surface owns the composer, Send stays enabled
+  // so pressing it can report "Job details are required." rather than doing
+  // nothing. The retired dialog's own submit button was always enabled too.
+  if (paneSurfaceComposerContract()) {
+    els['send'].disabled = false;
+    syncComposerStopButton(conv);
+    return;
+  }
   // Issue #1651 R6: a valid staged attachment is enough to enable Send even
   // with an empty textarea, same as a pending coaching job.
   const hasAttachment = hasValidStagedAttachment();
@@ -7094,9 +7109,11 @@ function renderLearningProposalTurn(conv) {
   if (existing) existing.remove();
   if (!conv || conv.jobId !== 'sleep-on-learnings' || !tfConvHasProposals(conv)) return;
 
-  // The scope the run was asked for decides where an approved lesson is written.
-  const scope = tfProposalScopeForConversation(conv);
-  const proposals = tfParsePendingProposals(conv, scope);
+  // Issue #1857: the scope arrives on the set rather than being re-derived from
+  // the kickoff text, so a decision reaches the corpus its run was scoped to even
+  // when the kickoff wording is ambiguous.
+  const scope = conv.proposalSet.runScope || tfProposalScopeForConversation(conv);
+  const proposals = tfPendingProposals(conv);
   if (!proposals.length) return;
   // Keep the relay pointed at this run even if the accordion never registered it.
   tfLearningRunByScope[tfLearningRunKey(scope)] = conv;
@@ -7110,7 +7127,7 @@ function renderLearningProposalTurn(conv) {
 
   const label = document.createElement('div');
   label.className = 'review-turn-label';
-  const undecided = proposals.filter((p) => !tfProposalDecisions[p.id]).length;
+  const undecided = proposals.filter((p) => !tfProposalDecisions[tfProposalKey(p)]).length;
   label.textContent = undecided
     ? `${undecided} lesson${undecided === 1 ? '' : 's'} waiting on your decision`
     : 'Lessons reviewed';
@@ -7122,6 +7139,34 @@ function renderLearningProposalTurn(conv) {
   for (const p of proposals) list.appendChild(tfProposalCard(p, scope));
   bubble.appendChild(list);
 
+  // Issue #1857 R7. A claim the manager already decided is not a card, because
+  // re-asking a settled question is the cost this issue exists to remove. It is
+  // also not silence: suppression the manager cannot see is a system quietly
+  // deciding on their behalf. One line each, after the batch.
+  const suppressed = tfSuppressedClaims(conv);
+  if (suppressed.length) {
+    const note = document.createElement('div');
+    note.className = 'learn-suppressed';
+    note.dataset.testid = 'learn-suppressed';
+    const lead = document.createElement('div');
+    lead.className = 'ls-lead';
+    lead.textContent = `Not shown: ${suppressed.length} claim${suppressed.length === 1 ? '' : 's'} `
+      + `matched ${suppressed.length === 1 ? 'a decision' : 'decisions'} you already made.`;
+    note.appendChild(lead);
+    for (const claim of suppressed) {
+      const line = document.createElement('div');
+      line.className = 'ls-row';
+      // D2: a claim a rule or skill now owns was promoted, not refused. Saying
+      // "you rejected this" about work the manager approved elsewhere is wrong.
+      line.textContent = claim.kind === 'superseded'
+        ? `"${claim.claimKey}" superseded ${claim.rejectedAt}: ${claim.reason}`
+        : `"${claim.claimKey}" rejected ${claim.rejectedAt}: "${claim.reason}"`
+          + (claim.newSourceCount ? ` - ${claim.newSourceCount} new source${claim.newSourceCount === 1 ? '' : 's'}` : '');
+      note.appendChild(line);
+    }
+    bubble.appendChild(note);
+  }
+
   row.appendChild(bubble);
   // Below the thread, above the deliverables/next-job furniture.
   host.insertBefore(row, host.querySelector('#deliverables-bar, #next-job-section'));
@@ -7485,6 +7530,93 @@ function managementActionContext(area) {
   return { area: area || (typeof tf !== 'undefined' && tf.area) || 'projects', projectPath: state.projectPath || '' };
 }
 
+// A pane surface takes over the conversation pane, so the focused conversation
+// has to be released before it opens. Skipping this leaves state.activeId set,
+// and the next poll-driven renderActive() evicts the surface a second later.
+// This is the same defocus focusComposerForDelegate() already performs; only
+// the composer seeding differs.
+function releasePaneForSurface() {
+  if (!state.activeId) return;
+  state.activeId = null;
+  persistActiveConversationSelection(null, tf.area === 'projects' ? { projectPath: state.projectPath || '' } : { scope: tf.area });
+  tf.areaSelection[tfAreaSelectionKey(tf.area)] = 'delegate';
+  if (tf.area === 'company' || tf.area === 'manager') {
+    tf.areaSelection[tf.area] = 'delegate';
+    if (tf.area === 'company') tfRenderCompany();
+    else tfRenderManager();
+  } else {
+    renderRail();
+    renderActive();
+  }
+}
+
+// Issue #1911: Coach's intake is a pane surface registered by
+// coach-intake-surface.js, which loads after this file. Guarded so a failed
+// asset load reports itself instead of throwing a ReferenceError on click.
+function openCoachSurface(actionContext) {
+  if (typeof openCoachIntake !== 'function') {
+    showStatus('The coaching panel is still loading. Reload FRAIM Hub if this persists.', true);
+    return;
+  }
+  const context = actionContext || managementActionContext();
+  // The pane surface host lives inside #area-projects. Switch there first so
+  // the panel is visible regardless of which tab the manager clicked Coach from,
+  // matching the Delegate behavior (focusComposerForDelegate does the same).
+  if (tf.area !== 'projects') tfShowArea('projects');
+  // Coach claims both the pane and the composer, so release both first. An
+  // armed Delegate job would otherwise reappear in the chip the moment Coach
+  // closes (the defect #1657's post-launch fix had to correct twice).
+  hideComposerSuggestions();
+  releasePaneForSurface();
+  if (els['coach-text']) els['coach-text'].value = '';
+  openCoachIntake(context);
+  syncSendButton();
+}
+
+// ---------------------------------------------------------------------------
+// Issue #1911 R16: main-pane surface host integration
+//
+// PaneSurfaces owns the #pane-surface-host slot and the optional composer
+// contract. Everything below is the app side of that contract: which pane
+// element wins, who writes the chip, and where Send goes. No surface-specific
+// knowledge lives here either - this works the same for Coach and for whatever
+// registers next.
+// ---------------------------------------------------------------------------
+
+function paneSurfaceOpen() {
+  return typeof PaneSurfaces !== 'undefined' && !!PaneSurfaces.activeId();
+}
+
+// Binds the generic host to this application's DOM. Every Hub-specific id lives
+// here rather than in pane-surfaces.js, which is what lets that module stay a
+// reusable slot instead of a Coach panel with a registry bolted on.
+function configurePaneSurfaceHost() {
+  if (typeof PaneSurfaces === 'undefined') return;
+  PaneSurfaces.configure({
+    hostSelector: '#pane-surface-host',
+    composerTextSelector: '#coach-text',
+    chipSelector: '#cp-armed-job-chip',
+    chipLabelSelector: '#cp-armed-job-chip .cp-armed-job-label',
+    chipClearSelector: '#cp-armed-job-clear',
+    composerErrorSelector: '#pane-surface-composer-error',
+    onChange: onPaneSurfaceChange,
+  });
+}
+
+function paneSurfaceComposerContract() {
+  return typeof PaneSurfaces === 'undefined' ? null : PaneSurfaces.composerContract();
+}
+
+// Called by PaneSurfaces on every open, close and successful submit. It must
+// never call renderActive(): renderActive() closes surfaces, and the two would
+// re-enter each other.
+function onPaneSurfaceChange() {
+  if (els['empty']) els['empty'].hidden = paneSurfaceOpen() || !!activeConversation();
+  if (paneSurfaceOpen() && els['active-conv']) els['active-conv'].hidden = true;
+  renderArmedJobChip();
+  syncSendButton();
+}
+
 function bindManagementDialog(overlay, close, trigger) {
   overlay.addEventListener('keydown', (event) => {
     if (event.key === 'Escape') { event.preventDefault(); event.stopPropagation(); close(); return; }
@@ -7518,7 +7650,7 @@ function appendManagementActions(rail, area) {
     button.textContent = label;
     if (!existing) button.addEventListener('click', () => {
       if (action === 'delegate') focusComposerForDelegate({ actionContext: managementActionContext(area) });
-      else if (action === 'coach') openTeachJobDialog(managementActionContext(area));
+      else if (action === 'coach') openCoachSurface(managementActionContext(area));
       else openRosterDialog(area);
     });
     actions.appendChild(button);
@@ -7592,209 +7724,14 @@ function teachJobOwnerLabel(job) {
   return persona ? (persona.displayName || key) : key;
 }
 
-function clearTeachFieldErrors() {
-  ['tjd-job-name-error', 'tjd-owner-error', 'tjd-details-error', 'tjd-submit-error'].forEach((id) => {
-    const el = document.getElementById(id);
-    if (el) el.textContent = '';
-  });
-}
-
-function setTeachSelectedMimic(job) {
-  state.teachSelectedMimicJob = job || null;
-  if (job && document.getElementById('tjd-mode')?.value === 'customize') {
-    document.getElementById('tjd-submit-error').textContent = '';
-  }
-  document.querySelectorAll('#tjd-mimic-results .tjd-mimic-row').forEach((row) => {
-    row.classList.toggle('selected', !!job && row.dataset.jobId === job.id);
-  });
-  const selected = document.getElementById('tjd-selected-mimic');
-  const label = document.getElementById('tjd-selected-mimic-label');
-  if (!selected || !label) return;
-  if (!job) {
-    selected.hidden = true;
-    label.textContent = '';
-    return;
-  }
-  const owner = teachJobOwnerLabel(job);
-  const prefix = document.getElementById('tjd-mode').value === 'customize' ? 'Correct' : 'Mimic';
-  label.textContent = `${prefix}: ${job.title || job.id} (${job.id}) · ${owner}`;
-  selected.hidden = false;
-}
-
-function buildTeachMimicRow(job) {
-  const row = document.createElement('button');
-  row.type = 'button';
-  row.className = 'cp-row tjd-mimic-row';
-  row.dataset.jobId = job.id;
-
-  const icon = document.createElement('span');
-  icon.className = 'cp-row-icon';
-  icon.textContent = 'J';
-  row.appendChild(icon);
-
-  const body = document.createElement('span');
-  body.className = 'cp-row-body';
-  const title = document.createElement('span');
-  title.className = 'cp-row-title';
-  title.textContent = job.title || job.id;
-  body.appendChild(title);
-  const sub = document.createElement('span');
-  sub.className = 'cp-row-sub';
-  sub.textContent = job.intent || job.id;
-  body.appendChild(sub);
-  row.appendChild(body);
-
-  const tag = document.createElement('span');
-  tag.className = 'cp-row-tag';
-  tag.textContent = job.requiredPersonaKey || 'general';
-  row.appendChild(tag);
-
-  row.addEventListener('click', () => setTeachSelectedMimic(job));
-  return row;
-}
-
-function renderTeachMimicJobs() {
-  const list = document.getElementById('tjd-mimic-results');
-  const empty = document.getElementById('tjd-mimic-empty');
-  const search = document.getElementById('tjd-mimic-search');
-  if (!list || !empty) return;
-  const q = search ? search.value.trim().toLowerCase() : '';
-  const jobs = teachJobCatalog().filter((job) => {
-    if (!q) return true;
-    const owner = teachJobOwnerLabel(job);
-    return [
-      job.id,
-      job.title,
-      job.intent,
-      job.requiredPersonaKey,
-      owner,
-    ].filter(Boolean).some((value) => String(value).toLowerCase().includes(q));
-  });
-  list.innerHTML = '';
-  jobs.forEach((job) => list.appendChild(buildTeachMimicRow(job)));
-  empty.hidden = jobs.length !== 0;
-  if (state.teachSelectedMimicJob && !jobs.some((job) => job.id === state.teachSelectedMimicJob.id)) {
-    setTeachSelectedMimic(state.teachSelectedMimicJob);
-  } else {
-    setTeachSelectedMimic(state.teachSelectedMimicJob || null);
-  }
-}
-
-function populateTeachOwners() {
-  const owner = document.getElementById('tjd-owner');
-  if (!owner) return;
-  owner.innerHTML = '';
-  const placeholder = document.createElement('option');
-  placeholder.value = '';
-  placeholder.textContent = 'Choose an employee...';
-  owner.appendChild(placeholder);
-  for (const option of teachOwnerOptions()) {
-    const el = document.createElement('option');
-    el.value = option.key;
-    el.textContent = `${option.label} - ${option.role}`;
-    owner.appendChild(el);
-  }
-}
-
-function setTeachMode(mode) {
-  const correcting = mode === 'customize';
-  document.getElementById('tjd-mode').value = correcting ? 'customize' : 'new';
-  document.querySelectorAll('.tjd-mode').forEach((button) => {
-    button.setAttribute('aria-pressed', String(button.dataset.mode === mode));
-  });
-  document.getElementById('tjd-new-fields').hidden = correcting;
-  document.getElementById('tjd-owner').disabled = correcting;
-  const mimic = document.getElementById('tjd-mimic');
-  // Keep keyboard order aligned with the visible mode-specific field order.
-  document.getElementById('tjd-form').insertBefore(mimic, correcting
-    ? document.querySelector('.tjd-details-field')
-    : document.getElementById('tjd-correction-hint'));
-  mimic.open = correcting;
-  document.getElementById('tjd-mimic-toggle').hidden = correcting;
-  document.querySelector('label[for="tjd-mimic-search"]').textContent = correcting ? 'Which job needs correcting?' : 'Mimic an existing job (optional)';
-  document.querySelector('label[for="tjd-details"]').textContent = correcting ? 'What should change?' : 'Details';
-  document.getElementById('tjd-details').placeholder = correcting ? 'Tell Mandy what to change.' : 'What should this job do?';
-  document.getElementById('tjd-correction-hint').hidden = !correcting;
-  document.getElementById('tjd-submit').textContent = correcting ? 'Send correction to Mandy' : 'Delegate to Mandy';
-  document.getElementById('tjd-mimic-empty').textContent = correcting ? 'No matching jobs. Try another search.' : 'No matching jobs. Mandy will still check for duplicates.';
-  clearTeachFieldErrors();
-  setTeachSelectedMimic(null);
-}
-
-function openTeachJobDialog(actionContext) {
-  const modal = document.getElementById('teach-job-modal');
-  const jobName = document.getElementById('tjd-job-name');
-  const owner = document.getElementById('tjd-owner');
-  const details = document.getElementById('tjd-details');
-  const search = document.getElementById('tjd-mimic-search');
-  if (!modal || !jobName || !owner || !details || !search) return;
-  clearTeachFieldErrors();
-  state.teachActionContext = actionContext || managementActionContext();
-  state.teachTrigger = document.activeElement;
-  setTeachMode('new');
-  const area = state.teachActionContext.area;
-  document.getElementById('tjd-scope-note').textContent = area === 'company' ? 'Company jobs' : area === 'manager' ? 'Your jobs' : "This project's jobs";
-  state.teachSelectedMimicJob = null;
-  jobName.value = '';
-  details.value = '';
-  search.value = '';
-  populateTeachOwners();
-  renderTeachMimicJobs();
-  modal.hidden = false;
-  jobName.focus();
-}
-
-function closeTeachJobDialog() {
-  const modal = document.getElementById('teach-job-modal');
-  if (modal) modal.hidden = true;
-  state.teachSelectedMimicJob = null;
-  if (state.teachTrigger && state.teachTrigger.isConnected) state.teachTrigger.focus();
-}
-
-function validateTeachJobDialog() {
-  clearTeachFieldErrors();
-  const jobName = document.getElementById('tjd-job-name');
-  const owner = document.getElementById('tjd-owner');
-  const details = document.getElementById('tjd-details');
-  if (!jobName || !owner || !details) return null;
-  const mode = document.getElementById('tjd-mode').value;
-  if (mode === 'customize' && !state.teachSelectedMimicJob) {
-    document.getElementById('tjd-submit-error').textContent = 'Choose the existing job to improve.';
-    document.getElementById('tjd-mimic-search').focus();
-    return null;
-  }
-  if (mode !== 'customize' && !jobName.value.trim()) {
-    const err = document.getElementById('tjd-job-name-error');
-    if (err) err.textContent = 'Job name is required.';
-    jobName.focus();
-    return null;
-  }
-  if (mode !== 'customize' && !owner.value) {
-    const err = document.getElementById('tjd-owner-error');
-    if (err) err.textContent = 'Choose the employee who should own this job.';
-    owner.focus();
-    return null;
-  }
-  if (!details.value.trim()) {
-    const err = document.getElementById('tjd-details-error');
-    if (err) err.textContent = 'Job details are required.';
-    details.focus();
-    return null;
-  }
-  const ownerKey = mode === 'customize' ? (state.teachSelectedMimicJob.requiredPersonaKey || GENERIC_WORKER_PERSONA.key) : owner.value;
-  const ownerOption = teachOwnerOptions().find((option) => option.key === ownerKey) || {
-    key: ownerKey,
-    label: ownerKey,
-    role: 'AI Employee',
-  };
-  return {
-    mode,
-    jobName: mode === 'customize' ? (state.teachSelectedMimicJob.title || state.teachSelectedMimicJob.id) : jobName.value.trim(),
-    owner: ownerOption,
-    details: details.value.trim(),
-    mimic: state.teachSelectedMimicJob || null,
-  };
-}
+// Issue #1911: the #teach-job-modal intake (clearTeachFieldErrors,
+// setTeachSelectedMimic, buildTeachMimicRow, renderTeachMimicJobs,
+// populateTeachOwners, setTeachMode, openTeachJobDialog, closeTeachJobDialog,
+// validateTeachJobDialog, submitTeachJobDialog) is retired. Coach now renders
+// into the main pane through PaneSurfaces; see coach-intake-surface.js, which
+// reuses teachJobCatalog/teachJobOwnerLabel/teachOwnerOptions and calls the
+// buildTeachIntakeInstructions() below unchanged - the payload and the
+// evolve-employee/Mandy run are the same ones spec #976 established.
 
 function buildTeachIntakeInstructions(intake) {
   const mimic = intake.mimic
@@ -7825,26 +7762,6 @@ function buildTeachIntakeInstructions(intake) {
   ].join('\n');
 }
 
-async function submitTeachJobDialog() {
-  const intake = validateTeachJobDialog();
-  if (!intake) return;
-  const submit = document.getElementById('tjd-submit');
-  if (submit) submit.disabled = true;
-  const employeeId = normalizeConfiguredAgentChoice((state.bootstrap && state.bootstrap.preferences && state.bootstrap.preferences.employeeId) || 'claude');
-  const result = await startRun({
-    id: 'evolve-employee',
-    title: intake.mode === 'customize' ? 'Improve a job' : 'Teach a new job',
-    requiredPersonaKey: 'mandy',
-  }, buildTeachIntakeInstructions(intake), employeeId, undefined, state.teachActionContext?.area, undefined, state.teachActionContext);
-  if (submit) submit.disabled = false;
-  if (result && result.ok === false) {
-    const err = document.getElementById('tjd-submit-error');
-    if (err) err.textContent = result.error || 'Could not start the teaching run.';
-    return;
-  }
-  closeTeachJobDialog();
-}
-
 // Comments follow the artifact (R7.7): a PR opens GitHub; local artifacts open
 // from disk; reports view inline. Nothing new is invented - these surface what
 // the read-side already knows and otherwise leave a status hint.
@@ -8092,6 +8009,9 @@ function wirePopovers() {
     }
 
     if (e.key === 'Escape') {
+      // Issue #1911 R15/R9a: an open pane surface is the innermost thing on
+      // screen, so it cancels first - and its own onEscape can veto.
+      if (typeof PaneSurfaces !== 'undefined' && PaneSurfaces.handleEscape()) return;
       const suggestions = document.getElementById('composer-suggestions');
       const assignModal = document.getElementById('dep-assignment-modal');
       const aomModal = document.getElementById('area-onboard-modal');
@@ -8103,7 +8023,7 @@ function wirePopovers() {
         assignModal.hidden = true;
       } else if (state.cpSelectedJob) {
         // Back out of an armed Delegate Job pick (see armSelectedJob) without
-        // sending. Teach has its own modal (#teach-job-modal), not covered here.
+        // sending. Coach is a pane surface and is handled above this branch.
         clearArmedJob();
       } else {
         closePopovers();
@@ -8130,6 +8050,10 @@ function parseCpSearch(text) {
 // personaKey/prefixSearch are still honored (e.g. a custom employee's own
 // "+" button) by seeding state and the composer text directly.
 function focusComposerForDelegate(opts) {
+  // Issue #1911 edge case: Delegate and Coach both want the pane and the
+  // composer. The second click replaces the first surface rather than layering a
+  // suggestion list over a panel.
+  if (paneSurfaceOpen()) PaneSurfaces.close({ restoreFocus: false });
   state.cpActionContext = (opts && opts.actionContext) || managementActionContext(state.pendingNextJobArea || undefined);
   state.activeId = null;
   persistActiveConversationSelection(null, tf.area === 'projects' ? { projectPath: state.projectPath || '' } : { scope: tf.area });
@@ -8227,13 +8151,21 @@ function renderComposerSuggestions(searchText) {
 
 // Issue #1657 R1 (Treatment 3): the single composer's send action, resolved
 // by context - an idle catalog/ad-hoc delegate, or coaching an
-// already-focused conversation. Teach has its own modal (#teach-job-modal)
-// and never reaches this function. This is the only thing that changes
+// already-focused conversation. A pane surface that claims the composer
+// (issue #1911) intercepts before any of that. This is the only thing that changes
 // across every path; the box itself never does.
 async function sendFromComposer() {
   const submitDelegate = (typeof window !== 'undefined' && typeof window.submitComposerDelegate === 'function')
     ? window.submitComposerDelegate
     : submitComposerDelegate;
+  // Issue #1911 R1: a pane surface that claims the composer takes Send first, so
+  // the detailed instruction typed here is submitted by that surface instead of
+  // being started as an ad-hoc delegate.
+  if (typeof PaneSurfaces !== 'undefined'
+    && await PaneSurfaces.submitFromComposer(els['coach-text'] ? els['coach-text'].value : '')) {
+    syncSendButton();
+    return;
+  }
   if (!state.activeId) {
     // Issue #989: a next-job-recommendation chip can pre-select a job (and
     // pre-fill instructions) without going through the suggestion list at
@@ -8573,7 +8505,8 @@ function renderCpHighlight() {
 // (armSelectedJob) and clears the box for the manager to type real
 // instructions, instead of sending immediately. Ad-hoc remains immediate-fire:
 // there is no separate name/instructions split - the typed text already IS the
-// instructions. Teach is out of this catalog entirely (see #teach-btn/#teach-job-modal).
+// instructions. Coach is out of this catalog entirely (see #teach-btn and the
+// coach-intake pane surface).
 function selectCpRow(index) {
   const row = state.cpRows[index];
   if (!row) return;
@@ -8591,6 +8524,7 @@ function selectCpRow(index) {
 // instructions) and waits for the manager to type real instructions, then
 // send (sendFromComposer's existing state.cpSelectedJob branch handles that).
 function armSelectedJob(job) {
+  if (paneSurfaceOpen()) PaneSurfaces.close({ restoreFocus: false });
   state.cpBrowseAll = false;
   state.cpActionContext = state.cpActionContext || managementActionContext(state.pendingNextJobArea || undefined);
   state.cpSelectedJob = job;
@@ -8615,21 +8549,25 @@ function clearArmedJob() {
 }
 
 // Shows the "Delegating to:" confirmation for a regular armed Delegate Job
-// catalog pick. Teach has its own modal (#teach-job-modal) and never sets
-// state.cpSelectedJob, so this chip needs no Teach-specific branch.
+// catalog pick. Coach never sets state.cpSelectedJob; it writes this chip
+// through the pane-surface composer contract instead.
 function renderArmedJobChip() {
   const chip = document.getElementById('cp-armed-job-chip');
-  const textEl = els['coach-text'];
   if (!chip) return;
+  // Issue #1911: an open pane surface owns the chip while it is up and says what
+  // it is coaching or configuring in its own words.
+  if (typeof PaneSurfaces !== 'undefined' && PaneSurfaces.renderComposerChrome()) return;
+  const label = chip.querySelector('.cp-armed-job-label');
+  const clear = document.getElementById('cp-armed-job-clear');
+  if (clear) clear.setAttribute('aria-label', 'Change job');
   if (state.cpSelectedJob) {
     chip.hidden = false;
-    const label = chip.querySelector('.cp-armed-job-label');
-    if (label) label.textContent = state.cpSelectedJob.title || state.cpSelectedJob.id;
-    syncComposerPlaceholder();
+    if (label) label.textContent = 'Delegating to: ' + (state.cpSelectedJob.title || state.cpSelectedJob.id);
   } else {
     chip.hidden = true;
-    syncComposerPlaceholder();
+    if (label) label.textContent = '';
   }
+  syncComposerPlaceholder();
 }
 
 async function submitComposerDelegate(job, instructionsRaw) {
@@ -9264,6 +9202,7 @@ async function continueRun(text, options) {
           ...(coachingJobId ? { coachingJobId } : {}),
           ...(deliveryIntent ? { deliveryIntent } : {}),
           ...(opts.selectedReviewAction ? { selectedReviewAction: opts.selectedReviewAction } : {}),
+          ...(opts.proposalDecisions ? { proposalDecisions: opts.proposalDecisions } : {}),
           ...(attachmentIds ? { attachmentIds } : {}),
         }),
       });
@@ -9294,6 +9233,7 @@ async function continueRun(text, options) {
             instructions: effectiveText,
             ...(coachingJobId ? { coachingJobId } : {}),
             ...(opts.selectedReviewAction ? { selectedReviewAction: opts.selectedReviewAction } : {}),
+          ...(opts.proposalDecisions ? { proposalDecisions: opts.proposalDecisions } : {}),
           }),
         });
         conv.runId = run.id; // bind the conversation to the freshly-resumed run
@@ -9315,6 +9255,7 @@ async function continueRun(text, options) {
             instructions: effectiveText,
             ...(coachingJobId ? { coachingJobId } : {}),
             ...(opts.selectedReviewAction ? { selectedReviewAction: opts.selectedReviewAction } : {}),
+          ...(opts.proposalDecisions ? { proposalDecisions: opts.proposalDecisions } : {}),
             ...(attachmentIds ? { attachmentIds } : {}),
           }),
         });
@@ -10310,39 +10251,7 @@ function wireEvents() {
   });
   if (els['teach-btn']) {
     els['teach-btn'].addEventListener('click', () => {
-      openTeachJobDialog();
-    });
-  }
-  const teachModal = document.getElementById('teach-job-modal');
-  const teachClose = document.getElementById('tjd-close');
-  const teachCancel = document.getElementById('tjd-cancel');
-  const teachSubmit = document.getElementById('tjd-submit');
-  const teachSearch = document.getElementById('tjd-mimic-search');
-  const teachClear = document.getElementById('tjd-clear-mimic');
-  if (teachClose) teachClose.addEventListener('click', closeTeachJobDialog);
-  if (teachCancel) teachCancel.addEventListener('click', closeTeachJobDialog);
-  if (teachSubmit) teachSubmit.addEventListener('click', submitTeachJobDialog);
-  if (teachSearch) teachSearch.addEventListener('input', renderTeachMimicJobs);
-  if (teachClear) teachClear.addEventListener('click', () => setTeachSelectedMimic(null));
-  document.querySelectorAll('.tjd-mode').forEach((button) => {
-    button.addEventListener('click', () => {
-      if (document.getElementById('tjd-mode').value !== button.dataset.mode) setTeachMode(button.dataset.mode);
-    });
-  });
-  if (teachModal) {
-    bindManagementDialog(teachModal, closeTeachJobDialog);
-    teachModal.addEventListener('click', (event) => {
-      if (event.target === teachModal) closeTeachJobDialog();
-    });
-    teachModal.addEventListener('keydown', (event) => {
-      if (event.key === 'Escape') {
-        closeTeachJobDialog();
-        return;
-      }
-      if (event.key === 'Enter' && event.target?.matches('input:not(#tjd-mimic-search)')) {
-        event.preventDefault();
-        submitTeachJobDialog();
-      }
+      openCoachSurface();
     });
   }
   // Issue #385: hire-required notice back button.
@@ -10442,7 +10351,11 @@ function wireEvents() {
     // which Teach also goes through now), the box holds instructions, not a
     // search query - re-triggering the catalog off that text would pop an
     // irrelevant suggestion list on top of the armed-job chip.
-    if (!state.activeId && !state.cpSelectedJob) {
+    // Issue #1911: while a pane surface owns the composer, this text is the
+    // instruction the surface will submit, not a job search - popping the
+    // catalog list under an open intake panel would squeeze the panel and
+    // offer a delegate the manager did not ask for.
+    if (!state.activeId && !state.cpSelectedJob && !paneSurfaceComposerContract()) {
       renderComposerSuggestions(els['coach-text'].value);
     }
   });
@@ -10458,6 +10371,9 @@ function wireEvents() {
     if (e.key === 'ArrowDown' && !document.getElementById('composer-suggestions').hidden) { e.preventDefault(); moveCpHighlight(+1); return; }
     if (e.key === 'ArrowUp' && !document.getElementById('composer-suggestions').hidden) { e.preventDefault(); moveCpHighlight(-1); return; }
     if (e.key === 'Escape') {
+      // Issue #1911 R15: Escape from inside the composer cancels the pane
+      // surface that owns it, before any Delegate-side arming is considered.
+      if (typeof PaneSurfaces !== 'undefined' && PaneSurfaces.handleEscape()) return;
       // Issue #1657 post-launch fix: an armed job (see armSelectedJob) must be
       // cleared via clearArmedJob (which also resets the chip/placeholder) -
       // hideComposerSuggestions also nulls state.cpSelectedJob but silently,
@@ -10536,7 +10452,15 @@ function wireEvents() {
   // returns this composer to its catalog.
   const armedJobClearBtn = document.getElementById('cp-armed-job-clear');
   if (armedJobClearBtn) {
-    armedJobClearBtn.addEventListener('click', () => clearArmedJob());
+    armedJobClearBtn.addEventListener('click', () => {
+      // Issue #1911 R5: while a pane surface owns the chip, the clear control
+      // cancels that surface instead of un-arming a Delegate job.
+      if (paneSurfaceOpen()) {
+        PaneSurfaces.close();
+        return;
+      }
+      clearArmedJob();
+    });
   }
   // Issue #442: Direct panel send button.
   const abDirectInput = document.getElementById('ab-direct-input');
@@ -10722,6 +10646,7 @@ function wireEvents() {
   // This is a belt-and-suspenders fallback in case the inline script was skipped.
 
   gatherElements();
+  configurePaneSurfaceHost();
   state.conversations = {};
   state.activeId = null;
   wirePopovers();
@@ -14840,11 +14765,32 @@ function tfLearningRunKey(scope) {
   return scope === 'org' ? 'org' : scope === 'reverse' ? 'reverse' : 'manager';
 }
 
-/** True when a conversation's employee turns carry a sleep-on-learnings proposal set. */
+/**
+ * A proposal's identity, and the only thing a decision is ever keyed on.
+ *
+ * Issue #1857 R18: these used to be `scope + '-live-' + index`, positional per
+ * render. A Skip therefore relayed a title, and `move-artifact-between-levels`
+ * says plainly that titles never select updates, so the one handle the Hub gave
+ * the job was the one the corpus forbids using. `proposalId` comes off the run
+ * journal and is what a recorded decision binds to.
+ */
+function tfProposalKey(p) {
+  return p && p.proposalId;
+}
+
+/**
+ * True when this run is carrying a structured proposal set.
+ *
+ * Issue #1857: this used to scan employee turns for a `## Pending Review` heading
+ * and `#### [P-LEVEL]` lines. No job or skill was ever instructed to emit that
+ * format, so the review surface #1755 shipped rendered nothing for a real run, and
+ * the two regexes that read the marker disagreed about whether a trailing date was
+ * allowed. The job now emits `evidence.proposalSet` through seekMentoring and the
+ * Hub stores it on the conversation, so there is one source and nothing to parse.
+ */
 function tfConvHasProposals(conv) {
-  if (!conv || !Array.isArray(conv.messages)) return false;
-  const text = conv.messages.filter((m) => m && m.role === 'employee').map((m) => m.text || '').join('\n');
-  return /pending review/i.test(text) && /####\s*\[?P-(CRITICAL|HIGH|MED|LOW)\]?/i.test(text);
+  return !!(conv && conv.proposalSet && Array.isArray(conv.proposalSet.proposals)
+    && conv.proposalSet.proposals.length > 0);
 }
 
 /**
@@ -14885,47 +14831,22 @@ function tfScopeMeta(scope) {
   return { cls: 'scope-proj', label: 'Project', note: 'Saves to project learnings · this project only' };
 }
 
-// Parse pending proposals out of a sleep-on-learnings run's surfaced text. The
-// job writes a "## Pending Review" section (see sleep-on-learnings.md
-// write-proposals phase); we read the employee turns for proposal headings so
-// what renders here is the job's own output, not a Hub invention.
-function tfParsePendingProposals(conv, scope) {
-  if (!conv) return [];
-  const text = (conv.messages || [])
-    .filter((m) => m.role === 'employee')
-    .map((m) => m.text || '')
-    .join('\n');
-  if (!text || !/pending review/i.test(text)) return [];
-  const out = [];
-  const re = /####\s*\[?P-(CRITICAL|HIGH|MED|LOW)\]?\s*(.+)/gi;
-  let m;
-  let i = 0;
-  while ((m = re.exec(text)) !== null) {
-    const title = m[2].trim();
-    // Issue #1002 R1/R13: the job states a level per proposal and, when it has a
-    // reason, a suggestion to move it. Read both out of the job's own output
-    // rather than assuming: the block after this heading carries them.
-    // Bound the block at the NEXT proposal heading. A fixed-length slice would
-    // read the following proposal's lines and attribute them to this one.
-    const rest = text.slice(m.index + m[0].length);
-    const nextHeading = rest.search(/####\s*\[?P-(?:CRITICAL|HIGH|MED|LOW)\]?/i);
-    const block = nextHeading === -1 ? rest : rest.slice(0, nextHeading);
-    const scopeLine = block.match(/\*\*Scope(?: if approved)?\*\*:\s*(.+)/i);
-    const suggestedLevel = scopeLine ? tfParseLevelWord(scopeLine[1]) : 'manager';
-    const suggestLine = block.match(/\*\*Suggested(?: level)?\*\*:\s*([a-z ]+?)[.:]\s*(.+)/i);
-    out.push({
-      id: scope + '-live-' + (i++),
-      scope,
-      title,
-      body: '',
-      example: '',
-      live: true,
-      suggestedLevel,
-      suggestion: suggestLine ? tfParseLevelWord(suggestLine[1]) : null,
-      suggestionReason: suggestLine ? suggestLine[2].trim() : null,
-    });
-  }
-  return out;
+/**
+ * The proposals this run is waiting on, straight off the structured set.
+ *
+ * Issue #1857 replaced a markdown parse with `evidence.proposalSet`. The scope the
+ * run was asked for still decides which corpus an approval lands in, and it now
+ * arrives on the set itself rather than being re-derived from the kickoff text.
+ */
+function tfPendingProposals(conv) {
+  if (!tfConvHasProposals(conv)) return [];
+  return conv.proposalSet.proposals;
+}
+
+/** Claims this run suppressed because the manager already decided them (R7). */
+function tfSuppressedClaims(conv) {
+  const suppressed = conv && conv.proposalSet && conv.proposalSet.suppressed;
+  return Array.isArray(suppressed) ? suppressed : [];
 }
 
 // Issue #1002 R13: the job states levels in the manager's words ("keep only for
@@ -14947,12 +14868,23 @@ async function tfRelayProposalDecision(scope, proposal, decision) {
   const conv = tfFindLearningRun(scope);
   const verb = decision === 'approve' ? 'Approve as-is' : 'Reject (skip)';
   const message = `${verb} the proposal: "${proposal.title}".`;
-  tfProposalDecisions[proposal.id] = decision;
+  tfProposalDecisions[tfProposalKey(proposal)] = decision;
   if (conv && conv.runId) {
     // Relay into the live run via the existing manager-turn path.
     const prev = state.activeId;
     state.activeId = conv.id;
-    try { await continueRun(message); } finally { state.activeId = prev; }
+    try {
+      // Issue #1857 R18: the decision names the proposal it decided. The prose is
+      // what the manager reads back; `proposalDecisions` is what the job records
+      // against, so it never has to re-derive a revision from a quoted title.
+      await continueRun(message, {
+        proposalDecisions: [{
+          proposalId: tfProposalKey(proposal), revision: proposal.revision,
+          action: decision === 'approve' ? 'approve' : 'reject',
+          level: tfProposalLevel(proposal), reason: tfProposalReason(proposal) || '',
+        }],
+      });
+    } finally { state.activeId = prev; }
   } else if (typeof showStatus === 'function') {
     showStatus(decision === 'approve'
       ? `Blessed “${proposal.title}”. Run Share learnings to let sleep-on-learnings write it.`
@@ -15017,6 +14949,103 @@ async function tfFetchPreservedLearnings(scope, level) {
   }
   return tfPreservedCache()[tfLearnKey(scope, level)];
 }
+/**
+ * Issue #1857 R19: what the manager said no to.
+ *
+ * Rejection is one of the three states a claim can be in, and it was the only one
+ * with no way to see it: approved claims are in the family files the library
+ * renders, pending ones are in the run's conversation, and a rejection lived only
+ * in a markdown file the manager would have to open by hand. Without this a
+ * manager cannot tell a claim that keeps returning from one they already settled,
+ * which is the judgment R8's reopening rule asks them to make.
+ *
+ * `undefined` = not loaded yet, so a first render can say "Loading" rather than
+ * claiming there are no rejections.
+ */
+function tfRejectionsCache() { return state._rejections; }
+async function tfFetchRejections() {
+  const params = new URLSearchParams();
+  if (state.projectPath) params.set('projectPath', state.projectPath);
+  try {
+    const data = await requestJson('/api/ai-hub/rejections?' + params.toString());
+    state._rejections = Array.isArray(data.entries) ? data.entries : [];
+  } catch {
+    state._rejections = [];
+  }
+  return state._rejections;
+}
+
+/**
+ * The collapsed rejected section, in the same shape as the Archived section above
+ * it: a count in the summary and the detail only when opened. Closed by default,
+ * because a refused claim is settled and the library is about what the employee
+ * knows, not what it was told no about.
+ */
+function tfRejectedSection(rejections) {
+  const box = document.createElement('details');
+  box.className = 'learn-dormant-section learn-rejected-section';
+  box.dataset.testid = 'rejected-section';
+  const summary = document.createElement('summary');
+  summary.className = 'learn-dormant-header';
+  summary.textContent = `Rejected (${rejections.length})`;
+  box.appendChild(summary);
+
+  for (const r of rejections) {
+    const card = document.createElement('div');
+    card.className = 'learn-prop lr-card';
+    card.dataset.testid = 'rejected-card';
+
+    const head = document.createElement('div');
+    head.className = 'lp-head';
+    const title = document.createElement('span');
+    title.className = 'lp-title';
+    title.textContent = r.title;
+    head.appendChild(title);
+    // A claim a rule or skill now owns was promoted, not refused. Labelling both
+    // "Rejected" would tell the manager they said no to work they approved.
+    const kind = document.createElement('span');
+    kind.className = 'lp-scope ' + (r.kind === 'superseded' ? 'sc-project' : 'sc-manager');
+    kind.dataset.testid = 'lr-kind';
+    kind.textContent = r.kind === 'superseded' ? 'Superseded' : 'Rejected';
+    head.appendChild(kind);
+    box.appendChild(card);
+    card.appendChild(head);
+
+    const why = document.createElement('div');
+    why.className = 'lp-body';
+    why.textContent = r.reason;
+    card.appendChild(why);
+
+    const meta = document.createElement('div');
+    meta.className = 'lp-stats';
+    if (r.rejectedAt) {
+      const when = document.createElement('span');
+      when.className = 'lp-score';
+      when.dataset.testid = 'lr-when';
+      when.textContent = String(r.rejectedAt).slice(0, 10);
+      meta.appendChild(when);
+    }
+    if (typeof r.recurrencesAtRejection === 'number') {
+      const count = document.createElement('span');
+      count.className = 'lp-score';
+      count.textContent = `${r.recurrencesAtRejection} occurrence${r.recurrencesAtRejection === 1 ? '' : 's'} then`;
+      meta.appendChild(count);
+    }
+    if (meta.children.length) card.appendChild(meta);
+
+    // The manager's own condition for looking at this again. R8 applies it as
+    // written, so showing it here is how they check what they actually asked for.
+    if (r.revisitCondition) {
+      const revisit = document.createElement('div');
+      revisit.className = 'lp-example';
+      revisit.dataset.testid = 'lr-revisit';
+      revisit.textContent = 'Revisit when: ' + r.revisitCondition;
+      card.appendChild(revisit);
+    }
+  }
+  return box;
+}
+
 // Re-render ONLY the surface that owns this (scope, level) after an async fetch or
 // a write — never the whole app. The project level lives in the project workspace;
 // machine/org levels live in their top-tab areas. (A broad re-render here caused
@@ -15711,6 +15740,15 @@ function tfRenderLearningsList(host, scope, level, emptyText) {
     tfFetchPreservedLearnings(scope, level).then(() => tfReRenderScope(scope, level)).catch(() => {});
   }
 
+  // Issue #1857 R19. Rejections are personal to the manager, so only their own
+  // surface asks for them; the company and reverse-mentoring lists are other
+  // people's corpora and a refusal there is not this manager's to review.
+  const showsRejections = scope === 'manager';
+  if (showsRejections && tfRejectionsCache() === undefined) {
+    tfFetchRejections().then(() => tfReRenderScope(scope, level)).catch(() => {});
+  }
+  const rejections = showsRejections && Array.isArray(tfRejectionsCache()) ? tfRejectionsCache() : [];
+
   const addCategories = scope === 'reverse' ? ['coaching']
     : scope === 'org' ? ['avoid', 'preference', 'repeat', 'coaching']
     : ['avoid', 'preference', 'repeat'];
@@ -15815,6 +15853,7 @@ function tfRenderLearningsList(host, scope, level, emptyText) {
 
   if (list.children.length) {
     host.appendChild(list);
+    if (rejections.length && !shareMode) host.appendChild(tfRejectedSection(rejections));
     if (!shareMode) { host.appendChild(addBtn); wireAdd(); }
     if (shareMode) tfUpdateShareBar(host, scope, level);
     return;
@@ -15837,6 +15876,7 @@ function tfRenderLearningsList(host, scope, level, emptyText) {
   empty.appendChild(emoji); empty.appendChild(title); empty.appendChild(sub);
   host.appendChild(empty);
   if (shareMode) { tfUpdateShareBar(host, scope, level); return; }
+  if (rejections.length) host.appendChild(tfRejectedSection(rejections));
   host.appendChild(addBtn);
   wireAdd();
 }
@@ -15872,7 +15912,7 @@ function tfLearningBatchBar(proposals, scope) {
   accept.dataset.testid = 'learn-batch-accept';
   // Do not say "Save all 5" when one of the five is skipped. Count what will
   // actually be written, and name the skips separately.
-  const skipped = proposals.filter((p) => (tfProposalOverrides[p.id] || {}).skipped);
+  const skipped = proposals.filter((p) => (tfProposalOverrides[tfProposalKey(p)] || {}).skipped);
   const saving = proposals.length - skipped.length;
   if (changed.length === 0) {
     accept.textContent = `✓ Keep all ${proposals.length}`;
@@ -15898,7 +15938,7 @@ function tfLearningBatchBar(proposals, scope) {
     if (accept.getAttribute('aria-disabled') === 'true') {
       const first = missingReason[0];
       const field = first
-        ? document.getElementById('lp-why-' + String(first.id).replace(/[^a-zA-Z0-9_-]/g, ''))
+        ? document.getElementById('lp-why-' + String(tfProposalKey(first)).replace(/[^a-zA-Z0-9_-]/g, ''))
         : null;
       if (field) {
         field.scrollIntoView({ block: 'center' });
@@ -15918,7 +15958,7 @@ function tfLearningBatchBar(proposals, scope) {
   skipAll.textContent = 'Skip all';
   skipAll.addEventListener('click', async () => {
     skipAll.disabled = true;
-    for (const p of proposals) tfProposalDecisions[p.id] = 'reject';
+    for (const p of proposals) tfProposalDecisions[tfProposalKey(p)] = 'reject';
     await tfRelayBatch(scope, `Skip all ${proposals.length} proposals.`);
   });
   bar.appendChild(skipAll);
@@ -15931,20 +15971,20 @@ function tfLearningBatchBar(proposals, scope) {
 // wiped by the next render before the manager could save it.
 const tfProposalOverrides = {};
 function tfProposalLevel(p) {
-  const o = tfProposalOverrides[p.id];
+  const o = tfProposalOverrides[tfProposalKey(p)];
   return (o && o.level) || p.suggestedLevel || 'manager';
 }
 function tfProposalReason(p) {
-  const o = tfProposalOverrides[p.id];
+  const o = tfProposalOverrides[tfProposalKey(p)];
   return (o && o.reason) || '';
 }
 function tfProposalChanged(p) {
-  const o = tfProposalOverrides[p.id];
+  const o = tfProposalOverrides[tfProposalKey(p)];
   if (!o) return false;
   return !!o.skipped || (!!o.level && o.level !== (p.suggestedLevel || 'manager'));
 }
 function tfSetProposalOverride(p, patch) {
-  tfProposalOverrides[p.id] = Object.assign({}, tfProposalOverrides[p.id] || {}, patch);
+  tfProposalOverrides[tfProposalKey(p)] = Object.assign({}, tfProposalOverrides[tfProposalKey(p)] || {}, patch);
 }
 
 // Issue #1002 R13: accepting the batch relays ONE turn describing every decision,
@@ -15952,18 +15992,22 @@ function tfSetProposalOverride(p, patch) {
 // the job persists as an unprocessed signal for the next run to read.
 async function tfAcceptLearningBatch(proposals, scope) {
   const lines = [];
+  const decisions = [];
   for (const p of proposals) {
     const level = tfProposalLevel(p);
-    const o = tfProposalOverrides[p.id] || {};
+    const o = tfProposalOverrides[tfProposalKey(p)] || {};
     if (o.skipped) {
-      tfProposalDecisions[p.id] = 'reject';
+      tfProposalDecisions[tfProposalKey(p)] = 'reject';
       lines.push(`- Skip "${p.title}". Reason: ${tfProposalReason(p) || '(none given)'}`);
+      decisions.push({ proposalId: tfProposalKey(p), revision: p.revision, action: 'reject', level, reason: tfProposalReason(p) || '' });
     } else if (tfProposalChanged(p)) {
-      tfProposalDecisions[p.id] = 'promote:' + level;
+      tfProposalDecisions[tfProposalKey(p)] = 'promote:' + level;
       lines.push(`- Save "${p.title}" at ${tfLevelWord(level)}. Reason: ${tfProposalReason(p) || '(none given)'}`);
+      decisions.push({ proposalId: tfProposalKey(p), revision: p.revision, action: 'approve', level, reason: tfProposalReason(p) || '' });
     } else {
-      tfProposalDecisions[p.id] = 'approve';
+      tfProposalDecisions[tfProposalKey(p)] = 'approve';
       lines.push(`- Keep "${p.title}" at ${tfLevelWord(level)}.`);
+      decisions.push({ proposalId: tfProposalKey(p), revision: p.revision, action: 'approve', level, reason: tfProposalReason(p) || '' });
     }
   }
   const msg = [
@@ -15971,7 +16015,10 @@ async function tfAcceptLearningBatch(proposals, scope) {
     ...lines,
     'Apply them all, and save each stated reason as a signal so future runs get this right without asking.',
   ].join('\n');
-  await tfRelayBatch(scope, msg);
+  // Issue #1857 R18. The prose above is what the manager reads back; `decisions`
+  // is what the job records against. Previously only the prose existed, naming
+  // each proposal by title, and the job had to guess which revision that was.
+  await tfRelayBatch(scope, msg, decisions);
 }
 
 function tfLevelWord(level) {
@@ -15982,12 +16029,17 @@ function tfLevelWord(level) {
 
 // Relay one manager turn into the running learning job, through the same
 // continueRun path every other in-conversation control uses. No second write path.
-async function tfRelayBatch(scope, message) {
+async function tfRelayBatch(scope, message, decisions) {
   const conv = tfFindLearningRun(scope);
   if (conv && conv.runId) {
     const prev = state.activeId;
     state.activeId = conv.id;
-    try { await continueRun(message); } finally { state.activeId = prev; }
+    try {
+      // Issue #1857 R18. Carried the same way an approve_delivery selection already
+      // travels: the job records against `proposalId` and `revision` rather than
+      // matching the titles quoted in the sentence the manager reads.
+      await continueRun(message, decisions && decisions.length ? { proposalDecisions: decisions } : undefined);
+    } finally { state.activeId = prev; }
   } else if (typeof showStatus === 'function') {
     showStatus('Run Sleep on learnings first, then your decisions go straight into that run.', false);
   }
@@ -16192,27 +16244,32 @@ async function tfLaunchShareWithOthers(scope, level, selectedTitles, job, projec
 }
 
 function tfProposalCard(p, scope) {
-  const meta = tfScopeMeta(scope);
   const card = document.createElement('div');
   card.className = 'learn-prop';
-  const decided = tfProposalDecisions[p.id];
+  const decided = tfProposalDecisions[p.proposalId];
   if (decided) card.classList.add('resolved');
 
+  // Issue #1857: no scope chip. It showed the run's scope while the level buttons
+  // below showed the proposal's, so a proposal suggested at Project inside a
+  // Manager-scoped run was chipped MANAGER and footed "this project only". The
+  // buttons already carry the level; two controls for one fact is how they came
+  // to disagree. Invisible before this change because a parsed proposal could not
+  // express a level that differed from its run.
   const head = document.createElement('div');
   head.className = 'lp-head';
-  const sc = document.createElement('span');
-  sc.className = 'lp-scope ' + meta.cls;
-  sc.textContent = meta.label;
   const title = document.createElement('span');
   title.className = 'lp-title';
   title.textContent = p.title;
-  head.appendChild(sc); head.appendChild(title);
+  head.appendChild(title);
   card.appendChild(head);
 
-  if (p.body) {
+  // The takeaway and the example. Both were authored by the job and both were
+  // thrown away by the parse, which hardcoded them empty, so until now a manager
+  // approved or rejected a lesson on its title alone.
+  if (p.takeaway) {
     const body = document.createElement('div');
     body.className = 'lp-body';
-    body.textContent = p.body;
+    body.textContent = p.takeaway;
     card.appendChild(body);
   }
   if (p.example) {
@@ -16222,6 +16279,43 @@ function tfProposalCard(p, scope) {
     card.appendChild(ex);
   }
 
+  // A reopening says so, in the manager's own words, with both counts. Without
+  // this a claim they already rejected would return looking like a new one.
+  if (p.reopening) {
+    const reopen = document.createElement('div');
+    reopen.className = 'lp-reopen';
+    reopen.dataset.testid = 'lp-reopen';
+    reopen.textContent = `You rejected this on ${p.reopening.rejectedAt}: "${p.reopening.reason}". `
+      + `It was at ${p.reopening.countAtRejection} occurrence`
+      + `${p.reopening.countAtRejection === 1 ? '' : 's'} then and is at ${p.recurrences} now.`;
+    card.appendChild(reopen);
+  }
+
+  // Evidence, using the library's own chips. The trajectory is what makes
+  // "leave it for now" a position rather than a deferred chore.
+  const stats = document.createElement('div');
+  stats.className = 'lp-stats';
+  if (p.domain) {
+    const domain = document.createElement('span');
+    domain.className = 'lp-domain';
+    domain.textContent = p.domain;
+    stats.appendChild(domain);
+  }
+  const occurrences = document.createElement('span');
+  occurrences.className = 'lp-score';
+  occurrences.dataset.testid = 'lp-occurrences';
+  occurrences.textContent = typeof p.recurrencesAtLastReview === 'number'
+    ? `${p.recurrences} occurrences, up from ${p.recurrencesAtLastReview}`
+    : `${p.recurrences} occurrence${p.recurrences === 1 ? '' : 's'}`;
+  stats.appendChild(occurrences);
+  const kind = document.createElement('span');
+  kind.className = 'lp-strength';
+  kind.textContent = p.changeType === 'update' ? 'Updates an existing lesson'
+    : p.changeType === 'cold' ? 'Revives a dormant lesson'
+      : 'New lesson';
+  stats.appendChild(kind);
+  card.appendChild(stats);
+
   // Issue #1755 R6: a proposal card reads as two zones: what you can still change
   // before deciding, and the decision itself. They used to sit on one undivided
   // row, so a level button and a Skip button looked like the same kind of control.
@@ -16258,14 +16352,18 @@ function tfProposalCard(p, scope) {
 
   // The recommendation, when the job made one, with its reason. R6: absent a
   // reason there is no nudge, so the row simply states where it will be saved.
-  if (p.suggestion && p.suggestionReason) {
+  // #1857: the level is `suggestedLevel` on the wire and is always present, so the
+  // reason alone decides whether there is a nudge to show. The old parse carried a
+  // separate `suggestion` field that was set only when the job named a level in
+  // prose, which made the nudge depend on how the sentence happened to be written.
+  if (p.suggestionReason) {
     const why = document.createElement('div');
     why.className = 'lp-reason';
     const b = document.createElement('b');
     // Name the suggested level with the SAME words the picker below uses. Reading
     // "Suggested: Project" above a button that says "This project" makes the
     // manager check whether those are the same thing.
-    b.textContent = 'Suggested: ' + (TF_LEVEL_PICK_LABEL[p.suggestion] || tfScopeMeta(p.suggestion).label || p.suggestion) + '.';
+    b.textContent = 'Suggested: ' + (TF_LEVEL_PICK_LABEL[suggested] || tfScopeMeta(suggested).label || suggested) + '.';
     why.appendChild(b);
     why.appendChild(document.createTextNode(' ' + p.suggestionReason));
     card.appendChild(why);
@@ -16282,7 +16380,7 @@ function tfProposalCard(p, scope) {
   // A skipped lesson is saved at no level, so no level is selected on that row.
   // Showing "Skipped" next to a highlighted level says two contradictory things
   // about the same row. The buttons stay live: picking a level clears the skip.
-  const isSkipped = !!(tfProposalOverrides[p.id] || {}).skipped;
+  const isSkipped = !!(tfProposalOverrides[p.proposalId] || {}).skipped;
   if (isSkipped) pick.classList.add('lp-moot');
   for (const [level, label] of LEVELS) {
     const b = document.createElement('button');
@@ -16312,13 +16410,13 @@ function tfProposalCard(p, scope) {
   decide.dataset.testid = 'lp-decide';
 
   const skip = document.createElement('button');
-  skip.className = 'lp-skip' + ((tfProposalOverrides[p.id] || {}).skipped || decided === 'reject' ? ' on' : '');
+  skip.className = 'lp-skip' + ((tfProposalOverrides[p.proposalId] || {}).skipped || decided === 'reject' ? ' on' : '');
   skip.type = 'button';
   skip.dataset.testid = 'lp-skip';
-  skip.textContent = (tfProposalOverrides[p.id] || {}).skipped || decided === 'reject' ? '✕ Skipped' : 'Skip';
+  skip.textContent = (tfProposalOverrides[p.proposalId] || {}).skipped || decided === 'reject' ? '✕ Skipped' : 'Skip';
   skip.disabled = !!decided;
   skip.addEventListener('click', () => {
-    const wasSkipped = !!(tfProposalOverrides[p.id] || {}).skipped;
+    const wasSkipped = !!(tfProposalOverrides[p.proposalId] || {}).skipped;
     tfSetProposalOverride(p, { skipped: !wasSkipped });
     reRender();
   });
@@ -16330,10 +16428,10 @@ function tfProposalCard(p, scope) {
   if (tfProposalChanged(p) && !decided) {
     const why = document.createElement('div');
     why.className = 'lp-why';
-    const id = 'lp-why-' + String(p.id).replace(/[^a-zA-Z0-9_-]/g, '');
+    const id = 'lp-why-' + String(p.proposalId).replace(/[^a-zA-Z0-9_-]/g, '');
     const label = document.createElement('label');
     label.setAttribute('for', id);
-    label.textContent = (tfProposalOverrides[p.id] || {}).skipped
+    label.textContent = (tfProposalOverrides[p.proposalId] || {}).skipped
       ? 'Why skip it?'
       : 'Why this level instead?';
     const ta = document.createElement('textarea');
@@ -16368,7 +16466,7 @@ function tfProposalCard(p, scope) {
   // R8: state what the decision will actually write, in the zone where it is made.
   const note = document.createElement('span');
   note.className = 'lp-scope-note';
-  note.textContent = (tfProposalOverrides[p.id] || {}).skipped
+  note.textContent = (tfProposalOverrides[p.proposalId] || {}).skipped
     ? 'Nothing is written. The skip is recorded as a signal.'
     : (TF_LEVEL_TITLE[currentLevel] || tfScopeMeta(scope).note);
   decide.appendChild(note);
@@ -16382,7 +16480,7 @@ async function tfPromoteProposal(originScope, proposal, targetScope) {
   const label = targetScope === 'org' ? 'Company' : targetScope === 'manager' ? 'Manager' : 'Project';
   const conv = tfFindLearningRun(originScope);
   const message = `Promote the proposal "${proposal.title}" to ${label.toLowerCase()} learnings (scope: ${targetScope}).`;
-  tfProposalDecisions[proposal.id] = 'promote:' + targetScope;
+  tfProposalDecisions[tfProposalKey(proposal)] = 'promote:' + targetScope;
   if (conv && conv.runId) {
     const prev = state.activeId;
     state.activeId = conv.id;
@@ -19016,4 +19114,9 @@ if (typeof window !== 'undefined') {
   // tests can assert the rendered controls without depending on click coordinates.
   window.openInlineCreateEmployee = openInlineCreateEmployee;
   window.submitComposerDelegate = submitComposerDelegate;
+  // Issue #1911: the pane-surface seams, so the suite can drive the host and
+  // the Coach entry point without clicking through three different rails.
+  window.openCoachSurface = openCoachSurface;
+  window.paneSurfaceOpen = paneSurfaceOpen;
+  window.buildTeachIntakeInstructions = buildTeachIntakeInstructions;
 }