feat(cli): format phone numbers in show vendors output

Table output uses FormatPhoneNumber with vendor locale fallback.
JSON output preserves raw phone values for machine consumption.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: cpcloud

cmd/micasa/show.go

@@ -12,7 +12,9 @@ import (
 	"text/tabwriter"
 	"time"
 
+	"github.com/micasa-dev/micasa/internal/config"
 	"github.com/micasa-dev/micasa/internal/data"
+	"github.com/micasa-dev/micasa/internal/locale"
 	"github.com/spf13/cobra"
 	"gorm.io/gorm"
 )
@@ -447,7 +449,13 @@ var vendorCols = []showCol[data.Vendor]{
 	{"NAME", func(v data.Vendor) string { return fmtStr(v.Name) }},
 	{"CONTACT", func(v data.Vendor) string { return fmtStr(v.ContactName) }},
 	{"EMAIL", func(v data.Vendor) string { return fmtStr(v.Email) }},
-	{"PHONE", func(v data.Vendor) string { return fmtStr(v.Phone) }},
+	{"PHONE", func(v data.Vendor) string {
+		region := strings.ToUpper(config.DetectCountry())
+		if v.Locale != "" {
+			region = strings.ToUpper(v.Locale)
+		}
+		return fmtStr(locale.FormatPhoneNumber(v.Phone, region))
+	}},
 	{"WEBSITE", func(v data.Vendor) string { return fmtStr(v.Website) }},
 }
 

cmd/micasa/show_test.go

@@ -169,14 +169,16 @@ func TestShowProjectsJSON(t *testing.T) {
 }
 
 func TestShowVendorsText(t *testing.T) {
-	t.Parallel()
+	// Not parallel: t.Setenv modifies process-global state.
+	// LC_ALL has highest precedence in DetectCountry().
+	t.Setenv("LC_ALL", "en_US.UTF-8")
 	store := newTestStoreWithMigration(t)
 
 	require.NoError(t, store.CreateVendor(&data.Vendor{
 		Name:        "Acme Plumbing",
 		ContactName: "John Doe",
 		Email:       "john@acme.com",
-		Phone:       "555-1234",
+		Phone:       "5551234567",
 	}))
 
 	var buf bytes.Buffer
@@ -187,7 +189,7 @@ func TestShowVendorsText(t *testing.T) {
 	assert.Contains(t, out, "Acme Plumbing")
 	assert.Contains(t, out, "John Doe")
 	assert.Contains(t, out, "john@acme.com")
-	assert.Contains(t, out, "555-1234")
+	assert.Contains(t, out, "(555) 123-4567")
 }
 
 func TestShowVendorsJSON(t *testing.T) {
@@ -210,6 +212,25 @@ func TestShowVendorsJSON(t *testing.T) {
 	assert.NotEmpty(t, result[0]["id"])
 }
 
+func TestShowVendorsJSONPhoneRaw(t *testing.T) {
+	t.Parallel()
+	store := newTestStoreWithMigration(t)
+
+	require.NoError(t, store.CreateVendor(&data.Vendor{
+		Name:  "Raw Phone Co",
+		Phone: "5551234567",
+	}))
+
+	var buf bytes.Buffer
+	require.NoError(t, runShow(&buf, store, "vendors", true, false))
+
+	var result []map[string]any
+	require.NoError(t, json.Unmarshal(buf.Bytes(), &result))
+	require.Len(t, result, 1)
+	assert.Equal(t, "5551234567", result[0]["phone"],
+		"JSON output must carry raw phone, not formatted")
+}
+
 func TestShowAppliancesText(t *testing.T) {
 	t.Parallel()
 	store := newTestStoreWithMigration(t)