Repository navigation
refactor: remove unused re-exports from pseudo-selectors - #1908
Conversation
Found with knip. src/pseudo-selectors/index.ts re-exported aliases, filters and pseudos, but nothing imports them from there and the package entry point does not expose them. The modules themselves are unchanged and still used by compilePseudoSelector. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe pseudo-selectors index no longer re-exports ChangesPseudo-selector exports
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Refactor Merge Risk: ⚪ Minimal · up to No affected supported consumer or actionable merge risk was found. The change is mergeable after normal checks. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. A rabbit peeks at exports gone, Comment |
There was a problem hiding this comment.
Note
Copilot was unable to run its full agentic suite in this review.
Copilot review overview
0 open findings
What changed in this PR
Removes the barrel re-exports from the src/pseudo-selectors entrypoint, leaving only compilePseudoSelector exported from this module.
Changes:
- Removed re-exports for
aliases,filters, andpseudosfromsrc/pseudo-selectors/index.ts.
| File | Description |
|---|---|
| src/pseudo-selectors/index.ts | Drops module re-exports from the package entrypoint, impacting how consumers import pseudo-selector helpers. |
🧠 Review effort: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Found with knip.
src/pseudo-selectors/index.ts re-exported aliases, filters and pseudos,
but nothing imports them from there and the package entry point does
not expose them. The modules themselves are unchanged and still used by
compilePseudoSelector.
🤖 Generated with Claude Code
Summary by CodeRabbit