shakers-cli 0.15.0 → 0.17.0

package/src/mcp-signup-tools.js

@@ -13,7 +13,7 @@ const { askAfterTexts, askConsent, optionFor, queueNotice, takeNotices } = requi
 const { expandHome, tildify } = require('./env-paths');
 const { NOOP_FUNNEL, weeklyHours, rateTypes, workSituationValue } = require('./onboarding-funnel');
 
-// MCP sign-up (ticket 39540): hub imports into an UNREGISTERED profile that the local window claims with a code kept only in memory.
+// MCP sign-up: hub imports into an UNREGISTERED profile that the local window claims with a code kept only in memory.
 
 const SIGNUP_WINDOW_TIMEOUT_MS = 15 * 60 * 1000;
 // The browser opens this long after the tool answers, so the AI's reply is on screen first.
@@ -302,6 +302,7 @@ function makeSignupTools(deps = {}) {
     pollDeviceToken = require('./device-login').pollDeviceToken,
     finalizeDeviceSession = require('./device-login').finalizeDeviceSession,
     appendProviderParam = require('./device-login').appendProviderParam,
+    deviceSessionRecord = require('./device-login').deviceSessionRecord,
     buildRegistrationContext = require('./signup-client').buildRegistrationContext,
     runLoopbackAuth = require('./mcp-auth-form').runLoopbackAuth,
     checkOnboardingCompleted = require('./onboarding-availability').checkOnboardingCompleted,
@@ -337,6 +338,7 @@ function makeSignupTools(deps = {}) {
     wake: null,
     importStartedAt: null,
     importReported: new Set(),
+    authAttempts: undefined,
   };
   currentSignup = state;
 
@@ -344,8 +346,12 @@ function makeSignupTools(deps = {}) {
     const before = state.account.state;
     state.account = next;
     if ((next.state === 'created' || next.state === 'existing') && before !== next.state) {
-      funnel.completed('account_creation', { auth_method: next.via, is_returning_account: next.state === 'existing' });
-      funnel.identified();
+      funnel.completed('account_creation', { auth_method: next.via, auth_attempts: state.authAttempts, is_returning_account: next.state === 'existing' });
+      funnel.identified({ newAccount: next.state === 'created' });
+    }
+    if (next.state === 'failed' && before !== 'failed') {
+      funnel.errored('account_creation', next.reason);
+      funnel.flush();
     }
     if (state.wake) state.wake();
   }
@@ -385,7 +391,7 @@ function makeSignupTools(deps = {}) {
       }
       const email = res.email || emailFromHubToken(fin.token);
       try {
-        resolved.saveAuthSession({ accessToken: fin.token, hubAccessToken: fin.token, email });
+        resolved.saveAuthSession(deviceSessionRecord({ token: fin.token, sessionToken: res.accessToken, sessionExpiresIn: res.expiresIn, email }));
       } catch {
         setAccount({ state: 'failed', reason: 'session-persist-failed' });
         return;
@@ -434,6 +440,7 @@ function makeSignupTools(deps = {}) {
         onUrl: (url) => { state.windowUrl = url; announce(url); },
       },
     ).then((outcome) => {
+      if (outcome && outcome.ok) state.authAttempts = outcome.attempts;
       if (outcome && outcome.viaSocial) return;
       if (!outcome || !outcome.ok) {
         setAccount({ state: 'failed', reason: (outcome && outcome.reason) || 'window-failed' });
@@ -482,6 +489,7 @@ function makeSignupTools(deps = {}) {
     // rides on the next question block, because models skip a relay that comes before another tool call.
     if (!nonEmpty(args.answer) && state.cvPath) {
       funnel.completed('start');
+      funnel.viewed('import_sources', { sources_offered: DRAFT_SOURCES });
       queueNotice(c.welcome(nonEmpty(args.firstName)));
       return {
         ok: true,
@@ -500,6 +508,7 @@ function makeSignupTools(deps = {}) {
     });
     if (asked.reply) return { ...asked.reply, language: lang() };
     funnel.completed('start');
+    funnel.viewed('import_sources', { sources_offered: DRAFT_SOURCES });
     // A CV path the talent gave wins over both answers: searching could read another candidate.
     const cvPath = state.cvPath;
     if (cvPath) {
@@ -576,6 +585,7 @@ function makeSignupTools(deps = {}) {
       return { ok: false, reason: 'draft-sources-required', message: "The draft needs sources: 'cv' if you read their CV, 'linkedin' only if you have their LinkedIn URL. Then call signup_draft again." };
     }
     if (!state.firstDraft) state.firstDraft = draft;
+    trackDraftShown(draft);
     const asked = await askAfterTexts(ctx, {
       tool: 'signup_draft',
       texts: [renderDraft(lang(), draft), legalCopy(lang()).signupNotice(sources), knownAccount() ? c.windowNoticeSignIn : c.windowNotice],
@@ -659,10 +669,10 @@ function makeSignupTools(deps = {}) {
       const attempted = DRAFT_SOURCES.filter((source) => (source === 'cv' ? !!cv : !!sources.linkedinUrl));
       state.importStartedAt = Object.fromEntries(attempted.map((source) => [source, now()]));
       for (const source of attempted) funnel.track('source_import_started', 'import_sources', { source_type: source });
-      funnel.completed('import_sources', { sources_attempted: attempted });
+      funnel.completed('import_sources', { sources_attempted: attempted, sources_connected: attempted, sources_connected_count: attempted.length });
     }
     if (!reused) {
-      funnel.viewed('account_creation');
+      funnel.viewed('account_creation', { auth_methods_offered: require('./mcp-auth-form').WINDOW_METHODS.register });
       await openWindow(identity);
     }
     return {
@@ -708,7 +718,7 @@ function makeSignupTools(deps = {}) {
 
   function nextSteps(account) {
     const interview = 'Then the onboarding interview: if onboardingInterview.completed is true, skip it without mentioning it; otherwise call onboarding_interview_start without disclaimerAcknowledged (it asks here now or later on the web). Then, also if they skip it, the main role: list_my_roles (waitSeconds 45), then set_main_role without clusterId only when its next says so. Finally open_web.';
-    const aiUsage = 'Then ai_usage without consent: it returns the two AI-usage disclaimers and the question (show them right before asking; no scan happens before a yes).';
+    const aiUsage = 'Then ai_usage without consent: it returns the two AI-usage disclaimers and the question (show them right before asking; no scan happens before a yes). After a yes, ai_usage_upload (waitSeconds 30) shows the talent what would be sent and asks; nothing is uploaded without their pick.';
     if (account.state === 'existing') {
       return `This talent already had an account and signed in. Call update_existing_profile: it shows what would change and asks whether to update, and its text tells the talent about the account, so say nothing about it yourself. ${aiUsage} ${interview}`;
     }
@@ -951,6 +961,21 @@ function makeSignupTools(deps = {}) {
     return c.ratesSaved(shown, !(state.draft && state.draft.ratesFromTalent));
   }
 
+  function trackDraftShown(draft) {
+    const prefilled = (step) => ({ is_prefilled: STEP_DRAFT_FIELDS[step].some((key) => draft[key] !== undefined) });
+    funnel.viewed('work_situation', prefilled('work_situation'));
+    funnel.viewed('availability', prefilled('availability'));
+    funnel.viewed('rates', { suggested_rate_shown: draft.ratesFromTalent !== true });
+    funnel.viewed('languages', prefilled('languages'));
+    funnel.viewed('location', prefilled('location'));
+    funnel.viewed('profile_summary');
+  }
+
+  function draftFieldsEdited() {
+    if (!state.firstDraft || !state.draft) return undefined;
+    return DRAFT_FIELDS.filter((key) => String(state.firstDraft[key] ?? '') !== String(state.draft[key] ?? '')).length;
+  }
+
   function wasEdited(step) {
     if (!state.firstDraft || !state.draft) return undefined;
     return STEP_DRAFT_FIELDS[step].some((key) => String(state.firstDraft[key] ?? '') !== String(state.draft[key] ?? ''));
@@ -958,7 +983,8 @@ function makeSignupTools(deps = {}) {
 
   function trackSavedSteps(details, results) {
     const saved = (section) => results[section] && results[section].ok === true;
-    const step = (name, props) => funnel.completed(name, { ...props, is_prefilled: true, was_edited: wasEdited(name) });
+    const draftTime = funnel.stepTime('profile_summary');
+    const step = (name, props) => funnel.completed(name, { ...props, is_prefilled: true, was_edited: wasEdited(name), time_on_step_ms: draftTime });
     if (saved('workSituation')) {
       step('work_situation', { work_situation: workSituationValue(String(details.workSituation.situation || '').trim().toUpperCase()) });
     }
@@ -976,12 +1002,16 @@ function makeSignupTools(deps = {}) {
     }
     if (saved('pricing')) {
       const types = rateTypes(pricingFromRates(details.pricing).input);
-      funnel.completed('rates', { rate_types_selected: types, rate_type: types[0], rate_currency: 'EUR', is_prefilled: !(state.draft && state.draft.ratesFromTalent), was_edited: wasEdited('rates') });
+      funnel.completed('rates', { rate_types_selected: types, rate_type: types[0], rate_currency: 'EUR', is_prefilled: !(state.draft && state.draft.ratesFromTalent), was_edited: wasEdited('rates'), time_on_step_ms: draftTime });
     }
     if (saved('languages')) {
       const codes = details.languages.map((row) => String(row.language || '').trim().toLowerCase()).filter(Boolean);
       step('languages', { languages_count: codes.length, languages: codes });
     }
+    const sections = Object.entries(results).filter(([name]) => name !== 'phone');
+    if (sections.length && sections.every(([, r]) => r.ok === true)) {
+      funnel.completed('profile_summary', { fields_edited_count: draftFieldsEdited(), time_on_step_ms: draftTime });
+    }
     funnel.flush();
   }
 

package/src/memory-structure-detector.js

@@ -4,7 +4,7 @@ const fs = require('fs');
 const path = require('path');
 const { getHomeDir } = require('./env-paths');
 
-// Deterministic (no-LLM) memory STRUCTURE detector (talents-ai-score, issue 016 / ADR-013-014).
+// Deterministic (no-LLM) memory STRUCTURE detector.
 
 const MAX_FILE_BYTES = 2_000_000; // safety cap: never read a pathologically huge file
 const MAX_IMPORT_DEPTH = 5; // mirrors Claude Code's own documented import-hop limit

package/src/model-call-record.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// What happened with each model call, written down (talents-ai-score, issue 109).
+// What happened with each model call, written down.
 
 // The three keys.
 const MODEL_CALL = Object.freeze({
@@ -163,7 +163,7 @@ function callPartial(report, key) {
   return !!c && c.ok === true && Array.isArray(c.omitted) && c.omitted.length > 0;
 }
 
-// THE CALL LOST TIME TO A DEADLINE — and this is NOT `reason === TIMEOUT` (issue 116, found by measuring, not by reading).
+// THE CALL LOST TIME TO A DEADLINE — and this is NOT `reason === TIMEOUT` (found by measuring, not by reading).
 function callTimedOut(report, key) {
   const c = modelCall(report, key);
   if (!c || c.ok !== false) return false;

package/src/onboarding-flow.js

@@ -168,7 +168,7 @@ function normalizeAmount(value) {
   return cents / 100;
 }
 
-// Pricing input (ADR-024).
+// Pricing input.
 function validatePricing(input = {}) {
   const fullSelected = input.fullTimeSelected === true;
   const partSelected = input.partTimeSelected === true;
@@ -212,7 +212,7 @@ function buildPricingUpsert({ fullSelected, fullAmount, fullCurrency, partSelect
   };
 }
 
-// Availability + location (ADR-028).
+// Availability + location.
 function validateAvailability(input = {}) {
   const body = { confirmed: true };
   if (typeof input.available === 'boolean') body.available = input.available;
@@ -299,7 +299,6 @@ function makeDeps(overrides = {}) {
     runMainRoleStep: require('./roles-flow').runOnboardingMainRoleStep,
     makeRolesDeps: require('./roles-flow').makeRolesDeps,
     requestDiscoveredInventory: inventory.requestDiscoveredInventory,
-    makeInterviewClient: (opts) => new (require('./interview-client').InterviewClient)(opts),
     // Used by reattachConsentIdentity (A) to re-anchor the ingest identity.
     recordConsent: require('./share-consent-state').recordConsent,
     ...overrides,
@@ -371,7 +370,7 @@ async function establishSessionFromCredentials(deps, { email, password } = {}) {
 
 // Register a talent via a social provider (device flow, RFC 8628); a live session results either way.
 async function registerWithSocialAccount(deps, { provider = 'google', freelanceType, language } = {}, { openBrowser, onAuthUrl } = {}) {
-  const { appendProviderParam } = require('./device-login');
+  const { appendProviderParam, deviceSessionRecord } = require('./device-login');
   const authorizeEndpoint = deps.getDeviceAuthorizeEndpoint();
   const tokenEndpoint = deps.getDeviceTokenEndpoint();
   const authTokenEndpoint = typeof deps.getAuthTokenEndpoint === 'function' ? deps.getAuthTokenEndpoint() : null;
@@ -397,13 +396,8 @@ async function registerWithSocialAccount(deps, { provider = 'google', freelanceT
   );
   if (!result.ok) return { ok: false, reason: result.reason || 'register-failed' };
 
-  // Persist the API JWT the exchange minted (no cookie), like the retired Google flow did.
   try {
-    deps.saveAuthSession({
-      accessToken: result.token,
-      hubAccessToken: result.token,
-      email: result.email || null,
-    });
+    deps.saveAuthSession(deviceSessionRecord(result));
   } catch {
     return { ok: false, reason: 'session-persist-failed' };
   }
@@ -675,18 +669,15 @@ function trackImport(funnel, attempted, importRes, durationMs) {
   const bySource = new Map();
   const report = importRes.ok && importRes.report ? importRes.report.sources : [];
   for (const src of report) bySource.set(sourceTypeFor(src.source), src);
-  const connected = [];
   for (const type of attempted) {
     const src = bySource.get(type);
     const ok = importRes.ok && (!src || src.status === 'imported');
     if (ok) {
-      connected.push(type);
       funnel.track('source_import_completed', 'import_sources', { source_type: type, duration_ms: durationMs });
     } else {
       funnel.track('source_import_failed', 'import_sources', { source_type: type, duration_ms: durationMs, failure_reason: (src && src.code) || importRes.reason });
     }
   }
-  return connected;
 }
 
 // Blocking orchestrator used by the CLI (`bin/onboarding.js`).
@@ -695,9 +686,16 @@ async function runOnboarding(io, deps = makeDeps(), { method = 'email', openBrow
   const c = getCatalog(io.lang).onboarding;
 
   funnel.begin();
+  funnel.viewed('start');
   funnel.completed('start');
   io.section(c.signupTitle);
-  funnel.viewed('account_creation');
+  funnel.viewed('account_creation', { auth_methods_offered: require('./auth-methods').methodIdsForFlow('register') });
+  funnel.flush();
+  const accountFailed = (reason) => {
+    funnel.errored('account_creation', reason);
+    funnel.flush();
+    return { ok: false, reason, step: 'signup' };
+  };
 
   if (method === 'google' || method === 'linkedin') {
     // The provider supplies name/email; freelanceType is derived later from work situation.
@@ -709,10 +707,10 @@ async function runOnboarding(io, deps = makeDeps(), { method = 'email', openBrow
     );
     if (!reg.ok) {
       io.error(c.signupError);
-      return { ok: false, reason: reg.reason, step: 'signup' };
+      return accountFailed(reg.reason);
     }
-    funnel.completed('account_creation', { auth_method: method, is_returning_account: reg.kind === 'account-already-exists' });
-    funnel.identified();
+    funnel.completed('account_creation', { auth_method: method, auth_attempts: 1, is_returning_account: reg.kind === 'account-already-exists' });
+    funnel.identified({ newAccount: reg.kind !== 'account-already-exists' });
     if (reg.kind === 'account-already-exists') {
       // Existing account: already signed in — stop, no wizard re-run.
       io.warn(c.accountExists);
@@ -721,39 +719,39 @@ async function runOnboarding(io, deps = makeDeps(), { method = 'email', openBrow
     notifyOk(io, c.accountReady);
   } else {
     const account = await io.askSignUp(c);
-    if (!account || !account.email) return { ok: false, reason: 'missing-fields', step: 'signup' };
+    if (!account || !account.email) return accountFailed('missing-fields');
     if (!isAcceptablePassword(account.password)) {
       io.error(c.weakPassword);
-      return { ok: false, reason: 'weak-password', step: 'signup' };
+      return accountFailed('weak-password');
     }
 
     const su = await signUp(deps, { ...account, preferredLanguage: account.preferredLanguage || io.lang });
     if (!su.ok) {
       io.error(su.message || c.signupError);
-      return { ok: false, reason: su.reason, step: 'signup' };
+      return accountFailed(su.reason);
     }
 
     if (su.accountExists) {
       // Existing account: sign them in, then STOP — no wizard re-run.
       io.warn(c.accountExists);
       const logged = await io.login(account.email);
-      if (!logged) return { ok: false, reason: 'login-failed', step: 'signup' };
-      funnel.completed('account_creation', { auth_method: 'email', is_returning_account: true });
-      funnel.identified();
+      if (!logged) return accountFailed('login-failed');
+      funnel.completed('account_creation', { auth_method: 'email', auth_attempts: 1, is_returning_account: true });
+      funnel.identified({ newAccount: false });
       return { ok: true, alreadyRegistered: true };
     }
     const sess = await establishSessionFromCredentials(deps, { email: account.email, password: account.password });
     if (!sess.ok) {
       io.error(c.sessionError);
-      return { ok: false, reason: sess.reason, step: 'signup' };
+      return accountFailed(sess.reason);
     }
-    funnel.completed('account_creation', { auth_method: 'email', is_returning_account: false });
+    funnel.completed('account_creation', { auth_method: 'email', auth_attempts: 1, is_returning_account: false });
     funnel.identified();
     notifyOk(io, c.accountReady);
   }
 
   io.section(c.title);
-  funnel.viewed('import_sources');
+  funnel.viewed('import_sources', { sources_offered: ['linkedin', 'cv', 'github', 'web_page'] });
   const linkedinUrl = (await io.ask(c.askLinkedin) || '').trim();
   if (!linkedinUrl) {
     funnel.errored('import_sources', 'no-linkedin', { error_field: 'linkedin' });
@@ -772,15 +770,15 @@ async function runOnboarding(io, deps = makeDeps(), { method = 'email', openBrow
   const importStartedAt = Date.now();
   const importRes = await withProgress(c.importChecking, () =>
     startProfileImport(deps, { linkedinUrl, cvPath, githubUrl, websiteUrl, language: io.lang }));
-  const connected = trackImport(funnel, attempted, importRes, Date.now() - importStartedAt);
-  funnel.completed('import_sources', { sources_attempted: attempted, sources_connected: connected, sources_connected_count: connected.length });
+  trackImport(funnel, attempted, importRes, Date.now() - importStartedAt);
+  funnel.completed('import_sources', { sources_attempted: attempted, sources_connected: attempted, sources_connected_count: attempted.length });
   funnel.flush();
   if (importRes.ok) notifyOk(io, c.importDone);
   else io.warn(c.importUnavailable);
 
   // Steps 4-8 are saved out of funnel order: sent together once all are answered.
   io.section(c.professionalTitle);
-  funnel.viewed('work_situation');
+  funnel.viewed('work_situation', { is_prefilled: false });
   const workSituationInput = await io.askWorkSituation(c);
   const workSituationRes = await saveWorkSituation(deps, workSituationInput);
   if (!workSituationRes.ok) {
@@ -788,11 +786,11 @@ async function runOnboarding(io, deps = makeDeps(), { method = 'email', openBrow
     funnel.flush();
     return { ok: false, reason: workSituationRes.reason, step: 'work-situation' };
   }
-  funnel.completed('work_situation', { work_situation: workSituationValue(String(workSituationInput.situation || '').trim().toUpperCase()) });
+  funnel.completed('work_situation', { work_situation: workSituationValue(String(workSituationInput.situation || '').trim().toUpperCase()), was_edited: true });
   notifyOk(io, c.professionalSaved);
 
   io.section(c.pricingTitle);
-  funnel.viewed('rates');
+  funnel.viewed('rates', { suggested_rate_shown: false });
   const pricingInput = await io.askPricing(c);
   const priceRes = await savePricing(deps, pricingInput);
   if (!priceRes.ok) {
@@ -805,23 +803,31 @@ async function runOnboarding(io, deps = makeDeps(), { method = 'email', openBrow
     rate_types_selected: selectedRates,
     rate_type: selectedRates[0],
     rate_currency: normalizeCurrency(pricingInput.partTimeSelected === true ? pricingInput.partTimeCurrency : pricingInput.fullTimeCurrency) || undefined,
+    was_edited: true,
   });
   notifyOk(io, c.pricingSaved);
 
   // Optional/skippable and fail-soft: a decline or a failed write never aborts the alta.
   io.section(c.availabilityTitle);
-  funnel.viewed('availability');
+  funnel.viewed('availability', { is_prefilled: false });
   if (await io.confirm(c.availabilityAsk)) {
-    const availabilityInput = await io.askAvailability(c);
+    let availabilityTime;
+    const availabilityInput = await io.askAvailability(c, {
+      onLocation: () => {
+        availabilityTime = funnel.stepTime('availability');
+        funnel.viewed('location', { is_prefilled: false });
+      },
+    });
     const availRes = await saveAvailability(deps, availabilityInput);
     if (availRes.ok) {
-      funnel.completed('availability', { availability_hours_per_week: weeklyHours(normalizeMonthlyHours(availabilityInput.monthlyHours)) });
+      funnel.completed('availability', { availability_hours_per_week: weeklyHours(normalizeMonthlyHours(availabilityInput.monthlyHours)), time_on_step_ms: availabilityTime, was_edited: true });
       const country = normalizeCountry(availabilityInput.country);
       const modes = Array.isArray(availabilityInput.workModes) ? availabilityInput.workModes.map((m) => String(m).trim().toUpperCase()) : [];
       funnel.completed('location', {
         location_country: country ? country.toLowerCase() : undefined,
         location_method: 'manual',
         is_remote_only: modes.length ? modes.every((m) => m === 'REMOTE') : undefined,
+        was_edited: true,
       });
       notifyOk(io, c.availabilitySaved);
     } else {
@@ -839,13 +845,13 @@ async function runOnboarding(io, deps = makeDeps(), { method = 'email', openBrow
   }
 
   io.section(c.languagesTitle);
-  funnel.viewed('languages');
+  funnel.viewed('languages', { is_prefilled: false });
   if (await io.confirm(c.languagesAsk)) {
     const languageRows = await io.askLanguages(c);
     const langRes = await saveLanguages(deps, languageRows);
     if (langRes.ok) {
       const codes = validateLanguageRows(languageRows).rows.map((r) => r.language);
-      funnel.completed('languages', { languages_count: codes.length, languages: codes });
+      funnel.completed('languages', { languages_count: codes.length, languages: codes, was_edited: true });
       notifyOk(io, c.languagesSaved);
     } else {
       funnel.errored('languages', langRes.reason);
@@ -888,20 +894,35 @@ async function runOnboarding(io, deps = makeDeps(), { method = 'email', openBrow
   // Every ending points the Talent at their profile, and opens it when a browser is at hand.
   const showProfile = () => {
     const url = profileUrl(deps);
+    funnel.viewed('profile_summary');
     notifyOk(io, url ? c.profileReadyUrl(url) : c.profileReadyNoUrl);
     if (url && typeof openProfile === 'function') {
       try { openProfile(url); } catch { /* the printed link covers it */ }
     }
+    funnel.completed('profile_summary');
+    funnel.flush();
     return url;
   };
 
   const mainRole = async () => {
-    const role = await runMainRoleFinalStep(io, deps);
-    if (role && role.ok && role.chosen) funnel.completed('supercluster', { supercluster: role.chosen });
-    else if (role && role.ok) funnel.skipped('supercluster');
+    let shown = null;
+    const role = await runMainRoleFinalStep(io, deps, {
+      onRolesShown: (info) => {
+        shown = info;
+        funnel.viewed('supercluster', { suggested_superclusters: info.suggested, suggestion_source: 'assigned_clusters', suggestions_count: info.suggested.length });
+      },
+    });
+    if (role && role.ok && role.chosen) {
+      funnel.completed('supercluster', {
+        supercluster: role.chosen,
+        clusters_count: shown ? shown.clustersCount : undefined,
+        was_suggestion_accepted: shown ? shown.suggested.includes(role.chosen) : undefined,
+      });
+    } else if (role && role.ok) funnel.skipped('supercluster');
     funnel.flush();
   };
 
+  funnel.holdUntil('profile_summary');
   io.section(c.interviewTitle);
   funnel.viewed('pre_interview');
   io.disclaimer(c.interviewInfoAccessed, c.interviewGoalDuration);
@@ -1010,14 +1031,14 @@ async function conductLivekitInterview(io, deps, { interviewId = null, language
 }
 
 // Runs the final main-role step, degrading softly (never aborts the flow / never throws).
-async function runMainRoleFinalStep(io, deps, session = null) {
-  const active = session || (typeof deps.loadAuthSession === 'function' ? deps.loadAuthSession() : null);
+async function runMainRoleFinalStep(io, deps, { onRolesShown } = {}) {
+  const active = typeof deps.loadAuthSession === 'function' ? deps.loadAuthSession() : null;
   if (typeof deps.runMainRoleStep !== 'function' || !active) return;
   const { getCatalog } = require('./i18n');
   const catalog = getCatalog(io.lang);
   try {
     const rolesDeps = typeof deps.makeRolesDeps === 'function' ? deps.makeRolesDeps() : undefined;
-    return await deps.runMainRoleStep({ io, session: active, catalog, deps: rolesDeps });
+    return await deps.runMainRoleStep({ io, session: active, catalog, deps: rolesDeps, onRolesShown });
   } catch {
     return null;
   }