#9457: move knip to repository root

[twschiller]

What does this PR do?

• Closes Run knip script from the root project package.json #9457
• Following directions at: https://knip.dev/features/monorepos-and-workspaces
• CI runs knip across all workspaces

Remaining Work

• Fix relative directory reference from storybook/webpack.sharedConfig.js
• Fix knip results (it looks like it's not picking up the knip config in the workspace)
• Verify production mode works locally
• Update CI commands
• Fix knip configuration for libraries workspace
• Verify CI commands

Future Work

• Update CI to run lint on all workspaces

For more information on our expectations for the PR process, see the
code review principles doc

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.81%. Comparing base (8318d74) to head (26dc6f0).
Report is 489 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9505      +/-   ##
==========================================
+ Coverage   74.24%   75.81%   +1.57%     
==========================================
  Files        1332     1413      +81     
  Lines       40817    42769    +1952     
  Branches     7634     7881     +247     
==========================================
+ Hits        30306    32427    +2121     
+ Misses      10511    10342     -169     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Playwright test results

 154 passed
 2 skipped

Details

 Open report ↗︎
 156 tests across 51 suites
 10 minutes, 53 seconds
 26dc6f0
 For more information on how to debug and view this report, see our readme

Skipped tests

chrome › tests/runtime/googleSheetsIntegration.spec.ts › can activate a google spreadsheet mod with config options
msedge › tests/runtime/googleSheetsIntegration.spec.ts › can activate a google spreadsheet mod with config options

[twschiller]

See discussion: #9503

[github-actions]

No loom links were found in the first post. Please add one there if you'd like to it to appear on Slack.

Do not edit this comment manually.

[fungairino]

Looks like knip is still warning of a few more things we need to handle in the CI run.

[twschiller]

Looks like knip is still warning of a few more things we need to handle in the CI run.

Yep, needed to fill out more of the package.json:

pixiebrix-extension/libraries/utils/package.json

Line 7 in 26dc6f0

"lint": "npm run lint:full -- --rule '{\"import/no-cycle\": \"off\"}'",