Skip to content

Format memory grants as KB/MB/GB#76

Merged
erikdarlingdata merged 1 commit intodevfrom
feature/issue-68-memory-grant-units
Mar 10, 2026
Merged

Format memory grants as KB/MB/GB#76
erikdarlingdata merged 1 commit intodevfrom
feature/issue-68-memory-grant-units

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • Add FormatMemoryGrantKB helper that picks human-readable units (KB/MB/GB)
  • Applied in text output (TextFormatter) and runtime summary pane (PlanViewerControl)
  • Reduces visual clutter for large grants (e.g., "2.14 GB" instead of "2,245,632 KB")

Closes #68

Test plan

  • Open a plan with a large memory grant — verify it shows MB or GB
  • Open a plan with a small grant (<1 MB) — verify it shows KB
  • Text output (Advice for Humans) also uses new format

🤖 Generated with Claude Code

Add FormatMemoryGrantKB helper to TextFormatter that picks the most readable
unit: KB under 1024, MB with 1 decimal up to 1 GB, GB with 2 decimals above.
Applied in both the text output and the runtime summary pane.

Closes #68

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit b220373 into dev Mar 10, 2026
2 checks passed
@erikdarlingdata erikdarlingdata deleted the feature/issue-68-memory-grant-units branch March 10, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant