fix lint issues

Author: 7h3Rabbit

eslint.config.js

@@ -5,7 +5,8 @@ export default [
     languageOptions: {
       globals: {
         node: true,
-        es6: true
+        es6: true,
+        URL: "readonly"
       },
       parserOptions: {
         ecmaVersion: 'latest',
@@ -22,5 +23,8 @@ export default [
       'unicorn/prefer-spread': 0
     }
   },
+  {
+    ignores: ["test/*"]
+  },
   js.configs.recommended
 ];

lib/harAnalyzer.js

@@ -69,7 +69,7 @@ export class HarAnalyzer {
         for (const htmlObj of data.htmls) {
             const dom = new JSDOM(htmlObj.content);
             const styleElements = dom.window.document.querySelectorAll('script');
-            styleElements.forEach((styleElement, index) => {
+            styleElements.forEach((styleElement) => {
                 const styleElementObj = {
                     'url': htmlObj.url,
                     'content': styleElement.textContent,
@@ -103,55 +103,44 @@ export class HarAnalyzer {
             return knowledgeData;
         }
 
-        try {
-            // Lint with both security and standard configurations
-            const lintPromises = analyzedData['all-scripts'].flatMap(entry => [
-                this.eslintSecurity.lintText(entry.content).then(results => {
-                    return results.flatMap(result =>
-                        result.messages
-                            .filter(message => message.ruleId !== null) // Ignore messages with null ruleId
-                            .map(message => ({
-                                url: entry.url,
-                                rule: message.ruleId,
-                                category: 'security',
-                                severity: message.severity === 1 ? "warning" : message.severity === 2 ? "error" : message.severity,
-                                text: message.message,
-                                line: message.line,
-                                column: message.column
-                            }))
-                    );
-                }),
-                this.eslintStandard.lintText(entry.content).then(results => {
-                    return results.flatMap(result =>
-                        result.messages
-                            .filter(message => message.ruleId !== null) // Ignore messages with null ruleId
-                            .map(message => ({
-                                url: entry.url,
-                                rule: message.ruleId,
-                                category: 'standard',
-                                severity: message.severity === 1 ? "warning" : message.severity === 2 ? "error" : message.severity,
-                                text: message.message,
-                                line: message.line,
-                                column: message.column
-                            }))
-                    );
-                })
-            ]);
-
-            // Wait for all linting promises to resolve and flatten the results
-            const lintResults = await Promise.all(lintPromises);
-            knowledgeData.issues = lintResults.flat();
-
-            // Populate "resolved-rules" with rules not mentioned in "issues"
-            // const allRules = [
-            //     ...Object.keys(this.configSecurity[0].rules),
-            //     ...Object.keys(this.configStandard[0].rules)
-            // ];
-            // const mentionedRules = new Set(knowledgeData.issues.map(issue => issue.rule));
-            // knowledgeData['resolved-rules'] = allRules.filter(rule => !mentionedRules.has(rule));
-        } catch (err) {
-            // console.error('Error during linting:', err);
-        }
+        // Lint with both security and standard configurations
+        const lintPromises = analyzedData['all-scripts'].flatMap(entry => [
+            this.eslintSecurity.lintText(entry.content).then(results => {
+                return results.flatMap(result =>
+                    result.messages
+                        .filter(message => message.ruleId !== null) // Ignore messages with null ruleId
+                        .map(message => ({
+                            url: entry.url,
+                            rule: message.ruleId,
+                            category: 'security',
+                            severity: message.severity === 1 ? "warning" : message.severity === 2 ? "error" : message.severity,
+                            text: message.message,
+                            line: message.line,
+                            column: message.column
+                        }))
+                );
+            }),
+            this.eslintStandard.lintText(entry.content).then(results => {
+                return results.flatMap(result =>
+                    result.messages
+                        .filter(message => message.ruleId !== null) // Ignore messages with null ruleId
+                        .map(message => ({
+                            url: entry.url,
+                            rule: message.ruleId,
+                            category: 'standard',
+                            severity: message.severity === 1 ? "warning" : message.severity === 2 ? "error" : message.severity,
+                            text: message.message,
+                            line: message.line,
+                            column: message.column
+                        }))
+                );
+            })
+        ]);
+
+        // Wait for all linting promises to resolve and flatten the results
+        const lintResults = await Promise.all(lintPromises);
+        knowledgeData.issues = lintResults.flat();
+
 
         return knowledgeData;
     }