fix(cli): preserve shaping joiners in human-readable results

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## Unreleased
 
+- CLI: preserve Unicode shaping joiners in human-readable names and emoji while retaining strict terminal-control sanitization in diagnostics.
 - Client: reject responses larger than 1 MiB without decoding truncated data, and report unhandled HTTP redirects as API errors instead of successful results.
 - Homebrew: install through a Formula (`brew install openclaw/tap/goplaces`) instead of the retired Cask; quarantine is preserved.
 

internal/cli/joiners_test.go

@@ -0,0 +1,40 @@
+package cli
+
+import (
+	"bytes"
+	"strings"
+	"testing"
+
+	"github.com/steipete/goplaces"
+)
+
+func TestRenderSearchPreservesUnicodeShapingJoiners(t *testing.T) {
+	name := "👩\u200d🔬 می\u200cخواهم"
+	output := renderSearch(NewColor(false), goplaces.SearchResponse{Results: []goplaces.PlaceSummary{{Name: name, Address: "👩\u200d🔬"}}})
+	if !strings.Contains(output, name) {
+		t.Fatalf("name changed: %q", output)
+	}
+	if strings.Count(output, "\u200d") != 2 {
+		t.Fatalf("address joiner lost: %q", output)
+	}
+}
+
+func TestRenderJoinersStillRemovesTerminalAndBidiControls(t *testing.T) {
+	name := "👩\u200d🔬 safe\u202egnirts\u2066\u200b\ufeff\x1b[31m\x07"
+	output := renderSearch(NewColor(false), goplaces.SearchResponse{Results: []goplaces.PlaceSummary{{Name: name}}})
+	if !strings.Contains(output, "👩\u200d🔬") {
+		t.Fatalf("emoji shaping lost: %q", output)
+	}
+	if strings.ContainsAny(output, "\u202e\u2066\u200b\ufeff\x1b\x07") {
+		t.Fatalf("unsafe display controls: %q", output)
+	}
+}
+
+func TestErrorSanitizerKeepsStrictFormatBoundary(t *testing.T) {
+	var buffer bytes.Buffer
+	writeError(&buffer, "safe\u200c\u200d\u202e\u2066\u200b\ufeff\x1b")
+	output := buffer.String()
+	if output != "safe\n" {
+		t.Fatalf("error sanitizer changed: %q", output)
+	}
+}

internal/cli/render.go

@@ -448,7 +448,17 @@ func directionsStepLine(step goplaces.DirectionsStep) string {
 	return strings.Join(parts, " · ")
 }
 
+// Human output preserves shaping joiners used in names and emoji, while
+// diagnostics keep the stricter format-control boundary.
 func sanitizeTerminalText(value string) string {
+	return sanitizeText(value, true)
+}
+
+func sanitizeTerminalErrorText(value string) string {
+	return sanitizeText(value, false)
+}
+
+func sanitizeText(value string, preserveJoiners bool) string {
 	if value == "" {
 		return ""
 	}
@@ -463,7 +473,7 @@ func sanitizeTerminalText(value string) string {
 		if unicode.IsControl(r) {
 			continue
 		}
-		if unicode.Is(unicode.Cf, r) {
+		if unicode.Is(unicode.Cf, r) && (!preserveJoiners || (r != '\u200c' && r != '\u200d')) {
 			continue
 		}
 		out.WriteRune(r)

internal/cli/run.go

@@ -144,5 +144,5 @@ func handleError(writer io.Writer, err error) int {
 }
 
 func writeError(writer io.Writer, message string) {
-	_, _ = fmt.Fprintln(writer, sanitizeTerminalText(message))
+	_, _ = fmt.Fprintln(writer, sanitizeTerminalErrorText(message))
 }