@productbrain/cli 0.1.0-beta.36 → 0.1.0-beta.3616

package/dist/lib/connectKeyLabel.js

@@ -0,0 +1,12 @@
+/**
+ * Labels for API keys from `pb connect` — keep in sync with convex/lib/connectRedeemDisplayName.ts.
+ * (Monorepo boundary: Convex cannot import from packages/cli and vice versa.)
+ */
+export const CONNECT_REDEEM_DEFAULT_API_KEY_NAME = 'Product Brain CLI';
+export const CONNECT_REDEEM_NAME_MAX_LENGTH = 200;
+const CONTROL_CHARS = /[\u0000-\u001F\u007F-\u009F]/g;
+/** Align client-side trimming with server `normalizeConnectRedeemDisplayName` (control strip + cap). */
+export function trimClientKeyLabelInput(raw) {
+    return raw.replace(CONTROL_CHARS, '').trim().slice(0, CONNECT_REDEEM_NAME_MAX_LENGTH).trimEnd();
+}
+//# sourceMappingURL=connectKeyLabel.js.map

package/dist/lib/connectKeyLabel.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"connectKeyLabel.js","sourceRoot":"","sources":["../../src/lib/connectKeyLabel.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,MAAM,CAAC,MAAM,mCAAmC,GAAG,mBAAmB,CAAC;AAEvE,MAAM,CAAC,MAAM,8BAA8B,GAAG,GAAG,CAAC;AAElD,MAAM,aAAa,GAAG,+BAA+B,CAAC;AAEtD,wGAAwG;AACxG,MAAM,UAAU,uBAAuB,CAAC,GAAW;IAClD,OAAO,GAAG,CAAC,OAAO,CAAC,aAAa,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,8BAA8B,CAAC,CAAC,OAAO,EAAE,CAAC;AACjG,CAAC"}

package/dist/lib/constants.d.ts

@@ -4,14 +4,39 @@
  */
 /** Production Convex HTTP Actions endpoint. */
 export declare const DEFAULT_SITE_URL = "https://gateway.productbrain.io";
+/** Canonical self-upgrade command. WP-425 fixes the stale @latest dist-tag trap. */
+export declare const UPGRADE_COMMAND = "npm install -g @productbrain/cli@beta";
 /**
  * Resolve the Convex site URL based on PB_ENV.
- * - PB_ENV=dev → local Convex dev server
+ * - PB_ENV=dev → DEV_SITE_URL (localhost — see caveat on DEV_SITE_URL)
  * - PB_ENV=production or unset → production URL
+ *
+ * NOTE: No production CLI command currently imports this function.
+ * For HTTP Actions (connect, etc.), use CONVEX_SITE_URL or DEFAULT_SITE_URL.
  */
 export declare function resolveSiteUrl(): string;
 /**
- * Resolve the web app URL based on PB_ENV.
+ * Gateway URL for `pb connect` token redemption (`POST …/auth/redeem-connect-token`).
+ *
+ * Resolution order:
+ * 1. `cliGateway` argument (from `pb connect --gateway <url>`)
+ * 2. `PB_CONNECT_GATEWAY_URL` env var
+ * 3. `CONVEX_SITE_URL` env var — the actual Convex HTTP Actions deployment
+ * 4. `DEFAULT_SITE_URL` (production gateway)
+ *
+ * Convex HTTP Actions are cloud-hosted even during local development
+ * (`npx convex dev` syncs to cloud, not localhost). Tokens minted by
+ * any web app (production or localhost) live in the cloud Convex DB,
+ * so redeem must target the cloud deployment — never localhost.
+ *
+ * For dev deployments: set CONVEX_SITE_URL to your *.convex.site URL
+ * or use --gateway.
+ */
+export declare function resolveConnectRedeemUrl(cliGateway?: string): string;
+/**
+ * Resolve the web app URL.
+ * - PB_APP_URL (explicit override) → wins, for any binding whose app host isn't
+ *   inferrable from PB_ENV (e.g. a preview deployment walked via a custom host).
  * - PB_ENV=dev → localhost:5173
  * - PB_ENV=production or unset → work.productbrain.io
  */

package/dist/lib/constants.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"constants.d.ts","sourceRoot":"","sources":["../../src/lib/constants.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,+CAA+C;AAC/C,eAAO,MAAM,gBAAgB,oCAAoC,CAAC;AAWlE;;;;GAIG;AACH,wBAAgB,cAAc,IAAI,MAAM,CAIvC;AAED;;;;GAIG;AACH,wBAAgB,aAAa,IAAI,MAAM,CAItC;AAED,8CAA8C;AAC9C,wBAAgB,SAAS,IAAI,OAAO,CAEnC"}
+{"version":3,"file":"constants.d.ts","sourceRoot":"","sources":["../../src/lib/constants.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,+CAA+C;AAC/C,eAAO,MAAM,gBAAgB,oCAAoC,CAAC;AAElE,oFAAoF;AACpF,eAAO,MAAM,eAAe,0CAA0C,CAAC;AAiBvE;;;;;;;GAOG;AACH,wBAAgB,cAAc,IAAI,MAAM,CAIvC;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,uBAAuB,CAAC,UAAU,CAAC,EAAE,MAAM,GAAG,MAAM,CAQnE;AAED;;;;;;GAMG;AACH,wBAAgB,aAAa,IAAI,MAAM,CAMtC;AAED,8CAA8C;AAC9C,wBAAgB,SAAS,IAAI,OAAO,CAEnC"}

package/dist/lib/constants.js

@@ -4,16 +4,26 @@
  */
 /** Production Convex HTTP Actions endpoint. */
 export const DEFAULT_SITE_URL = 'https://gateway.productbrain.io';
+/** Canonical self-upgrade command. WP-425 fixes the stale @latest dist-tag trap. */
+export const UPGRADE_COMMAND = 'npm install -g @productbrain/cli@beta';
+/** Hostname lists for banners + admin seed gates live in `deployment.ts` (PRODUCTION_HOSTS, DEV_CLOUD_HOSTS). */
 /** Production web app URL. */
 const PROD_APP_URL = 'https://work.productbrain.io';
-/** Local Convex dev server URL. */
+/**
+ * Legacy: intended for a local Convex dev server, but Convex HTTP Actions
+ * are always cloud-hosted (`npx convex dev` syncs to *.convex.site).
+ * Only used by resolveSiteUrl() which has no production consumers.
+ */
 const DEV_SITE_URL = 'http://localhost:5176';
 /** Local web app URL. */
 const DEV_APP_URL = 'http://localhost:5173';
 /**
  * Resolve the Convex site URL based on PB_ENV.
- * - PB_ENV=dev → local Convex dev server
+ * - PB_ENV=dev → DEV_SITE_URL (localhost — see caveat on DEV_SITE_URL)
  * - PB_ENV=production or unset → production URL
+ *
+ * NOTE: No production CLI command currently imports this function.
+ * For HTTP Actions (connect, etc.), use CONVEX_SITE_URL or DEFAULT_SITE_URL.
  */
 export function resolveSiteUrl() {
     const env = process.env.PB_ENV?.toLowerCase();
@@ -22,11 +32,45 @@ export function resolveSiteUrl() {
     return DEFAULT_SITE_URL;
 }
 /**
- * Resolve the web app URL based on PB_ENV.
+ * Gateway URL for `pb connect` token redemption (`POST …/auth/redeem-connect-token`).
+ *
+ * Resolution order:
+ * 1. `cliGateway` argument (from `pb connect --gateway <url>`)
+ * 2. `PB_CONNECT_GATEWAY_URL` env var
+ * 3. `CONVEX_SITE_URL` env var — the actual Convex HTTP Actions deployment
+ * 4. `DEFAULT_SITE_URL` (production gateway)
+ *
+ * Convex HTTP Actions are cloud-hosted even during local development
+ * (`npx convex dev` syncs to cloud, not localhost). Tokens minted by
+ * any web app (production or localhost) live in the cloud Convex DB,
+ * so redeem must target the cloud deployment — never localhost.
+ *
+ * For dev deployments: set CONVEX_SITE_URL to your *.convex.site URL
+ * or use --gateway.
+ */
+export function resolveConnectRedeemUrl(cliGateway) {
+    const fromCli = cliGateway?.trim();
+    if (fromCli)
+        return fromCli.replace(/\/$/, '');
+    const fromEnv = process.env.PB_CONNECT_GATEWAY_URL?.trim();
+    if (fromEnv)
+        return fromEnv.replace(/\/$/, '');
+    const fromConvex = process.env.CONVEX_SITE_URL?.trim();
+    if (fromConvex)
+        return fromConvex.replace(/\/$/, '');
+    return DEFAULT_SITE_URL;
+}
+/**
+ * Resolve the web app URL.
+ * - PB_APP_URL (explicit override) → wins, for any binding whose app host isn't
+ *   inferrable from PB_ENV (e.g. a preview deployment walked via a custom host).
  * - PB_ENV=dev → localhost:5173
  * - PB_ENV=production or unset → work.productbrain.io
  */
 export function resolveAppUrl() {
+    const override = process.env.PB_APP_URL?.trim();
+    if (override)
+        return override.replace(/\/$/, '');
     const env = process.env.PB_ENV?.toLowerCase();
     if (env === 'dev')
         return DEV_APP_URL;

package/dist/lib/constants.js.map

@@ -1 +1 @@
-{"version":3,"file":"constants.js","sourceRoot":"","sources":["../../src/lib/constants.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,+CAA+C;AAC/C,MAAM,CAAC,MAAM,gBAAgB,GAAG,iCAAiC,CAAC;AAElE,8BAA8B;AAC9B,MAAM,YAAY,GAAG,8BAA8B,CAAC;AAEpD,mCAAmC;AACnC,MAAM,YAAY,GAAG,uBAAuB,CAAC;AAE7C,yBAAyB;AACzB,MAAM,WAAW,GAAG,uBAAuB,CAAC;AAE5C;;;;GAIG;AACH,MAAM,UAAU,cAAc;IAC5B,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,WAAW,EAAE,CAAC;IAC9C,IAAI,GAAG,KAAK,KAAK;QAAE,OAAO,YAAY,CAAC;IACvC,OAAO,gBAAgB,CAAC;AAC1B,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,aAAa;IAC3B,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,WAAW,EAAE,CAAC;IAC9C,IAAI,GAAG,KAAK,KAAK;QAAE,OAAO,WAAW,CAAC;IACtC,OAAO,YAAY,CAAC;AACtB,CAAC;AAED,8CAA8C;AAC9C,MAAM,UAAU,SAAS;IACvB,OAAO,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,WAAW,EAAE,KAAK,KAAK,CAAC;AACrD,CAAC"}
+{"version":3,"file":"constants.js","sourceRoot":"","sources":["../../src/lib/constants.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,+CAA+C;AAC/C,MAAM,CAAC,MAAM,gBAAgB,GAAG,iCAAiC,CAAC;AAElE,oFAAoF;AACpF,MAAM,CAAC,MAAM,eAAe,GAAG,uCAAuC,CAAC;AAEvE,iHAAiH;AAEjH,8BAA8B;AAC9B,MAAM,YAAY,GAAG,8BAA8B,CAAC;AAEpD;;;;GAIG;AACH,MAAM,YAAY,GAAG,uBAAuB,CAAC;AAE7C,yBAAyB;AACzB,MAAM,WAAW,GAAG,uBAAuB,CAAC;AAE5C;;;;;;;GAOG;AACH,MAAM,UAAU,cAAc;IAC5B,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,WAAW,EAAE,CAAC;IAC9C,IAAI,GAAG,KAAK,KAAK;QAAE,OAAO,YAAY,CAAC;IACvC,OAAO,gBAAgB,CAAC;AAC1B,CAAC;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,UAAU,uBAAuB,CAAC,UAAmB;IACzD,MAAM,OAAO,GAAG,UAAU,EAAE,IAAI,EAAE,CAAC;IACnC,IAAI,OAAO;QAAE,OAAO,OAAO,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;IAC/C,MAAM,OAAO,GAAG,OAAO,CAAC,GAAG,CAAC,sBAAsB,EAAE,IAAI,EAAE,CAAC;IAC3D,IAAI,OAAO;QAAE,OAAO,OAAO,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;IAC/C,MAAM,UAAU,GAAG,OAAO,CAAC,GAAG,CAAC,eAAe,EAAE,IAAI,EAAE,CAAC;IACvD,IAAI,UAAU;QAAE,OAAO,UAAU,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;IACrD,OAAO,gBAAgB,CAAC;AAC1B,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,aAAa;IAC3B,MAAM,QAAQ,GAAG,OAAO,CAAC,GAAG,CAAC,UAAU,EAAE,IAAI,EAAE,CAAC;IAChD,IAAI,QAAQ;QAAE,OAAO,QAAQ,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;IACjD,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,WAAW,EAAE,CAAC;IAC9C,IAAI,GAAG,KAAK,KAAK;QAAE,OAAO,WAAW,CAAC;IACtC,OAAO,YAAY,CAAC;AACtB,CAAC;AAED,8CAA8C;AAC9C,MAAM,UAAU,SAAS;IACvB,OAAO,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,WAAW,EAAE,KAAK,KAAK,CAAC;AACrD,CAAC"}

package/dist/lib/conversation-engine.d.ts

@@ -1,47 +1,42 @@
 /**
- * Conversation engine — reusable phase evaluation and advancement.
- * WP-304: Deterministic Shell, Stochastic Core.
+ * Conversation engine — topic evaluation, XML directive builder, post-processing.
+ * WP-304: Coach pattern — LLM judges topic completion, heuristic backup.
  *
- * The engine evaluates exit criteria and decides when to advance.
+ * The engine evaluates topic completion and builds XML directives.
  * It does NOT generate content — that's the LLM's job.
  */
-import type { ConversationPhase, ConversationState } from './conversation-phases.js';
-/** Result of evaluating the current phase. */
-export interface PhaseAdvanceResult {
-    /** Whether the current phase is complete. */
+import type { ConversationState, ConversationSignals, TopicQA } from './conversation-phases.js';
+import type { OnboardingTopic } from './onboarding-topics.js';
+/** Result of evaluating the current topic. */
+export interface TopicEvalResult {
+    /** Whether the current topic is complete. */
     complete: boolean;
     /** Whether advancement was forced by maxExchanges. */
     forced: boolean;
-    /** Criteria that were met. */
-    metCriteria: string[];
-    /** Criteria that were NOT met. */
-    unmetCriteria: string[];
-    /** The next phase, or null if all phases are done. */
-    nextPhase: ConversationPhase | null;
-    /** True when all phases are complete — trigger extraction. */
-    readyToExtract: boolean;
+    /** Whether this topic was skipped (signal already present). */
+    skip: boolean;
 }
 /**
- * Evaluate whether the current phase is complete.
- * Returns advancement decision based on exit criteria and exchange count.
+ * Check if a topic should be skipped because its signal is already present.
+ * Only skips if we have enough content for quality extraction (>= 20 words).
  */
-export declare function evaluatePhase(phases: ConversationPhase[], state: ConversationState): PhaseAdvanceResult;
+export declare function shouldSkipTopic(topic: OnboardingTopic, signals: ConversationSignals, totalUserWords: number): boolean;
 /**
- * Build the phase directive to inject into the system prompt.
- * For bridge phase, injects accumulated quotable fragments so the LLM has
- * concrete user phrases to work with instead of re-extracting from history.
+ * Evaluate whether the current topic is complete.
+ * Priority: skip > LLM judgment > heuristic backup > forced > continue.
  */
-export declare function buildPhaseDirective(phase: ConversationPhase, phaseIndex: number, totalPhases: number, quotableFragments?: string[]): string;
+export declare function evaluateTopic(topic: OnboardingTopic, state: ConversationState, llmSaidComplete: boolean): TopicEvalResult;
+/**
+ * Build the XML topic directive to inject into the system prompt.
+ * Renders <topic> with goal/behaviour/criteria, plus <accumulated_context>
+ * with completed Q&A pairs from earlier topics.
+ */
+export declare function buildTopicDirective(topic: OnboardingTopic, topicIndex: number, totalTopics: number, completedQA: TopicQA[], workspaceName: string): string;
 /**
  * Strip banned patterns the LLM produces despite prompt instructions.
  * Safety net — Gemini Flash at temp 0.3 still produces these ~15% of the time.
  */
 export declare function sanitizeResponse(text: string): string;
-/**
- * If we're in bridge phase and the response ends with a question, strip it.
- * Bridge is a statement — the LLM should not ask another question.
- */
-export declare function enforceBridgeIsStatement(text: string, phaseId: string): string;
 /**
  * Detect if a response is too similar to recent ones (refusal loop).
  * Uses Jaccard similarity on word sets.

package/dist/lib/conversation-engine.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"conversation-engine.d.ts","sourceRoot":"","sources":["../../src/lib/conversation-engine.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,iBAAiB,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAC;AAErF,8CAA8C;AAC9C,MAAM,WAAW,kBAAkB;IACjC,6CAA6C;IAC7C,QAAQ,EAAE,OAAO,CAAC;IAElB,sDAAsD;IACtD,MAAM,EAAE,OAAO,CAAC;IAEhB,8BAA8B;IAC9B,WAAW,EAAE,MAAM,EAAE,CAAC;IAEtB,kCAAkC;IAClC,aAAa,EAAE,MAAM,EAAE,CAAC;IAExB,sDAAsD;IACtD,SAAS,EAAE,iBAAiB,GAAG,IAAI,CAAC;IAEpC,8DAA8D;IAC9D,cAAc,EAAE,OAAO,CAAC;CACzB;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAC3B,MAAM,EAAE,iBAAiB,EAAE,EAC3B,KAAK,EAAE,iBAAiB,GACvB,kBAAkB,CAwDpB;AAED;;;;GAIG;AACH,wBAAgB,mBAAmB,CACjC,KAAK,EAAE,iBAAiB,EACxB,UAAU,EAAE,MAAM,EAClB,WAAW,EAAE,MAAM,EACnB,iBAAiB,CAAC,EAAE,MAAM,EAAE,GAC3B,MAAM,CAgBR;AAcD;;;GAGG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAOrD;AAED;;;GAGG;AACH,wBAAgB,wBAAwB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAO9E;AAED;;;GAGG;AACH,wBAAgB,YAAY,CAAC,MAAM,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,EAAE,SAAS,SAAM,GAAG,OAAO,CAW3F"}
+{"version":3,"file":"conversation-engine.d.ts","sourceRoot":"","sources":["../../src/lib/conversation-engine.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,OAAO,EAAE,MAAM,0BAA0B,CAAC;AAChG,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAE9D,8CAA8C;AAC9C,MAAM,WAAW,eAAe;IAC9B,6CAA6C;IAC7C,QAAQ,EAAE,OAAO,CAAC;IAElB,sDAAsD;IACtD,MAAM,EAAE,OAAO,CAAC;IAEhB,+DAA+D;IAC/D,IAAI,EAAE,OAAO,CAAC;CACf;AASD;;;GAGG;AACH,wBAAgB,eAAe,CAAC,KAAK,EAAE,eAAe,EAAE,OAAO,EAAE,mBAAmB,EAAE,cAAc,EAAE,MAAM,GAAG,OAAO,CAIrH;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAC3B,KAAK,EAAE,eAAe,EACtB,KAAK,EAAE,iBAAiB,EACxB,eAAe,EAAE,OAAO,GACvB,eAAe,CAyBjB;AAED;;;;GAIG;AACH,wBAAgB,mBAAmB,CACjC,KAAK,EAAE,eAAe,EACtB,UAAU,EAAE,MAAM,EAClB,WAAW,EAAE,MAAM,EACnB,WAAW,EAAE,OAAO,EAAE,EACtB,aAAa,EAAE,MAAM,GACpB,MAAM,CAoBR;AAcD;;;GAGG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAOrD;AAED;;;GAGG;AACH,wBAAgB,YAAY,CAAC,MAAM,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,EAAE,SAAS,SAAM,GAAG,OAAO,CAW3F"}

package/dist/lib/conversation-engine.js

@@ -1,84 +1,75 @@
 /**
- * Conversation engine — reusable phase evaluation and advancement.
- * WP-304: Deterministic Shell, Stochastic Core.
+ * Conversation engine — topic evaluation, XML directive builder, post-processing.
+ * WP-304: Coach pattern — LLM judges topic completion, heuristic backup.
  *
- * The engine evaluates exit criteria and decides when to advance.
+ * The engine evaluates topic completion and builds XML directives.
  * It does NOT generate content — that's the LLM's job.
  */
 /**
- * Evaluate whether the current phase is complete.
- * Returns advancement decision based on exit criteria and exchange count.
+ * Minimum user words before topic skipping is allowed.
+ * Below this threshold, the conversation is too thin for quality extraction —
+ * keep asking even if LLM signals say the topic is covered.
  */
-export function evaluatePhase(phases, state) {
-    const currentIndex = phases.findIndex((p) => p.id === state.currentPhaseId);
-    const current = phases[currentIndex];
-    if (!current) {
-        return { complete: true, forced: false, metCriteria: [], unmetCriteria: [], nextPhase: null, readyToExtract: true };
+const MIN_WORDS_FOR_SKIP = 20;
+/**
+ * Check if a topic should be skipped because its signal is already present.
+ * Only skips if we have enough content for quality extraction (>= 20 words).
+ */
+export function shouldSkipTopic(topic, signals, totalUserWords) {
+    if (topic.skipIfSignal === null)
+        return false;
+    if (totalUserWords < MIN_WORDS_FOR_SKIP)
+        return false;
+    return signals[topic.skipIfSignal] === true;
+}
+/**
+ * Evaluate whether the current topic is complete.
+ * Priority: skip > LLM judgment > heuristic backup > forced > continue.
+ */
+export function evaluateTopic(topic, state, llmSaidComplete) {
+    // 1. Skip: signal already present from earlier topics (only if enough content)
+    if (shouldSkipTopic(topic, state.signals, state.totalUserWords)) {
+        return { complete: true, forced: false, skip: true };
     }
-    const exchangesInPhase = state.phaseExchanges[current.id] ?? 0;
-    const met = [];
-    const unmet = [];
-    for (const criterion of current.exitCriteria) {
-        if (criterion.check(state)) {
-            met.push(criterion.name);
-        }
-        else {
-            unmet.push(criterion.name);
-        }
+    const exchanges = state.topicExchanges[topic.id] ?? 0;
+    // 2. LLM says complete (require at least 1 exchange to prevent premature completion)
+    if (llmSaidComplete && exchanges >= 1) {
+        return { complete: true, forced: false, skip: false };
     }
-    // Check if all required criteria are met
-    const allRequiredMet = current.exitCriteria.filter((c) => c.required).every((c) => c.check(state));
-    // All criteria met (or all required met with at least some met) — advance naturally
-    if (unmet.length === 0 || (allRequiredMet && met.length > 0)) {
-        const nextPhase = phases[currentIndex + 1] ?? null;
-        return {
-            complete: true,
-            forced: false,
-            metCriteria: met,
-            unmetCriteria: unmet,
-            nextPhase,
-            readyToExtract: nextPhase === null,
-        };
+    // 3. Heuristic backup: signal detected even if LLM didn't say complete
+    if (topic.heuristicSignal && state.signals[topic.heuristicSignal] === true && exchanges >= 1) {
+        return { complete: true, forced: false, skip: false };
     }
-    // Max exchanges hit — force advancement (but only if required criteria are met or we're past max+1)
-    if (exchangesInPhase >= current.maxExchanges) {
-        if (!allRequiredMet && exchangesInPhase === current.maxExchanges) {
-            // Give one extra turn for required criteria
-            return { complete: false, forced: false, metCriteria: met, unmetCriteria: unmet, nextPhase: null, readyToExtract: false };
-        }
-        const nextPhase = phases[currentIndex + 1] ?? null;
-        return {
-            complete: true,
-            forced: true,
-            metCriteria: met,
-            unmetCriteria: unmet,
-            nextPhase,
-            readyToExtract: nextPhase === null,
-        };
+    // 4. Forced: max exchanges hit
+    if (exchanges >= topic.maxExchanges) {
+        return { complete: true, forced: true, skip: false };
     }
-    // Phase continues
-    return { complete: false, forced: false, metCriteria: met, unmetCriteria: unmet, nextPhase: null, readyToExtract: false };
+    // 5. Continue
+    return { complete: false, forced: false, skip: false };
 }
 /**
- * Build the phase directive to inject into the system prompt.
- * For bridge phase, injects accumulated quotable fragments so the LLM has
- * concrete user phrases to work with instead of re-extracting from history.
+ * Build the XML topic directive to inject into the system prompt.
+ * Renders <topic> with goal/behaviour/criteria, plus <accumulated_context>
+ * with completed Q&A pairs from earlier topics.
  */
-export function buildPhaseDirective(phase, phaseIndex, totalPhases, quotableFragments) {
-    let directive = phase.directive;
-    // Inject accumulated quotable fragments into bridge phase
-    if (phase.id === 'bridge' && quotableFragments && quotableFragments.length > 0) {
-        const fragmentList = quotableFragments
-            .slice(0, 8)
-            .map((f) => `  - "${f}"`)
-            .join('\n');
-        const fragmentBlock = `\nThe user said these things during the conversation — use them as bullet items:\n${fragmentList}\nPrefer these exact phrases over anything you'd summarize yourself.`;
-        directive = directive.replace('{{QUOTABLE_FRAGMENTS}}', fragmentBlock);
-    }
-    else {
-        directive = directive.replace('{{QUOTABLE_FRAGMENTS}}', '');
+export function buildTopicDirective(topic, topicIndex, totalTopics, completedQA, workspaceName) {
+    let xml = `
+<topic id="${topic.id}" index="${topicIndex + 1}" of="${totalTopics}">
+  <goal>${topic.goal}</goal>
+  <behaviour>${topic.behaviour}</behaviour>
+  <criteria>${topic.criteria}</criteria>
+</topic>`;
+    if (completedQA.length > 0) {
+        xml += '\n\n<accumulated_context>';
+        for (const qa of completedQA) {
+            xml += `\n  <topic_qa id="${qa.topicId}">`;
+            xml += `\n    <question>${qa.question}</question>`;
+            xml += `\n    <answer>${qa.answer}</answer>`;
+            xml += '\n  </topic_qa>';
+        }
+        xml += '\n</accumulated_context>';
     }
-    return `\nCURRENT PHASE (${phaseIndex + 1}/${totalPhases}): ${phase.id.toUpperCase()}\n${directive}\nDo NOT summarize, synthesize, or list items unless the directive above explicitly tells you to.`;
+    return xml;
 }
 // ── Post-processing ─────────────────────────────────────────────────────────
 const BANNED_PATTERNS = [
@@ -102,19 +93,6 @@ export function sanitizeResponse(text) {
     // Clean up double spaces and leading punctuation from removals
     return cleaned.replace(/\s{2,}/g, ' ').replace(/^\s*[.,]\s*/, '').trim();
 }
-/**
- * If we're in bridge phase and the response ends with a question, strip it.
- * Bridge is a statement — the LLM should not ask another question.
- */
-export function enforceBridgeIsStatement(text, phaseId) {
-    if (phaseId !== 'bridge')
-        return text;
-    const sentences = text.split(/(?<=[.!?])\s+/);
-    if (sentences.length > 1 && sentences[sentences.length - 1].trim().endsWith('?')) {
-        return sentences.slice(0, -1).join(' ').trim();
-    }
-    return text;
-}
 /**
  * Detect if a response is too similar to recent ones (refusal loop).
  * Uses Jaccard similarity on word sets.

package/dist/lib/conversation-engine.js.map

@@ -1 +1 @@
-{"version":3,"file":"conversation-engine.js","sourceRoot":"","sources":["../../src/lib/conversation-engine.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAyBH;;;GAGG;AACH,MAAM,UAAU,aAAa,CAC3B,MAA2B,EAC3B,KAAwB;IAExB,MAAM,YAAY,GAAG,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,KAAK,CAAC,cAAc,CAAC,CAAC;IAC5E,MAAM,OAAO,GAAG,MAAM,CAAC,YAAY,CAAC,CAAC;IAErC,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,WAAW,EAAE,EAAE,EAAE,aAAa,EAAE,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,cAAc,EAAE,IAAI,EAAE,CAAC;IACtH,CAAC;IAED,MAAM,gBAAgB,GAAG,KAAK,CAAC,cAAc,CAAC,OAAO,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC;IAC/D,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,KAAK,MAAM,SAAS,IAAI,OAAO,CAAC,YAAY,EAAE,CAAC;QAC7C,IAAI,SAAS,CAAC,KAAK,CAAC,KAAK,CAAC,EAAE,CAAC;YAC3B,GAAG,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAC3B,CAAC;aAAM,CAAC;YACN,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAC7B,CAAC;IACH,CAAC;IAED,yCAAyC;IACzC,MAAM,cAAc,GAAG,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC;IAEnG,oFAAoF;IACpF,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,cAAc,IAAI,GAAG,CAAC,MAAM,GAAG,CAAC,CAAC,EAAE,CAAC;QAC7D,MAAM,SAAS,GAAG,MAAM,CAAC,YAAY,GAAG,CAAC,CAAC,IAAI,IAAI,CAAC;QACnD,OAAO;YACL,QAAQ,EAAE,IAAI;YACd,MAAM,EAAE,KAAK;YACb,WAAW,EAAE,GAAG;YAChB,aAAa,EAAE,KAAK;YACpB,SAAS;YACT,cAAc,EAAE,SAAS,KAAK,IAAI;SACnC,CAAC;IACJ,CAAC;IAED,oGAAoG;IACpG,IAAI,gBAAgB,IAAI,OAAO,CAAC,YAAY,EAAE,CAAC;QAC7C,IAAI,CAAC,cAAc,IAAI,gBAAgB,KAAK,OAAO,CAAC,YAAY,EAAE,CAAC;YACjE,4CAA4C;YAC5C,OAAO,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,WAAW,EAAE,GAAG,EAAE,aAAa,EAAE,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,CAAC;QAC5H,CAAC;QAED,MAAM,SAAS,GAAG,MAAM,CAAC,YAAY,GAAG,CAAC,CAAC,IAAI,IAAI,CAAC;QACnD,OAAO;YACL,QAAQ,EAAE,IAAI;YACd,MAAM,EAAE,IAAI;YACZ,WAAW,EAAE,GAAG;YAChB,aAAa,EAAE,KAAK;YACpB,SAAS;YACT,cAAc,EAAE,SAAS,KAAK,IAAI;SACnC,CAAC;IACJ,CAAC;IAED,kBAAkB;IAClB,OAAO,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,WAAW,EAAE,GAAG,EAAE,aAAa,EAAE,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,CAAC;AAC5H,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,mBAAmB,CACjC,KAAwB,EACxB,UAAkB,EAClB,WAAmB,EACnB,iBAA4B;IAE5B,IAAI,SAAS,GAAG,KAAK,CAAC,SAAS,CAAC;IAEhC,0DAA0D;IAC1D,IAAI,KAAK,CAAC,EAAE,KAAK,QAAQ,IAAI,iBAAiB,IAAI,iBAAiB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC/E,MAAM,YAAY,GAAG,iBAAiB;aACnC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC;aACX,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,QAAQ,CAAC,GAAG,CAAC;aACxB,IAAI,CAAC,IAAI,CAAC,CAAC;QACd,MAAM,aAAa,GAAG,qFAAqF,YAAY,sEAAsE,CAAC;QAC9L,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,wBAAwB,EAAE,aAAa,CAAC,CAAC;IACzE,CAAC;SAAM,CAAC;QACN,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,wBAAwB,EAAE,EAAE,CAAC,CAAC;IAC9D,CAAC;IAED,OAAO,oBAAoB,UAAU,GAAG,CAAC,IAAI,WAAW,MAAM,KAAK,CAAC,EAAE,CAAC,WAAW,EAAE,KAAK,SAAS,mGAAmG,CAAC;AACxM,CAAC;AAED,+EAA+E;AAE/E,MAAM,eAAe,GAAG;IACtB,mBAAmB;IACnB,sBAAsB;IACtB,0GAA0G;IAC1G,kBAAkB;IAClB,2EAA2E;IAC3E,2DAA2D;IAC3D,qCAAqC;CACtC,CAAC;AAEF;;;GAGG;AACH,MAAM,UAAU,gBAAgB,CAAC,IAAY;IAC3C,IAAI,OAAO,GAAG,IAAI,CAAC;IACnB,KAAK,MAAM,OAAO,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;IACzC,CAAC;IACD,+DAA+D;IAC/D,OAAO,OAAO,CAAC,OAAO,CAAC,SAAS,EAAE,GAAG,CAAC,CAAC,OAAO,CAAC,aAAa,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;AAC3E,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,wBAAwB,CAAC,IAAY,EAAE,OAAe;IACpE,IAAI,OAAO,KAAK,QAAQ;QAAE,OAAO,IAAI,CAAC;IACtC,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;IAC9C,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,IAAI,SAAS,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;QACjF,OAAO,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;IACjD,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY,CAAC,MAAc,EAAE,UAAoB,EAAE,SAAS,GAAG,GAAG;IAChF,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC;IAC5D,KAAK,MAAM,IAAI,IAAI,UAAU,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QACxC,MAAM,SAAS,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC;QAC3D,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC5E,MAAM,KAAK,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,QAAQ,EAAE,GAAG,SAAS,CAAC,CAAC,CAAC;QACnD,IAAI,KAAK,CAAC,IAAI,GAAG,CAAC,IAAI,YAAY,CAAC,IAAI,GAAG,KAAK,CAAC,IAAI,GAAG,SAAS,EAAE,CAAC;YACjE,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC"}
+{"version":3,"file":"conversation-engine.js","sourceRoot":"","sources":["../../src/lib/conversation-engine.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAiBH;;;;GAIG;AACH,MAAM,kBAAkB,GAAG,EAAE,CAAC;AAE9B;;;GAGG;AACH,MAAM,UAAU,eAAe,CAAC,KAAsB,EAAE,OAA4B,EAAE,cAAsB;IAC1G,IAAI,KAAK,CAAC,YAAY,KAAK,IAAI;QAAE,OAAO,KAAK,CAAC;IAC9C,IAAI,cAAc,GAAG,kBAAkB;QAAE,OAAO,KAAK,CAAC;IACtD,OAAO,OAAO,CAAC,KAAK,CAAC,YAAY,CAAC,KAAK,IAAI,CAAC;AAC9C,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,aAAa,CAC3B,KAAsB,EACtB,KAAwB,EACxB,eAAwB;IAExB,+EAA+E;IAC/E,IAAI,eAAe,CAAC,KAAK,EAAE,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,cAAc,CAAC,EAAE,CAAC;QAChE,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC;IACvD,CAAC;IAED,MAAM,SAAS,GAAG,KAAK,CAAC,cAAc,CAAC,KAAK,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC;IAEtD,qFAAqF;IACrF,IAAI,eAAe,IAAI,SAAS,IAAI,CAAC,EAAE,CAAC;QACtC,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC;IACxD,CAAC;IAED,uEAAuE;IACvE,IAAI,KAAK,CAAC,eAAe,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,eAAe,CAAC,KAAK,IAAI,IAAI,SAAS,IAAI,CAAC,EAAE,CAAC;QAC7F,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC;IACxD,CAAC;IAED,+BAA+B;IAC/B,IAAI,SAAS,IAAI,KAAK,CAAC,YAAY,EAAE,CAAC;QACpC,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC;IACvD,CAAC;IAED,cAAc;IACd,OAAO,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC;AACzD,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,mBAAmB,CACjC,KAAsB,EACtB,UAAkB,EAClB,WAAmB,EACnB,WAAsB,EACtB,aAAqB;IAErB,IAAI,GAAG,GAAG;aACC,KAAK,CAAC,EAAE,YAAY,UAAU,GAAG,CAAC,SAAS,WAAW;UACzD,KAAK,CAAC,IAAI;eACL,KAAK,CAAC,SAAS;cAChB,KAAK,CAAC,QAAQ;SACnB,CAAC;IAER,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC3B,GAAG,IAAI,2BAA2B,CAAC;QACnC,KAAK,MAAM,EAAE,IAAI,WAAW,EAAE,CAAC;YAC7B,GAAG,IAAI,qBAAqB,EAAE,CAAC,OAAO,IAAI,CAAC;YAC3C,GAAG,IAAI,mBAAmB,EAAE,CAAC,QAAQ,aAAa,CAAC;YACnD,GAAG,IAAI,iBAAiB,EAAE,CAAC,MAAM,WAAW,CAAC;YAC7C,GAAG,IAAI,iBAAiB,CAAC;QAC3B,CAAC;QACD,GAAG,IAAI,0BAA0B,CAAC;IACpC,CAAC;IAED,OAAO,GAAG,CAAC;AACb,CAAC;AAED,+EAA+E;AAE/E,MAAM,eAAe,GAAG;IACtB,mBAAmB;IACnB,sBAAsB;IACtB,0GAA0G;IAC1G,kBAAkB;IAClB,2EAA2E;IAC3E,2DAA2D;IAC3D,qCAAqC;CACtC,CAAC;AAEF;;;GAGG;AACH,MAAM,UAAU,gBAAgB,CAAC,IAAY;IAC3C,IAAI,OAAO,GAAG,IAAI,CAAC;IACnB,KAAK,MAAM,OAAO,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;IACzC,CAAC;IACD,+DAA+D;IAC/D,OAAO,OAAO,CAAC,OAAO,CAAC,SAAS,EAAE,GAAG,CAAC,CAAC,OAAO,CAAC,aAAa,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;AAC3E,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY,CAAC,MAAc,EAAE,UAAoB,EAAE,SAAS,GAAG,GAAG;IAChF,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC;IAC5D,KAAK,MAAM,IAAI,IAAI,UAAU,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QACxC,MAAM,SAAS,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC;QAC3D,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC5E,MAAM,KAAK,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,QAAQ,EAAE,GAAG,SAAS,CAAC,CAAC,CAAC;QACnD,IAAI,KAAK,CAAC,IAAI,GAAG,CAAC,IAAI,YAAY,CAAC,IAAI,GAAG,KAAK,CAAC,IAAI,GAAG,SAAS,EAAE,CAAC;YACjE,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC"}

package/dist/lib/conversation-phases.d.ts

@@ -1,40 +1,17 @@
 /**
- * Conversation phase types — reusable phase-gated conversation control.
+ * Conversation types — state and signal types for coach-pattern conversations.
  * WP-304: LLM owns content, CLI owns flow.
  *
  * Pattern: Deterministic Shell, Stochastic Core.
- * - CLI defines phases with exit criteria (deterministic)
- * - LLM generates conversation content within each phase (stochastic)
- * - CLI evaluates criteria and decides when to advance
+ * - CLI defines topics with goal/behaviour/criteria (deterministic)
+ * - LLM generates conversation content and judges criteria (stochastic)
+ * - CLI evaluates with heuristic backup and decides when to advance
  */
-/** A conversation phase with a directive and exit criteria. */
-export interface ConversationPhase {
-    /** Unique phase identifier. */
-    id: string;
-    /**
-     * Injected into the system prompt. Tells the LLM what this phase
-     * is about and what kind of questions to ask.
-     */
-    directive: string;
-    /**
-     * CLI-enforced exit criteria. Required ones must pass to advance.
-     * Non-required ones are advisory — help quality but don't block.
-     */
-    exitCriteria: ExitCriterion[];
-    /**
-     * Max exchanges in this phase before forced advancement.
-     * Prevents getting stuck with uncooperative users.
-     */
-    maxExchanges: number;
-}
-/** An exit criterion — pure function over conversation state. */
-export interface ExitCriterion {
-    /** Human-readable name for debugging. */
-    name: string;
-    /** Returns true when the criterion is met. */
-    check: (state: ConversationState) => boolean;
-    /** If true, forced advancement cannot skip this. */
-    required: boolean;
+/** A completed topic's question-answer pair, accumulated as context for later topics. */
+export interface TopicQA {
+    topicId: string;
+    question: string;
+    answer: string;
 }
 /** Conversation state — the CLI's accumulated view of the conversation. */
 export interface ConversationState {
@@ -47,12 +24,20 @@ export interface ConversationState {
     userMessageCount: number;
     /** Total user words total. */
     totalUserWords: number;
-    /** Exchange count per phase. */
+    /** Exchange count per topic (coach pattern) or phase (legacy). */
+    topicExchanges: Record<string, number>;
+    /** @deprecated Alias for topicExchanges — used by legacy phase engine until Slice 3. */
     phaseExchanges: Record<string, number>;
-    /** Which phases have been completed. */
+    /** Which topics have been completed. */
+    completedTopics: Set<string>;
+    /** @deprecated Alias for completedTopics — used by legacy phase engine until Slice 3. */
     completedPhases: Set<string>;
-    /** Current phase ID. */
+    /** Current topic ID. */
+    currentTopicId: string;
+    /** @deprecated Alias for currentTopicId — used by legacy phase engine until Slice 3. */
     currentPhaseId: string;
+    /** Completed topic Q&A pairs — accumulated context for later topics. */
+    completedQA: TopicQA[];
     /** Heuristic signals detected from user messages. */
     signals: ConversationSignals;
     /** Count of consecutive refusals/non-answers. */
@@ -66,9 +51,9 @@ export interface ConversationSignals {
     mentionedChallenge: boolean;
     /** User mentioned why they're trying Product Brain. */
     mentionedWhyPB: boolean;
-    /** User gave at least one substantive answer (>= 8 words). */
+    /** User gave at least one substantive answer (>= 15 words). */
     hasSubstantiveAnswer: boolean;
-    /** Number of user messages with >= 5 words. */
+    /** Number of user messages with >= 8 words. */
     substantiveMessageCount: number;
 }
 //# sourceMappingURL=conversation-phases.d.ts.map

package/dist/lib/conversation-phases.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"conversation-phases.d.ts","sourceRoot":"","sources":["../../src/lib/conversation-phases.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,+DAA+D;AAC/D,MAAM,WAAW,iBAAiB;IAChC,+BAA+B;IAC/B,EAAE,EAAE,MAAM,CAAC;IAEX;;;OAGG;IACH,SAAS,EAAE,MAAM,CAAC;IAElB;;;OAGG;IACH,YAAY,EAAE,aAAa,EAAE,CAAC;IAE9B;;;OAGG;IACH,YAAY,EAAE,MAAM,CAAC;CACtB;AAED,iEAAiE;AACjE,MAAM,WAAW,aAAa;IAC5B,yCAAyC;IACzC,IAAI,EAAE,MAAM,CAAC;IAEb,8CAA8C;IAC9C,KAAK,EAAE,CAAC,KAAK,EAAE,iBAAiB,KAAK,OAAO,CAAC;IAE7C,oDAAoD;IACpD,QAAQ,EAAE,OAAO,CAAC;CACnB;AAED,2EAA2E;AAC3E,MAAM,WAAW,iBAAiB;IAChC,2BAA2B;IAC3B,QAAQ,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,GAAG,WAAW,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAEjE,qCAAqC;IACrC,gBAAgB,EAAE,MAAM,CAAC;IAEzB,8BAA8B;IAC9B,cAAc,EAAE,MAAM,CAAC;IAEvB,gCAAgC;IAChC,cAAc,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAEvC,wCAAwC;IACxC,eAAe,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IAE7B,wBAAwB;IACxB,cAAc,EAAE,MAAM,CAAC;IAEvB,qDAAqD;IACrD,OAAO,EAAE,mBAAmB,CAAC;IAE7B,iDAAiD;IACjD,aAAa,EAAE,MAAM,CAAC;CACvB;AAED,sEAAsE;AACtE,MAAM,WAAW,mBAAmB;IAClC,4CAA4C;IAC5C,gBAAgB,EAAE,OAAO,CAAC;IAE1B,0DAA0D;IAC1D,kBAAkB,EAAE,OAAO,CAAC;IAE5B,uDAAuD;IACvD,cAAc,EAAE,OAAO,CAAC;IAExB,8DAA8D;IAC9D,oBAAoB,EAAE,OAAO,CAAC;IAE9B,+CAA+C;IAC/C,uBAAuB,EAAE,MAAM,CAAC;CACjC"}
+{"version":3,"file":"conversation-phases.d.ts","sourceRoot":"","sources":["../../src/lib/conversation-phases.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,yFAAyF;AACzF,MAAM,WAAW,OAAO;IACtB,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,2EAA2E;AAC3E,MAAM,WAAW,iBAAiB;IAChC,2BAA2B;IAC3B,QAAQ,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,GAAG,WAAW,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAEjE,qCAAqC;IACrC,gBAAgB,EAAE,MAAM,CAAC;IAEzB,8BAA8B;IAC9B,cAAc,EAAE,MAAM,CAAC;IAEvB,kEAAkE;IAClE,cAAc,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACvC,wFAAwF;IACxF,cAAc,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAEvC,wCAAwC;IACxC,eAAe,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IAC7B,yFAAyF;IACzF,eAAe,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IAE7B,wBAAwB;IACxB,cAAc,EAAE,MAAM,CAAC;IACvB,wFAAwF;IACxF,cAAc,EAAE,MAAM,CAAC;IAEvB,wEAAwE;IACxE,WAAW,EAAE,OAAO,EAAE,CAAC;IAEvB,qDAAqD;IACrD,OAAO,EAAE,mBAAmB,CAAC;IAE7B,iDAAiD;IACjD,aAAa,EAAE,MAAM,CAAC;CACvB;AAED,sEAAsE;AACtE,MAAM,WAAW,mBAAmB;IAClC,4CAA4C;IAC5C,gBAAgB,EAAE,OAAO,CAAC;IAE1B,0DAA0D;IAC1D,kBAAkB,EAAE,OAAO,CAAC;IAE5B,uDAAuD;IACvD,cAAc,EAAE,OAAO,CAAC;IAExB,+DAA+D;IAC/D,oBAAoB,EAAE,OAAO,CAAC;IAE9B,+CAA+C;IAC/C,uBAAuB,EAAE,MAAM,CAAC;CACjC"}

package/dist/lib/conversation-phases.js

@@ -1,11 +1,11 @@
 /**
- * Conversation phase types — reusable phase-gated conversation control.
+ * Conversation types — state and signal types for coach-pattern conversations.
  * WP-304: LLM owns content, CLI owns flow.
  *
  * Pattern: Deterministic Shell, Stochastic Core.
- * - CLI defines phases with exit criteria (deterministic)
- * - LLM generates conversation content within each phase (stochastic)
- * - CLI evaluates criteria and decides when to advance
+ * - CLI defines topics with goal/behaviour/criteria (deterministic)
+ * - LLM generates conversation content and judges criteria (stochastic)
+ * - CLI evaluates with heuristic backup and decides when to advance
  */
 export {};
 //# sourceMappingURL=conversation-phases.js.map

package/dist/lib/conversation-signals.d.ts

@@ -10,6 +10,9 @@ import type { ConversationSignals } from './conversation-phases.js';
 /**
  * Detect signals from user messages using regex heuristics.
  * Returns accumulated signals across all user messages.
+ *
+ * Thresholds are intentionally strict — we'd rather ask one more question
+ * than advance on a vague answer. maxExchanges is the safety valve.
  */
 export declare function detectSignals(messages: Array<{
     role: string;

package/dist/lib/conversation-signals.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"conversation-signals.d.ts","sourceRoot":"","sources":["../../src/lib/conversation-signals.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,0BAA0B,CAAC;AAwBpE;;;GAGG;AACH,wBAAgB,aAAa,CAAC,QAAQ,EAAE,KAAK,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC,GAAG,mBAAmB,CAUrG;AAED;;;GAGG;AACH,wBAAgB,YAAY,CAC1B,SAAS,EAAE,mBAAmB,EAC9B,UAAU,EAAE;IAAE,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAAC,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAAC,aAAa,CAAC,EAAE,OAAO,CAAA;CAAE,GAC9F,mBAAmB,CAOrB"}
+{"version":3,"file":"conversation-signals.d.ts","sourceRoot":"","sources":["../../src/lib/conversation-signals.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,0BAA0B,CAAC;AA4BpE;;;;;;GAMG;AACH,wBAAgB,aAAa,CAAC,QAAQ,EAAE,KAAK,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC,GAAG,mBAAmB,CAgBrG;AAED;;;GAGG;AACH,wBAAgB,YAAY,CAC1B,SAAS,EAAE,mBAAmB,EAC9B,UAAU,EAAE;IAAE,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAAC,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAAC,aAAa,CAAC,EAAE,OAAO,CAAA;CAAE,GAC9F,mBAAmB,CAOrB"}

package/dist/lib/conversation-signals.js

@@ -20,6 +20,10 @@ const WHY_PB_PATTERNS = [
     /\b(?:context|memory|decisions?|knowledge|source of truth|ssot)\b/i,
     /\b(?:ai|llm|claude|cursor|copilot|agent)\b.*\b(?:context|remember|forget|know)\b/i,
     /\b(?:remember|forget|consistent|prompt)\b.*\b(?:ai|llm|agent|tool)\b/i,
+    // Broader: user describes the problem PB solves without naming PB
+    /\b(?:re-?explain|repeat myself|start from (?:zero|scratch)|re-?tell)\b/i,
+    /\b(?:docs?|documentation|md files?)\b.*\b(?:outdated|stale|rot|drift|wrong|maintain)\b/i,
+    /\b(?:cost|waste|spend)\b.*\b(?:tokens?|context|time)\b.*\b(?:re-?explain|repeat|maintain)\b/i,
 ];
 function countWords(text) {
     return text.trim().split(/\s+/).filter(Boolean).length;
@@ -27,15 +31,22 @@ function countWords(text) {
 /**
  * Detect signals from user messages using regex heuristics.
  * Returns accumulated signals across all user messages.
+ *
+ * Thresholds are intentionally strict — we'd rather ask one more question
+ * than advance on a vague answer. maxExchanges is the safety valve.
  */
 export function detectSignals(messages) {
     const userMessages = messages.filter((m) => m.role === 'user');
     return {
-        mentionedProduct: userMessages.some((m) => PRODUCT_PATTERNS.some((p) => p.test(m.content))),
+        // Product mention requires BOTH a pattern match AND enough words to be meaningful.
+        // "a cool app" matches the pattern but isn't substantive — need at least 10 words.
+        mentionedProduct: userMessages.some((m) => PRODUCT_PATTERNS.some((p) => p.test(m.content)) && countWords(m.content) >= 10),
         mentionedChallenge: userMessages.some((m) => CHALLENGE_PATTERNS.some((p) => p.test(m.content))),
         mentionedWhyPB: userMessages.some((m) => WHY_PB_PATTERNS.some((p) => p.test(m.content))),
-        hasSubstantiveAnswer: userMessages.some((m) => countWords(m.content) >= 8),
-        substantiveMessageCount: userMessages.filter((m) => countWords(m.content) >= 5).length,
+        // Substantive = enough detail to extract something meaningful from
+        hasSubstantiveAnswer: userMessages.some((m) => countWords(m.content) >= 15),
+        // Count messages with real content (not just "me" or "yes")
+        substantiveMessageCount: userMessages.filter((m) => countWords(m.content) >= 8).length,
     };
 }
 /**

package/dist/lib/conversation-signals.js.map

@@ -1 +1 @@
-{"version":3,"file":"conversation-signals.js","sourceRoot":"","sources":["../../src/lib/conversation-signals.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,MAAM,gBAAgB,GAAG;IACvB,yDAAyD;IACzD,qEAAqE;CACtE,CAAC;AAEF,MAAM,kBAAkB,GAAG;IACzB,wFAAwF;IACxF,kFAAkF;IAClF,kDAAkD;CACnD,CAAC;AAEF,MAAM,eAAe,GAAG;IACtB,8BAA8B;IAC9B,mEAAmE;IACnE,mFAAmF;IACnF,uEAAuE;CACxE,CAAC;AAEF,SAAS,UAAU,CAAC,IAAY;IAC9B,OAAO,IAAI,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,MAAM,CAAC;AACzD,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,aAAa,CAAC,QAAkD;IAC9E,MAAM,YAAY,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,MAAM,CAAC,CAAC;IAE/D,OAAO;QACL,gBAAgB,EAAE,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC;QAC3F,kBAAkB,EAAE,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,kBAAkB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC;QAC/F,cAAc,EAAE,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC;QACxF,oBAAoB,EAAE,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;QAC1E,uBAAuB,EAAE,YAAY,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM;KACvF,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY,CAC1B,SAA8B,EAC9B,UAA+F;IAE/F,OAAO;QACL,GAAG,SAAS;QACZ,gBAAgB,EAAE,SAAS,CAAC,gBAAgB,IAAI,CAAC,UAAU,CAAC,gBAAgB,IAAI,KAAK,CAAC;QACtF,kBAAkB,EAAE,SAAS,CAAC,kBAAkB,IAAI,CAAC,UAAU,CAAC,gBAAgB,IAAI,KAAK,CAAC;QAC1F,cAAc,EAAE,SAAS,CAAC,cAAc,IAAI,CAAC,UAAU,CAAC,aAAa,IAAI,KAAK,CAAC;KAChF,CAAC;AACJ,CAAC"}
+{"version":3,"file":"conversation-signals.js","sourceRoot":"","sources":["../../src/lib/conversation-signals.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,MAAM,gBAAgB,GAAG;IACvB,yDAAyD;IACzD,qEAAqE;CACtE,CAAC;AAEF,MAAM,kBAAkB,GAAG;IACzB,wFAAwF;IACxF,kFAAkF;IAClF,kDAAkD;CACnD,CAAC;AAEF,MAAM,eAAe,GAAG;IACtB,8BAA8B;IAC9B,mEAAmE;IACnE,mFAAmF;IACnF,uEAAuE;IACvE,kEAAkE;IAClE,yEAAyE;IACzE,yFAAyF;IACzF,8FAA8F;CAC/F,CAAC;AAEF,SAAS,UAAU,CAAC,IAAY;IAC9B,OAAO,IAAI,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,MAAM,CAAC;AACzD,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,aAAa,CAAC,QAAkD;IAC9E,MAAM,YAAY,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,MAAM,CAAC,CAAC;IAE/D,OAAO;QACL,mFAAmF;QACnF,mFAAmF;QACnF,gBAAgB,EAAE,YAAY,CAAC,IAAI,CACjC,CAAC,CAAC,EAAE,EAAE,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,UAAU,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,EAAE,CACtF;QACD,kBAAkB,EAAE,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,kBAAkB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC;QAC/F,cAAc,EAAE,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC;QACxF,mEAAmE;QACnE,oBAAoB,EAAE,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC;QAC3E,4DAA4D;QAC5D,uBAAuB,EAAE,YAAY,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM;KACvF,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY,CAC1B,SAA8B,EAC9B,UAA+F;IAE/F,OAAO;QACL,GAAG,SAAS;QACZ,gBAAgB,EAAE,SAAS,CAAC,gBAAgB,IAAI,CAAC,UAAU,CAAC,gBAAgB,IAAI,KAAK,CAAC;QACtF,kBAAkB,EAAE,SAAS,CAAC,kBAAkB,IAAI,CAAC,UAAU,CAAC,gBAAgB,IAAI,KAAK,CAAC;QAC1F,cAAc,EAAE,SAAS,CAAC,cAAc,IAAI,CAAC,UAAU,CAAC,aAAa,IAAI,KAAK,CAAC;KAChF,CAAC;AACJ,CAAC"}