@backstage/cli 0.7.14 → 0.8.1

package/CHANGELOG.md

@@ -1,5 +1,68 @@
 # @backstage/cli
 
+## 0.8.1
+
+### Patch Changes
+
+- f1e96dc5b1: Update usage of msw in default plugin template
+- b0dc1fd241: bump `@spotify/eslint-config-base` from 9.0.2 to 12.0.0
+- c5bb1df55d: Bump `msw` to `v0.35.0` to resolve [CVE-2021-32796](https://github.com/advisories/GHSA-5fg8-2547-mr8q).
+- 10615525f3: Switch to use the json and observable types from `@backstage/types`
+- Updated dependencies
+  - @backstage/config@0.1.11
+  - @backstage/cli-common@0.1.5
+  - @backstage/errors@0.1.4
+  - @backstage/config-loader@0.7.1
+
+## 0.8.0
+
+### Minor Changes
+
+- b486adb8c6: The Jest configuration that's included with the Backstage CLI has received several changes.
+
+  As a part of migrating to more widespread usage of ESM modules, the default configuration now transforms all source files everywhere, including those within `node_modules`. Due to this change the existing `transformModules` option has been removed and will be ignored. There is also a list of known packages that do not require transforms in the CLI, which will evolve over time. If needed there will also be an option to add packages to this list in the future, but it is not included yet to avoid clutter.
+
+  To counteract the slowdown of the additional transforms that have been introduced, the default configuration has also been reworked to enable caching across different packages. Previously each package in a Backstage monorepo would have its own isolated Jest cache, but it is now shared between packages that have a similar enough Jest configuration.
+
+  Another change that will speed up test execution is that the transformer for `.esm.js` files has been switched. It used to be an ESM transformer based on Babel, but it is also done by sucrase now since it is significantly faster.
+
+  The changes above are not strictly breaking as all tests should still work. It may however cause excessive slowdowns in projects that have configured custom transforms in the `jest` field within `package.json` files. In this case it is either best to consider removing the custom transforms, or overriding the `transformIgnorePatterns` to instead use Jest's default `'/node_modules/'` pattern.
+
+  This change also removes the `@backstage/cli/config/jestEsmTransform.js` transform, which can be replaced by using the `@backstage/cli/config/sucraseEsmTransform.js` transform instead.
+
+### Patch Changes
+
+- 36e67d2f24: Internal updates to apply more strict checks to throw errors.
+- Updated dependencies
+  - @backstage/config-loader@0.7.0
+  - @backstage/errors@0.1.3
+
+## 0.7.16
+
+### Patch Changes
+
+- 53bdc66623: add a --from <location> option to the plugin install command
+- 84e24fcdaf: Bump sucrase to version 3.20.2
+- 6583c6ac40: Add semicolon in template to make prettier happy
+- c6f927d819: Bump mini-css-extract-plugin to v2
+- 16f044cb6b: Update default backend ESLint configuration to allow usage of `__dirname` in tests.
+- 1ef9e64901: Add an experimental `install <plugin>` command.
+
+  Given a `pluginId`, the command looks for NPM packages matching `@backstage/plugin-{pluginId}` or `backstage-plugin-{pluginId}` or `{pluginId}`. It looks for the `experimentalInstallationRecipe` in their `package.json` for the steps of installation. Detailed documentation and API Spec to follow (and to be decided as well).
+
+## 0.7.15
+
+### Patch Changes
+
+- ae4680b88d: The `create-plugin` command now passes the extension name via the `name` key
+  in `createRoutableExtension()` calls in newly created plugins.
+- df1242ffe4: Adding `--inspect-brk` as an option when debugging backend for development
+- c7f2a2307d: When creating a backend plugin with `--backend` flag, don't add `-backend` if it's already suffixed
+- 185fec5c0c: The default jest configuration used by the `test` command now supports yarn workspaces. By running `backstage-cli test` in the root of a monorepo, all packages will now automatically be included in the test suite and it will run just like it does within a package. Each package in the monorepo will still use its own local jest configuration, and only packages that have `backstage-cli test` in the `test` script within `package.json` will be included.
+- Updated dependencies
+  - @backstage/config-loader@0.6.10
+  - @backstage/cli-common@0.1.4
+
 ## 0.7.14
 
 ### Patch Changes

package/config/eslint.backend.js

@@ -14,6 +14,15 @@
  * limitations under the License.
  */
 
+const globalRestrictedSyntax = [
+  {
+    message:
+      'Default import from winston is not allowed, import `* as winston` instead.',
+    selector:
+      'ImportDeclaration[source.value="winston"] ImportDefaultSpecifier',
+  },
+];
+
 module.exports = {
   extends: [
     '@spotify/eslint-config-base',
@@ -69,17 +78,12 @@ module.exports = {
     // Avoid default import from winston as it breaks at runtime
     'no-restricted-syntax': [
       'error',
-      {
-        message:
-          'Default import from winston is not allowed, import `* as winston` instead.',
-        selector:
-          'ImportDeclaration[source.value="winston"] ImportDefaultSpecifier',
-      },
       {
         message:
           "`__dirname` doesn't refer to the same dir in production builds, try `resolvePackagePath()` from `@backstage/backend-common` instead.",
         selector: 'Identifier[name="__dirname"]',
       },
+      ...globalRestrictedSyntax,
     ],
   },
   overrides: [
@@ -103,6 +107,7 @@ module.exports = {
             bundledDependencies: true,
           },
         ],
+        'no-restricted-syntax': ['error', ...globalRestrictedSyntax],
       },
     },
   ],

package/config/jest.js

@@ -16,20 +16,41 @@
 
 const fs = require('fs-extra');
 const path = require('path');
+const crypto = require('crypto');
+const glob = require('util').promisify(require('glob'));
+const { version } = require('../package.json');
 
-async function getConfig() {
+const transformIgnorePattern = [
+  '@material-ui',
+  '@rjsf',
+  'ajv',
+  'core-js',
+  'jest-.*',
+  'jsdom',
+  'knex',
+  'react',
+  'react-dom',
+  'highlight\\.js',
+  'prismjs',
+  'react-use',
+  'typescript',
+].join('|');
+
+async function getProjectConfig(targetPath, displayName) {
+  const configJsPath = path.resolve(targetPath, 'jest.config.js');
+  const configTsPath = path.resolve(targetPath, 'jest.config.ts');
   // If the package has it's own jest config, we use that instead.
-  if (await fs.pathExists('jest.config.js')) {
-    return require(path.resolve('jest.config.js'));
-  } else if (await fs.pathExists('jest.config.ts')) {
-    return require(path.resolve('jest.config.ts'));
+  if (await fs.pathExists(configJsPath)) {
+    return require(configJsPath);
+  } else if (await fs.pathExists(configTsPath)) {
+    return require(configTsPath);
   }
 
   // We read all "jest" config fields in package.json files all the way to the filesystem root.
   // All configs are merged together to create the final config, with longer paths taking precedence.
   // The merging of the configs is shallow, meaning e.g. all transforms are replaced if new ones are defined.
   const pkgJsonConfigs = [];
-  let currentPath = process.cwd();
+  let currentPath = targetPath;
 
   // Some sanity check to avoid infinite loop
   for (let i = 0; i < 100; i++) {
@@ -55,10 +76,7 @@ async function getConfig() {
     currentPath = newPath;
   }
 
-  // We add an additional Jest config parameter only known by the Backstage CLI
-  // called `transformModules`. It's a list of modules that we want to apply
-  // our configured jest transformations for.
-  // This is useful when packages are published in untranspiled ESM or TS form.
+  // This is an old deprecated option that is no longer used.
   const transformModules = pkgJsonConfigs
     .flatMap(conf => {
       const modules = conf.transformModules || [];
@@ -67,19 +85,23 @@ async function getConfig() {
     })
     .map(name => `${name}/`)
     .join('|');
-  const transformModulePattern = transformModules && `(?!${transformModules})`;
+  if (transformModules.length > 0) {
+    console.warn(
+      'The Backstage CLI jest transformModules option is no longer used and will be ignored. All modules are now always transformed.',
+    );
+  }
 
   const options = {
-    rootDir: path.resolve('src'),
-    coverageDirectory: path.resolve('coverage'),
+    ...(displayName && { displayName }),
+    rootDir: path.resolve(targetPath, 'src'),
+    coverageDirectory: path.resolve(targetPath, 'coverage'),
     collectCoverageFrom: ['**/*.{js,jsx,ts,tsx}', '!**/*.d.ts'],
     moduleNameMapper: {
       '\\.(css|less|scss|sss|styl)$': require.resolve('jest-css-modules'),
     },
 
     transform: {
-      '\\.esm\\.js$': require.resolve('./jestEsmTransform.js'), // See jestEsmTransform.js
-      '\\.(js|jsx|ts|tsx)$': require.resolve('@sucrase/jest-plugin'),
+      '\\.(js|jsx|ts|tsx)$': require.resolve('./jestSucraseTransform.js'),
       '\\.(bmp|gif|jpg|jpeg|png|frag|xml|svg|eot|woff|woff2|ttf)$':
         require.resolve('./jestFileTransform.js'),
       '\\.(yaml)$': require.resolve('jest-transform-yaml'),
@@ -88,19 +110,79 @@ async function getConfig() {
     // A bit more opinionated
     testMatch: ['**/?(*.)test.{js,jsx,mjs,ts,tsx}'],
 
-    // Default behaviour is to not apply transforms for node_modules, but we still want
-    // to apply the esm-transformer to .esm.js files, since that's what we use in backstage packages.
-    transformIgnorePatterns: [
-      `/node_modules/${transformModulePattern}.*\\.(?:(?<!esm\\.)js|json)$`,
-    ],
+    transformIgnorePatterns: [`/node_modules/(?:${transformIgnorePattern})/`],
   };
 
   // Use src/setupTests.ts as the default location for configuring test env
-  if (fs.existsSync('src/setupTests.ts')) {
+  if (fs.existsSync(path.resolve(targetPath, 'src/setupTests.ts'))) {
     options.setupFilesAfterEnv = ['<rootDir>/setupTests.ts'];
   }
 
-  return Object.assign(options, ...pkgJsonConfigs);
+  const config = Object.assign(options, ...pkgJsonConfigs);
+
+  // The config name is a cache key that lets us share the jest cache across projects.
+  // If no explicit name was configured, generated one based on the configuration.
+  if (!config.name) {
+    const configHash = crypto
+      .createHash('md5')
+      .update(version)
+      .update(Buffer.alloc(1))
+      .update(JSON.stringify(config.transform))
+      .digest('hex');
+    config.name = `backstage_cli_${configHash}`;
+  }
+
+  return config;
+}
+
+// This loads the root jest config, which in turn will either refer to a single
+// configuration for the current package, or a collection of configurations for
+// the target workspace packages
+async function getRootConfig() {
+  const targetPath = process.cwd();
+  const targetPackagePath = path.resolve(targetPath, 'package.json');
+  const exists = await fs.pathExists(targetPackagePath);
+
+  if (!exists) {
+    return getProjectConfig(targetPath);
+  }
+
+  // Check whether the current package is a workspace root or not
+  const data = await fs.readJson(targetPackagePath);
+  const workspacePatterns = data.workspaces && data.workspaces.packages;
+  if (!workspacePatterns) {
+    return getProjectConfig(targetPath);
+  }
+
+  // If the target package is a workspace root, we find all packages in the
+  // workspace and load those in as separate jest projects instead.
+  const projectPaths = await Promise.all(
+    workspacePatterns.map(pattern => glob(path.join(targetPath, pattern))),
+  ).then(_ => _.flat());
+
+  const configs = await Promise.all(
+    projectPaths.flat().map(async projectPath => {
+      const packagePath = path.resolve(projectPath, 'package.json');
+      if (!(await fs.pathExists(packagePath))) {
+        return undefined;
+      }
+
+      // We check for the presence of "backstage-cli test" in the package test
+      // script to determine whether a given package should be tested
+      const packageData = await fs.readJson(packagePath);
+      const testScript = packageData.scripts && packageData.scripts.test;
+      if (testScript && testScript.includes('backstage-cli test')) {
+        return await getProjectConfig(projectPath, packageData.name);
+      }
+
+      return undefined;
+    }),
+  ).then(cs => cs.filter(Boolean));
+
+  return {
+    rootDir: targetPath,
+    projects: configs,
+  };
 }
 
-module.exports = getConfig();
+module.exports = getRootConfig();

package/config/jestEsmTransform.js

@@ -1,5 +1,5 @@
 /*
- * Copyright 2020 The Backstage Authors
+ * Copyright 2021 The Backstage Authors
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -14,23 +14,6 @@
  * limitations under the License.
  */
 
-const babel = require('@babel/core');
-
-// We build .esm.js files with plugin:build, so to be able to load these in tests they need to be transformed
-// TODO: jest is working on module support, it's possible that we can remove this in the future
-module.exports = {
-  process(src) {
-    const result = babel.transform(src, {
-      babelrc: false,
-      compact: false,
-      plugins: [
-        // This transforms the regular ESM syntax, import and export statements
-        require.resolve('@babel/plugin-transform-modules-commonjs'),
-        // This transforms dynamic `import()`, which is not supported yet in the Node.js VM API
-        require.resolve('babel-plugin-dynamic-import-node'),
-      ],
-    });
-
-    return result.code;
-  },
-};
+throw new Error(
+  'The ./jestEsmTransform.js transform has been removed, switch to using ./jestSucraseTransform.js instead',
+);

package/config/jestSucraseTransform.js

@@ -0,0 +1,66 @@
+/*
+ * Copyright 2021 The Backstage Authors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+const { createHash } = require('crypto');
+const { transform } = require('sucrase');
+const sucrasePkg = require('sucrase/package.json');
+const sucrasePluginPkg = require('@sucrase/jest-plugin/package.json');
+
+const ESM_REGEX = /\b(?:import|export)\b/;
+
+function process(source, filePath) {
+  let transforms;
+
+  if (filePath.endsWith('.esm.js')) {
+    transforms = ['imports'];
+  } else if (filePath.endsWith('.js')) {
+    // This is a very rough filter to avoid transforming things that we quickly
+    // can be sure are definitely not ESM modules.
+    if (ESM_REGEX.test(source)) {
+      transforms = ['imports', 'jsx']; // JSX within .js is currently allowed
+    }
+  } else if (filePath.endsWith('.jsx')) {
+    transforms = ['jsx', 'imports'];
+  } else if (filePath.endsWith('.ts')) {
+    transforms = ['typescript', 'imports'];
+  } else if (filePath.endsWith('.tsx')) {
+    transforms = ['typescript', 'jsx', 'imports'];
+  }
+
+  // Only apply the jest transform to the test files themselves
+  if (transforms && filePath.includes('.test.')) {
+    transforms.push('jest');
+  }
+
+  if (transforms) {
+    return transform(source, { transforms, filePath }).code;
+  }
+
+  return source;
+}
+
+// TODO: contribute something like this to @sucrase/jest-plugin
+function getCacheKey(sourceText) {
+  return createHash('md5')
+    .update(sourceText)
+    .update(Buffer.alloc(1))
+    .update(sucrasePkg.version)
+    .update(Buffer.alloc(1))
+    .update(sucrasePluginPkg.version)
+    .digest('hex');
+}
+
+module.exports = { process, getCacheKey };

package/dist/cjs/{build-71c737ac.cjs.js → build-036a2109.cjs.js}

@@ -1,15 +1,16 @@
 'use strict';
 
-var packager = require('./packager-a538a0d7.cjs.js');
+var packager = require('./packager-679f1af5.cjs.js');
 require('fs-extra');
 require('rollup');
 require('chalk');
 require('path');
-require('./index-b0d38feb.cjs.js');
+require('./index-1db5f8d6.cjs.js');
 require('commander');
 require('semver');
 require('@backstage/cli-common');
 require('@backstage/config/package.json');
+require('@backstage/errors');
 require('rollup-plugin-peer-deps-external');
 require('@rollup/plugin-commonjs');
 require('@rollup/plugin-node-resolve');
@@ -29,4 +30,4 @@ var build = async () => {
 };
 
 exports.default = build;
-//# sourceMappingURL=build-71c737ac.cjs.js.map
+//# sourceMappingURL=build-036a2109.cjs.js.map

package/dist/cjs/{build-b73c8aec.cjs.js → build-3192edeb.cjs.js}

@@ -12,20 +12,21 @@ require('@backstage/cli-common');
 require('terser-webpack-plugin');
 var parallel = require('./parallel-a4714c72.cjs.js');
 require('mini-css-extract-plugin');
-var index = require('./index-b0d38feb.cjs.js');
-require('./run-e50c9fbb.cjs.js');
+var index = require('./index-1db5f8d6.cjs.js');
+require('./run-a312f9a6.cjs.js');
 require('lodash/pickBy');
 var yn = require('yn');
 var FileSizeReporter = require('react-dev-utils/FileSizeReporter');
 var formatWebpackMessages = require('react-dev-utils/formatWebpackMessages');
-var paths = require('./paths-38b204ca.cjs.js');
+var paths = require('./paths-9f678df9.cjs.js');
 var chalk = require('chalk');
 require('webpack-dev-server');
 require('react-dev-utils/openBrowser');
-var config = require('./config-f0ca8800.cjs.js');
+var config = require('./config-dab34362.cjs.js');
 require('commander');
 require('semver');
 require('@backstage/config/package.json');
+require('@backstage/errors');
 require('child_process');
 require('util');
 require('./svgrTemplate-2d0d15cf.cjs.js');
@@ -135,4 +136,4 @@ var build = async (cmd) => {
 };
 
 exports.default = build;
-//# sourceMappingURL=build-b73c8aec.cjs.js.map
+//# sourceMappingURL=build-3192edeb.cjs.js.map

package/dist/cjs/{build-61ca5bd1.cjs.js → build-32289a53.cjs.js}

@@ -1,15 +1,16 @@
 'use strict';
 
-var packager = require('./packager-a538a0d7.cjs.js');
+var packager = require('./packager-679f1af5.cjs.js');
 require('fs-extra');
 require('rollup');
 require('chalk');
 require('path');
-require('./index-b0d38feb.cjs.js');
+require('./index-1db5f8d6.cjs.js');
 require('commander');
 require('semver');
 require('@backstage/cli-common');
 require('@backstage/config/package.json');
+require('@backstage/errors');
 require('rollup-plugin-peer-deps-external');
 require('@rollup/plugin-commonjs');
 require('@rollup/plugin-node-resolve');
@@ -40,4 +41,4 @@ var build = async (cmd) => {
 };
 
 exports.default = build;
-//# sourceMappingURL=build-61ca5bd1.cjs.js.map
+//# sourceMappingURL=build-32289a53.cjs.js.map

package/dist/cjs/{build-b61079aa.cjs.js → build-7357e33b.cjs.js}

@@ -1,15 +1,16 @@
 'use strict';
 
-var packager = require('./packager-a538a0d7.cjs.js');
+var packager = require('./packager-679f1af5.cjs.js');
 require('fs-extra');
 require('rollup');
 require('chalk');
 require('path');
-require('./index-b0d38feb.cjs.js');
+require('./index-1db5f8d6.cjs.js');
 require('commander');
 require('semver');
 require('@backstage/cli-common');
 require('@backstage/config/package.json');
+require('@backstage/errors');
 require('rollup-plugin-peer-deps-external');
 require('@rollup/plugin-commonjs');
 require('@rollup/plugin-node-resolve');
@@ -29,4 +30,4 @@ var build = async () => {
 };
 
 exports.default = build;
-//# sourceMappingURL=build-b61079aa.cjs.js.map
+//# sourceMappingURL=build-7357e33b.cjs.js.map

package/dist/cjs/{buildImage-66df0236.cjs.js → buildImage-62344040.cjs.js}

@@ -3,9 +3,9 @@
 var chalk = require('chalk');
 var fs = require('fs-extra');
 var path = require('path');
-var index$1 = require('./index-639c9015.cjs.js');
-var index = require('./index-b0d38feb.cjs.js');
-var run = require('./run-e50c9fbb.cjs.js');
+var index$1 = require('./index-69ec4ee3.cjs.js');
+var index = require('./index-1db5f8d6.cjs.js');
+var run = require('./run-a312f9a6.cjs.js');
 var parallel = require('./parallel-a4714c72.cjs.js');
 require('os');
 require('tar');
@@ -13,6 +13,7 @@ require('commander');
 require('semver');
 require('@backstage/cli-common');
 require('@backstage/config/package.json');
+require('@backstage/errors');
 require('child_process');
 require('util');
 
@@ -73,4 +74,4 @@ async function findAppConfigs() {
 }
 
 exports.default = buildImage;
-//# sourceMappingURL=buildImage-66df0236.cjs.js.map
+//# sourceMappingURL=buildImage-62344040.cjs.js.map

package/dist/cjs/{buildWorkspace-a669db03.cjs.js → buildWorkspace-794159d5.cjs.js}

@@ -1,17 +1,18 @@
 'use strict';
 
 var fs = require('fs-extra');
-var index = require('./index-639c9015.cjs.js');
+var index = require('./index-69ec4ee3.cjs.js');
 require('path');
 require('os');
 require('tar');
-require('./index-b0d38feb.cjs.js');
+require('./index-1db5f8d6.cjs.js');
 require('commander');
 require('chalk');
 require('semver');
 require('@backstage/cli-common');
 require('@backstage/config/package.json');
-require('./run-e50c9fbb.cjs.js');
+require('@backstage/errors');
+require('./run-a312f9a6.cjs.js');
 require('child_process');
 require('util');
 
@@ -29,4 +30,4 @@ var buildWorkspace = async (dir, _cmd, packages) => {
 };
 
 exports.default = buildWorkspace;
-//# sourceMappingURL=buildWorkspace-a669db03.cjs.js.map
+//# sourceMappingURL=buildWorkspace-794159d5.cjs.js.map

package/dist/cjs/{bump-2c638ffb.cjs.js → bump-ab142f88.cjs.js}

@@ -3,11 +3,13 @@
 var fs = require('fs-extra');
 var chalk = require('chalk');
 var semver = require('semver');
+var errors = require('@backstage/errors');
 var path = require('path');
-var run = require('./run-e50c9fbb.cjs.js');
-var index = require('./index-b0d38feb.cjs.js');
+var run = require('./run-a312f9a6.cjs.js');
+var index = require('./index-1db5f8d6.cjs.js');
 var Lockfile = require('./Lockfile-80f0eec4.cjs.js');
-var lint = require('./lint-a5758aab.cjs.js');
+var packages = require('./packages-4fe3897c.cjs.js');
+var lint = require('./lint-95a1066a.cjs.js');
 require('child_process');
 require('util');
 require('commander');
@@ -22,48 +24,6 @@ var fs__default = /*#__PURE__*/_interopDefaultLegacy(fs);
 var chalk__default = /*#__PURE__*/_interopDefaultLegacy(chalk);
 var semver__default = /*#__PURE__*/_interopDefaultLegacy(semver);
 
-const PREFIX = "@backstage";
-const DEP_TYPES$1 = [
-  "dependencies",
-  "devDependencies",
-  "peerDependencies",
-  "optionalDependencies"
-];
-async function fetchPackageInfo(name) {
-  const output = await run.runPlain("yarn", "info", "--json", name);
-  if (!output) {
-    throw new index.NotFoundError(`No package information found for package ${name}`);
-  }
-  const info = JSON.parse(output);
-  if (info.type !== "inspect") {
-    throw new Error(`Received unknown yarn info for ${name}, ${output}`);
-  }
-  return info.data;
-}
-async function mapDependencies(targetDir) {
-  var _a;
-  const {Project} = require("@lerna/project");
-  const project = new Project(targetDir);
-  const packages = await project.getPackages();
-  const dependencyMap = new Map();
-  for (const pkg of packages) {
-    const deps = DEP_TYPES$1.flatMap((t) => {
-      var _a2;
-      return Object.entries((_a2 = pkg.get(t)) != null ? _a2 : {});
-    });
-    for (const [name, range] of deps) {
-      if (name.startsWith(PREFIX)) {
-        dependencyMap.set(name, ((_a = dependencyMap.get(name)) != null ? _a : []).concat({
-          range,
-          name: pkg.name,
-          location: pkg.location
-        }));
-      }
-    }
-  }
-  return dependencyMap;
-}
-
 const DEP_TYPES = [
   "dependencies",
   "devDependencies",
@@ -75,7 +35,7 @@ var bump = async () => {
   const lockfilePath = index.paths.resolveTargetRoot("yarn.lock");
   const lockfile = await Lockfile.Lockfile.load(lockfilePath);
   const findTargetVersion = createVersionFinder();
-  const dependencyMap = await mapDependencies(index.paths.targetDir);
+  const dependencyMap = await packages.mapDependencies(index.paths.targetDir);
   const versionBumps = new Map();
   const unlocked = Array();
   await workerThreads(16, dependencyMap.entries(), async ([name, pkgs]) => {
@@ -84,7 +44,7 @@ var bump = async () => {
     try {
       target = await findTargetVersion(name);
     } catch (error) {
-      if (error.name === "NotFoundError") {
+      if (errors.isError(error) && error.name === "NotFoundError") {
         console.log(`Package info not found, ignoring package ${name}`);
         return;
       }
@@ -114,7 +74,7 @@ var bump = async () => {
     try {
       target = await findTargetVersion(name);
     } catch (error) {
-      if (error.name === "NotFoundError") {
+      if (errors.isError(error) && error.name === "NotFoundError") {
         console.log(`Package info not found, ignoring package ${name}`);
         return;
       }
@@ -220,7 +180,7 @@ function createVersionFinder() {
       return existing;
     }
     console.log(`Checking for updates of ${name}`);
-    const info = await fetchPackageInfo(name);
+    const info = await packages.fetchPackageInfo(name);
     const latest = info["dist-tags"].latest;
     if (!latest) {
       throw new Error(`No latest version found for ${name}`);
@@ -243,4 +203,4 @@ async function workerThreads(count, items, fn) {
 }
 
 exports.default = bump;
-//# sourceMappingURL=bump-2c638ffb.cjs.js.map
+//# sourceMappingURL=bump-ab142f88.cjs.js.map