fix: show meaningful error messages instead of undefined on build/dev failures (#131)

Amp-Thread-ID: https://ampcode.com/threads/T-019d1b0d-6dca-7013-a8f8-af75363aae7d
Co-authored-by: Amp <amp@ampcode.com>

Author: gnzjgo

src/api/build.ts

@@ -223,7 +223,7 @@ export async function buildToTinybird(
     if (body.errors && body.errors.length > 0) {
       return body.errors.map(e => {
         const prefix = e.filename ? `[${e.filename}] ` : '';
-        return `${prefix}${e.error}`;
+        return `${prefix}${e.error ?? 'Unknown error'}`;
       }).join('\n');
     }
     return body.error || `HTTP ${response.status}: ${response.statusText}`;

src/cli/index.ts

@@ -541,6 +541,8 @@ function createCli(): Command {
           output.showBuildErrors(deploy.errors);
         } else if (result.error) {
           output.error(result.error);
+        } else {
+          output.error("Deploy failed (no error details available, run with TINYBIRD_DEBUG=1 for more info)");
         }
         output.showDeployFailure();
         process.exit(1);
@@ -732,6 +734,8 @@ function createCli(): Command {
                 output.showBuildErrors(deploy.errors);
               } else if (result.error) {
                 output.error(result.error);
+              } else {
+                output.error("Build failed (no error details available, run with TINYBIRD_DEBUG=1 for more info)");
               }
               output.showBuildFailure(true);
               return;