Skip to content

Remove unused "unused" stuff checkers - #315

Open
mym0404 wants to merge 2 commits into
toss:mainfrom
mym0404:feat/remove-dead-code-prune-tools
Open

Remove unused "unused" stuff checkers#315
mym0404 wants to merge 2 commits into
toss:mainfrom
mym0404:feat/remove-dead-code-prune-tools

Conversation

@mym0404

@mym0404 mym0404 commented Jun 17, 2026

Copy link
Copy Markdown

Overview

I believe knip and ts-prune have been neglecting their roles as static check gates. The initial commit includes those setup up front.

I'm not much of this project(getting familiar with this though). If I was wrong, feel free to close this PR.

This project is hard to verify whether some stuff is unused or not without an elaborated way to determine due to a complex Monorepo architecture and build chains. If that's okay with you, I suggest it to remove thin knip and ts-prune configuration for now, and to integrate those static checks later if need to again.

Knip report

I managed to run knip with some fixing and found out that results are a bit messed. So I concluded these old tools and scripts have lost their meanings at one point.

Category Count Examples Notes
Unused files 201 packages/cli/src/utils/compressToGzip.ts, packages/mpack/tsup.config.mts, example app files Includes configs, templates, examples, and generated-looking files.
Unused dependencies 58 @granite-js/utils, connect, zod, @babel/runtime, rimraf, yargs Reported from package manifests. Needs manual validation before removal.
Unused dev dependencies 50 @babel/preset-env, vitest, @vitest/coverage-v8 Some may be used by build/test conventions not detected by current config.
Unlisted or unresolved dependencies 25 react/package.json, custom-protocol:my-module, metro-babel-register Many are fixtures, protocol tests, or vendored Metro imports.
Unused exports 80 packages/mpack/src/vendors/metro/**, React Native constants High noise from vendored code and public API exports.
Unused exported types 46 BuildStatusPluginOptions, native component event types, React Native types Public or generated types may be falsely reported.
Duplicate exports 5 CookieManager, GraniteImage, LottieView, Video Needs API review before changing exports.

@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

@mym0404 is attempting to deploy a commit to the Toss Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot

changeset-bot Bot commented Jun 17, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e6fb4ba

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 28 packages
Name Type
@granite-js/cli Patch
@granite-js/mpack Patch
@granite-js/react-native Patch
babel-preset-granite Patch
create-granite-app Patch
@granite-js/blur-view Patch
@granite-js/brownfield-module Patch
@granite-js/cookies Patch
@granite-js/screen Patch
@granite-js/image Patch
@granite-js/jest Patch
@granite-js/lottie Patch
@granite-js/native Patch
@granite-js/naver-map Patch
@granite-js/plugin-core Patch
@granite-js/plugin-env Patch
@granite-js/plugin-hermes Patch
@granite-js/plugin-micro-frontend Patch
@granite-js/plugin-router Patch
@granite-js/plugin-rozenite Patch
@granite-js/plugin-sentry Patch
@granite-js/style-utils Patch
@granite-js/utils Patch
@granite-js/video Patch
@granite-js/vitest Patch
@granite-js/deployment-manager Patch
@granite-js/forge-cli Patch
@granite-js/pulumi-aws Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant