@zgeoff/format-codemod 0.0.1 → 0.0.2

package/dist/cli/expand-inputs.d.ts

@@ -1,5 +1,7 @@
 /**
  * Deduped: overlapping patterns (a `src/**` glob plus a file inside src/) must
- * not process — or report — the same file twice.
+ * not process — or report — the same file twice. Ignore globs filter the final
+ * list uniformly, so a file is skipped whether it arrived via a directory,
+ * a glob, or an explicit path argument.
  */
-export declare function expandInputs(patterns: readonly string[]): Promise<string[]>;
+export declare function expandInputs(patterns: readonly string[], ignore?: readonly string[]): Promise<string[]>;

package/dist/cli/parse-cli-args.d.ts

@@ -5,6 +5,7 @@ export interface CLIArgs {
     readonly bench: boolean;
     readonly help: boolean;
     readonly version: boolean;
+    readonly ignore: readonly string[];
     readonly inputs: readonly string[];
 }
 /**

package/dist/cli.js

@@ -20,9 +20,9 @@ function buildBenchStatsFromReports(reports, ms) {
 // src/cli/expand-inputs.ts
 import fs from "node:fs";
 import path from "node:path";
-async function expandInputs(patterns) {
+async function expandInputs(patterns, ignore = []) {
   const lists = await Promise.all(patterns.map((p) => expandPattern(p)));
-  return [...new Set(lists.flat())];
+  return [...new Set(lists.flat())].filter((file) => !isIgnored(file, ignore));
 }
 function expandPattern(p) {
   if (fs.existsSync(p)) {
@@ -41,6 +41,13 @@ function isExcluded(entry) {
   const p = typeof entry === "string" ? entry : path.join(entry.parentPath ?? "", entry.name);
   return p.split(/[\\/]/u).some((seg) => seg === "node_modules" || seg === ".git");
 }
+function isIgnored(file, patterns) {
+  if (patterns.length === 0) {
+    return false;
+  }
+  const relative = path.relative(process.cwd(), file);
+  return patterns.some((pattern) => path.matchesGlob(file, pattern) || path.matchesGlob(relative, pattern));
+}
 
 // src/cli/parse-cli-args.ts
 import { parseArgs } from "node:util";
@@ -54,7 +61,8 @@ function parseCLIArgs(argv) {
         bench: { type: "boolean", default: false },
         quiet: { type: "boolean", default: false },
         version: { type: "boolean", default: false },
-        help: { type: "boolean", default: false }
+        help: { type: "boolean", default: false },
+        ignore: { type: "string", multiple: true, default: [] }
       },
       allowPositionals: true,
       strict: true
@@ -65,6 +73,7 @@ function parseCLIArgs(argv) {
       bench: values.bench,
       help: values.help,
       version: values.version,
+      ignore: values.ignore,
       inputs: positionals
     };
   } catch (error) {
@@ -86,10 +95,11 @@ usage:
   format-codemod [options] <file|dir|glob> ...
 
 options:
-  --check     exit 1 if any file would change (no writes)
-  --dry       print unified diff to stdout (no writes)
-  --bench     print parse stats as JSON to stderr
-  --quiet     only print files that would change
+  --check            exit 1 if any file would change (no writes)
+  --dry              print unified diff to stdout (no writes)
+  --bench            print parse stats as JSON to stderr
+  --quiet            only print files that would change
+  --ignore <glob>    skip files matching the glob (repeatable)
   --version
   --help
 
@@ -102,6 +112,7 @@ examples:
   format-codemod 'src/**/*.ts'
   format-codemod --check 'src/**/*.{ts,tsx}'
   format-codemod --dry path/to/file.ts
+  format-codemod --ignore 'src/generated/**' .
 `);
 }
 
@@ -149,6 +160,36 @@ function applyEdits(src, edits) {
   return segments.toReversed().join("");
 }
 
+// src/transform/is-ast-node.ts
+function isASTNode(value) {
+  return typeof value === "object" && value !== null && "type" in value && typeof value.type === "string";
+}
+
+// src/transform/collect-ast-nodes.ts
+function collectASTNodes(value) {
+  if (!Array.isArray(value)) {
+    return isASTNode(value) ? [value] : [];
+  }
+  const nodes = [];
+  for (const item of value) {
+    if (isASTNode(item)) {
+      nodes.push(item);
+    }
+  }
+  return nodes;
+}
+
+// src/transform/collect-child-nodes.ts
+function collectChildNodes(node) {
+  const children = [];
+  for (const key of Object.keys(node)) {
+    if (key !== "loc" && key !== "parent") {
+      children.push(...collectASTNodes(node[key]));
+    }
+  }
+  return children;
+}
+
 // src/transform/plan-gap-edit.ts
 function planGapEdit(file, prev, next) {
   const gap = buildGap(file, prev, next);
@@ -269,7 +310,7 @@ function buildPairEdits(file, container, body) {
   for (let i = 0;i < body.length - 1; i++) {
     const prev = body[i];
     const next = body[i + 1];
-    if (prev !== undefined && next !== undefined && (needsBlankLine(container, prev, next) || isMultiline(file.src, prev) || isMultiline(file.src, next))) {
+    if (prev !== undefined && next !== undefined && !isImportPair(prev, next) && (needsBlankLine(container, prev, next) || isMultiline(file.src, prev) || isMultiline(file.src, next))) {
       const edit = planGapEdit(file, prev, next);
       if (edit !== null) {
         edits.push(edit);
@@ -278,6 +319,9 @@ function buildPairEdits(file, container, body) {
   }
   return edits;
 }
+function isImportPair(prev, next) {
+  return prev.type === "ImportDeclaration" && next.type === "ImportDeclaration";
+}
 function isMultiline(src, node) {
   return typeof node.start === "number" && typeof node.end === "number" && src.slice(node.start, node.end).includes(`
 `);
@@ -374,30 +418,6 @@ function isVarDecl(node) {
 function isFnOrClassDecl(node) {
   return node.type === "FunctionDeclaration" || node.type === "ClassDeclaration";
 }
-function collectChildNodes(node) {
-  const children = [];
-  for (const key of Object.keys(node)) {
-    if (key !== "loc" && key !== "parent") {
-      children.push(...collectASTNodes(node[key]));
-    }
-  }
-  return children;
-}
-function collectASTNodes(value) {
-  if (!Array.isArray(value)) {
-    return isASTNode(value) ? [value] : [];
-  }
-  const nodes = [];
-  for (const item of value) {
-    if (isASTNode(item)) {
-      nodes.push(item);
-    }
-  }
-  return nodes;
-}
-function isASTNode(value) {
-  return typeof value === "object" && value !== null && "type" in value && typeof value.type === "string";
-}
 
 // src/transform/parse-source.ts
 import { parseSync } from "oxc-parser";
@@ -633,7 +653,7 @@ if (typeof parsedArgs === "string") {
   console.error(parsedArgs);
   process.exit(2);
 }
-var { mode, quiet, bench, help, version, inputs } = parsedArgs;
+var { mode, quiet, bench, help, version, ignore, inputs } = parsedArgs;
 if (help || inputs.length === 0 && !version) {
   const exitCode = inputs.length === 0 ? 2 : 0;
   printHelp();
@@ -646,7 +666,7 @@ if (version) {
   process.exit(0);
 }
 var t0 = Date.now();
-var files = await expandInputs(inputs);
+var files = await expandInputs(inputs, ignore);
 if (files.length === 0) {
   console.error("no files matched");
   process.exit(2);

package/dist/index.js

@@ -10,6 +10,36 @@ function applyEdits(src, edits) {
   return segments.toReversed().join("");
 }
 
+// src/transform/is-ast-node.ts
+function isASTNode(value) {
+  return typeof value === "object" && value !== null && "type" in value && typeof value.type === "string";
+}
+
+// src/transform/collect-ast-nodes.ts
+function collectASTNodes(value) {
+  if (!Array.isArray(value)) {
+    return isASTNode(value) ? [value] : [];
+  }
+  const nodes = [];
+  for (const item of value) {
+    if (isASTNode(item)) {
+      nodes.push(item);
+    }
+  }
+  return nodes;
+}
+
+// src/transform/collect-child-nodes.ts
+function collectChildNodes(node) {
+  const children = [];
+  for (const key of Object.keys(node)) {
+    if (key !== "loc" && key !== "parent") {
+      children.push(...collectASTNodes(node[key]));
+    }
+  }
+  return children;
+}
+
 // src/transform/plan-gap-edit.ts
 function planGapEdit(file, prev, next) {
   const gap = buildGap(file, prev, next);
@@ -130,7 +160,7 @@ function buildPairEdits(file, container, body) {
   for (let i = 0;i < body.length - 1; i++) {
     const prev = body[i];
     const next = body[i + 1];
-    if (prev !== undefined && next !== undefined && (needsBlankLine(container, prev, next) || isMultiline(file.src, prev) || isMultiline(file.src, next))) {
+    if (prev !== undefined && next !== undefined && !isImportPair(prev, next) && (needsBlankLine(container, prev, next) || isMultiline(file.src, prev) || isMultiline(file.src, next))) {
       const edit = planGapEdit(file, prev, next);
       if (edit !== null) {
         edits.push(edit);
@@ -139,6 +169,9 @@ function buildPairEdits(file, container, body) {
   }
   return edits;
 }
+function isImportPair(prev, next) {
+  return prev.type === "ImportDeclaration" && next.type === "ImportDeclaration";
+}
 function isMultiline(src, node) {
   return typeof node.start === "number" && typeof node.end === "number" && src.slice(node.start, node.end).includes(`
 `);
@@ -235,30 +268,6 @@ function isVarDecl(node) {
 function isFnOrClassDecl(node) {
   return node.type === "FunctionDeclaration" || node.type === "ClassDeclaration";
 }
-function collectChildNodes(node) {
-  const children = [];
-  for (const key of Object.keys(node)) {
-    if (key !== "loc" && key !== "parent") {
-      children.push(...collectASTNodes(node[key]));
-    }
-  }
-  return children;
-}
-function collectASTNodes(value) {
-  if (!Array.isArray(value)) {
-    return isASTNode(value) ? [value] : [];
-  }
-  const nodes = [];
-  for (const item of value) {
-    if (isASTNode(item)) {
-      nodes.push(item);
-    }
-  }
-  return nodes;
-}
-function isASTNode(value) {
-  return typeof value === "object" && value !== null && "type" in value && typeof value.type === "string";
-}
 
 // src/transform/parse-source.ts
 import { parseSync } from "oxc-parser";

package/dist/transform/collect-ast-nodes.d.ts

@@ -0,0 +1,6 @@
+import type { ASTNode } from '../types.ts';
+/**
+ * The AST nodes in a property value: the node itself, an array's node
+ * elements, or nothing for non-node values.
+ */
+export declare function collectASTNodes(value: unknown): ASTNode[];

package/dist/transform/collect-child-nodes.d.ts

@@ -0,0 +1,6 @@
+import type { ASTNode } from '../types.ts';
+/**
+ * A node's direct child nodes across all properties, skipping the location
+ * and parent back-references a traversal must not follow.
+ */
+export declare function collectChildNodes(node: ASTNode): ASTNode[];

package/dist/transform/is-ast-node.d.ts

@@ -0,0 +1,2 @@
+import type { ASTNode } from '../types.ts';
+export declare function isASTNode(value: unknown): value is ASTNode;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zgeoff/format-codemod",
-  "version": "0.0.1",
+  "version": "0.0.2",
   "description": "Enforce blank-line padding conventions in TypeScript via a fast text-splice codemod",
   "license": "MIT",
   "repository": {