migrate-barrel-imports 1.3.0 → 1.5.0

package/README.md

@@ -30,7 +30,7 @@ npx migrate-barrel-imports <source-path> [target-path] [options]
 
 ### Arguments
 
-- `source-path`: Path to the package containing barrel files (required)
+- `source-path`: Directory pattern for source packages (e.g. "libs/*", "packages/{ui,core}") (required)
 - `target-path`: Path to the directory where imports should be migrated (default: current directory)
 
 ### Options
@@ -44,11 +44,17 @@ Options can be specified either before or after the arguments:
 ## Example
 
 ```bash
-# Options after arguments
+# Migrate a single package
 migrate-barrel-imports ./packages/my-lib --ignore-source-files "**/__tests__/**,**/__mocks__/**" --ignore-target-files "**/*.test.ts"
 
-# Options before arguments
-migrate-barrel-imports --no-extension ./packages/my-lib .
+# Migrate multiple packages using glob pattern
+migrate-barrel-imports "libs/*" --no-extension
+
+# Migrate specific packages using glob pattern
+migrate-barrel-imports "packages/{ui,core,utils}" --ignore-target-files "**/*.test.ts"
+
+# Migrate all packages in a monorepo
+migrate-barrel-imports "**/src" --no-extension
 
 # Mix of options before and after arguments
 migrate-barrel-imports --no-extension ./packages/my-lib --ignore-target-files "**/*.test.ts"

package/dist/index.js

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
-import{Command as e}from"commander";import{readFileSync as o,writeFileSync as t}from"node:fs";import r,{join as s}from"node:path";import i from"@babel/generator";import{parse as n}from"@babel/parser";import l from"@babel/traverse";import{isVariableDeclaration as a,isIdentifier as c,isExportSpecifier as p,isFunctionDeclaration as d,isImportSpecifier as u,importDeclaration as g,importSpecifier as f,stringLiteral as m}from"@babel/types";import h from"fast-glob";import x from"micromatch";let F=i.default||i,$=l.default||l;async function S(e){return JSON.parse(o(s(e,"package.json"),"utf-8"))}async function y({packagePath:e,ignoreSourceFiles:t=[],stats:r}){let i=[];console.log(`Scanning for TypeScript and JavaScript files in: ${e}`);let l=await h("**/*.{ts,tsx,js,jsx}",{cwd:e,ignore:["**/node_modules/**","**/dist/**","**/build/**"]});for(let u of(console.log(`Found ${l.length} files`),l)){let l=t.some(e=>x.isMatch(u,e));l&&(console.log(`File matches ignore pattern but will be preserved: ${u}`),r&&r.sourceFilesSkipped++);let g=s(e,u);console.log(`
-Processing file: ${u}`);let f=o(g,"utf-8");try{let e=n(f,{sourceType:"module",plugins:["typescript","jsx","decorators-legacy","classProperties","classPrivateProperties","classPrivateMethods","exportDefaultFrom","exportNamespaceFrom","functionBind","functionSent","dynamicImport","nullishCoalescingOperator","optionalChaining","objectRestSpread","asyncGenerators","doExpressions","importMeta","logicalAssignment","moduleBlocks","moduleStringNames","numericSeparator","partialApplication","privateIn","throwExpressions","topLevelAwait"]});$(e,{ExportNamedDeclaration(e){if(console.log(`Found named export in ${u}`),e.node.source){console.log(`Skipping re-export from ${e.node.source.value}`);return}if(e.node.declaration){if(a(e.node.declaration)){let o=e.node.declaration.declarations.map(e=>c(e.id)?e.id.name:null).filter(e=>null!==e);o.length>0&&(console.log(`Named exports found: ${o.join(", ")}`),i.push({source:u,exports:o,isIgnored:l}))}return}let o=e.node.specifiers.map(e=>{if(p(e)){let o=e.exported;return c(o)?o.name:o.value}return null}).filter(e=>null!==e);o.length>0&&(console.log(`Named exports found: ${o.join(", ")}`),i.push({source:u,exports:o,isIgnored:l}))},ExportDefaultDeclaration(e){console.log(`Found default export in ${u}`);let o=e.node.declaration;c(o)?(console.log(`Default export name: ${o.name}`),i.push({source:u,exports:[o.name],isIgnored:l})):d(o)&&o.id?(console.log(`Default export name: ${o.id.name}`),i.push({source:u,exports:[o.id.name],isIgnored:l})):console.log("Default export is not an identifier or named function")}})}catch(e){console.error(`Error parsing ${u}:`,e)}}return console.log(`
-Total exports found: ${i.length}`),i}async function w({packageName:e,monorepoRoot:t}){try{let r=new Set,s=await h(["**/*.{ts,tsx,js,jsx}"],{cwd:t,absolute:!0,ignore:["**/node_modules/**","**/dist/**","**/build/**"],followSymbolicLinks:!1});for(let t of(console.log(`Found ${s.length} files to scan`),s))try{let s=o(t,"utf-8"),i=n(s,{sourceType:"module",plugins:["typescript","jsx"]});$(i,{ImportDeclaration(o){let s=o.node.source.value;(s===e||s.startsWith(`${e}/`))&&r.add(t)}})}catch(e){console.error(`Error processing file ${t}:`,e)}let i=Array.from(r);if(i.length>0)for(let o of(console.log(`Found total of ${i.length} files with imports from ${e}`),console.log("Files found:"),i))console.log(`  ${o}`);else console.log(`No files found importing from ${e}`);return i}catch(e){return console.error("Error finding imports:",e),[]}}async function E({filePath:e,packageName:r,exports:s,includeExtension:i=!0}){console.log(`
-Processing file: ${e}`);let l=o(e,"utf-8");try{let o=n(l,{sourceType:"module",plugins:["typescript","jsx","decorators-legacy","classProperties","classPrivateProperties","classPrivateMethods","exportDefaultFrom","exportNamespaceFrom","functionBind","functionSent","dynamicImport","nullishCoalescingOperator","optionalChaining","objectRestSpread","asyncGenerators","doExpressions","importMeta","logicalAssignment","moduleBlocks","moduleStringNames","numericSeparator","partialApplication","privateIn","throwExpressions","topLevelAwait"]}),a=!1,p=0;if($(o,{ImportDeclaration(e){if(e.node.source.value===r){p++,console.log(`Found import from ${r}`);let o=e.node.specifiers,t=[];for(let e of o)if(u(e)){let o=e.imported,n=c(o)?o.name:o.value,l=s.find(e=>e.exports.includes(n));if(l){if(console.log(`  Found export ${n} in ${l.source}`),l.isIgnored){console.log(`  Keeping original import for ignored file: ${l.source}`);continue}let o=i?`${r}/${l.source}`:`${r}/${l.source.replace(/\.(js|jsx|ts|tsx|mjs|cjs)$/,"")}`;t.push(g([f(e.local,e.imported)],m(o))),a=!0}else console.log(`  Warning: Could not find export ${n}`)}if(t.length>0){let i=o.filter(e=>{if(!u(e))return!0;let o=c(e.imported)?e.imported.name:e.imported.value,t=s.find(e=>e.exports.includes(o));return!t||t.isIgnored});i.length>0&&t.unshift(g(i,m(r))),e.replaceWithMultiple(t),console.log("  Replaced import with direct imports from source files")}}}}),a){console.log(`Writing changes to ${e}`);let r=F(o,{retainLines:!1,retainFunctionParens:!0},l);t(e,r.code)}else p>0?console.log(`No changes needed for ${p} imports`):console.log("No imports found to update")}catch(o){console.error(`Error processing ${e}:`,o)}}async function P(e){let{sourcePath:t,targetPath:s,ignoreSourceFiles:i,ignoreTargetFiles:n}=e,l={totalFiles:0,filesProcessed:0,filesSkipped:0,importsUpdated:0,filesWithNoUpdates:0,errors:0,totalExports:0,sourceFilesScanned:0,sourceFilesWithExports:0,sourceFilesSkipped:0,targetFilesFound:[]},a=await S(t),c=await y({packagePath:t,ignoreSourceFiles:i,stats:l});l.totalExports=c.reduce((e,o)=>e+o.exports.length,0),l.sourceFilesWithExports=new Set(c.map(e=>e.source)).size,l.sourceFilesScanned=(await h("**/*.{ts,tsx}",{cwd:t,ignore:["**/node_modules/**","**/dist/**","**/build/**"]})).length;let p=await w({packageName:a.name,monorepoRoot:s});for(let t of(l.totalFiles=p.length,l.targetFilesFound=p,p)){let i=r.relative(s,t);if(n.some(e=>x.isMatch(i,e))){console.log(`Skipping ignored file: ${t} (matches pattern in ${n.join(", ")})`),l.filesSkipped++;continue}try{let r=o(t,"utf-8");await E({filePath:t,packageName:a.name,exports:c,includeExtension:e.includeExtension});let s=o(t,"utf-8");r!==s?l.importsUpdated++:l.filesWithNoUpdates++,l.filesProcessed++}catch(e){l.errors++,console.error(`Error processing ${t}:`,e)}}console.log("\nMigration Summary"),console.log(`Source files found: ${l.sourceFilesScanned}`),console.log(`Source files with exports: ${l.sourceFilesWithExports}`),console.log(`Source files skipped: ${l.sourceFilesSkipped}`),console.log(`Exports found: ${l.totalExports}`),console.log(`Target files found: ${l.totalFiles}`),console.log(`Target files processed: ${l.filesProcessed}`),console.log(`Target files with imports updated: ${l.importsUpdated}`),console.log(`Target files with no changes needed: ${l.filesWithNoUpdates}`),console.log(`Target files skipped: ${l.filesSkipped}`),l.errors>0&&console.log(`
-Warning: ${l.errors} errors encountered during processing`)}let b={targetPath:".",ignoreSourceFiles:[],ignoreTargetFiles:[],includeExtension:!1};async function j(){let o=new e;o.name("migrate-barrel-imports").description("CLI tool to migrate barrel files imports to direct imports").argument("<source-path>","Path to the package containing barrel files").argument("[target-path]","Path to the directory where imports should be migrated (default: current directory)").option("--ignore-source-files <patterns>","Comma-separated list of file patterns to ignore in source directory").option("--ignore-target-files <patterns>","Comma-separated list of file patterns to ignore in target directory").option("--no-extension","Exclude js|jsx|ts|tsx|mjs|cjs file extensions from import statements").allowUnknownOption(!1).parse(process.argv);let t=o.args;t[0]||(console.error("Error: source-path is required"),process.exit(1));let r=t[0],s=t[1]||b.targetPath,i=o.opts();await P({sourcePath:r,targetPath:s,ignoreSourceFiles:i.ignoreSourceFiles?i.ignoreSourceFiles.split(","):b.ignoreSourceFiles,ignoreTargetFiles:i.ignoreTargetFiles?i.ignoreTargetFiles.split(","):b.ignoreTargetFiles,includeExtension:!1!==i.extension||b.includeExtension})}(async()=>{await j()})();
+import{Command as e}from"commander";import o,{readFileSync as s,writeFileSync as r}from"node:fs";import t from"node:path";import n from"@babel/generator";import{parse as i}from"@babel/parser";import l from"@babel/traverse";import{isVariableDeclaration as a,isIdentifier as c,isFunctionDeclaration as p,isTSEnumDeclaration as d,isTSInterfaceDeclaration as g,isTSTypeAliasDeclaration as u,isClassDeclaration as f,isExportSpecifier as m,isImportSpecifier as h,importDeclaration as x,stringLiteral as $,importSpecifier as S}from"@babel/types";import F from"fast-glob";import k from"micromatch";let w=n.default||n,y=l.default||l;async function b(e){return JSON.parse(s(t.join(e,"package.json"),"utf-8"))}async function E({packagePath:e,ignoreSourceFiles:o=[],stats:r}){let n=[];console.log(`Scanning for TypeScript and JavaScript files in: ${e}`);let l=await F("**/*.{ts,tsx,js,jsx}",{cwd:e,ignore:["**/node_modules/**","**/dist/**","**/build/**"]});for(let h of(console.log(`Found ${l.length} files`),l)){let l=o.some(e=>k.isMatch(h,e));l&&(console.log(`File matches ignore pattern but will be preserved: ${h}`),r&&r.sourceFilesSkipped++);let x=t.join(e,h);console.log(`
+Processing file: ${h}`);let $=s(x,"utf-8");try{let e=i($,{sourceType:"module",plugins:["typescript","jsx","decorators-legacy","classProperties","classPrivateProperties","classPrivateMethods","exportDefaultFrom","exportNamespaceFrom","functionBind","functionSent","dynamicImport","nullishCoalescingOperator","optionalChaining","objectRestSpread","asyncGenerators","doExpressions","importMeta","logicalAssignment","moduleBlocks","moduleStringNames","numericSeparator","partialApplication","privateIn","throwExpressions","topLevelAwait"]});y(e,{ExportNamedDeclaration(e){if(console.log(`Found named export in ${h}`),e.node.source){let o=e.node.source.value;if(o.includes("node_modules")||!o.startsWith(".")){console.log(`Skipping re-export from external package: ${o}`);return}}if(e.node.declaration){if(a(e.node.declaration)){let o=e.node.declaration.declarations.map(e=>c(e.id)?e.id.name:null).filter(e=>null!==e);o.length>0&&(console.log(`Named exports found: ${o.join(", ")}`),n.push({source:h,exports:o,isIgnored:l}))}else if(p(e.node.declaration)){let o=e.node.declaration.id?.name;o&&(console.log(`Found function export: ${o}`),n.push({source:h,exports:[o],isIgnored:l}))}else if(d(e.node.declaration)){let o=e.node.declaration.id.name;console.log(`Found enum export: ${o}`),n.push({source:h,exports:[o],isIgnored:l})}else if(g(e.node.declaration)){let o=e.node.declaration.id.name;console.log(`Found interface export: ${o}`),n.push({source:h,exports:[o],isIgnored:l})}else if(u(e.node.declaration)){let o=e.node.declaration.id.name;console.log(`Found type alias export: ${o}`),n.push({source:h,exports:[o],isIgnored:l})}else if(f(e.node.declaration)&&e.node.declaration.id){let o=e.node.declaration.id.name;console.log(`Found class export: ${o}`),n.push({source:h,exports:[o],isIgnored:l})}return}let o=e.node.specifiers.map(e=>{if(m(e)){let o=e.exported;return c(o)?o.name:o.value}return null}).filter(e=>null!==e);o.length>0&&(console.log(`Named exports found: ${o.join(", ")}`),n.push({source:h,exports:o,isIgnored:l}))},ExportDefaultDeclaration(e){console.log(`Found default export in ${h}`);let o=e.node.declaration;c(o)?(console.log(`Default export name: ${o.name}`),n.push({source:h,exports:[o.name],isIgnored:l})):p(o)&&o.id?(console.log(`Default export name: ${o.id.name}`),n.push({source:h,exports:[o.id.name],isIgnored:l})):console.log("Default export is not an identifier or named function")}})}catch(e){console.error(`Error parsing ${h}:`,e)}}return console.log(`
+Total exports found: ${n.length}`),n}async function P({packageName:e,monorepoRoot:o}){try{let r=new Set,t=await F(["**/*.{ts,tsx,js,jsx}"],{cwd:o,absolute:!0,ignore:["**/node_modules/**","**/dist/**","**/build/**"],followSymbolicLinks:!1});for(let o of(console.log(`Found ${t.length} files to scan`),t))try{let t=s(o,"utf-8"),n=i(t,{sourceType:"module",plugins:["typescript","jsx"]});y(n,{ImportDeclaration(s){let t=s.node.source.value;(t===e||t.startsWith(`${e}/`))&&r.add(o)}})}catch(e){console.error(`Error processing file ${o}:`,e)}let n=Array.from(r);if(n.length>0)for(let o of(console.log(`Found total of ${n.length} files with imports from ${e}`),console.log("Files found:"),n))console.log(`  ${o}`);else console.log(`No files found importing from ${e}`);return n}catch(e){return console.error("Error finding imports:",e),[]}}async function j({filePath:e,packageName:o,exports:t,includeExtension:n=!0,warnings:l,stats:a}){console.log(`
+Processing file: ${e}`);let p=s(e,"utf-8");try{let s=i(p,{sourceType:"module",plugins:["typescript","jsx","decorators-legacy","classProperties","classPrivateProperties","classPrivateMethods","exportDefaultFrom","exportNamespaceFrom","functionBind","functionSent","dynamicImport","nullishCoalescingOperator","optionalChaining","objectRestSpread","asyncGenerators","doExpressions","importMeta","logicalAssignment","moduleBlocks","moduleStringNames","numericSeparator","partialApplication","privateIn","throwExpressions","topLevelAwait"]}),a=!1,d=0,g=new Set;if(y(s,{ImportDeclaration(s){let r=s.node.source.value;if(!g.has(r)&&r===o){d++,console.log(`Found import from ${o}`);let i=s.node.specifiers,p=new Map,u=[],f=!1;for(let s of i)if(h(s)){let r=s.imported,i=c(r)?r.name:r.value,d=t.find(e=>e.exports.includes(i));if(d){if(console.log(`  Found export ${i} in ${d.source}`),d.isIgnored){console.log(`  Keeping original import for ignored file: ${d.source}`),u.push(s);continue}let e=n?`${o}/${d.source}`:`${o}/${d.source.replace(/\.(js|jsx|ts|tsx|mjs|cjs)$/,"")}`;p.has(e)||p.set(e,[]),p.get(e)?.push({local:s.local,imported:s.imported}),a=!0,f=!0}else{if(t.some(e=>e.source.includes("node_modules")||!e.source.startsWith("."))){let o=`Skipping re-export from external package: ${i} in "${e}"`;console.log(`  ${o}`),l&&l.push(o),u.push(s);continue}let o=`Could not find export ${i} in ${e}`;console.log(`  Warning: ${o}`),l&&l.push(o),u.push(s)}}else u.push(s);if(!f){console.log("  Keeping original import due to no direct exports to migrate"),g.add(r);return}let m=[];for(let[e,s]of(u.length>0&&m.push(x(u,$(o))),p.entries()))m.push(x(s.map(e=>S(e.local,e.imported)),$(e)));s.replaceWithMultiple(m),console.log("  Replaced import with direct imports from source files"),g.add(r)}}}),a){console.log(`Writing changes to ${e}`);let o=w(s,{retainLines:!1,retainFunctionParens:!0},p);r(e,o.code)}else d>0?console.log(`No changes needed for ${d} imports`):console.log("No imports found to update")}catch(o){a&&a.errors++,console.error(`Error processing ${e}:`,o)}}async function I(e){let{sourcePath:r,targetPath:n,ignoreSourceFiles:i,ignoreTargetFiles:l}=e,a={totalFiles:0,filesProcessed:0,filesSkipped:0,importsUpdated:0,filesWithNoUpdates:0,errors:0,totalExports:0,sourceFilesScanned:0,sourceFilesWithExports:0,sourceFilesSkipped:0,targetFilesFound:[],packagesProcessed:0,packagesSkipped:0,warnings:[]},c=await F(r,{cwd:n,absolute:!0,ignore:["**/node_modules/**","**/dist/**","**/build/**","**/target-app/**"],followSymbolicLinks:!1,onlyDirectories:!0}),p=[];for(let e of c){let s=t.join(e,"package.json");try{await o.promises.access(s),p.push(s)}catch{console.log(`No package.json found in ${e}, skipping`)}}for(let o of(console.log(`Found ${p.length} source packages to process`),p))try{let r=t.dirname(o);console.log(`
+Processing package: ${r}`);let c=await b(r),p=await E({packagePath:r,ignoreSourceFiles:i,stats:a});a.totalExports+=p.reduce((e,o)=>e+o.exports.length,0),a.sourceFilesWithExports+=new Set(p.map(e=>e.source)).size,a.sourceFilesScanned+=(await F("**/*.{ts,tsx}",{cwd:r,ignore:["**/node_modules/**","**/dist/**","**/build/**"]})).length;let d=await P({packageName:c.name,monorepoRoot:n});for(let o of(a.totalFiles+=d.length,a.targetFilesFound.push(...d),d)){let r=t.relative(n,o);if(l.some(e=>k.isMatch(r,e))){console.log(`Skipping ignored file: ${o} (matches pattern in ${l.join(", ")})`),a.filesSkipped++;continue}try{let r=s(o,"utf-8");await j({filePath:o,packageName:c.name,exports:p,includeExtension:e.includeExtension,warnings:a.warnings,stats:a});let t=s(o,"utf-8");r!==t?a.importsUpdated++:a.filesWithNoUpdates++,a.filesProcessed++}catch(e){a.errors++,console.error(`Error processing ${o}:`,e)}}a.packagesProcessed++}catch(e){a.packagesSkipped++,console.error(`Error processing package ${o}:`,e)}if(console.log("\nMigration Summary"),console.log(`Source packages found: ${p.length}`),console.log(`Source packages processed: ${a.packagesProcessed}`),console.log(`Source packages skipped: ${a.packagesSkipped}`),console.log(`Source files found: ${a.sourceFilesScanned}`),console.log(`Source files with exports: ${a.sourceFilesWithExports}`),console.log(`Source files skipped: ${a.sourceFilesSkipped}`),console.log(`Exports found: ${a.totalExports}`),console.log(`Target files found: ${a.totalFiles}`),console.log(`Target files processed: ${a.filesProcessed}`),console.log(`Target files with imports updated: ${a.importsUpdated}`),console.log(`Target files with no changes needed: ${a.filesWithNoUpdates}`),console.log(`Target files skipped: ${a.filesSkipped}`),a.warnings.length>0)for(let e of(console.log("\nWarnings:"),a.warnings))console.log(`  - ${e}`);a.errors>0&&console.log(`
+Warning: ${a.errors} errors encountered during processing`)}let T={targetPath:".",ignoreSourceFiles:[],ignoreTargetFiles:[],includeExtension:!1};async function v(){let o=new e;o.name("migrate-barrel-imports").description("CLI tool to migrate barrel files imports to direct imports").argument("<source-path>",'Glob pattern for source packages containing barrel files (e.g. "libs/*")').argument("[target-path]","Path to the directory where imports should be migrated (default: current directory)").option("--ignore-source-files <patterns>","Comma-separated list of file patterns to ignore in source directory").option("--ignore-target-files <patterns>","Comma-separated list of file patterns to ignore in target directory").option("--no-extension","Exclude js|jsx|ts|tsx|mjs|cjs file extensions from import statements").allowUnknownOption(!1).parse(process.argv);let s=o.args;s[0]||(console.error("Error: source-path is required"),process.exit(1));let r=s[0],t=s[1]||T.targetPath,n=o.opts();await I({sourcePath:r,targetPath:t,ignoreSourceFiles:n.ignoreSourceFiles?n.ignoreSourceFiles.split(","):T.ignoreSourceFiles,ignoreTargetFiles:n.ignoreTargetFiles?n.ignoreTargetFiles.split(","):T.ignoreTargetFiles,includeExtension:!1!==n.extension||T.includeExtension})}(async()=>{await v()})();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "migrate-barrel-imports",
-  "version": "1.3.0",
+  "version": "1.5.0",
   "description": "A CLI tool to migrate barrel imports in JavaScript/TypeScript monorepos",
   "files": [
     "dist/"