@noctcore/eslint-plugin-async-safety 0.2.0 → 0.3.0

package/README.md

@@ -55,9 +55,11 @@ export default [
 | `require-fetch-timeout` | `error` | Precise and syntactic. |
 | `require-client-timeout` | `error` | Inert until you list `clients`, so it ships enabled but checks nothing by default. |
 | `no-shared-mutable-module-state` | `error` | Inert until you set `include` globs, so it ships enabled but off by default. |
-| `forward-abort-signal` | `warn` | Heuristic — advisory. |
-| `prefer-parallel-awaits` | `warn` | Heuristic — advisory suggestion. |
-| `no-concurrent-shared-mutation` | `warn` | Heuristic — advisory. |
+| `forward-abort-signal` | `error` | A dead `signal` is a real bug; any forwarding shape counts as a pass. |
+| `no-concurrent-shared-mutation` | `error` | A lost update is a real bug; order-tolerant writes are skipped. |
+| `prefer-parallel-awaits` | `off` | A latency hint, not a bug. Sequential awaits are often deliberate. Opt in where you want it. |
+
+Every rule is `error` or `off`, never `warn`: a warning is a rule nobody obeys.
 
 The 💡 rules provide editor suggestions (not autofixes) — parallelizing awaits and adding a timeout both change
 runtime behavior, so they are never applied automatically.

package/dist/index.cjs

@@ -34,10 +34,16 @@ var recommended = {
   "noctcore-async-safety/require-client-timeout": "error",
   // Inert until you set `include` globs, so it ships enabled but off by default.
   "noctcore-async-safety/no-shared-mutable-module-state": "error",
-  // Heuristic — advisory. Warns rather than blocking.
-  "noctcore-async-safety/forward-abort-signal": "warn",
-  "noctcore-async-safety/prefer-parallel-awaits": "warn",
-  "noctcore-async-safety/no-concurrent-shared-mutation": "warn"
+  // A dead `signal` parameter is a real bug (the cancel never reaches the I/O), and
+  // the rule counts any forwarding shape as a pass, so it errs toward silence.
+  "noctcore-async-safety/forward-abort-signal": "error",
+  // A lost update is a real bug, and the rule skips order-tolerant writes
+  // (`push`, `set`, distinct-index) and plain overwrites.
+  "noctcore-async-safety/no-concurrent-shared-mutation": "error",
+  // Ships OFF: a latency hint, not a correctness bug. Sequential awaits are often
+  // deliberate (one transaction client, rate limits, deterministic test setup), and
+  // the rule cannot see that. Enable it where you want the nudge.
+  "noctcore-async-safety/prefer-parallel-awaits": "off"
 };
 
 // src/rules/forward-abort-signal.ts

package/dist/index.js

@@ -6,10 +6,16 @@ var recommended = {
   "noctcore-async-safety/require-client-timeout": "error",
   // Inert until you set `include` globs, so it ships enabled but off by default.
   "noctcore-async-safety/no-shared-mutable-module-state": "error",
-  // Heuristic — advisory. Warns rather than blocking.
-  "noctcore-async-safety/forward-abort-signal": "warn",
-  "noctcore-async-safety/prefer-parallel-awaits": "warn",
-  "noctcore-async-safety/no-concurrent-shared-mutation": "warn"
+  // A dead `signal` parameter is a real bug (the cancel never reaches the I/O), and
+  // the rule counts any forwarding shape as a pass, so it errs toward silence.
+  "noctcore-async-safety/forward-abort-signal": "error",
+  // A lost update is a real bug, and the rule skips order-tolerant writes
+  // (`push`, `set`, distinct-index) and plain overwrites.
+  "noctcore-async-safety/no-concurrent-shared-mutation": "error",
+  // Ships OFF: a latency hint, not a correctness bug. Sequential awaits are often
+  // deliberate (one transaction client, rate limits, deterministic test setup), and
+  // the rule cannot see that. Enable it where you want the nudge.
+  "noctcore-async-safety/prefer-parallel-awaits": "off"
 };
 
 // src/rules/forward-abort-signal.ts

package/docs/rules/no-concurrent-shared-mutation.md

@@ -45,4 +45,4 @@ None.
 ## When not to use it
 
 If you intentionally accumulate into shared state and have externally serialized the callbacks (e.g. a mutex, or
-a concurrency limit of 1), this rule's warning is a false positive — disable it inline for that block.
+a concurrency limit of 1), this rule's report is a false positive: disable it inline for that block.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@noctcore/eslint-plugin-async-safety",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "ESLint rules for async correctness: fetch timeouts, AbortSignal forwarding, and shared-state / concurrency races.",
   "license": "MIT",
   "type": "module",