@tamagui/build 1.135.7 → 1.136.1

package/.turbo/turbo-test.log

@@ -2,26 +2,26 @@
  RUN  v4.0.4 /Users/n8/tamagui/code/packages/build
 
 stdout | __tests__/integration.test.ts > tamagui-build integration test > should rebuild the package on file change when --watch is used
-Watch process output: built tamagui-build-test-watch-package in 59 ms
+Watch process output: built tamagui-build-test-watch-package in 64 ms
 
 Initial build complete, modifying file...
 
 stdout | __tests__/integration.test.ts > tamagui-build integration test > should rebuild the package on file change when --watch is used
-Watch process output: built tamagui-build-test-watch-package in 347 ms
+Watch process output: built tamagui-build-test-watch-package in 349 ms
 
 Rebuild after file modification complete
 
- ✓ __tests__/integration.test.ts (7 tests) 9478ms
-     ✓ should build the package correctly  1377ms
-     ✓ should bundle the package correctly  1083ms
-     ✓ should skip mjs files when --skip-mjs is used  1121ms
-     ✓ should ignore base URL when --ignore-base-url is used  1128ms
-     ✓ should rebuild the package on file change when --watch is used  1323ms
-     ✓ should generate correct platform-specific output  1151ms
-     ✓ should minify the output when MINIFY=true is set  2278ms
+ ✓ __tests__/integration.test.ts (7 tests) 10037ms
+     ✓ should build the package correctly  1451ms
+     ✓ should bundle the package correctly  1172ms
+     ✓ should skip mjs files when --skip-mjs is used  1266ms
+     ✓ should ignore base URL when --ignore-base-url is used  1160ms
+     ✓ should rebuild the package on file change when --watch is used  1350ms
+     ✓ should generate correct platform-specific output  1222ms
+     ✓ should minify the output when MINIFY=true is set  2396ms
 
  Test Files  1 passed (1)
       Tests  7 passed (7)
-   Start at  05:32:06
-   Duration  9.58s (transform 36ms, setup 0ms, collect 41ms, tests 9.48s, environment 0ms, prepare 2ms)
+   Start at  16:45:26
+   Duration  10.14s (transform 36ms, setup 0ms, collect 41ms, tests 10.04s, environment 0ms, prepare 2ms)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tamagui/build",
-  "version": "1.135.7",
+  "version": "1.136.1",
   "bin": {
     "tamagui-build": "tamagui-build.js",
     "teesx": "./teesx.sh"
@@ -14,7 +14,7 @@
   "dependencies": {
     "@babel/core": "^7.25.2",
     "@swc/core": "^1.14.0",
-    "@tamagui/babel-plugin-fully-specified": "1.135.7",
+    "@tamagui/babel-plugin-fully-specified": "1.136.1",
     "@types/fs-extra": "^9.0.13",
     "chokidar": "^3.5.2",
     "esbuild": "^0.25.11",

package/pretty-print-errors.js

@@ -0,0 +1,130 @@
+/* eslint-disable no-console */
+
+/**
+ * Pretty print TypeScript diagnostics with file location and error codes
+ * @param {import('typescript').Diagnostic[]} diagnostics
+ * @param {import('typescript')} ts
+ */
+function printTypescriptDiagnostics(diagnostics, ts) {
+  console.error('\n❌ TypeScript compilation errors:\n')
+
+  diagnostics.forEach((diagnostic) => {
+    const messageText =
+      typeof diagnostic.messageText === 'string'
+        ? diagnostic.messageText
+        : diagnostic.messageText?.messageText || JSON.stringify(diagnostic.messageText)
+
+    if (diagnostic.file && diagnostic.start !== undefined) {
+      const { line, character } = ts.getLineAndCharacterOfPosition(
+        diagnostic.file,
+        diagnostic.start
+      )
+      console.error(
+        `  ${diagnostic.file.fileName}:${line + 1}:${character + 1} - error TS${diagnostic.code}: ${messageText}`
+      )
+    } else {
+      console.error(`  error TS${diagnostic.code}: ${messageText}`)
+    }
+  })
+
+  console.error('')
+}
+
+/**
+ * Pretty print esbuild errors with file location and code snippets
+ * @param {Error & { errors?: any[], warnings?: any[] }} err
+ */
+function printEsbuildError(err) {
+  console.error('\n❌ Build error:\n')
+
+  // esbuild errors have a formatted message that's much more readable
+  if (err.errors && err.errors.length > 0) {
+    err.errors.forEach((error) => {
+      if (error.location) {
+        console.error(
+          `  ${error.location.file}:${error.location.line}:${error.location.column}: ${error.text}`
+        )
+        if (error.location.lineText) {
+          console.error(`    ${error.location.lineText}`)
+        }
+      } else {
+        console.error(`  ${error.text}`)
+      }
+    })
+  } else {
+    // Fallback for non-esbuild errors
+    console.error(err.message || err)
+  }
+
+  if (err.warnings && err.warnings.length > 0) {
+    console.error('\n⚠️  Warnings:\n')
+    err.warnings.forEach((warning) => {
+      if (warning.location) {
+        console.error(
+          `  ${warning.location.file}:${warning.location.line}:${warning.location.column}: ${warning.text}`
+        )
+      } else {
+        console.error(`  ${warning.text}`)
+      }
+    })
+  }
+
+  console.error('')
+}
+
+/**
+ * Pretty print general build errors with stack traces
+ * @param {Error} error
+ * @param {string} packageName
+ * @param {string} cwd
+ */
+function printBuildError(error, packageName, cwd) {
+  console.error(`\n❌ Error building ${packageName} in ${cwd}:\n`)
+
+  if (error.stack) {
+    // Show a more readable error with the stack trace
+    const lines = error.stack.split('\n')
+    console.error(`  ${lines[0]}`) // The error message
+    if (lines.length > 1) {
+      console.error('\nStack trace:')
+      lines.slice(1, 6).forEach((line) => console.error(`  ${line.trim()}`))
+      if (lines.length > 6) {
+        console.error(`  ... and ${lines.length - 6} more`)
+      }
+    }
+  } else {
+    console.error(`  ${error.message || error}`)
+  }
+
+  console.error('')
+}
+
+/**
+ * Pretty print TypeScript compilation errors (network, config, etc)
+ * @param {Error & { code?: string }} err
+ * @param {string} packageName
+ */
+function printTypescriptCompilationError(err, packageName) {
+  console.error(`\n❌ Error during TypeScript compilation for ${packageName}:\n`)
+
+  if (err.code === 'ENOTFOUND' || err.code === 'ECONNREFUSED') {
+    console.error(`  Network error: ${err.message}`)
+  } else if (err.message) {
+    console.error(`  ${err.message}`)
+    if (err.stack && process.env.DEBUG) {
+      console.error('\nStack trace (DEBUG mode):')
+      err.stack.split('\n').slice(1, 6).forEach((line) => console.error(`  ${line.trim()}`))
+    }
+  } else {
+    console.error(`  ${err}`)
+  }
+
+  console.error('')
+}
+
+module.exports = {
+  printTypescriptDiagnostics,
+  printEsbuildError,
+  printBuildError,
+  printTypescriptCompilationError,
+}

package/tamagui-build.js

@@ -13,6 +13,12 @@ const { es5Plugin } = require('./esbuild-es5')
 const ts = require('typescript')
 const path = require('node:path')
 const childProcess = require('node:child_process')
+const {
+  printTypescriptDiagnostics,
+  printEsbuildError,
+  printBuildError,
+  printTypescriptCompilationError,
+} = require('./pretty-print-errors')
 
 const jsOnly = !!process.env.JS_ONLY
 const skipJS = !!(process.env.SKIP_JS || false)
@@ -178,7 +184,7 @@ async function build({ skipTypes } = {}) {
     // Run afterBuild script if defined
     await runAfterBuild()
   } catch (error) {
-    console.error(` ❌ Error building in ${process.cwd()}:\n\n`, error.stack + '\n')
+    printBuildError(error, pkg.name, process.cwd())
     if (!shouldWatch) {
       process.exit(1)
     }
@@ -267,9 +273,7 @@ async function buildTsc(allFiles) {
 
     // exit on errors
     if (diagnostics.some((x) => x.code) && !shouldWatch) {
-      console.error(
-        `Error building: ${diagnostics.map((x) => JSON.stringify(x.messageText)).join('\n')}`
-      )
+      printTypescriptDiagnostics(diagnostics, ts)
       if (shouldWatch) {
         return
       }
@@ -282,11 +286,7 @@ async function buildTsc(allFiles) {
       throw new Error('TypeScript compilation failed')
     }
   } catch (err) {
-    if (err.code === 'ENOTFOUND' || err.code === 'ECONNREFUSED') {
-      console.error(`Network error during compilation for ${pkg.name}:`, err.message)
-    } else {
-      console.error(`Error during TypeScript compilation for ${pkg.name}:`, err)
-    }
+    printTypescriptCompilationError(err, pkg.name)
     if (!shouldWatch) {
       process.exit(1)
     }
@@ -653,7 +653,7 @@ async function esbuildWriteIfChanged(
   try {
     built = await esbuild.build(buildSettings)
   } catch (err) {
-    console.error(`Error building`, err)
+    printEsbuildError(err)
     if (!shouldWatch) {
       process.exit(1)
     }