Skip to content

chore: remove dead composables#4

Merged
Patel230 merged 1 commit into
mainfrom
chore/remove-dead-composables
Jun 8, 2026
Merged

chore: remove dead composables#4
Patel230 merged 1 commit into
mainfrom
chore/remove-dead-composables

Conversation

@Patel230

@Patel230 Patel230 commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Removes ~200 lines of unused code surfaced during a code-organization review.

Deletes three composables imported by nothing, plus the composables/index.js barrel (never imported anywhere; the two composables it re-exported are imported directly by their consumers):

  • useMediaQuery.js
  • useScrollAnimation.js
  • useScrollSpy.js
  • index.js

Verified zero importers repo-wide for all three. Remaining composables (useAnalytics, useFocusTrap, useLightbox, useScrollReveal, useTypingEffect) are all actively used.

CI gate: lint ✅ · format ✅ · test ✅ · build ✅

Summary by CodeRabbit

  • Revert
    • Removed media query detection utilities for responsive design breakpoints and user preferences (mobile, tablet, desktop, dark mode, reduced motion)
    • Removed scroll animation and scroll tracking features
    • Removed composable re-exports from the composables index

Delete three composables that are imported by nothing, plus the unused
composables/index.js barrel (the barrel was never imported anywhere; the
two composables it re-exported are imported directly by their consumers):
- useMediaQuery.js
- useScrollAnimation.js
- useScrollSpy.js
- index.js

~200 lines of unused code. Verified zero importers repo-wide.
lint/format/test/build all pass.
@coderabbitai

coderabbitai Bot commented Jun 8, 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

Run ID: 79794822-ae21-434c-8079-1e9fb5d140a9

📥 Commits

Reviewing files that changed from the base of the PR and between 3fa073c and 0300822.

📒 Files selected for processing (4)
  • src/composables/index.js
  • src/composables/useMediaQuery.js
  • src/composables/useScrollAnimation.js
  • src/composables/useScrollSpy.js
💤 Files with no reviewable changes (4)
  • src/composables/useScrollAnimation.js
  • src/composables/useMediaQuery.js
  • src/composables/index.js
  • src/composables/useScrollSpy.js

📝 Walkthrough

Walkthrough

The PR removes two composable re-exports (useAnalytics and useScrollAnimation) from the composables index file. This change eliminates these utilities from the public API barrel export.

Changes

Composable exports cleanup

Layer / File(s) Summary
Remove composable re-exports from index
src/composables/index.js
Removes useAnalytics and useScrollAnimation re-exports from the composables index, reducing the public composables API.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Three composables fade to dust,
Export lines removed, as is just,
The index now holds fewer names,
A cleanup dance in refactor games!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description provides clear rationale and verification but does not follow the repository's template structure with required sections. Fill out the template with Type of Change, How Has This Been Tested checkboxes, and other required sections to match repository standards.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change: removing unused composables from the codebase.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 chore/remove-dead-composables

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

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

@Patel230 Patel230 merged commit 9df2166 into main Jun 8, 2026
2 checks passed
@Patel230 Patel230 deleted the chore/remove-dead-composables branch June 8, 2026 10:57
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