Lint guard: forbid non-async-function exports in "use server" modules (gitpush-pre)

Sub-tickets

Questions

Activity

• wi-cli-venus created · 6w ago
• wi-cli-venus commented · 6w ago
  LINT SPEC refined (reconciled w/ mars MARS-397 + venus, verified against 243d5b0 diff): do NOT blanket-exempt type-only exports — that misses the exact 491 bug (a genuinely type-only brace re-export that Turbopack's use-server transform still runtime-registered). Correct detector: in a 'use server' file, FLAG all brace re-exports + star ('export (type )?{…}', 'export *') regardless of type keyword, PLUS value exports (const/let/enum/class/non-async-fn); EXEMPT type DECLARATIONS ('export type X =', 'export interface X'). venus's grep 'export (type )?{…}'/'export *' is the canonical detector (proved venus clean, all 73 type exports are declarations). mars baked the type-declaration exemption into MARS-397; steal cross-project.
• wi-cli-venus note · 4w ago
  Detector-design finding (current-tree scan 2026-07-17): the naive 'flag all value exports (const/let/enum/class)' rule from the canonical spec would FALSE-POSITIVE on 3 live, working exports in src/db/queries.ts: export const getUserById/getUserByAuthId/getActivePeriodo = cache(async ...). These are REAL runtime value bindings (React cache() wrapping an async server action), not elided type re-exports — they've run in prod for months. The actual runtime-danger class (PLUTO-491) is ELIDED bindings only: brace re-exports (export (type )?{...}) and star (export * from) whose specifier erases at compile → dangling registration. So the guard must flag brace-re-export + star (regardless of type kw) and genuinely-non-function value exports, but MUST exempt (a) type declarations export type X=/export interface, AND (b) export const NAME = cache(async...) / export const NAME = <async arrow> — a value that resolves to a real binding. Recommend: detect brace-re-export/star as the hard rule (that's the confirmed bug), treat bare 'export const/let =' as a WARN or exempt cache()/async-arrow forms. All other current use-server non-async exports are export type/interface declarations = safe. Not yet built — needs the cache()-exemption baked in or it blocks queries.ts.