fix(import): blacklist some folders when importing

fix #74

Author: Murat

src/__tests__/unit/utils/upgrade/runUpgradeTasks.spec.ts

@@ -119,6 +119,22 @@ imports:
     expect(result.didRun).toBeTruthy();
     expect(mockFs.lstatSync).not.toHaveBeenCalled();
   });
+  it('should skip blacklisted upgrade.yml imports', async () => {
+    mockFs.writeFileSync('/oldProject/android/some.file', 'random');
+    mockFs.writeFileSync('/oldProject/node_modules/some.file', 'random');
+    mockFs.writeFileSync(
+      path.join(getProjectPath(), '.upgrade/upgrade.yml'),
+      `
+imports:
+  - android
+  - node_modules/some.file`
+    );
+    mockFs.lstatSync.mockClear();
+    const result = await runUpgradeTasks('/oldProject');
+
+    expect(result.didRun).toBeTruthy();
+    expect(mockFs.lstatSync).not.toHaveBeenCalled();
+  });
   it('should skip when no old project path specified', async () => {
     mockFs.writeFileSync(
       path.join(getProjectPath(), '.upgrade/upgrade.yml'),

src/utils/upgrade/other/importPackageJson.ts

@@ -180,7 +180,9 @@ export async function getInstallCommand(projectPath: string) {
   const isPnpmLockPresent = fs.existsSync(
     path.join(projectPath, 'pnpm-lock.yaml')
   );
-  const isBunLockPresent = fs.existsSync(path.join(projectPath, 'bun.lockb'));
+  const isBunLockPresent =
+    fs.existsSync(path.join(projectPath, 'bun.lockb')) ||
+    fs.existsSync(path.join(projectPath, 'bun.lock'));
 
   const options: SelectOption[] = [];
   if (isNpmLockPresent) options.push({ label: 'npm', value: 'npm install' });

src/utils/upgrade/runUpgradeTasks.ts

@@ -70,11 +70,34 @@ export async function runUpgradeTasks(
       startSpinner(`discovering files from ${color.yellow('old project')}`);
 
       const filesToCopy: string[] = [];
+      const blackListedPaths = [
+        'android',
+        'ios',
+        '.upgrade/',
+        'node_modules/',
+        'package.json',
+        'integrate-lock.json',
+      ];
       for (let i = 0; i < config.imports.length; i++) {
         updateSpinner(
           `discovering files from ${color.yellow('old project')} (${i + 1}/${config.imports.length})`
         );
         const relativePath = getText(config.imports[i]);
+        if (
+          blackListedPaths.some(p => {
+            if (p.endsWith('/'))
+              return (
+                relativePath.startsWith(p) ||
+                relativePath === p.substring(0, p.length - 1)
+              );
+            return relativePath === p;
+          })
+        ) {
+          logWarning(
+            `skipped import of ${color.yellow(relativePath)}, this path is handled internally so you can remove it from imports list.`
+          );
+          continue;
+        }
         const importPath = path.join(oldProjectPath, relativePath);
         if (!fs.existsSync(importPath)) {
           logWarning(