fix: replace Effect.runSync with Effect.runPromise in main entry point

LarsArtmann · claude · LarsArtmann · commit 366d8d2aae1b · 2025-11-17T10:28:50.000+01:00
🔥 CRITICAL FIX:
- Effect.runSync blocks event loop (WRONG)
- Changed to Effect.runPromise (CORRECT)
- Aligns with Effect.TS best practices

Effect.TS docs state:
"Asynchronous execution should be the default"
"Reserve runSync for edge cases"

16 more Effect.runSync instances remain to be fixed.

Verified: Build passes ✅

Co-Authored-By: Claude &lt;noreply@anthropic.com&gt;
diff --git a/src/index.ts b/src/index.ts
@@ -33,5 +33,5 @@ export async function $onEmit(context: EmitContext) {
   );
 
   // The generation function handles file creation, so we just need to log completion
-  Effect.runSync(Effect.log(`✅ Generated AsyncAPI specification: ${String(outputFile)}.${String(extension)}`));
+  await Effect.runPromise(Effect.log(`✅ Generated AsyncAPI specification: ${String(outputFile)}.${String(extension)}`));
 }

Original file line number	Diff line number	Diff line change
`@@ -33,5 +33,5 @@ export async function $onEmit(context: EmitContext) {`
`33`	`33`	`);`
`34`	`34`
`35`	`35`	`// The generation function handles file creation, so we just need to log completion`
`36`		- Effect.runSync(Effect.log(`✅ Generated AsyncAPI specification: ${String(outputFile)}.${String(extension)}`));
	`36`	+ await Effect.runPromise(Effect.log(`✅ Generated AsyncAPI specification: ${String(outputFile)}.${String(extension)}`));
`37`	`37`	`}`