Task 02: Full Audit - Unused Components Report#5
Task 02: Full Audit - Unused Components Report#5DeveloperAlly merged 2 commits intodocs-v2-previewfrom
Conversation
- Update CardCarousel.jsx to use theme CSS variables - Replace fallback colors with proper theme variables - Use --card-background, --accent, --border, --text - Update theme-colors.mdx wiki documentation - Document full ThemeData object with all properties - Add Global CSS Variables section documenting style.css - Update color values to match actual implementation - Update best practices section - Add completion report Closes task 05-homogenise-styling
- Audited 77 exports across 27 component files - Identified 24 definitely unused components for removal - Identified 10+ components with low/example-only usage - Found 2 consolidation opportunities (duplicate files) - Documented usage locations for all 58 actively used components
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 281f6e2b47
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | `display/CardCarousel.jsx` | `CardCarousel` | Never used | | ||
| | `display/embed.jsx` | `MarkdownEmbed` | Example only | | ||
| | `display/embed.jsx` | `EmbedMarkdown` | Example only, duplicate | | ||
| | `display/quote.jsx` | `Quote` | Never used | |
There was a problem hiding this comment.
This entry is incorrect: Quote is actively used in live v2 content (for example in v2/pages/00_home/introduction/vision.mdx and v2/pages/05_orchestrators/quickstart/join-a-pool.mdx), so treating it as a safe removal target can lead Task 02a to delete a component that current pages render. That would force follow-up break-fix edits across published docs pages instead of being a no-risk cleanup.
Useful? React with 👍 / 👎.
|
|
||
| | File | Component | Reason | | ||
| |------|-----------|--------| | ||
| | `display/frameMode.jsx` | `H4`, `H5`, `H6`, `P` | Not used | |
There was a problem hiding this comment.
The report says H5 is unused, but H5 is currently rendered in v2/pages/06_lptoken/token-portal.mdx (e.g., the "Definition" and "Quick Intro" headings). Leaving H5 in this "Not used" bucket makes the follow-up removal task likely to remove a component that is still part of a frame-mode portal page.
Useful? React with 👍 / 👎.
1 participant
Summary
Full audit of unused components in
snippets/components/directory.Key Findings:
Definite Removals (24 components):
Components never used or only in example files:
BlogDataLayout,CardInCardLayout,ResponseFieldGroupCardCarousel,MarkdownEmbed,EmbedMarkdownQuote,ShowcaseVideo,LinkedInEmbed,YouTubeVideoDownload,CardVideoBasicList,IconList,UpdateList,StepList,ListStepsBasicBtn,LivepeerSVG,LivepeerIconOld,LivepeerIconFlipped,LivepeerIconBlinkingTerminal,CopyText,HeroOverviewContentConsolidation Opportunities:
snippets/components/gateways/duplicatesdomain/04_GATEWAYS/content/data.jsxandlayout/cards.jsxFollow-Up Tasks:
Files Added:
docs/PLAN/reports/02-components-audit-unused-report.mdNote
Low Risk
Mostly documentation additions with small, low-impact theme/styling tweaks to UI components; risk is limited to minor visual regressions in
CardCarousel/frame-mode text.Overview
Adds a new unused components audit report (
02-components-audit-unused-report.md) that inventoriessnippets/components/exports, classifies production vs example-only usage, and calls out candidates for removal/consolidation.Adds a styling homogenization completion report (
05-homogenise-styling-report.md) and updates the theme documentation (theme-colors.mdx) to referenceThemeDataand global CSS variables.Makes minor UI consistency fixes by updating
CardCarouselto use theme CSS variables (removing hardcoded fallback colors) and correcting a variable naming bug in thePcomponent indisplay/frameMode.jsx.Written by Cursor Bugbot for commit 281f6e2. This will update automatically on new commits. Configure here.