feat: Added improved error logging for devcontainer feature.

Author: vercel-eddie

install-edge.sh

@@ -28,6 +28,13 @@ main() {
     os=$(get_os)
     arch=$(get_arch)
 
+    # Cache sudo credentials up front so the prompt happens before the
+    # download — avoids issues when the script is piped via curl | sh.
+    if [ ! -w "$INSTALL_DIR" ]; then
+        echo "Installation to ${INSTALL_DIR} requires sudo access."
+        sudo -v
+    fi
+
     binary_name="bridge-${os}-${arch}"
     download_url="https://github.com/${REPO}/releases/download/edge/${binary_name}"
 
@@ -36,15 +43,14 @@ main() {
     curl -fsSL -o bridge "${download_url}"
     chmod +x bridge
 
-    # Remove macOS quarantine attribute to prevent Gatekeeper from killing the binary
     if [ "$os" = "darwin" ]; then
+        # Remove macOS quarantine attribute to prevent Gatekeeper prompts
         xattr -d com.apple.quarantine bridge 2>/dev/null || true
     fi
 
     if [ -w "$INSTALL_DIR" ]; then
         mv bridge "${INSTALL_DIR}/bridge"
     else
-        echo "Moving binary to ${INSTALL_DIR} (requires sudo)..."
         sudo mv bridge "${INSTALL_DIR}/bridge"
     fi
 

pkg/commands/create.go

@@ -206,7 +206,7 @@ func runCreate(ctx context.Context, c *cli.Command) error {
 		if !yes {
 			p.Newline()
 			p.Warn("No bridge administrator found in the cluster.")
-			p.Info(fmt.Sprintf("Bridge can use your local credentials for cluster %q instead.", kubeContext))
+			p.Info(fmt.Sprintf("Should Bridge use your local credentials for cluster %q instead.", kubeContext))
 			fmt.Fprintf(w, "Continue? [y/N] ")
 
 			answer := promptYN(r)

pkg/devcontainer/client.go

@@ -1,9 +1,7 @@
 package devcontainer
 
 import (
-	"bytes"
 	"context"
-	"encoding/json"
 	"fmt"
 	"io"
 	"os"
@@ -33,7 +31,7 @@ func (c *Client) Up(ctx context.Context) error {
 	cmd.Stdout = nil
 	cmd.Stderr = nil
 	if out, err := cmd.CombinedOutput(); err != nil {
-		return fmt.Errorf("devcontainer up: %w\n%s", err, extractErrorMessage(out))
+		return fmt.Errorf("devcontainer up: %w\n%s", err, out)
 	}
 	return nil
 }
@@ -97,35 +95,6 @@ func (c *Client) Stop(ctx context.Context) error {
 	return nil
 }
 
-// extractErrorMessage parses the devcontainer CLI's JSON output to find the
-// error message. The CLI emits newline-delimited JSON; the line with
-// "outcome":"error" contains the human-readable description. If no such line
-// is found, the raw output is returned as-is.
-func extractErrorMessage(out []byte) string {
-	var result struct {
-		Outcome     string `json:"outcome"`
-		Message     string `json:"message"`
-		Description string `json:"description"`
-	}
-	for _, line := range bytes.Split(out, []byte("\n")) {
-		line = bytes.TrimSpace(line)
-		if len(line) == 0 || line[0] != '{' {
-			continue
-		}
-		if err := json.Unmarshal(line, &result); err != nil {
-			continue
-		}
-		if result.Outcome == "error" {
-			msg := result.Description
-			if msg == "" {
-				msg = result.Message
-			}
-			return msg
-		}
-	}
-	return string(out)
-}
-
 func (c *Client) stdinOrDefault() io.Reader {
 	if c.Stdin != nil {
 		return c.Stdin

pkg/interact/printer.go

@@ -3,6 +3,7 @@ package interact
 import (
 	"fmt"
 	"io"
+	"strings"
 )
 
 // Printer writes themed, styled messages to a writer.
@@ -31,10 +32,16 @@ func (p *Printer) Info(msg string) {
 	fmt.Fprintf(p.w, "%s %s\n", p.theme.Info.Render("→"), msg)
 }
 
-// Errorf prints a red error message with formatting.
+// Errorf prints a red error message with formatting. If the message contains
+// newlines, only the first line is styled to avoid lipgloss mangling
+// multi-line output (e.g. devcontainer build logs).
 func (p *Printer) Errorf(format string, a ...any) {
 	msg := fmt.Sprintf(format, a...)
-	fmt.Fprintf(p.w, "%s %s\n", p.theme.Error.Render("✗"), p.theme.Error.Render(msg))
+	first, rest, _ := strings.Cut(msg, "\n")
+	fmt.Fprintf(p.w, "%s %s\n", p.theme.Error.Render("✗"), p.theme.Error.Render(first))
+	if rest != "" {
+		fmt.Fprintln(p.w, rest)
+	}
 }
 
 // Header prints a bold, underlined header.