@d3lm/lint-preset 1.5.5 → 1.5.6

package/dist/comments.d.cts

@@ -75,7 +75,8 @@ interface ParagraphReport {
     /**
      * Holds the exact source lines for `startLine` through `endLine` when the paragraph is
      * greedily wrapped to the width, or `null` when the paragraph isn't prose, e.g. code,
-     * a table, a diagram, or `@example`.
+     * a table, a diagram, or `@example`. A paragraph that is accepted with the layout for a
+     * wider width within the tolerance keeps its lines.
      */
     readonly suggestion: readonly string[] | null;
 }
@@ -214,8 +215,9 @@ interface CommentsConfig {
     readonly singleLineWidth?: number;
     /**
      * Sets how many columns prose lines may go past `width` when that saves a line in a paragraph of
-     * three or more lines. It never goes past `maxWidth`. When `maxWidth` is omitted, it also sets a
-     * legacy hard limit of `width + tolerance`.
+     * three or more lines. A paragraph that already has the layout for a wider `width`, up to `width`
+     * plus this many columns, also passes as it is. It never goes past `maxWidth`. When `maxWidth` is
+     * omitted, it also sets a legacy hard limit of `width + tolerance`.
      */
     readonly tolerance?: number;
     /**

package/dist/comments.d.ts

@@ -75,7 +75,8 @@ interface ParagraphReport {
     /**
      * Holds the exact source lines for `startLine` through `endLine` when the paragraph is
      * greedily wrapped to the width, or `null` when the paragraph isn't prose, e.g. code,
-     * a table, a diagram, or `@example`.
+     * a table, a diagram, or `@example`. A paragraph that is accepted with the layout for a
+     * wider width within the tolerance keeps its lines.
      */
     readonly suggestion: readonly string[] | null;
 }
@@ -214,8 +215,9 @@ interface CommentsConfig {
     readonly singleLineWidth?: number;
     /**
      * Sets how many columns prose lines may go past `width` when that saves a line in a paragraph of
-     * three or more lines. It never goes past `maxWidth`. When `maxWidth` is omitted, it also sets a
-     * legacy hard limit of `width + tolerance`.
+     * three or more lines. A paragraph that already has the layout for a wider `width`, up to `width`
+     * plus this many columns, also passes as it is. It never goes past `maxWidth`. When `maxWidth` is
+     * omitted, it also sets a legacy hard limit of `width + tolerance`.
      */
     readonly tolerance?: number;
     /**

package/dist/comments.js

@@ -1059,9 +1059,9 @@ function analyzeParagraph(paragraph, options) {
     parentheticalOverflow: options.maxWidth === void 0 ? void 0 : options.maxWidth - options.width,
     keepFirstLine: paragraph.kind === "tag"
   };
-  const layOut = (from, settings) => {
-    const even2 = layoutTokens(from, firstLimit, restLimit, settings);
-    return { even: even2, wrapped: breakAfterSentence(even2, firstLimit, restLimit, settings) };
+  const layOut = (from, settings, extra) => {
+    const even2 = layoutTokens(from, firstLimit + extra, restLimit + extra, settings);
+    return { even: even2, wrapped: breakAfterSentence(even2, firstLimit + extra, restLimit + extra, settings) };
   };
   const units = bindLineStarts(paragraph.kind === "tag" ? bindTagHead(tokens) : tokens, startsLine);
   const hardLimit = restLimit + (options.maxWidth === void 0 ? options.tolerance : options.maxWidth - options.width);
@@ -1078,9 +1078,9 @@ function analyzeParagraph(paragraph, options) {
       return { ...first, text, verbatim: false, hardBreak: HARD_BREAK.test(text) };
     });
   };
-  const settle = (settings) => {
+  const settle = (settings, extra = 0) => {
     let current = units;
-    let { even: even2, wrapped: wrapped2 } = layOut(current, settings);
+    let { even: even2, wrapped: wrapped2 } = layOut(current, settings, extra);
     for (; ; ) {
       const body = bodyOf(wrapped2);
       const rewrap = structureOf(body);
@@ -1095,11 +1095,19 @@ function analyzeParagraph(paragraph, options) {
         return { tier: "manual", reason: `rewrap would read as ${reading}` };
       }
       current = current.toSpliced(start - 1, 2, pair);
-      ({ even: even2, wrapped: wrapped2 } = layOut(current, settings));
+      ({ even: even2, wrapped: wrapped2 } = layOut(current, settings, extra));
     }
   };
-  const settled = settle(layoutOptions);
-  const layout = "wrapped" in settled ? settled : settle({ ...layoutOptions, keepSteps: true });
+  const layoutAt = (extra) => {
+    const settings = {
+      ...layoutOptions,
+      tolerance: layoutOptions.tolerance - extra,
+      parentheticalOverflow: layoutOptions.parentheticalOverflow === void 0 ? void 0 : layoutOptions.parentheticalOverflow - extra
+    };
+    const settled = settle(settings, extra);
+    return "wrapped" in settled ? settled : settle({ ...settings, keepSteps: true }, extra);
+  };
+  const layout = layoutAt(0);
   if (!("wrapped" in layout)) {
     return keep(layout);
   }
@@ -1164,10 +1172,22 @@ function analyzeParagraph(paragraph, options) {
       }
     }
   }
-  const suggestion = wrapped.map(
+  const isVerdict = (issue) => ISSUE_TIERS[issue.type] === "verdict";
+  const matchesWiderLayout = () => {
+    for (let extra = 1; extra <= layoutOptions.tolerance; extra += 1) {
+      const wider = layoutAt(extra);
+      if ("wrapped" in wider && sameLayout(wider.wrapped, currentLines)) {
+        return true;
+      }
+    }
+    return false;
+  };
+  const accepted = lineIssues.some((issue) => isVerdict(issue)) && matchesWiderLayout();
+  const suggested = accepted ? currentLines : wrapped;
+  const suggestion = suggested.map(
     (words, position) => `${prefix}${position === 0 ? paragraph.marker : hang}${words.join(" ")}`
   );
-  const wrapIssues = lineIssues.map((issue) => lineIssue(issue.type, paragraph.lines[issue.line], options));
+  const wrapIssues = lineIssues.filter((issue) => !accepted || !isVerdict(issue)).map((issue) => lineIssue(issue.type, paragraph.lines[issue.line], options));
   const issues = byLine([...overflow, ...wrapIssues, ...styleIssues(paragraph, options)]);
   return { ...base, reflowable: true, issues, suggestion };
 }