fix: handle FunctionExpression also

Author: baseballyama

src/rules/no-store-async.ts

@@ -21,7 +21,12 @@ export default createRule("no-store-async", {
       Program() {
         for (const { node } of extractStoreReferences(context)) {
           const [, fn] = node.arguments
-          if (!fn || fn.type !== "ArrowFunctionExpression" || !fn.async) {
+          if (
+            !fn ||
+            (fn.type !== "ArrowFunctionExpression" &&
+              fn.type !== "FunctionExpression") ||
+            !fn.async
+          ) {
             continue
           }
 

tests/fixtures/rules/no-store-async/invalid/test04-errors.yaml

@@ -0,0 +1,12 @@
+- message: Do not pass async functions to svelte stores.
+  line: 3
+  column: 28
+  suggestions: null
+- message: Do not pass async functions to svelte stores.
+  line: 6
+  column: 28
+  suggestions: null
+- message: Do not pass async functions to svelte stores.
+  line: 9
+  column: 24
+  suggestions: null

tests/fixtures/rules/no-store-async/invalid/test04-input.js

@@ -0,0 +1,11 @@
+import { writable, readable, derived } from "svelte/store"
+
+const w2 = writable(false, async function () {
+  /** do nothing */
+})
+const r2 = readable(false, async function () {
+  /** do nothing */
+})
+const d2 = derived(a1, async function ($a1) {
+  /** do nothing */
+})