Refactor comment skipping logic into reusable method

[bartveneman]

Summary

This PR refactors the comment-skipping logic in the lexer into a dedicated _skip_comment() method to improve code maintainability and reduce duplication.

Key Changes

• Extracted the inline comment-skipping logic from the main tokenization loop into a new _skip_comment() method in the Lexer class
• Updated the comment-skipping code path to call the new method instead of duplicating the logic inline
• Added comprehensive test coverage for comment detection in various CSS contexts:
  • Comments in selectors, property values, and at-rule blocks
  • Comments in at-rule preludes (media queries, supports, layer, import, container)
• Updated test imports to include vi from vitest for mocking

Implementation Details

• The _skip_comment() method encapsulates the logic for advancing past /* and */ delimiters and handling the content in between
• This refactoring sets up the foundation for the on_comment callback to be properly invoked during comment processing
• The new tests verify that comments are correctly identified across all CSS rule types and positions

https://claude.ai/code/session_019FWMrZqm4cPwisLctbdiN1

[github-actions]

🎉 Package Size Decrease

📦 Package	📏 Base Size	📏 Source Size	📈 Size Change
@projectwallace/css-parser	45.9 kB	45.9 kB	-42 B

[bartveneman]

The Audit packages check is failing, but not because of this PR. pnpm audit reports 6 advisories (2 high) in brace-expansion < 1.1.19, pulled in transitively via tailwindcss > purgecss > glob > minimatch. It reproduces locally, and this PR only changes src/tokenize.ts and src/parse-options.test.ts, with no dependency changes. No fix is included here; it needs a dependency bump or override in a separate change. Unit tests, build and the other checks are unaffected.

---

Generated by Claude Code

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@projectwallace/css-parser@8abcf15

commit: 8abcf15