Repository navigation
Remove components that no other project uses - #91
Open
lucperkins wants to merge 2 commits into
Open
lucperkins wants to merge 2 commits into
lucperkins wants to merge 2 commits into
Conversation
flakehub and docs.determinate.systems are the only consumers of this package, and between them they import BusyIcon, CodeInline, ColorProvider, ColorSchemeToggle, CopyButton, LabeledRadioInput, LabeledTextInput, and PageLayout. This keeps those and everything they depend on, and removes the rest: Button, CodeBlock, CodeFile, DownloadButton, GitHubButton, Highlight, InstallCTA, Link, MacInstaller, and the useHighlight and useObjectURL hooks. The Shiki packages and @radix-ui/react-icons were only used by those, so they're gone too. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for determinate-ui ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Remove code display and download components src/hooks/*, src/Highlight/*, src/CodeBlock/*, src/CodeFile/*, src/DownloadButton/* |
Removes highlighting hooks and components, code display and download components, and their styles and stories. |
Remove installation components src/InstallCTA/*, src/MacInstaller/* |
Removes the installation CTA, target detection, tab components, macOS installer, and their styles and stories. |
Remove button and link components src/Button/*, src/GitHubButton/*, src/Link/* |
Removes the Button, GitHubButton, and Link components, along with their styles and stories. |
Update package dependencies and public exports src/index.ts, package.json |
Removes exports for the deleted components and hooks, and removes @radix-ui/react-icons and the five @shikijs/* packages from dependencies. |
Priority:
Estimated code review effort: 3 (Moderate) | ~25 minutes
Change: Other
Suggested reviewers: gustavderdrache, grahamc
Merge Risk: 
4f627
-
The removals appear consistent with the intended breaking change, with no identified issue requiring resolution before merge.
Pre-merge checks: 
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title clearly and concisely summarizes the main change: removing unused components, hooks, stories, styles, and dependencies. Docstring Coverage No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.
-
Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
# Conflicts: # package-lock.json # package.json
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
flakehubanddocs.determinate.systemsare the only projects that depend on this package, according to a code search across the org. Between them they import:BusyIconCodeInline(docs, through theexport *incomponents/DS.tsx)ColorProviderColorSchemeToggleCopyButtonLabeledRadioInputandLabeledTextInputPageLayoutThis keeps those and everything they depend on (
ClipboardIcon,ColorContext,DetSysIcon,Header,LabeledInput,Navigation,Placeholder), and removes the rest:Button,CodeBlock,CodeFile,DownloadButton,GitHubButton,Highlight,InstallCTA,Link, andMacInstalleruseHighlightanduseObjectURLhooks, which only those used@shikijs/*packages and@radix-ui/react-icons, which only those usedThis is a breaking change to the package's exports, but nothing that consumes it imports any of the removed ones.
Testing
npm run build,npm run format, andcodespell srcpass.🤖 Generated with Claude Code
Summary by CodeRabbit