@svelte-vitals/core 0.31.1 → 0.32.0

package/dist/index.d.ts

@@ -544,6 +544,20 @@ interface KitModuleFacts {
         name: string;
         line: number;
     }[];
+    /**
+     * `.set()`/`.update()` in a handler on an import resolving under the `$lib` server root.
+     * The call shape alone cannot tell a persistence client (`db.set(…)`) from a hand-rolled
+     * in-memory store, so the decision needs the target module — which this pure parse cannot
+     * read. `collectKitModuleFacts` resolves each one and promotes the in-memory ones into
+     * `importedStateWrites`; a consumer that ignores this field sees the pre-arbitration
+     * behaviour, i.e. every one of these exempt.
+     */
+    pendingServerStoreWrites: {
+        name: string;
+        imported: string;
+        resolved: string;
+        line: number;
+    }[];
     /** Value imports whose specifier resolves to a repo-local `.svelte.ts`/`.svelte.js` runes module (security/shared-state-import). */
     runesModuleImports: {
         source: string;

package/dist/index.js

@@ -1842,6 +1842,13 @@ function libServerRoot(aliases) {
   if (lib && lib.replacement === null) return void 0;
   return `${lib?.replacement ?? "src/lib"}/server`;
 }
+function serverRootRelativePath(spec, importerFile, aliases) {
+  const serverRoot = libServerRoot(aliases);
+  if (serverRoot === void 0) return void 0;
+  const path = resolveRepoLocalPath(spec, importerFile, aliases);
+  if (path === void 0) return void 0;
+  return path === serverRoot || path.startsWith(`${serverRoot}/`) ? path : void 0;
+}
 function isLocalStateSpecifier(spec, importerFile, aliases) {
   const serverRoot = libServerRoot(aliases);
   if (serverRoot === void 0) return false;
@@ -1849,12 +1856,32 @@ function isLocalStateSpecifier(spec, importerFile, aliases) {
   if (path === void 0) return false;
   return path !== serverRoot && !path.startsWith(`${serverRoot}/`);
 }
+var IN_MEMORY_CTORS = /* @__PURE__ */ new Set(["Map", "Set", "WeakMap", "WeakSet"]);
+function isInMemoryInit(init) {
+  if (!init) return false;
+  if (init.type === "ObjectExpression" || init.type === "ArrayExpression") return true;
+  return init.type === "NewExpression" && init.callee?.type === "Identifier" && IN_MEMORY_CTORS.has(init.callee.name);
+}
+function parseInMemoryExports(source, filename) {
+  const names = /* @__PURE__ */ new Set();
+  const { program } = parseModuleProgram(source, filename);
+  for (const stmt of program?.body ?? []) {
+    if (stmt?.type !== "ExportNamedDeclaration" || !stmt.declaration) continue;
+    const decl = stmt.declaration;
+    if (decl.type !== "VariableDeclaration") continue;
+    for (const d of decl.declarations ?? []) {
+      if (d?.id?.type === "Identifier" && isInMemoryInit(d.init)) names.add(d.id.name);
+    }
+  }
+  return names;
+}
 function parseKitModuleFacts(source, filename, aliases) {
   const suppressions = collectSuppressions(source);
   const { program, wrapped } = parseModuleProgram(source, filename);
   const moduleStateReassignments = [];
   const importedStateWrites = [];
   const importedStateWritesOutsideHandlers = [];
+  const pendingServerStoreWrites = [];
   const runesModuleImports = [];
   const lifecycleCalls = [];
   const browserGlobalRefs = [];
@@ -1863,6 +1890,7 @@ function parseKitModuleFacts(source, filename, aliases) {
       moduleStateReassignments,
       importedStateWrites,
       importedStateWritesOutsideHandlers,
+      pendingServerStoreWrites,
       runesModuleImports,
       lifecycleCalls,
       browserGlobalRefs,
@@ -1872,6 +1900,7 @@ function parseKitModuleFacts(source, filename, aliases) {
   }
   const line = (start) => Math.max(0, lineOf(wrapped, start) - 1);
   const importedSpecifiers = /* @__PURE__ */ new Map();
+  const importedNames = /* @__PURE__ */ new Map();
   for (const stmt of program.body ?? []) {
     if (stmt?.type !== "ImportDeclaration" || stmt.importKind === "type") continue;
     const spec = typeof stmt.source?.value === "string" ? stmt.source.value : "";
@@ -1880,6 +1909,10 @@ function parseKitModuleFacts(source, filename, aliases) {
       if (s?.importKind === "type" || s?.local?.type !== "Identifier") continue;
       names.push(s.local.name);
       importedSpecifiers.set(s.local.name, spec);
+      importedNames.set(
+        s.local.name,
+        s.type === "ImportSpecifier" && s.imported?.type === "Identifier" ? s.imported.name : s.local.name
+      );
     }
     if (names.length === 0) continue;
     const resolved = resolveRunesModuleSpecifier(spec, filename, aliases);
@@ -1956,8 +1989,21 @@ function parseKitModuleFacts(source, filename, aliases) {
       const method = n.callee.property?.type === "Identifier" ? n.callee.property.name : void 0;
       if (method === "set" || method === "update") {
         const r = importedRoot(n.callee.object);
-        if (r && isLocalStateSpecifier(importedSpecifiers.get(r), filename, aliases))
-          write = { name: r, via: "set-call" };
+        const spec = r ? importedSpecifiers.get(r) : void 0;
+        if (r && spec !== void 0) {
+          if (isLocalStateSpecifier(spec, filename, aliases)) write = { name: r, via: "set-call" };
+          else {
+            const resolved = serverRootRelativePath(spec, filename, aliases);
+            if (resolved !== void 0 && inHandler) {
+              pendingServerStoreWrites.push({
+                name: r,
+                imported: importedNames.get(r) ?? r,
+                resolved,
+                line: line(n.start)
+              });
+            }
+          }
+        }
       }
     } else if (n.type === "AssignmentExpression" && (n.left?.type === "ObjectPattern" || n.left?.type === "ArrayPattern")) {
       const scanPatternTargets = (pat) => {
@@ -2005,6 +2051,7 @@ function parseKitModuleFacts(source, filename, aliases) {
     moduleStateReassignments: byLine(moduleStateReassignments),
     importedStateWrites: byLine(importedStateWrites),
     importedStateWritesOutsideHandlers: byLine(importedStateWritesOutsideHandlers),
+    pendingServerStoreWrites: byLine(pendingServerStoreWrites),
     runesModuleImports: byLine(runesModuleImports),
     lifecycleCalls: byLine(lifecycleCalls),
     browserGlobalRefs: byLine(browserGlobalRefs),
@@ -2024,6 +2071,7 @@ function emptyKitModuleFacts(file, kind) {
     moduleStateReassignments: [],
     importedStateWrites: [],
     importedStateWritesOutsideHandlers: [],
+    pendingServerStoreWrites: [],
     runesModuleImports: [],
     lifecycleCalls: [],
     browserGlobalRefs: [],
@@ -2044,7 +2092,7 @@ async function collectKitModuleFacts(rt, cwd, aliases) {
   ];
   const lists = await Promise.all(patterns.map((p) => rt.glob(p, cwd)));
   const files = [...new Set(lists.flat())];
-  return Promise.all(
+  const facts = await Promise.all(
     files.sort().map(async (rel) => {
       const kind = kindOf(rel);
       try {
@@ -2055,6 +2103,38 @@ async function collectKitModuleFacts(rt, cwd, aliases) {
       }
     })
   );
+  return arbitrateServerStoreWrites(rt, cwd, facts);
+}
+function moduleCandidates(repoPath) {
+  if (repoPath.endsWith(".js")) return [repoPath, `${repoPath.slice(0, -3)}.ts`];
+  if (repoPath.endsWith(".ts")) return [repoPath];
+  return [`${repoPath}.ts`, `${repoPath}.js`, `${repoPath}/index.ts`, `${repoPath}/index.js`];
+}
+async function inMemoryExportsOf(rt, cwd, repoPath) {
+  for (const rel of moduleCandidates(repoPath)) {
+    try {
+      if (!await rt.exists(rt.join(cwd, rel))) continue;
+      return parseInMemoryExports(await rt.readFile(rt.join(cwd, rel)), rel);
+    } catch {
+      return /* @__PURE__ */ new Set();
+    }
+  }
+  return /* @__PURE__ */ new Set();
+}
+async function arbitrateServerStoreWrites(rt, cwd, facts) {
+  const targets = [...new Set(facts.flatMap((f) => f.pendingServerStoreWrites.map((w) => w.resolved)))];
+  if (targets.length === 0) return facts;
+  const byPath = new Map(
+    await Promise.all(targets.map(async (t) => [t, await inMemoryExportsOf(rt, cwd, t)]))
+  );
+  return facts.map((f) => {
+    const promoted = f.pendingServerStoreWrites.filter((w) => byPath.get(w.resolved)?.has(w.imported)).map((w) => ({ name: w.name, line: w.line, via: "set-call" }));
+    if (promoted.length === 0) return f;
+    return {
+      ...f,
+      importedStateWrites: [...f.importedStateWrites, ...promoted].sort((a, b) => a.line - b.line)
+    };
+  });
 }
 
 // src/config-object.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@svelte-vitals/core",
-  "version": "0.31.1",
+  "version": "0.32.0",
   "description": "Shared, runtime-agnostic core for svelte-vitals (types, rule engine, scorer, reporter).",
   "type": "module",
   "license": "MIT",