extract functions from file

Author: Idrinth

src/build-map.ts

@@ -1,52 +1,9 @@
 import css from 'css';
 import {readFileSync} from 'fs';
+import Rules from "./rules.js";
+import handleRule from "./handle-rule.js";
+import handleAtRule from "./handle-at-rule.js";
 
-interface Rules {
-    [selector:string]: {
-        [media:string]: {
-            [property:string]: string
-        }
-    }
-}
-const handleDeclaration = (selector: string, declaration: css.Declaration, position: css.Position|undefined, media: string, rules: Rules): boolean => {
-  let fails = false;
-  if (rules[selector]) {
-    if (rules[selector][media]) {
-      if (rules[selector][media][declaration.property || '']) {
-        fails = true;
-        console.error(`${declaration.property} of ${selector} overwrites a previous definition in line ${position?.line} for media ${media}.`);
-      }
-    }
-    if (media !== '' && rules[selector]['']) {
-      if (rules[selector][''][declaration.property || ''] && rules[selector][''][declaration.property || ''] === declaration.value || '') {
-        fails = true;
-        console.error(`${declaration.property} of ${selector} overwrites a previous definition in line ${position?.line} general with the same value.`);
-      }
-    }
-  }
-  rules[selector] = rules[selector] || {};
-  rules[selector][media] = rules[selector][media] || {};
-  rules[selector][media][declaration.property || ''] = declaration.value || '';
-  return fails;
-}
-const handleRule = (rule: css.Rule, rules: Rules, media: string = '') => {
-  let fails = false;
-  for (const selector of rule.selectors || []) {
-    for (const declaration of rule.declarations || []) {
-      if (declaration.type === 'declaration') {
-        fails = handleDeclaration(selector, declaration, rule.position?.start, media, rules) || fails;
-      }
-    }
-  }
-  return fails;
-};
-const handleAtRule = (rule: css.Media, rules: Rules): boolean => {
-  let fails = false;
-  for (const rul of rule.rules || []) {
-    fails = handleRule(rul, rules, rule.media) || fails;
-  }
-  return fails;
-};
 export default(file: string): boolean => {
   const rules: Rules = {};
   let fails = false;

src/handle-at-rule.ts

@@ -0,0 +1,11 @@
+import css from "css";
+import Rules from "./rules.js";
+import handleRule from "./handle-rule.js";
+
+export default (rule: css.Media, rules: Rules): boolean => {
+  let fails = false;
+  for (const rul of rule.rules || []) {
+    fails = handleRule(rul, rules, rule.media) || fails;
+  }
+  return fails;
+};

src/handle-declaration.ts

@@ -0,0 +1,24 @@
+import css from "css";
+import Rules from "./rules.js";
+
+export default (selector: string, declaration: css.Declaration, position: css.Position | undefined, media: string, rules: Rules): boolean => {
+  let fails = false;
+  if (rules[selector]) {
+    if (rules[selector][media]) {
+      if (rules[selector][media][declaration.property || '']) {
+        fails = true;
+        console.error(`${declaration.property} of ${selector} overwrites a previous definition in line ${position?.line} for media ${media}.`);
+      }
+    }
+    if (media !== '' && rules[selector]['']) {
+      if (rules[selector][''][declaration.property || ''] && rules[selector][''][declaration.property || ''] === declaration.value || '') {
+        fails = true;
+        console.error(`${declaration.property} of ${selector} overwrites a previous definition in line ${position?.line} general with the same value.`);
+      }
+    }
+  }
+  rules[selector] = rules[selector] || {};
+  rules[selector][media] = rules[selector][media] || {};
+  rules[selector][media][declaration.property || ''] = declaration.value || '';
+  return fails;
+}

src/handle-rule.ts

@@ -0,0 +1,15 @@
+import css from "css";
+import Rules from "./rules.js";
+import {handleDeclaration} from "./handle-declaration.js";
+
+export default (rule: css.Rule, rules: Rules, media: string = '') => {
+  let fails = false;
+  for (const selector of rule.selectors || []) {
+    for (const declaration of rule.declarations || []) {
+      if (declaration.type === 'declaration') {
+        fails = handleDeclaration(selector, declaration, rule.position?.start, media, rules) || fails;
+      }
+    }
+  }
+  return fails;
+};

src/index.ts

@@ -2,11 +2,11 @@ import {readdirSync} from 'node:fs';
 import buildMap from './build-map.js';
 
 export default (cwd: string): boolean => {
-  let fails = false;
+  let success = false;
   for (const file of readdirSync(cwd, {recursive: true, encoding: 'utf8'})) {
     if (file.endsWith('.css') && !file.includes('node_modules') && !file.includes('coverage') && !file.includes('dist')) {
-      fails = buildMap(cwd + '/' + file) || fails;
+      success = buildMap(cwd + '/' + file) || success;
     }
   }
-  return fails;
+  return success;
 }

src/rules.ts

@@ -0,0 +1,7 @@
+export default interface Rules {
+  [selector: string]: {
+    [media: string]: {
+      [property: string]: string
+    }
+  }
+}