@forsakringskassan/eslint-config 13.0.2 → 13.0.3

package/index.mjs

@@ -11,6 +11,7 @@ export { default as globals } from "globals";
 
 /**
  * @typedef {import("eslint").Linter.Config} Config
+ * @typedef {import("eslint").Linter.RulesRecord} RulesRecord
  */
 
 /**
@@ -36,6 +37,17 @@ function merge(result, it) {
     };
 }
 
+/**
+ * @param {RulesRecord} rules
+ * @param {(rule: string) => boolean} predicate
+ * @returns {RulesRecord}
+ */
+function filterRules(rules, predicate) {
+    return Object.fromEntries(
+        Object.entries(rules).filter(([key]) => predicate(key)),
+    );
+}
+
 export default [
     defineConfig({
         ...js.configs.recommended,
@@ -59,7 +71,27 @@ export default [
             },
         },
         rules: {
-            ...prettierConfig.rules,
+            ...filterRules(prettierConfig.rules, (rule) => {
+                if (rule.startsWith("@stylistic/")) {
+                    return false;
+                }
+                if (rule.startsWith("@babel/") || rule.startsWith("babel/")) {
+                    return false;
+                }
+                if (rule.startsWith("flowtype/")) {
+                    return false;
+                }
+                if (rule.startsWith("react/")) {
+                    return false;
+                }
+                if (rule.startsWith("standard/")) {
+                    return false;
+                }
+                if (rule.startsWith("unicorn/")) {
+                    return false;
+                }
+                return true;
+            }),
             ...prettierPlugin.configs.recommended.rules,
             ...importPlugin.configs.errors.rules,
             ...eslintCommentsPlugin.configs.recommended.rules,
@@ -125,8 +157,10 @@ export default [
             "sonarjs/no-duplicate-string": "off",
 
             "sonarjs/argument-type": "off", // handled by typescript (and this rule is sometimes wrong)
+            "sonarjs/arguments-order": "off", // another slow rule, would be nice to have enabled thought
             "sonarjs/deprecation": "off", // covered by @typescript-eslint/no-deprecated (and this rule crashes on .svelte files)
             "sonarjs/function-return-type": "off", // overly broad and opinionated, let typescript deal with this
+            "sonarjs/no-commented-code": "off", // neat rule but is very very slow (over 50% of the total linting time)
             "sonarjs/no-control-regex": "off", // covered by no-control-regexp
             "sonarjs/no-empty-test-file": "off", // could be useful but it does not handle it.each or similar constructs thus yields more false positives than its worth */
             "sonarjs/no-selector-parameter": "off", // not always possible (e.g. watcher handler in vue)
@@ -141,6 +175,27 @@ export default [
             "sonarjs/unused-named-groups": "off", // named groups can help readability even if not used
             "sonarjs/use-type-alias": "off", // overly broad, lets leave this to the discretion of the author
 
+            /* disable sonarjs aws related rules as we dont use aws */
+            "sonarjs/aws-apigateway-public-api": "off",
+            "sonarjs/aws-ec2-rds-dms-public": "off",
+            "sonarjs/aws-ec2-unencrypted-ebs-volume": "off",
+            "sonarjs/aws-efs-unencrypted": "off",
+            "sonarjs/aws-iam-all-privileges": "off",
+            "sonarjs/aws-iam-all-resources-accessible": "off",
+            "sonarjs/aws-iam-privilege-escalation": "off",
+            "sonarjs/aws-iam-public-access": "off",
+            "sonarjs/aws-opensearchservice-domain": "off",
+            "sonarjs/aws-rds-unencrypted-databases": "off",
+            "sonarjs/aws-restricted-ip-admin-access": "off",
+            "sonarjs/aws-s3-bucket-granted-access": "off",
+            "sonarjs/aws-s3-bucket-insecure-http": "off",
+            "sonarjs/aws-s3-bucket-public-access": "off",
+            "sonarjs/aws-s3-bucket-server-encryption": "off",
+            "sonarjs/aws-s3-bucket-versioning": "off",
+            "sonarjs/aws-sagemaker-unencrypted-notebook": "off",
+            "sonarjs/aws-sns-unencrypted-topics": "off",
+            "sonarjs/aws-sqs-unencrypted-queue": "off",
+
             /* Lower some errors to warnings, these are allowed on local builds (to
              * not prevent builds during development where code is unfinished and
              * might contain debugging code) but is disallowed when building from
@@ -148,7 +203,6 @@ export default [
             "no-console": "warn",
             "no-debugger": "warn",
             "prettier/prettier": "warn",
-            "sonarjs/no-commented-code": "warn",
 
             "import/default": "off",
             "import/extensions": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forsakringskassan/eslint-config",
-  "version": "13.0.2",
+  "version": "13.0.3",
   "description": "Försäkringskassans eslint shareable config",
   "keywords": [
     "eslint"
@@ -41,5 +41,5 @@
   "engines": {
     "node": ">= 22.0"
   },
-  "gitHead": "99ad23a49dac0b99b44be7190639b0dfb5d27355"
+  "gitHead": "23765c7782b10249c9c2e7c42fe7448e0afddce6"
 }