WIP

Author: mads-hartmann

.werft/build.ts

@@ -61,7 +61,7 @@ async function run(context: any) {
         await triggerUpgradeTests(werft, config, context.Owner);
         return;
     }
-    const a = ""
+
     await buildAndPublish(werft, config);
 
     if (config.withSelfHostedPreview) {

.werft/util/werft.ts

@@ -92,8 +92,6 @@ export class Werft {
             console.log(`[${slice}] tracing warning: No slice span by name ${slice}`);
         }
 
-        const errorMessage = err instanceof Error ? err.toString() : err;
-
         // Set the status on the span for the slice and also propagate the status to the phase and root span
         // as well so we can query on all phases that had an error regardless of which slice produced the error.
         [span, this.rootSpan, this.currentPhaseSpan].forEach((span: Span) => {
@@ -102,14 +100,18 @@ export class Werft {
             }
             span.setStatus({
                 code: SpanStatusCode.ERROR,
-                message: errorMessage,
+                message: err.toString(),
             });
         });
 
         // In case the error message is a multi-line string we want to ensure that we contain
         // the error message within the slice (otherwise they'll be moved to the "default" slice of the phase)
-        errorMessage.split("\n").forEach((line: string) => console.log(`[${slice}] ${line}`));
+        err.toString()
+            .split("\n")
+            .forEach((line: string) => console.log(`[${slice}] ${line}`));
 
+        // The UI shows the last log of the slice which might not make a lot of sense
+        // for multi-line error messages, so instead we tell the user to expand the slice.
         console.log(`[${slice}] Failed. Expand to see why`);
         console.log(`[${slice}|FAIL]`);
         throw new FailedSliceError(slice);