@playfast/reform-proof-vitest 0.0.5 → 0.1.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@playfast/reform-proof-vitest",
   "playbook": "./playbook",
-  "version": "0.0.5",
+  "version": "0.1.0",
   "type": "module",
   "description": "Vitest adapter for reform-proof — a vite plugin that auto-registers every *.proof.ts suite as native vitest tests, run through the proof engine.",
   "keywords": [

package/src/index.ts

@@ -1,5 +1,5 @@
 import { isProofSuite, type ProofSuite, withProofRunner } from '@playfast/reform-proof'
-import { describe, test } from 'vitest'
+import { describe, expect, test } from 'vitest'
 
 // reform-proof vitest adapter — turns a reform proof `ProofSuite` into native
 // vitest tests (a `describe` per product, a `test` per requirement), each run
@@ -30,21 +30,19 @@ export const setProofTestHook = (hook: () => void): void => {
 }
 
 const runSetupHooks = (): void => {
-  for (const hook of setupHooks) hook()
+  setupHooks.forEach((hook) => hook())
 }
 
 const registerSuite = (suite: ProofSuite): void => {
   describe(suite.product.manifest.name, () => {
-    for (const proof of suite.proofs) {
+    suite.proofs.forEach((proof) => {
       const statement = proof.requirement.manifest.statement
       test(statement, async () => {
         runSetupHooks()
-        const result = await withProofRunner((runner) => runner.executeProof(proof))
-        if (!result.ok) {
-          throw new Error(`reform-proof: ${statement} — ${result.error ?? 'failed'}`)
-        }
+        const outcome = await withProofRunner((runner) => runner.executeProof(proof))
+        expect(outcome.ok, `reform-proof: ${statement} — ${outcome.error ?? 'failed'}`).toBe(true)
       })
-    }
+    })
   })
 }
 
@@ -53,10 +51,12 @@ const registerSuite = (suite: ProofSuite): void => {
  * values are ignored, so a whole module's exports can be passed verbatim — which
  * is exactly what `reformProofPlugin` does.
  */
-export const registerSuites = (values: ReadonlyArray<unknown>): void => {
-  for (const value of values) {
-    if (!isProofSuite(value) || registered.has(value)) continue
-    registered.add(value)
-    registerSuite(value)
-  }
+export const registerSuites = (candidates: ReadonlyArray<unknown>): void => {
+  candidates.forEach((candidate) => {
+    if (!isProofSuite(candidate) || registered.has(candidate)) {
+      return
+    }
+    registered.add(candidate)
+    registerSuite(candidate)
+  })
 }

package/src/plugin.ts

@@ -9,8 +9,8 @@ import { init, parse } from 'es-module-lexer'
 
 const PROOF_RE = /\.proof\.ts$/
 
-/** Options for `reformProofPlugin`. */
-export interface ReformProofPluginOptions {
+/** Options for `reformProofPlugin`. ExternalApi: vite-plugin config boundary. */
+export interface ReformProofPluginOptionsExternalApi {
   /**
    * Module specifier(s) the plugin side-effect-imports into every `*.proof.ts`,
    * before registering its suites. Use to register per-proof-test setup
@@ -23,7 +23,7 @@ export interface ReformProofPluginOptions {
 
 /** The minimal vite plugin shape this adapter needs — structurally a `Plugin`,
  *  declared locally so the package need not depend on vite's types. */
-export interface ReformProofPlugin {
+export interface ReformProofPluginExternalApi {
   readonly name: string
   readonly enforce: 'post'
   transform(
@@ -32,16 +32,20 @@ export interface ReformProofPlugin {
   ): Promise<{ readonly code: string; readonly map: null } | null>
 }
 
-const sourcePath = (id: string): string => id.split('?')[0] ?? id
+const sourcePath = (id: string): string => id.replace(/\?.*$/, '')
 
-const setupList = (setup: ReformProofPluginOptions['setup']): ReadonlyArray<string> =>
+const setupList = (
+  setup: ReformProofPluginOptionsExternalApi['setup'],
+): ReadonlyArray<string> =>
   setup === undefined ? [] : typeof setup === 'string' ? [setup] : setup
 
 /**
  * The plugin. Runs `post` (after TS→JS), so it lexes plain JS exports reliably.
  * Only `*.proof.ts` modules are touched; everything else passes through.
  */
-export const reformProofPlugin = (options?: ReformProofPluginOptions): ReformProofPlugin => {
+export const reformProofPlugin = (
+  options?: ReformProofPluginOptionsExternalApi,
+): ReformProofPluginExternalApi => {
   const setupImports = setupList(options?.setup)
     .map((spec) => `import '${spec}'`)
     .join('\n')
@@ -49,7 +53,9 @@ export const reformProofPlugin = (options?: ReformProofPluginOptions): ReformPro
     name: 'reform-proof-vitest',
     enforce: 'post',
     async transform(code, id) {
-      if (!PROOF_RE.test(sourcePath(id))) return null
+      if (!PROOF_RE.test(sourcePath(id))) {
+        return null
+      }
       await init
       const [, exports] = parse(code)
       // Reference each export's LOCAL binding (in scope after the module body); a
@@ -57,7 +63,9 @@ export const reformProofPlugin = (options?: ReformProofPluginOptions): ReformPro
       const names = exports
         .map((exportSpec) => exportSpec.ln)
         .filter((local): local is string => typeof local === 'string' && local.length > 0)
-      if (names.length === 0) return null
+      if (names.length === 0) {
+        return null
+      }
       const epilogue =
         `\n${setupImports}\n` +
         `import { registerSuites as __registerReformProofSuites } from '@playfast/reform-proof-vitest'\n` +