Skip to content

perf: reduce react-doctor performance warnings#340

Open
bntvllnt wants to merge 2 commits into
mainfrom
fix/270-tosorted
Open

perf: reduce react-doctor performance warnings#340
bntvllnt wants to merge 2 commits into
mainfrom
fix/270-tosorted

Conversation

@bntvllnt
Copy link
Copy Markdown
Collaborator

Summary

  • Collapse hot map/filter/includes/find paths into cached Sets/Maps or single-pass reducers across package scripts, registry app code, and components.
  • Keep dynamic Intl formatting behind memoized cache helpers while avoiding raw constructor warnings in render-adjacent paths.
  • Regenerate registry shims so default registry sources mirror package source changes.

Validation

  • pnpm doctor:json -> 10 Performance warnings
  • pnpm -F @vllnt/ui lint
  • pnpm -F @vllnt/ui exec tsc --noEmit --project tsconfig.build.json
  • pnpm build
  • pnpm test:once
  • git diff --check

Closes #270

@bntvllnt bntvllnt added react-doctor Reported by react-doctor (codebase health) tech-debt Refactoring or cleanup performance Performance labels May 12, 2026
@bntvllnt bntvllnt self-assigned this May 12, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

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

Project Deployment Actions Updated (UTC)
storybook Ready Ready Preview, Comment May 12, 2026 8:35pm
ui.vllnt.ai Ready Ready Preview, Comment May 12, 2026 8:35pm

Request Review

# Conflicts:
#	packages/ui/src/components/conversation-thread/conversation-thread.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance Performance react-doctor Reported by react-doctor (codebase health) tech-debt Refactoring or cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

react-doctor: JS performance — hoist Intl, combine iterations, use Sets/Maps (~112 warnings)

1 participant