Skip to content

refactor: add knip json in monorepo root#2251

Open
edgarkhanzadian wants to merge 1 commit intorefactor/scope-reductionfrom
refactor/add-workspace-knip
Open

refactor: add knip json in monorepo root#2251
edgarkhanzadian wants to merge 1 commit intorefactor/scope-reductionfrom
refactor/add-workspace-knip

Conversation

@edgarkhanzadian
Copy link
Copy Markdown
Collaborator

@edgarkhanzadian edgarkhanzadian commented Apr 1, 2026

Use root level knip command to remove unused variables/exports/types/files/enums

@leather-bot
Copy link
Copy Markdown
Contributor

leather-bot commented Apr 1, 2026

Leather Extension build f45a5c0Extension build, Extension test report

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 1, 2026

Codecov Report

❌ Patch coverage is 11.11111% with 56 lines in your changes missing coverage. Please review.
✅ Project coverage is 12.47%. Comparing base (e70f67e) to head (f45a5c0).

Files with missing lines Patch % Lines
apps/web/app/components/table.tsx 0.00% 10 Missing ⚠️
apps/web/app/layouts/page/page.tsx 0.00% 5 Missing ⚠️
apps/web/app/store/addresses.ts 0.00% 5 Missing ⚠️
apps/web/app/constants/environment.ts 0.00% 2 Missing ⚠️
apps/web/app/mocks/extension/event-bus.ts 0.00% 2 Missing ⚠️
apps/web/app/utils/external-leather-navigator.ts 0.00% 2 Missing ⚠️
...mon/compliance-checker/compliance-checker.query.ts 0.00% 1 Missing ⚠️
...extension/test-app/webpack/webpack.config.base.cjs 0.00% 1 Missing ⚠️
apps/web/app/components/explainer.tsx 0.00% 1 Missing ⚠️
apps/web/app/components/info-card.tsx 0.00% 1 Missing ⚠️
... and 26 more
Additional details and impacted files
@@                     Coverage Diff                      @@
##           refactor/scope-reduction    #2251      +/-   ##
============================================================
+ Coverage                     12.43%   12.47%   +0.03%     
============================================================
  Files                          2430     2393      -37     
  Lines                        100279    98878    -1401     
  Branches                       4183     4125      -58     
============================================================
- Hits                          12471    12335     -136     
+ Misses                        86770    85504    -1266     
- Partials                       1038     1039       +1     
Files with missing lines Coverage Δ
apps/extension/src/app/common/date-utils.ts 85.71% <ø> (ø)
apps/extension/src/app/common/stacks-utils.ts 77.04% <ø> (ø)
...pp/common/transactions/stacks/transaction.utils.ts 0.00% <ø> (-20.38%) ⬇️
apps/extension/src/app/common/utils.ts 15.68% <ø> (-11.02%) ⬇️
.../src/app/common/validation/forms/fee-validators.ts 0.00% <ø> (ø)
.../features/fee-editor/bitcoin/bitcoin-fees.utils.ts 100.00% <ø> (ø)
.../app/features/ledger/utils/generic-ledger-utils.ts 0.00% <ø> (ø)
...y/bitcoin/address/transactions-by-address.query.ts 58.33% <ø> (ø)
...query/bitcoin/ordinals/brc20/brc20-tokens.hooks.ts 0.00% <ø> (ø)
.../query/common/remote-config/remote-config.query.ts 0.00% <ø> (ø)
... and 115 more

... and 1 file with indirect coverage changes

Components Coverage Δ
bitcoin 65.69% <ø> (+0.83%) ⬆️
query 24.38% <ø> (-0.10%) ⬇️
utils 87.20% <ø> (+0.21%) ⬆️
crypto 69.56% <ø> (-0.19%) ⬇️
stacks 48.83% <ø> (-0.53%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@edgarkhanzadian edgarkhanzadian force-pushed the refactor/add-workspace-knip branch from 703a9de to 1607773 Compare April 1, 2026 12:48
@leather-bot
Copy link
Copy Markdown
Contributor

leather-bot commented Apr 1, 2026

Leather Web build f45a5c0Web preview, Web test report

@edgarkhanzadian edgarkhanzadian force-pushed the refactor/add-workspace-knip branch from c236023 to d9282ef Compare April 1, 2026 16:36
@edgarkhanzadian edgarkhanzadian force-pushed the refactor/add-workspace-knip branch from d9282ef to f45a5c0 Compare April 1, 2026 16:39
@edgarkhanzadian edgarkhanzadian requested a review from alexp3y April 1, 2026 16:40
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.

2 participants