ic-mops 2.5.0 → 2.6.0

package/dist/commands/watch/warning-checker.js

@@ -4,7 +4,7 @@ import os from "node:os";
 import chalk from "chalk";
 import { getMocPath } from "../../helpers/get-moc-path.js";
 import { getGlobalMocArgs, getRootDir, readConfig } from "../../mops.js";
-import { sources } from "../sources.js";
+import { sourcesArgs } from "../sources.js";
 import { parallel } from "../../parallel.js";
 import { globMoFiles } from "./globMoFiles.js";
 export class WarningChecker {
@@ -50,7 +50,7 @@ export class WarningChecker {
         onProgress();
         let rootDir = getRootDir();
         let mocPath = getMocPath();
-        let deps = await sources({ cwd: rootDir });
+        let deps = (await sourcesArgs({ cwd: rootDir })).flat();
         let globalMocArgs = getGlobalMocArgs(readConfig());
         let paths = globMoFiles(rootDir);
         this.totalFiles = paths.length;
@@ -59,12 +59,7 @@ export class WarningChecker {
             let controller = new AbortController();
             let { signal } = controller;
             this.controllers.set(file, controller);
-            let { stderr } = await promisify(execFile)(mocPath, [
-                "--check",
-                ...deps.flatMap((x) => x.split(" ")),
-                ...globalMocArgs,
-                file,
-            ], { cwd: rootDir, signal }).catch((error) => {
+            let { stderr } = await promisify(execFile)(mocPath, ["--check", ...deps, ...globalMocArgs, file], { cwd: rootDir, signal }).catch((error) => {
                 if (error.code === "ABORT_ERR") {
                     return { stderr: "" };
                 }

package/dist/helpers/find-changelog-entry.js

@@ -15,7 +15,7 @@ export function findChangelogEntry(changelog, version) {
             }
         }
         else if (node.type === "heading" &&
-            toMarkdown(node).match(new RegExp(`\\b${version}\\b`))) {
+            toMarkdown(node).match(new RegExp(`\\b${version.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\b`))) {
             depth = node.depth;
             found = true;
         }

package/dist/integrity.js

@@ -50,7 +50,7 @@ export function getLocalFileHash(fileId) {
     return bytesToHex(sha256(fileData));
 }
 function getMopsTomlHash() {
-    return bytesToHex(sha256(fs.readFileSync(getRootDir() + "/mops.toml")));
+    return bytesToHex(sha256(fs.readFileSync(path.join(getRootDir(), "mops.toml"))));
 }
 function getMopsTomlDepsHash() {
     let config = readConfig();
@@ -90,7 +90,13 @@ export function readLockFile() {
     let rootDir = getRootDir();
     let lockFile = path.join(rootDir, "mops.lock");
     if (fs.existsSync(lockFile)) {
-        return JSON.parse(fs.readFileSync(lockFile).toString());
+        try {
+            return JSON.parse(fs.readFileSync(lockFile).toString());
+        }
+        catch {
+            console.error("mops.lock is corrupted. Delete it and run `mops install` to regenerate.");
+            process.exit(1);
+        }
     }
     return null;
 }
@@ -208,7 +214,7 @@ export async function checkLockFile(force = false) {
         }
         for (let [fileId, lockedHash] of Object.entries(hashes)) {
             // check if file belongs to package
-            if (!fileId.startsWith(packageId)) {
+            if (!fileId.startsWith(packageId + "/")) {
                 console.error("Integrity check failed");
                 console.error(`File ${fileId} in lock file does not belong to package ${packageId}`);
                 process.exit(1);

package/dist/mops.js

@@ -131,6 +131,9 @@ export async function getGithubCommit(repo, ref) {
         res = await fetch(`https://api.github.com/repos/${repo}/commits/main`);
         json = await res.json();
     }
+    if (!res.ok || !json.sha) {
+        throw new Error(`Failed to fetch commit for ${repo}#${ref}: ${json.message || `HTTP ${res.status}`}`);
+    }
     return json;
 }
 export function getDependencyType(version) {

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.5.0",
+  "version": "2.6.0",
   "type": "module",
   "bin": {
     "mops": "bin/mops.js",
@@ -34,7 +34,6 @@
     "@noble/hashes": "1.8.0",
     "as-table": "1.0.55",
     "buffer": "6.0.3",
-    "cacheable-request": "12.0.1",
     "chalk": "5.4.1",
     "change-case": "5.4.4",
     "chokidar": "3.6.0",
@@ -56,7 +55,7 @@
     "markdown-table": "3.0.4",
     "mdast-util-from-markdown": "2.0.2",
     "mdast-util-to-markdown": "2.1.2",
-    "minimatch": "10.0.1",
+    "minimatch": "10.2.4",
     "ncp": "2.0.0",
     "octokit": "3.1.2",
     "pem-file": "1.0.1",
@@ -69,7 +68,7 @@
     "semver": "7.7.1",
     "stream-to-promise": "3.0.0",
     "string-width": "7.2.0",
-    "tar": "7.5.9",
+    "tar": "7.5.11",
     "terminal-size": "4.0.0",
     "vscode-languageserver-textdocument": "1.0.12"
   },
@@ -78,7 +77,6 @@
     "@types/debounce": "1.2.4",
     "@types/decompress": "4.2.7",
     "@types/fs-extra": "11.0.4",
-    "@types/glob": "8.1.0",
     "@types/jsdom": "28.0.0",
     "@types/ncp": "2.0.8",
     "@types/node": "24.0.3",

package/dist/release-cli.js

@@ -29,7 +29,7 @@ fs.cpSync(path.resolve(__dirname, `../cli-releases/versions/${version}.tgz`), pa
 fs.writeFileSync(path.resolve(__dirname, `../cli-releases/tags/${tag}`), version);
 console.log(`Release '${version}' created with tag '${tag}'`);
 if (!fs.existsSync(path.resolve(__dirname, "../cli-releases/releases.json"))) {
-    fs.writeFileSync(path.resolve(__dirname, "../cli-releases/releases.json"), JSON.stringify({ tags: {}, versions: {} }, null, 2));
+    fs.writeFileSync(path.resolve(__dirname, "../cli-releases/releases.json"), JSON.stringify({ tags: {}, versions: {} }, null, 2) + "\n");
 }
 let releases = JSON.parse(fs.readFileSync(path.resolve(__dirname, "../cli-releases/releases.json"), "utf8"));
 releases.tags[tag] = version;
@@ -41,4 +41,4 @@ releases.versions[version] = {
     url: `https://cli.mops.one/versions/${version}.tgz`,
     hash,
 };
-fs.writeFileSync(path.resolve(__dirname, "../cli-releases/releases.json"), JSON.stringify(releases, null, 2));
+fs.writeFileSync(path.resolve(__dirname, "../cli-releases/releases.json"), JSON.stringify(releases, null, 2) + "\n");

package/dist/resolve-packages.js

@@ -21,8 +21,8 @@ export async function resolvePackages({ conflicts = "ignore", } = {}) {
     let packages = {};
     let versions = {};
     let compareVersions = (a = "0.0.0", b = "0.0.0") => {
-        let ap = a.split(".").map((x) => parseInt(x));
-        let bp = b.split(".").map((x) => parseInt(x));
+        let ap = a.split(".").map((x) => parseInt(x) || 0);
+        let bp = b.split(".").map((x) => parseInt(x) || 0);
         if (ap[0] - bp[0]) {
             return Math.sign(ap[0] - bp[0]);
         }
@@ -97,7 +97,7 @@ export async function resolvePackages({ conflicts = "ignore", } = {}) {
             }
             else if (version) {
                 let cacheDir = getDepCacheName(name, version);
-                nestedConfig = readConfig(getDepCacheDir(cacheDir) + "/mops.toml");
+                nestedConfig = readConfig(path.join(getDepCacheDir(cacheDir), "mops.toml"));
             }
             // collect nested deps
             if (nestedConfig) {
@@ -137,7 +137,7 @@ export async function resolvePackages({ conflicts = "ignore", } = {}) {
             if (majors.size > 1) {
                 console.error(chalk.reset("") +
                     chalk.redBright(conflicts === "error" ? "Error!" : "Warning!"), `Conflicting versions of dependency "${dep}"`);
-                for (let { version, dependencyOf } of vers.reverse()) {
+                for (let { version, dependencyOf } of [...vers].reverse()) {
                     console.error(chalk.reset("  ") +
                         `${dep} ${chalk.bold.red(version.split(".")[0])}.${version.split(".").slice(1).join(".")} is dependency of ${chalk.bold(dependencyOf)}`);
                 }

package/dist/tests/build.test.js

@@ -11,7 +11,7 @@ function cleanFixture(cwd, ...extras) {
     }
 }
 describe("build", () => {
-    // Several dfx/pocket-ic builds per test; slow CI (e.g. node 20 matrix) can exceed 60s default.
+    // Several dfx/pocket-ic builds per test; slow CI can exceed 60s default.
     jest.setTimeout(120_000);
     test("ok", async () => {
         const cwd = path.join(import.meta.dirname, "build/success");

package/dist/tests/check.test.js

@@ -94,4 +94,28 @@ describe("check", () => {
         expect(result.stderr).toMatch(/error/i);
         expect(result.stdout).not.toMatch(/Stable compatibility/);
     });
+    test("lint runs after moc check and passes", async () => {
+        const cwd = path.join(import.meta.dirname, "check/with-lint-pass");
+        const result = await cli(["check"], { cwd });
+        expect(result.exitCode).toBe(0);
+        expect(result.stdout).toMatch(/✓ Lint succeeded/);
+    });
+    test("check fails when lint finds errors", async () => {
+        const cwd = path.join(import.meta.dirname, "check/with-lint-fail");
+        const result = await cli(["check"], { cwd });
+        expect(result.exitCode).toBe(1);
+        expect(result.stderr).toMatch(/no-bool-switch/);
+    });
+    test("lint is skipped when lintoko not configured and no rules exist", async () => {
+        const cwd = path.join(import.meta.dirname, "check/canisters");
+        const result = await cli(["check"], { cwd });
+        expect(result.exitCode).toBe(0);
+        expect(result.stdout).not.toMatch(/Lint/);
+    });
+    test("--fix flag reaches lint step", async () => {
+        const cwd = path.join(import.meta.dirname, "check/with-lint-pass");
+        const result = await cli(["check", "--fix"], { cwd });
+        expect(result.exitCode).toBe(0);
+        expect(result.stdout).toMatch(/✓ Lint fixes applied/);
+    });
 });

package/dist/tests/helpers.js

@@ -2,8 +2,15 @@ import { expect } from "@jest/globals";
 import { execa } from "execa";
 import { dirname } from "path";
 import { fileURLToPath } from "url";
+// When MOPS_TEST_GLOBAL is set, invoke the globally-installed `mops` binary
+// directly rather than the npm script. This exercises the real global-install
+// code path where the binary lives outside the project tree.
+const useGlobalBinary = Boolean(process.env.MOPS_TEST_GLOBAL);
 export const cli = async (args, { cwd } = {}) => {
-    return await execa("npm", ["run", "--silent", "mops", "--", ...args], {
+    const [cmd, cmdArgs] = useGlobalBinary
+        ? ["mops", args]
+        : ["npm", ["run", "--silent", "mops", "--", ...args]];
+    return await execa(cmd, cmdArgs, {
         env: { ...process.env, ...(cwd != null && { MOPS_CWD: cwd }) },
         ...(cwd != null && { cwd }),
         stdio: "pipe",

package/dist/tests/lint.test.js

@@ -1,6 +1,6 @@
-import { describe, test } from "@jest/globals";
+import { describe, expect, test } from "@jest/globals";
 import path from "path";
-import { cliSnapshot } from "./helpers";
+import { cli, cliSnapshot } from "./helpers";
 describe("lint", () => {
     test("ok", async () => {
         const cwd = path.join(import.meta.dirname, "lint");
@@ -12,4 +12,30 @@ describe("lint", () => {
         await cliSnapshot(["lint", "NoBoolSwitch", "--verbose"], { cwd }, 1);
         await cliSnapshot(["lint", "DoesNotExist"], { cwd }, 1);
     });
+    test("[lint] rules - additional config rules directory is used", async () => {
+        const cwd = path.join(import.meta.dirname, "lint-config-rules");
+        const result = await cli(["lint"], { cwd });
+        expect(result.exitCode).toBe(1);
+        expect(result.stderr).toMatch(/no-bool-switch/);
+    });
+    test("[lint] extends - picks up rules/ from named dependency", async () => {
+        const cwd = path.join(import.meta.dirname, "lint-extends");
+        const result = await cli(["lint"], { cwd });
+        expect(result.exitCode).toBe(1);
+        expect(result.stderr).toMatch(/no-bool-switch/);
+    });
+    test("[lint] extends true - picks up rules/ from all dependencies", async () => {
+        const cwd = path.join(import.meta.dirname, "lint-extends-all");
+        const result = await cli(["lint"], { cwd });
+        expect(result.exitCode).toBe(1);
+        expect(result.stderr).toMatch(/no-bool-switch/);
+    });
+    test("[lint] extends - dep not in extends list is ignored", async () => {
+        // my-pkg has rules/ but extends only lists "other-pkg" (which doesn't exist),
+        // so no dep rules are loaded and NoBoolSwitch.mo passes with exit 0.
+        const cwd = path.join(import.meta.dirname, "lint-extends-ignored");
+        const result = await cli(["lint"], { cwd });
+        expect(result.exitCode).toBe(0);
+        expect(result.stderr).toMatch(/not found in dependencies/);
+    });
 });

package/dist/types.d.ts

@@ -29,6 +29,8 @@ export type Config = {
     };
     lint?: {
         args?: string[];
+        rules?: string[];
+        extends?: string[] | true;
     };
 };
 export type CanisterConfig = {

package/dist/vessel.d.ts

@@ -18,4 +18,4 @@ export declare const installFromGithub: (name: string, repo: string, { verbose,
     dep?: boolean | undefined;
     silent?: boolean | undefined;
     ignoreTransitive?: boolean | undefined;
-}) => Promise<void>;
+}) => Promise<boolean>;

package/dist/vessel.js

@@ -141,7 +141,7 @@ export const installFromGithub = async (name, repo, { verbose = false, dep = fal
         }
         catch (err) {
             deleteSync([cacheDir], { force: true });
-            process.exit(1);
+            return false;
         }
     }
     if (verbose) {
@@ -151,7 +151,7 @@ export const installFromGithub = async (name, repo, { verbose = false, dep = fal
         logUpdate.clear();
     }
     if (ignoreTransitive) {
-        return;
+        return true;
     }
     const config = await readVesselConfig(cacheDir, { silent });
     if (config) {
@@ -161,4 +161,5 @@ export const installFromGithub = async (name, repo, { verbose = false, dep = fal
             }
         }
     }
+    return true;
 };

package/helpers/find-changelog-entry.ts

@@ -16,7 +16,9 @@ export function findChangelogEntry(changelog: string, version: string): string {
       }
     } else if (
       node.type === "heading" &&
-      toMarkdown(node).match(new RegExp(`\\b${version}\\b`))
+      toMarkdown(node).match(
+        new RegExp(`\\b${version.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\b`),
+      )
     ) {
       depth = node.depth;
       found = true;

package/integrity.ts

@@ -87,7 +87,9 @@ export function getLocalFileHash(fileId: string): string {
 }
 
 function getMopsTomlHash(): string {
-  return bytesToHex(sha256(fs.readFileSync(getRootDir() + "/mops.toml")));
+  return bytesToHex(
+    sha256(fs.readFileSync(path.join(getRootDir(), "mops.toml"))),
+  );
 }
 
 function getMopsTomlDepsHash(): string {
@@ -137,7 +139,14 @@ export function readLockFile(): LockFile | null {
   let rootDir = getRootDir();
   let lockFile = path.join(rootDir, "mops.lock");
   if (fs.existsSync(lockFile)) {
-    return JSON.parse(fs.readFileSync(lockFile).toString()) as LockFile;
+    try {
+      return JSON.parse(fs.readFileSync(lockFile).toString()) as LockFile;
+    } catch {
+      console.error(
+        "mops.lock is corrupted. Delete it and run `mops install` to regenerate.",
+      );
+      process.exit(1);
+    }
   }
   return null;
 }
@@ -290,7 +299,7 @@ export async function checkLockFile(force = false) {
 
     for (let [fileId, lockedHash] of Object.entries(hashes)) {
       // check if file belongs to package
-      if (!fileId.startsWith(packageId)) {
+      if (!fileId.startsWith(packageId + "/")) {
         console.error("Integrity check failed");
         console.error(
           `File ${fileId} in lock file does not belong to package ${packageId}`,

package/mops.ts

@@ -152,6 +152,13 @@ export async function getGithubCommit(repo: string, ref: string): Promise<any> {
     res = await fetch(`https://api.github.com/repos/${repo}/commits/main`);
     json = await res.json();
   }
+
+  if (!res.ok || !json.sha) {
+    throw new Error(
+      `Failed to fetch commit for ${repo}#${ref}: ${json.message || `HTTP ${res.status}`}`,
+    );
+  }
+
   return json;
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.5.0",
+  "version": "2.6.0",
   "type": "module",
   "bin": {
     "mops": "dist/bin/mops.js",
@@ -55,7 +55,6 @@
     "@noble/hashes": "1.8.0",
     "as-table": "1.0.55",
     "buffer": "6.0.3",
-    "cacheable-request": "12.0.1",
     "chalk": "5.4.1",
     "change-case": "5.4.4",
     "chokidar": "3.6.0",
@@ -77,7 +76,7 @@
     "markdown-table": "3.0.4",
     "mdast-util-from-markdown": "2.0.2",
     "mdast-util-to-markdown": "2.1.2",
-    "minimatch": "10.0.1",
+    "minimatch": "10.2.4",
     "ncp": "2.0.0",
     "octokit": "3.1.2",
     "pem-file": "1.0.1",
@@ -90,7 +89,7 @@
     "semver": "7.7.1",
     "stream-to-promise": "3.0.0",
     "string-width": "7.2.0",
-    "tar": "7.5.9",
+    "tar": "7.5.11",
     "terminal-size": "4.0.0",
     "vscode-languageserver-textdocument": "1.0.12"
   },
@@ -99,7 +98,6 @@
     "@types/debounce": "1.2.4",
     "@types/decompress": "4.2.7",
     "@types/fs-extra": "11.0.4",
-    "@types/glob": "8.1.0",
     "@types/jsdom": "28.0.0",
     "@types/ncp": "2.0.8",
     "@types/node": "24.0.3",

package/release-cli.ts

@@ -79,7 +79,7 @@ type Releases = {
 if (!fs.existsSync(path.resolve(__dirname, "../cli-releases/releases.json"))) {
   fs.writeFileSync(
     path.resolve(__dirname, "../cli-releases/releases.json"),
-    JSON.stringify({ tags: {}, versions: {} }, null, 2),
+    JSON.stringify({ tags: {}, versions: {} }, null, 2) + "\n",
   );
 }
 
@@ -102,5 +102,5 @@ releases.versions[version] = {
 
 fs.writeFileSync(
   path.resolve(__dirname, "../cli-releases/releases.json"),
-  JSON.stringify(releases, null, 2),
+  JSON.stringify(releases, null, 2) + "\n",
 );

package/resolve-packages.ts

@@ -41,12 +41,12 @@ export async function resolvePackages({
   > = {};
 
   let compareVersions = (a: string = "0.0.0", b: string = "0.0.0") => {
-    let ap = a.split(".").map((x: string) => parseInt(x)) as [
+    let ap = a.split(".").map((x: string) => parseInt(x) || 0) as [
       number,
       number,
       number,
     ];
-    let bp = b.split(".").map((x: string) => parseInt(x)) as [
+    let bp = b.split(".").map((x: string) => parseInt(x) || 0) as [
       number,
       number,
       number,
@@ -138,7 +138,9 @@ export async function resolvePackages({
         }
       } else if (version) {
         let cacheDir = getDepCacheName(name, version);
-        nestedConfig = readConfig(getDepCacheDir(cacheDir) + "/mops.toml");
+        nestedConfig = readConfig(
+          path.join(getDepCacheDir(cacheDir), "mops.toml"),
+        );
       }
 
       // collect nested deps
@@ -193,7 +195,7 @@ export async function resolvePackages({
           `Conflicting versions of dependency "${dep}"`,
         );
 
-        for (let { version, dependencyOf } of vers.reverse()) {
+        for (let { version, dependencyOf } of [...vers].reverse()) {
           console.error(
             chalk.reset("  ") +
               `${dep} ${chalk.bold.red(version.split(".")[0])}.${version.split(".").slice(1).join(".")} is dependency of ${chalk.bold(dependencyOf)}`,

package/tests/build.test.ts

@@ -14,7 +14,7 @@ function cleanFixture(cwd: string, ...extras: string[]) {
 }
 
 describe("build", () => {
-  // Several dfx/pocket-ic builds per test; slow CI (e.g. node 20 matrix) can exceed 60s default.
+  // Several dfx/pocket-ic builds per test; slow CI can exceed 60s default.
   jest.setTimeout(120_000);
 
   test("ok", async () => {

package/tests/check/with-lint-fail/NoBoolSwitch.mo

@@ -0,0 +1,8 @@
+module {
+  public func boolSwitch(b : Bool) : Bool {
+    switch (b) {
+      case false { false };
+      case true { true };
+    };
+  };
+};

package/tests/check/with-lint-fail/lints/no-bool-switch.toml

@@ -0,0 +1,9 @@
+name = "no-bool-switch"
+description = "Don't switch on boolean values, use if instead"
+query = """
+(switch_exp
+ (case [
+   (lit_pat (bool_literal))
+   (tup_pat . (lit_pat (bool_literal)) @trailing)
+   ])) @error
+"""

package/tests/check/with-lint-fail/mops.toml

@@ -0,0 +1,9 @@
+[toolchain]
+moc = "1.3.0"
+lintoko = "0.7.0"
+
+[moc]
+args = ["--default-persistent-actors"]
+
+[canisters.backend]
+main = "NoBoolSwitch.mo"

package/tests/check/with-lint-pass/Ok.mo

@@ -0,0 +1,5 @@
+module {
+  public func greet(name : Text) : Text {
+    "Hello, " # name # "!";
+  };
+};

package/tests/check/with-lint-pass/lints/no-bool-switch.toml

@@ -0,0 +1,9 @@
+name = "no-bool-switch"
+description = "Don't switch on boolean values, use if instead"
+query = """
+(switch_exp
+ (case [
+   (lit_pat (bool_literal))
+   (tup_pat . (lit_pat (bool_literal)) @trailing)
+   ])) @error
+"""

package/tests/check/with-lint-pass/mops.toml

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

package/tests/check.test.ts

@@ -121,4 +121,32 @@ describe("check", () => {
     expect(result.stderr).toMatch(/error/i);
     expect(result.stdout).not.toMatch(/Stable compatibility/);
   });
+
+  test("lint runs after moc check and passes", async () => {
+    const cwd = path.join(import.meta.dirname, "check/with-lint-pass");
+    const result = await cli(["check"], { cwd });
+    expect(result.exitCode).toBe(0);
+    expect(result.stdout).toMatch(/✓ Lint succeeded/);
+  });
+
+  test("check fails when lint finds errors", async () => {
+    const cwd = path.join(import.meta.dirname, "check/with-lint-fail");
+    const result = await cli(["check"], { cwd });
+    expect(result.exitCode).toBe(1);
+    expect(result.stderr).toMatch(/no-bool-switch/);
+  });
+
+  test("lint is skipped when lintoko not configured and no rules exist", async () => {
+    const cwd = path.join(import.meta.dirname, "check/canisters");
+    const result = await cli(["check"], { cwd });
+    expect(result.exitCode).toBe(0);
+    expect(result.stdout).not.toMatch(/Lint/);
+  });
+
+  test("--fix flag reaches lint step", async () => {
+    const cwd = path.join(import.meta.dirname, "check/with-lint-pass");
+    const result = await cli(["check", "--fix"], { cwd });
+    expect(result.exitCode).toBe(0);
+    expect(result.stdout).toMatch(/✓ Lint fixes applied/);
+  });
 });

package/tests/helpers.ts

@@ -7,8 +7,16 @@ export interface CliOptions {
   cwd?: string;
 }
 
+// When MOPS_TEST_GLOBAL is set, invoke the globally-installed `mops` binary
+// directly rather than the npm script. This exercises the real global-install
+// code path where the binary lives outside the project tree.
+const useGlobalBinary = Boolean(process.env.MOPS_TEST_GLOBAL);
+
 export const cli = async (args: string[], { cwd }: CliOptions = {}) => {
-  return await execa("npm", ["run", "--silent", "mops", "--", ...args], {
+  const [cmd, cmdArgs] = useGlobalBinary
+    ? ["mops", args]
+    : ["npm", ["run", "--silent", "mops", "--", ...args]];
+  return await execa(cmd, cmdArgs, {
     env: { ...process.env, ...(cwd != null && { MOPS_CWD: cwd }) },
     ...(cwd != null && { cwd }),
     stdio: "pipe",

package/tests/lint-config-rules/extra-rules/no-bool-switch.toml

@@ -0,0 +1,9 @@
+name = "no-bool-switch"
+description = "Don't switch on boolean values, use if instead"
+query = """
+(switch_exp
+ (case [
+   (lit_pat (bool_literal))
+   (tup_pat . (lit_pat (bool_literal)) @trailing)
+   ])) @error
+"""

package/tests/lint-config-rules/mops.toml

@@ -0,0 +1,5 @@
+[toolchain]
+lintoko = "0.7.0"
+
+[lint]
+rules = ["extra-rules"]

package/tests/lint-config-rules/src/NoBoolSwitch.mo

@@ -0,0 +1,8 @@
+module {
+  public func boolSwitch(b : Bool) : Bool {
+    switch (b) {
+      case false { false };
+      case true { true };
+    };
+  };
+};

package/tests/lint-extends/mops.toml

@@ -0,0 +1,8 @@
+[dependencies]
+my-pkg = "./my-pkg"
+
+[toolchain]
+lintoko = "0.7.0"
+
+[lint]
+extends = ["my-pkg"]

package/tests/lint-extends/my-pkg/mops.toml

@@ -0,0 +1,3 @@
+[package]
+name = "my-pkg"
+version = "0.1.0"

package/tests/lint-extends/my-pkg/rules/no-bool-switch.toml

@@ -0,0 +1,9 @@
+name = "no-bool-switch"
+description = "Don't switch on boolean values, use if instead"
+query = """
+(switch_exp
+ (case [
+   (lit_pat (bool_literal))
+   (tup_pat . (lit_pat (bool_literal)) @trailing)
+   ])) @error
+"""