@peerigon/configs 15.1.0 → 15.3.0

package/ai/coding-styleguide-js-ts.mdc

@@ -81,6 +81,7 @@ globs: **/*.js, **/*.ts, **/*.mjs, **/*.cjs, **/*.mts, **/*.cts
 
 - Constructor **must** be at the top of the class
 - Then properties: public first, then private; order by importance in relation to the class purpose
+- Public class properties should be `readonly` when the value is not reassigned after construction or is only changed within the class. If the value needs internal mutation but should stay stable to consumers, expose it via a getter backed by a private property.
 - Then methods: public first, then private; order by importance in relation to the class purpose
 
 ## Types

package/dist/eslint/rules/javascript.js

@@ -145,20 +145,18 @@ export const javascript = [
             "no-only-tests": noOnlyTests,
         },
         rules: {
-            // Top-level await might slow down the test suite start up
-            "unicorn/prefer-top-level-await": "off",
-            // Nesting is pretty common in tests when you group tests
-            "max-nested-callbacks": "off",
-            // await in loops is pretty common in Playwright tests
-            "no-await-in-loop": "off",
-            // Unreachable loops are sometimes used in tests to assert loop body runs at most once
-            "no-unreachable-loop": "off",
-            // In case you want to test errors thrown by a constructor
-            "no-new": "off",
-            // Generators without yield are common in test helpers/setup
-            "require-yield": "off",
-            // should show a warning when a test is focussed (overridden to off when using vitest config)
-            "no-only-tests/no-only-tests": "error",
+            camelcase: "off", // Mock payloads often mirror external casing conventions.
+            "max-depth": "off", // Nested describe blocks are often the clearest structure.
+            "max-nested-callbacks": "off", // Nested test callbacks are idiomatic in test frameworks.
+            "no-await-in-loop": "off", // Await in loops is pretty common in Playwright tests.
+            "no-new": "off", // Constructor side effects are often asserted directly in tests.
+            "no-only-tests/no-only-tests": "error", // Should show a warning when a test is focussed (overridden to off when using vitest config).
+            "no-constant-condition": "off", // Constants are often used in tests to assert loop body runs at most once.
+            "no-unreachable-loop": "off", // Unreachable loops are sometimes used in tests to assert loop body runs at most once.
+            "no-unused-expressions": "off", // Assertion styles (e.g. chai) use expression statements.
+            "prefer-arrow-callback": "off", // Function callbacks are needed for this-binding tests.
+            "require-yield": "off", // Generators without yield are common in test helpers/setup.
+            "unicorn/prefer-top-level-await": "off", // Top-level await might slow down the test suite start up.
         },
     },
 ];

package/dist/eslint/rules/react.js

@@ -146,5 +146,13 @@ export const react = [
             "@eslint-react/no-leaked-conditional-rendering": "warn", // https://eslint-react.xyz/docs/rules/no-leaked-conditional-rendering
         },
     },
+    {
+        files: globPatterns.tests,
+        rules: {
+            "react-hooks/exhaustive-deps": "off", // Effect dependency permutations are often intentionally incomplete in tests.
+            "react-refresh/only-export-components": "off", // Test files export helpers/constants alongside components.
+            "react/display-name": "off", // Anonymous inline components are common in tests.
+        },
+    },
 ];
 export default react;

package/dist/eslint/rules/typescript.js

@@ -185,6 +185,7 @@ export const typescript =
         // We allow any to be used in tests, so returning it is ok
         "@typescript-eslint/no-unsafe-return": "off", // https://typescript-eslint.io/rules/no-unsafe-return
         // chai uses these as assertions
+        "@typescript-eslint/no-unnecessary-condition": "off", // https://typescript-eslint.io/rules/no-unnecessary-condition
         "@typescript-eslint/no-unused-expressions": "off", // https://typescript-eslint.io/rules/no-unused-expressions
         // It's uncommon to use async/await in Cypress tests
         // https://docs.cypress.io/guides/core-concepts/introduction-to-cypress.html#Commands-Are-Asynchronous

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@peerigon/configs",
-  "version": "15.1.0",
+  "version": "15.3.0",
   "description": "Configs for ESLint, Prettier, TypeScript & friends",
   "keywords": [
     "eslint",
@@ -84,26 +84,26 @@
     "@prettier/plugin-oxc": "^0.1.3",
     "@sebbo2002/semantic-release-jsr": "^3.2.1",
     "@semantic-release/exec": "^7.1.0",
-    "@tanstack/eslint-plugin-query": "^5.91.4",
+    "@tanstack/eslint-plugin-query": "^5.95.2",
     "@types/eslint-config-prettier": "^6.11.3",
     "@types/eslint-plugin-jsx-a11y": "^6.10.1",
-    "@vitest/eslint-plugin": "^1.6.12",
+    "@vitest/eslint-plugin": "^1.6.13",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-jsx-a11y": "^6.10.2",
     "eslint-plugin-no-only-tests": "^3.3.0",
-    "eslint-plugin-playwright": "^2.10.0",
+    "eslint-plugin-playwright": "^2.10.1",
     "eslint-plugin-prefer-arrow": "^1.2.3",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^7.0.1",
     "eslint-plugin-react-refresh": "^0.5.2",
-    "eslint-plugin-react-you-might-not-need-an-effect": "^0.9.1",
+    "eslint-plugin-react-you-might-not-need-an-effect": "^0.9.2",
     "eslint-plugin-unicorn": "^63.0.0",
     "globals": "^17.4.0",
     "prettier-plugin-css-order": "^2.2.0",
     "prettier-plugin-jsdoc": "^1.8.0",
     "prettier-plugin-packagejson": "^3.0.2",
     "prettier-plugin-tailwindcss": "^0.7.2",
-    "typescript-eslint": "^8.57.0"
+    "typescript-eslint": "^8.57.2"
   },
   "devDependencies": {
     "@types/jest": "^30.0.0",