workspace-tools 0.18.2 → 0.19.0

package/CHANGELOG.json

@@ -2,7 +2,52 @@
   "name": "workspace-tools",
   "entries": [
     {
-      "date": "Fri, 07 Jan 2022 18:15:30 GMT",
+      "date": "Thu, 05 May 2022 19:40:19 GMT",
+      "tag": "workspace-tools_v0.19.0",
+      "version": "0.19.0",
+      "comments": {
+        "minor": [
+          {
+            "author": "kchau@microsoft.com",
+            "package": "workspace-tools",
+            "comment": "adds a new API to allow retrieving a list of packages affected by files and also by git ref range",
+            "commit": "8d3b9dd8a8e1831b56a5f1606491be403debcbed"
+          }
+        ]
+      }
+    },
+    {
+      "date": "Wed, 20 Apr 2022 16:49:02 GMT",
+      "tag": "workspace-tools_v0.18.4",
+      "version": "0.18.4",
+      "comments": {
+        "patch": [
+          {
+            "author": "kchau@microsoft.com",
+            "package": "workspace-tools",
+            "comment": "fixes a potential security issue where fetch --upload-pack can allow for command injection",
+            "commit": "9bc7e65ce497f87e1f363fd47b8f802f3d3cd978"
+          }
+        ]
+      }
+    },
+    {
+      "date": "Sat, 09 Apr 2022 15:51:14 GMT",
+      "tag": "workspace-tools_v0.18.3",
+      "version": "0.18.3",
+      "comments": {
+        "patch": [
+          {
+            "author": "4123478+tido64@users.noreply.github.com",
+            "package": "workspace-tools",
+            "comment": "Fix Rush not being detected correctly. When Rush is set up to use Yarn or pnpm, the lock file for the latter are found first.",
+            "commit": "b99f6f82a6f22da37b67d74b519a204abd631c87"
+          }
+        ]
+      }
+    },
+    {
+      "date": "Fri, 07 Jan 2022 18:15:36 GMT",
       "tag": "workspace-tools_v0.18.2",
       "version": "0.18.2",
       "comments": {

package/CHANGELOG.md

@@ -1,12 +1,36 @@
 # Change Log - workspace-tools
 
-This log was last generated on Fri, 07 Jan 2022 18:15:30 GMT and should not be manually modified.
+This log was last generated on Thu, 05 May 2022 19:40:19 GMT and should not be manually modified.
 
 <!-- Start content -->
 
+## 0.19.0
+
+Thu, 05 May 2022 19:40:19 GMT
+
+### Minor changes
+
+- adds a new API to allow retrieving a list of packages affected by files and also by git ref range (kchau@microsoft.com)
+
+## 0.18.4
+
+Wed, 20 Apr 2022 16:49:02 GMT
+
+### Patches
+
+- fixes a potential security issue where fetch --upload-pack can allow for command injection (kchau@microsoft.com)
+
+## 0.18.3
+
+Sat, 09 Apr 2022 15:51:14 GMT
+
+### Patches
+
+- Fix Rush not being detected correctly. When Rush is set up to use Yarn or pnpm, the lock file for the latter are found first. (4123478+tido64@users.noreply.github.com)
+
 ## 0.18.2
 
-Fri, 07 Jan 2022 18:15:30 GMT
+Fri, 07 Jan 2022 18:15:36 GMT
 
 ### Patches
 

package/lib/__tests__/getChangedPackages.test.js

@@ -122,4 +122,21 @@ describe("getChangedPackages()", () => {
         // assert
         expect(changedPkgs).toEqual([]);
     });
+    it("can detect changed packages between two refs", () => {
+        // arrange
+        const root = (0, setupFixture_1.setupFixture)("monorepo");
+        const newFile = path_1.default.join(root, "packages/package-a/footest.txt");
+        fs_1.default.writeFileSync(newFile, "hello foo test");
+        (0, git_1.git)(["add", newFile], { cwd: root });
+        (0, git_1.stageAndCommit)(["packages/package-a/footest.txt"], "test commit in a", root);
+        const newFile2 = path_1.default.join(root, "packages/package-b/footest2.txt");
+        fs_1.default.writeFileSync(newFile2, "hello foo test");
+        (0, git_1.git)(["add", newFile2], { cwd: root });
+        (0, git_1.stageAndCommit)(["packages/package-b/footest2.txt"], "test commit in b", root);
+        // act
+        const changedPkgs = (0, getChangedPackages_1.getChangedPackagesBetweenRefs)(root, "HEAD^1", "HEAD");
+        // assert
+        expect(changedPkgs).toContain("package-b");
+        expect(changedPkgs).not.toContain("package-a");
+    });
 });

package/lib/__tests__/getPackagesByFiles.test.d.ts

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

package/lib/__tests__/getPackagesByFiles.test.js

@@ -0,0 +1,54 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const path_1 = __importDefault(require("path"));
+const fs_1 = __importDefault(require("fs"));
+const setupFixture_1 = require("../helpers/setupFixture");
+const getPackagesByFiles_1 = require("../workspaces/getPackagesByFiles");
+describe("getPackagesByFiles()", () => {
+    afterAll(() => {
+        (0, setupFixture_1.cleanupFixtures)();
+    });
+    it("can find all packages that contain the files in a monorepo", () => {
+        // arrange
+        const root = (0, setupFixture_1.setupFixture)("monorepo");
+        const newFile = path_1.default.join(root, "packages/package-a/footest.txt");
+        fs_1.default.writeFileSync(newFile, "hello foo test");
+        // act
+        const packages = (0, getPackagesByFiles_1.getPackagesByFiles)(root, ["packages/package-a/footest.txt"]);
+        // assert
+        expect(packages).toContain("package-a");
+        expect(packages).not.toContain("package-b");
+    });
+    it("can find can ignore changes in a glob pattern", () => {
+        // arrange
+        const root = (0, setupFixture_1.setupFixture)("monorepo");
+        const newFileA = path_1.default.join(root, "packages/package-a/footest.txt");
+        fs_1.default.writeFileSync(newFileA, "hello foo test");
+        const newFileB = path_1.default.join(root, "packages/package-b/footest.txt");
+        fs_1.default.writeFileSync(newFileB, "hello foo test");
+        // act
+        const packages = (0, getPackagesByFiles_1.getPackagesByFiles)(root, ["packages/package-a/footest.txt", "packages/package-b/footest.txt"], ["packages/package-b/**"]);
+        // assert
+        expect(packages).toContain("package-a");
+        expect(packages).not.toContain("package-b");
+    });
+    it("can find can handle empty files", () => {
+        // arrange
+        const root = (0, setupFixture_1.setupFixture)("monorepo");
+        // act
+        const packages = (0, getPackagesByFiles_1.getPackagesByFiles)(root, []);
+        // assert
+        expect(packages.length).toBe(0);
+    });
+    it("can find can handle unrelated files", () => {
+        // arrange
+        const root = (0, setupFixture_1.setupFixture)("monorepo");
+        // act
+        const packages = (0, getPackagesByFiles_1.getPackagesByFiles)(root, ["package.json"]);
+        // assert
+        expect(packages.length).toBe(0);
+    });
+});

package/lib/__tests__/getWorkspaces.test.js

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 const path_1 = __importDefault(require("path"));
 const setupFixture_1 = require("../helpers/setupFixture");
+const implementations_1 = require("../workspaces/implementations");
 const yarn_1 = require("../workspaces/implementations/yarn");
 const pnpm_1 = require("../workspaces/implementations/pnpm");
 const rush_1 = require("../workspaces/implementations/rush");
@@ -17,6 +18,7 @@ describe("getWorkspaces", () => {
     describe("yarn", () => {
         it("gets the name and path of the workspaces", () => {
             const packageRoot = (0, setupFixture_1.setupFixture)("monorepo");
+            expect((0, implementations_1.getWorkspaceImplementation)(packageRoot, {})).toBe("yarn");
             const workspacesPackageInfo = (0, yarn_1.getYarnWorkspaces)(packageRoot);
             const packageAPath = path_1.default.join(packageRoot, "packages", "package-a");
             const packageBPath = path_1.default.join(packageRoot, "packages", "package-b");
@@ -27,6 +29,7 @@ describe("getWorkspaces", () => {
         });
         it("gets the name and path of the workspaces against a packages spec of an individual package", () => {
             const packageRoot = (0, setupFixture_1.setupFixture)("monorepo-globby");
+            expect((0, implementations_1.getWorkspaceImplementation)(packageRoot, {})).toBe("yarn");
             const workspacesPackageInfo = (0, yarn_1.getYarnWorkspaces)(packageRoot);
             const packageAPath = path_1.default.join(packageRoot, "packages", "package-a");
             const packageBPath = path_1.default.join(packageRoot, "packages", "package-b");
@@ -41,6 +44,7 @@ describe("getWorkspaces", () => {
     describe("pnpm", () => {
         it("gets the name and path of the workspaces", () => {
             const packageRoot = (0, setupFixture_1.setupFixture)("monorepo-pnpm");
+            expect((0, implementations_1.getWorkspaceImplementation)(packageRoot, {})).toBe("pnpm");
             const workspacesPackageInfo = (0, pnpm_1.getPnpmWorkspaces)(packageRoot);
             const packageAPath = path_1.default.join(packageRoot, "packages", "package-a");
             const packageBPath = path_1.default.join(packageRoot, "packages", "package-b");
@@ -50,9 +54,23 @@ describe("getWorkspaces", () => {
             ]);
         });
     });
-    describe("rush", () => {
+    describe("rush + pnpm", () => {
         it("gets the name and path of the workspaces", () => {
             const packageRoot = (0, setupFixture_1.setupFixture)("monorepo-rush-pnpm");
+            expect((0, implementations_1.getWorkspaceImplementation)(packageRoot, {})).toBe("rush");
+            const workspacesPackageInfo = (0, rush_1.getRushWorkspaces)(packageRoot);
+            const packageAPath = path_1.default.join(packageRoot, "packages", "package-a");
+            const packageBPath = path_1.default.join(packageRoot, "packages", "package-b");
+            expect(workspacesPackageInfo).toMatchObject([
+                { name: "package-a", path: packageAPath },
+                { name: "package-b", path: packageBPath },
+            ]);
+        });
+    });
+    describe("rush + yarn", () => {
+        it("gets the name and path of the workspaces", () => {
+            const packageRoot = (0, setupFixture_1.setupFixture)("monorepo-rush-yarn");
+            expect((0, implementations_1.getWorkspaceImplementation)(packageRoot, {})).toBe("rush");
             const workspacesPackageInfo = (0, rush_1.getRushWorkspaces)(packageRoot);
             const packageAPath = path_1.default.join(packageRoot, "packages", "package-a");
             const packageBPath = path_1.default.join(packageRoot, "packages", "package-b");
@@ -65,6 +83,7 @@ describe("getWorkspaces", () => {
     describe("npm", () => {
         it("gets the name and path of the workspaces", () => {
             const packageRoot = (0, setupFixture_1.setupFixture)("monorepo-npm");
+            expect((0, implementations_1.getWorkspaceImplementation)(packageRoot, {})).toBe("npm");
             const workspacesPackageInfo = (0, npm_1.getNpmWorkspaces)(packageRoot);
             const packageAPath = path_1.default.join(packageRoot, "packages", "package-a");
             const packageBPath = path_1.default.join(packageRoot, "packages", "package-b");
@@ -75,6 +94,7 @@ describe("getWorkspaces", () => {
         });
         it("gets the name and path of the workspaces using the shorthand configuration", () => {
             const packageRoot = (0, setupFixture_1.setupFixture)("monorepo-shorthand");
+            expect((0, implementations_1.getWorkspaceImplementation)(packageRoot, {})).toBe("npm");
             const workspacesPackageInfo = (0, npm_1.getNpmWorkspaces)(packageRoot);
             const packageAPath = path_1.default.join(packageRoot, "packages", "package-a");
             const packageBPath = path_1.default.join(packageRoot, "packages", "package-b");
@@ -88,7 +108,8 @@ describe("getWorkspaces", () => {
     });
     describe("lerna", () => {
         it("gets the name and path of the workspaces", async () => {
-            const packageRoot = await (0, setupFixture_1.setupFixture)("monorepo-lerna-npm");
+            const packageRoot = (0, setupFixture_1.setupFixture)("monorepo-lerna-npm");
+            expect((0, implementations_1.getWorkspaceImplementation)(packageRoot, {})).toBe("lerna");
             const workspacesPackageInfo = (0, lerna_1.getLernaWorkspaces)(packageRoot);
             const packageAPath = path_1.default.join(packageRoot, "packages", "package-a");
             const packageBPath = path_1.default.join(packageRoot, "packages", "package-b");

package/lib/git.js

@@ -104,14 +104,14 @@ function getUntrackedChanges(cwd) {
 }
 exports.getUntrackedChanges = getUntrackedChanges;
 function fetchRemote(remote, cwd) {
-    const results = git(["fetch", remote], { cwd });
+    const results = git(["fetch", "--", remote], { cwd });
     if (!results.success) {
         throw gitError(`Cannot fetch remote: ${remote}`);
     }
 }
 exports.fetchRemote = fetchRemote;
 function fetchRemoteBranch(remote, remoteBranch, cwd) {
-    const results = git(["fetch", remote, remoteBranch], { cwd });
+    const results = git(["fetch", "--", remote, remoteBranch], { cwd });
     if (!results.success) {
         throw gitError(`Cannot fetch remote: ${remote} ${remoteBranch}`);
     }
@@ -145,17 +145,12 @@ exports.getChanges = getChanges;
  * @param cwd
  */
 function getBranchChanges(branch, cwd) {
-    try {
-        return processGitOutput(git(["--no-pager", "diff", "--name-only", "--relative", branch + "..."], { cwd }));
-    }
-    catch (e) {
-        throw gitError(`Cannot gather information about branch changes`, e);
-    }
+    return getChangesBetweenRefs(branch, "", [], "", cwd);
 }
 exports.getBranchChanges = getBranchChanges;
 function getChangesBetweenRefs(fromRef, toRef, options, pattern, cwd) {
     try {
-        return processGitOutput(git(["--no-pager", "diff", "--relative", "--name-only", ...options, `${fromRef}...${toRef}`, "--", pattern], {
+        return processGitOutput(git(["--no-pager", "diff", "--name-only", "--relative", ...options, `${fromRef}...${toRef}`, ...(pattern ? ["--", pattern] : [])], {
             cwd,
         }));
     }

package/lib/helpers/setupFixture.js

@@ -35,6 +35,8 @@ function setupFixture(fixtureName) {
     fs_extra_1.default.mkdirpSync(cwd);
     fs_extra_1.default.copySync(fixturePath, cwd);
     (0, git_1.init)(cwd, "test@test.email", "test user");
+    // Ensure GPG signing doesn't interfere with tests
+    (0, git_1.gitFailFast)(["config", "commit.gpgsign", "false"], { cwd });
     // Make the 'main' branch the default in the test repo
     // ensure that the configuration for this repo does not collide
     // with any global configuration the user had made, so we have

package/lib/index.d.ts

@@ -14,5 +14,6 @@ export * from "./workspaces/implementations/pnpm";
 export * from "./workspaces/implementations/rush";
 export * from "./workspaces/implementations/yarn";
 export * from "./workspaces/getChangedPackages";
+export * from "./workspaces/getPackagesByFiles";
 export * from "./workspaces/listOfWorkspacePackageNames";
 export * from "./workspaces/workspaces";

package/lib/index.js

@@ -26,5 +26,6 @@ __exportStar(require("./workspaces/implementations/pnpm"), exports);
 __exportStar(require("./workspaces/implementations/rush"), exports);
 __exportStar(require("./workspaces/implementations/yarn"), exports);
 __exportStar(require("./workspaces/getChangedPackages"), exports);
+__exportStar(require("./workspaces/getPackagesByFiles"), exports);
 __exportStar(require("./workspaces/listOfWorkspacePackageNames"), exports);
 __exportStar(require("./workspaces/workspaces"), exports);

package/lib/workspaces/getChangedPackages.d.ts

@@ -1,3 +1,21 @@
+/**
+ * Finds all packages that had been changed between two refs in the repo under cwd
+ *
+ * executes a "git diff $fromRef...$toRef" to get changes given a merge-base
+ *
+ * further explanation with the three dots:
+ *
+ * > git diff [--options] <commit>...<commit> [--] [<path>...]
+ * >
+ * >    This form is to view the changes on the branch containing and up to
+ * >    the second <commit>, starting at a common ancestor of both
+ * >    <commit>. "git diff A...B" is equivalent to "git diff
+ * >    $(git-merge-base A B) B". You can omit any one of <commit>, which
+ * >    has the same effect as using HEAD instead.
+ *
+ * @returns string[] of package names that have changed
+ */
+export declare function getChangedPackagesBetweenRefs(cwd: string, fromRef: string, toRef?: string, ignoreGlobs?: string[]): string[];
 /**
  * Finds all packages that had been changed in the repo under cwd
  *

package/lib/workspaces/getChangedPackages.js

@@ -1,13 +1,37 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getChangedPackages = void 0;
+exports.getChangedPackages = exports.getChangedPackagesBetweenRefs = void 0;
 const git_1 = require("../git");
-const getWorkspaces_1 = require("./getWorkspaces");
-const multimatch_1 = __importDefault(require("multimatch"));
-const path_1 = __importDefault(require("path"));
+const getPackagesByFiles_1 = require("./getPackagesByFiles");
+/**
+ * Finds all packages that had been changed between two refs in the repo under cwd
+ *
+ * executes a "git diff $fromRef...$toRef" to get changes given a merge-base
+ *
+ * further explanation with the three dots:
+ *
+ * > git diff [--options] <commit>...<commit> [--] [<path>...]
+ * >
+ * >    This form is to view the changes on the branch containing and up to
+ * >    the second <commit>, starting at a common ancestor of both
+ * >    <commit>. "git diff A...B" is equivalent to "git diff
+ * >    $(git-merge-base A B) B". You can omit any one of <commit>, which
+ * >    has the same effect as using HEAD instead.
+ *
+ * @returns string[] of package names that have changed
+ */
+function getChangedPackagesBetweenRefs(cwd, fromRef, toRef = "", ignoreGlobs = []) {
+    let changes = [
+        ...new Set([
+            ...((0, git_1.getUntrackedChanges)(cwd) || []),
+            ...((0, git_1.getUnstagedChanges)(cwd) || []),
+            ...((0, git_1.getChangesBetweenRefs)(fromRef, toRef, [], "", cwd) || []),
+            ...((0, git_1.getStagedChanges)(cwd) || []),
+        ]),
+    ];
+    return (0, getPackagesByFiles_1.getPackagesByFiles)(cwd, changes, ignoreGlobs, true);
+}
+exports.getChangedPackagesBetweenRefs = getChangedPackagesBetweenRefs;
 /**
  * Finds all packages that had been changed in the repo under cwd
  *
@@ -26,32 +50,15 @@ const path_1 = __importDefault(require("path"));
  * @returns string[] of package names that have changed
  */
 function getChangedPackages(cwd, target, ignoreGlobs = []) {
-    const workspaceInfo = (0, getWorkspaces_1.getWorkspaces)(cwd);
-    target = target || (0, git_1.getDefaultRemoteBranch)(undefined, cwd);
+    const targetBranch = target || (0, git_1.getDefaultRemoteBranch)(undefined, cwd);
     let changes = [
         ...new Set([
             ...((0, git_1.getUntrackedChanges)(cwd) || []),
             ...((0, git_1.getUnstagedChanges)(cwd) || []),
-            ...((0, git_1.getBranchChanges)(target, cwd) || []),
+            ...((0, git_1.getBranchChanges)(targetBranch, cwd) || []),
             ...((0, git_1.getStagedChanges)(cwd) || []),
         ]),
     ];
-    const ignoreSet = new Set((0, multimatch_1.default)(changes, ignoreGlobs));
-    changes = changes.filter((change) => !ignoreSet.has(change));
-    const changeSet = new Set();
-    for (const change of changes) {
-        const candidates = workspaceInfo.filter((pkgPath) => change.indexOf(path_1.default.relative(cwd, pkgPath.path).replace(/\\/g, "/")) ===
-            0);
-        if (candidates && candidates.length > 0) {
-            const found = candidates.reduce((found, item) => {
-                return found.path.length > item.path.length ? found : item;
-            }, candidates[0]);
-            changeSet.add(found.name);
-        }
-        else {
-            return workspaceInfo.map((pkg) => pkg.name);
-        }
-    }
-    return [...changeSet];
+    return (0, getPackagesByFiles_1.getPackagesByFiles)(cwd, changes, ignoreGlobs, true);
 }
 exports.getChangedPackages = getChangedPackages;

package/lib/workspaces/getPackagesByFiles.d.ts

@@ -0,0 +1,10 @@
+/**
+ * Given a list of files, finds all packages names that contain those files
+ *
+ * @param workspaceRoot - The root of the workspace
+ * @param files - files to search for
+ * @param ignoreGlobs - glob patterns to ignore
+ * @param returnAllPackagesOnNoMatch - if true, will return all packages if no matches are found
+ * @returns package names that have changed
+ */
+export declare function getPackagesByFiles(workspaceRoot: string, files: string[], ignoreGlobs?: string[], returnAllPackagesOnNoMatch?: boolean): string[];

package/lib/workspaces/getPackagesByFiles.js

@@ -0,0 +1,38 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.getPackagesByFiles = void 0;
+const multimatch_1 = __importDefault(require("multimatch"));
+const path_1 = __importDefault(require("path"));
+const getWorkspaces_1 = require("./getWorkspaces");
+/**
+ * Given a list of files, finds all packages names that contain those files
+ *
+ * @param workspaceRoot - The root of the workspace
+ * @param files - files to search for
+ * @param ignoreGlobs - glob patterns to ignore
+ * @param returnAllPackagesOnNoMatch - if true, will return all packages if no matches are found
+ * @returns package names that have changed
+ */
+function getPackagesByFiles(workspaceRoot, files, ignoreGlobs = [], returnAllPackagesOnNoMatch = false) {
+    const workspaceInfo = (0, getWorkspaces_1.getWorkspaces)(workspaceRoot);
+    const ignoreSet = new Set((0, multimatch_1.default)(files, ignoreGlobs));
+    files = files.filter((change) => !ignoreSet.has(change));
+    const packages = new Set();
+    for (const file of files) {
+        const candidates = workspaceInfo.filter((pkgPath) => file.indexOf(path_1.default.relative(workspaceRoot, pkgPath.path).replace(/\\/g, "/")) === 0);
+        if (candidates && candidates.length > 0) {
+            const found = candidates.reduce((found, item) => {
+                return found.path.length > item.path.length ? found : item;
+            }, candidates[0]);
+            packages.add(found.name);
+        }
+        else if (returnAllPackagesOnNoMatch) {
+            return workspaceInfo.map((pkg) => pkg.name);
+        }
+    }
+    return [...packages];
+}
+exports.getPackagesByFiles = getPackagesByFiles;

package/lib/workspaces/implementations/index.d.ts

@@ -3,8 +3,12 @@ export interface ImplementationAndLockFile {
     implementation: WorkspaceImplementations | undefined;
     lockFile: string;
 }
-export declare function getWorkspaceImplementationAndLockFile(cwd: string): {
+export declare function getWorkspaceImplementationAndLockFile(cwd: string, cache?: {
+    [cwd: string]: ImplementationAndLockFile;
+}): {
     implementation: WorkspaceImplementations | undefined;
     lockFile: string;
 } | undefined;
-export declare function getWorkspaceImplementation(cwd: string): WorkspaceImplementations | undefined;
+export declare function getWorkspaceImplementation(cwd: string, cache?: {
+    [cwd: string]: ImplementationAndLockFile;
+}): WorkspaceImplementations | undefined;

package/lib/workspaces/implementations/index.js

@@ -6,12 +6,12 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.getWorkspaceImplementation = exports.getWorkspaceImplementationAndLockFile = void 0;
 const find_up_1 = __importDefault(require("find-up"));
 const path_1 = __importDefault(require("path"));
-const cache = {};
-function getWorkspaceImplementationAndLockFile(cwd) {
+const workspaceCache = {};
+function getWorkspaceImplementationAndLockFile(cwd, cache = workspaceCache) {
     if (cache[cwd]) {
         return cache[cwd];
     }
-    const lockFile = find_up_1.default.sync(["lerna.json", "yarn.lock", "pnpm-workspace.yaml", "rush.json", "package-lock.json"], {
+    const lockFile = find_up_1.default.sync(["lerna.json", "rush.json", "yarn.lock", "pnpm-workspace.yaml", "package-lock.json"], {
         cwd,
     });
     if (!lockFile) {
@@ -52,8 +52,8 @@ function getWorkspaceImplementationAndLockFile(cwd) {
     return cache[cwd];
 }
 exports.getWorkspaceImplementationAndLockFile = getWorkspaceImplementationAndLockFile;
-function getWorkspaceImplementation(cwd) {
+function getWorkspaceImplementation(cwd, cache = workspaceCache) {
     var _a;
-    return (_a = getWorkspaceImplementationAndLockFile(cwd)) === null || _a === void 0 ? void 0 : _a.implementation;
+    return (_a = getWorkspaceImplementationAndLockFile(cwd, cache)) === null || _a === void 0 ? void 0 : _a.implementation;
 }
 exports.getWorkspaceImplementation = getWorkspaceImplementation;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "workspace-tools",
-  "version": "0.18.2",
+  "version": "0.19.0",
   "license": "MIT",
   "repository": {
     "type": "git",