@engine-room/after-effects-mcp 0.1.2 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@engine-room/after-effects-mcp",
-  "version": "0.1.2",
+  "version": "0.2.1",
   "type": "module",
   "description": "Control Adobe After Effects with AI — describe the animation you want and it gets built: layers, keyframes, effects, expressions and text, all editable afterwards.",
   "license": "MIT",
@@ -43,7 +43,7 @@
     "win32"
   ],
   "engines": {
-    "node": ">=20"
+    "node": ">=22"
   },
   "scripts": {
     "build": "tsc -p tsconfig.json",
@@ -53,13 +53,13 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.4",
-    "ws": "^8.18.0",
+    "ws": "^8.21.3",
     "zod": "^3.23.8",
     "zod-to-json-schema": "^3.23.5"
   },
   "devDependencies": {
     "@engineroom/shared": "*",
-    "@types/node": "^20.14.0",
+    "@types/node": "^22",
     "@types/ws": "^8.5.13",
     "esbuild": "^0.28.2",
     "typescript": "^5.6.0"

package/panel/CSXS/manifest.xml

@@ -1,8 +1,8 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<ExtensionManifest Version="11.0" ExtensionBundleId="games.engine-room.ae-mcp" ExtensionBundleVersion="0.1.2"
+<ExtensionManifest Version="11.0" ExtensionBundleId="games.engine-room.ae-mcp" ExtensionBundleVersion="0.2.1"
                    ExtensionBundleName="AE MCP Bridge" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
   <ExtensionList>
-    <Extension Id="games.engine-room.ae-mcp.panel" Version="0.1.2" />
+    <Extension Id="games.engine-room.ae-mcp.panel" Version="0.2.1" />
   </ExtensionList>
   <ExecutionEnvironment>
     <HostList>

package/panel/client/csinterface.js

@@ -11,6 +11,13 @@ CSInterface.prototype.evalScript = function (script, callback) {
 CSInterface.prototype.getSystemPath = function (pathType) {
   var path = window.__adobe_cep__.getSystemPath(pathType);
   if (path && path.indexOf("file://") === 0) path = decodeURIComponent(path.substring(7));
+  // CEP hands back a file URL, so on Windows the drive letter arrives still
+  // carrying the URL's leading slash: /C:/Users/... Node reads that as
+  // root-relative — path.join turns it into \C:\...\jsx\bundle.jsx, which no
+  // fs call can open, and the panel reports the bundle missing when it is
+  // sitting right there. Strip it here rather than at each call site: this is
+  // the one place a URL becomes a native path.
+  if (path) path = path.replace(/^\/([A-Za-z]:)/, "$1");
   return path;
 };
 

package/panel/client/main.js

@@ -10,14 +10,13 @@
   var fs = require("fs");
   var os = require("os");
   var http = require("http");
-  var WebSocket;
-  try { WebSocket = require("ws"); }
-  catch (e) {
-    // ws is bundled in packages/ae-panel/node_modules; resolve manually if normal require fails.
-    var alt = path.join(__dirname, "..", "node_modules", "ws");
-    WebSocket = require(alt);
-  }
+  var crypto = require("crypto");
 
+  // The DOM handles and the logger come first so that everything below is able
+  // to report its own failure. `require("ws")` used to run before this point,
+  // so when it threw the exception escaped this whole function before a single
+  // line could be written — the panel sat on "starting…" indefinitely with the
+  // reason nowhere to be seen.
   var $status = document.getElementById("status");
   var $port = document.getElementById("port");
   var $ae = document.getElementById("ae");
@@ -40,6 +39,24 @@
     while ($log.childNodes.length > 80) $log.removeChild($log.lastChild);
   }
 
+  var WebSocket;
+  try { WebSocket = require("ws"); }
+  catch (primary) {
+    // ws is bundled in packages/ae-panel/node_modules; resolve manually if normal require fails.
+    var alt = path.join(__dirname, "..", "node_modules", "ws");
+    try { WebSocket = require(alt); }
+    catch (fallback) {
+      // Nothing below can be built without ws, so this stops here — but it
+      // stops saying why, and naming the fix, rather than looking like a panel
+      // that is still starting up.
+      setStatus("cannot start — the ws module is missing", "err");
+      log("error", "require('ws') failed: " + primary.message);
+      log("error", "and " + alt + ": " + fallback.message);
+      log("error", "Quit After Effects, run the setup_panel tool, then reopen it.");
+      return;
+    }
+  }
+
   var cs = new CSInterface();
   var extDir = cs.getSystemPath(SystemPath.EXTENSION);
   var bundlePath = path.join(extDir, "jsx", "bundle.jsx");
@@ -84,6 +101,24 @@
   }
 
   // ---------- Load bundle.jsx ----------
+  // Hash of the bundle *actually evaluated into ExtendScript*, reported on
+  // /health. The server compares it against the bundle it ships to decide
+  // whether the running panel understands the ops it is about to send.
+  //
+  // It has to be captured here rather than read off disk on demand: after
+  // setup_panel refreshes the extension folder, the file on disk is new while
+  // this process is still running the old code, and that gap — between
+  // installing an update and restarting AE — is exactly when calls fail.
+  var loadedBundleHash = null;
+
+  function hashFile(file) {
+    try {
+      return crypto.createHash("sha256").update(fs.readFileSync(file)).digest("hex");
+    } catch (e) {
+      return null;
+    }
+  }
+
   function loadJsxBundle() {
     setStatus("Loading bundle.jsx…");
     if (!fs.existsSync(bundlePath)) {
@@ -95,7 +130,12 @@
     var loadScript = "$.evalFile(" + JSON.stringify(bundlePath) + "); typeof dispatch === 'function' ? 'ok' : 'no-dispatch';";
     return new Promise(function (resolve, reject) {
       cs.evalScript(loadScript, function (raw) {
-        if (raw === "ok") { $jsx.textContent = "loaded"; $jsx.className = "ok"; resolve(); }
+        if (raw === "ok") {
+          loadedBundleHash = hashFile(bundlePath);
+          $jsx.textContent = "loaded";
+          $jsx.className = "ok";
+          resolve();
+        }
         else { $jsx.textContent = "fail: " + raw; $jsx.className = "err"; reject(new Error("dispatch not defined after loading bundle.jsx: " + raw)); }
       });
     });
@@ -245,7 +285,15 @@
         if (url === "/health") {
           res.statusCode = 200;
           res.setHeader("content-type", "application/json");
-          res.end(JSON.stringify({ ok: true, port: port, bundleLoaded: true, ts: Date.now() }));
+          // bundleHash identifies the code this panel is *running*, which is what
+          // the server needs to know before sending an op the panel may predate.
+          res.end(JSON.stringify({
+            ok: true,
+            port: port,
+            bundleLoaded: true,
+            bundleHash: loadedBundleHash,
+            ts: Date.now()
+          }));
           return;
         }
 

package/panel/jsx/bundle.jsx

@@ -1,5 +1,5 @@
 // Auto-generated bundle. Do not edit directly — edit files in packages/jsx/.
-// Generated 2026-08-11T09:11:46.741Z
+// Generated 2026-08-12T07:48:29.949Z
 
 // ===== core.jsx =====
 
@@ -1222,6 +1222,123 @@ OPS.screenshot_layer = noUndo(function (args) {
 });
 
 
+// ===== style.jsx =====
+
+// style.jsx — the project's house style, read from and written to a plain
+// markdown file sitting next to the .aep.
+//
+// Why here rather than in the MCP server: the bridge into After Effects is the
+// one channel every client has, because the whole product already depends on it.
+// A server-side file would need a project folder, and the clients that matter
+// most here are exactly the ones that do not have one — Claude Desktop starts
+// its servers at the filesystem root. Reading the style over the bridge needs no
+// working directory, no filesystem tools on the client, and no configuration.
+//
+// The cost is that the project must have been saved once. `app.project.file` is
+// null until then, and there is no folder to write into. That is reported, never
+// guessed around.
+
+var HOUSE_STYLE_FILENAME = "house-style.md";
+
+/** The .aep's folder, or null when the project has never been saved. */
+function __projectFolder() {
+  var f = app.project.file;
+  if (!f) return null;
+  return f.parent;
+}
+
+function __houseStyleFile() {
+  var folder = __projectFolder();
+  if (!folder) return null;
+  return new File(folder.fsName + "/" + HOUSE_STYLE_FILENAME);
+}
+
+/** Shared shape for "there is nowhere to put it", so both ops explain it the same way. */
+function __unsavedProject() {
+  return {
+    found: false,
+    projectSaved: false,
+    path: null,
+    content: null,
+    reason:
+      "This After Effects project has never been saved, so there is no folder to keep the style guide in. " +
+      "Ask the user to save the project, then try again."
+  };
+}
+
+OPS.get_house_style = noUndo(function () {
+  var file = __houseStyleFile();
+  if (!file) return __unsavedProject();
+
+  if (!file.exists) {
+    return {
+      found: false,
+      projectSaved: true,
+      path: file.fsName,
+      content: null,
+      reason: "No style guide for this project yet. Offer to capture one from a comp the user likes."
+    };
+  }
+
+  // UTF-8 explicitly: ExtendScript otherwise decodes with the system encoding,
+  // which mangles any non-ASCII the designer typed (curly quotes, em dashes,
+  // accented font names).
+  file.encoding = "UTF-8";
+  if (!file.open("r")) throw new Error("Could not open " + file.fsName + " for reading");
+  var content;
+  try { content = file.read(); }
+  finally { file.close(); }
+
+  return {
+    found: true,
+    projectSaved: true,
+    path: file.fsName,
+    content: content,
+    bytes: content.length
+  };
+});
+
+OPS.set_house_style = noUndo(function (args) {
+  var content = args && args.content;
+  if (typeof content !== "string" || content.length === 0) {
+    throw new Error("content is required and must not be empty");
+  }
+
+  var file = __houseStyleFile();
+  if (!file) {
+    var unsaved = __unsavedProject();
+    throw new Error(unsaved.reason);
+  }
+
+  var existed = file.exists;
+  // Replacing someone's hand-written style guide is not something to do on a
+  // half-remembered version of it, so the caller has to have read it first and
+  // send the whole document back. Refusing here is cheaper than an apology.
+  if (existed && args.overwrite !== true) {
+    throw new Error(
+      "A style guide already exists at " + file.fsName + ". Read it with get_house_style, " +
+      "merge your changes into the full document, and call again with overwrite: true."
+    );
+  }
+
+  file.encoding = "UTF-8";
+  if (!file.open("w")) throw new Error("Could not open " + file.fsName + " for writing");
+  try {
+    if (!file.write(content)) throw new Error("Write failed for " + file.fsName);
+  } finally {
+    file.close();
+  }
+
+  return {
+    ok: true,
+    path: file.fsName,
+    bytes: content.length,
+    created: !existed,
+    replaced: existed
+  };
+});
+
+
 // ===== batch.jsx =====
 
 // batch.jsx — execute many ops in one ExtendScript pass.

package/panel/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@engineroom/ae-panel",
-  "version": "0.1.2",
+  "version": "0.2.1",
   "private": true,
   "description": "Invisible CEP extension hosting an HTTP+WS bridge inside After Effects.",
   "dependencies": {
-    "ws": "^8.18.0"
+    "ws": "^8.21.3"
   }
 }