🔄 Production Mode considers dependencies since v5.42.0

[y-yagi]

Prerequisites

• I've read the relevant documentation
• I've searched for existing issues
• I've read the issue reproduction guide

Reproduction url

https://github.com/y-yagi/knip-issue-unlisted-binaries

Reproduction access

• I've made sure the reproduction is publicly accessible

Good version

5.42.0

Bad version

5.41.1

Description of the regression

We run the following steps on CI. And, we have turbo as devDependencies.

steps:
  - name: Run knip
    run: pnpm knip --production

  - name: Build
    run: pnpm turbo run build 

Since v5.42.0, Knip reports turbo as Unlisted binaries.

Unlisted binaries (2)
turbo  .github/workflows/ci.yml 

I understand the intention, but I could not find any information about these changes in the release notes. So let me ask this change if this is intended or not.

[webpro]

Thanks for the report, @y-yagi!

I think this was introduced in https://github.com/webpro-nl/knip/releases/tag/5.42.1, started with the 1731ee5 refactor.

There have been some improvements in this area, hopefully things got better after my fixes.

[webpro]

🚀 This issue has been resolved in v5.42.2. See Release 5.42.2 for release notes.

Using Knip in a commercial project? Please consider becoming a sponsor.

[y-yagi]

Thanks for your quick fix!