fix: postcss and emit failures in CI (#1577)

.changeset/beige-geese-impress.md

@@ -0,0 +1,7 @@
+---
+'@pandacss/postcss': patch
+'@pandacss/node': patch
+---
+
+Fix persistent error that causes CI builds to fail due to PostCSS plugin emitting artifacts in the middle of a build
+process.

packages/node/src/builder.ts

@@ -33,14 +33,14 @@ const contentFilesCache = new WeakMap<PandaContext, ContentData>()
 
 const limit = pLimit(20)
 
-let setupCount = 0
-
 export class Builder {
   /**
    * The current panda context
    */
   context: PandaContext | undefined
 
+  hasEmitted = false
+
   configDependencies: Set<string> = new Set()
 
   writeFileCss = (file: string, css: string) => {
@@ -75,10 +75,6 @@ export class Builder {
       delete require.cache[file]
     }
 
-    if (setupCount > 0) {
-      logger.debug('builder', '⚙️ Config changed, reloading')
-    }
-
     return { isModified: true, modifiedMap: newModified }
   }
 
@@ -92,6 +88,8 @@ export class Builder {
     return configPath
   }
 
+  hasConfigChanged = false
+
   setup = async (options: { configPath?: string; cwd?: string } = {}) => {
     logger.debug('builder', '🚧 Setup')
 
@@ -106,13 +104,17 @@ export class Builder {
     this.configDependencies = configDeps
 
     const deps = this.checkConfigDeps(configPath, configDeps)
+    this.hasConfigChanged = deps.isModified
 
     if (deps.isModified) {
       await this.setupContext({
         configPath,
         depsModifiedMap: deps.modifiedMap,
       })
-      const ctx = this.context!
+
+      const ctx = this.getContextOrThrow()
+
+      logger.debug('builder', '⚙️ Config changed, reloading')
       await ctx.hooks.callHook('config:change', ctx.config)
     }
 
@@ -129,20 +131,22 @@ export class Builder {
         depsModifiedMap: deps.modifiedMap,
       })
     }
+  }
 
-    setupCount++
+  emit() {
+    // ensure emit is only called when the config is changed
+    if (this.hasEmitted && this.hasConfigChanged) {
+      emitArtifacts(this.getContextOrThrow())
+    }
+
+    this.hasEmitted = true
   }
 
   setupContext = async (options: { configPath: string; depsModifiedMap: Map<string, number> }) => {
     const { configPath, depsModifiedMap } = options
 
     this.context = await loadConfigAndCreateContext({ configPath })
 
-    // don't emit artifacts on first setup
-    if (setupCount > 0) {
-      emitArtifacts(this.context) // no need to await this
-    }
-
     configCache.set(configPath, {
       context: this.context,
       deps: new Set(this.context.dependencies ?? []),

packages/postcss/src/index.ts

@@ -25,6 +25,8 @@ export const pandacss: PluginCreator<{ configPath?: string; cwd?: string }> = (o
           return
         }
 
+        builder.emit()
+
         await builder.extract()
 
         builder.registerDependency((dep) => {