feat(mag): transform bare numbers in LLM prose, not just dollar amounts

magTransformText previously only matched prefixed amounts. Bare counts
like "1" in "There is 1 flooring project" were left untouched. Now uses
a combined regex that matches both dollar amounts and standalone numbers
in a single pass (so replacement digits like the 4 in 🠡4 are never
re-matched).

refs #195

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

Author: cpcloud

internal/app/mag.go

@@ -82,18 +82,22 @@ 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).
+// magTextRe matches dollar amounts ($1,234.56, -$5.00) and standalone bare
+// numbers (42, 1,000, 3.14) in prose. Dollar amounts are tried first via
+// alternation so their digits aren't consumed by the bare-number branch.
+// A single-pass replace ensures output digits (like the 4 in 🠡4) are never
+// re-matched.
+var magTextRe = regexp.MustCompile(`-?\$[\d,]+(?:\.\d+)?|\b\d[\d,]*(?:\.\d+)?\b`)
+
+// magTransformText replaces dollar amounts and bare numbers in free-form
+// text with magnitude notation. Used to post-process LLM responses when
+// mag mode is on.
 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, " ")
+	return magTextRe.ReplaceAllStringFunc(s, func(match string) string {
+		if strings.ContainsRune(match, '$') {
+			return magFormat(cell{Value: match, Kind: cellMoney}, true)
+		}
+		return magFormat(cell{Value: match, Kind: cellDrilldown}, false)
 	})
 }
 

internal/app/mag_test.go

@@ -173,7 +173,7 @@ func TestMagTransformText(t *testing.T) {
 			"Loss of -$ \U0001F8213 this month.",
 		},
 		{
-			"no amounts",
+			"no amounts or numbers",
 			"The project is underway.",
 			"The project is underway.",
 		},
@@ -182,6 +182,26 @@ func TestMagTransformText(t *testing.T) {
 			"Just $5.00.",
 			"Just $ \U0001F8211.",
 		},
+		{
+			"bare count",
+			"There is 1 flooring project.",
+			"There is \U0001F8210 flooring project.",
+		},
+		{
+			"larger bare count",
+			"You have 42 maintenance items.",
+			"You have \U0001F8212 maintenance items.",
+		},
+		{
+			"bare number with commas",
+			"Total is 1,000 items.",
+			"Total is \U0001F8213 items.",
+		},
+		{
+			"mixed dollars and bare numbers",
+			"Found 3 projects totaling $15,000.00.",
+			"Found \U0001F8210 projects totaling $ \U0001F8214.",
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {