Skip to content

chore: remove dead code, unused fixtures and dev dependencies - #1481

Open
l2ysho wants to merge 2 commits into
masterfrom
claude/cli-dead-code-detection-98d4f9
Open

l2ysho wants to merge 2 commits into
masterfrom
claude/cli-dead-code-detection-98d4f9

Conversation

@l2ysho

@l2ysho l2ysho commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

dead code removal

Found with knip and verified by grep and git history:
- getNpmCmd, mapGroupBy, consoleDatasetUrl and link have no callers
- badUserClient and three dataset schema fixtures are unused in tests
- @types/adm-zip (adm-zip ships its own types) and cross-env are unused
- stale TODO about migrating to yargs

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@l2ysho l2ysho added adhoc Ad-hoc unplanned task added during the sprint. t-builders Issues owned by the Builders team. labels Oct 8, 2026
@l2ysho
l2ysho marked this pull request as ready for review October 8, 2026 20:02
@l2ysho
l2ysho requested a review from DaveHanns as a code owner October 8, 2026 20:02
@apify-service-account apify-service-account added the tested Temporary label used only programatically for some analytics. label Oct 8, 2026

This branch has not been deployed

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

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-builders Issues owned by the Builders team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants