Skip to content

Configure Knip source discovery - #33

Merged
nritschel merged 2 commits into
masterfrom
tools/update-knip
Sep 24, 2026
Merged

nritschel merged 2 commits into
masterfrom
tools/update-knip

Conversation

@SybelBlue

Copy link
Copy Markdown
Collaborator

Summary

  • anchor the Python packaging /lib/ ignore rule so nested frontend sources remain visible to Knip
  • configure root and sketch_tool workspace entry points and project files
  • point Knip at the nonstandard Webpack config filenames and account for HTML- and runtime-loaded JavaScript
  • ignore the buffer package false positive caused by its Node built-in name

Testing

  • git diff --check master...HEAD
  • npm exec --yes --package=knip@6.38.0 -- knip --include files --no-exit-code --no-progress
    • reports the two existing cleanup candidates: lib/config-validator.js and postcss.config.js

PostCSS cleanup is intentionally out of scope and will be handled separately.

@SybelBlue
SybelBlue marked this pull request as ready for review September 23, 2026 20:45
@SybelBlue
SybelBlue requested review from a team and austinbillings-pl and removed request for a team September 23, 2026 20:45
@SybelBlue
SybelBlue added this pull request to stack #35 September 23, 2026 21:01
@nritschel
nritschel merged commit 2a7aa2c into master Sep 24, 2026
5 checks passed
@nritschel
nritschel deleted the tools/update-knip branch September 24, 2026 01:09
@SybelBlue SybelBlue mentioned this pull request Sep 24, 2026
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.

3 participants