eslint-plugin-nextfriday 1.10.0 → 1.10.2

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # eslint-plugin-nextfriday
 
+## 1.10.2
+
+### Patch Changes
+
+- [#51](https://github.com/next-friday/eslint-plugin-nextfriday/pull/51) [`b940720`](https://github.com/next-friday/eslint-plugin-nextfriday/commit/b940720b356ee0bf0da0addac5bd4a35c20b4141) Thanks [@nextfridaydeveloper](https://github.com/nextfridaydeveloper)! - fix(newline-after-multiline-block): exclude import declarations from requiring blank lines between them
+
+  Multi-line imports followed by other imports no longer require a blank line, avoiding conflicts with `import-x/order` rule. Blank lines are still required when a multi-line import is followed by a non-import statement.
+
+## 1.10.1
+
+### Patch Changes
+
+- [#49](https://github.com/next-friday/eslint-plugin-nextfriday/pull/49) [`baecdc9`](https://github.com/next-friday/eslint-plugin-nextfriday/commit/baecdc9810eda995c0ced73bcf1536027feae2e6) Thanks [@nextfridaydeveloper](https://github.com/nextfridaydeveloper)! - fix(no-inline-nested-object): allow arrays with only primitive values to stay inline
+
+  Arrays containing only primitives (literals, identifiers, template literals) are now allowed to be inline. Objects still require multiple lines.
+  - `options: ["primary", "foreground", "danger"]` is now valid
+  - `config: { enabled: true }` still requires multiline
+  - `items: [{ a: 1 }, { b: 2 }]` still requires multiline
+
 ## 1.10.0
 
 ### Minor Changes

package/lib/index.cjs

@@ -40,7 +40,7 @@ module.exports = __toCommonJS(index_exports);
 // package.json
 var package_default = {
   name: "eslint-plugin-nextfriday",
-  version: "1.10.0",
+  version: "1.10.2",
   description: "A comprehensive ESLint plugin providing custom rules and configurations for Next Friday development workflows.",
   keywords: [
     "eslint",
@@ -1049,6 +1049,13 @@ function getInnerExpression(node) {
   }
   return node;
 }
+function arrayContainsOnlyPrimitives(node) {
+  return node.elements.every((el) => {
+    if (el === null) return true;
+    const inner = getInnerExpression(el);
+    return inner.type === import_utils14.AST_NODE_TYPES.Literal || inner.type === import_utils14.AST_NODE_TYPES.Identifier || inner.type === import_utils14.AST_NODE_TYPES.TemplateLiteral || inner.type === import_utils14.AST_NODE_TYPES.UnaryExpression;
+  });
+}
 var noInlineNestedObject = createRule14({
   name: "no-inline-nested-object",
   meta: {
@@ -1081,6 +1088,9 @@ var noInlineNestedObject = createRule14({
         if (elements.length === 0) {
           return;
         }
+        if (valueNode.type === import_utils14.AST_NODE_TYPES.ArrayExpression && arrayContainsOnlyPrimitives(valueNode)) {
+          return;
+        }
         const isMultiline = valueNode.loc.start.line !== valueNode.loc.end.line;
         if (isMultiline) {
           return;
@@ -1366,6 +1376,9 @@ var import_utils20 = require("@typescript-eslint/utils");
 var createRule18 = import_utils20.ESLintUtils.RuleCreator(
   (name) => `https://github.com/next-friday/eslint-plugin-nextfriday/blob/main/docs/rules/${name.replaceAll("-", "_").toUpperCase()}.md`
 );
+function isImportDeclaration(node) {
+  return node.type === import_utils20.AST_NODE_TYPES.ImportDeclaration;
+}
 function checkStatements(statements, context) {
   const { sourceCode } = context;
   statements.forEach((current, index) => {
@@ -1380,6 +1393,9 @@ function checkStatements(statements, context) {
     if (!isMultiline) {
       return;
     }
+    if (isImportDeclaration(current) && isImportDeclaration(next)) {
+      return;
+    }
     const lineGap = next.loc.start.line - current.loc.end.line;
     if (lineGap < 2) {
       context.report({