oxlint-plugin-arch 0.6.2 → 0.6.3

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.6.3
+
+- Add a trailing comma when `object-multiline` splits an object, so the fix agrees with multiline comma rules.
+
+- Add opt-in `ternary-layout` and `call-arguments-multiline` layout rules with configurable thresholds and indentation. Presets remain unchanged.
+
 ## 0.6.2
 
 - Export `oxlint-plugin-arch/rules/dillon-anti-slop` and `oxlint-plugin-arch/rules/module-policy` with TypeScript declarations. Resolve the preset's three first-party plugins through public package entry points, without changing rules or policy. Document standalone registration and third-party dependency ownership; verify built exports expose every supporting rule.

package/README.md

@@ -6,7 +6,7 @@ Keep a codebase shaped the way you designed it: files in the right folders, expo
 
 - 🗂️ **Filename-aware.** Rules derive expected names from the file they run on (`user-action.create.ts` → `makeUserActionCreate`).
 - 🎛️ **Configurable.** No app names, folders, or libraries are baked in. Globs and policy live in your config.
-- 🛟 **Safe autofix.** Layout and comment rules only touch whitespace or comments. Type-import fixes qualify scope-resolved references and skip unsafe rewrites.
+- 🛟 **Safe autofix.** Layout and comment rules touch whitespace or comments; `object-multiline` and `call-arguments-multiline` can also add trailing commas. Type-import fixes qualify scope-resolved references and skip unsafe rewrites.
 - ⚡ **Fast.** Built on Oxlint's JS plugin API and ESTree, with no extra parser.
 
 ---
@@ -427,7 +427,9 @@ Options that take one regex string (`pattern`, `allowPattern`, `forbid`, `requir
 | Rule | What it enforces |
 | --- | --- |
 | `padding-between-statements` 🔧 | Blank lines around functions and classes, around control flow, and before `return` in longer blocks; `multilineVariables: true` also separates multiline declarations, including exports (default `false`). |
-| `object-multiline` 🔧 | Objects with 3+ properties passed to a call go one property per line (`scope: 'all'` for every object). |
+| `object-multiline` 🔧 | Objects with 3+ properties passed to a call go one property per line with a trailing comma (`scope: 'all'` for every object). |
+| `ternary-layout` 🔧 | Join short conditionals or split long ones before `?` and `:`. [Details ↓](#ternary-layout) |
+| `call-arguments-multiline` 🔧 | Calls and constructors with 3+ arguments go one argument per line. [Details ↓](#call-arguments-multiline) |
 | `key-value-same-line` 🔧 | An object key and the start of its value stay on the same line. |
 | `chain-newline` 🔧 | Long method chains go one call per line. You pick which chains with `groups`. |
 | `call-array-multiline` 🔧 | Arrays passed to `Promise.all` (or callees you list) go one element per line. |
@@ -440,6 +442,18 @@ Options that take one regex string (`pattern`, `allowPattern`, `forbid`, `requir
 
 ## 🔍 Rule details
 
+### ternary-layout
+
+Options: `maxLength` (integer ≥ 1, default `160`) and `indent` (integer ≥ 1, default `2` spaces). The full resulting line, including its prefix and suffix, determines whether a conditional fits. Short multiline conditionals join; long single-line conditionals split before `?` and `:`, one indent deeper than their starting line. Direct template interpolations stay on one line regardless of length.
+
+Conditionals with a conditional parent or operand, comments inside, or multiline operands are skipped. Existing long multiline conditionals are left unchanged.
+
+### call-arguments-multiline
+
+Options: `minArguments` (integer ≥ 2, default `3`) and `indent` (integer ≥ 1, default `2` spaces). Calls and `new` expressions at the threshold put each argument and the closing `)` on separate lines. Arguments use one indent deeper than the line containing `(`. Fixes add a trailing comma, including after spread arguments.
+
+Calls already using separate lines for every argument and `)` are left unchanged, including their indentation and optional trailing comma. Calls containing comments, parenthesized arguments, or multiline arrow/function arguments are skipped. These are code fixes because they can add punctuation. Both new rules are opt-in and absent from presets.
+
 ### File size
 
 `limits.maxFileLines?: number` sets the existing ESLint `max-lines` ceiling on checked safety scopes; default `400`. Blank lines and comments remain skipped. Use native Oxlint overrides or the existing `ruleExclusions` configuration for deliberate per-file exceptions. File size is a ceiling, not a cohesion proof.

package/dist/index.js

@@ -1,7 +1,7 @@
 // src/index.ts
 import { eslintCompatPlugin } from "@oxlint/plugins";
 
-// src/rules/call-array-multiline.ts
+// src/rules/call-arguments-multiline.ts
 import { defineRule } from "@oxlint/plugins";
 
 // src/utils/helpers/ast.ts
@@ -981,7 +981,91 @@ function boundaryResolveTarget(options, file, source) {
   const alias = Object.entries(options.aliases).sort(([a], [b]) => b.length - a.length).find(([prefix]) => source === prefix || source.startsWith(`${prefix}/`));
   return alias ? resolve(alias[1], source.slice(alias[0].length).replace(/^\//, "")).replaceAll("\\", "/") : source.startsWith(".") ? resolve(dirname(file), source).replaceAll("\\", "/") : source.startsWith("@/") && web ? `${web}/${source.slice(2)}` : source.startsWith("#/") && owner ? `${owner}/${source.slice(2)}` : source.startsWith("#services/") && owner ? `${owner}/services/${source.slice(10)}` : source;
 }
+// src/rules/call-arguments-multiline.ts
+function callArgumentsIsCallback(node) {
+  return (node.type === "ArrowFunctionExpression" || node.type === "FunctionExpression") && node.loc.start.line !== node.loc.end.line;
+}
+var callArgumentsMultiline = defineRule({
+  meta: {
+    type: "layout",
+    fixable: "code",
+    schema: [
+      {
+        type: "object",
+        additionalProperties: false,
+        properties: {
+          minArguments: {
+            type: "integer",
+            minimum: 2,
+            default: 3
+          },
+          indent: {
+            type: "integer",
+            minimum: 1,
+            default: 2
+          }
+        }
+      }
+    ],
+    messages: {
+      multiline: "A call with {{min}}+ arguments puts each argument on its own line."
+    }
+  },
+  createOnce(context) {
+    let minArguments = 3;
+    let indent = 2;
+    const check = (node) => {
+      const args = node.arguments;
+      if (args.length < minArguments || args.some(callArgumentsIsCallback) || context.sourceCode.getCommentsInside(node).length > 0) {
+        return;
+      }
+      const text = context.sourceCode.text;
+      const open = text.indexOf("(", (node.typeArguments ?? node.callee).range[1]);
+      const close = node.range[1] - 1;
+      const first = args[0];
+      const last = args[args.length - 1];
+      const gaps = [[open + 1, first.range[0]]];
+      for (let index = 1;index < args.length; index++) {
+        gaps.push([args[index - 1].range[1], args[index].range[0]]);
+      }
+      gaps.push([last.range[1], close]);
+      const between = gaps.map(([start, end]) => text.slice(start, end));
+      if (text[open] !== "(" || text[close] !== ")" || !/^\s*$/.test(between[0]) || between.slice(1, -1).some((gap) => !/^\s*,\s*$/.test(gap)) || !/^\s*,?\s*$/.test(between[between.length - 1])) {
+        return;
+      }
+      if (between.every((gap) => gap.includes(`
+`))) {
+        return;
+      }
+      const baseIndent = layoutLineIndent(text, open);
+      const newline = layoutNewline(text);
+      const argumentIndent = `${baseIndent}${" ".repeat(indent)}`;
+      context.report({
+        node,
+        messageId: "multiline",
+        data: { min: String(minArguments) },
+        fix: (fixer) => gaps.map((gap, index) => {
+          if (index === 0) {
+            return fixer.replaceTextRange(gap, `${newline}${argumentIndent}`);
+          }
+          return fixer.replaceTextRange(gap, index === gaps.length - 1 ? `,${newline}${baseIndent}` : `,${newline}${argumentIndent}`);
+        })
+      });
+    };
+    return {
+      before() {
+        const options2 = optionsFirst(context, {});
+        minArguments = options2.minArguments ?? 3;
+        indent = options2.indent ?? 2;
+      },
+      CallExpression: check,
+      NewExpression: check
+    };
+  }
+});
+
 // src/rules/call-array-multiline.ts
+import { defineRule as defineRule2 } from "@oxlint/plugins";
 function callArrayMultilineCallee(node) {
   if (node.type === "Identifier")
     return { name: node.name, root: node };
@@ -1048,7 +1132,7 @@ function callArrayMultilineEdits(array, source, text, elementIndent, baseIndent,
   });
   return edits;
 }
-var callArrayMultiline = defineRule({
+var callArrayMultiline = defineRule2({
   meta: {
     type: "layout",
     fixable: "whitespace",
@@ -1110,7 +1194,7 @@ var callArrayMultiline = defineRule({
 });
 
 // src/rules/chain-newline.ts
-import { defineRule as defineRule2 } from "@oxlint/plugins";
+import { defineRule as defineRule3 } from "@oxlint/plugins";
 function chainNewlineCollect(node) {
   const links = [];
   let current = node;
@@ -1147,7 +1231,7 @@ function chainNewlineMatches(root, links, group) {
   const rootName = chainNewlineRootIdentifier(root);
   return rootName !== null && optionsPatternsTest(group.rootPatterns, rootName);
 }
-var chainNewline = defineRule2({
+var chainNewline = defineRule3({
   meta: {
     type: "layout",
     fixable: "whitespace",
@@ -1253,7 +1337,7 @@ var chainNewline = defineRule2({
 });
 
 // src/rules/component-props.ts
-import { defineRule as defineRule3 } from "@oxlint/plugins";
+import { defineRule as defineRule4 } from "@oxlint/plugins";
 function unwrapExport(statement) {
   return statement.type === "ExportNamedDeclaration" || statement.type === "ExportDefaultDeclaration" ? statement.declaration : statement;
 }
@@ -1303,7 +1387,7 @@ function isPassThrough(type) {
   const argument = type.typeArguments?.params.length === 1 ? type.typeArguments.params[0] : undefined;
   return name === "ComponentProps" && (argument?.type === "TSTypeQuery" || argument?.type === "TSLiteralType" && argument.literal.type === "Literal" && typeof argument.literal.value === "string");
 }
-var componentProps = defineRule3({
+var componentProps = defineRule4({
   meta: {
     type: "problem",
     schema: [
@@ -1374,8 +1458,8 @@ var componentProps = defineRule3({
 });
 
 // src/rules/declaration-name.ts
-import { defineRule as defineRule4 } from "@oxlint/plugins";
-var declarationName = defineRule4({
+import { defineRule as defineRule5 } from "@oxlint/plugins";
+var declarationName = defineRule5({
   meta: {
     type: "problem",
     schema: [
@@ -1452,8 +1536,8 @@ var declarationName = defineRule4({
 });
 
 // src/rules/export-file-prefix.ts
-import { defineRule as defineRule5 } from "@oxlint/plugins";
-var exportFilePrefix = defineRule5({
+import { defineRule as defineRule6 } from "@oxlint/plugins";
+var exportFilePrefix = defineRule6({
   meta: {
     type: "problem",
     schema: [
@@ -1518,8 +1602,8 @@ var exportFilePrefix = defineRule5({
 });
 
 // src/rules/export-name-pattern.ts
-import { defineRule as defineRule6 } from "@oxlint/plugins";
-var exportNamePattern = defineRule6({
+import { defineRule as defineRule7 } from "@oxlint/plugins";
+var exportNamePattern = defineRule7({
   meta: {
     type: "problem",
     schema: [
@@ -1572,8 +1656,8 @@ var exportNamePattern = defineRule6({
 });
 
 // src/rules/filename-export-name.ts
-import { defineRule as defineRule7 } from "@oxlint/plugins";
-var filenameExportName = defineRule7({
+import { defineRule as defineRule8 } from "@oxlint/plugins";
+var filenameExportName = defineRule8({
   meta: {
     type: "problem",
     schema: [
@@ -1638,8 +1722,8 @@ var filenameExportName = defineRule7({
 });
 
 // src/rules/filename-match.ts
-import { defineRule as defineRule8 } from "@oxlint/plugins";
-var filenameMatch = defineRule8({
+import { defineRule as defineRule9 } from "@oxlint/plugins";
+var filenameMatch = defineRule9({
   meta: {
     type: "problem",
     schema: [
@@ -1676,8 +1760,8 @@ var filenameMatch = defineRule8({
 });
 
 // src/rules/folder-prefix.ts
-import { defineRule as defineRule9 } from "@oxlint/plugins";
-var folderPrefix = defineRule9({
+import { defineRule as defineRule10 } from "@oxlint/plugins";
+var folderPrefix = defineRule10({
   meta: {
     type: "problem",
     schema: [
@@ -1735,7 +1819,7 @@ var folderPrefix = defineRule9({
 });
 
 // src/rules/jsx-attributes-multiline.ts
-import { defineRule as defineRule10 } from "@oxlint/plugins";
+import { defineRule as defineRule11 } from "@oxlint/plugins";
 function jsxAttributesMultilineEdits(node, text, attributeIndent, baseIndent, newline) {
   const first = node.attributes[0];
   const last = node.attributes.at(-1);
@@ -1766,7 +1850,7 @@ function jsxAttributesMultilineEdits(node, text, attributeIndent, baseIndent, ne
   edits.push({ range: closingRange, text: `${newline}${baseIndent}` });
   return edits;
 }
-var jsxAttributesMultiline = defineRule10({
+var jsxAttributesMultiline = defineRule11({
   meta: {
     type: "layout",
     fixable: "whitespace",
@@ -1815,8 +1899,8 @@ var jsxAttributesMultiline = defineRule10({
 });
 
 // src/rules/key-value-same-line.ts
-import { defineRule as defineRule11 } from "@oxlint/plugins";
-var keyValueSameLine = defineRule11({
+import { defineRule as defineRule12 } from "@oxlint/plugins";
+var keyValueSameLine = defineRule12({
   meta: {
     type: "layout",
     fixable: "whitespace",
@@ -1865,7 +1949,7 @@ var keyValueSameLine = defineRule11({
 });
 
 // src/rules/no-comments.ts
-import { defineRule as defineRule12 } from "@oxlint/plugins";
+import { defineRule as defineRule13 } from "@oxlint/plugins";
 var NO_COMMENTS_LINE_GAP = /^[\t \r\n\u2028\u2029]*$/;
 var NO_COMMENTS_LINE_TERMINATOR = /\r\n|[\n\r\u2028\u2029]/;
 var NO_COMMENTS_TRAILING_LINES = /^[\t ]*(?:\r\n|[\n\r\u2028\u2029])(?:[\t ]*(?:\r\n|[\n\r\u2028\u2029]))*/;
@@ -1957,7 +2041,7 @@ function noCommentsJsxEmptyContainer(containers, start, end) {
   const container = containers[low - 1];
   return container?.range[1] >= end ? container : undefined;
 }
-var noComments = defineRule12({
+var noComments = defineRule13({
   meta: {
     type: "suggestion",
     fixable: "code",
@@ -2089,8 +2173,8 @@ var noComments = defineRule12({
 });
 
 // src/rules/no-extra-exports.ts
-import { defineRule as defineRule13 } from "@oxlint/plugins";
-var noExtraExports = defineRule13({
+import { defineRule as defineRule14 } from "@oxlint/plugins";
+var noExtraExports = defineRule14({
   meta: {
     type: "problem",
     schema: [
@@ -2149,7 +2233,7 @@ var noExtraExports = defineRule13({
 });
 
 // src/rules/no-extra-factory-keys.ts
-import { defineRule as defineRule14 } from "@oxlint/plugins";
+import { defineRule as defineRule15 } from "@oxlint/plugins";
 function compileFactoryKeys(pattern, keys, requireKeys) {
   const allowedList = keys.concat(requireKeys.filter((key) => !keys.includes(key)));
   return {
@@ -2159,7 +2243,7 @@ function compileFactoryKeys(pattern, keys, requireKeys) {
     requireKeys
   };
 }
-var noExtraFactoryKeys = defineRule14({
+var noExtraFactoryKeys = defineRule15({
   meta: {
     type: "problem",
     schema: [
@@ -2263,8 +2347,8 @@ var noExtraFactoryKeys = defineRule14({
 });
 
 // src/rules/no-file-level-helpers.ts
-import { defineRule as defineRule15 } from "@oxlint/plugins";
-var noFileLevelHelpers = defineRule15({
+import { defineRule as defineRule16 } from "@oxlint/plugins";
+var noFileLevelHelpers = defineRule16({
   meta: {
     type: "problem",
     schema: [
@@ -2310,7 +2394,7 @@ var noFileLevelHelpers = defineRule15({
 });
 
 // src/rules/no-imported-type-alias.ts
-import { defineRule as defineRule16 } from "@oxlint/plugins";
+import { defineRule as defineRule17 } from "@oxlint/plugins";
 function importedAliasTarget(declaration, imports) {
   if (declaration?.type !== "TSTypeAliasDeclaration" || declaration.typeAnnotation.type !== "TSTypeReference" || declaration.typeAnnotation.typeName.type !== "Identifier" || Boolean(declaration.typeAnnotation.typeArguments?.params.length) || !imports.has(declaration.typeAnnotation.typeName.name)) {
     return;
@@ -2320,7 +2404,7 @@ function importedAliasTarget(declaration, imports) {
 function specifierLocalName(specifier) {
   return specifier.local.type === "Identifier" ? specifier.local.name : specifier.local.value;
 }
-var noImportedTypeAlias = defineRule16({
+var noImportedTypeAlias = defineRule17({
   meta: {
     type: "problem",
     schema: [],
@@ -2394,8 +2478,8 @@ var noImportedTypeAlias = defineRule16({
 });
 
 // src/rules/no-inline-schema-elements.ts
-import { defineRule as defineRule17 } from "@oxlint/plugins";
-var noInlineSchemaElements = defineRule17({
+import { defineRule as defineRule18 } from "@oxlint/plugins";
+var noInlineSchemaElements = defineRule18({
   meta: {
     type: "problem",
     schema: [
@@ -2456,7 +2540,7 @@ var noInlineSchemaElements = defineRule17({
 });
 
 // src/rules/no-inline-types.ts
-import { defineRule as defineRule18 } from "@oxlint/plugins";
+import { defineRule as defineRule19 } from "@oxlint/plugins";
 function parameterAnnotation(parameter) {
   if (parameter.type === "TSParameterProperty") {
     return parameterAnnotation(parameter.parameter);
@@ -2466,7 +2550,7 @@ function parameterAnnotation(parameter) {
   }
   return parameter.typeAnnotation;
 }
-var noInlineTypes = defineRule18({
+var noInlineTypes = defineRule19({
   meta: {
     type: "problem",
     schema: [
@@ -2536,7 +2620,7 @@ var noInlineTypes = defineRule18({
 });
 
 // src/rules/no-literal-in.ts
-import { defineRule as defineRule19 } from "@oxlint/plugins";
+import { defineRule as defineRule20 } from "@oxlint/plugins";
 function literalKey(node) {
   if (node.type === "Literal" && typeof node.value === "string") {
     return node.value;
@@ -2545,7 +2629,7 @@ function literalKey(node) {
     return node.quasis[0]?.value.cooked ?? node.quasis[0]?.value.raw;
   }
 }
-var noLiteralIn = defineRule19({
+var noLiteralIn = defineRule20({
   meta: {
     type: "suggestion",
     schema: [
@@ -2586,13 +2670,13 @@ var noLiteralIn = defineRule19({
 });
 
 // src/rules/no-local-schema-construction.ts
-import { defineRule as defineRule20 } from "@oxlint/plugins";
+import { defineRule as defineRule21 } from "@oxlint/plugins";
 var defaultRuntimeImportMessage = "Do not import runtime schema constructors from {{source}}; import a named schema instead.";
 var defaultConstructionMessage = "Do not construct schemas locally; import a named schema instead.";
 function formatMessage(template, data) {
   return Object.entries(data).reduce((result, [key, value]) => result.replaceAll(`{{${key}}}`, value), template);
 }
-var noLocalSchemaConstruction = defineRule20({
+var noLocalSchemaConstruction = defineRule21({
   meta: {
     type: "problem",
     schema: [
@@ -2714,8 +2798,8 @@ var noLocalSchemaConstruction = defineRule20({
 });
 
 // src/rules/no-member-comments.ts
-import { defineRule as defineRule21 } from "@oxlint/plugins";
-var noMemberComments = defineRule21({
+import { defineRule as defineRule22 } from "@oxlint/plugins";
+var noMemberComments = defineRule22({
   meta: {
     type: "suggestion",
     fixable: "code",
@@ -2811,7 +2895,7 @@ var noMemberComments = defineRule21({
 });
 
 // src/rules/no-module-mutable-state.ts
-import { defineRule as defineRule22 } from "@oxlint/plugins";
+import { defineRule as defineRule23 } from "@oxlint/plugins";
 function moduleMutableStateBindings(pattern) {
   if (pattern.type === "Identifier") {
     return [{ name: pattern.name, node: pattern }];
@@ -2850,7 +2934,7 @@ function moduleMutableStateInsideAmbient(node) {
   }
   return false;
 }
-var noModuleMutableState = defineRule22({
+var noModuleMutableState = defineRule23({
   meta: {
     type: "problem",
     schema: [
@@ -2902,7 +2986,7 @@ var noModuleMutableState = defineRule22({
 });
 
 // src/rules/no-promise-all-mutation.ts
-import { defineRule as defineRule23 } from "@oxlint/plugins";
+import { defineRule as defineRule24 } from "@oxlint/plugins";
 function mutationBinding(node) {
   if (node.type === "Identifier") {
     return node;
@@ -2930,7 +3014,7 @@ function mutationBindings(node) {
     }
   }
 }
-var noPromiseAllMutation = defineRule23({
+var noPromiseAllMutation = defineRule24({
   meta: {
     type: "problem",
     schema: [
@@ -3030,8 +3114,8 @@ var noPromiseAllMutation = defineRule23({
 });
 
 // src/rules/no-rederive-schema.ts
-import { defineRule as defineRule24 } from "@oxlint/plugins";
-var noRederiveSchema = defineRule24({
+import { defineRule as defineRule25 } from "@oxlint/plugins";
+var noRederiveSchema = defineRule25({
   meta: {
     type: "problem",
     schema: [
@@ -3075,7 +3159,7 @@ var noRederiveSchema = defineRule24({
 });
 
 // src/rules/no-restricted-constructor.ts
-import { defineRule as defineRule25 } from "@oxlint/plugins";
+import { defineRule as defineRule26 } from "@oxlint/plugins";
 function restrictedConstructorEntries(constructors) {
   const entries = new Map;
   for (const item of constructors) {
@@ -3091,7 +3175,7 @@ function restrictedConstructorEntries(constructors) {
   }
   return entries;
 }
-var noRestrictedConstructor = defineRule25({
+var noRestrictedConstructor = defineRule26({
   meta: {
     type: "problem",
     schema: [
@@ -3166,8 +3250,8 @@ var noRestrictedConstructor = defineRule25({
 });
 
 // src/rules/no-restricted-files.ts
-import { defineRule as defineRule26 } from "@oxlint/plugins";
-var noRestrictedFiles = defineRule26({
+import { defineRule as defineRule27 } from "@oxlint/plugins";
+var noRestrictedFiles = defineRule27({
   meta: {
     type: "problem",
     schema: [
@@ -3199,7 +3283,7 @@ var noRestrictedFiles = defineRule26({
 });
 
 // src/rules/no-restricted-token.ts
-import { defineRule as defineRule27 } from "@oxlint/plugins";
+import { defineRule as defineRule28 } from "@oxlint/plugins";
 function tokenIsTypeOnlyImport(node) {
   const parent = node.parent;
   if (parent?.type === "ImportSpecifier") {
@@ -3210,7 +3294,7 @@ function tokenIsTypeOnlyImport(node) {
 function memberMatches(node, restriction) {
   return restriction.startsWith("*.") ? astStaticMemberName(node) === restriction.slice(2) : astDottedName(node) === restriction;
 }
-var noRestrictedToken = defineRule27({
+var noRestrictedToken = defineRule28({
   meta: {
     type: "problem",
     schema: [
@@ -3307,7 +3391,7 @@ var noRestrictedToken = defineRule27({
 });
 
 // src/rules/no-runtime-in-types.ts
-import { defineRule as defineRule28 } from "@oxlint/plugins";
+import { defineRule as defineRule29 } from "@oxlint/plugins";
 function isTypeOnlyImport(statement) {
   return statement.importKind === "type" || statement.specifiers.length > 0 && statement.specifiers.every((specifier) => specifier.type === "ImportSpecifier" && specifier.importKind === "type");
 }
@@ -3524,7 +3608,7 @@ function isRuntimeReExport(statement, runtimeImportLocals, valueNames, isAmbient
   }
   return statement.specifiers.some((specifier) => specifier.exportKind !== "type" && runtimeImportLocals.has(exportLocalName(specifier)));
 }
-var noRuntimeInTypes = defineRule28({
+var noRuntimeInTypes = defineRule29({
   meta: {
     type: "problem",
     schema: [
@@ -3668,7 +3752,7 @@ var noRuntimeInTypes = defineRule28({
 });
 
 // src/rules/no-single-use-scalar-schema.ts
-import { defineRule as defineRule29 } from "@oxlint/plugins";
+import { defineRule as defineRule30 } from "@oxlint/plugins";
 function isScalarSchema(node, namespaces, structuralMethods) {
   if (node.type !== "CallExpression" || node.callee.type !== "MemberExpression") {
     return false;
@@ -3702,7 +3786,7 @@ function isAllowedStructuralElement(identifier, initializer, ancestors, options2
   const method = call.callee.type === "MemberExpression" ? astStaticMemberName(call.callee) : undefined;
   return Boolean(method && options2.methods.includes(method) && schemasElements(call).includes(identifier));
 }
-var noSingleUseScalarSchema = defineRule29({
+var noSingleUseScalarSchema = defineRule30({
   meta: {
     type: "suggestion",
     schema: [
@@ -3787,8 +3871,8 @@ var noSingleUseScalarSchema = defineRule29({
 });
 
 // src/rules/no-top-level-functions.ts
-import { defineRule as defineRule30 } from "@oxlint/plugins";
-var noTopLevelFunctions = defineRule30({
+import { defineRule as defineRule31 } from "@oxlint/plugins";
+var noTopLevelFunctions = defineRule31({
   meta: {
     type: "problem",
     schema: [
@@ -3847,7 +3931,7 @@ var noTopLevelFunctions = defineRule30({
 });
 
 // src/rules/no-trivial-functions.ts
-import { defineRule as defineRule31 } from "@oxlint/plugins";
+import { defineRule as defineRule32 } from "@oxlint/plugins";
 
 // src/constants/trivial.constants.ts
 var trivialBannedNames = [
@@ -3863,7 +3947,7 @@ var trivialBannedNames = [
 ];
 
 // src/rules/no-trivial-functions.ts
-var noTrivialFunctions = defineRule31({
+var noTrivialFunctions = defineRule32({
   meta: {
     type: "problem",
     schema: [
@@ -3980,8 +4064,8 @@ var noTrivialFunctions = defineRule31({
 });
 
 // src/rules/no-type-declarations.ts
-import { defineRule as defineRule32 } from "@oxlint/plugins";
-var noTypeDeclarations = defineRule32({
+import { defineRule as defineRule33 } from "@oxlint/plugins";
+var noTypeDeclarations = defineRule33({
   meta: {
     type: "problem",
     schema: [
@@ -4026,8 +4110,8 @@ var noTypeDeclarations = defineRule32({
 });
 
 // src/rules/no-unescaped-like.ts
-import { defineRule as defineRule33 } from "@oxlint/plugins";
-var noUnescapedLike = defineRule33({
+import { defineRule as defineRule34 } from "@oxlint/plugins";
+var noUnescapedLike = defineRule34({
   meta: {
     type: "problem",
     schema: [
@@ -4078,7 +4162,7 @@ var noUnescapedLike = defineRule33({
 });
 
 // src/rules/object-multiline.ts
-import { defineRule as defineRule34 } from "@oxlint/plugins";
+import { defineRule as defineRule35 } from "@oxlint/plugins";
 function objectMultilineIsCallArg(node) {
   const parent = node.parent;
   return (parent.type === "CallExpression" || parent.type === "NewExpression") && parent.arguments.includes(node);
@@ -4111,14 +4195,14 @@ function objectMultilineGaps(text, node) {
     if (!objectMultilineIsWhitespace(between.slice(0, comma)) || !objectMultilineIsWhitespace(between.slice(comma + 1))) {
       return null;
     }
-    gaps.push([after + comma + 1, limit]);
+    gaps.push([next ? after + comma + 1 : after, limit]);
   }
   return gaps;
 }
-var objectMultiline = defineRule34({
+var objectMultiline = defineRule35({
   meta: {
     type: "layout",
-    fixable: "whitespace",
+    fixable: "code",
     schema: [
       {
         type: "object",
@@ -4168,7 +4252,7 @@ var objectMultiline = defineRule34({
             if (!gaps) {
               return null;
             }
-            return gaps.map((gap, index) => fixer.replaceTextRange(gap, `${newline}${index === gaps.length - 1 ? baseIndent : propIndent}`));
+            return gaps.map((gap, index) => fixer.replaceTextRange(gap, index === gaps.length - 1 ? `,${newline}${baseIndent}` : `${newline}${propIndent}`));
           }
         });
       }
@@ -4177,8 +4261,8 @@ var objectMultiline = defineRule34({
 });
 
 // src/rules/only-export-components.ts
-import { defineRule as defineRule35 } from "@oxlint/plugins";
-var onlyExportComponents = defineRule35({
+import { defineRule as defineRule36 } from "@oxlint/plugins";
+var onlyExportComponents = defineRule36({
   meta: {
     type: "problem",
     schema: [
@@ -4258,7 +4342,7 @@ var onlyExportComponents = defineRule35({
 });
 
 // src/rules/only-export-constants.ts
-import { defineRule as defineRule36 } from "@oxlint/plugins";
+import { defineRule as defineRule37 } from "@oxlint/plugins";
 function bindingNames(pattern) {
   switch (pattern.type) {
     case "Identifier":
@@ -4313,7 +4397,7 @@ function collectBindings(program) {
   }
   return bindings;
 }
-var onlyExportConstants = defineRule36({
+var onlyExportConstants = defineRule37({
   meta: {
     type: "problem",
     schema: [
@@ -4401,7 +4485,7 @@ var onlyExportConstants = defineRule36({
 });
 
 // src/rules/padding-between-statements.ts
-import { defineRule as defineRule37 } from "@oxlint/plugins";
+import { defineRule as defineRule38 } from "@oxlint/plugins";
 function paddingKind(node) {
   if (node.type === "ExportNamedDeclaration" || node.type === "ExportDefaultDeclaration") {
     return node.declaration ? paddingKind(node.declaration) : "other";
@@ -4453,7 +4537,7 @@ function paddingBlankLineFix(sourceText, comments2, prev, curr, newline) {
   }
   return { newlineEnd, newline };
 }
-var paddingBetweenStatements = defineRule37({
+var paddingBetweenStatements = defineRule38({
   meta: {
     type: "layout",
     fixable: "whitespace",
@@ -4527,7 +4611,7 @@ var paddingBetweenStatements = defineRule37({
 });
 
 // src/rules/prefer-namespace-type-import.ts
-import { defineRule as defineRule38 } from "@oxlint/plugins";
+import { defineRule as defineRule39 } from "@oxlint/plugins";
 var bindingName = /^[A-Za-z_$][\w$]*$/;
 var reservedNames = new Set("await break case catch class const continue debugger default delete do else enum export extends false finally for function if import in instanceof new null return super switch this throw true try typeof var void while with yield implements interface let package private protected public static arguments eval".split(" "));
 function namespaceName(source, names) {
@@ -4551,7 +4635,7 @@ function safeReference(node) {
   }
   return parent?.type === "TSTypeReference" && parent.typeName === current || parent?.type === "TSTypeQuery" && parent.exprName === current || (parent?.type === "TSInterfaceHeritage" || parent?.type === "TSClassImplements") && parent.expression === current;
 }
-var preferNamespaceTypeImport = defineRule38({
+var preferNamespaceTypeImport = defineRule39({
   meta: {
     type: "suggestion",
     fixable: "code",
@@ -4668,8 +4752,8 @@ var preferNamespaceTypeImport = defineRule38({
 });
 
 // src/rules/require-file-factory.ts
-import { defineRule as defineRule39 } from "@oxlint/plugins";
-var requireFileFactory = defineRule39({
+import { defineRule as defineRule40 } from "@oxlint/plugins";
+var requireFileFactory = defineRule40({
   meta: {
     type: "problem",
     schema: [
@@ -4717,8 +4801,8 @@ var requireFileFactory = defineRule39({
 });
 
 // src/rules/require-object-params.ts
-import { defineRule as defineRule40 } from "@oxlint/plugins";
-var requireObjectParams = defineRule40({
+import { defineRule as defineRule41 } from "@oxlint/plugins";
+var requireObjectParams = defineRule41({
   meta: {
     type: "problem",
     schema: [
@@ -4776,8 +4860,8 @@ var requireObjectParams = defineRule40({
 });
 
 // src/rules/require-orpc-output.ts
-import { defineRule as defineRule41 } from "@oxlint/plugins";
-var requireOrpcOutput = defineRule41({
+import { defineRule as defineRule42 } from "@oxlint/plugins";
+var requireOrpcOutput = defineRule42({
   meta: {
     type: "problem",
     schema: [
@@ -4825,8 +4909,8 @@ var requireOrpcOutput = defineRule41({
 });
 
 // src/rules/require-paired-call.ts
-import { defineRule as defineRule42 } from "@oxlint/plugins";
-var requirePairedCall = defineRule42({
+import { defineRule as defineRule43 } from "@oxlint/plugins";
+var requirePairedCall = defineRule43({
   meta: {
     type: "problem",
     schema: [
@@ -4900,8 +4984,8 @@ var requirePairedCall = defineRule42({
 });
 
 // src/rules/route-surface.ts
-import { defineRule as defineRule43 } from "@oxlint/plugins";
-var routeSurface = defineRule43({
+import { defineRule as defineRule44 } from "@oxlint/plugins";
+var routeSurface = defineRule44({
   meta: {
     type: "problem",
     schema: [
@@ -4946,8 +5030,102 @@ var routeSurface = defineRule43({
   }
 });
 
+// src/rules/ternary-layout.ts
+import { defineRule as defineRule45 } from "@oxlint/plugins";
+function ternaryLayoutLineStart(text, offset) {
+  return text.lastIndexOf(`
+`, offset - 1) + 1;
+}
+function ternaryLayoutLineEnd(text, offset) {
+  const end = text.indexOf(`
+`, offset);
+  if (end === -1) {
+    return text.length;
+  }
+  return text[end - 1] === "\r" ? end - 1 : end;
+}
+function ternaryLayoutBreak(between, operator, newline) {
+  const at = between.indexOf(operator);
+  return `${between.slice(0, at).trimEnd()}${newline}${operator} ${between.slice(at + 1).trimStart()}`;
+}
+var ternaryLayout = defineRule45({
+  meta: {
+    type: "layout",
+    fixable: "whitespace",
+    schema: [
+      {
+        type: "object",
+        additionalProperties: false,
+        properties: {
+          maxLength: {
+            type: "integer",
+            minimum: 1,
+            default: 160
+          },
+          indent: {
+            type: "integer",
+            minimum: 1,
+            default: 2
+          }
+        }
+      }
+    ],
+    messages: {
+      join: "Keep this conditional on one line ({{length}} columns; limit {{max}}, except in template literals).",
+      split: "This conditional does not fit on one line ({{length}} of {{max}} columns); break before `?` and `:`."
+    }
+  },
+  createOnce(context) {
+    let maxLength = 160;
+    let indent = 2;
+    return {
+      before() {
+        const options2 = optionsFirst(context, {});
+        maxLength = options2.maxLength ?? 160;
+        indent = options2.indent ?? 2;
+      },
+      ConditionalExpression(node) {
+        const source = context.sourceCode;
+        const text = source.text;
+        const operands = [node.test, node.consequent, node.alternate];
+        if (node.parent.type === "ConditionalExpression" || operands.some((operand) => operand.type === "ConditionalExpression" || operand.loc.start.line !== operand.loc.end.line) || source.getCommentsInside(node).length > 0) {
+          return;
+        }
+        const prefix = text.slice(ternaryLayoutLineStart(text, node.range[0]), node.range[0]);
+        const suffix = text.slice(node.range[1], ternaryLayoutLineEnd(text, node.range[1]));
+        const joined = text.slice(node.range[0], node.range[1]).replace(/\s*\n\s*/g, " ");
+        const length = prefix.length + joined.length + suffix.length;
+        const onOneLine = node.loc.start.line === node.loc.end.line;
+        const interpolated = node.parent.type === "TemplateLiteral";
+        if (!onOneLine && (interpolated || length <= maxLength)) {
+          context.report({
+            node,
+            messageId: "join",
+            data: { length: String(length), max: String(maxLength) },
+            fix: (fixer) => fixer.replaceTextRange(node.range, joined)
+          });
+        }
+        if (onOneLine && !interpolated && length > maxLength) {
+          const newline = `${layoutNewline(text)}${layoutLineIndent(text, node.range[0])}${" ".repeat(indent)}`;
+          const question = [node.test.range[1], node.consequent.range[0]];
+          const colon = [node.consequent.range[1], node.alternate.range[0]];
+          context.report({
+            node,
+            messageId: "split",
+            data: { length: String(length), max: String(maxLength) },
+            fix: (fixer) => [
+              fixer.replaceTextRange(question, ternaryLayoutBreak(text.slice(...question), "?", newline)),
+              fixer.replaceTextRange(colon, ternaryLayoutBreak(text.slice(...colon), ":", newline))
+            ]
+          });
+        }
+      }
+    };
+  }
+});
+
 // src/rules/test-title-pattern.ts
-import { defineRule as defineRule44 } from "@oxlint/plugins";
+import { defineRule as defineRule46 } from "@oxlint/plugins";
 var TEST_MODIFIERS = new Set(["only", "skip", "concurrent", "sequential", "todo", "each", "for"]);
 function isConfiguredName(name, callees) {
   if (!name) {
@@ -4987,7 +5165,7 @@ function regexMatches(regex, value) {
 function titlePatterns(pattern, flags) {
   return pattern === undefined ? [] : [pattern].flat().map((source) => ({ regex: new RegExp(source, flags), pattern: source }));
 }
-var testTitlePattern = defineRule44({
+var testTitlePattern = defineRule46({
   meta: {
     type: "suggestion",
     schema: [
@@ -5066,6 +5244,7 @@ var testTitlePattern = defineRule44({
 var src_default = eslintCompatPlugin({
   meta: { name: "arch" },
   rules: {
+    "call-arguments-multiline": callArgumentsMultiline,
     "call-array-multiline": callArrayMultiline,
     "chain-newline": chainNewline,
     "component-props": componentProps,
@@ -5109,6 +5288,7 @@ var src_default = eslintCompatPlugin({
     "require-orpc-output": requireOrpcOutput,
     "require-paired-call": requirePairedCall,
     "route-surface": routeSurface,
+    "ternary-layout": ternaryLayout,
     "test-title-pattern": testTitlePattern
   }
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oxlint-plugin-arch",
-  "version": "0.6.2",
+  "version": "0.6.3",
   "description": "Configurable architecture rules for Oxlint JavaScript plugins.",
   "type": "module",
   "main": "./dist/index.js",