chore: improve the codebase with react doctor

[MaheshtheDev]

No description provided.

[cloudflare-workers-and-pages]

Deploying with    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	8f6c16d	May 09 2026, 07:13 PM

[cloudflare-workers-and-pages]

Deploying with    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	8f6c16d	Commit Preview URL	May 09 2026, 07:14 PM

[MaheshtheDev]

• chore: improve the codebase with react doctor #917 👈 (View in Graphite)
• main

---

How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[claude]

Claude finished @MaheshtheDev's task in 3m 20s —— View job

---

Review Complete

• Fetch PR diff and understand the changes
• Check for existing review comments
• Review code for bugs, security issues, and logic errors
• Submit review summary

Overview: This PR applies automated "react doctor" improvements - SVG optimization, Tailwind shorthand conversion, typography fixes, and a useEffect cleanup.

Critical Issue Found: The mobile usage display in apps/web/components/add-document/index.tsx references undefined components and functions (UsageMeter, tokensToCredits, tokensPercent, etc.) that will cause a runtime crash on mobile devices.

Score: 4/10 - Cannot merge until the mobile view bug is fixed.

[graphite-app]

Merge activity

• May 9, 7:11 PM UTC: MaheshtheDev added this pull request to the Graphite merge queue.
• May 9, 7:13 PM UTC: The Graphite merge queue couldn't merge this PR because it failed for an unknown reason (Changes have been requested by reviewers on this PR).
• May 9, 7:17 PM UTC: MaheshtheDev added this pull request to the Graphite merge queue.
• May 9, 7:17 PM UTC: Merged by the Graphite merge queue.

[claude]

Review Summary

Overview: This PR applies automated improvements from "react doctor" - SVG path optimization, Tailwind size-* shorthand conversion, typography improvements (... → …, em-dashes → colons), and a proper useEffect cleanup fix.

Issues found:

1. Critical Bug in apps/web/components/add-document/index.tsx: The mobile usage display section references undefined variables and components:

   • UsageMeter component doesn't exist
   • tokensToCredits function doesn't exist
   • tokensPercent, searchesPercent, tokensLimit, searchesLimit are not returned by useTokenUsage

   This will cause a runtime crash on mobile devices.

What looks good:

• The documents-command-palette.tsx change correctly adds cleanup for a setTimeout in useEffect (good memory leak fix)
• SVG path truncation and CSS shorthand changes are safe cosmetic improvements
• Typography changes (ellipsis, punctuation) are benign

Score: 4/10

The PR cannot be merged as-is because the mobile view will crash. The add-document/index.tsx changes need to either be reverted or completed with the missing component and hook updates.

don't need