Fix ESLint issues

Author: mrmckeb

.eslintrc

@@ -10,5 +10,6 @@
   "parser": "@typescript-eslint/parser",
   "parserOptions": {
     "project": ["./tsconfig.json"]
-  }
+  },
+  "ignorePatterns": ["dist", "jest.config.js", "src/helpers/__tests__/fixtures"]
 }

package.json

@@ -26,8 +26,8 @@
   ],
   "scripts": {
     "build": "rm -rf ./dist && tsc --project tsconfig.build.json",
-    "lint": "eslint --max-warnings 0 . && yarn prettier -c .",
-    "prepublishOnly": "yarn build",
+    "lint": "eslint --max-warnings 0 . && pnpm prettier -c .",
+    "prepublishOnly": "pnpm build",
     "test": "jest",
     "prepare": "husky install"
   },

src/helpers/createDtsExports.ts

@@ -55,7 +55,7 @@ export default classes;
     const smc = new SourceMapConsumer(cssExports.sourceMap);
 
     // Split original CSS file into lines.
-    const cssLines = cssExports.css?.split('\n') || [];
+    const cssLines = cssExports.css?.split('\n') ?? [];
 
     // Create new equal size array of empty strings.
     const dtsLines = Array.from(Array(cssLines.length), () => '');

src/helpers/getCssExports.ts

@@ -52,7 +52,7 @@ export const getCssExports = ({
 }): CSSExportsWithSourceMap => {
   try {
     const fileType = getFileType(fileName);
-    const rendererOptions = options.rendererOptions || {};
+    const rendererOptions = options.rendererOptions ?? {};
 
     let transformedCss = '';
     let sourceMap: RawSourceMap | undefined;
@@ -73,7 +73,7 @@ export const getCssExports = ({
             {
               syncImport: true,
               filename: fileName,
-              ...(rendererOptions.less || {}),
+              ...(rendererOptions.less ?? {}),
             } as Less.Options,
             (
               error: Less.RenderError | undefined,
@@ -93,7 +93,7 @@ export const getCssExports = ({
         case FileType.scss:
         case FileType.sass: {
           const filePath = getFilePath(fileName);
-          const { loadPaths, ...sassOptions } = rendererOptions.sass || {};
+          const { loadPaths, ...sassOptions } = rendererOptions.sass ?? {};
           const { baseUrl, paths } = compilerOptions;
           const matchPath =
             baseUrl && paths
@@ -111,7 +111,7 @@ export const getCssExports = ({
 
           const result = sass.compile(fileName, {
             importers,
-            loadPaths: [filePath, 'node_modules', ...(loadPaths || [])],
+            loadPaths: [filePath, 'node_modules', ...(loadPaths ?? [])],
             sourceMap: true,
             // sourceMapIncludeSources: true,
             ...sassOptions,
@@ -124,7 +124,7 @@ export const getCssExports = ({
 
         case FileType.styl:
           transformedCss = stylus(css, {
-            ...(rendererOptions.stylus || {}),
+            ...(rendererOptions.stylus ?? {}),
             filename: fileName,
           }).render();
           break;

src/helpers/getDtsSnapshot.ts

@@ -21,7 +21,7 @@ export const getDtsSnapshot = (
    * https://github.com/mrmckeb/typescript-plugin-css-modules/issues/41
    * Needs investigation for a more elegant solution.
    */
-  if (/export default classes/.test(css)) {
+  if (css.includes('export default classes')) {
     return scriptSnapshot;
   }
 

src/helpers/logger.ts

@@ -10,7 +10,7 @@ export const createLogger = (info: ts.server.PluginCreateInfo): Logger => {
     );
   };
   const error = (error: unknown) => {
-    log(`Failed with error: ${error}`);
+    log(`Failed with error: ${error as string}`);
   };
 
   return {

src/helpers/validVarRegexp.ts

@@ -20,7 +20,6 @@ const getPostCssConfigPlugins = (directory: string) => {
   }
 };
 
-// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
 function init({ typescript: ts }: { typescript: typeof tsModule }) {
   let _isCSS: isCSSFn;
 
@@ -33,16 +32,18 @@ function init({ typescript: ts }: { typescript: typeof tsModule }) {
     process.chdir(directory);
 
     // User options for plugin.
-    const options: Options = info.config.options || {};
+
+    const config = info.config as { options?: Options };
+    const options = config.options ?? {};
     logger.log(`options: ${JSON.stringify(options)}`);
 
     // Load environment variables like SASS_PATH.
     // TODO: Add tests for this option.
-    const dotenvOptions = options.dotenvOptions || {};
+    const dotenvOptions = options.dotenvOptions;
     if (dotenvOptions) {
       dotenvOptions.path = path.resolve(
         directory,
-        dotenvOptions.path || '.env',
+        dotenvOptions.path ?? '.env',
       );
     }
     dotenv.config(dotenvOptions);
@@ -60,7 +61,7 @@ function init({ typescript: ts }: { typescript: typeof tsModule }) {
 
     // Add postCSS config if enabled.
     const postcssOptions =
-      options.postcssOptions || options.postCssOptions || {};
+      options.postcssOptions ?? options.postCssOptions ?? {};
 
     let userPlugins: AcceptedPlugin[] = [];
     if (postcssOptions.useConfig) {
@@ -212,7 +213,7 @@ function init({ typescript: ts }: { typescript: typeof tsModule }) {
 
               // Filter to only one extension type, and remove that extension. This leaves us with the actual filename.
               // Example: "usr/person/project/src/dir/File.module.css/index.d.ts" > "usr/person/project/src/dir/File.module.css"
-              const normalizedLocations = failedLocations.reduce(
+              const normalizedLocations = failedLocations.reduce<string[]>(
                 (locations, location) => {
                   if (
                     (baseUrl ? location.includes(baseUrl) : true) &&
@@ -222,7 +223,7 @@ function init({ typescript: ts }: { typescript: typeof tsModule }) {
                   }
                   return locations;
                 },
-                [] as string[],
+                [],
               );
 
               // Find the imported CSS module, if it exists.

src/index.ts

@@ -1,5 +1,6 @@
 // TODO: Remove this when the related issue is resolved.
 // https://github.com/css-modules/postcss-icss-keyframes/issues/3
+// eslint-disable-next-line @typescript-eslint/unbound-method
 const warn = global.console.warn;
 global.console.warn = (...args: unknown[]) => {
   const isPostCSSDeprecationWarning =

src/setup-tests.ts

@@ -1,4 +1,4 @@
 {
   "extends": "./tsconfig.json",
-  "exclude": ["**/__tests__/**", "setup-tests.js"]
+  "exclude": ["dist", "**/__tests__/**", "setup-tests.js"]
 }