ember-codemod-remove-global-styles 0.6.0 → 0.6.2

package/README.md

@@ -68,13 +68,13 @@ pnpx ember-codemod-remove-global-styles --convert routes
 By default, the codemod considers all files and folders for components and routes. Pass `--folder` to limit the search to 1 folder. (You may use glob patterns to specify multiple folders.)
 
 ```sh
-# `ui` folder only
+# `ui` folder only (search `app/{components,templates}/ui`)
 pnpx ember-codemod-remove-global-styles --folder ui
 
-# `ui/form` folder only
+# `ui/form` folder only (search `app/{components,templates}/ui/form`)
 pnpx ember-codemod-remove-global-styles --folder ui/form
 
-# `route1` and `route2` folders only
+# `route1` and `route2` folders only (search `app/templates/{route1,route2}`)
 pnpx ember-codemod-remove-global-styles --convert routes --folder "{route1,route2}"
 ```
 

package/dist/src/steps/update-project/update-classes.js

@@ -3,12 +3,18 @@ import { updateClass } from '../../utils/update-project/index.js';
 export function updateClasses(project, options) {
     const fileMap = new Map();
     project.components.forEach((_data, templateFilePath) => {
-        const { classFile, classFilePath } = updateClass(templateFilePath, options);
-        fileMap.set(classFilePath, classFile);
+        const { output, status } = updateClass(templateFilePath, options);
+        if (status === 'error') {
+            return;
+        }
+        fileMap.set(output.classFilePath, output.classFile);
     });
     project.routes.forEach((_data, templateFilePath) => {
-        const { classFile, classFilePath } = updateClass(templateFilePath, options);
-        fileMap.set(classFilePath, classFile);
+        const { output, status } = updateClass(templateFilePath, options);
+        if (status === 'error') {
+            return;
+        }
+        fileMap.set(output.classFilePath, output.classFile);
     });
     createFiles(fileMap, options);
 }

package/dist/src/steps/update-project/update-templates.js

@@ -3,12 +3,18 @@ import { updateTemplate } from '../../utils/update-project/index.js';
 export function updateTemplates(project, options) {
     const fileMap = new Map();
     project.components.forEach((_data, templateFilePath) => {
-        const templateFile = updateTemplate(templateFilePath, options);
-        fileMap.set(templateFilePath, templateFile);
+        const { output, status } = updateTemplate(templateFilePath, options);
+        if (status === 'error') {
+            return;
+        }
+        fileMap.set(templateFilePath, output.templateFile);
     });
     project.routes.forEach((_data, templateFilePath) => {
-        const templateFile = updateTemplate(templateFilePath, options);
-        fileMap.set(templateFilePath, templateFile);
+        const { output, status } = updateTemplate(templateFilePath, options);
+        if (status === 'error') {
+            return;
+        }
+        fileMap.set(templateFilePath, output.templateFile);
     });
     createFiles(fileMap, options);
 }

package/dist/src/utils/css/get-class-to-styles.js

@@ -31,6 +31,8 @@ export function getClassToStyles(file) {
     const plugins = [
         {
             postcssPlugin: 'postcss-get-class-to-styles',
+            // @ts-expect-error: Incorrect type
+            // eslint-disable-next-line @typescript-eslint/explicit-function-return-type
             prepare() {
                 return {
                     Rule: processRule,
@@ -38,7 +40,6 @@ export function getClassToStyles(file) {
             },
         },
     ];
-    // @ts-expect-error: Incorrect type
     // eslint-disable-next-line @typescript-eslint/no-unused-expressions
     postcss(plugins).process(file).css;
     return classToStyles;

package/dist/src/utils/update-project/update-class.js

@@ -10,20 +10,36 @@ export function updateClass(templateFilePath, options) {
         isTemplateTag: ext === '.gjs' || ext === '.gts',
         isTypeScript: ext === '.gts' || ext === '.ts',
     };
-    if (data.isTemplateTag) {
-        classFile = updateJavaScript(classFile, (code) => {
-            return importStylesheet(code, data);
-        });
+    try {
+        if (data.isTemplateTag) {
+            classFile = updateJavaScript(classFile, (code) => {
+                return importStylesheet(code, data);
+            });
+            return {
+                output: {
+                    classFile,
+                    classFilePath,
+                },
+                status: 'success',
+            };
+        }
+        classFile = replaceTemplateOnlyComponent(classFile, data);
+        classFile = importStylesheet(classFile, data);
+        classFile = addStylesToClass(classFile, data);
         return {
-            classFile,
-            classFilePath,
+            output: {
+                classFile,
+                classFilePath,
+            },
+            status: 'success',
+        };
+    }
+    catch (error) {
+        console.log(`WARNING: ${classFilePath} could not be updated.`);
+        console.log(error.message);
+        return {
+            output: undefined,
+            status: 'error',
         };
     }
-    classFile = replaceTemplateOnlyComponent(classFile, data);
-    classFile = importStylesheet(classFile, data);
-    classFile = addStylesToClass(classFile, data);
-    return {
-        classFile,
-        classFilePath,
-    };
 }

package/dist/src/utils/update-project/update-template.js

@@ -4,16 +4,38 @@ import { updateTemplates } from '@codemod-utils/ast-template-tag';
 import { addLocalClasses, getClassToStyles, getModuleFilePath, } from '../css/index.js';
 export function updateTemplate(templateFilePath, options) {
     const { projectRoot } = options;
+    let templateFile = readFileSync(join(projectRoot, templateFilePath), 'utf8');
     const cssModuleFile = readFileSync(join(projectRoot, getModuleFilePath(templateFilePath)), 'utf8');
-    const templateFile = readFileSync(join(projectRoot, templateFilePath), 'utf8');
     const data = {
         classToStyles: getClassToStyles(cssModuleFile),
         isHbs: templateFilePath.endsWith('.hbs'),
     };
-    if (data.isHbs) {
-        return addLocalClasses(templateFile, data);
+    try {
+        if (data.isHbs) {
+            templateFile = addLocalClasses(templateFile, data);
+            return {
+                output: {
+                    templateFile,
+                },
+                status: 'success',
+            };
+        }
+        templateFile = updateTemplates(templateFile, (code) => {
+            return addLocalClasses(code, data);
+        });
+        return {
+            output: {
+                templateFile,
+            },
+            status: 'success',
+        };
+    }
+    catch (error) {
+        console.log(`WARNING: ${templateFilePath} could not be updated.`);
+        console.log(error.message);
+        return {
+            output: undefined,
+            status: 'error',
+        };
     }
-    return updateTemplates(templateFile, (code) => {
-        return addLocalClasses(code, data);
-    });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ember-codemod-remove-global-styles",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "Codemod to localize global styles",
   "keywords": [
     "codemod",
@@ -20,32 +20,29 @@
   "type": "module",
   "main": "dist/src/index.js",
   "bin": "dist/bin/ember-codemod-remove-global-styles.js",
-  "directories": {
-    "test": "tests"
-  },
   "files": [
     "dist"
   ],
   "dependencies": {
-    "@codemod-utils/ast-javascript": "^2.1.2",
-    "@codemod-utils/ast-template": "^2.1.2",
-    "@codemod-utils/ast-template-tag": "^1.2.0",
-    "@codemod-utils/files": "^3.2.3",
+    "@codemod-utils/ast-javascript": "^2.1.3",
+    "@codemod-utils/ast-template": "^2.1.3",
+    "@codemod-utils/ast-template-tag": "^1.2.1",
+    "@codemod-utils/files": "^3.2.4",
     "postcss": "^8.5.6",
     "yargs": "^18.0.0"
   },
   "devDependencies": {
-    "@codemod-utils/tests": "^2.2.2",
+    "@codemod-utils/tests": "^2.2.3",
     "@sondr3/minitest": "^0.1.2",
-    "@types/node": "^20.19.27",
+    "@types/node": "^20.19.30",
     "@types/yargs": "^17.0.35",
     "concurrently": "^9.2.1",
     "eslint": "^9.39.2",
-    "prettier": "^3.7.4",
+    "prettier": "^3.8.1",
     "typescript": "^5.9.3",
     "@shared-configs/eslint-config-node": "0.0.0",
-    "@shared-configs/prettier": "0.0.0",
-    "@shared-configs/typescript": "0.0.0"
+    "@shared-configs/typescript": "0.0.0",
+    "@shared-configs/prettier": "0.0.0"
   },
   "engines": {
     "node": "20.* || >= 22"