shakers-cli-beta 0.11.0 → 0.13.0

package/src/mcp-signup-tools.js

@@ -8,7 +8,7 @@ const { signupCopy, legalCopy, renderDraft, amount } = require('./signup-copy');
 const { fixSignupLanguage, signupLanguage } = require('./signup-language');
 const { readLocalCv } = require('./cv-file');
 const { emailFromHubToken } = require('./auth-session-store');
-const { askAfterTexts, queueNotice, takeNotices } = require('./mcp-choice');
+const { askAfterTexts, optionFor, queueNotice, takeNotices } = require('./mcp-choice');
 
 // MCP sign-up (ticket 39540): hub imports into an UNREGISTERED profile that the local window claims with a code kept only in memory.
 
@@ -36,10 +36,32 @@ const SECTION_HELP = {
   'bad-opinion': 'workSituation.opinion is required unless situation is FREELANCE: pick your best match from its enum.',
   'bad-motivation': 'workSituation.changeMotivators is required when situation is FREELANCE or opinion is WAS_FREELANCE_BEFORE or OPEN_TO_FREELANCE: pick your best match from its enum.',
 };
+// What the closing words may say about the profile import, by the state read at that moment.
+const IMPORT_CLOSING = {
+  done: 'The profile import finished: if you mention it, say it is done, never that it is pending.',
+  running: 'The profile import is still running: if you mention it, say it finishes on its own in a few minutes.',
+  failed: 'Say nothing more about the profile import than the fixed lines already shown.',
+  none: 'Say nothing about the state of the profile import.',
+};
 const VERBATIM = 'Print relayVerbatim word for word now, as its own block, before any other tool call, with nothing of your own added to it.';
 
+// Providers that serve the same personal mailboxes under many country domains (hotmail.co.uk, yahoo.com.br, outlook.it).
+const PERSONAL_EMAIL_PROVIDERS = new Set(['gmail', 'googlemail', 'outlook', 'hotmail', 'live', 'msn', 'yahoo', 'ymail', 'icloud', 'proton', 'protonmail', 'gmx', 'aol', 'zoho', 'tutanota', 'libero', 'virgilio', 'tiscali', 'sapo', 'yandex', 'fastmail', 'uol', 'bol', 'terra']);
+// Reserved for documentation and tests (RFC 2606, RFC 6761): never anybody's employer.
+const RESERVED_EMAIL_DOMAIN_RE = /(^|\.)(example\.(com|net|org)|example|test|invalid|localhost)$/;
+
+function emailDomain(email) {
+  return String(email).split('@').pop().toLowerCase();
+}
+
 function isPersonalEmail(email) {
-  return PERSONAL_EMAIL_DOMAINS.has(String(email).split('@').pop().toLowerCase());
+  const domain = emailDomain(email);
+  return PERSONAL_EMAIL_DOMAINS.has(domain) || PERSONAL_EMAIL_PROVIDERS.has(domain.split('.')[0]);
+}
+
+// Only an address that looks like a company one gets the hint to prefer a personal email.
+function looksLikeWorkEmail(email) {
+  return !isPersonalEmail(email) && !RESERVED_EMAIL_DOMAIN_RE.test(emailDomain(email));
 }
 
 function isLinkedinProfileUrl(url) {
@@ -134,7 +156,8 @@ const WELCOME_SCHEMA = {
     language: { type: 'string', description: "ISO code of the language the talent wrote their first message in (es, en, it, pt...). It fixes the language of every sign-up text from here on." },
     firstName: { type: 'string', description: 'Only if you already know it: the welcome greets them by name.' },
     answer: { type: 'string', description: 'The option the talent picked when you asked the question in the chat.' },
-    cvPath: { type: 'string', description: 'When the talent\'s answer (either option) already gives their CV as a path on this machine: that exact file is read, with no search and no request for the file.' },
+    cvPath: { type: 'string', description: 'When the talent\'s first message or answer already gives their CV as a path on this machine: that exact file is read, with no CV question, no search and no request for the file.' },
+    linkedinMissing: { type: 'boolean', description: 'true only after read_cv, when the CV has no LinkedIn URL: returns the LinkedIn question to print as your whole reply.' },
   },
   required: ['language'],
 };
@@ -148,6 +171,9 @@ const EMAIL_SCHEMA = {
   },
 };
 
+// In the order the data notice names them.
+const DRAFT_SOURCES = ['linkedin', 'cv'];
+
 const DRAFT_SCHEMA = {
   type: 'object',
   description: 'Your draft of the profile, from the CV, LinkedIn and what you know. Every field is shown as you write it, in the talent\'s language.',
@@ -163,9 +189,10 @@ const DRAFT_SCHEMA = {
     hourlyRate: { type: 'number', description: `Euros per hour (part-time projects), at most ${MAX_HOURLY_RATE}: the talent's figure, else your estimate from role, seniority, stack and location.` },
     annualRate: { type: 'number', description: `Euros per year (full-time projects), at least ${MIN_ANNUAL_RATE}: the talent's figure, else your estimate.` },
     ratesFromTalent: { type: 'boolean', description: 'true only when the talent gave you both rates; omit it when you estimated them (they are shown as your estimate).' },
+    sources: { type: 'array', items: { type: 'string', enum: DRAFT_SOURCES }, description: "What the draft comes from: 'cv' when you read their CV, 'linkedin' only when you have their LinkedIn URL. The data notice names exactly these." },
     answer: { type: 'string', description: 'The option the talent picked when you asked the question in the chat.' },
   },
-  required: ['name', 'role', 'hourlyRate', 'annualRate'],
+  required: ['name', 'role', 'hourlyRate', 'annualRate', 'sources'],
 };
 
 const CREATE_SCHEMA = {
@@ -261,6 +288,7 @@ function makeSignupTools(deps = {}) {
     fetchPricingRate = (opts) => require('./rate-client').fetchPricingRate({}, opts),
     fetchAvailability = (opts) => require('./availability-client').fetchAvailability({}, opts),
     fetchLanguages = (opts) => require('./profile-client').fetchLanguages({}, opts),
+    fetchTelephone = (opts) => require('./profile-client').fetchTelephone({}, opts),
     openBrowser = require('./open-file').openPath,
     sleep = defaultSleep,
     now = () => Date.now(),
@@ -275,6 +303,10 @@ function makeSignupTools(deps = {}) {
     draft: null,
     draftConfirmed: false,
     keepProfile: false,
+    // Fields the update question already told the talent they keep.
+    toldKept: new Set(),
+    // The confirmed draft already said the account exists (this machine knew it).
+    toldExisting: false,
     sources: null,
     account: { state: 'not-started' },
     reported: { account: null, import: null },
@@ -385,28 +417,53 @@ function makeSignupTools(deps = {}) {
     return Promise.race([announced, sleep(2000).then(() => state.windowUrl)]);
   }
 
-  // Step 1: welcome, value proposal, the three steps and one question (may I look for your CV here?).
+  // Step 1: welcome, value proposal, the three steps and one question (send me your CV, or shall I look for it here?).
   async function signupStart(args = {}, ctx = {}) {
+    // The LinkedIn question is a tool block, so a welcome still pending is read before it and the AI words nothing.
+    if (args.linkedinMissing === true && !nonEmpty(args.answer)) {
+      const c = copy();
+      return {
+        ok: true,
+        step: 'linkedin-ask',
+        say: [...takeNotices(), c.linkedinAsk].join('\n\n'),
+        linkedinOpening: c.linkedinOpening,
+        message: 'Print `say` word for word as your whole reply, with nothing of your own before or after it, and call no tool until the talent answers. With their URL, call signup_email. Only if they say they do not know where to find it, say `linkedinOpening` word for word and call open_linkedin_profile.',
+      };
+    }
     if (!nonEmpty(args.answer)) {
       fixSignupLanguage(args.language);
       state.draft = null;
       state.draftConfirmed = false;
       state.email = null;
       state.failedSources.clear();
+      state.toldKept = new Set();
+      state.toldExisting = false;
       state.cvPath = null;
       takeNotices();
     }
     if (nonEmpty(args.cvPath)) state.cvPath = nonEmpty(args.cvPath);
     const c = copy();
+    const linkedin = 'LinkedIn: use the URL in the CV (read_cv returns links.linkedin) and call signup_email. Only if the CV has none, call signup_start with linkedinMissing:true: it returns the LinkedIn question to print. Never ask for LinkedIn in your own words.';
+    // A CV path in the first message already answers the CV question: that CV is read, and the welcome
+    // rides on the next question block, because models skip a relay that comes before another tool call.
+    if (!nonEmpty(args.answer) && state.cvPath) {
+      queueNotice(c.welcome(nonEmpty(args.firstName)));
+      return {
+        ok: true,
+        step: 'cv-given',
+        cvPath: state.cvPath,
+        language: lang(),
+        message: `Never explain to the talent what you are doing or what the CV holds: every text they read in this step comes from the tools, the welcome included. Now read_cv cvPath with talentConsent:true, asking nothing about the CV. ${linkedin}`,
+      };
+    }
     const asked = await askAfterTexts(ctx, {
       tool: 'signup_start',
       texts: [c.welcome(nonEmpty(args.firstName))],
       question: c.welcomeQuestion,
-      options: [c.cvSearch, c.cvAttach],
+      options: [c.cvAttach, c.cvSearch],
       answer: args.answer,
     });
     if (asked.reply) return { ...asked.reply, language: lang() };
-    const linkedin = `LinkedIn: use the URL in the CV (read_cv returns links.linkedin). Only if it has none, ask \`linkedinAsk\` word for word; only if they say they do not know where to find it, say \`linkedinOpening\` and call open_linkedin_profile. Then call signup_email.`;
     // A CV path the talent gave wins over both answers: searching could read another candidate.
     const cvPath = state.cvPath;
     if (cvPath) {
@@ -414,8 +471,6 @@ function makeSignupTools(deps = {}) {
         ok: true,
         step: 'cv-given',
         cvPath,
-        linkedinAsk: c.linkedinAsk,
-        linkedinOpening: c.linkedinOpening,
         message: `The talent already gave their CV: read_cv it now with talentConsent:true, saying nothing about attaching it. ${linkedin}`,
       };
     }
@@ -424,8 +479,6 @@ function makeSignupTools(deps = {}) {
         ok: true,
         step: 'cv-search',
         cvNotFound: c.cvNotFound,
-        linkedinAsk: c.linkedinAsk,
-        linkedinOpening: c.linkedinOpening,
         message: `The talent allowed you to look for their CV: call suggest_register_context, then read_cv on its first candidate without asking again, even when nameMatch is null (their name is just unknown). Only if there is no candidate, or every one comes back mentionsTalent:false, show \`cvNotFound\` word for word and wait for the file. ${linkedin}`,
       };
     }
@@ -433,8 +486,6 @@ function makeSignupTools(deps = {}) {
       ok: true,
       step: 'cv-attach',
       relayVerbatim: c.cvAttachAsk,
-      linkedinAsk: c.linkedinAsk,
-      linkedinOpening: c.linkedinOpening,
       message: `If their message already holds the CV (attached, pasted or as a path), do not print relayVerbatim: use that CV. Otherwise ${VERBATIM} Then wait for the CV in the chat. ${linkedin}`,
     };
   }
@@ -451,7 +502,7 @@ function makeSignupTools(deps = {}) {
     if (!email) {
       return { ok: false, reason: 'email-required', relayVerbatim: c.emailUnknown, message: `${VERBATIM} It is the whole question. Then call signup_email with email set to what they write and typed:true.` };
     }
-    const question = isPersonalEmail(email) ? c.emailKnown(email) : `${c.emailKnown(email)} ${c.emailWorkHint}`;
+    const question = looksLikeWorkEmail(email) ? `${c.emailKnown(email)} ${c.emailWorkHint}` : c.emailKnown(email);
     const asked = await askAfterTexts(ctx, { tool: 'signup_email', texts: [], question, options: [c.emailYes, c.emailOther], answer: args.answer });
     if (asked.reply) return asked.reply;
     if (asked.answer === c.emailYes) {
@@ -461,6 +512,13 @@ function makeSignupTools(deps = {}) {
     return { ok: false, reason: 'email-required', relayVerbatim: c.emailUnknown, message: `${VERBATIM} Then call signup_email with email set to what they write and typed:true.` };
   }
 
+  // A session this machine kept for the same email means the account exists, so the window will ask to sign in.
+  function knownAccount() {
+    const stored = resolved.loadAuthSession();
+    const email = stored && typeof stored.email === 'string' ? stored.email.toLowerCase() : null;
+    return !!email && !!state.email && email === state.email.toLowerCase();
+  }
+
   // Step 3: the draft, the data notice and how the window works; confirming it is what opens the window.
   async function signupDraft(args = {}, ctx = {}) {
     const c = copy();
@@ -475,9 +533,14 @@ function makeSignupTools(deps = {}) {
       return { ok: false, reason: 'draft-rates-required', message: 'The draft needs both rates: if the talent gave them, send them with ratesFromTalent:true; otherwise estimate hourlyRate (€/h) and annualRate (€/year) from role, seniority, stack and location, and send them without it. Then call signup_draft again.' };
     }
     if (args.ratesFromTalent === true) draft.ratesFromTalent = true;
+    const sources = Array.isArray(args.sources) ? DRAFT_SOURCES.filter((s) => args.sources.includes(s)) : [];
+    // A confirmation of the printed block stands even if the model drops sources on the answer call.
+    if (!sources.length && !optionFor(args.answer, [c.draftOk, c.draftChange])) {
+      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." };
+    }
     const asked = await askAfterTexts(ctx, {
       tool: 'signup_draft',
-      texts: [renderDraft(lang(), draft), legalCopy(lang()).signupNotice, c.windowNotice],
+      texts: [renderDraft(lang(), draft), legalCopy(lang()).signupNotice(sources), knownAccount() ? c.windowNoticeSignIn : c.windowNotice],
       question: c.draftQuestion,
       options: [c.draftOk, c.draftChange],
       answer: args.answer,
@@ -489,6 +552,7 @@ function makeSignupTools(deps = {}) {
     }
     state.draft = draft;
     state.draftConfirmed = true;
+    state.toldExisting = knownAccount();
     return {
       ok: true,
       next: 'signup_create_account',
@@ -574,10 +638,10 @@ function makeSignupTools(deps = {}) {
   }
 
   function nextSteps(account) {
-    const interview = 'Then the onboarding interview: if onboardingInterview.completed is true do not offer 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 20), then set_main_role without clusterId only when its next says so. Finally open_web.';
+    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 20), 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).';
     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. ${aiUsage} ${interview}`;
+      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}`;
     }
     const unclaimed = account.claimed === false ? ' This account did not take over the imported profile: call import_profile with the same LinkedIn/CV first.' : '';
     return `The account exists.${unclaimed} Now, without asking: save_profile_details with the confirmed draft (situation, motivation, both rates, availability and location, languages). ${aiUsage} ${interview}`;
@@ -684,18 +748,32 @@ function makeSignupTools(deps = {}) {
     };
   }
 
-  // Fill-empty only: what of the confirmed draft would land in an existing profile, plus the import that only adds.
+  // Fill-empty only: per draft field, what it fills or the value the profile keeps, plus the import that only adds.
   async function existingDiff() {
     const c = copy();
     const draft = state.draft || {};
     const current = await currentProfile();
     const lines = [];
+    const kept = new Set();
     const has = (v) => v !== undefined && v !== null && v !== '';
-    if (current.pricing && current.pricing.hourly === null && has(draft.hourlyRate)) lines.push(c.existingRate(c.existingEmpty, money(draft.hourlyRate, c.perHour)));
-    if (current.pricing && current.pricing.annual === null && has(draft.annualRate)) lines.push(c.existingAnnual(c.existingEmpty, money(draft.annualRate, c.perYear)));
-    if (current.availability && !has(current.availability.monthlyHours) && has(draft.monthlyHours)) lines.push(c.existingHours(c.existingEmpty, c.hours(draft.monthlyHours)));
-    lines.push(c.existingLists);
-    return `${c.existingTitle}\n\n${lines.map((l) => `· ${l}`).join('\n')}`;
+    // An unreadable section (undefined) counts as set, as fillEmpty never writes it.
+    const field = (key, now, filled, label, shown) => {
+      if (!has(draft[key])) return;
+      if (now === null) { lines.push(filled); return; }
+      kept.add(key);
+      lines.push(c.existingStays(label, now === undefined ? null : shown(now), shown(draft[key])));
+    };
+    const pricing = current.pricing || { hourly: undefined, annual: undefined };
+    const hours = current.availability ? (has(current.availability.monthlyHours) ? current.availability.monthlyHours : null) : undefined;
+    field('hourlyRate', pricing.hourly, c.existingRate(c.existingEmpty, money(draft.hourlyRate, c.perHour)), c.hourlyRateLabel, (v) => money(v, c.perHour));
+    field('annualRate', pricing.annual, c.existingAnnual(c.existingEmpty, money(draft.annualRate, c.perYear)), c.annualRateLabel, (v) => money(v, c.perYear));
+    field('monthlyHours', hours, c.existingHours(c.existingEmpty, c.hours(draft.monthlyHours)), c.keptLabels.availability, (v) => c.hours(v));
+    if (has(draft.languages) && current.languageCount !== 0) { kept.add('languages'); lines.push(c.existingStays(c.keptLabels.languages)); }
+    if (kept.size) lines.push(c.existingStaysNote);
+    state.toldKept = kept;
+    const given = state.sources ? [state.sources.linkedinUrl && 'linkedin', state.sources.cvPath && 'cv'].filter(Boolean) : [];
+    if (given.length) lines.push(c.existingLists(given));
+    return `${state.toldExisting ? c.existingUpdateQuestion : c.existingTitle}\n\n${lines.map((l) => `· ${l}`).join('\n')}`;
   }
 
   // The sections of save_profile_details an existing profile can take (only fields it leaves empty), and what it keeps instead.
@@ -731,25 +809,25 @@ function makeSignupTools(deps = {}) {
     return { details: out, kept };
   }
 
-  // What the existing profile kept, for the model per field and for the talent as one fixed line.
+  // What the existing profile kept, for the model per field and for the talent as one fixed line without what the update question already told.
   function keptReport(kept) {
     const c = copy();
     const fields = [];
     const items = [];
-    const value = (v, shown) => (v === null || v === undefined ? null : shown(v));
+    const told = (key) => state.toldKept.has(key);
     if (kept.workSituation) { fields.push('workSituation: not changed (the profile keeps what it had)'); items.push(c.keptLabels.workSituation); }
     for (const [key, label, unit] of [['hourlyRate', c.hourlyRateLabel, c.perHour], ['annualRate', c.annualRateLabel, c.perYear]]) {
       const k = kept.pricing && kept.pricing[key];
       if (!k) continue;
       fields.push(k.kept === null ? `${key}: not changed (the profile keeps what it had, not ${k.requested})` : `${key}: kept existing value ${k.kept} (not changed, not ${k.requested})`);
-      items.push(k.kept === null ? label : `${label}: ${money(k.kept, unit)}`);
+      if (!told(key)) items.push(k.kept === null ? label : `${label}: ${money(k.kept, unit)}`);
     }
     for (const [key, k] of Object.entries(kept.availability || {})) {
       fields.push(k.kept === null ? `${key}: not changed (the profile keeps what it had)` : `${key}: kept existing value ${Array.isArray(k.kept) ? k.kept.join('/') : k.kept} (not changed, not ${Array.isArray(k.requested) ? k.requested.join('/') : k.requested})`);
     }
     const hours = kept.availability && kept.availability.monthlyHours;
-    if (hours) items.push(hours.kept === null ? c.keptLabels.availability : `${c.keptLabels.availability}: ${c.hours(hours.kept)}`);
-    if (kept.languages) { fields.push('languages: not changed (the profile already had languages)'); items.push(c.keptLabels.languages); }
+    if (hours && !told('monthlyHours')) items.push(hours.kept === null ? c.keptLabels.availability : `${c.keptLabels.availability}: ${c.hours(hours.kept)}`);
+    if (kept.languages) { fields.push('languages: not changed (the profile already had languages)'); if (!told('languages')) items.push(c.keptLabels.languages); }
     if (kept.phone) { fields.push('phone: not changed (the profile keeps what it had)'); items.push(c.keptLabels.phone); }
     return { fields, line: items.length ? c.existingUnchanged(items) : null };
   }
@@ -805,17 +883,26 @@ function makeSignupTools(deps = {}) {
       ['availability', details.availability, (v) => flow.saveAvailability(resolved, v), saved.availability],
       ['languages', Array.isArray(details.languages) && details.languages.length ? details.languages : null, (v) => flow.saveLanguages(resolved, v), saved.languages],
       ['phone', details.phone, (v) => flow.savePhone(resolved, v), saved.phone],
-    ];
+    ].filter(([, value]) => value && typeof value === 'object');
+    // Hub's async profile-completion recalculation saves the whole users row from a copy it read earlier, so a
+    // phone written while it runs comes back NULL: the phone goes first, and is read back once the rest is saved.
+    const phoneFirst = [...sections.filter(([name]) => name === 'phone'), ...sections.filter(([name]) => name !== 'phone')];
     const results = {};
-    const confirmations = [];
-    for (const [name, value, save, saved] of sections) {
-      if (!value || typeof value !== 'object') continue;
+    for (const [name, value, save] of phoneFirst) {
       const res = await save(value);
-      if (res.ok) confirmations.push(saved);
       if (!res.ok && res.reason === 'no-session') return { ok: false, reason: 'no-session', message: 'The talent is not signed in yet.' };
       const help = res.message || SECTION_HELP[res.reason];
       results[name] = res.ok ? { ok: true } : { ok: false, reason: res.reason, ...(help ? { message: help } : {}) };
     }
+    if (results.phone && results.phone.ok) {
+      const session = resolved.loadAuthSession();
+      const stored = await fetchTelephone({ hubAccessToken: session && session.hubAccessToken }).catch(() => ({ ok: false }));
+      if (stored.ok && !stored.telephone) {
+        const again = await flow.savePhone(resolved, details.phone);
+        results.phone = again.ok ? { ok: true } : { ok: false, reason: again.reason };
+      }
+    }
+    const confirmations = sections.filter(([name]) => results[name] && results[name].ok).map(([, , , text]) => text);
     const ok = Object.values(results).every((r) => r.ok);
     const report = keptReport(kept);
     if (report.line) queueNotice(report.line);
@@ -845,27 +932,32 @@ function makeSignupTools(deps = {}) {
     if (!url) return { ok: false, reason: 'no-endpoint' };
     // The address reaches the AI only when the talent needs a link: the AI comments on hosts it sees.
     const opened = args.open !== false && openBrowser(url) !== false;
+    // Read right before the closing words, so they never state an import state that is already stale.
+    const imp = await readImport();
+    // A source that failed after the last signup_status gets its fixed line here, as there is no question left.
+    const failed = DRAFT_SOURCES.filter((s) => imp.sources && imp.sources[s] && imp.sources[s].state === 'failed');
+    noticeFailed(failed);
     // The flow ends here, so a fixed line no question carried is printed now.
     const unseen = takeNotices();
     const relay = unseen.length ? { relayVerbatim: unseen.join('\n\n') } : {};
-    const relayFirst = unseen.length ? `${VERBATIM} ` : '';
-    if (opened) {
-      return {
-        ok: true,
-        opened: true,
-        loggedIn,
-        ...relay,
-        message: relayFirst + (loggedIn ? 'Shakers opened in the talent\'s browser, signed in.' : 'Shakers opened in the talent\'s browser at the sign-in page; they sign in there.')
-          + ' If they say it did not open, call open_web with open:false and give them its link.',
-      };
-    }
+    const importLine = failed.length
+      ? `The import of ${failed.join(' and ')} failed and its fixed line is the only thing to say about it: never say or imply it was imported; if you mention the import, name only what did import.`
+      : IMPORT_CLOSING[imp.state] || IMPORT_CLOSING.none;
+    const where = opened
+      ? `Shakers is open in their browser${loggedIn ? ', already signed in' : ' at the sign-in page, where they sign in'}`
+      : `they open Shakers from \`link\` (give it to click, with no comment on the address${loggedIn ? '; it works once, for 2 minutes' : ''})`;
+    const closing = `one short closing sentence of your own, in the talent's language, saying ${where}.`;
+    const reply = unseen.length
+      ? `Your reply has two parts and needs both: first relayVerbatim word for word as its own block, then ${closing}`
+      : `End with ${closing}`;
     return {
       ok: true,
-      opened: false,
+      opened,
       loggedIn,
-      link: url,
+      ...(opened ? {} : { link: url }),
+      import: { state: imp.state },
       ...relay,
-      message: relayFirst + 'Give the talent `link` to click, with no comment on the address.' + (loggedIn ? ' It works once, for 2 minutes.' : ''),
+      message: `${reply}${opened ? ' If they say it did not open, call open_web with open:false and give them its link.' : ''} ${importLine}`,
     };
   }
 
@@ -882,7 +974,7 @@ function makeSignupTools(deps = {}) {
   return [
     {
       name: 'signup_start',
-      description: "FIRST call when the talent asks to sign up or register on Shakers, before you write anything: pass `language` = the language of their message. It returns the welcome to show word for word and its one question (may I look for your CV on this computer?). Then follow its message: CV, LinkedIn, signup_email, signup_draft, signup_create_account, signup_status. Every text it and the next sign-up tools return in `say` or relayVerbatim is printed word for word, in that language.",
+      description: "FIRST call when the talent asks to sign up or register on Shakers, before you write anything: pass `language` = the language of their message. It returns the welcome to show word for word and its one question (send me your CV, or shall I look for it on this computer?). Then follow its message: CV, LinkedIn, signup_email, signup_draft, signup_create_account, signup_status. Every text it and the next sign-up tools return in `say` or relayVerbatim is printed word for word, in that language.",
       inputSchema: WELCOME_SCHEMA,
       handler: signupStart,
     },
@@ -912,7 +1004,7 @@ function makeSignupTools(deps = {}) {
     },
     {
       name: 'update_existing_profile',
-      description: 'When signup_status says the talent already had an account: shows what the confirmed draft would fill in the empty fields of their profile and asks whether to update it; on yes it imports their LinkedIn/CV filling only what is empty.',
+      description: 'When signup_status says the talent already had an account: shows what the confirmed draft would fill in the empty fields of their profile and the value each field they already had keeps, and asks whether to update it; on yes it imports their LinkedIn/CV filling only what is empty.',
       inputSchema: UPDATE_EXISTING_SCHEMA,
       handler: updateExistingProfile,
     },
@@ -943,4 +1035,4 @@ function makeSignupTools(deps = {}) {
   ];
 }
 
-module.exports = { makeSignupTools, isSignupPending, signupPhase, readSignupCv, isPersonalEmail, WELCOME_SCHEMA, EMAIL_SCHEMA, DRAFT_SCHEMA, CREATE_SCHEMA, STATUS_SCHEMA, IMPORT_SCHEMA };
+module.exports = { makeSignupTools, isSignupPending, signupPhase, readSignupCv, isPersonalEmail, looksLikeWorkEmail, WELCOME_SCHEMA, EMAIL_SCHEMA, DRAFT_SCHEMA, CREATE_SCHEMA, STATUS_SCHEMA, IMPORT_SCHEMA };

package/src/profile-client.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// AI-profile client (hub `works/me/ai-profile`, talent Bearer). The "My work
+// AI-profile client. The "My work
 // with AI" read model: setup/usage tier+level, 3x3 cell, vision, howIWork.
 
 const { postJsonWithTimeout } = require('./backend-request');
@@ -87,14 +87,12 @@ async function getMapped(endpoint, hubAccessToken, timeoutMs, map) {
   return { ok: true, ...map(data) };
 }
 
-// `works/me/profile` -> headline (basic profile text).
 async function fetchMeProfile(deps = {}, { hubAccessToken, timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
   const getEndpoint = deps.getMeProfileEndpoint || require('./config').getMeProfileEndpoint;
   const req = deps.getMapped || getMapped;
   return req(getEndpoint(), hubAccessToken, timeoutMs, (d) => ({ headline: str(d && d.headline) }));
 }
 
-// `works/talents/me/profile` -> completion %, freelance type, onboarding status.
 async function fetchTalentMeProfile(deps = {}, { hubAccessToken, timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
   const getEndpoint = deps.getTalentMeProfileEndpoint || require('./config').getTalentMeProfileEndpoint;
   const req = deps.getMapped || getMapped;
@@ -105,7 +103,6 @@ async function fetchTalentMeProfile(deps = {}, { hubAccessToken, timeoutMs = DEF
   }));
 }
 
-// `works/talents/me/work-details/languages` -> count + codes.
 async function fetchLanguages(deps = {}, { hubAccessToken, timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
   const getEndpoint = deps.getLanguagesEndpoint || require('./config').getLanguagesEndpoint;
   const req = deps.getMapped || getMapped;
@@ -116,10 +113,21 @@ async function fetchLanguages(deps = {}, { hubAccessToken, timeoutMs = DEFAULT_T
   });
 }
 
+// `works/users/me` -> the stored phone, null when hub has none.
+async function fetchTelephone(deps = {}, { hubAccessToken, timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
+  const getEndpoint = deps.getUserMeEndpoint || require('./config').getUserMeEndpoint;
+  const req = deps.getMapped || getMapped;
+  return req(getEndpoint(), hubAccessToken, timeoutMs, (d) => {
+    const t = d && d.telephone;
+    return { telephone: t && str(t.code) && str(t.number) ? { code: str(t.code), number: str(t.number) } : null };
+  });
+}
+
 module.exports = {
   requestAiProfile,
   fetchAiProfile,
   fetchMeProfile,
   fetchTalentMeProfile,
   fetchLanguages,
+  fetchTelephone,
 };

package/src/rate-client.js

@@ -1,7 +1,6 @@
 'use strict';
 
-// Pricing-rate client (hub `works/talents/me/work-details/pricing-rate`, talent
-// Bearer). READ-ONLY here: setting the rate is a financial PUT, left to register.
+// Pricing-rate client. READ-ONLY here: setting the rate is a financial PUT, left to register.
 
 const { postJsonWithTimeout } = require('./backend-request');
 const { reasonForError } = require('./onboarding-client');

package/src/roles-client.js

@@ -43,7 +43,6 @@ function normalizeRole(r) {
   };
 }
 
-// GET works/certifications/me/available-roles -> { roles:[{clusterId,name,category,proposalKind}] }.
 async function requestAvailableRoles({ hubAccessToken } = {}, { endpoint, timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
   if (!endpoint) return { ok: false, reason: 'no-endpoint' };
   if (!hubAccessToken) return { ok: false, reason: 'no-hub-token' };
@@ -60,7 +59,6 @@ async function requestAvailableRoles({ hubAccessToken } = {}, { endpoint, timeou
   return { ok: true, roles: rows.map(normalizeRole).filter(Boolean) };
 }
 
-// GET works/talents/me/work-details/assigned-clusters -> the talent's GROWTH/RECOMMENDED roles.
 async function requestAssignedClusters({ hubAccessToken } = {}, { endpoint, timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
   if (!endpoint) return { ok: false, reason: 'no-endpoint' };
   if (!hubAccessToken) return { ok: false, reason: 'no-hub-token' };
@@ -77,7 +75,6 @@ async function requestAssignedClusters({ hubAccessToken } = {}, { endpoint, time
   return { ok: true, clusters: rows.map(normalizeRole).filter(Boolean) };
 }
 
-// POST works/talents/me/work-details/assigned-clusters { clusterId } -> adds a GROWTH role.
 async function requestAddGrowthRole({ clusterId, hubAccessToken } = {}, { endpoint, timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
   if (!endpoint) return { ok: false, reason: 'no-endpoint' };
   if (!hubAccessToken) return { ok: false, reason: 'no-hub-token' };
@@ -92,7 +89,6 @@ async function requestAddGrowthRole({ clusterId, hubAccessToken } = {}, { endpoi
   return { ok: true, clusterId };
 }
 
-// PATCH works/certifications/me/main-role { clusterId } -> sets the preferred main role.
 async function requestSetMainRole({ clusterId, hubAccessToken } = {}, { endpoint, timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
   if (!endpoint) return { ok: false, reason: 'no-endpoint' };
   if (!hubAccessToken) return { ok: false, reason: 'no-hub-token' };

package/src/saved-positions-client.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// Saved-positions client (hub `works/saved-positions`, talent Bearer). Both
+// Saved-positions client. Both
 // operations are idempotent server-side, so no read-before-write is needed.
 
 const { postJsonWithTimeout } = require('./backend-request');

package/src/show-project-client.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// Position detail client (hub `works/positions/:id/detail`, talent Bearer).
+// Position detail client.
 // A "project" here is the Position + its parent project's fields.
 
 const { postJsonWithTimeout } = require('./backend-request');