remove unused shadcn elements and radix-ui deps. also remove unused donutchart/chart class

[sawka]

No description provided.

[coderabbitai]

Walkthrough

This PR removes five UI component and utility modules from the codebase alongside dependency updates. The changes delete: the DonutChart component and its Storybook story, the entire Shadcn chart integration system (ChartContainer, ChartStyle, ChartTooltip, ChartLegend, and supporting helpers), the Shadcn form utility module, and the Shadcn label component wrapper. The package.json is updated to remove @radix-ui/react-label and @radix-ui/react-slot dependencies and bump versions for dayjs, mermaid, and remark-flexible-toc.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

• Import verification: Verify no remaining files reference the deleted modules (DonutChart, ChartContainer, ChartStyle, ChartTooltip, ChartLegend, form utilities, Label) to prevent build failures
• Dependency chain check: Confirm that removal of @radix-ui/react-label and @radix-ui/react-slot doesn't break other components that may depend on these packages indirectly
• Build validation: Ensure the codebase builds successfully after all deletions and dependency removals
• Orphaned exports: Check if these deleted components were re-exported from index files or public entry points

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	No description was provided by the author, making it impossible to assess relevance to the changeset.	Add a brief description explaining the rationale for removing these components and dependencies to help reviewers understand the motivation.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main changes: removal of unused shadcn elements, radix-ui dependencies, and unused chart components.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch sawka/remove-unused

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4da6a39 and b92271f.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (6)

• frontend/app/element/donutchart.stories.tsx (0 hunks)
• frontend/app/element/donutchart.tsx (0 hunks)
• frontend/app/shadcn/chart.tsx (0 hunks)
• frontend/app/shadcn/form.tsx (0 hunks)
• frontend/app/shadcn/label.tsx (0 hunks)
• package.json (3 hunks)

💤 Files with no reviewable changes (5)

• frontend/app/element/donutchart.tsx
• frontend/app/shadcn/form.tsx
• frontend/app/element/donutchart.stories.tsx
• frontend/app/shadcn/label.tsx
• frontend/app/shadcn/chart.tsx

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)

• GitHub Check: Build Docsite
• GitHub Check: Analyze (javascript-typescript)
• GitHub Check: Analyze (go)
• GitHub Check: Build for TestDriver.ai
• GitHub Check: merge-gatekeeper

🔇 Additional comments (2)

package.json (2)

114-114: Version bumps are safe and aligned with PR scope.

The minor/patch version updates to dayjs (1.11.15 → 1.11.19), mermaid (11.12.0 → 11.12.1), and remark-flexible-toc (1.2.3 → 1.2.4) are conservative and unlikely to introduce breaking changes. These updates appear to be pulled in alongside the cleanup work and don't seem directly tied to the removal refactor.

Also applies to: 123-123, 146-146

---

91-163: Dependency cleanup is complete and verified.

The removal of @radix-ui/react-label and @radix-ui/react-slot has been verified—no remaining references exist in the codebase, and no orphaned @radix-ui packages remain. The version bumps (dayjs, mermaid, remark-flexible-toc) are all patch-level updates and pose no compatibility risks.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}