fix: return JSONRenderer when JSON output is requested

Author: leandroh

internal/renderer/json.go

@@ -4,24 +4,26 @@ import (
 	"bytes"
 	"encoding/json"
 	"fmt"
-
-	"github.com/go-openapi/swag"
 )
 
 type JSONRenderer struct{}
 
 func (jr JSONRenderer) Render(data []ResponseData) {
-	if !swag.IsZero(data) {
-		JSONString, err := json.Marshal(data)
-		if err != nil {
-			fmt.Println("Could not decode the result as JSON.")
-		}
+	if len(data) == 0 {
+		return
+	}
 
-		var prettyJSON bytes.Buffer
-		if err := json.Indent(&prettyJSON, JSONString, "", "    "); err != nil {
-			fmt.Println("JSON format error")
-		}
+	JSONString, err := json.Marshal(data)
+	if err != nil {
+		fmt.Println("Could not decode the result as JSON.")
+		return
+	}
 
-		fmt.Println(prettyJSON.String())
+	var prettyJSON bytes.Buffer
+	if err := json.Indent(&prettyJSON, JSONString, "", "    "); err != nil {
+		fmt.Println("JSON format error")
+		return
 	}
+
+	fmt.Println(prettyJSON.String())
 }

internal/renderer/main.go

@@ -5,6 +5,7 @@ import (
 
 	outputTable "github.com/latitudesh/lsh/internal/output/table"
 	"github.com/spf13/viper"
+	"golang.org/x/term"
 )
 
 type ResponseData interface {
@@ -22,10 +23,14 @@ func GetRenderer() Renderer {
 		return TableRenderer{} // Old ASCII
 	}
 
-	// Check if JSON was requested
-	if viper.GetBool("json") {
-		// You can create a JSONRenderer later
-		return TableRenderer{} // fallback
+	// Check if JSON was requested via --json flag or -o json
+	if viper.GetBool("json") || viper.GetString("output") == "json" {
+		return JSONRenderer{}
+	}
+
+	// If stdout is not a terminal (e.g., pipe), use table output
+	if !term.IsTerminal(int(os.Stdout.Fd())) {
+		return TableRenderer{}
 	}
 
 	// Default: use interactive Bubble Tea