@dfinity/eslint-config-oisy-wallet 0.1.14 → 0.2.1

package/README.md

@@ -84,9 +84,9 @@ module.exports = require("@dfinity/eslint-config-oisy-wallet/eslint-local-rules"
 > [!NOTE]
 > This is necessary because the `eslint-plugin-local-rules` plugin we use for custom rules requires a file located at the root and does not offer any customizable location option.
 
-## 🔧 Overriding or Disabling Rules
+## 🔧 Overriding, Enabling, or Disabling Rules
 
-You can override or disable any of the rules provided by this configuration — including custom **local rules** — just like you would with any ESLint config.
+You can override, enable, or disable any of the rules provided by this configuration — including custom **local rules** — just like you would with any ESLint config.
 
 In your `.eslintrc.js`, simply add a `rules` section:
 
@@ -100,6 +100,9 @@ module.exports = {
     // Disable a local custom rule
     "local/use-nullish-checks": "off",
 
+    // Enable a built-in rule
+    "local-rules/prefer-object-params": "warn",
+
     // Customize severity or options
     "@typescript-eslint/no-unused-vars": ["warn", { argsIgnorePattern: "^_" }],
   },
@@ -124,6 +127,9 @@ export default [
       // Disable a local custom rule
       "local/use-nullish-checks": "off",
 
+      // Enable a built-in rule
+      "local-rules/prefer-object-params": "warn",
+
       // Customize severity or options
       "@typescript-eslint/no-unused-vars": [
         "warn",

package/configs/eslint.rules.mjs

@@ -53,12 +53,6 @@ export const eslintRules = [
 
       "local-rules/prefer-object-params": "warn",
 
-      // TODO: Disabled for now to allow the migration of ESLint v9 in OISY, as the rule is not yet enforced.
-      // It should also be introduced as an optional rule.
-      // "local-rules/use-nullish-checks": "warn",
-
-      "local-rules/use-option-type-wrapper": "warn",
-
       "import/no-duplicates": ["error", { "prefer-inline": true }],
       "import/no-relative-parent-imports": "error",
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dfinity/eslint-config-oisy-wallet",
-  "version": "0.1.14",
+  "version": "0.2.1",
   "license": "Apache-2.0",
   "description": "Shared ESLint configurations from the Oisy Wallet team",
   "repository": {
@@ -64,23 +64,23 @@
   ],
   "peerDependencies": {
     "@dfinity/utils": "*",
-    "@eslint/compat": "^1.2.6",
-    "@eslint/eslintrc": "^3.2.0",
-    "@eslint/js": "^9.20.0",
-    "@typescript-eslint/eslint-plugin": "^8.24.1",
-    "@vitest/eslint-plugin": "^1.1.39",
-    "eslint": "^9.20.1",
-    "eslint-config-love": "^118.0.0",
-    "eslint-config-prettier": "^10.0.1",
-    "eslint-plugin-import": "^2.31.0",
+    "@eslint/compat": "^1.3.1",
+    "@eslint/eslintrc": "^3.3.1",
+    "@eslint/js": "^9.30.1",
+    "@typescript-eslint/eslint-plugin": "^8.36.0",
+    "@vitest/eslint-plugin": "^1.3.4",
+    "eslint": "^9.30.1",
+    "eslint-config-love": "^121.0.0",
+    "eslint-config-prettier": "^10.1.5",
+    "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-local-rules": "^3.0.2",
-    "eslint-plugin-n": "^17.15.1",
+    "eslint-plugin-n": "^17.21.0",
     "eslint-plugin-prefer-arrow": "^1.2.3",
-    "eslint-plugin-prettier": "^5.2.3",
+    "eslint-plugin-prettier": "^5.5.1",
     "eslint-plugin-promise": "^7.2.1",
     "eslint-plugin-svelte": "^3",
-    "globals": "^15.15.0",
-    "prettier": "^3.5.1",
+    "globals": "^16.3.0",
+    "prettier": "^3.6.2",
     "prettier-plugin-organize-imports": "^4.1.0",
     "typescript": "^5"
   },

package/rules/use-nullish-checks.cjs

@@ -4,7 +4,7 @@ module.exports = {
     docs: {
       description: "Enforce the use of isNullish functions for nullish checks",
       category: "Best Practices",
-      recommended: true,
+      recommended: false,
     },
     messages: {
       isNullish:

package/rules/use-option-type-wrapper.cjs

@@ -4,7 +4,7 @@ module.exports = {
     docs: {
       description: "Enforce use of Option<T> instead of T | null | undefined",
       category: "Best Practices",
-      recommended: true,
+      recommended: false,
     },
     messages: {
       useOption:

package/vitest.mjs

@@ -11,9 +11,12 @@ export default [
 
       "vitest/no-alias-methods": ["error"],
       "vitest/no-duplicate-hooks": ["error"],
+      "vitest/no-importing-vitest-globals": ["error"],
+      "vitest/no-test-prefixes": ["error"],
       "vitest/no-test-return-statement": ["error"],
       "vitest/padding-around-all": ["error"],
       "vitest/prefer-comparison-matcher": ["error"],
+      "vitest/prefer-called-once": ["error"],
       "vitest/prefer-each": ["error"],
       "vitest/prefer-equality-matcher": ["error"],
       "vitest/prefer-expect-resolves": ["error"],
@@ -26,7 +29,9 @@ export default [
       "vitest/prefer-to-contain": ["error"],
       "vitest/prefer-to-have-length": ["error"],
       "vitest/prefer-todo": ["error"],
+      "vitest/prefer-vi-mocked": ["error"],
       "vitest/require-top-level-describe": ["error"],
+      "vitest/valid-expect-in-promise": ["error"],
     },
   },
   languageOptions(),