> [!NOTE] > **TL;DR** — related to https://github.com/apify/apify-cli/pull/1481, lets ensure we are not letting dead code in PRs ## Proposal - Add `knip` as a dev dependency and a `knip.json` config: - entry points: `src/entrypoints/*.ts`, `scripts/**/*.ts`, tests - ignore `website/` (separate Docusaurus app) - ignore `__APIFY_KEYRING_NATIVE_SUBPACKAGE__` (build-time placeholder) - turn off `types` and `exports` rules at first — about 47 hits today are symbols used inside their own file, so only the `export` keyword is extra - Add a `pnpm run knip` script and run it in the lint job. - Check the install-size impact. knip is a dev dependency, so the published package does not change. ## Open questions - Fail the build or only warn at first? - Turn on the `exports` rule later, after a cleanup pass? Related: #1481 (one-off cleanup found with knip). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Note
TL;DR — related to #1481, lets ensure we are not letting dead code in PRs
Proposal
knipas a dev dependency and aknip.jsonconfig:src/entrypoints/*.ts,scripts/**/*.ts, testswebsite/(separate Docusaurus app)__APIFY_KEYRING_NATIVE_SUBPACKAGE__(build-time placeholder)typesandexportsrules at first — about 47 hits today are symbols used inside their own file, so only theexportkeyword is extrapnpm run knipscript and run it in the lint job.Open questions
exportsrule later, after a cleanup pass?Related: #1481 (one-off cleanup found with knip).
🤖 Generated with Claude Code