Skip to content

chore: remove unused dead code#457

Merged
aidenybai merged 2 commits into
mainfrom
chore/dead-code-cleanup
May 28, 2026
Merged

chore: remove unused dead code#457
aidenybai merged 2 commits into
mainfrom
chore/dead-code-cleanup

Conversation

@aidenybai

@aidenybai aidenybai commented May 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • Prune unused helpers, stale types, debug/event tracking state, and unnecessary internal exports found by deslop-cli.
  • Align package metadata by removing unused dependencies and adding the missing react-component-name/loader build entry.
  • Refresh the generated website auto.global.js and lockfile after dependency/build changes.

Test plan

  • pnpm build
  • pnpm typecheck
  • pnpm dlx deslop-cli analyze . --tsconfig tsconfig.json --ignore "**/dist/**" "**/.next/**" "**/node_modules/**" "**/build/**"
  • pnpm lint still fails on the repo's existing warning set; no linter errors were reported for the edited files via IDE diagnostics.

Note

Low Risk
Mostly deletion of unreferenced symbols and quote/formatting churn; the storage guard is a small hardening change with no auth or data-pipeline impact.

Overview
This PR removes unused and internal-only code across the extension, react-scan core, web UI, and notifications paths—helpers like debounce, profile/debug interaction stores, fiber adjacency utilities, aggregation helpers, and several outline/instrumentation exports that were not referenced elsewhere.

Behavioral tweaks are small: extension chrome.storage read/write now validates stored objects before indexing; react-component-name option resolution types/helpers that were unused are dropped from options.ts.

Packaging: packages/scan/package.json is reorganized (metadata, publishConfig, entry fields) and react-component-name/loader is wired through typesVersions and exports alongside the existing build entry in tsup.

Reviewed by Cursor Bugbot for commit 5adb676. Bugbot is set up for automated code reviews on this repo. Configure here.

Prune stale helpers and internal-only exports found by deslop, and align package metadata with actual build/runtime usage.
@vercel

vercel Bot commented May 28, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
react-scan Ready Ready Preview, Comment May 28, 2026 11:07am

Restore the react-doctor dependency because the publish workflow invokes it through pnpm exec.
@aidenybai
aidenybai merged commit d6e1fa8 into main May 28, 2026
4 of 6 checks passed
@aidenybai
aidenybai deleted the chore/dead-code-cleanup branch May 28, 2026 22:33
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.

1 participant