@productbrain/cli 0.1.0-beta.1093 → 0.1.0-beta.10935

package/dist/lib/runner.d.ts

@@ -2,7 +2,7 @@
  * runCliCommand — shared command runner for the Product Brain CLI.
  *
  * Owns:
- *   (a) TTY detection via process.stdout.isTTY
+ *   (a) auto-mode JSON opt-in via PB_JSON=1 (piping alone does NOT switch modes — WP-744 / 1e-B)
  *   (b) JSON vs human output branching
  *   (c) stripConvexInternals() application in JSON mode
  *   (d) --json and --pretty explicit overrides that take precedence over auto-detection
@@ -15,7 +15,15 @@ export declare function getOutputMode(): 'auto' | 'json' | 'pretty';
 /** Returns true if the current output mode resolves to JSON. */
 export declare function isJsonMode(): boolean;
 export declare function setQuietMode(quiet: boolean): void;
+/**
+ * The RAW quiet flag, independent of JSON mode. isQuietMode() conflates JSON mode,
+ * so read this when you need the literal quiet state (e.g. to assert that a window
+ * left quiet — and therefore the interactive-prompt gate — untouched).
+ */
+export declare function getQuietMode(): boolean;
 export declare function isQuietMode(): boolean;
+export declare function setSpinnersSilenced(silenced: boolean): void;
+export declare function areSpinnersSilenced(): boolean;
 export interface RunCliCommandOptions<T> {
     /** The async function that performs the MCP call and returns data. */
     fn: () => Promise<T>;

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

@@ -1 +1 @@
-{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/lib/runner.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAWH,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,QAAQ,GAAG,IAAI,CAEpE;AAED,wBAAgB,aAAa,IAAI,MAAM,GAAG,MAAM,GAAG,QAAQ,CAE1D;AAED,gEAAgE;AAChE,wBAAgB,UAAU,IAAI,OAAO,CAKpC;AASD,wBAAgB,YAAY,CAAC,KAAK,EAAE,OAAO,GAAG,IAAI,CAEjD;AAED,wBAAgB,WAAW,IAAI,OAAO,CAIrC;AAED,MAAM,WAAW,oBAAoB,CAAC,CAAC;IACrC,sEAAsE;IACtE,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,CAAC;IACrB,kDAAkD;IAClD,YAAY,EAAE,CAAC,IAAI,EAAE,CAAC,KAAK,MAAM,CAAC;CACnC;AAED;;;;;;GAMG;AACH,wBAAsB,aAAa,CAAC,CAAC,EAAE,OAAO,EAAE,oBAAoB,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,CAwBtF"}
+{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/lib/runner.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAWH,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,QAAQ,GAAG,IAAI,CAEpE;AAED,wBAAgB,aAAa,IAAI,MAAM,GAAG,MAAM,GAAG,QAAQ,CAE1D;AAED,gEAAgE;AAChE,wBAAgB,UAAU,IAAI,OAAO,CAQpC;AASD,wBAAgB,YAAY,CAAC,KAAK,EAAE,OAAO,GAAG,IAAI,CAEjD;AAED;;;;GAIG;AACH,wBAAgB,YAAY,IAAI,OAAO,CAEtC;AAED,wBAAgB,WAAW,IAAI,OAAO,CAIrC;AAYD,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,OAAO,GAAG,IAAI,CAE3D;AAED,wBAAgB,mBAAmB,IAAI,OAAO,CAE7C;AAED,MAAM,WAAW,oBAAoB,CAAC,CAAC;IACrC,sEAAsE;IACtE,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,CAAC;IACrB,kDAAkD;IAClD,YAAY,EAAE,CAAC,IAAI,EAAE,CAAC,KAAK,MAAM,CAAC;CACnC;AAED;;;;;;GAMG;AACH,wBAAsB,aAAa,CAAC,CAAC,EAAE,OAAO,EAAE,oBAAoB,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,CAwBtF"}

package/dist/lib/runner.js

@@ -2,7 +2,7 @@
  * runCliCommand — shared command runner for the Product Brain CLI.
  *
  * Owns:
- *   (a) TTY detection via process.stdout.isTTY
+ *   (a) auto-mode JSON opt-in via PB_JSON=1 (piping alone does NOT switch modes — WP-744 / 1e-B)
  *   (b) JSON vs human output branching
  *   (c) stripConvexInternals() application in JSON mode
  *   (d) --json and --pretty explicit overrides that take precedence over auto-detection
@@ -14,7 +14,7 @@ import { stripConvexInternals } from './strip.js';
 import { CLIError } from './errors.js';
 /**
  * Global output mode — set by the CLI entry point when --json or --pretty flags are parsed.
- * 'auto' = detect via TTY; 'json' = force JSON; 'pretty' = force human.
+ * 'auto' = card unless PB_JSON=1; 'json' = force JSON; 'pretty' = force human.
  */
 let outputMode = 'auto';
 export function setOutputMode(mode) {
@@ -29,8 +29,11 @@ export function isJsonMode() {
         return true;
     if (outputMode === 'pretty')
         return false;
-    // auto: non-TTY (piped) → JSON, TTY → human
-    return !process.stdout.isTTY;
+    // auto: only an explicit PB_JSON=1 opts into JSON. Piping alone does NOT — the card is the
+    // default in every shell, TTY or not; `--json` (or PB_JSON=1) is the only way to ask for the
+    // warehouse (WP-744 / 1e-B). Formerly `!process.stdout.isTTY`, which made piped output
+    // indistinguishable from `--json` and left the card unreachable from a pipe.
+    return process.env.PB_JSON === '1';
 }
 /**
  * Quiet mode — suppresses non-essential output (banners, hints, progress).
@@ -41,12 +44,35 @@ let quietMode = false;
 export function setQuietMode(quiet) {
     quietMode = quiet;
 }
+/**
+ * The RAW quiet flag, independent of JSON mode. isQuietMode() conflates JSON mode,
+ * so read this when you need the literal quiet state (e.g. to assert that a window
+ * left quiet — and therefore the interactive-prompt gate — untouched).
+ */
+export function getQuietMode() {
+    return quietMode;
+}
 export function isQuietMode() {
     // JSON mode is inherently quiet — no decorative output needed
     if (isJsonMode())
         return true;
     return quietMode;
 }
+/**
+ * Spinner suppression — independent of quiet/json mode. Silences withSpinner()
+ * output WITHOUT making the CLI "quiet" (quiet also gates interactive prompts via
+ * prompts.ts/assertInteractive). `pb connect` flips this to hide the raw per-kernelCall
+ * spinner labels ("Loading chain getVocabulary", …) while still letting the handshake's
+ * adopt/revert prompts for PB-managed tampered files run (TEN-2561 / WP-2: legible
+ * output must never silence a prompt the user needs to answer).
+ */
+let spinnersSilenced = false;
+export function setSpinnersSilenced(silenced) {
+    spinnersSilenced = silenced;
+}
+export function areSpinnersSilenced() {
+    return spinnersSilenced;
+}
 /**
  * Execute a CLI command with unified output handling.
  *

package/dist/lib/runner.js.map

@@ -1 +1 @@
-{"version":3,"file":"runner.js","sourceRoot":"","sources":["../../src/lib/runner.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,oBAAoB,EAAE,MAAM,YAAY,CAAC;AAClD,OAAO,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAEvC;;;GAGG;AACH,IAAI,UAAU,GAA+B,MAAM,CAAC;AAEpD,MAAM,UAAU,aAAa,CAAC,IAAgC;IAC5D,UAAU,GAAG,IAAI,CAAC;AACpB,CAAC;AAED,MAAM,UAAU,aAAa;IAC3B,OAAO,UAAU,CAAC;AACpB,CAAC;AAED,gEAAgE;AAChE,MAAM,UAAU,UAAU;IACxB,IAAI,UAAU,KAAK,MAAM;QAAE,OAAO,IAAI,CAAC;IACvC,IAAI,UAAU,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAC1C,4CAA4C;IAC5C,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC;AAC/B,CAAC;AAED;;;;GAIG;AACH,IAAI,SAAS,GAAG,KAAK,CAAC;AAEtB,MAAM,UAAU,YAAY,CAAC,KAAc;IACzC,SAAS,GAAG,KAAK,CAAC;AACpB,CAAC;AAED,MAAM,UAAU,WAAW;IACzB,8DAA8D;IAC9D,IAAI,UAAU,EAAE;QAAE,OAAO,IAAI,CAAC;IAC9B,OAAO,SAAS,CAAC;AACnB,CAAC;AASD;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CAAI,OAAgC;IACrE,MAAM,IAAI,GAAG,UAAU,EAAE,CAAC;IAE1B,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,OAAO,CAAC,EAAE,EAAE,CAAC;QAEhC,IAAI,IAAI,EAAE,CAAC;YACT,MAAM,QAAQ,GAAG,oBAAoB,CAAC,IAAI,CAAC,CAAC;YAC5C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,GAAG,IAAI,CAAC,CAAC;QACxD,CAAC;aAAM,CAAC;YACN,MAAM,SAAS,GAAG,OAAO,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC;YAC7C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,SAAS,GAAG,IAAI,CAAC,CAAC;QACzC,CAAC;IACH,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,6EAA6E;QAC7E,+DAA+D;QAC/D,IAAI,GAAG,YAAY,QAAQ,EAAE,CAAC;YAC5B,MAAM,GAAG,CAAC;QACZ,CAAC;QACD,MAAM,IAAI,QAAQ,CAChB,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,EAChD,EAAE,IAAI,EAAE,UAAU,EAAE,QAAQ,EAAE,UAAU,EAAE,CAC3C,CAAC;IACJ,CAAC;AACH,CAAC"}
+{"version":3,"file":"runner.js","sourceRoot":"","sources":["../../src/lib/runner.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,oBAAoB,EAAE,MAAM,YAAY,CAAC;AAClD,OAAO,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAEvC;;;GAGG;AACH,IAAI,UAAU,GAA+B,MAAM,CAAC;AAEpD,MAAM,UAAU,aAAa,CAAC,IAAgC;IAC5D,UAAU,GAAG,IAAI,CAAC;AACpB,CAAC;AAED,MAAM,UAAU,aAAa;IAC3B,OAAO,UAAU,CAAC;AACpB,CAAC;AAED,gEAAgE;AAChE,MAAM,UAAU,UAAU;IACxB,IAAI,UAAU,KAAK,MAAM;QAAE,OAAO,IAAI,CAAC;IACvC,IAAI,UAAU,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAC1C,2FAA2F;IAC3F,6FAA6F;IAC7F,uFAAuF;IACvF,6EAA6E;IAC7E,OAAO,OAAO,CAAC,GAAG,CAAC,OAAO,KAAK,GAAG,CAAC;AACrC,CAAC;AAED;;;;GAIG;AACH,IAAI,SAAS,GAAG,KAAK,CAAC;AAEtB,MAAM,UAAU,YAAY,CAAC,KAAc;IACzC,SAAS,GAAG,KAAK,CAAC;AACpB,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,YAAY;IAC1B,OAAO,SAAS,CAAC;AACnB,CAAC;AAED,MAAM,UAAU,WAAW;IACzB,8DAA8D;IAC9D,IAAI,UAAU,EAAE;QAAE,OAAO,IAAI,CAAC;IAC9B,OAAO,SAAS,CAAC;AACnB,CAAC;AAED;;;;;;;GAOG;AACH,IAAI,gBAAgB,GAAG,KAAK,CAAC;AAE7B,MAAM,UAAU,mBAAmB,CAAC,QAAiB;IACnD,gBAAgB,GAAG,QAAQ,CAAC;AAC9B,CAAC;AAED,MAAM,UAAU,mBAAmB;IACjC,OAAO,gBAAgB,CAAC;AAC1B,CAAC;AASD;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CAAI,OAAgC;IACrE,MAAM,IAAI,GAAG,UAAU,EAAE,CAAC;IAE1B,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,OAAO,CAAC,EAAE,EAAE,CAAC;QAEhC,IAAI,IAAI,EAAE,CAAC;YACT,MAAM,QAAQ,GAAG,oBAAoB,CAAC,IAAI,CAAC,CAAC;YAC5C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,GAAG,IAAI,CAAC,CAAC;QACxD,CAAC;aAAM,CAAC;YACN,MAAM,SAAS,GAAG,OAAO,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC;YAC7C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,SAAS,GAAG,IAAI,CAAC,CAAC;QACzC,CAAC;IACH,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,6EAA6E;QAC7E,+DAA+D;QAC/D,IAAI,GAAG,YAAY,QAAQ,EAAE,CAAC;YAC5B,MAAM,GAAG,CAAC;QACZ,CAAC;QACD,MAAM,IAAI,QAAQ,CAChB,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,EAChD,EAAE,IAAI,EAAE,UAAU,EAAE,QAAQ,EAAE,UAAU,EAAE,CAC3C,CAAC;IACJ,CAAC;AACH,CAAC"}

package/dist/lib/seatSentence.d.ts

@@ -0,0 +1,62 @@
+/**
+ * TASK-633 S3c (WP-808, DEC-1954): the one sentence a seat or unseat reply reads as, on the CLI
+ * (`pb role seat` / `pb role unseat`) and the MCP `role` tool alike. Both doors call the same
+ * kernel routes (`organisation.seatInRole` / `organisation.unseatFromRole`) and render this
+ * sentence from the result, so the two replies cannot drift (RULE-657: one place decides it).
+ * The invite reply (`pb people invite`, the MCP people tool's `invite`; TASK-637) lives here too: one organisation
+ * reply file, one generated copy per connector.
+ *
+ * Pure, no I/O, zero imports: copied into each connector by scripts/generate-mcp-embedded-copies.mjs.
+ * Field sanitization is the caller's (each connector's own `sanitizeSingleLineField`), passed in
+ * as `clean`, the same split ownerLine.ts uses.
+ *
+ * A seat names the person's id after their name: a placeholder has no email, and the id is the
+ * handle `pb role unseat --person` and the MCP `unseat` take (PR #1549 review; TASK-636, DEC-1810).
+ */
+/**
+ * TASK-712 (WP-808, DEC-2007, RULE-664): the one word a person or an agent reads for a person's state: the Rail holder
+ * chip, the Assign rows, a team's members, a role's history line, the seat refusal, the CLI holder tag and the MCP role
+ * and people reads. A person with no login reads Not invited; Draft stays the word for entries, roles and teams waiting
+ * for an accept. Keys are the stored people.status values (convex/lib/constants.ts PERSON_STATUS); the stored value
+ * placeholder is unchanged, so nothing migrates. It lives in this file because this is the organisation words file
+ * already copied into both connectors; change a word here and every surface follows.
+ */
+export declare const PERSON_STATUS_WORDS: {
+    readonly active: "Active";
+    readonly invited: "Invited";
+    readonly placeholder: "Not invited";
+    readonly archived: "Archived";
+};
+export type PersonStatusKey = keyof typeof PERSON_STATUS_WORDS;
+/** The tag beside a person's name: none for an active person (DEC-1810), the state word for every other stored status. */
+export declare function personStatusTag(status: string | null | undefined): string | null;
+export interface SeatReply {
+    outcome: 'applied' | 'refused';
+    created?: boolean;
+    removed?: boolean;
+    person?: string;
+    personId?: string;
+    personStatus?: string;
+    role?: string;
+    roleName?: string;
+    teamName?: string;
+    refusal?: {
+        code: string;
+        reason: string;
+        requiredAction?: string;
+    };
+}
+export declare function formatSeatSentence(action: 'seat' | 'unseat', reply: SeatReply, clean: (value: string) => string): string;
+/** TASK-637 (DEC-1953): the one sentence `pb people invite` and the MCP people tool's `invite` reply with.
+ *  `inPlace` is a placeholder that became invited on its own row, so it keeps the roles it holds. */
+export interface InviteReply {
+    person: string;
+    personId: string;
+    email: string;
+    inPlace: boolean;
+}
+export declare function formatInviteSentence(reply: InviteReply, clean: (value: string) => string): string;
+/** TASK-849: organisation refusals a caller fixes by changing the input (validation, not authority); `pb` and the MCP
+ *  `role` tool both classify by this one list. A census across four sources, not a derived union. */
+export declare const ORGANISATION_VALIDATION_REFUSALS: ReadonlySet<string>;
+//# sourceMappingURL=seatSentence.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"seatSentence.d.ts","sourceRoot":"","sources":["../../src/lib/seatSentence.ts"],"names":[],"mappings":"AAIA;;;;;;;;;;;;;;GAcG;AAEH;;;;;;;GAOG;AACH,eAAO,MAAM,mBAAmB;;;;;CAKtB,CAAC;AAEX,MAAM,MAAM,eAAe,GAAG,MAAM,OAAO,mBAAmB,CAAC;AAE/D,0HAA0H;AAC1H,wBAAgB,eAAe,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,MAAM,GAAG,IAAI,CAGhF;AAED,MAAM,WAAW,SAAS;IACzB,OAAO,EAAE,SAAS,GAAG,SAAS,CAAC;IAC/B,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAC;QAAC,cAAc,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;CACpE;AAED,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,MAAM,GAAG,QAAQ,EAAE,KAAK,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,MAAM,GAAG,MAAM,CAexH;AAED;qGACqG;AACrG,MAAM,WAAW,WAAW;IAC3B,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,OAAO,CAAC;CACjB;AAED,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,WAAW,EAAE,KAAK,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,MAAM,GAAG,MAAM,CAKjG;AAED;qGACqG;AACrG,eAAO,MAAM,gCAAgC,EAAE,WAAW,CAAC,MAAM,CAkB/D,CAAC"}

package/dist/lib/seatSentence.js

@@ -0,0 +1,82 @@
+// GENERATED by scripts/generate-mcp-embedded-copies.mjs from kernel/organisationalAuthority/seatSentence.ts.
+// DO NOT EDIT. Run `npm run mcp:embedded-copies` to regenerate.
+// Local runtime copy for the isolated connector build.
+/**
+ * TASK-633 S3c (WP-808, DEC-1954): the one sentence a seat or unseat reply reads as, on the CLI
+ * (`pb role seat` / `pb role unseat`) and the MCP `role` tool alike. Both doors call the same
+ * kernel routes (`organisation.seatInRole` / `organisation.unseatFromRole`) and render this
+ * sentence from the result, so the two replies cannot drift (RULE-657: one place decides it).
+ * The invite reply (`pb people invite`, the MCP people tool's `invite`; TASK-637) lives here too: one organisation
+ * reply file, one generated copy per connector.
+ *
+ * Pure, no I/O, zero imports: copied into each connector by scripts/generate-mcp-embedded-copies.mjs.
+ * Field sanitization is the caller's (each connector's own `sanitizeSingleLineField`), passed in
+ * as `clean`, the same split ownerLine.ts uses.
+ *
+ * A seat names the person's id after their name: a placeholder has no email, and the id is the
+ * handle `pb role unseat --person` and the MCP `unseat` take (PR #1549 review; TASK-636, DEC-1810).
+ */
+/**
+ * TASK-712 (WP-808, DEC-2007, RULE-664): the one word a person or an agent reads for a person's state: the Rail holder
+ * chip, the Assign rows, a team's members, a role's history line, the seat refusal, the CLI holder tag and the MCP role
+ * and people reads. A person with no login reads Not invited; Draft stays the word for entries, roles and teams waiting
+ * for an accept. Keys are the stored people.status values (convex/lib/constants.ts PERSON_STATUS); the stored value
+ * placeholder is unchanged, so nothing migrates. It lives in this file because this is the organisation words file
+ * already copied into both connectors; change a word here and every surface follows.
+ */
+export const PERSON_STATUS_WORDS = {
+    active: 'Active',
+    invited: 'Invited',
+    placeholder: 'Not invited',
+    archived: 'Archived',
+};
+/** The tag beside a person's name: none for an active person (DEC-1810), the state word for every other stored status. */
+export function personStatusTag(status) {
+    if (!status || status === 'active' || !Object.prototype.hasOwnProperty.call(PERSON_STATUS_WORDS, status))
+        return null;
+    return PERSON_STATUS_WORDS[status];
+}
+export function formatSeatSentence(action, reply, clean) {
+    if (reply.outcome === 'refused') {
+        return `Refused: ${reply.refusal?.reason ?? reply.refusal?.code ?? 'not allowed'}${reply.refusal?.requiredAction ? ` ${reply.refusal.requiredAction}` : ''}`;
+    }
+    const tag = personStatusTag(reply.personStatus);
+    const who = `${clean(reply.person ?? '')}${tag ? ` (${tag})` : ''}`;
+    const roleLabel = `${clean(reply.roleName ?? '')} (${reply.role ?? ''})`;
+    const teamLabel = clean(reply.teamName ?? '');
+    if (action === 'seat') {
+        const id = reply.personId ? ` [${reply.personId}]` : '';
+        return `${reply.created ? 'Seated' : 'Confirmed'} ${who}${id} as ${roleLabel} in ${teamLabel}.`;
+    }
+    return reply.removed
+        ? `Unseated ${who} from ${roleLabel} in ${teamLabel}.`
+        : `${who} did not hold ${roleLabel} in ${teamLabel}; nothing changed.`;
+}
+export function formatInviteSentence(reply, clean) {
+    const who = `${clean(reply.person)} [${reply.personId}]`;
+    return reply.inPlace
+        ? `Invited ${who} at ${clean(reply.email)}; they keep every role they already hold.`
+        : `Invited ${who} at ${clean(reply.email)} as a new person.`;
+}
+/** TASK-849: organisation refusals a caller fixes by changing the input (validation, not authority); `pb` and the MCP
+ *  `role` tool both classify by this one list. A census across four sources, not a derived union. */
+export const ORGANISATION_VALIDATION_REFUSALS = new Set([
+    'duplicate_membership',
+    'last_team_lead',
+    'child_team_invalid',
+    'parent_team_invalid',
+    'existing_parent_invalid',
+    'hierarchy_ambiguous',
+    'self_parent',
+    'hierarchy_cycle',
+    'hierarchy_too_deep',
+    'hierarchy_too_large',
+    'hierarchy_repair_required',
+    'archived_parent_authority_invalid',
+    'role_still_held',
+    'move_not_needed',
+    'no_pending_move',
+    'why_required',
+    'role_unplaced',
+]);
+//# sourceMappingURL=seatSentence.js.map

package/dist/lib/seatSentence.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"seatSentence.js","sourceRoot":"","sources":["../../src/lib/seatSentence.ts"],"names":[],"mappings":"AAAA,6GAA6G;AAC7G,gEAAgE;AAChE,uDAAuD;AAEvD;;;;;;;;;;;;;;GAcG;AAEH;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG;IAClC,MAAM,EAAE,QAAQ;IAChB,OAAO,EAAE,SAAS;IAClB,WAAW,EAAE,aAAa;IAC1B,QAAQ,EAAE,UAAU;CACX,CAAC;AAIX,0HAA0H;AAC1H,MAAM,UAAU,eAAe,CAAC,MAAiC;IAChE,IAAI,CAAC,MAAM,IAAI,MAAM,KAAK,QAAQ,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,cAAc,CAAC,IAAI,CAAC,mBAAmB,EAAE,MAAM,CAAC;QAAE,OAAO,IAAI,CAAC;IACtH,OAAO,mBAAmB,CAAC,MAAyB,CAAC,CAAC;AACvD,CAAC;AAeD,MAAM,UAAU,kBAAkB,CAAC,MAAyB,EAAE,KAAgB,EAAE,KAAgC;IAC/G,IAAI,KAAK,CAAC,OAAO,KAAK,SAAS,EAAE,CAAC;QACjC,OAAO,YAAY,KAAK,CAAC,OAAO,EAAE,MAAM,IAAI,KAAK,CAAC,OAAO,EAAE,IAAI,IAAI,aAAa,GAAG,KAAK,CAAC,OAAO,EAAE,cAAc,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,OAAO,CAAC,cAAc,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC;IAC9J,CAAC;IACD,MAAM,GAAG,GAAG,eAAe,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC;IAChD,MAAM,GAAG,GAAG,GAAG,KAAK,CAAC,KAAK,CAAC,MAAM,IAAI,EAAE,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC,KAAK,GAAG,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC;IACpE,MAAM,SAAS,GAAG,GAAG,KAAK,CAAC,KAAK,CAAC,QAAQ,IAAI,EAAE,CAAC,KAAK,KAAK,CAAC,IAAI,IAAI,EAAE,GAAG,CAAC;IACzE,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC;IAC9C,IAAI,MAAM,KAAK,MAAM,EAAE,CAAC;QACvB,MAAM,EAAE,GAAG,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,KAAK,KAAK,CAAC,QAAQ,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;QACxD,OAAO,GAAG,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,WAAW,IAAI,GAAG,GAAG,EAAE,OAAO,SAAS,OAAO,SAAS,GAAG,CAAC;IACjG,CAAC;IACD,OAAO,KAAK,CAAC,OAAO;QACnB,CAAC,CAAC,YAAY,GAAG,SAAS,SAAS,OAAO,SAAS,GAAG;QACtD,CAAC,CAAC,GAAG,GAAG,iBAAiB,SAAS,OAAO,SAAS,oBAAoB,CAAC;AACzE,CAAC;AAWD,MAAM,UAAU,oBAAoB,CAAC,KAAkB,EAAE,KAAgC;IACxF,MAAM,GAAG,GAAG,GAAG,KAAK,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,KAAK,CAAC,QAAQ,GAAG,CAAC;IACzD,OAAO,KAAK,CAAC,OAAO;QACnB,CAAC,CAAC,WAAW,GAAG,OAAO,KAAK,CAAC,KAAK,CAAC,KAAK,CAAC,2CAA2C;QACpF,CAAC,CAAC,WAAW,GAAG,OAAO,KAAK,CAAC,KAAK,CAAC,KAAK,CAAC,mBAAmB,CAAC;AAC/D,CAAC;AAED;qGACqG;AACrG,MAAM,CAAC,MAAM,gCAAgC,GAAwB,IAAI,GAAG,CAAC;IAC5E,sBAAsB;IACtB,gBAAgB;IAChB,oBAAoB;IACpB,qBAAqB;IACrB,yBAAyB;IACzB,qBAAqB;IACrB,aAAa;IACb,iBAAiB;IACjB,oBAAoB;IACpB,qBAAqB;IACrB,2BAA2B;IAC3B,mCAAmC;IACnC,iBAAiB;IACjB,iBAAiB;IACjB,iBAAiB;IACjB,cAAc;IACd,eAAe;CACf,CAAC,CAAC"}

package/dist/lib/session.d.ts

@@ -5,6 +5,17 @@
  *
  * WP-322 S2: session.json also serves as a closed marker (closedAt present)
  * for auto-resume logic when the last session closed <2h ago.
+ *
+ * WP-479 E2 (client half): when a conversation identity resolves (see ./conversation.ts —
+ * PB_CONVERSATION_ID / CLAUDE_CODE_SESSION_ID / CURSOR_CONVERSATION_ID / hashed
+ * CODEX_TUI_SESSION_LOG_PATH), the marker moves to `.productbrain/sessions/<conversationId>.json`
+ * so concurrent harness conversations sharing a worktree don't collide on one marker. Every
+ * function below resolves the conversation id ONCE (via resolveConversationId) and derives its
+ * path from that single value. With no identity signal present, resolveConversationId() returns
+ * null and every function falls back to the exact legacy `.productbrain/session.json` path and
+ * behavior — byte-identical, zero-config invariant. An explicit `conversationIdOverride` (only
+ * threaded from the CLI's `--conversation` flag on `pb session start`) takes priority over env
+ * detection; every other call site across the CLI calls these with no argument.
  */
 export interface SessionState {
     sessionId: string;
@@ -16,16 +27,37 @@ export interface SessionState {
     closedAt?: string;
     /** Effective profile name at session start — backstop for detecting a profile switch. */
     profile?: string;
+    /**
+     * Fingerprint of the CREDENTIAL this session was started under (TEN-3240).
+     *
+     * `profile` alone cannot see a raw exported `PRODUCTBRAIN_API_KEY`: it names profiles, and an
+     * exported key supplies no profile name. Swap that key for another workspace's between two
+     * invocations and the name is unchanged, so the name-only backstop reports the OLD workspace's
+     * session as still active. This is derived from the resolved key + gateway, so it moves
+     * whenever the binding does — profile switch, preview flip, or bare key swap alike.
+     *
+     * Optional: sessions written before this field existed carry only `profile`, and the backstop
+     * still honours that. Never the key itself — a one-way digest, and only its head is kept.
+     */
+    bindingFingerprint?: string;
+    /** WP-479 E2: the conversation identity this marker is scoped to, when one resolved. */
+    conversationId?: string;
 }
+/** 2-hour window for auto-resume (WP-322 E1) — also the GC threshold for closed conversation
+ *  markers (WP-479 E2): a closed marker past this window will never be auto-resumed, so it's
+ *  garbage. Exported so commands/session.ts's resume logic uses the exact same constant. */
+export declare const RESUME_WINDOW_MS: number;
 /** Shape returned by readClosedMarker when a closed marker is present. */
 export interface ClosedMarker {
     sessionId: string;
     closedAt: string;
     workspaceId: string;
     workspaceName: string;
+    /** WP-479 E2: the conversation identity this marker was scoped to, when one resolved. */
+    conversationId?: string;
 }
-export declare function readSession(): SessionState | null;
-export declare function writeSession(state: SessionState): void;
+export declare function readSession(conversationIdOverride?: string): SessionState | null;
+export declare function writeSession(state: SessionState, conversationIdOverride?: string): void;
 /**
  * Writes a closed marker instead of deleting the file.
  * If the file is already a closed marker (closedAt present), deletes it outright
@@ -33,20 +65,62 @@ export declare function writeSession(state: SessionState): void;
  * Uses atomic write (tmp → rename) to prevent partial writes on active sessions.
  * WP-322 S2 — enables auto-resume within 2h window.
  */
-export declare function clearSession(): void;
+export declare function clearSession(conversationIdOverride?: string): void;
+/**
+ * Delete the CURRENT conversation's marker outright — NO closed marker. Use when discarding a
+ * single conversation's session (clearSession writes a resume marker; that is only correct for
+ * session close).
+ */
+export declare function discardSession(conversationIdOverride?: string): void;
+/**
+ * Delete EVERY session marker in this project — the legacy `session.json` AND every per-conversation
+ * marker under `.productbrain/sessions/`. Use on a WORKSPACE change (profile switch / connect
+ * redirect) so the 2h auto-resume can't re-pin the old workspace from ANY marker.
+ *
+ * Review fix (Codex): `discardSession()` resolves only the CURRENT conversation's marker, so a
+ * workspace-switch caller with no conversation env would leave other conversations' active markers
+ * behind — later reusable against the NEW workspace. A workspace change invalidates ALL of them, so
+ * this clears the whole set rather than one marker.
+ */
+export declare function discardAllSessions(): void;
+/** True if a session marker file exists at all — active session OR closed marker. */
+export declare function sessionFileExists(conversationIdOverride?: string): boolean;
 /**
- * Delete session.json outright — NO closed marker. Use on a workspace change
- * (profile switch / connect redirect) so the 2h auto-resume can't re-pin the old
- * workspace. (clearSession writes a resume marker; that is only correct for session close.)
+ * True if ANY session marker exists — the legacy `session.json` OR any per-conversation marker under
+ * `.productbrain/sessions/`. Used by `pb session reset` so a hard reset (and the binding-flip recovery
+ * it backs) can detect + clear a sibling conversation's marker it has no env to resolve (Codex
+ * re-review — the flip guard refuses on ANY active marker, so reset must see them ALL).
+ */
+export declare function hasAnySessionMarker(): boolean;
+/**
+ * True if ANY marker — legacy `session.json` OR any per-conversation marker — represents an ACTIVE
+ * session (no `closedAt`). Used by `pb session reset` so its confirmation/`--force` guard fires when
+ * a SIBLING conversation's session is active, not only the current one (Codex re-review: an
+ * all-marker reset must not silently strand an active sibling server session). Closed markers are
+ * inert and don't require a prompt.
+ */
+export declare function hasAnyActiveSession(): boolean;
+/**
+ * Find the ACTIVE marker carrying `sessionId`, across both storage shapes, or null.
+ *
+ * TEN-3315: the write-binding guard compares the bound credential against a marker's fingerprint,
+ * but a command can select a DIFFERENT conversation's session and pass its id explicitly (e.g.
+ * `pb rework report --conversation A`). Judging the ambient marker there would judge the wrong
+ * session, so the guard used to skip — blind exactly where the caller was most deliberate about
+ * which session it meant. This lets it judge the marker the call actually uses.
+ *
+ * Enumerates the same two places `hasAnyActiveSession()` does — the legacy `session.json` and the
+ * per-conversation directory — because a session id says nothing about which conversation owns it.
+ * Closed markers are not sessions, so they never match; a malformed one is skipped rather than
+ * throwing, since this sits in front of every session-attributed write.
  */
-export declare function discardSession(): void;
-/** True if a session.json file exists at all — active session OR closed marker. */
-export declare function sessionFileExists(): boolean;
+export declare function readSessionBySessionId(sessionId: string): SessionState | null;
 /**
- * Reads the closed marker from session.json.
+ * Reads the closed marker from the resolved conversation's marker file (or the legacy
+ * session.json when no conversation identity resolves).
  * Returns the marker if closedAt is present, otherwise null.
  * WP-322 S2 — used by runSessionStart to detect recent closed sessions.
  */
-export declare function readClosedMarker(): ClosedMarker | null;
-export declare function addCapturedEntry(entryId: string): void;
+export declare function readClosedMarker(conversationIdOverride?: string): ClosedMarker | null;
+export declare function addCapturedEntry(entryId: string, conversationIdOverride?: string): void;
 //# sourceMappingURL=session.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"session.d.ts","sourceRoot":"","sources":["../../src/lib/session.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAMH,MAAM,WAAW,YAAY;IAC3B,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC;IAClB,eAAe,EAAE,MAAM,EAAE,CAAC;IAC1B,sFAAsF;IACtF,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,yFAAyF;IACzF,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAED,0EAA0E;AAC1E,MAAM,WAAW,YAAY;IAC3B,SAAS,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;CACvB;AAwBD,wBAAgB,WAAW,IAAI,YAAY,GAAG,IAAI,CAWjD;AAED,wBAAgB,YAAY,CAAC,KAAK,EAAE,YAAY,GAAG,IAAI,CAItD;AAED;;;;;;GAMG;AACH,wBAAgB,YAAY,IAAI,IAAI,CA0BnC;AAED;;;;GAIG;AACH,wBAAgB,cAAc,IAAI,IAAI,CAGrC;AAED,mFAAmF;AACnF,wBAAgB,iBAAiB,IAAI,OAAO,CAE3C;AAED;;;;GAIG;AACH,wBAAgB,gBAAgB,IAAI,YAAY,GAAG,IAAI,CAetD;AAED,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAKtD"}
+{"version":3,"file":"session.d.ts","sourceRoot":"","sources":["../../src/lib/session.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAOH,MAAM,WAAW,YAAY;IAC3B,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC;IAClB,eAAe,EAAE,MAAM,EAAE,CAAC;IAC1B,sFAAsF;IACtF,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,yFAAyF;IACzF,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB;;;;;;;;;;;OAWG;IACH,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,wFAAwF;IACxF,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAED;;4FAE4F;AAC5F,eAAO,MAAM,gBAAgB,QAAqB,CAAC;AAEnD,0EAA0E;AAC1E,MAAM,WAAW,YAAY;IAC3B,SAAS,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;IACtB,yFAAyF;IACzF,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAiFD,wBAAgB,WAAW,CAAC,sBAAsB,CAAC,EAAE,MAAM,GAAG,YAAY,GAAG,IAAI,CAahF;AAED,wBAAgB,YAAY,CAAC,KAAK,EAAE,YAAY,EAAE,sBAAsB,CAAC,EAAE,MAAM,GAAG,IAAI,CAUvF;AAED;;;;;;GAMG;AACH,wBAAgB,YAAY,CAAC,sBAAsB,CAAC,EAAE,MAAM,GAAG,IAAI,CA6BlE;AAED;;;;GAIG;AACH,wBAAgB,cAAc,CAAC,sBAAsB,CAAC,EAAE,MAAM,GAAG,IAAI,CAIpE;AAED;;;;;;;;;GASG;AACH,wBAAgB,kBAAkB,IAAI,IAAI,CAmBzC;AAED,qFAAqF;AACrF,wBAAgB,iBAAiB,CAAC,sBAAsB,CAAC,EAAE,MAAM,GAAG,OAAO,CAG1E;AAED;;;;;GAKG;AACH,wBAAgB,mBAAmB,IAAI,OAAO,CAS7C;AAED;;;;;;GAMG;AACH,wBAAgB,mBAAmB,IAAI,OAAO,CAkB7C;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,sBAAsB,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY,GAAG,IAAI,CA0B7E;AAED;;;;;GAKG;AACH,wBAAgB,gBAAgB,CAAC,sBAAsB,CAAC,EAAE,MAAM,GAAG,YAAY,GAAG,IAAI,CAkBrF;AAED,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,MAAM,EAAE,sBAAsB,CAAC,EAAE,MAAM,GAAG,IAAI,CAKvF"}