@zeroheight/adoption-cli 0.2.13 → 0.3.0

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Release notes
 
+## [0.3.0](https://www.npmjs.com/package/@zeroheight/adoption-cli/v/0.3.0) - 29th July 2024
+
+- Replace acorn parser with oxc-parser
+
 ## [0.2.13](https://www.npmjs.com/package/@zeroheight/adoption-cli/v/0.2.13) - 25th July 2024
 
 - Fix ws dependency security issue — https://github.com/advisories/GHSA-3h5v-q93c-6h6q

package/dist/ast/analyze.d.ts

@@ -1,4 +1,8 @@
-import type { Node } from "acorn";
+import type { Node, Program, Identifier } from "acorn";
+export interface PlainIdentifier extends Node {
+    type: "Identifier";
+    local: Identifier;
+}
 export type VisitorState = {
     components: Map<string, number>;
     imports: Map<string, {
@@ -11,4 +15,4 @@ export type RawUsage = {
     count: number;
     package: string;
 };
-export declare function analyze(ast: Node): RawUsage[];
+export declare function analyze(ast: Program): RawUsage[];

package/dist/ast/analyze.js

@@ -23,27 +23,98 @@ export function analyze(ast) {
         components: new Map(),
         imports: new Map(),
     };
-    const visitors = {
-        JSXElement(node, state, _nodePath) {
+    const visitorFunctions = {
+        CatchClause() { },
+        CatchParameter() { },
+        ComputedMemberExpression() { },
+        JSXText() { },
+        NullLiteral() { },
+        NumericLiteral() { },
+        BooleanLiteral() { },
+        ObjectExpression() { },
+        ObjectProperty() { },
+        StringLiteral() { },
+        TSAsExpression() { },
+        TSEnumDeclaration() { },
+        TSInterfaceDeclaration() { },
+        TSTypeAliasDeclaration() { },
+        TSModuleDeclaration() { },
+        TSSatisfiesExpression() { },
+        StaticMemberExpression() { },
+        RegExpLiteral() { },
+        TSNonNullExpression() { },
+        JSXEmptyExpression() { },
+        JSXFragment(node, state) {
+            node.children.forEach((child) => walk.recursive(child, state, visitorFunctions, walk.base));
+        },
+        IfStatement(node, state) {
+            walk.recursive(node.consequent, state, visitorFunctions, walk.base);
+            if (node.alternate) {
+                walk.recursive(node.alternate, state, visitorFunctions, walk.base);
+            }
+        },
+        VariableDeclaration(node, state) {
+            node.declarations.forEach((child) => {
+                walk.recursive(child, state, visitorFunctions, walk.base);
+            });
+        },
+        ArrowFunctionExpression(node, state) {
+            if (node.body.type === "FunctionBody") {
+                walk.recursive(node.body, state, visitorFunctions, walk.base);
+            }
+        },
+        ParenthesizedExpression(node, state) {
+            walk.recursive(node.expression, state, visitorFunctions, walk.base);
+        },
+        ReturnStatement(node, state) {
+            if (node.argument) {
+                walk.recursive(node.argument, state, visitorFunctions, walk.base);
+            }
+        },
+        FunctionBody(node, state) {
+            node.statements.forEach((child) => walk.recursive(child, state, visitorFunctions, walk.base));
+        },
+        Program(node, state) {
+            node.body.forEach((child) => {
+                walk.recursive(child, state, visitorFunctions, walk.base);
+            });
+        },
+        JSXElement(node, state) {
             const el = node.openingElement;
             const elName = el.name.name;
-            if (!elName)
-                return;
             // Ignore html tags e.g. div, h1, span
-            if (elName[0] === elName[0]?.toLocaleLowerCase())
-                return;
-            state.components.set(elName, (state.components.get(elName) ?? 0) + 1);
+            if (elName?.[0] !== elName?.[0]?.toLocaleLowerCase()) {
+                state.components.set(elName, (state.components.get(elName) ?? 0) + 1);
+            }
+            node.children.forEach((child) => {
+                walk.recursive(child, state, visitorFunctions, walk.base);
+            });
+        },
+        CallExpression(node, state) {
+            node.arguments.forEach((child) => walk.recursive(child, state, visitorFunctions, walk.base));
+        },
+        ExportDefaultDeclaration(node, state) {
+            if (node.declaration.type === "FunctionDeclaration") {
+                const { statements } = node.declaration.body;
+                statements.forEach((child) => {
+                    walk.recursive(child, state, visitorFunctions, walk.base);
+                });
+            }
+        },
+        JSXExpressionContainer(node, state) {
+            walk.recursive(node.expression, state, visitorFunctions, walk.base);
         },
-        ImportDeclaration(node, state, _nodePath) {
-            const packageName = node.source.value;
-            node.specifiers.forEach((specifier) => {
+        ImportDeclaration(node, state) {
+            const packageName = node.source.value?.toString() ?? "";
+            node.specifiers?.forEach((specifier) => {
                 // not handling namespace imports as we can't determine the actual name
                 // e.g. import * as MyIconLibrary from 'my-icon-library'
                 if (specifier.type === "ImportNamespaceSpecifier") {
                     return;
                 }
                 let name = specifier.local.name;
-                if (specifier.type === "ImportDefaultSpecifier") {
+                if (specifier.type === "ImportDefaultSpecifier" ||
+                    specifier.type === "Identifier") {
                     // handles the following cases:
                     // import Icon from 'my-icon-library'
                     name = specifier.local.name;
@@ -63,23 +134,10 @@ export function analyze(ast) {
                 });
             });
         },
-    };
-    const base = {
-        ...walk.base,
-        TSInterfaceDeclaration() { },
-        TSModuleDeclaration() { },
-        TSAsExpression() { },
-        TSDeclareFunction() { },
-        TSTypeAliasDeclaration() { },
-        TSNonNullExpression() { },
-        TSEnumDeclaration() { },
-        TSParameterProperty() { },
-        TSImportEqualsDeclaration() { },
-        TSInstantiationExpression() { },
-        TSDeclareMethod() { },
+        // FIXME: Not the best typing but there are additional types not in acorn
     };
     try {
-        walk.ancestor(ast, visitors, base, visitorState);
+        walk.recursive(ast, visitorState, visitorFunctions, walk.base);
     }
     catch (e) {
         // We don't want to log the error to the users - maybe could add some actually logging in the future

package/dist/ast/parser.d.ts

@@ -1,11 +1,10 @@
-import * as acorn from "acorn";
 /**
  * Parse code and return AST
  *
  * This extends the acorn parser with support for TypeScript and JSX
  *
- * @param code The string containing JS/TS
- * @param version a version of ECMA Script to use, defaults to latest
+ * @param code The string containing JS?X/TS?X
+ * @param sourceFilename a filename to help with inference
  * @returns Parsed AST
  */
-export declare function parse(code: string, version?: acorn.ecmaVersion): acorn.Program;
+export declare function parse(code: string, sourceFilename?: string): any;

package/dist/ast/parser.js

@@ -1,23 +1,16 @@
-import * as acorn from "acorn";
-import { tsPlugin } from "acorn-typescript";
-import * as walk from "acorn-walk";
-import { extend } from "acorn-jsx-walk";
+import oxc from "oxc-parser";
 /**
  * Parse code and return AST
  *
  * This extends the acorn parser with support for TypeScript and JSX
  *
- * @param code The string containing JS/TS
- * @param version a version of ECMA Script to use, defaults to latest
+ * @param code The string containing JS?X/TS?X
+ * @param sourceFilename a filename to help with inference
  * @returns Parsed AST
  */
-export function parse(code, version = "latest") {
-    const typescriptPlugin = tsPlugin();
-    extend(walk.base);
-    const JSXParser = acorn.Parser.extend(typescriptPlugin);
-    return JSXParser.parse(code, {
-        ecmaVersion: version,
-        sourceType: "module",
-        locations: true,
-    });
+export function parse(code, sourceFilename) {
+    const result = oxc.parseSync(code, { sourceFilename, sourceType: "module" });
+    // TODO: Surface these errors to help in debugging
+    // console.error(result.errors)
+    return JSON.parse(result.program);
 }

package/dist/commands/analyze.utils.js

@@ -55,7 +55,7 @@ export async function analyzeFiles(extensions, ignorePattern) {
     for (const file of files) {
         try {
             const fileContents = await readFile(file, "utf-8");
-            const ast = parse(fileContents);
+            const ast = parse(fileContents, file);
             const usage = analyze(ast);
             if (usage.length > 0) {
                 const relativePath = file.slice(process.cwd().length);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zeroheight/adoption-cli",
-  "version": "0.2.13",
+  "version": "0.3.0",
   "license": "ISC",
   "main": "dist/cli.js",
   "bin": {
@@ -26,8 +26,6 @@
   ],
   "dependencies": {
     "acorn": "^8.11.3",
-    "acorn-jsx-walk": "^2.0.0",
-    "acorn-typescript": "^1.4.13",
     "acorn-walk": "^8.3.2",
     "chalk": "^5.2.0",
     "commander": "^12.0.0",
@@ -39,6 +37,7 @@
     "ink-select-input": "^5.0.0",
     "ink-spinner": "^5.0.0",
     "ink-text-input": "^5.0.1",
+    "oxc-parser": "^0.22.0",
     "react": "^18.2.0",
     "yn": "^5.0.0"
   },