kensington-eslint-plugin 0.3.1 → 0.3.2

package/README.md

@@ -470,7 +470,7 @@ Auto-fixable when the group's members are contiguous in the source. Non-contiguo
 
 ### `prefer-array-for-multiline-content`
 
-Mirrors what `html-to-kensington` emits: when a tag's content can't fit on the same line as the opening paren, it goes in an array, even when it's the only item. The array form makes line-by-line edits easier (no need to add `[ ]` when adding a sibling).
+Mirrors what `html-to-kensington` emits: when a tag's content occupies its own line(s) — separated from both the opening and closing paren — it goes in an array, even when it's the only item. The array form makes line-by-line edits easier (no need to add `[ ]` when adding a sibling).
 
 ```js
 // Bad
@@ -482,6 +482,12 @@ t.div({ class: 'x' },
 t.div({ class: 'x' }, [
   t.p('only'),
 ]);
+
+// Also good — content trails on the closing-paren line, no array needed
+t.a({
+  href: 'https://example.com',
+  target: '_blank',
+}, 'VS Code');
 ```
 
 Auto-fixable. Single-line calls (`t.div({…}, t.p('inner'))`) are left alone.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kensington-eslint-plugin",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "ESLint rules for kensington signal correctness",
   "type": "module",
   "main": "index.js",

package/rules/prefer-array-for-multiline-content.js

@@ -1,6 +1,8 @@
-// Tag content on a separate line from the call's opening paren must be wrapped
-// in an array, even when it's the only item. Mirrors what html-to-kensington
-// emits when a tag's content can't fit on a single line.
+// Tag content that occupies its own line(s) — separated from both the call's
+// opening paren and its closing paren — must be wrapped in an array, even when
+// it's the only item. Mirrors what html-to-kensington emits when a tag's
+// content can't fit on a single line. Content that trails on the closing-paren
+// line stays bare.
 
 import { isTagCall, getObjectNames, objectNamesSchema } from './_utils.js';
 
@@ -48,15 +50,18 @@ export default {
         if (content.type === 'SpreadElement') { return; }
 
         // The "opening line" is where the open paren sits — same line as the
-        // callee's last token. Content on that line stays bare.
+        // callee's last token. Content that touches either the opening-paren
+        // line or the closing-paren line stays bare.
         const openParenLine = node.callee.loc.end.line;
         if (content.loc.start.line <= openParenLine) { return; }
 
+        const closeParen = sourceCode.getLastToken(node);
+        if (closeParen && closeParen.value === ')' && content.loc.end.line >= closeParen.loc.start.line) { return; }
+
         context.report({
           node: content,
           messageId: 'wrapInArray',
           *fix(fixer) {
-            const closeParen = sourceCode.getLastToken(node);
             const tokenAfterContent = sourceCode.getTokenAfter(content);
             const hasTrailingComma = tokenAfterContent
               && tokenAfterContent.value === ','