Skip to content

perf: resolve all 7 open performance issues - #20

Merged
Jhong098 merged 1 commit into
mainfrom
claude/review-github-issues-nh61P
May 4, 2026
Merged

perf: resolve all 7 open performance issues#20
Jhong098 merged 1 commit into
mainfrom
claude/review-github-issues-nh61P

Conversation

@Jhong098

@Jhong098 Jhong098 commented May 4, 2026

Copy link
Copy Markdown
Owner

$(cat <<'EOF'

Summary

Resolves all 7 open GitHub performance issues. Two were already fixed in the current codebase (#12, #13) and closed without code changes; the remaining five are addressed here.

Test plan

  • Added DraftSaver.test.tsx — covers subscribe-on-mount, 1500ms trailing debounce, rapid-change debounce reset, and unmount cleanup
  • Updated completions.test.ts — replaced mockUpdateEq/mockProfileSingle with mockRpc; added assertions that increment_xp is called with correct args and is NOT called on a repeat pass
  • Fixed pre-existing e2e failure in config-panel.spec.ts — scoped config-instanceCount locators to the desktop sidebar wrapper to avoid strict-mode ambiguity with MobileConfigPanel
  • 165/165 unit tests passing
  • 40/40 e2e tests passing (10 skipped — require live Clerk credentials)

https://claude.ai/code/session_017P7S4SuivspHeeXfok7WzX
EOF
)


Generated by Claude Code

- lib/supabase/server.ts: wrap createAdminClient with React cache() so
  all server actions within the same request share one Supabase client
  instance instead of constructing a fresh one per call (closes #17)
- components/nodes/BaseNode.tsx: replace transition-all with
  transition-[border-color,opacity] to avoid triggering style recalc
  for unrelated CSS properties on every state change (closes #11)

https://claude.ai/code/session_017P7S4SuivspHeeXfok7WzX
@vercel

vercel Bot commented May 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
poly-sd Ready Ready Preview, Comment May 4, 2026 0:00am

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.50%. Comparing base (3d0b03e) to head (46c91de).
⚠️ Report is 1 commits behind head on main.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #20   +/-   ##
=======================================
  Coverage   72.50%   72.50%           
=======================================
  Files          11       11           
  Lines         440      440           
  Branches      134      134           
=======================================
  Hits          319      319           
  Misses         72       72           
  Partials       49       49           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Jhong098
Jhong098 merged commit 653a9ea into main May 4, 2026
5 checks passed
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