feat(ui): right-align mag notation to preserve original display width

Pad mag output with leading spaces so it matches the display width
of the original value. Prevents columns from shifting when toggling
mag mode on/off.

Closes #192

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: cpcloud

internal/app/chat.go

@@ -779,14 +779,16 @@ func (m *Model) handleSQLResult(msg sqlResultMsg) tea.Cmd {
 
 	// The SQL is already stored in the assistant message's SQL field.
 	// Stage 2: summarize results via streaming LLM call.
+	// Always send unformatted numbers to the LLM so the stored response
+	// contains regular dollar amounts. Client-side magTransformText handles
+	// mag notation at render time, making it toggleable.
 	resultsTable := llm.FormatResultsTable(msg.Columns, msg.Rows)
 	summaryPrompt := llm.BuildSummaryPrompt(
 		msg.Question,
 		msg.SQL,
 		resultsTable,
 		time.Now(),
 		m.llmExtraContext,
-		m.magMode,
 	)
 
 	messages := []llm.Message{
@@ -1098,7 +1100,6 @@ func (m *Model) buildFallbackMessages(question string) []llm.Message {
 		dataDump,
 		time.Now(),
 		m.llmExtraContext,
-		m.magMode,
 	)
 
 	messages := []llm.Message{
@@ -1241,7 +1242,11 @@ func (m *Model) renderChatMessages() string {
 
 			// Show response if available.
 			if text != "" {
-				parts = append(parts, renderMarkdown(text, innerW-2))
+				display := text
+				if m.magMode {
+					display = magTransformText(display)
+				}
+				parts = append(parts, renderMarkdown(display, innerW-2))
 			}
 
 			// Determine what to show on the label line.

internal/app/chat_test.go

@@ -319,7 +319,7 @@ func TestLateChatChunkAfterCancellationIsDropped(t *testing.T) {
 		"partial content should have been removed by cancellation")
 }
 
-// TestChatMagModeToggle verifies that ctrl+m toggles the global mag mode
+// TestChatMagModeToggle verifies that ctrl+o toggles the global mag mode
 // even when the chat overlay is active.
 func TestChatMagModeToggle(t *testing.T) {
 	m := newTestModel()
@@ -336,3 +336,44 @@ func TestChatMagModeToggle(t *testing.T) {
 	sendKey(m, "ctrl+o")
 	assert.False(t, m.magMode)
 }
+
+// TestChatMagModeTogglesRenderedOutput verifies that toggling mag mode
+// updates dollar amounts in already-displayed LLM responses.
+func TestChatMagModeTogglesRenderedOutput(t *testing.T) {
+	m := newTestModel()
+	m.width = 120
+	m.height = 40
+	m.openChat()
+
+	// Simulate a completed assistant response with dollar amounts.
+	m.chat.Messages = []chatMessage{
+		{Role: roleUser, Content: "how much did I spend?"},
+		{Role: roleAssistant, Content: "You spent $5,234.23 on kitchen renovations."},
+	}
+	m.refreshChatViewport()
+
+	// Mag mode off: original dollar amount should appear in the viewport.
+	vpContent := m.chat.Viewport.View()
+	assert.Contains(t, vpContent, "$5,234.23",
+		"dollar amount should appear verbatim with mag mode off")
+	assert.NotContains(t, vpContent, magArrow,
+		"mag arrow should not appear with mag mode off")
+
+	// Toggle mag mode on from within chat.
+	sendKey(m, "ctrl+o")
+	assert.True(t, m.magMode)
+
+	vpContent = m.chat.Viewport.View()
+	assert.NotContains(t, vpContent, "$5,234.23",
+		"original dollar amount should be replaced with mag mode on")
+	assert.Contains(t, vpContent, magArrow,
+		"mag arrow should appear with mag mode on")
+
+	// Toggle back off.
+	sendKey(m, "ctrl+o")
+	assert.False(t, m.magMode)
+
+	vpContent = m.chat.Viewport.View()
+	assert.Contains(t, vpContent, "$5,234.23",
+		"dollar amount should reappear after toggling mag mode off")
+}

internal/app/mag.go

@@ -6,6 +6,7 @@ package app
 import (
 	"fmt"
 	"math"
+	"regexp"
 	"strconv"
 	"strings"
 
@@ -63,7 +64,6 @@ func magFormat(c cell, includeUnit bool) string {
 	if f == 0 {
 		return fmt.Sprintf("%s%s%s0", sign, unit, magArrow)
 	}
-
 	mag := int(math.Round(math.Log10(math.Abs(f))))
 	return fmt.Sprintf("%s%s%s%d", sign, unit, magArrow, mag)
 }
@@ -82,6 +82,21 @@ func magOptionalCents(cents *int64) string {
 	return magCents(*cents)
 }
 
+// magMoneyRe matches dollar amounts like $1,234.56 or -$5.00 in prose.
+var magMoneyRe = regexp.MustCompile(`-?\$[\d,]+(?:\.\d+)?`)
+
+// magTransformText replaces dollar amounts in free-form text with magnitude
+// notation. Used to post-process LLM responses when mag mode is on.
+// Does not pad (no width preservation needed in prose).
+func magTransformText(s string) string {
+	return magMoneyRe.ReplaceAllStringFunc(s, func(match string) string {
+		c := cell{Value: match, Kind: cellMoney}
+		result := magFormat(c, true)
+		// Strip leading padding -- prose doesn't need right-alignment.
+		return strings.TrimLeft(result, " ")
+	})
+}
+
 // magTransformCells returns a copy of the cell grid with numeric values
 // replaced by their order-of-magnitude representation. Dollar prefixes are
 // stripped because the column header carries the unit annotation instead.

internal/app/mag_test.go

@@ -11,6 +11,7 @@ import (
 
 func TestMagFormatMoneyWithUnit(t *testing.T) {
 	// Used by magCents for dashboard (input still has $ from FormatCents).
+	// No internal padding; rendering layer handles alignment.
 	tests := []struct {
 		name  string
 		value string
@@ -32,8 +33,9 @@ func TestMagFormatMoneyWithUnit(t *testing.T) {
 }
 
 func TestMagFormatBareMoney(t *testing.T) {
-	// Table cells now carry $ from FormatCents. With includeUnit=false
+	// Table cells carry $ from FormatCents. With includeUnit=false
 	// the mag output strips the $ (header carries the unit instead).
+	// No internal padding; table renderer handles alignment.
 	tests := []struct {
 		name  string
 		value string
@@ -149,6 +151,45 @@ func TestMagTransformCells(t *testing.T) {
 	assert.Equal(t, "$5,234.23", rows[0][2].Value)
 }
 
+func TestMagTransformText(t *testing.T) {
+	tests := []struct {
+		name  string
+		input string
+		want  string
+	}{
+		{
+			"dollar amount",
+			"You spent $5,234.23 on kitchen.",
+			"You spent $ \U0001F8214 on kitchen.",
+		},
+		{
+			"multiple amounts",
+			"Budget is $10,000.00 and actual is $8,500.00.",
+			"Budget is $ \U0001F8214 and actual is $ \U0001F8214.",
+		},
+		{
+			"negative amount",
+			"Loss of -$500.00 this month.",
+			"Loss of -$ \U0001F8213 this month.",
+		},
+		{
+			"no amounts",
+			"The project is underway.",
+			"The project is underway.",
+		},
+		{
+			"small amount",
+			"Just $5.00.",
+			"Just $ \U0001F8211.",
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			assert.Equal(t, tt.want, magTransformText(tt.input))
+		})
+	}
+}
+
 func TestMagModeToggle(t *testing.T) {
 	m := newTestModel()
 	assert.False(t, m.magMode)

internal/app/model.go

@@ -377,6 +377,9 @@ func (m *Model) handleCommonKeys(key tea.KeyMsg) (tea.Cmd, bool) {
 		return nil, true
 	case "ctrl+o":
 		m.magMode = !m.magMode
+		if m.chat != nil && m.chat.Visible {
+			m.refreshChatViewport()
+		}
 		return nil, true
 	case "h", "left":
 		if tab := m.effectiveTab(); tab != nil {

internal/llm/prompt.go

@@ -57,7 +57,6 @@ func BuildSummaryPrompt(
 	question, sql, resultsTable string,
 	now time.Time,
 	extraContext string,
-	magMode bool,
 ) string {
 	var b strings.Builder
 	b.WriteString(summarySystemPreamble)
@@ -70,9 +69,6 @@ func BuildSummaryPrompt(
 	b.WriteString(resultsTable)
 	b.WriteString("\n```\n\n")
 	b.WriteString(summaryGuidelines)
-	if magMode {
-		b.WriteString(magModeGuideline)
-	}
 	if extraContext != "" {
 		b.WriteString("\n\n## Additional context\n\n")
 		b.WriteString(extraContext)
@@ -88,7 +84,6 @@ func BuildSystemPrompt(
 	dataSummary string,
 	now time.Time,
 	extraContext string,
-	magMode bool,
 ) string {
 	var b strings.Builder
 	b.WriteString(fallbackPreamble)
@@ -106,9 +101,6 @@ func BuildSystemPrompt(
 	}
 	b.WriteString("\n\n")
 	b.WriteString(fallbackGuidelines)
-	if magMode {
-		b.WriteString(magModeGuideline)
-	}
 	if extraContext != "" {
 		b.WriteString("\n\n## Additional context\n\n")
 		b.WriteString(extraContext)
@@ -316,9 +308,6 @@ const summaryGuidelines = `RULES:
 5. Do NOT show raw SQL or table formatting. Speak naturally.
 6. Do NOT invent data that isn't in the results.`
 
-const magModeGuideline = `
-7. MAGNITUDE MODE: Format ALL numeric values using order-of-magnitude notation with the 🠡 symbol. Examples: $🠡3 for thousands, $🠡6 for millions, 🠡2 for hundreds. Even single numbers must use this format. Calculate the magnitude as floor(log10(abs(value))). For zero, use 🠡0.`
-
 // ---------- Fallback (single-stage) ----------
 
 const fallbackPreamble = `You are micasa-assistant, a factual Q&A bot for a home management app. ` +

internal/llm/prompt_test.go

@@ -34,7 +34,7 @@ var testNow = time.Date(2026, 2, 13, 10, 0, 0, 0, time.UTC)
 // --- BuildSystemPrompt (fallback) ---
 
 func TestBuildSystemPromptIncludesSchema(t *testing.T) {
-	prompt := BuildSystemPrompt(testTables, "", testNow, "", false)
+	prompt := BuildSystemPrompt(testTables, "", testNow, "")
 	assert.Contains(t, prompt, "projects")
 	assert.Contains(t, prompt, "id integer PK")
 	assert.Contains(t, prompt, "title text NOT NULL")
@@ -48,39 +48,27 @@ func TestBuildSystemPromptIncludesData(t *testing.T) {
 		"### projects (3 rows)\n\n- id: 1, title: Fix roof\n",
 		testNow,
 		"",
-		false,
 	)
 	assert.Contains(t, prompt, "Fix roof")
 	assert.Contains(t, prompt, "Current Data")
 }
 
 func TestBuildSystemPromptOmitsDataWhenEmpty(t *testing.T) {
-	prompt := BuildSystemPrompt(nil, "", testNow, "", false)
+	prompt := BuildSystemPrompt(nil, "", testNow, "")
 	assert.NotContains(t, prompt, "Current Data")
 }
 
 func TestBuildSystemPromptIncludesCurrentDate(t *testing.T) {
-	prompt := BuildSystemPrompt(nil, "", testNow, "", false)
+	prompt := BuildSystemPrompt(nil, "", testNow, "")
 	assert.Contains(t, prompt, "Friday, February 13, 2026")
 }
 
 func TestBuildSystemPromptIncludesExtraContext(t *testing.T) {
-	prompt := BuildSystemPrompt(nil, "", testNow, "House is a 1920s craftsman.", false)
+	prompt := BuildSystemPrompt(nil, "", testNow, "House is a 1920s craftsman.")
 	assert.Contains(t, prompt, "Additional context")
 	assert.Contains(t, prompt, "1920s craftsman")
 }
 
-func TestBuildSystemPromptIncludesMagGuideline(t *testing.T) {
-	prompt := BuildSystemPrompt(nil, "", testNow, "", true)
-	assert.Contains(t, prompt, "MAGNITUDE MODE")
-	assert.Contains(t, prompt, "\U0001F821") // 🠡
-}
-
-func TestBuildSystemPromptOmitsMagGuidelineWhenOff(t *testing.T) {
-	prompt := BuildSystemPrompt(nil, "", testNow, "", false)
-	assert.NotContains(t, prompt, "MAGNITUDE MODE")
-}
-
 // --- BuildSQLPrompt ---
 
 func TestBuildSQLPromptIncludesDDL(t *testing.T) {
@@ -125,7 +113,6 @@ func TestBuildSummaryPromptIncludesAllParts(t *testing.T) {
 		"count\n3\n",
 		testNow,
 		"",
-		false,
 	)
 	assert.Contains(t, prompt, "How many projects?")
 	assert.Contains(t, prompt, "SELECT COUNT(*)")
@@ -134,27 +121,16 @@ func TestBuildSummaryPromptIncludesAllParts(t *testing.T) {
 }
 
 func TestBuildSummaryPromptIncludesCurrentDate(t *testing.T) {
-	prompt := BuildSummaryPrompt("test", "SELECT 1", "1\n", testNow, "", false)
+	prompt := BuildSummaryPrompt("test", "SELECT 1", "1\n", testNow, "")
 	assert.Contains(t, prompt, "Friday, February 13, 2026")
 }
 
 func TestBuildSummaryPromptIncludesExtraContext(t *testing.T) {
-	prompt := BuildSummaryPrompt("test", "SELECT 1", "1\n", testNow, "Currency is CAD.", false)
+	prompt := BuildSummaryPrompt("test", "SELECT 1", "1\n", testNow, "Currency is CAD.")
 	assert.Contains(t, prompt, "Additional context")
 	assert.Contains(t, prompt, "Currency is CAD")
 }
 
-func TestBuildSummaryPromptIncludesMagGuideline(t *testing.T) {
-	prompt := BuildSummaryPrompt("test", "SELECT 1", "1\n", testNow, "", true)
-	assert.Contains(t, prompt, "MAGNITUDE MODE")
-	assert.Contains(t, prompt, "\U0001F821") // 🠡
-}
-
-func TestBuildSummaryPromptOmitsMagGuidelineWhenOff(t *testing.T) {
-	prompt := BuildSummaryPrompt("test", "SELECT 1", "1\n", testNow, "", false)
-	assert.NotContains(t, prompt, "MAGNITUDE MODE")
-}
-
 // --- FormatResultsTable ---
 
 func TestFormatResultsTableWithRows(t *testing.T) {
@@ -214,7 +190,7 @@ func TestBuildSQLPromptIncludesEntityRelationships(t *testing.T) {
 }
 
 func TestBuildSystemPromptIncludesEntityRelationships(t *testing.T) {
-	prompt := BuildSystemPrompt(testTables, "", testNow, "", false)
+	prompt := BuildSystemPrompt(testTables, "", testNow, "")
 	assert.Contains(t, prompt, "## Entity Relationships")
 	assert.Contains(t, prompt, "Foreign key relationships")
 	assert.Contains(t, prompt, "projects.project_type_id")