mcpill 1.10.0 → 1.12.0

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## 1.12.0
+
+- Fix: `runValidate` used `console.log` for the `✓ Valid:` summary line, writing plain text to stdout and breaking stdio MCP transport (clients tried to parse it as JSON-RPC and crashed). Output is now sent to `process.stderr.write` instead.
+- Bumped `mcpster` dep to `^0.3.2`, which carries the same fix in its `setup.js`
+
+## 1.11.0
+
+- `trigger: UserPromptSubmit` is a valid hook trigger in `PILL.md` `## Hook:` sections and `.mcpill/server/hooks/*.md` files
+- `mcpill compile` writes `UserPromptSubmit` hooks to `.claude/settings.json` without a `matcher` field (not supported by this event type)
+- `matcher` or `blocking: true` on a `UserPromptSubmit` hook emits a compile-time warning and the field is dropped
+
 ## 1.10.0
 
 - `mcpill compile` emits `.mcpill/dist/<pill>/` — a standalone npm package containing `install.js`, `index.js`, `tools.js`, `package.json`, and (for blocking pills) `hooks/pre-tool-gate.js` + `hooks/post-analysis-flag.js`

package/dist/cli.js

@@ -1076,8 +1076,13 @@ function mergeUserHooksIntoSettings(baseDir, hookBlocks) {
       ...block.matcher !== void 0 ? { matcher: block.matcher } : {},
       hooks: [{ type: "command", command: block.command }]
     };
-    const matcherKey = block.matcher ?? "";
-    const idx = existing.findIndex((e) => (e.matcher ?? "") === matcherKey);
+    let idx;
+    if (block.event === "UserPromptSubmit") {
+      idx = existing.findIndex((e) => e.hooks[0]?.command === block.command);
+    } else {
+      const matcherKey = block.matcher ?? "";
+      idx = existing.findIndex((e) => (e.matcher ?? "") === matcherKey);
+    }
     if (idx !== -1) {
       existing[idx] = newEntry;
     } else {
@@ -1227,7 +1232,8 @@ async function validateOne(mcpillDir) {
     }
     process.exit(1);
   }
-  console.log(`\u2713 Valid: ${toolCount} tools, ${promptCount} prompts, ${resourceCount} resources`);
+  process.stderr.write(`\u2713 Valid: ${toolCount} tools, ${promptCount} prompts, ${resourceCount} resources
+`);
 }
 async function runValidate(baseDir) {
   const mcpillDir = path6.join(baseDir, ".mcpill");
@@ -2215,8 +2221,24 @@ async function runCompile(opts) {
     }
   }
   const allUserHooks = [...pillHooks, ...fileHooks];
-  const blockingHooks = allUserHooks.filter((h) => h.blocking);
-  const regularHooks = allUserHooks.filter((h) => !h.blocking);
+  const normalizedUserHooks = allUserHooks.map((h) => {
+    if (h.event !== "UserPromptSubmit") return h;
+    const norm = { ...h };
+    if (norm.blocking) {
+      console.warn(`warn: UserPromptSubmit hooks do not support blocking \u2014 field ignored`);
+      delete norm.blocking;
+      delete norm.required_tool;
+      delete norm.gate_window_seconds;
+      delete norm.instructions;
+    }
+    if (norm.matcher !== void 0) {
+      console.warn(`warn: UserPromptSubmit hooks do not support matcher \u2014 field ignored`);
+      delete norm.matcher;
+    }
+    return norm;
+  });
+  const blockingHooks = normalizedUserHooks.filter((h) => h.blocking);
+  const regularHooks = normalizedUserHooks.filter((h) => !h.blocking);
   for (const hook of blockingHooks) {
     if (!hook.name) throw new Error(`blocking hook is missing a name`);
     if (!hook.required_tool) throw new Error(`blocking hook '${hook.name}' is missing required_tool`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcpill",
-  "version": "1.10.0",
+  "version": "1.12.0",
   "type": "module",
   "description": "CLI for building, validating, and publishing MCP servers using the pill format.",
   "homepage": "https://mcpill.ruco.dev",
@@ -27,7 +27,7 @@
   },
   "dependencies": {
     "mcpill-runtime": "^0.1.2",
-    "mcpster": "^0.3.0",
+    "mcpster": "^0.3.2",
     "commander": "^12"
   },
   "devDependencies": {