@eslint-sh/eslint-plugin 0.7.0 → 0.8.0

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@eslint-sh/eslint-plugin",
   "description": "Shell script linting for ESLint, with built-in policy rules, shared presets, and custom rules over a typed shell AST",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "author": "Mike North <michael.l.north@gmail.com>",
   "dependencies": {
     "@eslint/core": "^1.2.1",
     "@eslint/plugin-kit": "^0.7.2",
-    "sh-ast": "^0.7.0"
+    "sh-ast": "^0.9.0"
   },
   "devDependencies": {
     "eslint": "^10.7.0"

package/dist/language/end-of-input-marker.d.ts

@@ -1,29 +0,0 @@
-/**
- * Detects a source whose U+0080 made the parser silently drop text.
- *
- * mvdan/sh's reader uses U+0080 (`utf8.RuneSelf`) as its internal end-of-input marker. In some
- * positions the parser reads it as an ordinary character and returns the whole tree; in others
- * it stops there and returns a tree for only the text before it, with no error. A linter that
- * accepted the second kind would skip every rule on the lost text, so those sources must be
- * refused while the first kind keeps linting.
- *
- * The two are told apart by asking the parser what it does with the same source when U+0080 is
- * an ordinary character: the parse is complete exactly when the printed tree is the same as the
- * printed tree of the source with every U+0080 swapped for `X`. Text the parser dropped is
- * missing from the first print and present in the second. Nothing is inferred from tree
- * extents, so a trailing comment or heredoc body cannot make a truncated file look complete.
- */
-import { type ParseOptions } from 'sh-ast';
-/**
- * The UTF-16 index of the first U+0080 whose presence makes the parser drop text, or
- * `undefined` when `text` holds none or the parser handles every one of them completely.
- *
- * A source the parser rejects with an error is also `undefined`: that is a fatal parse error the
- * caller reports through its normal path, not a silent loss. Every other doubt is a refusal:
- * a source that parses but prints differently from its swapped counterpart is refused, and so is
- * one whose swapped counterpart no longer parses, so an undecidable position is never accepted.
- *
- * @param text - The source about to be parsed.
- * @param options - The parse options the caller will use, so the dialect matches.
- */
-export declare function firstDroppedTextIndex(text: string, options: ParseOptions): number | undefined;

package/dist/language/end-of-input-marker.js

@@ -1,88 +0,0 @@
-/**
- * Detects a source whose U+0080 made the parser silently drop text.
- *
- * mvdan/sh's reader uses U+0080 (`utf8.RuneSelf`) as its internal end-of-input marker. In some
- * positions the parser reads it as an ordinary character and returns the whole tree; in others
- * it stops there and returns a tree for only the text before it, with no error. A linter that
- * accepted the second kind would skip every rule on the lost text, so those sources must be
- * refused while the first kind keeps linting.
- *
- * The two are told apart by asking the parser what it does with the same source when U+0080 is
- * an ordinary character: the parse is complete exactly when the printed tree is the same as the
- * printed tree of the source with every U+0080 swapped for `X`. Text the parser dropped is
- * missing from the first print and present in the second. Nothing is inferred from tree
- * extents, so a trailing comment or heredoc body cannot make a truncated file look complete.
- */
-import { parseSync, printSync } from 'sh-ast';
-const END_OF_INPUT_MARKER = '\u0080';
-/** An ordinary one-UTF-16-unit word character, so offsets are the same in both parses. */
-const ORDINARY_CHARACTER = 'X';
-/** What the parser makes of a source: the printed tree, or `ERROR` when it does not parse. */
-const ERROR = Symbol('parse error');
-/**
- * Prints `text` the way the parser sees it. Every U+0080 in the result is swapped for the
- * ordinary character so the print of the original can be compared with the print of the
- * swapped source.
- */
-function printed(text, options) {
-    try {
-        const output = printSync(parseSync(text, options));
-        const decoded = typeof output === 'string' ? output : new TextDecoder().decode(output);
-        return decoded.replaceAll(END_OF_INPUT_MARKER, ORDINARY_CHARACTER);
-    }
-    catch {
-        return ERROR;
-    }
-}
-/**
- * The UTF-16 index of the first U+0080 whose presence makes the parser drop text, or
- * `undefined` when `text` holds none or the parser handles every one of them completely.
- *
- * A source the parser rejects with an error is also `undefined`: that is a fatal parse error the
- * caller reports through its normal path, not a silent loss. Every other doubt is a refusal:
- * a source that parses but prints differently from its swapped counterpart is refused, and so is
- * one whose swapped counterpart no longer parses, so an undecidable position is never accepted.
- *
- * @param text - The source about to be parsed.
- * @param options - The parse options the caller will use, so the dialect matches.
- */
-export function firstDroppedTextIndex(text, options) {
-    const indices = [];
-    for (let at = text.indexOf(END_OF_INPUT_MARKER); at !== -1; at = text.indexOf(END_OF_INPUT_MARKER, at + 1)) {
-        indices.push(at);
-    }
-    if (indices.length === 0)
-        return undefined;
-    const ordinary = text.replaceAll(END_OF_INPUT_MARKER, ORDINARY_CHARACTER);
-    const expected = printed(ordinary, options);
-    const actual = printed(text, options);
-    if (actual === ERROR)
-        return undefined;
-    if (expected !== ERROR && actual === expected)
-        return undefined;
-    /** Whether keeping only the first `count` U+0080 characters (the rest swapped) still differs. */
-    const dropsText = (count) => {
-        let candidate = '';
-        let from = 0;
-        for (const index of indices.slice(0, count)) {
-            candidate += ordinary.slice(from, index) + END_OF_INPUT_MARKER;
-            from = index + 1;
-        }
-        candidate += ordinary.slice(from);
-        const result = printed(candidate, options);
-        return result === ERROR || expected === ERROR || result !== expected;
-    };
-    // The full source differs, so the smallest differing prefix of U+0080 characters exists; the
-    // search assumes that keeping more of them never repairs the parse, and only ever lands on a
-    // count that was itself observed to differ.
-    let low = 1;
-    let high = indices.length;
-    while (low < high) {
-        const middle = (low + high) >> 1;
-        if (dropsText(middle))
-            high = middle;
-        else
-            low = middle + 1;
-    }
-    return indices[high - 1];
-}