fixed: Made crucial functions async

Austin1serb · Austin1serb · commit f55ce9da568f · 2025-06-17T17:23:46.000-07:00
diff --git a/package.json b/package.json
@@ -8,6 +8,7 @@
   "packageManager": "pnpm@10.12.1",
   "scripts": {
     "preinstall": "npx only-allow pnpm",
+    "reset": "git clean -fdx && pnpm install --frozen-lockfile && pnpm prepack:core && pnpm i-tarball",
     "test": "cd packages/core && pnpm test:all",
     "prepack:core": "pnpm -F @austinserb/react-zero-ui pack --pack-destination ./dist",
     "i-tarball": "node scripts/install-local-tarball.js",
diff --git a/packages/core/__tests__/config/playwright.next.config.js b/packages/core/__tests__/config/playwright.next.config.js
@@ -38,5 +38,7 @@ export default defineConfig({
 		command: "pnpm run dev",
 		cwd: path.resolve(__dirname, "../fixtures/next"),
 		url: BASE_URL,
+		timeout: 60_000, // Give more time for CI environments
+		reuseExistingServer: !process.env.CI, // Don't reuse in CI
 	},
 });
diff --git a/packages/core/__tests__/config/playwright.vite.config.js b/packages/core/__tests__/config/playwright.vite.config.js
@@ -37,5 +37,7 @@ export default defineConfig({
 		command: "pnpm run dev",
 		cwd: path.resolve(__dirname, "../fixtures/vite"),
 		url: BASE_URL,
+		timeout: 60_000, // Give more time for CI environments
+		reuseExistingServer: !process.env.CI, // Don't reuse in CI
 	},
 });
diff --git a/packages/core/__tests__/fixtures/next/package.json b/packages/core/__tests__/fixtures/next/package.json
@@ -8,6 +8,7 @@
     "clean": "rm -rf .next node_modules package-lock.json"
   },
   "dependencies": {
+    "@austinserb/react-zero-ui": "file:/Users/austinserb/Desktop/React-Zero/react-zero-ui/dist/austinserb-react-zero-ui-1.0.18.tgz",
     "next": "^15.0.0",
     "react": "^18.2.0",
     "react-dom": "^18.2.0"
diff --git a/packages/core/__tests__/fixtures/vite/package.json b/packages/core/__tests__/fixtures/vite/package.json
@@ -11,6 +11,7 @@
     "clean": "rm -rf node_modules package-lock.json"
   },
   "dependencies": {
+    "@austinserb/react-zero-ui": "file:/Users/austinserb/Desktop/React-Zero/react-zero-ui/dist/austinserb-react-zero-ui-1.0.18.tgz",
     "react": "^19.1.0",
     "react-dom": "^19.1.0"
   },
diff --git a/packages/core/__tests__/helpers/globalSetup.next.js b/packages/core/__tests__/helpers/globalSetup.next.js
@@ -11,7 +11,7 @@ export default async function globalSetup() {
 
   // Reset and setup the Next.js fixture
   console.log('[Global Setup] Setting up Next.js fixture...');
-  resetZeroUiState(projectDir, true);
+  await resetZeroUiState(projectDir, true);
 
   const zeroUiCli = await loadCliFromFixture(projectDir);
   await zeroUiCli([]);
diff --git a/packages/core/__tests__/helpers/globalSetup.vite.js b/packages/core/__tests__/helpers/globalSetup.vite.js
@@ -10,7 +10,7 @@ export default async function globalSetup() {
 
   // Reset and setup the Vite fixture
   console.log('[Global Setup] Setting up Vite fixture...');
-  resetZeroUiState(projectDir, false);
+  await resetZeroUiState(projectDir, false);
 
   const zeroUiCli = await loadCliFromFixture(projectDir);
   await zeroUiCli([]);
diff --git a/packages/core/__tests__/unit/cli.test.cjs b/packages/core/__tests__/unit/cli.test.cjs
@@ -37,6 +37,7 @@ function cleanupTestDir(testDir) {
                 fs.unlinkSync(fullPath);
               } catch (e) {
                 // Ignore
+                console.log(`Error deleting file ${fullPath}: ${e.message}`);
               }
             }
           }
@@ -306,12 +307,12 @@ test('CLI script imports and executes library CLI', async () => {
     // Create mock CLI module that matches the actual structure
     const mockCLI = `
       // Mock implementation of the CLI
-      function runZeroUiInit() {
+      async function runZeroUiInit() {
         console.log('Mock CLI executed successfully');
       }
       
       function cli(argv = process.argv.slice(2)) {
-        runZeroUiInit(argv);
+        await runZeroUiInit(argv);
         return Promise.resolve();
       }
       
@@ -406,7 +407,7 @@ export function TestComponent() {
     process.chdir(testDir);
 
     try {
-      runZeroUiInit();
+      await runZeroUiInit();
 
       // Check that initialization messages were logged
       const logOutput = logMessages.join('\n');
@@ -454,7 +455,7 @@ test('Library CLI handles errors gracefully', async () => {
       const { runZeroUiInit } = require('../../src/cli/postInstall.cjs');
 
       // This should complete without errors in most cases
-      runZeroUiInit();
+      await runZeroUiInit();
 
       console.log('✅ Library CLI handles execution without crashing');
 
@@ -723,7 +724,7 @@ export function Toggle() {
     process.chdir(testDir);
 
     try {
-      runZeroUiInit();
+      await runZeroUiInit();
 
       const logOutput = logMessages.join('\n');
       assert(logOutput.includes('[Zero-UI] Initializing...'), 'Should log initialization');
diff --git a/packages/core/src/cli/init.cjs b/packages/core/src/cli/init.cjs
@@ -7,12 +7,17 @@ const { runZeroUiInit } = require('./postInstall.cjs');
 
 
 // Take command line arguments (defaulting to process.argv.slice(2) which are the args after node <scriptname>) and pass them to runZeroUiInit
-function cli(argv = process.argv.slice(2)) {
-  runZeroUiInit(argv);
+async function cli(argv = process.argv.slice(2)) {
+  return await runZeroUiInit(argv);
 }
 
 /* -------- CL I  -------- */
-if (require.main === module) cli();   // `npx init-react-zero-ui`
+if (require.main === module) {
+  cli().catch(error => {
+    console.error('CLI failed:', error);
+    process.exit(1);
+  });
+}
 
 /* -------- CJS  -------- */
 module.exports = cli;                 // `require('@…/cli')()`
diff --git a/packages/core/src/cli/postInstall.cjs b/packages/core/src/cli/postInstall.cjs
@@ -8,28 +8,28 @@ const {
   hasViteConfig,
 } = require('../postcss/helpers.cjs');
 
-function runZeroUiInit() {
+async function runZeroUiInit() {
   try {
     console.log('[Zero-UI] Initializing...');
 
     // Patch Vite config for Vite projects
     if (hasViteConfig()) {
-      patchViteConfig();
+      await patchViteConfig();
     }
 
 
     if (!hasViteConfig()) {
       // Patch config for module resolution 
-      patchConfigAlias();
+      await patchConfigAlias();
       // Patch PostCSS config for Next.js projects
-      patchPostcssConfig();
+      await patchPostcssConfig();
     }
 
     // Process all variants using the shared helper
-    const { finalVariants, initialValues, sourceFiles } = processVariants();
+    const { finalVariants, initialValues, sourceFiles } = await processVariants();
 
     // Generate attribute files using the shared helper
-    generateAttributesFile(finalVariants, initialValues);
+    await generateAttributesFile(finalVariants, initialValues);
 
 
     console.log(`[Zero-UI] ✅ Initialized with ${finalVariants.length} variants from ${sourceFiles.length} files`);
diff --git a/packages/core/src/postcss/helpers.cjs b/packages/core/src/postcss/helpers.cjs
@@ -58,7 +58,7 @@ function findAllSourceFiles(rootDirs = ['src', 'app']) {
  * @param {string[]} files - Array of file paths to process - if not provided, all source files will be found and processed
  * @returns {Object} - Object containing final variants, initial values, and source files
  */
-function processVariants(files = null) {
+async function processVariants(files = null) {
   const sourceFiles = files || findAllSourceFiles();
   const allVariants = sourceFiles.flatMap(file => {
     return extractVariants(file);
@@ -116,7 +116,7 @@ function buildCss(variants) {
   return css;
 }
 
-function generateAttributesFile(finalVariants, initialValues) {
+async function generateAttributesFile(finalVariants, initialValues) {
   const cwd = process.cwd();
   const ATTR_DIR = path.join(cwd, CONFIG.ZERO_UI_DIR);
   const ATTR_FILE = path.join(ATTR_DIR, 'attributes.js');
@@ -196,7 +196,7 @@ function isZeroUiInitialized() {
  * are inside the TypeScript program. Writes to ts/ jsconfig only if something
  * actually changes.
  */
-function patchConfigAlias() {
+async function patchConfigAlias() {
   const cwd = process.cwd();
 
   const configFile = fs.existsSync(path.join(cwd, 'tsconfig.json'))
@@ -256,7 +256,7 @@ function patchConfigAlias() {
  * Patches postcss.config.js to include Zero-UI plugin before Tailwind CSS
  * Only runs for Next.js projects and uses AST parsing for robust config modification
  */
-function patchPostcssConfig() {
+async function patchPostcssConfig() {
   const cwd = process.cwd();
   const postcssConfigJsPath = path.join(cwd, 'postcss.config.js');
   const postcssConfigMjsPath = path.join(cwd, 'postcss.config.mjs');
@@ -330,7 +330,7 @@ export default {
  * Patches vite.config.ts/js to include Zero-UI plugin and replace Tailwind CSS v4+ plugin if present
  * Only runs for Vite projects and uses AST parsing for robust config modification
  */
-function patchViteConfig() {
+async function patchViteConfig() {
   const cwd = process.cwd();
   const viteConfigTsPath = path.join(cwd, 'vite.config.ts');
   const viteConfigJsPath = path.join(cwd, 'vite.config.js');
diff --git a/packages/core/src/postcss/index.cjs b/packages/core/src/postcss/index.cjs
@@ -16,12 +16,11 @@ module.exports = () => {
     postcssPlugin: 'postcss-react-zero-ui',
 
     // Once runs BEFORE Root, so Tailwind will see our variants
-    Once(root, { result }) {
-      const startTime = Date.now();
+    async Once(root, { result }) {
 
       try {
         // Process all variants using the shared helper
-        const { finalVariants, initialValues, sourceFiles } = processVariants();
+        const { finalVariants, initialValues, sourceFiles } = await processVariants();
 
         // Generate CSS
         const cssBlock = buildCss(finalVariants);
@@ -44,24 +43,14 @@ module.exports = () => {
         // Run initialization only once (fallback if user hasn't run npx zero-ui init)
         if (!isZeroUiInitialized()) {
           console.log('[Zero-UI] Auto-initializing (first time setup)...');
-          runZeroUiInit();
+          await runZeroUiInit();
         }
 
         // Generate body attributes file and TypeScript definitions
-        const { jsChanged, tsChanged } = generateAttributesFile(finalVariants, initialValues);
-
-        // Optional: Log performance metrics in development
-        if (process.env.NODE_ENV === 'development') {
-          const processingTime = Date.now() - startTime;
-          console.log(`[Zero-UI] Processed ${sourceFiles.length} files in ${processingTime}ms`);
-          if (jsChanged || tsChanged) {
-            console.log(`[Zero-UI] Updated: ${jsChanged ? 'attributes.js' : ''} ${tsChanged ? 'attributes.d.ts' : ''}`);
-          }
-        }
+        await generateAttributesFile(finalVariants, initialValues);
 
       } catch (error) {
-        console.error('[Zero-UI] PostCSS plugin error: ', error.message);
-        // Don't throw to prevent build failures
+        throw new Error('[Zero-UI] PostCSS plugin error: ', error.message);
       }
     }
   };
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
