Skip to content

Skip comments in Ruby files when checking for class names #19243

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
thecrypticace merged 13 commits into from
Nov 13, 2025
+137 −5
Merged

Skip comments in Ruby files when checking for class names #19243

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
5c54516
Skip over strings in Ruby files
thecrypticace Oct 31, 2025
54617bc
Replace comments in Ruby code
thecrypticace Oct 31, 2025
a413e53
Discard arbitrary properties with `!` at the top-level
thecrypticace Oct 31, 2025
956ce18
Fix formatting
thecrypticace Oct 31, 2025
d9163d8
Add tests
thecrypticace Nov 11, 2025
3778c28
Support single line string literal specifiers
thecrypticace Nov 11, 2025
14f888f
Only match expected string literal specifiers
thecrypticace Nov 11, 2025
5662aaf
Fix formatting
thecrypticace Nov 11, 2025
f381118
Update changelog
thecrypticace Nov 12, 2025
7f34f3f
Handle `#` as a string format delimiter
thecrypticace Nov 12, 2025
54d4cac
Remove `!important` thing
thecrypticace Nov 13, 2025
defb469
Update comment
thecrypticace Nov 13, 2025
5ae0787
Update changelog
thecrypticace Nov 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add tests
  • Loading branch information
@thecrypticace
thecrypticace committed Nov 12, 2025
commit d9163d848f6c251dac575b7d6a35185b24c37266
24 changes: 24 additions & 0 deletions crates/oxide/src/extractor/pre_processors/ruby.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -239,9 +239,14 @@ mod tests {
"%w(flex data-[state=pending]:bg-(--my-color) flex-col)",
"%w flex data-[state=pending]:bg-(--my-color) flex-col ",
),

// %w …\n
("%w flex px-2.5\n", "%w flex px-2.5\n"),

// Use backslash to embed spaces in the strings.
(r#"%w[foo\ bar baz\ bat]"#, r#"%w foo bar baz bat "#),
(r#"%W[foo\ bar baz\ bat]"#, r#"%W foo bar baz bat "#),

// The nested delimiters evaluated to a flat array of strings
// (not nested array).
(r#"%w[foo[bar baz]qux]"#, r#"%w foo[bar baz]qux "#),
Expand All @@ -257,6 +262,23 @@ mod tests {
r#"def call = tag.span "Foo", class: %w[rounded-full h-0.75 w-0.75]"#,
r#"def call = tag.span "Foo", class: %w rounded-full h-0.75 w-0.75 "#
),

(r#"%w[foo ' bar]"#, r#"%w foo ' bar "#),
(r#"%w[foo " bar]"#, r#"%w foo " bar "#),
(r#"%W[foo ' bar]"#, r#"%W foo ' bar "#),
(r#"%W[foo " bar]"#, r#"%W foo " bar "#),

(r#"%p foo ' bar "#, r#"%p foo ' bar "#),
(r#"%p foo " bar "#, r#"%p foo " bar "#),

(
"%p has a ' quote\n# this should be removed\n%p has a ' quote",
"%p has a ' quote\n \n%p has a ' quote"
),
(
"%p has a \" quote\n# this should be removed\n%p has a \" quote",
"%p has a \" quote\n \n%p has a \" quote"
),
] {
Ruby::test(input, expected);
}
Expand Down Expand Up @@ -297,6 +319,8 @@ mod tests {

(r#""foo # bar""#, vec!["foo", "bar"]),
(r#"'foo # bar'"#, vec!["foo", "bar"]),

(r#"%w[foo ' bar]"#, vec!["foo", "bar"]),
] {
Ruby::test_extract_contains(input, expected);
}
Expand Down