Skip to content

fix: Usagemeter not defined error [P0]#922

Merged
Dhravya merged 1 commit into
mainfrom
ishaan/add-note
May 11, 2026
Merged

fix: Usagemeter not defined error [P0]#922
Dhravya merged 1 commit into
mainfrom
ishaan/add-note

Conversation

@ishaanxgupta
Copy link
Copy Markdown
Contributor

@ishaanxgupta ishaanxgupta commented May 10, 2026

image

@claude
Copy link
Copy Markdown

claude Bot commented May 10, 2026

Claude finished @ishaanxgupta's task in 1m 37s —— View job


Review Complete

  • Fetching PR diff
  • Analyzing changes for bugs/security issues
  • Checking surrounding code context
  • Submitting review

Overview: This PR fixes a runtime error by replacing the undefined UsageMeter component with an inline usage progress bar implementation.

Verified:

  • planUsagePct is properly sourced from useTokenUsage hook
  • calculateUsagePercent clamps values to 0-100, preventing CSS overflow
  • All other dependencies are correctly imported

Score: 10/10 — Clean fix, no issues found.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp 260261d May 10 2026, 02:00 PM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 260261d Commit Preview URL

Branch Preview URL
May 10 2026, 02:01 PM

Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview: Replaces the undefined UsageMeter component with an inline usage progress bar implementation for the mobile view.

Issues found: None — this looks good to ship.

Verified:

  • planUsagePct is properly defined from useTokenUsage hook
  • calculateUsagePercent already clamps values to 0-100, preventing width overflow
  • All other variables (tokensUsed, searchesUsed, formatUsageNumber, cn, dmSansClassName) are correctly imported/defined
  • Edge case handling for < 1% display is correct

Score: 10/10

Copy link
Copy Markdown
Member

@ved015 ved015 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ishaanxgupta ishaanxgupta changed the title fix Usagemeter not defined error fix: Usagemeter not defined error [P0] May 10, 2026
@Dhravya Dhravya merged commit ae35b7a into main May 11, 2026
9 checks passed
vorflux Bot pushed a commit that referenced this pull request May 11, 2026
@graphite-app graphite-app Bot requested a review from Dhravya May 13, 2026 11:11
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.

3 participants