Merge pull request #16 from CodeshiftCommunity/new-publishing

New publishing approach

Author: danieldelcore

community/@atlaskit__avatar/codeshift.config.js

@@ -1,3 +1,7 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '18.0.0': require('./18.0.0/transform'),
+    '19.0.0': require('./19.0.0/transform'),
+  },
 };

community/@atlaskit__breadcrumbs/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '11.0.0': require('./11.0.0/transform'),
+  },
 };

community/@atlaskit__button/codeshift.config.js

@@ -1,3 +1,7 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '15.0.0': require('./15.0.0/transform'),
+    '15.1.1': require('./15.1.1/transform'),
+  },
 };

community/@atlaskit__calendar/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '11.0.0': require('./11.0.0/transform'),
+  },
 };

community/@atlaskit__checkbox/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '12.0.0': require('./12.0.0/transform'),
+  },
 };

community/@atlaskit__icon/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '21.2.0': require('./21.2.0/transform'),
+  },
 };

community/@atlaskit__menu/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '0.7.0': require('./0.7.0/transform'),
+  },
 };

community/@atlaskit__popper/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '5.0.0': require('./5.0.0/transform'),
+  },
 };

community/@atlaskit__popup/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '1.0.0': require('./1.0.0/transform'),
+  },
 };

community/@atlaskit__progress-indicator/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '9.0.0': require('./9.0.0/transform'),
+  },
 };

community/@atlaskit__range/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '4.0.0': require('./4.0.0/transform'),
+  },
 };

community/@atlaskit__section-message/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '6.0.0': require('./6.0.0/transform'),
+  },
 };

community/@atlaskit__side-navigation/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '0.8.0': require('./0.8.0/transform'),
+  },
 };

community/@atlaskit__spinner/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '13.0.0': require('./13.0.0/transform'),
+  },
 };

community/@atlaskit__tag/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '11.0.0': require('./11.0.0/transform'),
+  },
 };

community/@atlaskit__textarea/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '4.0.0': require('./4.0.0/transform'),
+  },
 };

community/@atlaskit__textfield/codeshift.config.js

@@ -1,3 +1,6 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '5.0.0': require('./5.0.0/transform'),
+  },
 };

community/@atlaskit__toggle/codeshift.config.js

@@ -1,3 +1,7 @@
 export default {
   maintainers: ['danieldelcore'],
+  transforms: {
+    '11.0.0': require('./11.0.0/transform'),
+    '12.0.0': require('./12.0.0/transform'),
+  },
 };

packages/initializer/package.json

@@ -11,6 +11,7 @@
   },
   "dependencies": {
     "fs-extra": "^9.1.0",
+    "recast": "^0.20.4",
     "semver": "^7.3.5",
     "ts-node": "^9.1.1"
   }

packages/initializer/src/index.ts

@@ -1,5 +1,6 @@
 import fs from 'fs-extra';
 import semver from 'semver';
+import * as recast from 'recast';
 
 function transformPackageName(packageName: string) {
   return packageName.replace('/', '__');
@@ -38,10 +39,51 @@ function main(packageName: string, version: string) {
     fs.writeFileSync(
       configPath,
       `export default {
-  maintainers: ['danieldelcore'],
+  maintainers: [],
+  transforms: {
+    '${version}': require('./${version}/transform'),
+  }
 };
 `,
     );
+  } else {
+    const source = fs.readFileSync(configPath, 'utf8');
+    const ast = recast.parse(source);
+    const b = recast.types.builders;
+
+    recast.visit(ast, {
+      visitProperty(path) {
+        // @ts-ignore
+        if (path.node.key.name !== 'transforms') return false;
+        // @ts-ignore
+        const properties = path.node.value.properties;
+        // @ts-ignore
+        properties.forEach(property => {
+          if (semver.eq(property.key.value, version)) {
+            throw new Error(
+              `Transform for ${packageName} version ${version} already exists`,
+            );
+          }
+        });
+
+        properties.push(
+          b.property(
+            'init',
+            b.stringLiteral(version),
+            b.callExpression(b.identifier('require'), [
+              b.stringLiteral(`./${version}/transform`),
+            ]),
+          ),
+        );
+
+        return false;
+      },
+    });
+
+    fs.writeFileSync(
+      configPath,
+      recast.prettyPrint(ast, { quote: 'single', trailingComma: true }).code,
+    );
   }
 
   fs.copyFileSync(

packages/publisher/src/generate.ts

@@ -45,28 +45,15 @@ function getPackageJson(packageName: string, version: string) {
   );
 }
 
-function getIndexFile(path: string) {
-  return fs
-    .readdirSync(path)
-    .filter(subDirPath => fs.lstatSync(`${path}/${subDirPath}`).isDirectory())
-    .map(
-      transformDir =>
-        `export { default as transform${transformDir.replace(
-          /\./gi,
-          '_',
-        )} } from \'./${transformDir}/transform\';\n`,
-    )
-    .join('')
-    .trim();
-}
-
 export default async function generatePackages(
   sourcePath: string,
   targetPath: string,
   changedPackages: string[],
 ) {
-  fs.mkdirSync(targetPath);
-  fs.readdirSync(sourcePath)
+  await fs.mkdir(targetPath);
+  const directories = await fs.readdir(sourcePath);
+
+  directories
     .filter(dir => changedPackages.includes(dir))
     .forEach(async dir => {
       const packageName = `@codeshift/mod-${dir
@@ -77,20 +64,20 @@ export default async function generatePackages(
       const nextPackageVersion = semver.inc(packageVersion, 'patch');
 
       const basePath = `${targetPath}/${dir}`;
-      fs.copySync(`${sourcePath}/${dir}`, `${basePath}/src`);
-      fs.copyFileSync(
+      await fs.copy(`${sourcePath}/${dir}`, `${basePath}/src`);
+      await fs.copyFile(
         `${__dirname}/../template/LICENSE`,
         `${basePath}/LICENSE`,
       );
-      fs.copyFileSync(
+      await fs.copyFile(
         `${__dirname}/../template/.npmignore`,
         `${basePath}/.npmignore`,
       );
-      fs.writeFileSync(
-        `${basePath}/src/index.ts`,
-        getIndexFile(`${basePath}/src`),
+      await fs.rename(
+        `${basePath}/src/codeshift.config.js`,
+        `${basePath}/src/index.js`,
       );
-      fs.writeFileSync(
+      await fs.writeFile(
         `${basePath}/package.json`,
         getPackageJson(packageName, nextPackageVersion!),
       );

packages/publisher/src/index.ts

@@ -29,4 +29,6 @@ async function main(sourcePath: string, targetPath: string) {
   cleanTargetDir(targetPath);
 }
 
-main(process.argv[2], process.argv[3]);
+main(process.argv[2], process.argv[3]).catch(error => {
+  console.error('Publishing error:', error.message);
+});