Skip to content

Remove empty env.d.ts ambient declaration - #3030

Merged
tanem merged 1 commit into
masterfrom
cleanup/dead-code
Aug 26, 2026
Merged

Remove empty env.d.ts ambient declaration#3030
tanem merged 1 commit into
masterfrom
cleanup/dead-code

Conversation

@tanem

@tanem tanem commented Aug 26, 2026

Copy link
Copy Markdown
Owner

A dead-code pass across the repo turned up one orphaned file.

src/env.d.ts once held a single declare module 'react-lifecycles-compat' ambient declaration. That dependency was removed during the February 2026 tooling modernisation (commit 58b607d), leaving the file empty. Nothing includes it by name and it emits nothing, so it is dead. This removes it.

The rest of the repo was audited in the same pass and is clean: source helpers and exports, scripts, jest configs, devDependencies, README (references all nine examples), MIGRATION.md (the withNProgress and UMD mentions are intentional upgrade history), the .github workflows, codecov.yml (live, uploaded from ci.yml), and the test/react matrix (matches the boundary list in AGENTS.md). No other dead code was found.

Verified with a full npm test run: format and type checks, lint, build, size gate, bundle tests, the full React version matrix, and 100% source coverage.

The file held a single `declare module 'react-lifecycles-compat'`
ambient declaration. That dependency was removed during the February
2026 tooling modernisation, leaving the file empty. Nothing includes
it by name and it emits nothing, so it is dead.
@tanem tanem added the internal label Aug 26, 2026
@tanem
tanem merged commit 42c7f03 into master Aug 26, 2026
2 checks passed
@tanem
tanem deleted the cleanup/dead-code branch August 26, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant