ic-mops 2.3.0 → 2.3.1

package/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## Next
 
+## 2.3.1
+- Fix `mops build` and `mops check-candid` failing with "Wasm bindings have not been set" when installed via `npm i -g ic-mops`
+
 ## 2.3.0
 - Add `mops check-stable` command for stable variable compatibility checking
 - `mops check` now falls back to canister entrypoints from `mops.toml` when no files are specified

package/dist/bin/mops.js

@@ -1,3 +1,3 @@
 #!/usr/bin/env node
 
-import "../cli.js";
+import "../environments/nodejs/cli.js";

package/dist/fix-dist.js

@@ -7,3 +7,8 @@ delete json.scripts;
 json.bin.mops = "bin/mops.js";
 json.bin["ic-mops"] = "bin/mops.js";
 writeFileSync("dist/package.json", JSON.stringify(json, null, 2));
+// Route the npm entry point through the Node.js environment wrapper
+// so setWasmBindings() is called before the CLI runs.
+// The source bin/mops.js imports ../cli.js (needed for the single-file bundle),
+// but dist/ has the full directory structure with environments/nodejs/cli.js.
+writeFileSync("dist/bin/mops.js", '#!/usr/bin/env node\n\nimport "../environments/nodejs/cli.js";\n');

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "type": "module",
   "bin": {
     "mops": "bin/mops.js",

package/dist/tests/build.test.js

@@ -1,6 +1,9 @@
 import { describe, expect, test } from "@jest/globals";
+import { execa } from "execa";
+import { existsSync } from "node:fs";
 import path from "path";
 import { cliSnapshot } from "./helpers";
+const distBin = path.resolve(import.meta.dirname, "../dist/bin/mops.js");
 describe("build", () => {
     test("ok", async () => {
         const cwd = path.join(import.meta.dirname, "build/success");
@@ -15,4 +18,19 @@ describe("build", () => {
         expect((await cliSnapshot(["build", "bar"], { cwd }, 1)).stderr).toMatch("Candid compatibility check failed for canister bar");
         expect((await cliSnapshot(["build", "foo", "bar"], { cwd }, 1)).stderr).toMatch("Candid compatibility check failed for canister bar");
     });
+    // Regression: bin/mops.js must route through environments/nodejs/cli.js
+    // so that setWasmBindings() is called before any command runs.
+    // The dev entry point (npm run mops) uses tsx and always worked;
+    // this test exercises the compiled dist binary (same path as npm i -g ic-mops).
+    const hasDistBin = existsSync(distBin);
+    (hasDistBin ? test : test.skip)("wasm bindings initialized via dist entry point", async () => {
+        const cwd = path.join(import.meta.dirname, "build/success");
+        const result = await execa("node", [distBin, "build", "foo"], {
+            cwd,
+            stdio: "pipe",
+            reject: false,
+        });
+        expect(result.stderr).not.toContain("Wasm bindings have not been set");
+        expect(result.exitCode).toBe(0);
+    });
 });

package/fix-dist.ts

@@ -10,3 +10,12 @@ json.bin.mops = "bin/mops.js";
 json.bin["ic-mops"] = "bin/mops.js";
 
 writeFileSync("dist/package.json", JSON.stringify(json, null, 2));
+
+// Route the npm entry point through the Node.js environment wrapper
+// so setWasmBindings() is called before the CLI runs.
+// The source bin/mops.js imports ../cli.js (needed for the single-file bundle),
+// but dist/ has the full directory structure with environments/nodejs/cli.js.
+writeFileSync(
+  "dist/bin/mops.js",
+  '#!/usr/bin/env node\n\nimport "../environments/nodejs/cli.js";\n',
+);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "type": "module",
   "bin": {
     "mops": "dist/bin/mops.js",

package/tests/build.test.ts

@@ -1,7 +1,11 @@
 import { describe, expect, test } from "@jest/globals";
+import { execa } from "execa";
+import { existsSync } from "node:fs";
 import path from "path";
 import { cliSnapshot } from "./helpers";
 
+const distBin = path.resolve(import.meta.dirname, "../dist/bin/mops.js");
+
 describe("build", () => {
   test("ok", async () => {
     const cwd = path.join(import.meta.dirname, "build/success");
@@ -21,4 +25,24 @@ describe("build", () => {
       (await cliSnapshot(["build", "foo", "bar"], { cwd }, 1)).stderr,
     ).toMatch("Candid compatibility check failed for canister bar");
   });
+
+  // Regression: bin/mops.js must route through environments/nodejs/cli.js
+  // so that setWasmBindings() is called before any command runs.
+  // The dev entry point (npm run mops) uses tsx and always worked;
+  // this test exercises the compiled dist binary (same path as npm i -g ic-mops).
+  const hasDistBin = existsSync(distBin);
+  (hasDistBin ? test : test.skip)(
+    "wasm bindings initialized via dist entry point",
+    async () => {
+      const cwd = path.join(import.meta.dirname, "build/success");
+      const result = await execa("node", [distBin, "build", "foo"], {
+        cwd,
+        stdio: "pipe",
+        reject: false,
+      });
+
+      expect(result.stderr).not.toContain("Wasm bindings have not been set");
+      expect(result.exitCode).toBe(0);
+    },
+  );
 });