Exclude patterns

Author: engthiago

README.md

@@ -55,6 +55,9 @@ Folder containing `.cs` files. Subfolders are included.
 `-f`, `--file-filter` optional  
 Glob-style filter for source files, such as `*.cs` or `*Dto.cs`. Default: `*.cs`.
 
+`-x`, `--exclude-pattern` optional  
+Glob-style source file exclude pattern. Can be repeated or passed as a comma-separated list, such as `*Attributes.cs,*Internal.cs`. Patterns are matched against file names and source-relative paths. Default: none.
+
 `-t`, `--type-filter` optional  
 Glob-style filter for parsed type names, such as `Person*`. Default: all parsed types.
 
@@ -87,6 +90,7 @@ Create `typesharp.json`:
 {
   "source": "./samples/SampleModels",
   "fileFilter": "*.cs",
+  "excludePatterns": ["*Attributes.cs", "*Internal.cs"],
   "typeFilter": "*",
   "destination": "./samples/SampleModels/Typescript",
   "watch": false,

src/typesharp-ts/dist/main.mjs

@@ -1,6 +1,6 @@
 import { test } from "vitest";
 import { strict as assert } from "node:assert";
-import { mkdtemp, readFile, rm, writeFile } from "node:fs/promises";
+import { mkdir, mkdtemp, readFile, rm, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { dirname, join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
@@ -186,6 +186,7 @@ test("generates sample outputs for current golden DTOs", async () => {
     await convert({
       source: join(root, "samples/SampleModels"),
       fileFilter: "*.cs",
+      excludePatterns: [],
       destination: temp,
       watch: false,
       exportModule: false,
@@ -212,6 +213,9 @@ test("supports current CLI arguments", () => {
       "-f",
       "*.cs",
       "--type-filter=Person*",
+      "--exclude-pattern",
+      "*Attributes.cs",
+      "-x=*Internal.cs",
       "-d",
       "out",
       "-w",
@@ -226,6 +230,7 @@ test("supports current CLI arguments", () => {
       configPath: "./config/tssharp.json",
       source: "samples",
       fileFilter: "*.cs",
+      excludePatterns: ["*Attributes.cs", "*Internal.cs"],
       typeFilter: "Person*",
       destination: "out",
       watch: true,
@@ -246,6 +251,7 @@ test("loads typesharp.json and lets CLI override file values", async () => {
       JSON.stringify({
         source: "models",
         fileFilter: "*.cs",
+        excludePatterns: ["*Attributes.cs", "*Internal.cs"],
         destination: "generated",
         exportModule: true,
         dictionaryStyle: "index-signature",
@@ -257,6 +263,7 @@ test("loads typesharp.json and lets CLI override file values", async () => {
     assert.deepEqual(options, {
       source: join(temp, "models"),
       fileFilter: "*.cs",
+      excludePatterns: ["*Attributes.cs", "*Internal.cs"],
       destination: join(temp, "generated"),
       typeFilter: undefined,
       watch: false,
@@ -286,6 +293,7 @@ test("defaults fileFilter to C# source files", async () => {
     assert.deepEqual(options, {
       source: join(temp, "models"),
       fileFilter: "*.cs",
+      excludePatterns: [],
       destination: join(temp, "generated"),
       typeFilter: undefined,
       watch: false,
@@ -300,6 +308,38 @@ test("defaults fileFilter to C# source files", async () => {
   }
 });
 
+test("excludes source files by configured patterns", async () => {
+  const temp = await mkdtemp(join(tmpdir(), "typesharp-"));
+  try {
+    const source = join(temp, "models");
+    const destination = join(temp, "generated");
+    await writeFile(join(temp, "typesharp.json"), JSON.stringify({
+      source: "models",
+      destination: "generated",
+      excludePatterns: ["*Attributes.cs", "Internal/*Internal.cs"],
+    }));
+    await mkdir(join(source, "Internal"), { recursive: true });
+    await writeFile(join(source, "Person.cs"), "public class Person { public string Name { get; set; } }");
+    await writeFile(
+      join(source, "OptionalAttributes.cs"),
+      "public class OptionalAttributes { public string Name { get; set; } }",
+    );
+    await writeFile(
+      join(source, "Internal", "HiddenInternal.cs"),
+      "public class HiddenInternal { public string Name { get; set; } }",
+    );
+
+    const options = await resolveOptions([], temp);
+    await convert(options);
+
+    assert.equal(await readFile(join(destination, "Person.ts"), "utf8").then(() => true), true);
+    await assert.rejects(() => readFile(join(destination, "OptionalAttributes.ts"), "utf8"), /ENOENT/);
+    await assert.rejects(() => readFile(join(destination, "HiddenInternal.ts"), "utf8"), /ENOENT/);
+  } finally {
+    await rm(temp, { recursive: true, force: true });
+  }
+});
+
 test("fails clearly on unsupported property bodies", () => {
   assert.throws(
     () => parseSourceFile("bad.cs", 'public class Bad { public string Name { get { return "x"; } set { } } }'),

src/typesharp-ts/spec/node_test.ts

@@ -9,6 +9,7 @@ import { fileURLToPath } from "node:url";
 export interface CliOptions extends GenerateOptions {
   source: string;
   fileFilter: string;
+  excludePatterns: string[];
   destination: string;
   typeFilter?: string;
   watch: boolean;
@@ -30,7 +31,7 @@ export async function run(args: string[]): Promise<void> {
 }
 
 export async function convert(options: CliOptions): Promise<void> {
-  const sourceFiles = await findCSharpFiles(options.source, options.fileFilter);
+  const sourceFiles = await findCSharpFiles(options.source, options.fileFilter, options.excludePatterns);
   const parsed = [];
   for (const path of sourceFiles) {
     parsed.push(parseSourceFile(path, await readFile(path, "utf8")));
@@ -75,12 +76,14 @@ export function parseArgs(args: string[]): ParsedCliOptions {
     c: "config",
     s: "source",
     f: "file-filter",
+    x: "exclude-pattern",
     t: "type-filter",
     d: "destination",
     w: "watch",
     m: "export-module",
   };
   const booleanFlags = new Set(["watch", "export-module", "readonly-properties", "semicolons"]);
+  const arrayFlags = new Set(["exclude-pattern", "exclude-patterns"]);
 
   for (let i = 0; i < args.length; i++) {
     const raw = args[i];
@@ -94,6 +97,10 @@ export function parseArgs(args: string[]): ParsedCliOptions {
     const name = aliases[flagName] ?? flagName;
     if (booleanFlags.has(name)) {
       values.set(name, inlineValue === undefined ? true : parseBoolean(inlineValue, name));
+    } else if (arrayFlags.has(name)) {
+      const value = inlineValue ?? args[++i];
+      if (!value || value.startsWith("-")) throw new Error(`Missing value for --${name}`);
+      appendListValue(values, "exclude-patterns", value);
     } else {
       const value = inlineValue ?? args[++i];
       if (!value || value.startsWith("-")) throw new Error(`Missing value for --${name}`);
@@ -141,6 +148,7 @@ function finalizeOptions(options: ConfigFileOptions): CliOptions {
   return {
     source: options.source,
     fileFilter: options.fileFilter ?? "*.cs",
+    excludePatterns: normalizeStringList(options.excludePatterns, "excludePatterns"),
     destination: options.destination,
     typeFilter: options.typeFilter,
     watch: options.watch ?? false,
@@ -157,6 +165,7 @@ function mapRawOptions(values: Map<string, string | boolean>): ParsedCliOptions
   setString(options, "configPath", values.get("config"));
   setString(options, "source", values.get("source"));
   setString(options, "fileFilter", values.get("file-filter"));
+  setStringList(options, "excludePatterns", values.get("exclude-patterns"));
   setString(options, "typeFilter", values.get("type-filter"));
   setString(options, "destination", values.get("destination"));
   setBoolean(options, "watch", values.get("watch"));
@@ -168,6 +177,11 @@ function mapRawOptions(values: Map<string, string | boolean>): ParsedCliOptions
   return options;
 }
 
+function appendListValue(values: Map<string, string | boolean>, key: string, value: string): void {
+  const existing = values.get(key);
+  values.set(key, [typeof existing === "string" ? existing : "", value].filter(Boolean).join(","));
+}
+
 function setString<T extends Record<string, unknown>>(
   target: T,
   key: keyof T,
@@ -176,6 +190,14 @@ function setString<T extends Record<string, unknown>>(
   if (typeof value === "string") target[key] = value as T[keyof T];
 }
 
+function setStringList<T extends Record<string, unknown>>(
+  target: T,
+  key: keyof T,
+  value: string | boolean | undefined,
+): void {
+  if (typeof value === "string") target[key] = splitList(value) as T[keyof T];
+}
+
 function setBoolean<T extends Record<string, unknown>>(
   target: T,
   key: keyof T,
@@ -184,17 +206,35 @@ function setBoolean<T extends Record<string, unknown>>(
   if (typeof value === "boolean") target[key] = value as T[keyof T];
 }
 
+function normalizeStringList(value: unknown, optionName: string): string[] {
+  if (value === undefined) return [];
+  if (typeof value === "string") return splitList(value);
+  if (Array.isArray(value) && value.every((item) => typeof item === "string")) {
+    return value.map((item) => item.trim()).filter(Boolean);
+  }
+  throw new Error(`${optionName} must be an array of strings`);
+}
+
+function splitList(value: string): string[] {
+  return value.split(",").map((item) => item.trim()).filter(Boolean);
+}
+
 function parseBoolean(value: string, flag: string): boolean {
   if (["true", "1", "yes"].includes(value.toLowerCase())) return true;
   if (["false", "0", "no"].includes(value.toLowerCase())) return false;
   throw new Error(`--${flag} must be true or false`);
 }
 
-async function findCSharpFiles(source: string, filter: string): Promise<string[]> {
+async function findCSharpFiles(source: string, filter: string, excludePatterns: string[] = []): Promise<string[]> {
   const files: string[] = [];
   const matcher = globToRegExp(filter === "*" ? "*.cs" : filter);
+  const excludeMatchers = excludePatterns.map(globToRegExp);
   for await (const entry of walk(source)) {
-    if (entry.isFile && entry.path.endsWith(".cs") && matcher.test(basename(entry.path))) {
+    const relativePath = relativePathFrom(source, entry.path);
+    const excluded = excludeMatchers.some((excludeMatcher) =>
+      excludeMatcher.test(basename(entry.path)) || excludeMatcher.test(relativePath)
+    );
+    if (entry.isFile && entry.path.endsWith(".cs") && matcher.test(basename(entry.path)) && !excluded) {
       files.push(entry.path);
     }
   }
@@ -267,6 +307,12 @@ function normalizePath(path: string): string {
   return normalized === "/" ? normalized : normalized.replace(/\/$/, "");
 }
 
+function relativePathFrom(root: string, path: string): string {
+  const normalizedRoot = `${normalizePath(root)}/`;
+  const normalizedPath = normalizePath(path);
+  return normalizedPath.startsWith(normalizedRoot) ? normalizedPath.slice(normalizedRoot.length) : basename(path);
+}
+
 function resolvePath(base: string, path: string): string {
   return normalizePath(isAbsolute(path) ? path : resolve(base, path));
 }

src/typesharp-ts/src/main.ts

@@ -1,6 +1,7 @@
 {
   "source": "./../../samples/SampleModels",
   "fileFilter": "*.cs",
+  "excludePatterns": [],
   "typeFilter": "*",
   "destination": "./../../samples/SampleModels/Typescript",
   "watch": false,
@@ -9,4 +10,4 @@
   "readonlyProperties": false,
   "quoteStyle": "double",
   "semicolons": true
-}
+}