miro/files
1
0
Fork 0
Code Issues 1 Pull requests 3 Projects Releases 11 Packages 1 Activity Actions

Add knip workflow and remove dead code #44

Merged
miro merged 1 commit from claude/knip-workflow-install-f5f373 into main 2026-10-10 19:49:28 +00:00
Conversation 0 Commits 1 Files changed 15 +587 -151
miro commented 2026-10-10 18:59:14 +00:00
Owner
Copy link

Summary

  • Add knip as a pinned root devDependency, a pnpm knip script, knip.json, and a knip job in the Forgejo CI workflow.
  • Remove dead code it found: unused files (ContextMenu.tsx, support/config.ts), the unused @radix-ui/react-context-menu dependency, and unused exports/types.
  • src/components/ui/** (vendored shadcn components) is ignored on purpose, along with its @radix-ui/react-navigation-menu dependency.
  • docker/bootstrap.mjs is declared as an entry since the Dockerfile runs it.

Test plan

  • pnpm knip, pnpm lint, pnpm typecheck, pnpm format:check pass locally
  • CI green
## Summary - Add [knip](https://knip.dev) as a pinned root devDependency, a `pnpm knip` script, `knip.json`, and a `knip` job in the Forgejo CI workflow. - Remove dead code it found: unused files (`ContextMenu.tsx`, `support/config.ts`), the unused `@radix-ui/react-context-menu` dependency, and unused exports/types. - `src/components/ui/**` (vendored shadcn components) is ignored on purpose, along with its `@radix-ui/react-navigation-menu` dependency. - `docker/bootstrap.mjs` is declared as an entry since the Dockerfile runs it. ## Test plan - [x] `pnpm knip`, `pnpm lint`, `pnpm typecheck`, `pnpm format:check` pass locally - [ ] CI green
Add knip workflow and remove dead code
Some checks failed
CI / lint (pull_request) Successful in 25s
Details
CI / format (pull_request) Successful in 37s
Details
CI / build (pull_request) Successful in 38s
Details
CI / knip (pull_request) Successful in 38s
Details
CI / typecheck (pull_request) Successful in 40s
Details
CI / docker (amd64, docker-amd64) (pull_request) Successful in 28s
Details
CI / manifest (pull_request) Has been cancelled
Details
E2E / playwright (pull_request) Has been cancelled
Details
CI / docker (arm64, docker-arm64) (pull_request) Has been cancelled
Details
e170377779
miro force-pushed claude/knip-workflow-install-f5f373 from e170377779
Some checks failed
CI / lint (pull_request) Successful in 25s
Details
CI / format (pull_request) Successful in 37s
Details
CI / build (pull_request) Successful in 38s
Details
CI / knip (pull_request) Successful in 38s
Details
CI / typecheck (pull_request) Successful in 40s
Details
CI / docker (amd64, docker-amd64) (pull_request) Successful in 28s
Details
CI / manifest (pull_request) Has been cancelled
Details
E2E / playwright (pull_request) Has been cancelled
Details
CI / docker (arm64, docker-arm64) (pull_request) Has been cancelled
Details
to 87e907eaf5
All checks were successful
CI / typecheck (pull_request) Successful in 30s
Details
CI / lint (pull_request) Successful in 40s
Details
CI / knip (pull_request) Successful in 36s
Details
CI / format (pull_request) Successful in 26s
Details
CI / docker (arm64, docker-arm64) (pull_request) Successful in 40s
Details
CI / build (pull_request) Successful in 46s
Details
CI / docker (amd64, docker-amd64) (pull_request) Successful in 28s
Details
CI / manifest (pull_request) Has been skipped
Details
E2E / playwright (pull_request) Successful in 1m3s
Details
2026-10-10 19:00:31 +00:00 Compare
miro merged commit 5b7b66353e into main 2026-10-10 19:49:28 +00:00
miro deleted branch claude/knip-workflow-install-f5f373 2026-10-10 19:49:28 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
miro/files!44
No description provided.