feat: filter migration list to package dependencies (#48)

Author: 43081j

src/analyze/replacements.ts

@@ -1,6 +1,7 @@
 import * as replacements from 'module-replacements';
-import {PackageJsonLike, ReportPluginResult} from '../types.js';
+import {ReportPluginResult} from '../types.js';
 import type {FileSystem} from '../file-system.js';
+import {getPackageJson} from '../file-system-utils.js';
 
 /**
  * Generates a standard URL to the docs of a given rule
@@ -27,25 +28,9 @@ export async function runReplacements(
     messages: []
   };
 
-  let packageJsonText: string;
+  const packageJson = await getPackageJson(fileSystem);
 
-  try {
-    packageJsonText = await fileSystem.readFile('/package.json');
-  } catch {
-    // No package.json found
-    return result;
-  }
-
-  let packageJson: PackageJsonLike;
-
-  try {
-    packageJson = JSON.parse(packageJsonText);
-  } catch {
-    // Not parseable
-    return result;
-  }
-
-  if (!packageJson.dependencies) {
+  if (!packageJson || !packageJson.dependencies) {
     // No dependencies
     return result;
   }

src/commands/migrate.ts

@@ -6,19 +6,37 @@ import {glob} from 'tinyglobby';
 import {readFile, writeFile} from 'node:fs/promises';
 import {fixableReplacements} from './fixable-replacements.js';
 import type {Replacement} from '../types.js';
-
-const fixableReplacementsTargets = new Set(
-  fixableReplacements.map((rep) => rep.from)
-);
+import {LocalFileSystem} from '../local-file-system.js';
+import {getPackageJson} from '../file-system-utils.js';
 
 export async function run(ctx: CommandContext<typeof meta.args>) {
   const [_commandName, ...targetModules] = ctx.positionals;
   const dryRun = ctx.values['dry-run'] === true;
   const interactive = ctx.values.interactive === true;
   const include = ctx.values.include;
+  const fileSystem = new LocalFileSystem(process.cwd());
+  const packageJson = await getPackageJson(fileSystem);
 
   prompts.intro(`Migrating packages...`);
 
+  if (!packageJson) {
+    prompts.cancel(
+      'Error: No package.json found in the current directory. Please run this command in a project directory.'
+    );
+    return;
+  }
+
+  const dependencies = {
+    ...packageJson.dependencies,
+    ...packageJson.devDependencies
+  };
+
+  const fixableReplacementsTargets = new Set(
+    fixableReplacements
+      .filter((rep) => Object.hasOwn(dependencies, rep.from))
+      .map((rep) => rep.from)
+  );
+
   if (interactive) {
     const additionalTargets = await prompts.autocompleteMultiselect({
       message: 'Select packages to migrate',

src/file-system-utils.ts

@@ -0,0 +1,22 @@
+import type {FileSystem} from './file-system.js';
+import type {PackageJsonLike} from './types.js';
+
+export async function getPackageJson(
+  fileSystem: FileSystem
+): Promise<PackageJsonLike | null> {
+  let packageJsonText: string;
+
+  try {
+    packageJsonText = await fileSystem.readFile('/package.json');
+  } catch {
+    // No package.json found
+    return null;
+  }
+
+  try {
+    return JSON.parse(packageJsonText);
+  } catch {
+    // Not parseable
+    return null;
+  }
+}

test/fixtures/basic-chalk/src/main.ts renamed to test/fixtures/basic-chalk/lib/main.js

@@ -1,5 +1,10 @@
 {
   "name": "foo",
+  "type": "module",
   "private": true,
-  "version": "0.0.1"
+  "version": "0.0.1",
+  "main": "lib/main.js",
+  "dependencies": {
+    "chalk": "^4.0.0"
+  }
 }