fix: prevent race condition during GraphQL scan when using skipLocalScan with extends

Author: productdevbook

src/nitro/setup/scanner.ts

@@ -124,6 +124,14 @@ export async function performGraphQLScan(nitro: Nitro, options: ScanOptions = {}
   const scanLocal = shouldScanLocalFiles(nitro)
   const extendSources = getExtendSources(nitro)
 
+  // Skip rescan when using skipLocalScan with extends
+  // The dev:start hook triggers rescan but Nitro doesn't await async hooks.
+  // This causes a race condition where tests/code run before rescan completes.
+  // Since extend packages don't change during dev, skipping rescan is correct.
+  if (isRescan && !scanLocal && extendSources?.length) {
+    return
+  }
+
   // Step 1: Handle skipLocalScan mode
   if (!scanLocal) {
     if (!isRescan && !silent) {

tests/e2e/extend-schema-merge.test.ts

@@ -46,10 +46,6 @@ describe('extend Schema Merge E2E', () => {
     await prepare(nitro)
     await build(nitro)
 
-    // BUG: After build(), the dev:start hook triggers performGraphQLScan
-    // which clears and re-populates scanSchemas/scanResolvers.
-    // During rescan with skipLocalScan:true, extend packages are NOT fully loaded.
-
     devServer = createDevServer(nitro)
     const server = await devServer.listen({ port: 0 })
     const url = server.url || `http://localhost:${(server as unknown as { port: number }).port}`