@productbrain/mcp 0.0.1-beta.4849 → 0.0.1-beta.4869

package/dist/{chunk-U3LG62QG.js → chunk-EW72I5PP.js}

@@ -42,7 +42,7 @@ import {
   trackSessionCaptureRate,
   trackWriteBackHintServed,
   trackZeroCaptureAuditFired
-} from "./chunk-42YIJFZI.js";
+} from "./chunk-Q6JFCKKI.js";
 
 // src/server.ts
 import { McpServer as McpServer2 } from "@modelcontextprotocol/sdk/server/mcp.js";
@@ -397,7 +397,14 @@ var TOOL_NAME_MIGRATIONS = /* @__PURE__ */ new Map([
   ["gather-context", 'context action="gather"'],
   ["get-build-context", 'context action="build"'],
   ["list-collections", 'collections action="list"'],
-  ["create-collection", 'collections action="create"'],
+  // WP-691 S4 (TASK-126, review round 1): 'collections action="create"' is gone — the member
+  // collection-create door closed for every caller. This row is a NOTICE, not an executable
+  // replacement: `collections action="update"` only edits an EXISTING collection, so mapping
+  // create-collection's genuine create-intent onto it would coach the agent to call a command
+  // that cannot do what it asked for. translateStaleToolNames renders a non-tool-call value
+  // (detected by its shape — no bare `noun action="value"` form) as prose, never as `\`old\` →
+  // \`current\`` — see its isExecutableReference check.
+  ["create-collection", 'closed \u2014 member collection-create is refused for every caller; ask a platform admin to provision it, or enable the module ontology pack that owns the drawer. `collections action="update"` only edits an EXISTING collection.'],
   ["update-collection", 'collections action="update"'],
   ["agent-start", 'session action="start"'],
   ["agent-close", 'session action="close"'],
@@ -439,12 +446,17 @@ var TOOL_NAME_MIGRATIONS = /* @__PURE__ */ new Map([
   ["chain-version", 'chain action="version.*" (commit/list/diff/revert/history \u2192 version.<same>)'],
   ["chain-branch", 'chain-review action="branch.*" (create/list/merge/conflicts \u2192 branch.<same>)']
 ]);
+function isExecutableReference(value) {
+  return /^[\w./-]+(\s+action="[^"]*")?$/.test(value);
+}
 function translateStaleToolNames(text) {
   const found = [];
   for (const [old, current] of TOOL_NAME_MIGRATIONS) {
     const pattern = new RegExp(`\\b${old.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\b`, "g");
     if (pattern.test(text)) {
-      found.push(`\`${old}\` \u2192 \`${current}\``);
+      found.push(
+        isExecutableReference(current) ? `\`${old}\` \u2192 \`${current}\`` : `\`${old}\` \u2014 ${current}`
+      );
     }
   }
   if (found.length === 0) return null;
@@ -7330,7 +7342,6 @@ async function handleLabelsRemove(slug, entryId) {
 // src/tools/collections.ts
 var COLLECTIONS_ACTIONS = [
   "list",
-  "create",
   "update",
   "describe",
   "audit",
@@ -7374,7 +7385,7 @@ var fieldSchema = z16.object({
 });
 var collectionsSchema = z16.object({
   action: z16.enum(COLLECTIONS_ACTIONS).describe(
-    "'list': browse all collections. 'create': create a new collection. 'update': update an existing collection. 'describe': full documentation for one collection \u2014 fields, option guides, usage guidance, examples. 'audit': health report for all collections \u2014 missing classification, icon, displayHint coverage, and field schema gaps. 'export': full system_collection_definitions export with classification metadata (thinkingLayer, classificationPriority, classificationCheck, classificationSignals, governanceRole, governanceFunction, timelineRole, canBeElementOf, descriptionFieldKey). Admin only. 'label-list'/'label-create'/'label-update'/'label-delete'/'label-apply'/'label-remove': manage workspace labels (absorbs the `labels` tool)."
+    "'list': browse all collections. 'update': update an existing collection. 'describe': full documentation for one collection \u2014 fields, option guides, usage guidance, examples. 'audit': health report for all collections \u2014 missing classification, icon, displayHint coverage, and field schema gaps. 'export': full system_collection_definitions export with classification metadata (thinkingLayer, classificationPriority, classificationCheck, classificationSignals, governanceRole, governanceFunction, timelineRole, canBeElementOf, descriptionFieldKey). Admin only. 'label-list'/'label-create'/'label-update'/'label-delete'/'label-apply'/'label-remove': manage workspace labels (absorbs the `labels` tool). Creating a new collection is not offered here \u2014 ask a platform admin to provision it, or enable the module ontology pack that owns the drawer."
   ),
   slug: z16.string().max(200).optional().describe("URL-safe identifier for create/update, e.g. 'glossary', 'tech-debt'. For label-*: label slug."),
   name: z16.string().max(500).optional().describe("Display name for create, or new name for update. For label-create: label display name."),
@@ -7408,24 +7419,6 @@ var collectionsSchema = z16.object({
 });
 var collectionsListVariant = z16.object({ action: z16.literal("list") });
 var collectionsDescribeVariant = z16.object({ action: z16.literal("describe"), slug: z16.string().max(200) });
-var collectionsCreateVariant = z16.object({
-  action: z16.literal("create"),
-  slug: z16.string().max(200),
-  name: z16.string().max(500),
-  description: z16.string().max(2e4).optional(),
-  purpose: z16.string().max(2e3).optional(),
-  icon: z16.string().max(50).optional(),
-  navGroup: z16.enum(["daily", "strategic", "governance", "reference", "collections"]).optional(),
-  fields: z16.array(fieldSchema).min(1),
-  axis: z16.enum(["work", "none", "pending"]),
-  // required, mirrors the Convex door's axisValidator (PRRT_kwDORS4ils6ZiYjo)
-  stageLabelMap: z16.record(z16.string(), z16.enum(["open", "in_progress", "done", "stopped", "tag"])).optional(),
-  defaultCanonicalKey: z16.string().max(200).optional(),
-  defaultWorkflowStatus: z16.string().max(200).optional(),
-  validWorkflowStatuses: z16.array(z16.string().max(200)).optional(),
-  classificationCheck: z16.string().max(500).optional(),
-  classificationPriority: z16.number().optional()
-});
 var collectionsUpdateVariant = z16.object({
   action: z16.literal("update"),
   slug: z16.string().max(200),
@@ -7474,7 +7467,6 @@ var collectionsLabelRemoveVariant = z16.object({ action: z16.literal("label-remo
 var collectionsActionUnion = z16.discriminatedUnion("action", [
   collectionsListVariant,
   collectionsDescribeVariant,
-  collectionsCreateVariant,
   collectionsUpdateVariant,
   collectionsAuditVariant,
   collectionsExportVariant,
@@ -7488,7 +7480,6 @@ var collectionsActionUnion = z16.discriminatedUnion("action", [
 var COLLECTIONS_ACTION_SPECS = {
   list: { params: [], description: "Browse all collections; no params." },
   describe: { params: ["slug"], description: "slug is required." },
-  create: { params: ["slug", "name", "description", "purpose", "icon", "navGroup", "fields", "axis", "stageLabelMap", "defaultCanonicalKey", "defaultWorkflowStatus", "validWorkflowStatuses", "classificationCheck", "classificationPriority"], description: "slug, name, fields, and axis are required. axis:'work' also requires stageLabelMap + validWorkflowStatuses." },
   update: { params: ["slug", "name", "description", "purpose", "icon", "navGroup", "fields", "axis", "stageLabelMap", "defaultCanonicalKey", "defaultWorkflowStatus", "validWorkflowStatuses", "classificationCheck", "classificationPriority", "qualityCriteria", "usageGuidance"], description: "slug is required; at least one other field must be provided. axis/stageLabelMap are only settable on a non-system-stratum collection (the repair path for a legacy custom collection missing its axis declaration)." },
   audit: { params: [], description: "Health report for all collections; no params." },
   export: { params: [], description: "Full system_collection_definitions export; no params. Admin only." },
@@ -7503,16 +7494,6 @@ function registerCollectionsTools(server) {
   const collectionsHandlers = {
     list: () => handleList2(),
     describe: (data) => handleDescribe(data.slug),
-    create: (data) => handleCreate2(
-      data.slug,
-      data.name,
-      data.description,
-      data.purpose,
-      data.icon,
-      data.navGroup ?? "collections",
-      data.fields,
-      { axis: data.axis, stageLabelMap: data.stageLabelMap, defaultCanonicalKey: data.defaultCanonicalKey, defaultWorkflowStatus: data.defaultWorkflowStatus, validWorkflowStatuses: data.validWorkflowStatuses, classificationCheck: data.classificationCheck, classificationPriority: data.classificationPriority }
-    ),
     update: (data) => {
       const hasChanges = data.name || data.description || data.purpose || data.icon || data.navGroup || data.fields || data.axis !== void 0 || data.stageLabelMap !== void 0 || data.defaultCanonicalKey !== void 0 || data.defaultWorkflowStatus !== void 0 || data.validWorkflowStatuses !== void 0 || data.classificationCheck !== void 0 || data.classificationPriority !== void 0 || data.qualityCriteria !== void 0 || data.usageGuidance !== void 0;
       if (!hasChanges) {
@@ -7542,7 +7523,7 @@ function registerCollectionsTools(server) {
     "collections",
     {
       title: "Collections",
-      description: "Manage knowledge collections and workspace labels. Twelve actions:\n\n- **list**: Browse all collections \u2014 glossary, business rules, tracking events, etc. Returns slug, name, description, and field schema. Use before capture to see what exists.\n- **describe**: Full documentation for a single collection. Returns field help text, option decision guides, usage guidance, examples, and cross-references. Use when you need to understand a collection's purpose, field semantics, or option values.\n- **create**: Create a new collection. Provide slug, name, and field schema. Use when setting up a workspace or tracking a new type of knowledge.\n- **update**: Update an existing collection's name, description, purpose, icon, navGroup, fields, or (non-system collections only) axis/stageLabelMap. Only provide the fields you want to change.\n- **audit**: Health report for all workspace collections. Checks classification metadata, icon presence, displayHint coverage per field, and field schema gaps vs system definitions. Returns total collections, count with issues, and per-collection issue list.\n- **export**: Full system_collection_definitions export as JSON. Includes all classification metadata (thinkingLayer, classificationPriority, classificationCheck, timelineRole, governanceRole, governanceFunction, etc.). Admin only.\n- **label-list/label-create/label-update/label-delete/label-apply/label-remove**: manage workspace labels (absorbs the `labels` tool) \u2014 list all labels, create/update/delete a label, or apply/remove a label on an entry. Labels can be applied to any entry across any collection for cross-domain filtering.",
+      description: "Manage knowledge collections and workspace labels. Eleven actions:\n\n- **list**: Browse all collections \u2014 glossary, business rules, tracking events, etc. Returns slug, name, description, and field schema. Use before capture to see what exists.\n- **describe**: Full documentation for a single collection. Returns field help text, option decision guides, usage guidance, examples, and cross-references. Use when you need to understand a collection's purpose, field semantics, or option values.\n- **update**: Update an existing collection's name, description, purpose, icon, navGroup, fields, or (non-system collections only) axis/stageLabelMap. Only provide the fields you want to change. Creating a NEW collection is not offered here \u2014 ask a platform admin to provision it, or enable the module ontology pack that owns the drawer.\n- **audit**: Health report for all workspace collections. Checks classification metadata, icon presence, displayHint coverage per field, and field schema gaps vs system definitions. Returns total collections, count with issues, and per-collection issue list.\n- **export**: Full system_collection_definitions export as JSON. Includes all classification metadata (thinkingLayer, classificationPriority, classificationCheck, timelineRole, governanceRole, governanceFunction, etc.). Admin only.\n- **label-list/label-create/label-update/label-delete/label-apply/label-remove**: manage workspace labels (absorbs the `labels` tool) \u2014 list all labels, create/update/delete a label, or apply/remove a label on an entry. Labels can be applied to any entry across any collection for cross-domain filtering.",
       inputSchema: collectionsSchema,
       annotations: { readOnlyHint: false, destructiveHint: true, openWorldHint: false }
     },
@@ -7667,8 +7648,7 @@ async function handleList2() {
     return successResult(
       "No collections found in this workspace.",
       "No collections found yet.",
-      { collections: [], total: 0 },
-      [{ tool: "collections", description: "Create collection", parameters: { action: "create" } }]
+      { collections: [], total: 0 }
     );
   }
   const formatted = collections.map((c) => {
@@ -7705,69 +7685,6 @@ ${formatted}` }],
 function metaFields(m) {
   return Object.fromEntries(Object.entries(m).filter(([, v]) => v !== void 0));
 }
-async function handleCreate2(slug, name, description, purpose, icon, navGroup, fields, meta = {}) {
-  requireWriteAccess();
-  try {
-    await kernelMutation("chain.createCollection", {
-      slug,
-      name,
-      description,
-      purpose,
-      icon,
-      navGroup,
-      fields,
-      ...metaFields(meta),
-      createdBy: getAgentSessionId() ? `agent:${getAgentSessionId()}` : "mcp"
-    });
-    invalidateCollectionCacheForScope();
-    invalidateProfileCacheForScope();
-    invalidateHubCacheForScope();
-    const fieldList = fields.map((f) => `  - \`${f.key}\` (${f.type}${f.required ? ", required" : ""}${f.searchable ? ", searchable" : ""})`).join("\n");
-    return {
-      content: [{
-        type: "text",
-        text: `# Collection Created: ${name}
-
-**Slug:** \`${slug}\`
-` + (description ? `**Description:** ${description}
-` : "") + (purpose ? `**Purpose:** ${purpose}
-` : "") + `**Nav group:** ${navGroup}
-
-**Fields:**
-${fieldList}
-
-You can now capture entries: \`capture collection="${slug}" name="..." description="..."\``
-      }],
-      structuredContent: success(
-        `Created collection '${name}' (${slug}) with ${fields.length} fields.`,
-        { slug, name, description, purpose, navGroup, fieldCount: fields.length },
-        [{ tool: "capture", description: "Capture an entry", parameters: { collection: slug } }]
-      )
-    };
-  } catch (error) {
-    const msg = error instanceof Error ? error.message : String(error);
-    if (msg.includes("already exists")) {
-      return failureResult(
-        `Collection '${slug}' already exists. Use collections action=update to modify it, or choose a different slug.`,
-        "DUPLICATE",
-        `Collection '${slug}' already exists.`,
-        "Use collections action=update to modify it.",
-        [{ tool: "collections", description: "Update collection", parameters: { action: "update", slug } }]
-      );
-    }
-    const code = error.code ?? error.data?.code;
-    if (code === "COLLECTION_PREFIX_TAKEN" || msg.includes("already used by collection")) {
-      return failureResult(
-        msg,
-        "DUPLICATE",
-        msg,
-        "Choose a different prefix by passing the idPrefix parameter, or let the system derive one from a different slug.",
-        [{ tool: "collections", description: "List collections to see taken prefixes", parameters: { action: "list" } }]
-      );
-    }
-    throw error;
-  }
-}
 async function handleUpdate(slug, name, description, purpose, icon, navGroup, fields, meta = {}) {
   requireWriteAccess();
   await kernelMutation("chain.updateCollection", {
@@ -14339,7 +14256,7 @@ var CALL_CATEGORIES = {
   "chain.createLabel": "label",
   "chain.updateLabel": "label",
   "chain.deleteLabel": "label",
-  "chain.createCollection": "write",
+  // WP-691 S4 (TASK-126): 'chain.createCollection' route removed — closed for every caller.
   "chain.updateCollection": "write",
   "chain.listCollections": "meta",
   "chain.getCollection": "meta",
@@ -15896,7 +15813,7 @@ ${stds}`);
     async (uri) => {
       const collections = await kernelQuery("chain.listCollections") ?? [];
       if (collections.length === 0) {
-        return { contents: [{ uri: uri.href, text: "No collections in this workspace. Use `collections action=create` or `start` with a preset to get started.", mimeType: "text/markdown" }] };
+        return { contents: [{ uri: uri.href, text: "No collections in this workspace. Ask a platform admin to provision one, or enable a module ontology pack \u2014 `collections action=create` is no longer self-serve.", mimeType: "text/markdown" }] };
       }
       const formatted = collections.map((c) => {
         const fieldList = (c.fields ?? []).map((f) => `  - \`${f.key}\` (${f.type}${f.required ? ", required" : ""}${f.searchable ? ", searchable" : ""})`).join("\n");
@@ -16409,8 +16326,9 @@ var INSTRUCTIONS = [
   "If the work is outside the scope of active work packages \u2014 stop, name the gap, and ask the user",
   "to confirm before designing. Do not propose implementation for out-of-scope work without a go-ahead.",
   "",
-  "Workspace setup: use `collections action=create` and `collections action=update` to shape the workspace",
-  "structure with the user. Ask what they need to track; presets are starting points, not fixed.",
+  "Workspace setup: use `collections action=update` to shape existing collections with the user.",
+  "Creating a brand-new collection is not self-serve \u2014 ask a platform admin to provision it, or",
+  "enable the module ontology pack that owns the drawer.",
   "",
   "Personalization: if you have context about the user from memory (prior work, recent",
   "conversations, team context), use it to personalize recommendations. For example,",
@@ -16577,4 +16495,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-U3LG62QG.js.map
+//# sourceMappingURL=chunk-EW72I5PP.js.map