Merge pull request #130 from nksaraf/directives-fix

Add onModuleFound callback to shimExportsPlugin

.changeset/thin-windows-glow.md

@@ -0,0 +1,6 @@
+---
+"@vinxi/plugin-directives": patch
+"vinxi": patch
+---
+
+fix: @vinxi/plugin-directives: Add onModuleFound callback to shimExportsPlugin, fixes #129, solidjs/solid-start#1261

packages/vinxi-directives/plugins/shim-exports.js

@@ -297,6 +297,7 @@ export function shimExportsPlugin({
 								],
 							),
 						);
+						onModuleFound?.(id);
 						this.traverse(path);
 					}
 					return false;
@@ -312,7 +313,9 @@ export function shimExportsPlugin({
 				});
 
 				return {
-					code: `import { ${runtime.function} } from '${runtime.module}';\n` + result.code,
+					code:
+						`import { ${runtime.function} } from '${runtime.module}';\n` +
+						result.code,
 					map: result.map,
 				};
 			}

packages/vinxi-directives/transform.test.js

@@ -1,5 +1,5 @@
 import { prettyPrint } from "recast";
-import { describe, expect, it } from "vitest";
+import { describe, expect, it, vi } from "vitest";
 
 import { parseAdvanced } from "./parse.js";
 import {
@@ -36,12 +36,15 @@ async function transformSSR(
 		pragma: "use runtime",
 	},
 ) {
+	const onModuleFound = vi.fn();
 	const instance = plugin({
 		...args,
+		onModuleFound,
 	});
 
 	let result = await instance.transform(code, args.id, args.options);
 	const data = js(result);
+	expect(onModuleFound.mock.calls.length).toBeGreaterThan(0);
 	return data;
 }
 async function transformClient(