Handle parser corpus edge cases

Accept comments between JSX attributes so valid TSX files do not fall back to unchanged output.

Keep dynamic quoted Svelte attributes in source-preservation checks without treating them as static scrambling candidates.

Author: praveenperera

rustywind-core/src/jsx_parser.rs

@@ -203,6 +203,15 @@ impl<'a> JsxParser<'a> {
                 return None;
             }
 
+            if remaining(input).starts_with("//") {
+                consume_line_comment(input, "//")?;
+                continue;
+            }
+            if remaining(input).starts_with("/*") {
+                consume_block_comment(input)?;
+                continue;
+            }
+
             if remaining(input).starts_with('{') {
                 consume_character(input, '{')?;
                 self.parse_javascript(input, Some('}'))?;
@@ -436,6 +445,22 @@ mod tests {
         assert_eq!(values(source), Some(vec!["p-4 flex"]));
     }
 
+    #[test]
+    fn comments_can_separate_attributes() {
+        let source = r#"
+            const view = (
+                <Button
+                    onClick={() => submit()}
+                    // keep this prop documented
+                    className="p-4 flex"
+                    /* and this one */ disabled
+                />
+            );
+        "#;
+
+        assert_eq!(values(source), Some(vec!["p-4 flex"]));
+    }
+
     #[test]
     fn generic_and_relational_syntax_is_not_jsx() {
         let source = r#"

tests/tailwind-compare/lib.mjs

@@ -65,16 +65,31 @@ function findReactAttributes(source, kind) {
   return attributes;
 }
 
-function findSvelteAttributes(source) {
+function findStaticSvelteAttributes(source) {
   const ast = parseSvelte(source, { modern: true });
   const attributes = [];
   walk(ast.fragment, (node) => {
     if (
       node.type !== "Attribute" ||
-      !isClassAttributeName(node.name)
+      !isClassAttributeName(node.name) ||
+      node.value?.length !== 1 ||
+      node.value[0].type !== "Text"
     ) {
       return;
     }
+    const { start, end } = node.value[0];
+    const quote = source[start - 1];
+    if ((quote !== '"' && quote !== "'") || source[end] !== quote) return;
+    attributes.push({ end, start });
+  });
+  return attributes;
+}
+
+function findQuotedSvelteAttributes(source) {
+  const ast = parseSvelte(source, { modern: true });
+  const attributes = [];
+  walk(ast.fragment, (node) => {
+    if (node.type !== "Attribute" || !isClassAttributeName(node.name)) return;
     const valueStart = source.indexOf("=", node.name_loc.end.character) + 1;
     if (valueStart === 0 || valueStart >= node.end) return;
     const quoteStart = source
@@ -166,7 +181,7 @@ function findStaticAttributes(source, kind) {
   if (kind === "jsx" || kind === "tsx") {
     attributes = findReactAttributes(source, kind);
   } else if (kind === "svelte") {
-    attributes = findSvelteAttributes(source);
+    attributes = findStaticSvelteAttributes(source);
   } else if (kind === "astro") {
     attributes = findAstroAttributes(source);
   } else {
@@ -194,7 +209,7 @@ function findQuotedAttributes(source, kind) {
   if (kind === "jsx" || kind === "tsx") {
     attributes = findReactAttributes(source, kind);
   } else if (kind === "svelte") {
-    attributes = findSvelteAttributes(source);
+    attributes = findQuotedSvelteAttributes(source);
   } else if (kind === "astro") {
     attributes = findAstroAttributes(source);
   } else {

tests/tailwind-compare/test/lib.test.mjs

@@ -88,6 +88,18 @@ test("permits changes only inside real quoted class attributes", () => {
   );
 });
 
+test("preserves dynamic quoted Svelte attributes without scrambling them", () => {
+  const source = '<div class="p-4 {active ? \'flex\' : \'grid\'}"></div>';
+  const changed = source.replace("p-4", "m-4");
+
+  assert.deepEqual(extractAttributes(source, "svelte"), []);
+  assert.equal(scrambleAttributes(source, "svelte"), source);
+  assert.equal(
+    preservesSourceOutsideAttributes(source, changed, "svelte"),
+    true,
+  );
+});
+
 test("ignores markup text in source comments and string literals", () => {
   const fixtures = [
     {