fraim-hub 2.0.210 → 2.0.211

package/dist/src/ai-hub/conversation-store.js

@@ -211,7 +211,7 @@ function normalizeConversation(projectPath, raw) {
         return null;
     if (typeof value.jobId !== 'string')
         return null;
-    if (value.agentName !== 'codex' && value.agentName !== 'claude' && value.agentName !== 'gemini')
+    if (typeof value.agentName !== 'string' || value.agentName.trim().length === 0)
         return null;
     if (value.status !== 'running' && value.status !== 'completed' && value.status !== 'failed')
         return null;
@@ -301,11 +301,8 @@ class AiHubConversationStore {
     lockPath(bucketDir) {
         return path_1.default.join(bucketDir, '.lock');
     }
-    // ---- migration (one-time split of the legacy monolith) ----
-    ensureMigrated() {
-        if (this.migrated)
-            return;
-        this.migrated = true;
+    // ---- migration (split of the legacy monolith) ----
+    importLegacyMonolithIfPresent() {
         let stat;
         try {
             stat = fs_1.default.statSync(this.stateFilePath);
@@ -338,6 +335,17 @@ class AiHubConversationStore {
             /* best-effort backup; another process may have already migrated */
         }
     }
+    ensureMigrated() {
+        if (!this.migrated) {
+            this.migrated = true;
+            this.importLegacyMonolithIfPresent();
+            return;
+        }
+        // Compatibility for tests and old processes that still recreate the legacy
+        // monolith after this store has already been opened. Shards remain the
+        // source of truth; a newly observed monolith is imported and moved aside.
+        this.importLegacyMonolithIfPresent();
+    }
     // ---- bucket read helpers (lock-free) ----
     readAllConversations(bucketDir, bucketKey) {
         let files;

package/dist/src/ai-hub/desktop-main.js

@@ -14,6 +14,7 @@ const remote_hub_gateway_1 = require("./remote-hub-gateway");
 const project_fraim_paths_1 = require("../core/utils/project-fraim-paths");
 const version_utils_1 = require("../cli/utils/version-utils");
 const hub_runtime_file_1 = require("./hub-runtime-file");
+const window_open_decision_1 = require("./window-open-decision");
 // ---------------------------------------------------------------------------
 // State
 // ---------------------------------------------------------------------------
@@ -54,66 +55,6 @@ function applyUserDataOverride() {
     fs_1.default.mkdirSync(userDataDir, { recursive: true });
     electron_1.app.setPath('userData', userDataDir);
 }
-function isTrustedInAppNavigation(targetUrl, hubUrl) {
-    try {
-        const parsed = new URL(targetUrl);
-        if (parsed.protocol !== 'http:' && parsed.protocol !== 'https:')
-            return false;
-        return parsed.origin === new URL(hubUrl).origin || parsed.origin === new URL((0, remote_hub_gateway_1.resolveFraimRemoteUrl)()).origin;
-    }
-    catch {
-        return false;
-    }
-}
-function connectedSurfaceParam(value) {
-    return value === 'account' || value === 'analytics' || value === 'brain' ? value : null;
-}
-function connectedSurfaceForHostedUrl(targetUrl, hubUrl) {
-    try {
-        const parsed = new URL(targetUrl);
-        const hub = new URL(hubUrl);
-        const remote = new URL((0, remote_hub_gateway_1.resolveFraimRemoteUrl)());
-        if (parsed.origin === hub.origin || parsed.origin !== remote.origin)
-            return null;
-        const pathname = parsed.pathname.replace(/\/+$/, '') || '/';
-        if (pathname === '/account')
-            return 'account';
-        if (pathname === '/analytics')
-            return 'analytics';
-        if (pathname === '/fraim-brain' || pathname === '/fraim-brain.html')
-            return 'brain';
-        if (pathname === '/ai-hub') {
-            return connectedSurfaceParam(parsed.searchParams.get('connected') || parsed.searchParams.get('open'));
-        }
-        if (pathname === '/auth/sign-in.html' || pathname === '/auth/recovery.html' || pathname === '/auth/error.html') {
-            return connectedSurfaceParam(parsed.searchParams.get('surface'));
-        }
-        return null;
-    }
-    catch {
-        return null;
-    }
-}
-function localHubConnectedUrl(hubUrl, surface, sourceUrl) {
-    const target = new URL(hubUrl);
-    target.search = '';
-    target.hash = '';
-    target.searchParams.set('connected', surface);
-    if (sourceUrl) {
-        try {
-            const source = new URL(sourceUrl);
-            const apiKey = source.searchParams.get('api-key') || source.searchParams.get('apiKey');
-            if (apiKey)
-                target.searchParams.set('api-key', apiKey);
-        }
-        catch { /* ignore malformed source URLs */ }
-    }
-    return target.toString();
-}
-function wrappedHostedNavigationUrl(targetUrl, hubUrl) {
-    const surface = connectedSurfaceForHostedUrl(targetUrl, hubUrl);
-    return surface ? localHubConnectedUrl(hubUrl, surface, targetUrl) : null;
-}
 // ---------------------------------------------------------------------------
 // Tray icon resolution — prefers bundled icon, falls back to a 1×1 empty image
 // so the app never crashes if assets aren't present.
@@ -281,18 +222,22 @@ async function createWindow(url) {
         mainWindow.on('closed', () => electron_1.nativeTheme.removeListener('updated', applyOverlay));
     }
     mainWindow.webContents.setWindowOpenHandler(({ url: targetUrl }) => {
-        const wrappedUrl = wrappedHostedNavigationUrl(targetUrl, url);
-        if (wrappedUrl) {
-            void mainWindow?.loadURL(wrappedUrl);
-            return { action: 'deny' };
+        const decision = (0, window_open_decision_1.resolveWindowOpenDecision)(targetUrl, url, (0, remote_hub_gateway_1.resolveFraimRemoteUrl)());
+        if (decision.action === 'load-in-app') {
+            void mainWindow?.loadURL(decision.url);
         }
-        if (isTrustedInAppNavigation(targetUrl, url)) {
-            void mainWindow?.loadURL(targetUrl);
+        else if (decision.action === 'open-external') {
+            // e.g. the "Open PR on GitHub" review action — hand it to the OS default
+            // browser (issue #835). A bare target="_blank" anchor would otherwise be
+            // swallowed here.
+            void electron_1.shell.openExternal(decision.url);
         }
+        // A new BrowserWindow is never opened; we navigate the Hub window in-app or
+        // defer to the OS browser, so the popup request itself is always denied.
         return { action: 'deny' };
     });
     mainWindow.webContents.on('will-navigate', (event, targetUrl) => {
-        const wrappedUrl = wrappedHostedNavigationUrl(targetUrl, url);
+        const wrappedUrl = (0, window_open_decision_1.wrappedHostedNavigationUrl)(targetUrl, url, (0, remote_hub_gateway_1.resolveFraimRemoteUrl)());
         if (!wrappedUrl)
             return;
         event.preventDefault();
@@ -303,7 +248,7 @@ async function createWindow(url) {
     mainWindow.webContents.on('will-redirect', (details) => {
         if (!details.isMainFrame)
             return;
-        const wrappedUrl = wrappedHostedNavigationUrl(details.url, url);
+        const wrappedUrl = (0, window_open_decision_1.wrappedHostedNavigationUrl)(details.url, url, (0, remote_hub_gateway_1.resolveFraimRemoteUrl)());
         if (!wrappedUrl)
             return;
         details.preventDefault();
@@ -312,7 +257,7 @@ async function createWindow(url) {
         }
     });
     mainWindow.webContents.on('did-navigate', (_event, targetUrl) => {
-        const wrappedUrl = wrappedHostedNavigationUrl(targetUrl, url);
+        const wrappedUrl = (0, window_open_decision_1.wrappedHostedNavigationUrl)(targetUrl, url, (0, remote_hub_gateway_1.resolveFraimRemoteUrl)());
         if (!wrappedUrl || mainWindow?.webContents.getURL() === wrappedUrl)
             return;
         void mainWindow?.loadURL(wrappedUrl);

package/dist/src/ai-hub/server.js

@@ -98,6 +98,16 @@ function buildReviewApprovalSystemEventText(instructions) {
         return 'review_approved merge_pr_work_completion';
     if (/^Approved and clean up branch\.$/i.test(text))
         return 'review_approved cleanup_branch';
+    if (/^Approved and send\.$/i.test(text))
+        return 'review_approved send_delivery';
+    if (/^Approved and send, then push to\b/i.test(text))
+        return 'review_approved send_delivery push_default_branch';
+    if (/^Approved and send, then merge PR\.$/i.test(text))
+        return 'review_approved send_delivery merge_pr';
+    if (/^Approved and send, then merge PR and complete issue\.$/i.test(text))
+        return 'review_approved send_delivery merge_pr_work_completion';
+    if (/^Approved and send, then clean up branch\.$/i.test(text))
+        return 'review_approved send_delivery cleanup_branch';
     return null;
 }
 function loadManagerHiringModule() {

package/dist/src/ai-hub/window-open-decision.js

@@ -0,0 +1,86 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.wrappedHostedNavigationUrl = wrappedHostedNavigationUrl;
+exports.resolveWindowOpenDecision = resolveWindowOpenDecision;
+const url_safety_1 = require("./url-safety");
+function isTrustedInAppNavigation(targetUrl, hubUrl, remoteUrl) {
+    try {
+        const parsed = new URL(targetUrl);
+        if (parsed.protocol !== 'http:' && parsed.protocol !== 'https:')
+            return false;
+        return parsed.origin === new URL(hubUrl).origin || parsed.origin === new URL(remoteUrl).origin;
+    }
+    catch {
+        return false;
+    }
+}
+function connectedSurfaceParam(value) {
+    return value === 'account' || value === 'analytics' || value === 'brain' ? value : null;
+}
+function connectedSurfaceForHostedUrl(targetUrl, hubUrl, remoteUrl) {
+    try {
+        const parsed = new URL(targetUrl);
+        const hub = new URL(hubUrl);
+        const remote = new URL(remoteUrl);
+        if (parsed.origin === hub.origin || parsed.origin !== remote.origin)
+            return null;
+        const pathname = parsed.pathname.replace(/\/+$/, '') || '/';
+        if (pathname === '/account')
+            return 'account';
+        if (pathname === '/analytics')
+            return 'analytics';
+        if (pathname === '/fraim-brain' || pathname === '/fraim-brain.html')
+            return 'brain';
+        if (pathname === '/ai-hub') {
+            return connectedSurfaceParam(parsed.searchParams.get('connected') || parsed.searchParams.get('open'));
+        }
+        if (pathname === '/auth/sign-in.html' || pathname === '/auth/recovery.html' || pathname === '/auth/error.html') {
+            return connectedSurfaceParam(parsed.searchParams.get('surface'));
+        }
+        return null;
+    }
+    catch {
+        return null;
+    }
+}
+function localHubConnectedUrl(hubUrl, surface, sourceUrl) {
+    const target = new URL(hubUrl);
+    target.search = '';
+    target.hash = '';
+    target.searchParams.set('connected', surface);
+    if (sourceUrl) {
+        try {
+            const source = new URL(sourceUrl);
+            const apiKey = source.searchParams.get('api-key') || source.searchParams.get('apiKey');
+            if (apiKey)
+                target.searchParams.set('api-key', apiKey);
+        }
+        catch { /* ignore malformed source URLs */ }
+    }
+    return target.toString();
+}
+function wrappedHostedNavigationUrl(targetUrl, hubUrl, remoteUrl) {
+    const surface = connectedSurfaceForHostedUrl(targetUrl, hubUrl, remoteUrl);
+    return surface ? localHubConnectedUrl(hubUrl, surface, targetUrl) : null;
+}
+// Route a window-open / new-window request. Precedence:
+//   1. A hosted FRAIM surface (account/analytics/brain) is wrapped back into the
+//      Hub shell and loaded in-app.
+//   2. A same-origin (Hub or FRAIM remote) URL is loaded in-app.
+//   3. Any other http/https URL (e.g. a GitHub/GitLab/ADO pull-request link from
+//      the review handoff) is opened in the OS default browser. Without this
+//      branch the desktop shell silently denies the click (issue #835).
+//   4. Anything else (javascript:, file:, data:, malformed, ...) is denied —
+//      safeHttpUrl is the same http/https gate the client and server review
+//      handoff use, so nothing unsafe can reach shell.openExternal.
+function resolveWindowOpenDecision(targetUrl, hubUrl, remoteUrl) {
+    const wrappedUrl = wrappedHostedNavigationUrl(targetUrl, hubUrl, remoteUrl);
+    if (wrappedUrl)
+        return { action: 'load-in-app', url: wrappedUrl };
+    if (isTrustedInAppNavigation(targetUrl, hubUrl, remoteUrl))
+        return { action: 'load-in-app', url: targetUrl };
+    const external = (0, url_safety_1.safeHttpUrl)(targetUrl);
+    if (external)
+        return { action: 'open-external', url: external };
+    return { action: 'deny' };
+}

package/dist/src/local-mcp-server/learning-context-builder.js

@@ -385,13 +385,11 @@ function buildLearningContextSection(workspaceRoot, userId, forJob, domain) {
         coach: { ft: 'manager-coaching', gated: false, label: '(manager-facing; all entries)' },
         validated: { ft: 'validated-patterns', gated: true, label: '(entries above score threshold)' },
     };
-    // Domain axis (issue #806, Option 1): `fraim_connect` (forJob=false) loads only
-    // the interaction-preference files + the L0 nudge, since it runs with no job.
-    // A job (forJob=true) loads every category globally, plus the current job's
-    // domain files. Category order within a tier is preserved from prior behavior;
-    // domain files are appended after the global files.
-    const l2Cats = forJob ? ['mistake', 'pref', 'coach', 'validated'] : ['pref'];
-    const l1Cats = forJob ? ['pref', 'coach', 'mistake', 'validated'] : ['pref'];
+    // Domain axis (issue #806): startup and job contexts both include the global
+    // learning set. Job contexts may additionally append the current job's domain
+    // files below the global files.
+    const l2Cats = ['mistake', 'pref', 'coach', 'validated'];
+    const l1Cats = ['pref', 'coach', 'mistake', 'validated'];
     const activeDomain = forJob && domain ? domain : null;
     const dormantOf = (meta, filePath) => meta.gated ? scanMistakePatternFile(filePath, threshold, meta.ft).dormant : 0;
     // Resolve one tier (L2 org or L1 personal) into ordered global + domain files.

package/dist/src/services/email-service.js

@@ -4,7 +4,7 @@ exports.EmailService = void 0;
 const resend_1 = require("resend");
 class EmailService {
     constructor() {
-        const apiKey = process.env.RESEND_API_KEY;
+        const apiKey = process.env.RESEND_API_KEY || (process.env.FRAIM_SUPPRESS_EMAILS === 'true' ? 'test-resend-key' : undefined);
         if (!apiKey) {
             throw new Error('RESEND_API_KEY environment variable is required');
         }
@@ -13,16 +13,20 @@ class EmailService {
         this.fromEmail = process.env.RESEND_FROM_EMAIL || 'FRAIM <onboarding@resend.dev>';
         this.baseUrl = (process.env.BASE_URL || 'https://fraimworks.ai').replace(/\/$/, '');
     }
+    static shouldSuppressEmail() {
+        return process.env.FRAIM_SUPPRESS_EMAILS === 'true'
+            || (process.env.NODE_ENV === 'test' && process.env.ALLOW_REAL_EMAILS_IN_TEST !== 'true');
+    }
     /** Wrapper around resend.emails.send — no-ops in test mode to avoid burning real API quota. */
     async sendEmail(payload) {
-        if (process.env.NODE_ENV === 'test') {
+        if (EmailService.shouldSuppressEmail()) {
             console.log(`[TEST] Email suppressed — would send to ${Array.isArray(payload.to) ? payload.to.join(', ') : payload.to}: "${payload.subject}"`);
             return { data: { id: 'test-suppressed' }, error: null, headers: null };
         }
         return this.sendWithResend(payload);
     }
     async sendWithResend(payload) {
-        if (process.env.NODE_ENV === 'test' && process.env.ALLOW_REAL_EMAILS_IN_TEST !== 'true') {
+        if (EmailService.shouldSuppressEmail()) {
             console.log(`[TEST] Email suppressed - would send to ${Array.isArray(payload.to) ? payload.to.join(', ') : payload.to}: "${payload.subject}"`);
             return { data: { id: 'test-suppressed' }, error: null, headers: null };
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim-hub",
-  "version": "2.0.210",
+  "version": "2.0.211",
   "description": "FRAIM Hub local companion package.",
   "main": "index.js",
   "bin": {
@@ -90,7 +90,7 @@
     "dotenv": "^16.4.7",
     "electron": "^41.2.2",
     "express": "^5.2.1",
-    "fraim": "2.0.210",
+    "fraim": "2.0.211",
     "mongodb": "^7.0.0",
     "node-cron": "4.2.1",
     "node-edge-tts": "^1.2.10",

package/public/ai-hub/script.js

@@ -3472,7 +3472,6 @@ function convAwaitingReview(conv) {
   if (conv.jobId === '__freeform__') return false;
   const handoff = reviewHandoffForConversation(conv);
   if (handoff) return handoff.reviewRequired === true;
-  if (reviewHandoffIssueForConversation(conv)) return true;
   // Delegation orchestration runs (fully-delegate with an active ledger) only enter review
   // via an explicit review_handoff — Mandy's delegation artifacts are orchestration working
   // output, not deliverables submitted to the human manager. Other manager jobs (e.g.,
@@ -3716,47 +3715,6 @@ function delegationLedgerForConversation(conv) {
   return null;
 }
 
-function hasReviewHandoffSignal(text) {
-  return !!(text && /reviewRequired|reviewTarget|review_handoff/i.test(text));
-}
-
-function reviewSubmissionClaimText(text) {
-  return String(stripMarkdownForDisplay(text) || '')
-    .replace(/\s+/g, ' ')
-    .trim();
-}
-
-function hasExplicitReviewSubmissionClaim(text) {
-  const cleaned = reviewSubmissionClaimText(text);
-  if (!cleaned) return false;
-  if (/\b(?:not\s+ready|not\s+yet\s+ready|will\s+be\s+ready|soon\s+be\s+ready)\b.{0,60}\breview\b/i.test(cleaned)) {
-    return false;
-  }
-  return /\bready\s+for\s+(?:your\s+)?review\b/i.test(cleaned)
-    || /\bplease\s+review\b/i.test(cleaned)
-    || /\bsubmitted\s+(?:the\s+|this\s+|my\s+)?(?:work|deliverable|artifact|bundle|pr|pull request|document|doc|file|files)\s+for\s+(?:your\s+)?review\b/i.test(cleaned)
-    || /\b(?:review|open)\s+(?:the\s+)?(?:pr|pull request|artifact|document|doc|file|files|deliverable|bundle)\b/i.test(cleaned);
-}
-
-function reviewHandoffIssueForConversation(conv) {
-  if (!conv) return '';
-  if (conv.reviewHandoff && !normalizeReviewHandoff(conv.reviewHandoff)) {
-    return 'The review handoff contract is invalid.';
-  }
-  const messages = conv.messages || [];
-  const hasRunArtifacts = Array.isArray(conv.artifacts) && conv.artifacts.length > 0;
-  const latestEmployee = [...messages].reverse().find((message) => message.role === 'employee');
-  // #770 R3: only the latest employee turn can raise a malformed-contract issue;
-  // a stale malformed tag from an earlier, superseded turn must not re-latch it.
-  if (latestEmployee && hasReviewHandoffSignal(latestEmployee.text) && !extractReviewHandoffFromText(latestEmployee.text)) {
-    return 'The review handoff contract is malformed or missing required fields.';
-  }
-  if (!conv.reviewHandoff && !hasRunArtifacts && latestEmployee && hasExplicitReviewSubmissionClaim(latestEmployee.text)) {
-    return 'This run says it is ready for review, but did not include a structured review handoff.';
-  }
-  return '';
-}
-
 function reviewHandoffForConversation(conv) {
   if (!conv) return null;
   // Authoritative source: the live server run snapshot, refreshed every poll by
@@ -3891,20 +3849,11 @@ function pullRequestActionLabel(url) {
   return 'Open pull request';
 }
 
-// Derive the deliverable format from the job/deliverable type (R7.2). Returns a
-// format key + a label + the action descriptors to render in the artifact strip.
+// Derive the deliverable format from the job/deliverable type (R7.2). Returns
+// metadata used by the review card and deliverables panel.
 function deriveDeliverableFormat(conv) {
   const structured = deriveFormatFromReviewHandoff(reviewHandoffForConversation(conv));
   if (structured) return structured;
-  const handoffIssue = reviewHandoffIssueForConversation(conv);
-  if (handoffIssue) {
-    return {
-      key: 'contract_error',
-      label: 'invalid review handoff contract',
-      actions: [{ id: 'resend-review-contract', label: 'Ask for corrected contract', primary: true, kind: 'resend-contract' }],
-      issue: handoffIssue,
-    };
-  }
 
   const hay = ((conv && (conv.jobId || '')) + ' ' + (conv && (conv.jobTitle || ''))).toLowerCase();
   // Prefer an explicit artifact path captured from the run when present.
@@ -4113,23 +4062,8 @@ function renderReviewExperience(conv) {
 
   // --- Format-derived artifact strip (R7.2) ---
   // #770 A1: real deliverables (artifact_set / pull_request / deck / report /
-  // markdown) now render in the Deliverables panel (renderDeliverablesPanel), not
-  // pinned in the thread. Only the contract_error correction prompt stays inline
-  // with the review card, since it is a decision-blocking ask, not a deliverable.
-  if (fmt.key === 'contract_error' && fmt.actions && fmt.actions.length) {
-    const strip = document.createElement('div');
-    strip.className = 'rc-artifact-strip';
-    for (const action of fmt.actions) {
-      const btn = document.createElement('button');
-      btn.type = 'button';
-      btn.className = 'rc-art-btn' + (action.primary ? '' : ' ghost');
-      btn.textContent = action.label;
-      btn.dataset.artAction = action.id;
-      btn.addEventListener('click', () => handleArtifactAction(conv, action));
-      strip.appendChild(btn);
-    }
-    host.appendChild(strip);
-  }
+  // markdown) render in the Deliverables panel (renderDeliverablesPanel), not
+  // pinned in the thread.
 
   // --- Unified action bar: review actions appear beside the always-on chips ---
   actions.hidden = false;
@@ -4137,9 +4071,7 @@ function renderReviewExperience(conv) {
   const handoff = els['review-handoff'];
   if (handoff) {
     const hasArtifact = !!(fmt.actions && fmt.actions.length);
-    handoff.textContent = fmt.key === 'contract_error'
-      ? `${fmt.issue || 'The review handoff contract is invalid.'} Ask the employee to resend it before review.`
-      : fmt.key === 'pull_request'
+    handoff.textContent = fmt.key === 'pull_request'
         ? 'Comment on the PR to leave inline notes, or approve / type changes below.'
         : isManagerTemplateJob(conv && conv.jobId)
           ? 'Open the updated files, then approve or type changes below.'
@@ -4239,6 +4171,15 @@ function approvalMenuLabel(action) {
   return label.replace(/^Approve\s*\+\s*/i, 'Approve and ');
 }
 
+function approvalCommandFromLabel(action) {
+  const label = approvalMenuLabel(action)
+    .replace(/^Approve\s*,\s*/i, '')
+    .replace(/^Approve\s+and\s+/i, '')
+    .replace(/^Approve\s+/i, '')
+    .trim();
+  return label ? `Approved and ${label}.` : APPROVAL_MESSAGE;
+}
+
 // #770 A1: single-run deliverables live in a stable collapsible panel in the
 // support stack, decoupled from the transient approve/reject decision UI. Fed by
 // the same format derivation as the old in-thread strip, so a genuine structured
@@ -4583,6 +4524,7 @@ function refreshStatusSurfaces() {
 }
 
 const APPROVAL_MESSAGE = 'Approved.';
+const DEFAULT_REQUEST_CHANGES_MESSAGE = 'Request changes. Please revise the submitted work and send it back for review.';
 
 function buildApprovalDeliveryPreparationMessage(reviewAction) {
   if (!reviewAction || reviewAction.kind !== 'approve_delivery' || !reviewAction.deliveryActionId) {
@@ -4601,8 +4543,7 @@ function buildApprovalDeliveryPreparationMessage(reviewAction) {
   if (reviewAction.deliveryActionId === 'cleanup_branch') {
     return 'Approved and clean up branch.';
   }
-  const label = approvalMenuLabel(reviewAction).replace(/^Approve\s+and\s+/i, '');
-  return label ? `Approved and ${label}.` : APPROVAL_MESSAGE;
+  return approvalCommandFromLabel(reviewAction);
 }
 
 // Approve closes the review gate. From the dedicated review approval button,
@@ -4646,20 +4587,18 @@ async function approveReview(options) {
 
 // Request changes sends the manager's note as a coaching turn that RESUMES the
 // run via the existing continueRun path. The note follows the artifact: a typed
-// comment becomes a coaching message (R7.7). Empty note → focus the input.
+// comment becomes a coaching message (R7.7). Empty note sends a plain revision
+// request so the visible Request changes button always advances the review flow.
 async function requestChangesReview() {
   const conv = activeConversation();
   if (!conv) return;
-  const text = (els['coach-text'] && els['coach-text'].value.trim()) || '';
-  if (!text) {
-    showStatus('Type what to change below, then Request changes resumes the run with your note.', false);
-    if (els['coach-text']) els['coach-text'].focus();
-    return;
-  }
+  const typedText = (els['coach-text'] && els['coach-text'].value.trim()) || '';
+  const text = typedText || DEFAULT_REQUEST_CHANGES_MESSAGE;
   // Re-open the review gate: a change request means the run is no longer "done".
   conv.reviewApproved = false;
-  els['coach-text'].value = '';
+  if (els['coach-text']) els['coach-text'].value = '';
   syncSendButton();
+  showStatus(typedText ? 'Change request sent.' : 'Default change request sent.', false);
   await continueRun(text);
 }