@evref-bl/pharo-agent 0.5.1 → 0.5.2

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pharo-agent",
   "displayName": "Pharo Agent",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "Terminal-first Pharo image setup for coding agents.",
   "author": {
     "name": "Evref-BL",

package/bin/pharo-agent.js

@@ -1,7 +1,11 @@
 #!/usr/bin/env node
 import { main } from "../src/cli.js";
 
-main(process.argv.slice(2)).catch((error) => {
-  console.error(error instanceof Error ? error.message : String(error));
-  process.exitCode = 1;
-});
+main(process.argv.slice(2))
+  .then((exitCode) => {
+    if (exitCode) process.exitCode = exitCode;
+  })
+  .catch((error) => {
+    console.error(error instanceof Error ? error.message : String(error));
+    process.exitCode = 1;
+  });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@evref-bl/pharo-agent",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "private": false,
   "description": "Terminal-first Pharo integration for coding agents.",
   "license": "Apache-2.0",
@@ -32,8 +32,8 @@
   },
   "dependencies": {
     "@evref-bl/pharo-mcp": "2.1.0",
-    "@evref-bl/plexus": "0.5.3",
-    "@evref-bl/plexus-gateway": "0.5.3",
+    "@evref-bl/plexus": "0.5.4",
+    "@evref-bl/plexus-gateway": "0.5.4",
     "@modelcontextprotocol/sdk": "1.32.1"
   }
 }

package/plugin.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://agent-plugins.org/schemas/1.0.0/plugin.schema.json",
   "name": "pharo-agent",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "Terminal-first Pharo image setup for coding agents.",
   "author": {
     "name": "Evref-BL",

package/skills/pharo-setup/SKILL.md

@@ -8,7 +8,7 @@ Run this only when the user asks to enable Pharo tooling in this project.
 From the project root, run the released CLI that matches this plugin:
 
 ```sh
-npx --yes @evref-bl/pharo-agent@0.5.1 init --host claude
+npx --yes @evref-bl/pharo-agent@0.5.2 init --host claude
 ```
 
 Name the host this chat runs in: `--host claude` in Claude Code,

package/src/cli.js

@@ -110,6 +110,14 @@ function parseToolArguments(value) {
   return parsed;
 }
 
+// A tool the image answered with an error is still an answer: the MCP result
+// carries the Pharo exception, and a caller wants to read it. Print it as
+// always, and report the failure in the exit status too, so `set -e` and
+// subprocess.run(check=True) do not read a failed load as a success.
+export function exitCodeForToolResult(result) {
+  return result?.isError === true ? 1 : 0;
+}
+
 export async function main(args) {
   const [command, ...rest] = args;
   if (command === "init") {
@@ -194,13 +202,14 @@ export async function main(args) {
     if (!toolName || extra.length > 0) {
       throw new Error("pharo requires one tool name and an optional JSON object");
     }
-    console.log(JSON.stringify(await callImageTool(
+    const result = await callImageTool(
       projectRoot,
       readOpenImage(projectRoot, image),
       toolName,
       parseToolArguments(argumentsJson),
-    ), null, 2));
-    return;
+    );
+    console.log(JSON.stringify(result, null, 2));
+    return exitCodeForToolResult(result);
   }
   throw new Error(`Unsupported command: ${command}\n${usage()}`);
 }