Skip to content

fix(react-doctor): clear architecture structure warnings#344

Open
bntvllnt wants to merge 1 commit into
mainfrom
fix/269-architecture
Open

fix(react-doctor): clear architecture structure warnings#344
bntvllnt wants to merge 1 commit into
mainfrom
fix/269-architecture

Conversation

@bntvllnt
Copy link
Copy Markdown
Collaborator

Closes #269

Summary

  • add a Knip workspace config so generated and framework-owned files are excluded from file-structure diagnostics
  • remove unused public registry exports and keep helper types private where they are only used internally
  • replace the BlogCard duplicate alias with a wrapper and regenerate the registry copy from the source component

Validation

  • pnpm -F @vllnt/ui lint
  • pnpm -F @vllnt/ui exec tsc --noEmit --project tsconfig.build.json
  • pnpm -F @vllnt/ui-registry exec tsc --noEmit --project tsconfig.json
  • pnpm build
  • pnpm test:once
  • git diff --check
  • npx --yes react-doctor . --diff origin/main --annotations --fail-on error --offline
  • npx --yes react-doctor . --offline --json: targeted files/exports/types/duplicates warnings = 0

Notes

  • Next.js build still emits the known multiple-lockfile root warning in worktrees.
  • Vitest still emits the known jsdom navigation, InteractiveTimeline duplicate-key, and ShareDialog description warnings.

@bntvllnt bntvllnt added react-doctor Reported by react-doctor (codebase health) tech-debt Refactoring or cleanup dx Developer experience 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 10:48pm
ui.vllnt.ai Ready Ready Preview, Comment May 12, 2026 10:48pm

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dx Developer experience 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: file/exports/types structure (files + exports + types + duplicates, 336 warnings)

1 participant