Skip to content

refactor: remove unused site scaffolding - #31

Open
bermanto wants to merge 2 commits into
mainfrom
cleanup/ponytail-audit
Open

refactor: remove unused site scaffolding#31
bermanto wants to merge 2 commits into
mainfrom
cleanup/ponytail-audit

Conversation

@bermanto

@bermanto bermanto commented Aug 22, 2026

Copy link
Copy Markdown
Member

Summary

  • remove unused UI components, font files, image assets, and duplicate token scaffolding
  • stop exporting UI helpers that are only used inside their modules

Verification

  • pnpm check
  • Knip readback reviewed

Notes

The four public assets still reported by Knip are loaded by HTML paths or CSS imports at runtime.

Summary by CodeRabbit

  • New Features

    • Added a Community Clients entry to the “Using HQBase” documentation navigation.
    • Added documentation coverage for the Community Clients page and overview link.
  • Refactor

    • Streamlined shared interface components and styling utilities.
    • Simplified application configuration for more reliable project setup.
  • Chores

    • Removed unused design-token styles and font license files.
    • Updated site validation checks to reflect the current project structure.

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d697e06e-43a0-4499-b534-4b85702fadf5

📥 Commits

Reviewing files that changed from the base of the PR and between 5791efc and 15a3cf3.

📒 Files selected for processing (2)
  • astro.config.mjs
  • test/site.test.mjs

Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The PR updates Astro root resolution, adds community-clients documentation navigation and checks, narrows UI exports, removes Card and Table components, and deletes token and font license assets.

Changes

Site cleanup

Layer / File(s) Summary
Astro configuration and documentation checks
astro.config.mjs, test/site.test.mjs
Astro uses import.meta.dirname for root. The sidebar and site tests now cover the community-clients page. Tests read public/tokens.css.
UI component surface reduction
src/components/ui/badge.tsx, src/components/ui/button.tsx, src/components/ui/dialog.tsx, src/components/ui/card.tsx, src/components/ui/table.tsx, test/site.test.mjs
The badge, button, and dialog modules expose fewer exports. The Card and Table modules are deleted. The scaffold check no longer requires Card.
Theme and font asset removal
tokens.css, public/fonts/Geist-OFL.txt, public/fonts/OFL.txt
The shared token stylesheet and two font license files are deleted.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 15a3c

This cleanup is merge-ready after normal checks and review; no actionable merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the removal of unused UI components, assets, exports, and token scaffolding.
Docstring Coverage ✅ Passed Docstring check was indeterminate for this PR — some files could not be analyzed in time. Not blocking.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cleanup/ponytail-audit

Comment @coderabbitai help to get the list of available commands.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant