fix(logs): don't render numbers with >13 digits using commas

[JoshuaKGoldberg]

Extracts a formatNumber helper for the numeric field renderer. It bypasses formatFloat for numbers that have over 13 digits.

Tangentially related:

• fix(dashboards): Format y-axis numbers with abbreviation helper #110360
• fix(dashboards): Reduce max fractional digits for number type to 4 #110858

Fixes EXP-866.

[linear-code]

EXP-866 Don't show commas for numbers with >=13 digits

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Formatting function has inconsistent return type
  • Changed formatNumber to return String(value) for large numbers instead of raw number, ensuring consistent string return type across all code paths and aligning with other formatting utilities.

Or push these changes by commenting:

@cursor push 0f0e338f92

Preview (0f0e338f92)

diff --git a/static/app/utils/number/formatNumber.spec.ts b/static/app/utils/number/formatNumber.spec.ts
--- a/static/app/utils/number/formatNumber.spec.ts
+++ b/static/app/utils/number/formatNumber.spec.ts
@@ -10,11 +10,11 @@
   });
 
   it('returns the value when the value has 14 digits', () => {
-    expect(formatNumber(12_345_678_901_234)).toBe(12345678901234);
+    expect(formatNumber(12_345_678_901_234)).toBe('12345678901234');
   });
 
   it('returns the value when the value has 15 digits', () => {
-    expect(formatNumber(123_456_789_012_345)).toBe(123456789012345);
+    expect(formatNumber(123_456_789_012_345)).toBe('123456789012345');
   });
 
   it('returns the value when the value does not have digits', () => {

diff --git a/static/app/utils/number/formatNumber.ts b/static/app/utils/number/formatNumber.ts
--- a/static/app/utils/number/formatNumber.ts
+++ b/static/app/utils/number/formatNumber.ts
@@ -4,7 +4,7 @@
 
 export function formatNumber(value: number) {
   if (value >= 10_000_000_000_000) {
-    return value;
+    return String(value);
   }
 
   return formatFloat(value, NUMBER_MAX_FRACTION_DIGITS).toLocaleString(undefined, {

_{This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 5d90b14. Configure here.}

[nsdeschenes]

All looks good, can you update the PR title to match the changes as well, just so the commit message matches everything 🙏

[JoshuaKGoldberg]

D'oh!