ic-mops 2.2.1 → 2.3.1

package/dist/helpers/resolve-canisters.js

@@ -0,0 +1,31 @@
+import { cliError } from "../error.js";
+export function resolveCanisterConfigs(config) {
+    if (!config.canisters) {
+        return {};
+    }
+    return Object.fromEntries(Object.entries(config.canisters).map(([name, c]) => typeof c === "string" ? [name, { main: c }] : [name, c]));
+}
+export function resolveCanisterEntrypoints(config) {
+    const canisters = resolveCanisterConfigs(config);
+    return Object.values(canisters)
+        .map((c) => c.main)
+        .filter((main) => Boolean(main));
+}
+export function resolveSingleCanister(config, canisterName) {
+    const canisters = resolveCanisterConfigs(config);
+    const names = Object.keys(canisters);
+    if (names.length === 0) {
+        cliError("No canisters defined in mops.toml [canisters] section");
+    }
+    if (canisterName) {
+        const canister = canisters[canisterName];
+        if (!canister) {
+            cliError(`Canister '${canisterName}' not found in mops.toml. Available: ${names.join(", ")}`);
+        }
+        return { name: canisterName, canister };
+    }
+    if (names.length > 1) {
+        cliError(`Multiple canisters defined in mops.toml. Please specify one: ${names.join(", ")}`);
+    }
+    return { name: names[0], canister: canisters[names[0]] };
+}

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.2.1",
+  "version": "2.3.1",
   "type": "module",
   "bin": {
     "mops": "bin/mops.js",
@@ -63,7 +63,7 @@
     "pic-ic": "0.5.4",
     "pic-js-mops": "0.14.8",
     "prettier": "3.5.3",
-    "prettier-plugin-motoko": "0.12.5",
+    "prettier-plugin-motoko": "0.13.0",
     "promisify-child-process": "4.1.2",
     "prompts": "2.4.2",
     "semver": "7.7.1",

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/dist/tests/check-fix.test.js

@@ -86,4 +86,21 @@ describe("check --fix", () => {
         expect(result.stdout).toContain("Attempting to fix files");
         expect(result.stdout).toContain("No fixes were needed");
     });
+    test("fix with remaining warnings", async () => {
+        const runFilePath = copyFixture("fix-with-warning.mo");
+        const result = await cli(["check", runFilePath, "--fix", "--", warningFlags], { cwd: fixDir });
+        expect(result.exitCode).toBe(0);
+        expect(result.stdout).toContain("1 fix applied");
+        expect(result.stdout).toMatch(/✓/);
+        expect(result.stderr).toMatch(/warning \[M0194\]/);
+        expect(result.stderr).toMatch(/unused identifier/);
+    });
+    test("fix with remaining errors", async () => {
+        const runFilePath = copyFixture("fix-with-error.mo");
+        const result = await cli(["check", runFilePath, "--fix", "--", warningFlags], { cwd: fixDir });
+        expect(result.exitCode).toBe(1);
+        expect(result.stdout).toContain("1 fix applied");
+        expect(result.stderr).toMatch(/error/i);
+        expect(result.stdout).not.toMatch(/✓ run/);
+    });
 });

package/dist/tests/check-stable.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/tests/check-stable.test.js

@@ -0,0 +1,51 @@
+import { describe, expect, test } from "@jest/globals";
+import { mkdtemp, rm, writeFile } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import path from "path";
+import { cli, cliSnapshot } from "./helpers";
+describe("check-stable", () => {
+    test("compatible upgrade from .mo file", async () => {
+        const cwd = path.join(import.meta.dirname, "check-stable/compatible");
+        await cliSnapshot(["check-stable", "old.mo"], { cwd }, 0);
+    });
+    test("incompatible upgrade from .mo file", async () => {
+        const cwd = path.join(import.meta.dirname, "check-stable/incompatible");
+        const result = await cliSnapshot(["check-stable", "old.mo"], { cwd }, 1);
+        expect(result.stderr).toMatch(/compatibility/i);
+    });
+    test("compatible upgrade with verbose", async () => {
+        const cwd = path.join(import.meta.dirname, "check-stable/compatible");
+        const result = await cli(["check-stable", "old.mo", "--verbose"], { cwd });
+        expect(result.exitCode).toBe(0);
+        expect(result.stdout).toMatch(/Generating stable types for old\.mo/);
+        expect(result.stdout).toMatch(/Generating stable types for new\.mo/);
+        expect(result.stdout).toMatch(/--stable-compatible/);
+        expect(result.stdout).toMatch(/Stable compatibility check passed/);
+    });
+    test("old file in subdirectory (.old/src/ pattern)", async () => {
+        const cwd = path.join(import.meta.dirname, "check-stable/subdirectory");
+        const result = await cli(["check-stable", ".old/src/main.mo"], { cwd });
+        expect(result.exitCode).toBe(0);
+        expect(result.stdout).toMatch(/Stable compatibility check passed/);
+    });
+    test("compatible upgrade from .most file", async () => {
+        const cwd = path.join(import.meta.dirname, "check-stable/compatible");
+        const tempDir = await mkdtemp(path.join(tmpdir(), "mops-test-most-"));
+        try {
+            const mostPath = path.join(tempDir, "old.most");
+            await writeFile(mostPath, "actor {\n  stable var counter : Nat\n};\n");
+            const result = await cli(["check-stable", mostPath], { cwd });
+            expect(result.exitCode).toBe(0);
+            expect(result.stdout).toMatch(/Stable compatibility check passed/);
+        }
+        finally {
+            await rm(tempDir, { recursive: true, force: true });
+        }
+    });
+    test("errors when old file does not exist", async () => {
+        const cwd = path.join(import.meta.dirname, "check-stable/compatible");
+        const result = await cli(["check-stable", "nonexistent.mo"], { cwd });
+        expect(result.exitCode).toBe(1);
+        expect(result.stderr).toMatch(/File not found/);
+    });
+});

package/dist/tests/check.test.js

@@ -1,6 +1,6 @@
 import { describe, expect, test } from "@jest/globals";
 import path from "path";
-import { cliSnapshot } from "./helpers";
+import { cli, cliSnapshot } from "./helpers";
 describe("check", () => {
     test("ok", async () => {
         const cwd = path.join(import.meta.dirname, "check/success");
@@ -34,4 +34,57 @@ describe("check", () => {
         const cwd = path.join(import.meta.dirname, "check/moc-args");
         await cliSnapshot(["check", "Warning.mo"], { cwd }, 1);
     });
+    test("no args falls back to [canisters] entrypoints", async () => {
+        const cwd = path.join(import.meta.dirname, "check/canisters");
+        await cliSnapshot(["check"], { cwd }, 0);
+    });
+    test("[moc] args applied when using canister fallback", async () => {
+        const cwd = path.join(import.meta.dirname, "check/canisters-moc-args");
+        const result = await cli(["check"], { cwd });
+        expect(result.exitCode).toBe(1);
+        expect(result.stderr).toMatch(/warning \[M0194\]/);
+    });
+    test("canister entrypoint with errors", async () => {
+        const cwd = path.join(import.meta.dirname, "check/canisters-error");
+        const result = await cli(["check"], { cwd });
+        expect(result.exitCode).toBe(1);
+        expect(result.stderr).toMatch(/error/i);
+    });
+    test("--fix with canister fallback", async () => {
+        const cwd = path.join(import.meta.dirname, "check/canisters");
+        const result = await cli(["check", "--fix"], { cwd });
+        expect(result.exitCode).toBe(0);
+    });
+    test("deployed: runs stable check when deployed file exists", async () => {
+        const cwd = path.join(import.meta.dirname, "check/deployed-compatible");
+        const result = await cli(["check"], { cwd });
+        expect(result.exitCode).toBe(0);
+        expect(result.stdout).toMatch(/Stable compatibility check passed/);
+    });
+    test("deployed: silently skips when file missing and skipIfMissing", async () => {
+        const cwd = path.join(import.meta.dirname, "check/deployed-missing-skip");
+        const result = await cli(["check"], { cwd });
+        expect(result.exitCode).toBe(0);
+        expect(result.stdout).not.toMatch(/stable/i);
+    });
+    test("deployed: errors when file missing without deployedSkipIfFileMissing", async () => {
+        const cwd = path.join(import.meta.dirname, "check/deployed-missing-error");
+        const result = await cli(["check"], { cwd });
+        expect(result.exitCode).toBe(1);
+        expect(result.stderr).toMatch(/Deployed file not found/);
+        expect(result.stderr).toMatch(/skipIfMissing/);
+    });
+    test("--fix runs stable check after fixing", async () => {
+        const cwd = path.join(import.meta.dirname, "check/deployed-compatible");
+        const result = await cli(["check", "--fix"], { cwd });
+        expect(result.exitCode).toBe(0);
+        expect(result.stdout).toMatch(/Stable compatibility check passed/);
+    });
+    test("stable check is skipped when type-checking fails", async () => {
+        const cwd = path.join(import.meta.dirname, "check/deployed-compile-error");
+        const result = await cli(["check"], { cwd });
+        expect(result.exitCode).toBe(1);
+        expect(result.stderr).toMatch(/error/i);
+        expect(result.stdout).not.toMatch(/Stable compatibility/);
+    });
 });

package/dist/tests/moc-args.test.js

@@ -6,12 +6,12 @@ describe("moc-args", () => {
         const cwd = path.join(import.meta.dirname, "check/moc-args");
         const result = await cli(["moc-args"], { cwd });
         expect(result.exitCode).toBe(0);
-        expect(result.stdout).toBe("-Werror");
+        expect(result.stdout).toBe("--default-persistent-actors\n-Werror");
     });
-    test("prints nothing when no [moc] config", async () => {
+    test("prints only global args when no extra [moc] args", async () => {
         const cwd = path.join(import.meta.dirname, "check/success");
         const result = await cli(["moc-args"], { cwd });
         expect(result.exitCode).toBe(0);
-        expect(result.stdout).toBe("");
+        expect(result.stdout).toBe("--default-persistent-actors");
     });
 });

package/dist/types.d.ts

@@ -32,10 +32,14 @@ export type Config = {
     };
 };
 export type CanisterConfig = {
-    main: string;
+    main?: string;
     args?: string[];
     candid?: string;
     initArg?: string;
+    "check-stable"?: {
+        path: string;
+        skipIfMissing?: boolean;
+    };
 };
 export type Dependencies = Record<string, Dependency>;
 export type Dependency = {

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/helpers/resolve-canisters.ts

@@ -0,0 +1,52 @@
+import { CanisterConfig, Config } from "../types.js";
+import { cliError } from "../error.js";
+
+export function resolveCanisterConfigs(
+  config: Config,
+): Record<string, CanisterConfig> {
+  if (!config.canisters) {
+    return {};
+  }
+  return Object.fromEntries(
+    Object.entries(config.canisters).map(([name, c]) =>
+      typeof c === "string" ? [name, { main: c }] : [name, c],
+    ),
+  );
+}
+
+export function resolveCanisterEntrypoints(config: Config): string[] {
+  const canisters = resolveCanisterConfigs(config);
+  return Object.values(canisters)
+    .map((c) => c.main)
+    .filter((main): main is string => Boolean(main));
+}
+
+export function resolveSingleCanister(
+  config: Config,
+  canisterName?: string,
+): { name: string; canister: CanisterConfig } {
+  const canisters = resolveCanisterConfigs(config);
+  const names = Object.keys(canisters);
+
+  if (names.length === 0) {
+    cliError("No canisters defined in mops.toml [canisters] section");
+  }
+
+  if (canisterName) {
+    const canister = canisters[canisterName];
+    if (!canister) {
+      cliError(
+        `Canister '${canisterName}' not found in mops.toml. Available: ${names.join(", ")}`,
+      );
+    }
+    return { name: canisterName, canister };
+  }
+
+  if (names.length > 1) {
+    cliError(
+      `Multiple canisters defined in mops.toml. Please specify one: ${names.join(", ")}`,
+    );
+  }
+
+  return { name: names[0]!, canister: canisters[names[0]!]! };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.2.1",
+  "version": "2.3.1",
   "type": "module",
   "bin": {
     "mops": "dist/bin/mops.js",
@@ -84,7 +84,7 @@
     "pic-ic": "0.5.4",
     "pic-js-mops": "0.14.8",
     "prettier": "3.5.3",
-    "prettier-plugin-motoko": "0.12.5",
+    "prettier-plugin-motoko": "0.13.0",
     "promisify-child-process": "4.1.2",
     "prompts": "2.4.2",
     "semver": "7.7.1",

package/tests/README.md

@@ -0,0 +1,16 @@
+# CLI Test Fixtures
+
+Each subdirectory under `check/`, `check-stable/`, etc. is a self-contained test fixture with its own `mops.toml`.
+
+## Adding a new fixture
+
+1. Create a directory with a `mops.toml` and the `.mo` files your test needs.
+2. Only declare `[dependencies]` if your `.mo` files actually import from them. Unused dependencies cause `moc` to receive `--package` flags pointing to directories that may not exist on CI.
+3. If your fixture declares `[dependencies]`, add a `mops install` step for it in `.github/workflows/ci.yml` under the "Pre-cache" step. Test fixtures' `.mops/` directories are gitignored and don't exist on CI unless explicitly installed.
+4. If your fixture uses a `[toolchain]` moc version that isn't already pre-cached in CI, add a download step for it in the same CI pre-cache block.
+
+## Why this matters
+
+Jest runs test suites in parallel. Without pre-installation:
+- Multiple workers race to download the same `moc` binary, corrupting the cache.
+- `moc` fails when `--package` flags point to missing `.mops/` directories.

package/tests/__snapshots__/check-fix.test.ts.snap

@@ -4,7 +4,7 @@ exports[`check --fix M0223 1`] = `
 "// M0223: Redundant type instantiation
 // The type annotation is not needed when it can be inferred
 
-persistent actor {
+actor {
   public func testM0223() : async () {
     func identity<T>(x : T) : T = x;
     let varArray : [var Nat] = [var 1];
@@ -18,7 +18,8 @@ persistent actor {
 exports[`check --fix M0223: fix output 1`] = `
 "Fixed run/M0223.mo (1 fix: M0223)
 
-✓ 1 fix applied to 1 file"
+✓ 1 fix applied to 1 file
+✓ <TEST_DIR>/check/fix/run/M0223.mo"
 `;
 
 exports[`check --fix M0236 1`] = `
@@ -27,7 +28,7 @@ exports[`check --fix M0236 1`] = `
 import List "mo:core/List";
 import Nat "mo:core/Nat";
 
-persistent actor {
+actor {
   public func testM0236() : async () {
     let list = List.fromArray<Nat>([1, 2, 3]);
     list.sortInPlace();
@@ -39,7 +40,8 @@ persistent actor {
 exports[`check --fix M0236: fix output 1`] = `
 "Fixed run/M0236.mo (1 fix: M0236)
 
-✓ 1 fix applied to 1 file"
+✓ 1 fix applied to 1 file
+✓ <TEST_DIR>/check/fix/run/M0236.mo"
 `;
 
 exports[`check --fix M0237 1`] = `
@@ -48,7 +50,7 @@ exports[`check --fix M0237 1`] = `
 import List "mo:core/List";
 import Nat "mo:core/Nat";
 
-persistent actor {
+actor {
   public func testM0237() : async () {
     let list = List.fromArray<Nat>([3, 2, 1]);
     list.sortInPlace();
@@ -60,7 +62,8 @@ persistent actor {
 exports[`check --fix M0237: fix output 1`] = `
 "Fixed run/M0237.mo (1 fix: M0237)
 
-✓ 1 fix applied to 1 file"
+✓ 1 fix applied to 1 file
+✓ <TEST_DIR>/check/fix/run/M0237.mo"
 `;
 
 exports[`check --fix edit-suggestions 1`] = `
@@ -205,7 +208,8 @@ do {
 exports[`check --fix edit-suggestions: fix output 1`] = `
 "Fixed run/edit-suggestions.mo (30 fixes: M0223, M0236, M0237)
 
-✓ 30 fixes applied to 1 file"
+✓ 30 fixes applied to 1 file
+✓ <TEST_DIR>/check/fix/run/edit-suggestions.mo"
 `;
 
 exports[`check --fix overlapping edits 1`] = `
@@ -224,14 +228,16 @@ do {
 exports[`check --fix overlapping edits: fix output 1`] = `
 "Fixed run/overlapping.mo (4 fixes: M0223, M0236)
 
-✓ 4 fixes applied to 1 file"
+✓ 4 fixes applied to 1 file
+✓ <TEST_DIR>/check/fix/run/overlapping.mo"
 `;
 
 exports[`check --fix transitive imports: fix output 1`] = `
 "Fixed run/transitive-lib.mo (1 fix: M0236)
 Fixed run/transitive-main.mo (1 fix: M0223)
 
-✓ 2 fixes applied to 2 files"
+✓ 2 fixes applied to 2 files
+✓ <TEST_DIR>/check/fix/run/transitive-main.mo"
 `;
 
 exports[`check --fix transitive imports: lib file 1`] = `
@@ -250,7 +256,7 @@ module {
 exports[`check --fix transitive imports: main file 1`] = `
 "import Lib "./transitive-lib";
 
-persistent actor {
+actor {
   public func run() : async () {
     func identity<T>(x : T) : T = x;
     let _ = identity(1);

package/tests/__snapshots__/check-stable.test.ts.snap

@@ -0,0 +1,29 @@
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
+
+exports[`check-stable compatible upgrade from .mo file 1`] = `
+{
+  "exitCode": 0,
+  "stderr": "",
+  "stdout": "✓ Stable compatibility check passed for canister 'backend'",
+}
+`;
+
+exports[`check-stable incompatible upgrade from .mo file 1`] = `
+{
+  "exitCode": 1,
+  "stderr": "(unknown location): Compatibility error [M0170], the new type of stable variable \`counter\` is not compatible with the previous version.
+ The previous type
+  var Nat
+ is not a subtype of
+  var Text
+ because the type 
+  Nat
+ is not compatible with type 
+  Text
+ in \`counter\`.
+ Write an explicit migration function, please see https://internetcomputer.org/docs/motoko/fundamentals/actors/compatibility#explicit-migration-using-a-migration-function.
+(unknown location): Compatibility error [M0169], the stable variable \`name\` of the previous version cannot be implicitly discarded. The variable can only be dropped by an explicit migration function, please see https://internetcomputer.org/docs/motoko/fundamentals/actors/compatibility#explicit-migration-using-a-migration-function
+✗ Stable compatibility check failed for canister 'backend'",
+  "stdout": "",
+}
+`;

package/tests/__snapshots__/check.test.ts.snap

@@ -3,9 +3,10 @@
 exports[`check [moc] args are passed to moc 1`] = `
 {
   "exitCode": 1,
-  "stderr": "Warning.mo:3.9-3.15: warning [M0194], unused identifier unused (delete or rename to wildcard \`_\` or \`_unused\`)
+  "stderr": "Warning.mo:3.9-3.15: warning [M0194], unused identifier: \`unused\`
+help: if this is intentional, prefix it with an underscore: \`_unused\`
 ✗ Check failed for file Warning.mo (exit code: 1)",
-  "stdout": "moc < 1.3.0: some diagnostic hints may be missing",
+  "stdout": "",
 }
 `;
 
@@ -18,7 +19,7 @@ cannot produce expected type
   ()
 Error.mo:7.1-7.21: type error [M0057], unbound variable thisshouldnotcompile
 ✗ Check failed for file Error.mo (exit code: 1)",
-  "stdout": "moc < 1.3.0: some diagnostic hints may be missing",
+  "stdout": "",
 }
 `;
 
@@ -27,7 +28,15 @@ exports[`check error 2`] = `
   "exitCode": 1,
   "stderr": "Ok.mo: No such file or directory
 ✗ Check failed for file Ok.mo (exit code: 1)",
-  "stdout": "moc < 1.3.0: some diagnostic hints may be missing",
+  "stdout": "",
+}
+`;
+
+exports[`check no args falls back to [canisters] entrypoints 1`] = `
+{
+  "exitCode": 0,
+  "stderr": "",
+  "stdout": "✓ Ok.mo",
 }
 `;
 
@@ -35,8 +44,7 @@ exports[`check ok 1`] = `
 {
   "exitCode": 0,
   "stderr": "",
-  "stdout": "moc < 1.3.0: some diagnostic hints may be missing
-✓ Ok.mo",
+  "stdout": "✓ Ok.mo",
 }
 `;
 
@@ -44,9 +52,9 @@ exports[`check ok 2`] = `
 {
   "exitCode": 0,
   "stderr": "",
-  "stdout": "moc < 1.3.0: some diagnostic hints may be missing
+  "stdout": "check Using --all-libs for richer diagnostics
 check Running moc:
-moc-wrapper ["Ok.mo","--check","--package","core",".mops/core@2.0.0/src"]
+<CACHE>moc-wrapper ["Ok.mo","--check","--all-libs","--default-persistent-actors"]
 ✓ Ok.mo",
 }
 `;
@@ -54,19 +62,20 @@ moc-wrapper ["Ok.mo","--check","--package","core",".mops/core@2.0.0/src"]
 exports[`check warning 1`] = `
 {
   "exitCode": 0,
-  "stderr": "Warning.mo:3.9-3.15: warning [M0194], unused identifier unused (delete or rename to wildcard \`_\` or \`_unused\`)",
-  "stdout": "moc < 1.3.0: some diagnostic hints may be missing
-✓ Warning.mo",
+  "stderr": "Warning.mo:3.9-3.15: warning [M0194], unused identifier: \`unused\`
+help: if this is intentional, prefix it with an underscore: \`_unused\`",
+  "stdout": "✓ Warning.mo",
 }
 `;
 
 exports[`check warning verbose 1`] = `
 {
   "exitCode": 0,
-  "stderr": "Warning.mo:3.9-3.15: warning [M0194], unused identifier unused (delete or rename to wildcard \`_\` or \`_unused\`)",
-  "stdout": "moc < 1.3.0: some diagnostic hints may be missing
+  "stderr": "Warning.mo:3.9-3.15: warning [M0194], unused identifier: \`unused\`
+help: if this is intentional, prefix it with an underscore: \`_unused\`",
+  "stdout": "check Using --all-libs for richer diagnostics
 check Running moc:
-moc-wrapper ["Warning.mo","--check","--package","core",".mops/core@2.0.0/src"]
+<CACHE>moc-wrapper ["Warning.mo","--check","--all-libs","--default-persistent-actors"]
 ✓ Warning.mo",
 }
 `;
@@ -74,8 +83,9 @@ moc-wrapper ["Warning.mo","--check","--package","core",".mops/core@2.0.0/src"]
 exports[`check warning with -Werror flag 1`] = `
 {
   "exitCode": 1,
-  "stderr": "Warning.mo:3.9-3.15: warning [M0194], unused identifier unused (delete or rename to wildcard \`_\` or \`_unused\`)
+  "stderr": "Warning.mo:3.9-3.15: warning [M0194], unused identifier: \`unused\`
+help: if this is intentional, prefix it with an underscore: \`_unused\`
 ✗ Check failed for file Warning.mo (exit code: 1)",
-  "stdout": "moc < 1.3.0: some diagnostic hints may be missing",
+  "stdout": "",
 }
 `;

package/tests/build/no-dfx/mops.toml

@@ -1,5 +1,8 @@
 [toolchain]
 moc = "1.3.0"
 
+[moc]
+args = ["--default-persistent-actors"]
+
 [canisters.main]
 main = "src/Main.mo"

package/tests/build/no-dfx/src/Main.mo

@@ -1,4 +1,4 @@
-persistent actor {
+actor {
   public func greet(name : Text) : async Text {
     "Hello, " # name # "!";
   };

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);
+    },
+  );
 });

package/tests/check/canisters/Ok.mo

@@ -0,0 +1,5 @@
+actor {
+  public func hello() : async Text {
+    "Hello, World!";
+  };
+};

package/tests/check/canisters/mops.toml

@@ -0,0 +1,8 @@
+[toolchain]
+moc = "1.3.0"
+
+[moc]
+args = ["--default-persistent-actors"]
+
+[canisters.backend]
+main = "Ok.mo"