pipebrain-mcp 0.42.0 → 0.43.0

package/dist/main.js

@@ -507,6 +507,19 @@ function apiBindingLabel(apiBaseUrl) {
 // ../../libs/agent-shared/dist/doctrine.js
 var DOCTRINE = "# Pipebrain doctrine\n\nHow an agent works inside Pipebrain. These rules ship with the product and read the same in every organization, because each exists where disagreeing breaks something the product guarantees rather than merely changing house style.\n\nHow your organization *writes* is its own text, edited in Settings, and follows this document under a heading naming each scope.\n\n## Decisions are yours; escalate on consequence\n\nInside an approved requirement you choose the design, the libraries, the structure, and the sequencing. Do not ask, report, or seek ratification for the choice. Escalate on consequence, not uncertainty: a choice that changes what a user sees, costs money, or contradicts an approved requirement is filed as a question with the options you propose; everything else you decide and carry on. A technical choice worth keeping is a decision record, never a paragraph.\n\n## Every loose end has one destination, and none of them is prose\n\nAnything you notice that is not the work in front of you goes to exactly one of three places: a question, when the operator has to choose; a new issue at triage, when it is real work outside this item's scope; or nowhere. Nowhere is the default, and the right outcome whenever knowing the thing would change nothing anyone does. Never open a section called notes, caveats, observations, things to know, or next steps in a body or a comment. Filing is the report: an item recorded and then repeated in prose asks the operator to triage it twice.\n\n## A decision you need is filed as a question, never written into a body\n\nAn open decision belongs in a question on the issue, filed with `ask_questions`, carrying the options you propose. It does not belong in a body \u2014 not under a heading, and not in a field. A decision written into prose prompts nobody and records no answer, so the operator has no place to make it and a later session cannot tell whether it ever was. Retract one your own work has since settled, so nothing asks the operator to decide something nobody waits on.\n\n## Search before you create\n\nLook for the existing record before adding one. A duplicate splits one thing's history across two identifiers, and the split is permanent \u2014 nothing merges them afterward.\n\n## Link by key; never paste another record's text\n\nReference a requirement, a decision, or an issue by its key, with its title on first mention. Never paste its text into a body. The source changes and the copy does not, and the stale copy is what gets built.\n\n## Cross-cutting requirements arrive on their own\n\nThey bind every epic automatically and come back with a work-item read, at their latest approved revision. Never restate them in a body and never add them as explicit requirement links. An explicit link pins a document at an approved revision, which is a different act.\n\n## Conventions arrive in force; yours is a proposal\n\nApproved conventions arrive with the same work-item read, at their latest approved revision, and they bind work in every project in the organization, not this one alone. Neither restate one in a body nor add one as an explicit link: delivery is already automatic, so a copy is a second version waiting to drift.\n\nOne you create or amend over the agent surface records a proposal. It is not in force until a person approves it in a browser, and only an approved revision reaches any session \u2014 so in the interval, do not work from your own unapproved wording, because adopting a rule for the organization is a person's act, not yours.\n\nThe tier test is a statement's lifetime: one that stops being true when this project ends is a cross-cutting requirement, and one that would still be true on the next project is a convention. The moments worth capturing are an evaluator rejecting work, a criterion moving to failed, and a reason recorded on blocked work. File at moments like those and rarely otherwise: every approved convention is delivered on every read, and it spends a person's review attention before it gets there.\n\n## Verification modes follow what can be driven\n\nBefore assigning a mode, establish what this project can drive end to end: a harness, and the command that runs it. A criterion is `auto` when an evaluator can drive the input and read the result against the running system, including behavior shown in an interface wherever such a harness exists. What qualifies it is the evaluator's own run, never the existence of a test the implementer wrote.\n\nMark `manual` only when the verdict needs a person's judgment. A criterion you cannot describe as drivable is usually worded too vaguely: reword it against the observable rather than fall back to a person. A `manual` criterion carries instructions for whoever checks it by hand, written for someone who has not seen the code or the conversation that produced it.\n\nA manual criterion carrying no recorded artifact is the human gate working as designed rather than a coverage gap, so it is neither a finding nor weaker than one carrying evidence.\n\n## The bar for filing a bug\n\nFile a bug only when the observed behavior breaks an approved requirement or a defined acceptance criterion, and name which one before you file. Failing that, it is not a bug: code you find unclear, an edge case nobody specified, a performance number nobody set, and anything you would phrase as \"this could be better\" are opinions about the product, so file each as feedback or not at all. The bar exists because a queue holding both real defects and unshared taste has to be read entirely before any of it can be trusted.\n\n## A decision record is written once\n\nIt states what was decided and why, at the time it was decided, and is not edited as understanding moves. A superseding record replaces it, and the chain of the two is the account of how the thinking changed \u2014 which an edited record destroys. Read the existing records before re-opening a settled question.\n\n## A secret value never travels in a tool call\n\nNo tool here returns one, and that is deliberate rather than an omission to be fixed. Nor should you put one in: a value typed into a tool call is in the transcript before the server can encrypt it, so it is already disclosed and the only remedy left is rotation. Set values with the command-line tool, from a file or on standard input, and inject them at run time rather than reading them into context.\n\n## Setting an instruction replaces the whole body\n\nThere is no partial update. Writing an instruction stores exactly the body you supply and discards everything that was there, so a partial write destroys the rest of the text. The procedure is: read the live value, compose the complete replacement, write it, then read the live value again to confirm what landed.\n\nWhat you are reading now \u2014 the composed guidance \u2014 is a snapshot taken when this session started, and so are the tool descriptions. An edit reaches the next session's guidance, never this one's, so this document re-read after a write still shows the pre-edit text. The stored rows are a different read: `list_org_instructions` returns the current saved values mid-session, which is what makes the confirmation read above possible. An agent that reports its write as absent has re-read the snapshot instead of the record.\n";
 
+// ../../libs/agent-shared/dist/org-reference.js
+var CANONICAL_UUID_RE = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+async function resolveOrgReference(reference, listOrgs) {
+  if (CANONICAL_UUID_RE.test(reference))
+    return reference;
+  const orgs = await listOrgs();
+  const match = orgs.find((org) => org.slug === reference);
+  if (match !== void 0)
+    return match.id;
+  const available = orgs.map((org) => org.slug).sort();
+  throw new Error(available.length === 0 ? `no organization is named "${reference}", and this credential can see none at all.` : `no organization is named "${reference}" \u2014 you are a member of: ${available.join(", ")}.`);
+}
+
 // ../../libs/agent-shared/dist/resolve-agent-env.js
 function pick(env, name) {
   const value = env[name];
@@ -515,7 +528,8 @@ function pick(env, name) {
 function resolveAgentEnv(env) {
   return {
     apiUrl: pick(env, "PIPEBRAIN_API_URL"),
-    token: pick(env, "PIPEBRAIN_TOKEN")
+    token: pick(env, "PIPEBRAIN_TOKEN"),
+    org: pick(env, "PIPEBRAIN_ORG")
   };
 }
 
@@ -15820,12 +15834,14 @@ var commentSchema = external_exports.object({
   updatedAt: external_exports.iso.datetime({ offset: true }),
   deletedAt: external_exports.iso.datetime({ offset: true }).nullable()
 });
+var apiTokenKindSchema = external_exports.enum(["connector", "manual"]);
 var apiTokenSchema = external_exports.object({
   id: uuidSchema,
   orgId: uuidSchema,
   userId: uuidSchema,
   label: external_exports.string().min(1),
   tokenPrefix: tokenPrefixSchema,
+  kind: apiTokenKindSchema,
   lastUsedAt: external_exports.iso.datetime({ offset: true }).nullable(),
   revokedAt: external_exports.iso.datetime({ offset: true }).nullable(),
   createdAt: external_exports.iso.datetime({ offset: true })
@@ -16589,7 +16605,14 @@ var ApiClient = class _ApiClient {
    * leave the header pointing at the old org (the cross-org cache-bleed and
    * post-leave `/login` lockout root cause). Org-INDEPENDENT calls (`me`/`listOrgs`/
    * `createOrg`/the invitation inbox + by-token routes) stay on the base client,
-   * which sends NO header; bearer callers (MCP) ignore the header entirely.
+   * which sends NO header.
+   *
+   * A BEARER CALLER USES IT TOO, and it is honoured there (PIPE-410): this is how
+   * `pipebrain-mcp` scopes a session to `PIPEBRAIN_ORG`, how it re-scopes one call that
+   * names another organization, and how `ppb` scopes an invocation to `--org`. The same
+   * per-call binding is what makes that safe under the MCP SDK's unqueued dispatch — two
+   * tool calls naming two organizations are genuinely in flight together, and a shared
+   * mutable would place one call's write through the other's client.
    */
   forOrg(orgId) {
     return new _ApiClient({
@@ -17133,9 +17156,13 @@ var ApiClient = class _ApiClient {
    * (PIPE-87): revisions awaiting approval, questions awaiting an answer, and work
    * in `verify` with a criterion not yet passed, longest wait first.
    *
-   * Takes NO project and NO org: a queue you point at a project is the per-project
-   * surface this replaces, and org scope is the client's own (`X-Org-Id` for a
-   * session, the token's org for a bearer caller). `mine` narrows to the gates whose
+   * Takes NO project and NO org PARAMETER: a queue you point at a project is the
+   * per-project surface this replaces, and the organization is a property of the CLIENT,
+   * which is where every caller already binds it — `X-Org-Id` for a session and for a
+   * bearer caller alike, falling back to the credential's own organization when nothing
+   * stamped one (PIPE-410 withdrew the rule that a bearer call's organization came off
+   * the token). The agent tool DOES take an `org`, and it is spent on `forOrg` before it
+   * gets here. `mine` narrows to the gates whose
    * work the viewer claimed — plus the requirement gates, which have no claimant at
    * all and would otherwise be waited on by nobody; omitting it is the wide view.
    */
@@ -17417,6 +17444,7 @@ function safeJsonParse(text) {
 }
 
 // ../../libs/mcp-server/dist/server.js
+import { AsyncLocalStorage } from "node:async_hooks";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 
 // ../../libs/mcp-server/dist/instructions.js
@@ -17541,10 +17569,22 @@ async function resolveDeliverableTargetId(client2, projectKey, ref) {
   }
   return match.id;
 }
-function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */ new Map(), opts = {}) {
+var ORG_ARGUMENT_DESCRIPTION = "The organization this call acts in \u2014 its slug or its uuid. Omit it to act in this session\u2019s default.";
+var orgArgumentSchema = external_exports.string().min(1).describe(ORG_ARGUMENT_DESCRIPTION).optional();
+function createMcpServer(baseClient, apiBaseUrl, orgInstructions2 = /* @__PURE__ */ new Map(), opts = {}) {
   const surface = opts.surface ?? "local";
+  const scope = new AsyncLocalStorage();
+  function client2() {
+    return scope.getStore()?.client ?? baseClient;
+  }
   const banner = bindingBanner(apiBaseUrl);
   const guidance = composeGuidance(banner, orgInstructions2);
+  async function callGuidance() {
+    const fetchFor = opts.orgInstructionsFor;
+    if (fetchFor === void 0)
+      return guidance;
+    return composeGuidance(banner, await fetchFor(scope.getStore()?.orgId));
+  }
   const server2 = new McpServer({ name: "pipebrain", version: opts.version ?? DEFAULT_VERSION }, {
     // A BOUNDED INDEX, not a document container (PIPE-D30). The client truncates this
     // field at 2KB and shares roughly 4KB across all configured servers, so what goes
@@ -17562,15 +17602,31 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
   });
   function registerStrict(name, config3, cb) {
     const declared = config3.inputSchema;
-    const inputSchema = declared === void 0 ? external_exports.strictObject({}) : "_zod" in declared || "_def" in declared ? declared : external_exports.strictObject(declared);
+    const strict = declared === void 0 ? external_exports.strictObject({}) : "_zod" in declared || "_def" in declared ? declared : external_exports.strictObject(declared);
+    const inputSchema = strict.extend({ org: orgArgumentSchema });
     server2.registerTool(
       name,
       { ...config3, inputSchema },
       // The cast retells what the generic already guarantees: the SDK calls the
       // handler with the parse result of the schema derived from `InputArgs`, and a
       // strict wrap changes which calls REACH the handler, never the shape of one
-      // that does.
-      cb
+      // that does. The wrapper below hands `cb` the same parse result minus `org`,
+      // which no handler declared and none can see.
+      (async (raw, extra) => {
+        const { org, ...args } = raw;
+        const handler = cb;
+        if (org === void 0)
+          return handler(args, extra);
+        let scoped;
+        let orgId;
+        try {
+          orgId = await resolveOrgReference(org, () => baseClient.listOrgs());
+          scoped = baseClient.forOrg(orgId);
+        } catch (err) {
+          return toolError(err);
+        }
+        return scope.run({ client: scoped, orgId }, () => handler(args, extra));
+      })
     );
   }
   registerStrict("list_projects", {
@@ -17581,24 +17637,25 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       limit: external_exports.number().int().min(1).max(500).optional(),
       offset: external_exports.number().int().nonnegative().optional()
     }
-  }, (args) => run(() => client2.listProjects(args), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().listProjects(args), GUIDANCE_POINTER));
   registerStrict("create_project", {
     description: "Create a new project \u2014 the workflow bootstrap entry point, since issues and requirements are all project-scoped and nothing can be filed until a project exists. Both `name` (its display label) and `key` are required; the key is 2\u20135 uppercase letters (A\u2013Z, e.g. `DWB`), unique per org, and becomes the prefix of every display id in the project. Returns the created project (\xA77).",
     // Inline shape: @pipeline/domain has no standalone create-project schema — the
     // same `{ name, key }` pair lives only inside createOrgInputSchema.project, so
     // this mirrors it exactly (key = 2–5 A–Z via projectKeySchema) and matches
-    // client.createProject's `{ name, key }` contract.
+    // `ApiClient.createProject`'s `{ name, key }` contract.
     inputSchema: {
       name: external_exports.string().min(1),
       key: projectKeySchema
     }
-  }, (args) => run(() => client2.createProject(args)));
+  }, (args) => run(() => client2().createProject(args)));
   registerStrict("list_waiting", {
-    description: "Every gate awaiting a PERSON across ALL projects in the token\u2019s org, longest wait first: a requirement revision awaiting approval, a question awaiting an answer, and work in `verify` holding at least one criterion not yet passed. Each entry names its project, the ref the action is taken on (`KEY-R{n}` for a revision, the ISSUE\u2019s `KEY-{n}` for a question or for work at verify), and `waitingSince` \u2014 the instant the wait began, not a duration. Nothing else is a gate: work in `triage` never appears, and neither does work in verify whose criteria have ALL passed. THIS TOOL is READ-ONLY and clears nothing; what clears each gate differs. Answering a question has no tool on this surface, by design. A requirement revision is approved with `approve_requirement`. A verify gate clears as its criteria pass \u2014 an `auto` one through `set_criterion_state`, a `manual` one only ever by a person in the web app. Use it to see what your work is blocked on before asking, rather than to act on it."
-    // NO arguments — not even an org. Org scope arrives from the token, exactly as
-    // it does for `list_projects`, and an org parameter would open a cross-org read
-    // path that has no precedent anywhere on this surface. The viewer narrowing the
-    // web queue offers is filter state on that view, not something an agent picks.
+    description: "Every gate awaiting a PERSON across ALL projects in the organization this call acts in \u2014 name that organization with `org`, or omit it to read the one this session defaults to \u2014 longest wait first: a requirement revision awaiting approval, a question awaiting an answer, and work in `verify` holding at least one criterion not yet passed. Each entry names its project, the ref the action is taken on (`KEY-R{n}` for a revision, the ISSUE\u2019s `KEY-{n}` for a question or for work at verify), and `waitingSince` \u2014 the instant the wait began, not a duration. Nothing else is a gate: work in `triage` never appears, and neither does work in verify whose criteria have ALL passed. THIS TOOL is READ-ONLY and clears nothing; what clears each gate differs. Answering a question has no tool on this surface, by design. A requirement revision is approved with `approve_requirement`. A verify gate clears as its criteria pass \u2014 an `auto` one through `set_criterion_state`, a `manual` one only ever by a person in the web app. Use it to see what your work is blocked on before asking, rather than to act on it."
+    // NO NARROWING ARGUMENTS. The one argument this tool does take — `org`, which every
+    // tool takes — is not a narrowing: it says WHICH ORGANIZATION the call acts in, not
+    // which of that organization's gates come back, and the caller's membership there is
+    // checked on the way in (PIPE-410). The viewer narrowing the web queue offers is
+    // filter state on that view, not something an agent picks.
   }, () => (
     // `{ items }`, mirroring the HTTP envelope rather than unwrapping to a bare array
     // like the sibling list tools. The reason is the envelope itself: this tool is a
@@ -17606,12 +17663,12 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
     // place for that answer's shape to be decided.
     //
     // The two surfaces are NOT interchangeable, and nothing here should suggest they
-    // are. This tool takes no narrowing arguments and returns the org's WHOLE queue,
-    // while the web view narrows what it DRAWS — by project, by category, and by viewer
-    // scope. One endpoint decides what counts as a gate, so that definition cannot drift
-    // between them; what an agent reads here is everything waiting, rather than whatever
-    // the operator happens to be looking at.
-    run(async () => ({ items: await client2.listWaiting() }), GUIDANCE_POINTER)
+    // are. This tool returns the WHOLE queue of the organization it acts in, while the
+    // web view narrows what it DRAWS — by project, by category, and by viewer scope. One
+    // endpoint decides what counts as a gate, so that definition cannot drift between
+    // them; what an agent reads here is everything waiting, rather than whatever the
+    // operator happens to be looking at.
+    run(async () => ({ items: await client2().listWaiting() }), GUIDANCE_POINTER)
   ));
   registerStrict(
     "get_work_context",
@@ -17649,7 +17706,22 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
     // THE COMPOSED GUIDANCE RIDES THE RESULT, as a second text block after the API's JSON.
     // This is the delivery PIPE-D30 chose: a tool result is not subject to the 2KB cap that
     // cut the previous mechanism to a tenth of the document.
-    (args) => run(() => client2.getWorkContext(args), guidance)
+    //
+    // AND IT IS COMPOSED FOR THE ORGANIZATION THE CALL ACTED IN (PIPE-410). This is the one
+    // call site the composition has, so a call naming another organization would otherwise
+    // return that organization's work item, requirements and conventions under THIS one's
+    // writing rules, in one document, with nothing in it saying so. See
+    // `opts.orgInstructionsFor`, which is what a caller supplies to close that; without it
+    // the construction-time composition is returned exactly as before.
+    async (args) => {
+      let appended;
+      try {
+        appended = await callGuidance();
+      } catch (err) {
+        return toolError(err);
+      }
+      return run(() => client2().getWorkContext(args), appended);
+    }
   );
   registerStrict("create_milestone", {
     description: "Create a milestone on a project. `token` is the short ref issues point at via their `milestone` field (e.g. `M1`) and is unique per project (a duplicate is rejected); `label` is its display name (e.g. `Feature Verification`). Optional `position` orders it (0-based, defaults to last); optional `active` marks it the current milestone \u2014 AT MOST ONE milestone per project is active, so `active: true` transactionally DEMOTES the previously active one rather than failing. Returns the created milestone (\xA77).",
@@ -17660,7 +17732,7 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       position: external_exports.number().int().nonnegative().optional(),
       active: external_exports.boolean().optional()
     }
-  }, ({ project, ...input }) => run(() => client2.createMilestone(project, input)));
+  }, ({ project, ...input }) => run(() => client2().createMilestone(project, input)));
   registerStrict("update_milestone", {
     description: "Rename or reorder a milestone, addressed by its token (e.g. `M1`) or uuid. Supply at least one of `token`, `label`, or `position` \u2014 an empty edit is rejected. Renaming the `token` does NOT rewrite issues pointing at the old one (`issues.milestone` is a loose ref), so prefer changing `label`. Use set_active_milestone to change `active` (\xA77).",
     inputSchema: {
@@ -17670,14 +17742,14 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       label: external_exports.string().min(1).optional(),
       position: external_exports.number().int().nonnegative().optional()
     }
-  }, ({ project, milestone, ...fields }) => run(async () => client2.updateMilestone(project, await resolveMilestoneId(client2, project, milestone), fields)));
+  }, ({ project, milestone, ...fields }) => run(async () => client2().updateMilestone(project, await resolveMilestoneId(client2(), project, milestone), fields)));
   registerStrict("set_active_milestone", {
     description: "Make a milestone the project\u2019s active one, addressed by its token (e.g. `M2`) or uuid. At most one milestone per project is active; this clears the previous one in the same transaction. Returns the activated milestone (\xA77).",
     inputSchema: {
       project: projectKeySchema,
       milestone: external_exports.string().min(1)
     }
-  }, ({ project, milestone }) => run(async () => client2.activateMilestone(project, await resolveMilestoneId(client2, project, milestone))));
+  }, ({ project, milestone }) => run(async () => client2().activateMilestone(project, await resolveMilestoneId(client2(), project, milestone))));
   registerStrict("delete_milestone", {
     description: "Soft-delete a milestone, addressed by its token (e.g. `M3`) or uuid; its token frees up for reuse. Issues still naming it are NOT rewritten and keep the now-dangling token (`issues.milestone` is a loose ref) \u2014 re-point them first. Returns confirmation metadata only (\xA77).",
     inputSchema: {
@@ -17685,8 +17757,8 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       milestone: external_exports.string().min(1)
     }
   }, ({ project, milestone }) => run(async () => {
-    const id = await resolveMilestoneId(client2, project, milestone);
-    await client2.deleteMilestone(project, id);
+    const id = await resolveMilestoneId(client2(), project, milestone);
+    await client2().deleteMilestone(project, id);
     return { deleted: true, project, milestone, id };
   }));
   registerStrict("create_deliverable_target", {
@@ -17698,7 +17770,7 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       position: external_exports.number().int().nonnegative().optional(),
       active: external_exports.boolean().optional()
     }
-  }, ({ project, ...input }) => run(() => client2.createDeliverableTarget(project, input)));
+  }, ({ project, ...input }) => run(() => client2().createDeliverableTarget(project, input)));
   registerStrict("update_deliverable_target", {
     description: "Update a deliverable target, addressed by its slug (e.g. `bsky`) or uuid. Supply at least one of `label`, `position`, or `active` \u2014 an empty edit is rejected. The `slug` is immutable (it is the value stored on existing deliverables). Set `active: false` to remove it from the add-deliverable menu while keeping existing deliverables working.",
     inputSchema: {
@@ -17708,7 +17780,7 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       position: external_exports.number().int().nonnegative().optional(),
       active: external_exports.boolean().optional()
     }
-  }, ({ project, target, ...fields }) => run(async () => client2.updateDeliverableTarget(project, await resolveDeliverableTargetId(client2, project, target), fields)));
+  }, ({ project, target, ...fields }) => run(async () => client2().updateDeliverableTarget(project, await resolveDeliverableTargetId(client2(), project, target), fields)));
   registerStrict("delete_deliverable_target", {
     description: "Delete a deliverable target, addressed by its slug (e.g. `bsky`) or uuid. Only a target with NO deliverables can be deleted; one already in use is rejected (deactivate it with update_deliverable_target instead). Returns confirmation metadata only.",
     inputSchema: {
@@ -17716,8 +17788,8 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       target: external_exports.string().min(1)
     }
   }, ({ project, target }) => run(async () => {
-    const id = await resolveDeliverableTargetId(client2, project, target);
-    await client2.deleteDeliverableTarget(project, id);
+    const id = await resolveDeliverableTargetId(client2(), project, target);
+    await client2().deleteDeliverableTarget(project, id);
     return { deleted: true, project, target, id };
   }));
   registerStrict(
@@ -17736,7 +17808,7 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
     // and an explicit filter, terminal values and all, is forwarded verbatim as
     // it always was. (The work page kept its terminal rows by naming every
     // status on its reads, so nothing over there leant on the defaultless API.)
-    (args) => run(() => client2.listIssues(args), GUIDANCE_POINTER)
+    (args) => run(() => client2().listIssues(args), GUIDANCE_POINTER)
   );
   registerStrict("get_issue", {
     description: `Full issue detail by display id or uuid: the header \u2014 \`summary\`, \`why\`, \`affectedSurfaces\`, with \`headerDerivedAt\` and \`headerStale\`, true when nobody has written a header or the body has moved since one was written, and \`set_issue_header\` is what writes it \u2014 body, criteria (states, modes, evidence), explicit requirement links (pinned revision + staleness), the project\u2019s cross-cutting requirements at their latest approved revision, this organization\u2019s approved conventions \u2014 standing guidance that BINDS work in every project here, delivered in full at the revision a person approved \u2014 children, and comments (\xA77). Carries \`terminal\`: true when the issue is finished (\`done\`) or deliberately abandoned (\`killed\`) and is therefore NOT buildable work \u2014 check it before acting on an issue you were handed by key. Reading criterion state: \`state_evidence\` is required only when a TOKEN sets one, so a \`manual\` criterion \u2014 settable only from a web session \u2014 carries none by design, and the attribution the server already stores (who set it, and when) IS its record. What this payload tells you about ANOTHER issue goes stale. Reference that issue by key and say what you need from it; never restate its status, its criteria, or whether those criteria passed. Long records read in PARTS: \`bodyOffset\`/\`bodyLimit\` window the body (UTF-16 offsets; parts concatenate back exactly), \`commentOffset\`/\`commentLimit\` window the oldest-first comment list. Unpaged, the body is capped at ${ISSUE_BODY_CHAR_LIMIT} chars and comments at the newest ${ISSUE_COMMENT_LIMIT}; a response that left anything out says so in \`omitted\` \u2014 the true totals, the omitted comment count, and in \`next\` the exact arguments for the adjacent part (body walks forward, comments toward older) \u2014 follow \`next\` until null to retrieve everything. An issue under both caps reads whole in ONE call, no notice.`,
@@ -17759,7 +17831,7 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
     // comment tail needs no offset — an omitted `commentOffset` beside a present
     // `commentLimit` IS the newest tail, resolved server-side, so no extra round
     // trip asks how many comments exist (PIPE-R15 rule 4).
-    client2.getIssue(issue2, {
+    client2().getIssue(issue2, {
       bodyOffset,
       bodyLimit: bodyLimit ?? ISSUE_BODY_CHAR_LIMIT,
       commentOffset,
@@ -17769,14 +17841,14 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
   registerStrict("create_issue", {
     description: "Create an issue (epic/task/bug/feedback). Omitted status defaults to triage (still subject to \xA76 guards); omitted environment resolves explicit \u2192 parent\u2019s environment \u2192 project default (prod). Optional parent (a parent already `done` or `killed` is rejected `parent_terminal`, unless the child is itself terminal; a new issue is a leaf, so no cycle is possible here), priority, milestone, requirement_links (pins each at its approved revision; a target with no approved revision is rejected), deliverable_links (content deliverables like DWB-C3/bsky \u2014 pins each at its current revision; the promotion path), and ordered {text, verification, howTo} criteria (\xA77). A `status` other than `triage` or `killed` REQUIRES `milestone` (PIPE-R10 rule 9): work past triage belongs to a release, and the create is refused (`milestone_required`) without one. A `manual` criterion must carry `howTo` \u2014 one or two sentences saying where to look, what to do, and what counts as passing; writing or changing one without it is refused (`how_to_required`), and so is supplying it on an `auto` criterion, which a machine checks. Work of any kind carries criteria \u2014 an epic, a task, a bug alike \u2014 with feedback the one exception, refused with `criteria_not_supported` because it is a report rather than agreed work; `promote_feedback` retypes it into work before anything is built. When a bug is filed against a defined acceptance criterion, pass that criterion\u2019s uuid as `criterionId` and the bug records which criterion it was filed against \u2014 what has already been reported arrives as `filedBugs` on every criteria read. Filing a second bug against one criterion is supported: one row can fail for two different reasons at once. Bugs only: any other type naming a criterion is refused (`criterion_requires_bug`). Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CSearch before you create\u201D and \u201CThe bar for filing a bug\u201D.",
     inputSchema: createIssueInputSchema.shape
-  }, (args) => run(() => client2.createIssue(args)));
+  }, (args) => run(() => client2().createIssue(args)));
   registerStrict("promote_feedback", {
     description: "Promote a FEEDBACK issue into a NEW epic or task (PIPE-143): creates the issue in the feedback\u2019s project (`title` + `type` required; omitted environment inherits the FEEDBACK\u2019s environment, not the project default; optional status/body/priority/milestone/requirement_links/criteria as in create_issue), records the feedback as its permanent source (`promotedFrom`/`promotedTo` on both details), closes the feedback as done, and comments the new key on it. The feedback\u2019s own title/body are never touched \u2014 write the new issue\u2019s body as real work. Rejected (409) for non-feedback, terminal, or already-promoted sources, and refused (422 `status_guard`, naming each offending child and its status) when the feedback carries a child issue that is not yet `done` or `killed` \u2014 closing the source would strand that child under a finished parent, so finish or kill the child first. Returns the created issue. A `status` other than `triage` or `killed` REQUIRES `milestone` (PIPE-R10 rule 9), exactly as in create_issue; the SOURCE feedback needs none to be closed by the promotion. A `manual` criterion must carry `howTo` \u2014 one or two sentences saying where to look, what to do, and what counts as passing; writing or changing one without it is refused (`how_to_required`), and so is supplying it on an `auto` criterion, which a machine checks. Work of any kind carries criteria \u2014 an epic, a task, a bug alike \u2014 with feedback the one exception, refused with `criteria_not_supported` because it is a report rather than agreed work; this tool is what retypes it into work before anything is built. Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CSearch before you create\u201D and \u201CThe bar for filing a bug\u201D.",
     inputSchema: {
       issue: external_exports.string().min(1),
       ...promoteFeedbackInputSchema.shape
     }
-  }, ({ issue: issue2, ...fields }) => run(() => client2.promoteFeedback(issue2, fields)));
+  }, ({ issue: issue2, ...fields }) => run(() => client2().promoteFeedback(issue2, fields)));
   registerStrict("open_review", {
     description: "Open the COMPLETION REVIEW \u2014 the reading of an epic\u2019s WHOLE accumulated change as one body of work, by a third fresh session that neither built it nor evaluated it. It runs at one moment: once the epic\u2019s contract is green and before anyone sets it `done`. It creates one ORDINARY `task` at `ready` carrying the review checklist as its body. The epic form (`epic`) titles it `Completion review \u2014 <KEY-n> <epic title>`, files it as a CHILD of the epic and inherits the epic\u2019s milestone; the milestone form (`project` + `milestone`) titles it `Completion review \u2014 milestone <token>`, carries that token and has no parent. Pass EXACTLY ONE of `epic` or `project` + `milestone`: both, or neither, is refused. One review at a time per scope \u2014 a second is refused (`review_already_open`) NAMING the standing one, so read, finish or kill that task instead. The scopes overlap ONE way: the epic form\u2019s review also stands for that same project and milestone, so a later milestone-form open is refused; a milestone-form review blocks no epic-form open. A review opens only on a non-terminal epic \u2014 `review_target_not_epic` otherwise, `review_target_terminal` on a `done`/`killed` epic (a re-opened epic reviews again). NO NEW GATE was added to `done`: file each finding as ordinary work \u2014 a CHILD OF THE EPIC, a sibling of this review \u2014 and the children clause already on `done` holds the epic open until every finding is done or killed. A finding filed UNDER the review gates nothing, because that gate reads one level deep. Killing the review, or any finding, is the recorded waiver: deliberately ungated, and written to the status history with who did it and through which token. Writing guidance for the checklist and for each finding lives in the product doctrine, which `get_work_context` returns.",
     // The WHOLE domain schema rather than its `.shape`, for `get_work_context`'s
@@ -17784,51 +17856,51 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
     // shape drops it — the refusal naming the two forms would never reach the
     // caller, and both arguments (or neither) would be accepted here.
     inputSchema: openReviewInputSchema
-  }, (args) => run(() => client2.openReview(args)));
+  }, (args) => run(() => client2().openReview(args)));
   registerStrict("update_issue", {
     description: "Update any subset of an issue\u2019s title, body, type, priority, milestone, environment, parent (self/cycle rejected, and moving an OPEN issue onto a parent already `done` or `killed` is rejected with `parent_terminal` \u2014 clearing the parent with null is always allowed), requirement_links (add/remove; adding a link to an unapproved requirement is rejected), deliverable_links (add/remove content-deliverable links, add pins at the current revision), or decision_links (add/remove decision records this work sits under \u2014 `KEY-D{n}`, from `list_decisions`). Changing type to `feedback` is rejected while the issue has criteria (\xA77), since feedback carries none; epic, task and bug interchange freely and the criteria ride along, keeping their AC-{n} numbers and their states. Clearing `milestone` (passing null) is rejected while the issue sits past `triage` \u2014 work already scheduled keeps a milestone (PIPE-R10 rule 9); swap it for a different one, or move the issue back to `triage` first. Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CLink by key; never paste another record\u2019s text\u201D and \u201CA decision you need is filed as a question, never written into a body\u201D.",
     inputSchema: updateIssueInputSchema.shape
-  }, ({ issue: issue2, ...fields }) => run(() => client2.updateIssue(issue2, fields)));
+  }, ({ issue: issue2, ...fields }) => run(() => client2().updateIssue(issue2, fields)));
   registerStrict("set_issue_header", {
     description: "Write an issue\u2019s HEADER \u2014 the three lines an operator reads before deciding anything: `summary` (the change, in one sentence), `why` (the user, business, cost or operational reason rather than the technical one; when nothing is user-visible, the cost of leaving it alone) and `affectedSurfaces` (which parts of the product this touches, and who feels it). All three are REQUIRED and the write is whole-value: the header is replaced entire and never merged, so a partial write cannot leave the stored provenance vouching for lines it did not touch. Returns the full issue detail. WRITE ALL THREE IN PLAIN WORDS: no identifiers, no file paths, no type names, no jargon. Their reader is deciding, not building, and the depth belongs in the body underneath them. A `summary` shaped like code \u2014 a repository path, a line number, a symbol written as a call or in backticks \u2014 is refused (400) naming what was found. `summary` is capped at 350 characters; over it the write is refused naming both the cap and the length received. THE HEADER CAN BE WRITTEN NO OTHER WAY: `summary`, `why` or `affectedSurfaces` passed to `create_issue`, `update_issue` or `promote_feedback` is refused (400) naming the field, rather than silently dropped \u2014 create the issue first, then write its header here. Every read of the issue carries `headerStale`, true when the body has changed since these three lines were derived from it and true when no header was ever written; writing here clears it. Writing guidance for these three lines is stated above; the organization\u2019s own writing rules, which `get_work_context` returns, govern the body beneath them.",
     inputSchema: setIssueHeaderInputSchema.shape
-  }, ({ issue: issue2, ...fields }) => run(() => client2.setIssueHeader(issue2, fields)));
+  }, ({ issue: issue2, ...fields }) => run(() => client2().setIssueHeader(issue2, fields)));
   registerStrict("set_status", {
     description: "Set an issue\u2019s status; the \xA76 guards apply and a guard failure returns the specific reason, naming the rows it refused on (\xA77). The criteria gates read whatever criteria the issue carries, whatever its type: verify needs every auto criterion passed, done needs every criterion passed. ONE guard remains EPIC-ONLY, and the asymmetry is the rule rather than an oversight \u2014 an epic that has stated NO criteria is refused at both gates, because an empty list would clear them having committed to nothing, while a task or a bug carrying none moves through both untouched (most work is stated adequately by its body). `done` additionally requires every child of the issue being closed to be terminal \u2014 an epic, a task and a bug alike \u2014 read from each child\u2019s STATUS and never from its criteria. A BLOCKED issue \u2014 one carrying an explicit mark or waiting on a blocker that is not yet `done`/`killed` \u2014 cannot move to `ready` or `in_progress`, and the refusal names the reason and every unresolved blocker, so clear the mark (`clear_blocked`) or finish, kill or `unlink_blocker` each blocker first; every other target still moves while blocked. An issue with NO milestone cannot move to any status but `triage` and `killed` (PIPE-R10 rule 9) \u2014 set one with `update_issue` first; `killed` is exempt, so work nobody scheduled can still be abandoned.",
     inputSchema: setStatusInputSchema.shape
-  }, (args) => run(() => client2.setStatus(args.issue, args.status)));
+  }, (args) => run(() => client2().setStatus(args.issue, args.status)));
   registerStrict("claim_issue", {
     description: "Atomically claim an issue (\xA76 rule 1): succeeds only if unclaimed; under a race exactly one wins and the loser\u2019s error names the current claimer. Claim is orthogonal to status (\xA77).",
     inputSchema: { issue: issueRefSchema }
-  }, (args) => run(() => client2.claimIssue(args.issue)));
+  }, (args) => run(() => client2().claimIssue(args.issue)));
   registerStrict("release_issue", {
     description: "Release an issue\u2019s claim, clearing claimed_by (\xA77).",
     inputSchema: { issue: issueRefSchema }
-  }, (args) => run(() => client2.releaseIssue(args.issue)));
+  }, (args) => run(() => client2().releaseIssue(args.issue)));
   registerStrict("add_comment", {
     description: "Add a markdown comment to an issue, attributed to the operator (and token label over MCP). A comment EXPLAINS a blockage \u2014 the full account of what you hit and why you stopped \u2014 but never records one: `set_blocked` (with a reason) and `link_blocker` (when you are waiting on another issue) are what put an issue in the blocked state (\xA77/WORKFLOW). Writing guidance for a comment lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CEvery loose end has one destination, and none of them is prose\u201D.",
     inputSchema: commentInputSchema.shape
-  }, (args) => run(() => client2.addComment(args.issue, args.body)));
+  }, (args) => run(() => client2().addComment(args.issue, args.body)));
   registerStrict("set_blocked", {
     description: "Record an issue as BLOCKED with a required human-readable `reason` \u2014 state on the issue, not a comment: it shows on the issue and counts on the project\u2019s Work page, and only clearing it (`clear_blocked`) ends it. Setting it again on an already-blocked issue replaces the standing reason and keeps the old one as history, attributed to whoever set it. The reason is one or two lines saying what the work is waiting on; the full account of what you tried still belongs in a comment. Returns the updated issue detail, whose `blocked` carries the reason and who set it. Writing guidance for the reason lives in the product doctrine, which `get_work_context` returns.",
     inputSchema: setBlockedInputSchema.shape
-  }, (args) => run(() => client2.setBlocked(args.issue, args.reason)));
+  }, (args) => run(() => client2().setBlocked(args.issue, args.reason)));
   registerStrict("clear_blocked", {
     description: "Clear an issue\u2019s blocked mark \u2014 the blockage is over. Idempotent: clearing an issue that is not blocked is a no-op, not an error. The cleared mark is kept as history with who ended it, so a past blockage stays readable. Returns the updated issue detail (`blocked` is then null).",
     inputSchema: { issue: issueRefSchema }
-  }, (args) => run(() => client2.clearBlocked(args.issue)));
+  }, (args) => run(() => client2().clearBlocked(args.issue)));
   registerStrict("link_blocker", {
     description: "Record that `issue` is WAITING ON `blocker` \u2014 the dependency between two issues, written from the waiting issue\u2019s side. One call records both directions: `get_issue` on the waiting issue lists it under `blockedBy` (with the blocker\u2019s display id, title, and whether it is already `done`/`killed`), and `get_issue` on the blocker lists it under `blocks`, so finishing work shows what it frees. Both issues must be in the SAME project. An issue cannot block itself, and an edge that would close a cycle (directly or through other issues) is refused with nothing recorded. Adding the same blocker twice records no second edge. Returns the waiting issue\u2019s updated detail.",
     inputSchema: linkBlockerInputSchema.shape
-  }, (args) => run(() => client2.addBlocker(args.issue, args.blocker)));
+  }, (args) => run(() => client2().addBlocker(args.issue, args.blocker)));
   registerStrict("unlink_blocker", {
     description: "Remove the blocked-by edge between `issue` and `blocker` \u2014 the dependency is gone, and one call clears it from BOTH issues\u2019 views. Idempotent: unlinking a pair that is not linked is a no-op, not an error. Returns the waiting issue\u2019s updated detail.",
     inputSchema: linkBlockerInputSchema.shape
-  }, (args) => run(() => client2.removeBlocker(args.issue, args.blocker)));
+  }, (args) => run(() => client2().removeBlocker(args.issue, args.blocker)));
   registerStrict("set_criteria", {
     description: "Define or replace the ordered acceptance criteria list ({text, verification, howTo}) on a piece of work. Matched rows keep their stable AC-{n} number; new items get fresh never-reused numbers; removed items are soft-deleted. A matched row keeps its state only when BOTH its text and verification mode are unchanged \u2014 a mode flip resets it to pending (\xA77). A `manual` criterion must carry `howTo` \u2014 one or two sentences saying where to look, what to do, and what counts as passing; writing or changing one without it is refused (`how_to_required`), and so is supplying it on an `auto` criterion, which a machine checks. Work of any kind carries criteria \u2014 an epic, a task, a bug alike \u2014 with feedback the one exception, refused with `criteria_not_supported` because it is a report rather than agreed work; `promote_feedback` retypes it into work before anything is built. Raising a criterion from `manual` to `auto` both resets its state and clears the stored `howTo`, so a criterion moved to `auto` has lost its checking instructions \u2014 moving it back to `manual` does not restore them, and is refused (`how_to_required`) until you write new ones. Writing guidance for a criterion, and for choosing its mode, lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CVerification modes follow what can be driven\u201D.",
     inputSchema: setCriteriaInputSchema.shape
-  }, (args) => run(() => client2.setCriteria(args.issue, args.criteria)));
+  }, (args) => run(() => client2().setCriteria(args.issue, args.criteria)));
   registerStrict("set_criterion_state", {
     description: "Set a criterion\u2019s state (pending/passed/failed) on whatever work carries it \u2014 an epic, a task and a bug alike \u2014 addressed by criterionId ALONE, or by issue + criterionNumber (the stable AC-{n}); exactly one form. Feedback never carries criteria, so neither address resolves to one. A token-authenticated call is REJECTED on a manual criterion (manual state is web-session-only, \xA74); evidence is required when a token marks passed/failed (\xA77). Writing guidance for that evidence lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CVerification modes follow what can be driven\u201D.",
     // `issue` is OPTIONAL: the criterionId-only form needs no issue ref. The
@@ -17849,12 +17921,12 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
   }, (args) => run(() => {
     const input = setCriterionStateInputSchema.parse(args);
     if (input.criterionId !== void 0) {
-      return client2.setCriterionStateById(input.criterionId, {
+      return client2().setCriterionStateById(input.criterionId, {
         state: input.state,
         evidence: input.evidence
       });
     }
-    return client2.setCriterionState(input.issue, {
+    return client2().setCriterionState(input.issue, {
       criterionNumber: input.criterionNumber,
       state: input.state,
       evidence: input.evidence
@@ -17863,23 +17935,23 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
   registerStrict("ask_questions", {
     description: "Ask the operator a BATCH of questions on an issue \u2014 `questions` takes 1 or more {text, options?}, and a session with one open decision usually has two or three, so ask them in ONE call rather than one at a time (a half-asked set costs the operator a second pass). `options` are the choices you propose, in the order you offer them; omit it to ask in open words. Each question gets a stable per-issue `Q-{n}` number that is NEVER reused. Returns the updated issue detail, and every issue read from then on carries `questions` with their answers \u2014 so read the answer in the `get_issue` you were going to make anyway, and READ the existing questions and answers BEFORE asking again rather than re-asking a decision already taken. Answering is reserved for a person in the web UI: there is deliberately no tool for it on this surface (PIPE-R22 rule 4). A question is withdrawn with `withdraw_question`. Writing guidance for a question lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CA decision you need is filed as a question, never written into a body\u201D.",
     inputSchema: askQuestionsInputSchema.shape
-  }, (args) => run(() => client2.askQuestions(args.issue, args.questions)));
+  }, (args) => run(() => client2().askQuestions(args.issue, args.questions)));
   registerStrict("withdraw_question", {
-    description: "Withdraw a question by its stable `Q-{n}` number \u2014 asker only: the asker is the token\u2019s OWNING user, so another agent\u2019s question is not yours to clear. Deliberately NOT human-gated, because the point is to clear a question you no longer need \u2014 the work answered it, or it was wrong to ask \u2014 instead of leaving the operator to answer something nobody is waiting on. It is recorded, never an answer: the question is soft-deleted (kept for the audit trail), its number retires and is not reissued, and the next issue read no longer carries it in `questions`. Returns the updated issue detail.",
+    description: "Withdraw a question by its stable `Q-{n}` number \u2014 asker only: the asker is the PERSON your credential belongs to, whichever organization the call names in `org`, so another agent\u2019s question is not yours to clear. Deliberately NOT human-gated, because the point is to clear a question you no longer need \u2014 the work answered it, or it was wrong to ask \u2014 instead of leaving the operator to answer something nobody is waiting on. It is recorded, never an answer: the question is soft-deleted (kept for the audit trail), its number retires and is not reissued, and the next issue read no longer carries it in `questions`. Returns the updated issue detail.",
     inputSchema: withdrawQuestionInputSchema.shape
-  }, (args) => run(() => client2.withdrawQuestion(args.issue, args.number)));
+  }, (args) => run(() => client2().withdrawQuestion(args.issue, args.number)));
   registerStrict("list_requirements", {
     description: "List requirements filtered by project and text: display id, title, summary (the latest revision\u2019s one/two-line outline description), area (product-area label), cross_cutting flag, lifecycle (draft/approved/amended), latest + approved revision numbers, and linked-epic count (\xA77).",
     inputSchema: listRequirementsFilterSchema.shape
-  }, (args) => run(() => client2.listRequirements(args), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().listRequirements(args), GUIDANCE_POINTER));
   registerStrict("get_requirement", {
     description: "Get a requirement by display id or uuid. Defaults to the approved revision (latest if never approved) unless `revision` is given; includes lifecycle, revision list, approvals, and linked issues (\xA77).",
     inputSchema: { requirement: requirementRefSchema, revision: positiveIntSchema.optional() }
-  }, (args) => run(() => client2.getRequirement(args.requirement, args.revision), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().getRequirement(args.requirement, args.revision), GUIDANCE_POINTER));
   registerStrict("create_requirement", {
     description: 'Create a project-scoped requirement document with revision 1 (title, body). Optional summary is a short one/two-line outline description (versioned on the revision, like title/body); optional area is a free-text product-area label ("Backup", "Billing") used to group/filter the outline (metadata, like cross_cutting). Optional cross_cutting flags it as binding every epic in the project automatically \u2014 never link cross-cutting documents (\xA75 rule 6, \xA77). Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CSearch before you create\u201D and \u201CLink by key; never paste another record\u2019s text\u201D.',
     inputSchema: createRequirementInputSchema.shape
-  }, (args) => run(() => client2.createRequirement(args)));
+  }, (args) => run(() => client2().createRequirement(args)));
   registerStrict("update_requirement", {
     description: "Update a requirement: title, body, and/or summary APPENDS a new revision (nothing is edited in place; summary versions like title/body); optional note records why. Optional area (product-area label), cross_cutting, and decision_links (add/remove decision records this rule sits under \u2014 `KEY-D{n}`, from `list_decisions`) are metadata changes (no revision) \u2014 toggling cross_cutting ON is rejected while explicit links exist; unlink first. Supplying none of title/body/summary/area/cross_cutting/decision_links is an empty edit (400) (\xA77). Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CLink by key; never paste another record\u2019s text\u201D.",
     // Inline shape (updateRequirementInputSchema is a refine, no .shape) — add
@@ -17895,23 +17967,23 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       crossCutting: external_exports.boolean().optional(),
       decisionLinks: decisionLinkChangesSchema.optional()
     }
-  }, ({ requirement, ...fields }) => run(() => client2.updateRequirement(requirement, fields)));
+  }, ({ requirement, ...fields }) => run(() => client2().updateRequirement(requirement, fields)));
   registerStrict("approve_requirement", {
     description: "Approve (pin) a requirement\u2019s latest revision, or a specified `revision`, recording it in approvals history and advancing the approved revision that downstream links and reads build against (\xA75, \xA77).",
     inputSchema: approveRequirementInputSchema.shape
-  }, (args) => run(() => client2.approveRequirement(args.requirement, args.revision)));
+  }, (args) => run(() => client2().approveRequirement(args.requirement, args.revision)));
   registerStrict("list_conventions", {
     description: "List this organization\u2019s conventions \u2014 standing guidance that binds work in EVERY project here, unlike a requirement, which binds one project. Returns display id (`CONV-{n}`), title, the standing text, lifecycle (draft/approved/amended), latest + approved revision numbers, and whether it has been superseded and by which convention. Optional `text` matches over title and body. Retired conventions are INCLUDED and marked, so you can see what the organization used to require rather than proposing it again. Only APPROVED, unretired conventions are actually delivered to agents \u2014 every `get_issue` and `get_work_context` read carries them.",
     inputSchema: listConventionsFilterSchema.shape
-  }, (args) => run(() => client2.listConventions(args), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().listConventions(args), GUIDANCE_POINTER));
   registerStrict("get_convention", {
     description: "Get one convention by display id (`CONV-{n}`) or uuid: the standing text at the approved revision, its full revision history with who wrote each and when, its approval history, and \u2014 when it has been retired \u2014 which convention replaced it. A convention that was never approved is readable here and is delivered to nobody.",
     inputSchema: { convention: conventionRefSchema }
-  }, (args) => run(() => client2.getConvention(args.convention), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().getConvention(args.convention), GUIDANCE_POINTER));
   registerStrict("create_convention", {
     description: "Write a convention: organization-wide standing guidance, with no project attached. THE GOVERNING TEST \u2014 if it stops being true when this project ends it is a cross-cutting requirement; if it would still be true on the next project it is a convention. You must assert that test by passing `outlivesAnyProject: true`; the write is refused without it. A statement naming one specific project \u2014 its key, a display id built on it, a repository path, or a database object \u2014 is also refused (422 `convention_names_project_scope`), and the message says to file it as a cross-cutting requirement on that project instead. IT BINDS NOBODY UNTIL IT IS APPROVED, and approval is NOT available on this surface: it requires a signed-in person in the browser, so there is no tool for it here and asking for one is asking for a tool that deliberately does not exist. A new convention starts as a draft; write it, then ask a person to review and approve it. Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CSearch before you create\u201D and \u201CLink by key; never paste another record\u2019s text\u201D.",
     inputSchema: createConventionInputSchema.shape
-  }, (args) => run(() => client2.createConvention(args)));
+  }, (args) => run(() => client2().createConvention(args)));
   registerStrict("update_convention", {
     description: "Amend a convention: `title` and/or `body` APPEND a new revision \u2014 nothing is ever edited in place \u2014 and optional `note` records why. Supplying neither is an empty edit (400); a note alone amends nothing. The same project-scope refusal applies as on create, over the amended text. THE AMENDMENT BINDS NOBODY UNTIL IT IS ITSELF APPROVED: until then agents keep receiving the approved revision, so amending is not a way to change what is being delivered \u2014 a signed-in person approving the new revision is. Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CConventions arrive in force; yours is a proposal\u201D.",
     // Inline shape (updateConventionInputSchema is a refine, so it has no `.shape`);
@@ -17922,19 +17994,19 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       body: markdownSchema.optional(),
       note: external_exports.string().min(1).optional()
     }
-  }, ({ convention, ...fields }) => run(() => client2.updateConvention(convention, fields)));
+  }, ({ convention, ...fields }) => run(() => client2().updateConvention(convention, fields)));
   registerStrict("list_prompts", {
     description: "List this organization\u2019s prompt library \u2014 reusable prompt text, addressed `PROMPT-{n}`. A prompt belongs to the ORGANIZATION rather than to a project, unlike a requirement or a content draft, so with no filter this returns the whole library. Returns display id, title, the current text, the latest revision number, and the attached project when it has one. `project` narrows to the prompts attached to that project \u2014 that attachment is metadata saying what a prompt is ABOUT, never its scope, so a prompt attached elsewhere is still one you may read and run. `text` matches over TITLE AND BODY, because someone looking for a prompt remembers what it says rather than what somebody called it.",
     inputSchema: listPromptsFilterSchema.shape
-  }, (args) => run(() => client2.listPrompts(args), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().listPrompts(args), GUIDANCE_POINTER));
   registerStrict("get_prompt", {
     description: "Get one prompt by display id (`PROMPT-{n}`) or uuid: its current text \u2014 the LATEST revision, which is simply the prompt, since nothing approves a prompt and nothing pins to an older wording \u2014 plus the full revision history with who wrote each save, through which token, when, and the note saying why. Nothing here was edited in place, so a wording that has been replaced is still readable rather than lost; read the history before rewriting a prompt that already works.",
     inputSchema: { prompt: promptRefSchema }
-  }, (args) => run(() => client2.getPrompt(args.prompt), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().getPrompt(args.prompt), GUIDANCE_POINTER));
   registerStrict("create_prompt", {
     description: "Write a prompt: reusable prompt text kept ORGANIZATION-WIDE (`PROMPT-{n}`) and available in every project here \u2014 unlike a requirement or a content draft, which belong to the one project they are filed under. `body` is the prompt itself and must not be empty; `title` names it; optional `note` records why it was written. Optional `project` ATTACHES the prompt to a project, which says what it is about and nothing more \u2014 the prompt is still the organization\u2019s, and a prompt about no project in particular simply omits it. There is no approval step: it is usable the moment it is written. Nothing deletes a prompt either \u2014 later wording is added as a revision, so what is here now stays readable. Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CSearch before you create\u201D.",
     inputSchema: createPromptInputSchema.shape
-  }, (args) => run(() => client2.createPrompt(args)));
+  }, (args) => run(() => client2().createPrompt(args)));
   registerStrict("update_prompt", {
     description: "Amend a prompt: `title` and/or `body` APPEND a new revision \u2014 nothing is ever edited in place \u2014 and optional `note` records why. Amending is therefore safe rather than destructive: the earlier text stays readable, the revision chain is the record of how this prompt got to where it is, and the new wording answers immediately because there is nothing to approve. `project` moves the attachment and `project: null` DETACHES it, leaving a prompt the whole organization still has; either way NO revision is written, because the revision chain is the history of the TEXT and moving the attachment changes no text. Supplying none of `title`, `body` or `project` is an empty edit (400) \u2014 a note alone amends nothing. Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns.",
     // Inline shape (updatePromptInputSchema is a refine, so it has no `.shape`),
@@ -17949,51 +18021,51 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       note: external_exports.string().min(1).optional(),
       project: projectRefSchema.nullable().optional()
     }
-  }, ({ prompt, ...fields }) => run(() => client2.updatePrompt(prompt, fields)));
+  }, ({ prompt, ...fields }) => run(() => client2().updatePrompt(prompt, fields)));
   registerStrict("create_decision", {
     description: "Record ONE decision as a project-scoped decision record (`KEY-D{n}`): `title` names the decision and `body` says what was decided and why. A new record starts `proposed`; moving it to `accepted` IS the acceptance \u2014 there is no separate approval step. Returns the created record. Title and body can be corrected only while the record is `proposed`; from `accepted` onward it is frozen, and the way to change a decision is to write a NEW record and `supersede_decision` the old one \u2014 which leaves the original readable and says where the current decision went. Nothing on any surface deletes one. Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CA decision record is written once\u201D and \u201CSearch before you create\u201D.",
     inputSchema: createDecisionInputSchema.shape
-  }, (args) => run(() => client2.createDecision(args)));
+  }, (args) => run(() => client2().createDecision(args)));
   registerStrict("get_decision", {
     description: "Get a decision record by display id (`KEY-D{n}`) or uuid: title, body, lifecycle state (`proposed`/`accepted`/`superseded`), who accepted it and when, the record that replaced it, where the CURRENT decision lives when this one has been superseded, and everything that references it \u2014 every requirement and every work item, gathered from both directions.",
     inputSchema: getDecisionInputSchema.shape
-  }, (args) => run(() => client2.getDecision(args.decision), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().getDecision(args.decision), GUIDANCE_POINTER));
   registerStrict("list_decisions", {
     description: "List decision records filtered by `project` and `text`: display id, title, lifecycle state, the replacement when superseded, and how many requirements and work items cite each one. `text` matches over TITLE AND BODY, so search on what a decision was about rather than on what someone chose to call it.",
     inputSchema: listDecisionsFilterSchema.shape
-  }, (args) => run(() => client2.listDecisions(args), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().listDecisions(args), GUIDANCE_POINTER));
   registerStrict("supersede_decision", {
     description: "Replace an ACCEPTED decision record with another one: `decision` moves to `superseded` and names `replacedBy` as where the current decision now lives, so whoever opens the old record is sent to the standing one. This is the only way a decision changes \u2014 the original is never edited and never removed, and the whole chain stays readable. Refused when the record is still `proposed` (nothing has been decided yet, so correct it instead), when a record is named as its own replacement, when the replacement belongs to another project, or when the replacement\u2019s own chain already leads back to this record. Returns the superseded record.",
     inputSchema: supersedeDecisionInputSchema.shape
-  }, (args) => run(() => client2.supersedeDecision(args.decision, args.replacedBy)));
+  }, (args) => run(() => client2().supersedeDecision(args.decision, args.replacedBy)));
   registerStrict("list_content", {
     description: "List content drafts filtered by project, text, area, and author: display id (KEY-C{n}), title, summary (latest revision), area, author, latest revision number, deliverable count, and linked-issue count. Content is a project-scoped sibling of requirements \u2014 raw material that gets promoted into work, not an issue type.",
     inputSchema: listContentFilterSchema.shape
-  }, (args) => run(() => client2.listContent(args), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().listContent(args), GUIDANCE_POINTER));
   registerStrict("get_content", {
     description: "Get a content draft by display id (KEY-C{n}) or uuid: its revision history (default latest, or `revision` to pin) plus every per-platform deliverable with its own revisions, posted metadata (url/postedAt), linked issues (with staleness), and linked assets.",
     inputSchema: { content: contentRefSchema, revision: positiveIntSchema.optional() }
-  }, (args) => run(() => client2.getContent(args.content, args.revision), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().getContent(args.content, args.revision), GUIDANCE_POINTER));
   registerStrict("create_content", {
     description: 'Create a project-scoped content draft with revision 1 (title, body). Optional summary is a short outline description (versioned on the revision, like title/body); optional area is a free-text product-area label (metadata). Optional author is the editorial byline the draft is credited to (free text, e.g. "jason") \u2014 distinct from the acting operator, and the field the list filters by. Optional metadata is an open bag for capture provenance (tags, score, sensitivity, source refs) \u2014 stored passively, never acted on. This is the capture-sink target: a captured topic becomes a draft here. Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CSearch before you create\u201D.',
     inputSchema: createContentInputSchema.shape
-  }, (args) => run(() => client2.createContent(args)));
+  }, (args) => run(() => client2().createContent(args)));
   registerStrict("update_content", {
     description: "Update a content draft: title, body, and/or summary APPENDS a new revision (nothing is edited in place); optional note records why. Optional area, author, and metadata are metadata changes (no revision). Supplying none of title/body/summary/area/author/metadata is an empty edit (400). Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns.",
     inputSchema: updateContentInputSchema.shape
-  }, ({ content, ...fields }) => run(() => client2.updateContent(content, fields)));
+  }, ({ content, ...fields }) => run(() => client2().updateContent(content, fields)));
   registerStrict("create_deliverable", {
     description: "Add a per-platform deliverable to a draft with revision 1 (the post text). `platform` is a slug configured for the project \u2014 one of its ACTIVE deliverable targets (see list_projects \u2192 deliverableTargets; defaults are bsky/x/reddit/hn/blog, but projects can add/remove them); an unconfigured slug is rejected. Optional body seeds revision 1. There is one deliverable per platform per draft \u2014 a variant is a revision (update_deliverable), not a second deliverable. Rendered id is DRAFT-KEY/platform (e.g. DWB-C3/bsky). Writing guidance for the post text lives in the product doctrine, which `get_work_context` returns.",
     inputSchema: createDeliverableInputSchema.shape
-  }, ({ content, ...fields }) => run(() => client2.createDeliverable(content, fields)));
+  }, ({ content, ...fields }) => run(() => client2().createDeliverable(content, fields)));
   registerStrict("update_deliverable", {
     description: "Append a new revision (post text) to a deliverable, addressed as DRAFT-KEY/platform (e.g. DWB-C3/bsky); optional note records why. Platform identity is immutable \u2014 this only versions the post text. Writing guidance for the post text lives in the product doctrine, which `get_work_context` returns.",
     inputSchema: updateDeliverableInputSchema.shape
-  }, ({ deliverable, ...fields }) => run(() => client2.updateDeliverable(deliverable, fields)));
+  }, ({ deliverable, ...fields }) => run(() => client2().updateDeliverable(deliverable, fields)));
   registerStrict("record_post", {
     description: "Record that a deliverable (DRAFT-KEY/platform) was posted: its public url and when (postedAt defaults to now), pinning the deliverable\u2019s current revision as the posted one. Pipebrain never posts \u2014 a human posts, then records the url here. Re-calling overwrites (a correction).",
     inputSchema: recordPostInputSchema.shape
-  }, ({ deliverable, ...fields }) => run(() => client2.recordPost(deliverable, fields)));
+  }, ({ deliverable, ...fields }) => run(() => client2().recordPost(deliverable, fields)));
   registerStrict("promote_deliverable", {
     description: "Promote a deliverable (DRAFT-KEY/platform, e.g. DWB-C3/bsky) to a NEW issue: creates the issue (`title` + `type` required; optional `status` defaults to triage, optional `milestone` schedules it, optional `body` seeds it \u2014 typically the current post text) AND a PROTECTED link pinning the deliverable\u2019s current revision. That link CANNOT be unlinked \u2014 the issue exists to ship this deliverable. (To link an EXISTING issue instead, use update_issue.deliverable_links, which stays removable.) Returns the created issue. A `status` other than `triage` or `killed` REQUIRES `milestone` (PIPE-R10 rule 9), exactly as in create_issue: work past triage belongs to a release, and the promotion is refused (`milestone_required`) without one. Seeding the body with the deliverable\u2019s own text is the exception to linking rather than pasting \u2014 it is the point of this tool, and the protected link pins the revision that text came from. Writing guidance for this tool lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CSearch before you create\u201D and \u201CLink by key; never paste another record\u2019s text\u201D.",
     inputSchema: {
@@ -18017,33 +18089,33 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       // that sentence before anything is called.
       ...refusedIssueHeaderFields
     }
-  }, ({ deliverable, ...fields }) => run(() => client2.promoteDeliverable(deliverable, fields)));
+  }, ({ deliverable, ...fields }) => run(() => client2().promoteDeliverable(deliverable, fields)));
   registerStrict("list_assets", {
     description: "List project assets (S3-backed files of any type) filtered by project, status (pending/active/rejected), category (image/video/document), and filename text, sorted by date or size (asc/desc): id, filename, contentType, size, sha256, status, hasThumbnail, and linkCount (live references). Metadata only \u2014 the bytes move by path, never through this result.",
     inputSchema: listAssetsFilterSchema.shape
-  }, (args) => run(() => client2.listAssets(args), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().listAssets(args), GUIDANCE_POINTER));
   registerStrict("get_asset", {
     description: "Get an asset\u2019s metadata by uuid: filename, contentType, size, sha256 (client-computed, server-verified by read-back), status, hasThumbnail, linkCount (live references), and timestamps. Metadata only \u2014 no value/URL is returned over MCP. To get the bytes, use download_asset (a LOCAL server only \u2014 it writes to a path on the machine running the server) or `ppb assets download`.",
     inputSchema: { asset: uuidSchema }
-  }, (args) => run(() => client2.getAsset(args.asset), GUIDANCE_POINTER));
+  }, (args) => run(() => client2().getAsset(args.asset), GUIDANCE_POINTER));
   if (surface === "local") {
     registerStrict("upload_asset", {
       description: "Upload a LOCAL FILE (by path) as a project asset and activate it: reads and SHA-256-hashes the file locally, registers the asset, PUTs the bytes directly to storage, and completes (server verifies the digest by read-back). Returns the asset metadata (status active on success). Only the file PATH crosses the wire \u2014 the bytes never do; do NOT pass base64 or file contents. contentType is guessed from the extension when omitted. Then use link_asset to attach the returned asset to a deliverable revision or a content draft.",
       inputSchema: uploadAssetInputSchema.shape
-    }, (args) => run(() => uploadAssetFromPath(client2, apiBaseUrl, args)));
+    }, (args) => run(() => uploadAssetFromPath(client2(), apiBaseUrl, args)));
     registerStrict("download_asset", {
       description: "Download an ACTIVE asset\u2019s bytes to a LOCAL FILE at `path` (created/overwritten), so you can read or view an uploaded image, mockup, or document. The bytes are written to disk and re-verified against the recorded sha256; the RESULT is metadata + the written path \u2014 never the bytes or a URL. Addressed by asset uuid (from list_assets, get_asset, or get_content).",
       inputSchema: downloadAssetInputSchema.shape
-    }, (args) => run(() => downloadAssetToPath(client2, args)));
+    }, (args) => run(() => downloadAssetToPath(client2(), args)));
   }
   registerStrict("link_asset", {
     description: "Reference an active asset FROM a target. Two targets are wired: `deliverable_revision` (targetId is a content_deliverable_revision uuid, e.g. from get_content) and `content_draft` (targetId is a content_draft uuid \u2014 the asset attaches to the topic doc and is snapshot-copied onto deliverables when they are created). Rejects a non-active asset or a cross-project link.",
     inputSchema: linkAssetInputSchema.shape
-  }, (args) => run(() => client2.linkAsset(args)));
+  }, (args) => run(() => client2().linkAsset(args)));
   registerStrict("unlink_asset", {
     description: "Remove an asset reference by its link id (soft-delete). The asset itself is untouched \u2014 this only drops the reference from its target. A link whose target is a CHAT MESSAGE is author-only: anyone but the person who wrote that message is refused with `not_message_author` (403), so an attachment you did not post is not yours to detach. Every other target type is removable by any member.",
     inputSchema: unlinkAssetInputSchema.shape
-  }, (args) => run(() => client2.unlinkAsset(args.linkId)));
+  }, (args) => run(() => client2().unlinkAsset(args.linkId)));
   registerStrict("set_secret", {
     description: "Create a secret or add a new version under a project + environment. THE VALUE CROSSES INBOUND ONLY \u2014 the response NEVER echoes it \u2014 but inbound is a direction, not a safe one: for a sensitive value prefer the CLI (`ppb secrets set` via stdin/`--from-file`), because a value typed here lands in the chat transcript before the server can encrypt it. `name` is UPPER_SNAKE_CASE (e.g. API_KEY); `environment` is the environment slug (dev/prod); optional `description`. Writing guidance for that description lives in the product doctrine, which `get_work_context` returns.",
     inputSchema: {
@@ -18053,14 +18125,14 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       value: external_exports.string().min(1),
       description: external_exports.string().optional()
     }
-  }, ({ project, environment, name, value, description }) => run(() => client2.setSecret(project, { env: environment, name, value, description })));
+  }, ({ project, environment, name, value, description }) => run(() => client2().setSecret(project, { env: environment, name, value, description })));
   registerStrict("list_secrets", {
     description: "List a project's secrets as METADATA ONLY (id, name, environment, description, current version, timestamps) \u2014 never any value. Optional `environment` slug filter.",
     inputSchema: {
       project: projectKeySchema,
       environment: external_exports.string().min(1).optional()
     }
-  }, ({ project, environment }) => run(() => client2.listSecrets(project, { env: environment }), GUIDANCE_POINTER));
+  }, ({ project, environment }) => run(() => client2().listSecrets(project, { env: environment }), GUIDANCE_POINTER));
   registerStrict("describe_secret", {
     description: "Describe one secret: its metadata plus its version history (version, created_at, created_by, and whether each version has been crypto-shredded). METADATA ONLY \u2014 no value is ever returned.",
     inputSchema: {
@@ -18068,7 +18140,7 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       environment: external_exports.string().min(1),
       name: secretNameSchema
     }
-  }, ({ project, environment, name }) => run(() => client2.describeSecret(project, name, { env: environment }), GUIDANCE_POINTER));
+  }, ({ project, environment, name }) => run(() => client2().describeSecret(project, name, { env: environment }), GUIDANCE_POINTER));
   registerStrict("rotate_secret", {
     description: "Rotate an EXISTING secret: store a NEW version and advance the pointer. Same value rules as `set_secret`: the value crosses INBOUND ONLY and the response NEVER echoes it, and for a sensitive value prefer the CLI (`ppb secrets set` adds a new version from stdin/`--from-file`), because a value typed here lands in the chat transcript before the server can encrypt it.",
     inputSchema: {
@@ -18077,7 +18149,7 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       name: secretNameSchema,
       value: external_exports.string().min(1)
     }
-  }, ({ project, environment, name, value }) => run(() => client2.rotateSecret(project, name, { env: environment, value })));
+  }, ({ project, environment, name, value }) => run(() => client2().rotateSecret(project, name, { env: environment, value })));
   registerStrict("update_secret_description", {
     description: "Update a secret's description (metadata-only edit \u2014 does not change the value or add a version). Writing guidance for that description lives in the product doctrine, which `get_work_context` returns.",
     inputSchema: {
@@ -18086,7 +18158,7 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       name: secretNameSchema,
       description: external_exports.string()
     }
-  }, ({ project, environment, name, description }) => run(() => client2.updateSecretDescription(project, name, { env: environment, description })));
+  }, ({ project, environment, name, description }) => run(() => client2().updateSecretDescription(project, name, { env: environment, description })));
   registerStrict("delete_secret", {
     description: "Delete a secret: soft-delete the entry and crypto-shred all its versions (the value becomes unrecoverable). Returns confirmation metadata only.",
     inputSchema: {
@@ -18095,20 +18167,20 @@ function createMcpServer(client2, apiBaseUrl, orgInstructions2 = /* @__PURE__ */
       name: secretNameSchema
     }
   }, ({ project, environment, name }) => run(async () => {
-    await client2.deleteSecret(project, name, { env: environment });
+    await client2().deleteSecret(project, name, { env: environment });
     return { deleted: true, project, environment, name };
   }));
   registerStrict("list_org_instructions", {
     description: "List the org\u2019s agent instructions \u2014 the per-scope texts this server composes into the guidance `get_work_context` returns, each under a heading naming its scope. Edits apply to NEW sessions only; use this to see the CURRENT saved values mid-session (at most one row per scope)."
-  }, () => run(() => client2.listOrgInstructions(), GUIDANCE_POINTER));
+  }, () => run(() => client2().listOrgInstructions(), GUIDANCE_POINTER));
   registerStrict("set_org_instruction", {
-    description: "Set (create or replace) one scope\u2019s org-level agent instruction text, max 4000 characters \u2014 over-cap is rejected, never truncated. Scopes: `general` (server-wide), `requirements`, `content`, and `work.<type>` (one per issue type, e.g. `work.epic`). Requires the ADMIN role in the token\u2019s org. Takes effect at the NEXT session start \u2014 running sessions keep the text they launched with. Returns the saved instruction. Writing guidance for an instruction body lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CSetting an instruction replaces the whole body\u201D.",
+    description: "Set (create or replace) one scope\u2019s org-level agent instruction text, max 4000 characters \u2014 over-cap is rejected, never truncated. Scopes: `general` (server-wide), `requirements`, `content`, and `work.<type>` (one per issue type, e.g. `work.epic`). WHICH ORGANIZATION IS WRITTEN is whichever one this call acts in: pass `org` and you are rewriting THAT organization\u2019s standing instructions to every agent working there, not your own. It requires the ADMIN role in the organization named, checked against your live membership of it, so a call naming an organization you are only a CONTRIBUTOR in is refused. Takes effect at the NEXT session start \u2014 running sessions keep the text they launched with. Returns the saved instruction. Writing guidance for an instruction body lives in the product doctrine, which `get_work_context` returns \u2014 see \u201CSetting an instruction replaces the whole body\u201D.",
     inputSchema: setOrgInstructionInputSchema.shape
-  }, (args) => run(() => client2.setOrgInstruction(args.scope, args.body)));
+  }, (args) => run(() => client2().setOrgInstruction(args.scope, args.body)));
   registerStrict("delete_org_instruction", {
-    description: "Clear one scope\u2019s org-level agent instruction (soft delete; an unset scope is an error naming it). Requires the ADMIN role in the token\u2019s org. Takes effect at the NEXT session start. Returns the cleared instruction.",
+    description: "Clear one scope\u2019s org-level agent instruction (soft delete; an unset scope is an error naming it). WHICH ORGANIZATION IS CLEARED is whichever one this call acts in: pass `org` and you are removing THAT organization\u2019s standing instruction to every agent working there, not your own. It requires the ADMIN role in the organization named, checked against your live membership of it, so a call naming an organization you are only a CONTRIBUTOR in is refused. Takes effect at the NEXT session start. Returns the cleared instruction.",
     inputSchema: deleteOrgInstructionInputSchema.shape
-  }, (args) => run(() => client2.deleteOrgInstruction(args.scope)));
+  }, (args) => run(() => client2().deleteOrgInstruction(args.scope)));
   return server2;
 }
 
@@ -18170,22 +18242,49 @@ function loadMcpConfig() {
   if (apiUrl === void 0 || token === void 0) {
     throw new EnvValidationError(issues, variables);
   }
-  return { apiUrl, token };
+  const org = resolved.org?.value;
+  return { apiUrl, token, org: org !== void 0 && isUnexpandedPlaceholder(org) ? void 0 : org };
 }
 
 // src/version.ts
-var VERSION = "0.42.0";
+var VERSION = "0.43.0";
 
 // src/main.ts
+async function scopedClient(base, org) {
+  if (org === void 0) return base;
+  try {
+    return base.forOrg(await resolveOrgReference(org, () => base.listOrgs()));
+  } catch (err) {
+    throw new Error(
+      `PIPEBRAIN_ORG is set to "${org}", and this server could not act there: ${err instanceof Error ? err.message : String(err)}`
+    );
+  }
+}
 var config2 = loadMcpConfig();
-var client = new ApiClient({
-  baseUrl: config2.apiUrl,
-  auth: { bearer: config2.token }
-});
+var client = await scopedClient(
+  new ApiClient({
+    baseUrl: config2.apiUrl,
+    auth: { bearer: config2.token }
+  }),
+  config2.org
+);
 var orgInstructions = await fetchOrgInstructionsSafe(client);
 var server = createMcpServer(client, config2.apiUrl, orgInstructions, {
   version: VERSION,
-  surface: "local"
+  surface: "local",
+  // The writing rules follow the organization the CALL acts in (PIPE-410). Without this,
+  // a `get_work_context` naming another organization returns that organization's work,
+  // requirements and conventions under THIS one's rules, in one document, with nothing in
+  // it saying so.
+  //
+  // A call naming none is served the startup snapshot, so the common session — every call
+  // in the default organization — still pays exactly one fetch, at launch, as it always
+  // has. A call naming another organization fetches that organization's text then and
+  // there rather than caching it: `get_work_context` is read once or twice in a session,
+  // so the round trip is cheap, and a per-session cache would freeze a FAILED fetch —
+  // fail-open returns an empty map — into "this organization has written nothing" for the
+  // rest of the session.
+  orgInstructionsFor: (actingOrgId) => actingOrgId === void 0 ? Promise.resolve(orgInstructions) : fetchOrgInstructionsSafe(client.forOrg(actingOrgId))
 });
 var transport = new StdioServerTransport();
 await server.connect(transport);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pipebrain-mcp",
-  "version": "0.42.0",
+  "version": "0.43.0",
   "description": "Pipebrain MCP server for Claude Code and Claude Desktop — issues, requirements, decision records, conventions, prompts, content, assets, metadata-only secrets, and more of the platform's modules over the Pipebrain API.",
   "type": "module",
   "license": "MIT",