@warlock.js/core 5.34.3 → 5.36.0

package/esm/tooling/check/api-route-references.mjs

@@ -0,0 +1,146 @@
+import { keyName, parseSourceFile } from "../syntax/ast.mjs";
+import { calleeName, literalString, unwrap, walk } from "./source-ast.mjs";
+import path from "node:path";
+import { readFileSync } from "node:fs";
+
+//#region ../../packages/core/src/tooling/check/api-route-references.ts
+const EMPTY_API_ROUTE_FACTS = {
+	clientHiddenNames: /* @__PURE__ */ new Set(),
+	declaresApiRoutes: false
+};
+/** Browser-side route-taking helpers from `@warlock.js/web`. */
+const ROUTE_CALLS = /* @__PURE__ */ new Set(["apiHref", "useSubmitForm"]);
+/** The value of a literal key on an object literal argument, when there is one. */
+function objectProperty(object, name) {
+	const value = unwrap(object);
+	if (value.type !== "ObjectExpression") return void 0;
+	for (const property of value.properties) {
+		if (property.type !== "Property") continue;
+		if (keyName(property.key) === name) return property.value;
+	}
+}
+/**
+* Names of the routes a source file registers with `clientVisible: false`
+* inside an options object that also carries a literal `name`. A group prefix
+* is not applied here, so a route whose name comes only from a group is not
+* recognised; name such a route with a literal `name` option.
+*/
+function clientHiddenRouteNamesOf(source) {
+	const names = [];
+	walk(source, (node) => {
+		if (node.type !== "CallExpression") return;
+		for (const argument of node.arguments) {
+			const hidden = objectProperty(argument, "clientVisible");
+			const flag = hidden === void 0 ? void 0 : unwrap(hidden);
+			const name = literalString(objectProperty(argument, "name"));
+			if (flag?.type === "Literal" && flag.value === false && name !== void 0) names.push(name);
+		}
+	}, true);
+	return names;
+}
+/** Whether a setup file's exported `config` object declares an `apiRoutes` key. */
+function declaresApiRoutesOf(source) {
+	let declared = false;
+	walk(source, (node) => {
+		if (declared) return false;
+		if (node.type !== "VariableDeclarator" || keyName(node.id) !== "config") return;
+		if (node.init === null || node.init === void 0) return;
+		if (objectProperty(node.init, "apiRoutes") !== void 0) declared = true;
+	}, true);
+	return declared;
+}
+/**
+* The static name a route-taking call receives: a string literal, a template
+* with no substitutions, or the static prefix of a template. `undefined` means
+* the name is only known at run time.
+*/
+function staticNameOf(node) {
+	const value = unwrap(node);
+	const literal = literalString(value);
+	if (literal !== void 0) return {
+		name: literal,
+		prefix: false
+	};
+	if (value.type === "TemplateLiteral" && value.expressions.length > 0) {
+		const head = value.quasis[0]?.value.cooked ?? "";
+		return head.length > 0 ? {
+			name: head,
+			prefix: true
+		} : void 0;
+	}
+}
+/** The route-name argument of an `apiHref(name)` or `useSubmitForm({ route })` call. */
+function routeArgumentOf(call) {
+	const first = call.arguments[0];
+	if (first === void 0) return void 0;
+	if (calleeName(call) === "apiHref") return first;
+	return objectProperty(first, "route");
+}
+/**
+* The browser route-reference conventions:
+*
+* - `WL9020` (error) a web module passes a name of a `clientVisible: false`
+*   route to `apiHref()` or `useSubmitForm({ route })`, literally or as a
+*   template prefix;
+* - `WL9019` (warning) such a call takes a name that cannot be read statically
+*   and no page declares `config.apiRoutes`, so the page's route table cannot
+*   be computed. The declaration is checked app-wide, not per page.
+*
+* Applies to `src/web/**` modules, outside test files.
+*/
+function apiRouteReferenceFindings(source, facts = EMPTY_API_ROUTE_FACTS) {
+	if (!/^src\/web\/.*\.[cm]?[jt]sx?$/.test(source.fileName)) return [];
+	if (/\.(?:test|spec)\.[cm]?[jt]sx?$/i.test(source.fileName)) return [];
+	const findings = [];
+	walk(source, (node) => {
+		if (node.type !== "CallExpression") return;
+		if (!ROUTE_CALLS.has(calleeName(node) ?? "")) return;
+		const argument = routeArgumentOf(node);
+		if (argument === void 0) return;
+		const target = staticNameOf(argument);
+		if (target === void 0) {
+			if (!facts.declaresApiRoutes) findings.push({
+				code: "WL9019",
+				node: argument,
+				severity: "warning",
+				message: "This route name is only known at run time, so the page's browser route table cannot include it. Pass a string literal, or declare the names with config.apiRoutes in the page's setup file."
+			});
+			return;
+		}
+		const hidden = [...facts.clientHiddenNames].find((name) => target.prefix ? name.startsWith(target.name) : name === target.name);
+		if (hidden !== void 0) findings.push({
+			code: "WL9020",
+			node: argument,
+			severity: "error",
+			message: `Route "${hidden}" is registered with clientVisible: false and cannot be referenced from browser code.`
+		});
+	}, true);
+	return findings;
+}
+/**
+* Gathers the project-wide facts from `files` (project-relative). Unreadable or
+* unparsable files contribute nothing, as the per-file checks report them.
+*/
+function collectApiRouteFacts(cwd, files) {
+	const clientHiddenNames = /* @__PURE__ */ new Set();
+	let declaresApiRoutes = false;
+	for (const file of files) {
+		if (!/^src\/.*\.[cm]?[jt]sx?$/.test(file)) continue;
+		let sourceFile;
+		try {
+			sourceFile = parseSourceFile(file, readFileSync(path.join(cwd, file), "utf8"));
+		} catch {
+			continue;
+		}
+		for (const name of clientHiddenRouteNamesOf(sourceFile)) clientHiddenNames.add(name);
+		if (/^src\/web\/.*\.setup\.[cm]?[jt]sx?$/.test(file) && declaresApiRoutesOf(sourceFile)) declaresApiRoutes = true;
+	}
+	return {
+		clientHiddenNames,
+		declaresApiRoutes
+	};
+}
+
+//#endregion
+export { EMPTY_API_ROUTE_FACTS, apiRouteReferenceFindings, clientHiddenRouteNamesOf, collectApiRouteFacts, declaresApiRoutesOf };
+//# sourceMappingURL=api-route-references.mjs.map

package/esm/tooling/check/api-route-references.mjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"api-route-references.mjs","names":["ast.keyName","ast.parseSourceFile"],"sources":["../../../../../../../../../packages/core/src/tooling/check/api-route-references.ts"],"sourcesContent":["import { readFileSync } from \"node:fs\";\nimport path from \"node:path\";\nimport * as ast from \"../syntax/ast\";\nimport { calleeName, literalString, unwrap, walk, type SourceFinding } from \"./source-ast\";\n\n/**\n * Project-wide facts the browser-reference checks need. Gathered once per run\n * (`runConventions`) because a route is registered in one file and referenced\n * from another.\n */\nexport type ApiRouteFacts = {\n  /** Names of routes registered with `clientVisible: false`. */\n  readonly clientHiddenNames: ReadonlySet<string>;\n  /** Whether any page, layout or root setup file declares `config.apiRoutes`. */\n  readonly declaresApiRoutes: boolean;\n};\n\nexport const EMPTY_API_ROUTE_FACTS: ApiRouteFacts = {\n  clientHiddenNames: new Set(),\n  declaresApiRoutes: false,\n};\n\n/** Browser-side route-taking helpers from `@warlock.js/web`. */\nconst ROUTE_CALLS = new Set([\"apiHref\", \"useSubmitForm\"]);\n\n/** The value of a literal key on an object literal argument, when there is one. */\nfunction objectProperty(object: ast.Node, name: string): ast.Node | undefined {\n  const value = unwrap(object);\n  if (value.type !== \"ObjectExpression\") return undefined;\n  for (const property of value.properties) {\n    if (property.type !== \"Property\") continue;\n    if (ast.keyName(property.key) === name) return property.value;\n  }\n  return undefined;\n}\n\n/**\n * Names of the routes a source file registers with `clientVisible: false`\n * inside an options object that also carries a literal `name`. A group prefix\n * is not applied here, so a route whose name comes only from a group is not\n * recognised; name such a route with a literal `name` option.\n */\nexport function clientHiddenRouteNamesOf(source: ast.SourceFile): string[] {\n  const names: string[] = [];\n\n  walk(\n    source,\n    (node) => {\n      if (node.type !== \"CallExpression\") return;\n      for (const argument of node.arguments) {\n        const hidden = objectProperty(argument, \"clientVisible\");\n        const flag = hidden === undefined ? undefined : unwrap(hidden);\n        const name = literalString(objectProperty(argument, \"name\"));\n        if (flag?.type === \"Literal\" && flag.value === false && name !== undefined)\n          names.push(name);\n      }\n    },\n    true,\n  );\n\n  return names;\n}\n\n/** Whether a setup file's exported `config` object declares an `apiRoutes` key. */\nexport function declaresApiRoutesOf(source: ast.SourceFile): boolean {\n  let declared = false;\n\n  walk(\n    source,\n    (node) => {\n      if (declared) return false;\n      if (node.type !== \"VariableDeclarator\" || ast.keyName(node.id) !== \"config\") return;\n      if (node.init === null || node.init === undefined) return;\n      if (objectProperty(node.init, \"apiRoutes\") !== undefined) declared = true;\n    },\n    true,\n  );\n\n  return declared;\n}\n\n/**\n * The static name a route-taking call receives: a string literal, a template\n * with no substitutions, or the static prefix of a template. `undefined` means\n * the name is only known at run time.\n */\nfunction staticNameOf(node: ast.Node): { name: string; prefix: boolean } | undefined {\n  const value = unwrap(node);\n  const literal = literalString(value);\n  if (literal !== undefined) return { name: literal, prefix: false };\n  if (value.type === \"TemplateLiteral\" && value.expressions.length > 0) {\n    const head = value.quasis[0]?.value.cooked ?? \"\";\n    return head.length > 0 ? { name: head, prefix: true } : undefined;\n  }\n  return undefined;\n}\n\n/** The route-name argument of an `apiHref(name)` or `useSubmitForm({ route })` call. */\nfunction routeArgumentOf(call: ast.CallExpression): ast.Node | undefined {\n  const first = call.arguments[0];\n  if (first === undefined) return undefined;\n  if (calleeName(call) === \"apiHref\") return first;\n  return objectProperty(first, \"route\");\n}\n\n/**\n * The browser route-reference conventions:\n *\n * - `WL9020` (error) a web module passes a name of a `clientVisible: false`\n *   route to `apiHref()` or `useSubmitForm({ route })`, literally or as a\n *   template prefix;\n * - `WL9019` (warning) such a call takes a name that cannot be read statically\n *   and no page declares `config.apiRoutes`, so the page's route table cannot\n *   be computed. The declaration is checked app-wide, not per page.\n *\n * Applies to `src/web/**` modules, outside test files.\n */\nexport function apiRouteReferenceFindings(\n  source: ast.SourceFile,\n  facts: ApiRouteFacts = EMPTY_API_ROUTE_FACTS,\n): SourceFinding[] {\n  if (!/^src\\/web\\/.*\\.[cm]?[jt]sx?$/.test(source.fileName)) return [];\n  if (/\\.(?:test|spec)\\.[cm]?[jt]sx?$/i.test(source.fileName)) return [];\n\n  const findings: SourceFinding[] = [];\n\n  walk(\n    source,\n    (node) => {\n      if (node.type !== \"CallExpression\") return;\n      if (!ROUTE_CALLS.has(calleeName(node) ?? \"\")) return;\n      const argument = routeArgumentOf(node);\n      if (argument === undefined) return;\n      const target = staticNameOf(argument);\n\n      if (target === undefined) {\n        if (!facts.declaresApiRoutes)\n          findings.push({\n            code: \"WL9019\",\n            node: argument,\n            severity: \"warning\",\n            message:\n              \"This route name is only known at run time, so the page's browser route table cannot include it. Pass a string literal, or declare the names with config.apiRoutes in the page's setup file.\",\n          });\n        return;\n      }\n\n      const hidden = [...facts.clientHiddenNames].find((name) =>\n        target.prefix ? name.startsWith(target.name) : name === target.name,\n      );\n\n      if (hidden !== undefined)\n        findings.push({\n          code: \"WL9020\",\n          node: argument,\n          severity: \"error\",\n          message: `Route \"${hidden}\" is registered with clientVisible: false and cannot be referenced from browser code.`,\n        });\n    },\n    true,\n  );\n\n  return findings;\n}\n\n/**\n * Gathers the project-wide facts from `files` (project-relative). Unreadable or\n * unparsable files contribute nothing, as the per-file checks report them.\n */\nexport function collectApiRouteFacts(cwd: string, files: readonly string[]): ApiRouteFacts {\n  const clientHiddenNames = new Set<string>();\n  let declaresApiRoutes = false;\n\n  for (const file of files) {\n    if (!/^src\\/.*\\.[cm]?[jt]sx?$/.test(file)) continue;\n\n    let sourceFile: ast.SourceFile;\n\n    try {\n      sourceFile = ast.parseSourceFile(file, readFileSync(path.join(cwd, file), \"utf8\"));\n    } catch {\n      continue;\n    }\n\n    for (const name of clientHiddenRouteNamesOf(sourceFile)) clientHiddenNames.add(name);\n    if (/^src\\/web\\/.*\\.setup\\.[cm]?[jt]sx?$/.test(file) && declaresApiRoutesOf(sourceFile))\n      declaresApiRoutes = true;\n  }\n\n  return { clientHiddenNames, declaresApiRoutes };\n}\n"],"mappings":";;;;;;AAiBA,MAAa,wBAAuC;CAClD,mCAAmB,IAAI,IAAI;CAC3B,mBAAmB;AACrB;;AAGA,MAAM,8BAAc,IAAI,IAAI,CAAC,WAAW,eAAe,CAAC;;AAGxD,SAAS,eAAe,QAAkB,MAAoC;CAC5E,MAAM,QAAQ,OAAO,MAAM;CAC3B,IAAI,MAAM,SAAS,oBAAoB,OAAO;CAC9C,KAAK,MAAM,YAAY,MAAM,YAAY;EACvC,IAAI,SAAS,SAAS,YAAY;EAClC,IAAIA,QAAY,SAAS,GAAG,MAAM,MAAM,OAAO,SAAS;CAC1D;AAEF;;;;;;;AAQA,SAAgB,yBAAyB,QAAkC;CACzE,MAAM,QAAkB,CAAC;CAEzB,KACE,SACC,SAAS;EACR,IAAI,KAAK,SAAS,kBAAkB;EACpC,KAAK,MAAM,YAAY,KAAK,WAAW;GACrC,MAAM,SAAS,eAAe,UAAU,eAAe;GACvD,MAAM,OAAO,WAAW,SAAY,SAAY,OAAO,MAAM;GAC7D,MAAM,OAAO,cAAc,eAAe,UAAU,MAAM,CAAC;GAC3D,IAAI,MAAM,SAAS,aAAa,KAAK,UAAU,SAAS,SAAS,QAC/D,MAAM,KAAK,IAAI;EACnB;CACF,GACA,IACF;CAEA,OAAO;AACT;;AAGA,SAAgB,oBAAoB,QAAiC;CACnE,IAAI,WAAW;CAEf,KACE,SACC,SAAS;EACR,IAAI,UAAU,OAAO;EACrB,IAAI,KAAK,SAAS,wBAAwBA,QAAY,KAAK,EAAE,MAAM,UAAU;EAC7E,IAAI,KAAK,SAAS,QAAQ,KAAK,SAAS,QAAW;EACnD,IAAI,eAAe,KAAK,MAAM,WAAW,MAAM,QAAW,WAAW;CACvE,GACA,IACF;CAEA,OAAO;AACT;;;;;;AAOA,SAAS,aAAa,MAA+D;CACnF,MAAM,QAAQ,OAAO,IAAI;CACzB,MAAM,UAAU,cAAc,KAAK;CACnC,IAAI,YAAY,QAAW,OAAO;EAAE,MAAM;EAAS,QAAQ;CAAM;CACjE,IAAI,MAAM,SAAS,qBAAqB,MAAM,YAAY,SAAS,GAAG;EACpE,MAAM,OAAO,MAAM,OAAO,EAAE,EAAE,MAAM,UAAU;EAC9C,OAAO,KAAK,SAAS,IAAI;GAAE,MAAM;GAAM,QAAQ;EAAK,IAAI;CAC1D;AAEF;;AAGA,SAAS,gBAAgB,MAAgD;CACvE,MAAM,QAAQ,KAAK,UAAU;CAC7B,IAAI,UAAU,QAAW,OAAO;CAChC,IAAI,WAAW,IAAI,MAAM,WAAW,OAAO;CAC3C,OAAO,eAAe,OAAO,OAAO;AACtC;;;;;;;;;;;;;AAcA,SAAgB,0BACd,QACA,QAAuB,uBACN;CACjB,IAAI,CAAC,+BAA+B,KAAK,OAAO,QAAQ,GAAG,OAAO,CAAC;CACnE,IAAI,kCAAkC,KAAK,OAAO,QAAQ,GAAG,OAAO,CAAC;CAErE,MAAM,WAA4B,CAAC;CAEnC,KACE,SACC,SAAS;EACR,IAAI,KAAK,SAAS,kBAAkB;EACpC,IAAI,CAAC,YAAY,IAAI,WAAW,IAAI,KAAK,EAAE,GAAG;EAC9C,MAAM,WAAW,gBAAgB,IAAI;EACrC,IAAI,aAAa,QAAW;EAC5B,MAAM,SAAS,aAAa,QAAQ;EAEpC,IAAI,WAAW,QAAW;GACxB,IAAI,CAAC,MAAM,mBACT,SAAS,KAAK;IACZ,MAAM;IACN,MAAM;IACN,UAAU;IACV,SACE;GACJ,CAAC;GACH;EACF;EAEA,MAAM,SAAS,CAAC,GAAG,MAAM,iBAAiB,CAAC,CAAC,MAAM,SAChD,OAAO,SAAS,KAAK,WAAW,OAAO,IAAI,IAAI,SAAS,OAAO,IACjE;EAEA,IAAI,WAAW,QACb,SAAS,KAAK;GACZ,MAAM;GACN,MAAM;GACN,UAAU;GACV,SAAS,UAAU,OAAO;EAC5B,CAAC;CACL,GACA,IACF;CAEA,OAAO;AACT;;;;;AAMA,SAAgB,qBAAqB,KAAa,OAAyC;CACzF,MAAM,oCAAoB,IAAI,IAAY;CAC1C,IAAI,oBAAoB;CAExB,KAAK,MAAM,QAAQ,OAAO;EACxB,IAAI,CAAC,0BAA0B,KAAK,IAAI,GAAG;EAE3C,IAAI;EAEJ,IAAI;GACF,aAAaC,gBAAoB,MAAM,aAAa,KAAK,KAAK,KAAK,IAAI,GAAG,MAAM,CAAC;EACnF,QAAQ;GACN;EACF;EAEA,KAAK,MAAM,QAAQ,yBAAyB,UAAU,GAAG,kBAAkB,IAAI,IAAI;EACnF,IAAI,sCAAsC,KAAK,IAAI,KAAK,oBAAoB,UAAU,GACpF,oBAAoB;CACxB;CAEA,OAAO;EAAE;EAAmB;CAAkB;AAChD"}

package/esm/tooling/check/changed-files.d.mts

@@ -22,6 +22,8 @@ export declare function listChangedFiles(options: {
   cwd: string;
   base?: string;
   git?: GitRunner;
+  /** Keep deleted paths as seeds for reverse dependency selection. */
+  includeDeleted?: boolean;
 }): Promise<{
   files: string[];
 } | {

package/esm/tooling/check/changed-files.d.mts.map

@@ -1 +1 @@
-{"version":3,"file":"changed-files.d.mts","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/changed-files.ts"],"mappings":";;;YAMY;EAAc;EAAc;EAAgB;;;YAG5C,aAAa,yBAAyB,gBAAgB,QAAQ;;qBAG7D,QAAQ;;;;;;;;;wBAoCC,iBAAiB;EACrC;EACA;EACA,MAAM;IACJ;EAAU;;EAAsB,SAAS"}
+{"version":3,"file":"changed-files.d.mts","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/changed-files.ts"],"mappings":";;;YAMY;EAAc;EAAc;EAAgB;;;YAG5C,aAAa,yBAAyB,gBAAgB,QAAQ;;qBAG7D,QAAQ;;;;;;;;;wBAoCC,iBAAiB;EACrC;EACA;EACA,MAAM;;EAEN;IACE;EAAU;;EAAsB,SAAS"}

package/esm/tooling/check/changed-files.mjs

@@ -55,7 +55,7 @@ async function listChangedFiles(options) {
 		"diff",
 		"--name-only",
 		"--relative",
-		"--diff-filter=ACMR",
+		options.includeDeleted ? "--diff-filter=ACMRD" : "--diff-filter=ACMR",
 		ref
 	], options.cwd);
 	if (diff.code !== 0) return { problem: gitProblem(`git diff against ${ref} failed: ${diff.stderr.trim() || `exit ${diff.code}`}`) };
@@ -65,7 +65,7 @@ async function listChangedFiles(options) {
 		"--exclude-standard"
 	], options.cwd);
 	if (untracked.code !== 0) return { problem: gitProblem(`git ls-files failed: ${untracked.stderr.trim() || `exit ${untracked.code}`}`) };
-	return { files: [.../* @__PURE__ */ new Set([...lines(diff.stdout), ...lines(untracked.stdout)])].map((file) => file.split(path.sep).join("/")).filter((file) => SOURCE_FILE.test(file) && existsSync(path.join(options.cwd, file))).sort() };
+	return { files: [.../* @__PURE__ */ new Set([...lines(diff.stdout), ...lines(untracked.stdout)])].map((file) => file.split(path.sep).join("/")).filter((file) => SOURCE_FILE.test(file) && (options.includeDeleted || existsSync(path.join(options.cwd, file)))).sort() };
 }
 
 //#endregion

package/esm/tooling/check/changed-files.mjs.map

@@ -1 +1 @@
-{"version":3,"file":"changed-files.mjs","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/changed-files.ts"],"sourcesContent":["import { execFile } from \"node:child_process\";\nimport { existsSync } from \"node:fs\";\nimport path from \"node:path\";\nimport { createProblem, type ToolingProblem } from \"../problem\";\n\n/** The result of one git invocation. */\nexport type GitResult = { code: number; stdout: string; stderr: string };\n\n/** Runs `git <args>` in `cwd`. Injectable so tests need no repository. */\nexport type GitRunner = (args: readonly string[], cwd: string) => Promise<GitResult>;\n\n/** The default {@link GitRunner}: the `git` on PATH. */\nexport const runGit: GitRunner = (args, cwd) =>\n  new Promise((resolve) => {\n    execFile(\n      \"git\",\n      [...args],\n      { cwd, encoding: \"utf8\", windowsHide: true, maxBuffer: 16 * 1024 * 1024 },\n      (error, stdout, stderr) => {\n        const code = error === null ? 0 : typeof error.code === \"number\" ? error.code : 127;\n\n        resolve({ code, stdout, stderr: stderr || (error?.message ?? \"\") });\n      },\n    );\n  });\n\n/** Files `warlock check --changed` considers: JavaScript and TypeScript sources. */\nconst SOURCE_FILE = /\\.(?:[cm]?[jt]sx?)$/i;\n\nfunction lines(output: string): string[] {\n  return output\n    .split(/\\r?\\n/)\n    .map((line) => line.trim())\n    .filter((line) => line.length > 0);\n}\n\nfunction gitProblem(message: string): ToolingProblem {\n  return createProblem({ code: \"WL8006\", source: \"environment\", message });\n}\n\n/**\n * The source files changed in the working tree — tracked changes against\n * `base` (or HEAD) plus untracked files — relative to `cwd`, existing only.\n *\n * With `base`, the comparison point is `git merge-base <base> HEAD`, so a\n * branch is compared with where it left `base` rather than with `base`'s tip.\n * A git failure is a `WL8006` environment problem.\n */\nexport async function listChangedFiles(options: {\n  cwd: string;\n  base?: string;\n  git?: GitRunner;\n}): Promise<{ files: string[] } | { problem: ToolingProblem }> {\n  const git = options.git ?? runGit;\n  let ref = \"HEAD\";\n\n  if (options.base) {\n    const mergeBase = await git([\"merge-base\", options.base, \"HEAD\"], options.cwd);\n\n    if (mergeBase.code !== 0) {\n      return {\n        problem: gitProblem(\n          `git merge-base ${options.base} HEAD failed: ${mergeBase.stderr.trim() || `exit ${mergeBase.code}`}`,\n        ),\n      };\n    }\n\n    ref = mergeBase.stdout.trim();\n  }\n\n  // `--relative` makes the paths relative to cwd and limits them to it.\n  const diff = await git(\n    [\"diff\", \"--name-only\", \"--relative\", \"--diff-filter=ACMR\", ref],\n    options.cwd,\n  );\n\n  if (diff.code !== 0) {\n    return {\n      problem: gitProblem(\n        `git diff against ${ref} failed: ${diff.stderr.trim() || `exit ${diff.code}`}`,\n      ),\n    };\n  }\n\n  const untracked = await git([\"ls-files\", \"--others\", \"--exclude-standard\"], options.cwd);\n\n  if (untracked.code !== 0) {\n    return {\n      problem: gitProblem(\n        `git ls-files failed: ${untracked.stderr.trim() || `exit ${untracked.code}`}`,\n      ),\n    };\n  }\n\n  const files = [...new Set([...lines(diff.stdout), ...lines(untracked.stdout)])]\n    .map((file) => file.split(path.sep).join(\"/\"))\n    .filter((file) => SOURCE_FILE.test(file) && existsSync(path.join(options.cwd, file)))\n    .sort();\n\n  return { files };\n}\n"],"mappings":";;;;;;;AAYA,MAAa,UAAqB,MAAM,QACtC,IAAI,SAAS,YAAY;CACvB,SACE,OACA,CAAC,GAAG,IAAI,GACR;EAAE;EAAK,UAAU;EAAQ,aAAa;EAAM,WAAW;CAAiB,IACvE,OAAO,QAAQ,WAAW;EAGzB,QAAQ;GAAE,MAFG,UAAU,OAAO,IAAI,OAAO,MAAM,SAAS,WAAW,MAAM,OAAO;GAEhE;GAAQ,QAAQ,WAAW,OAAO,WAAW;EAAI,CAAC;CACpE,CACF;AACF,CAAC;;AAGH,MAAM,cAAc;AAEpB,SAAS,MAAM,QAA0B;CACvC,OAAO,OACJ,MAAM,OAAO,CAAC,CACd,KAAK,SAAS,KAAK,KAAK,CAAC,CAAC,CAC1B,QAAQ,SAAS,KAAK,SAAS,CAAC;AACrC;AAEA,SAAS,WAAW,SAAiC;CACnD,OAAO,cAAc;EAAE,MAAM;EAAU,QAAQ;EAAe;CAAQ,CAAC;AACzE;;;;;;;;;AAUA,eAAsB,iBAAiB,SAIwB;CAC7D,MAAM,MAAM,QAAQ,OAAO;CAC3B,IAAI,MAAM;CAEV,IAAI,QAAQ,MAAM;EAChB,MAAM,YAAY,MAAM,IAAI;GAAC;GAAc,QAAQ;GAAM;EAAM,GAAG,QAAQ,GAAG;EAE7E,IAAI,UAAU,SAAS,GACrB,OAAO,EACL,SAAS,WACP,kBAAkB,QAAQ,KAAK,gBAAgB,UAAU,OAAO,KAAK,KAAK,QAAQ,UAAU,QAC9F,EACF;EAGF,MAAM,UAAU,OAAO,KAAK;CAC9B;CAGA,MAAM,OAAO,MAAM,IACjB;EAAC;EAAQ;EAAe;EAAc;EAAsB;CAAG,GAC/D,QAAQ,GACV;CAEA,IAAI,KAAK,SAAS,GAChB,OAAO,EACL,SAAS,WACP,oBAAoB,IAAI,WAAW,KAAK,OAAO,KAAK,KAAK,QAAQ,KAAK,QACxE,EACF;CAGF,MAAM,YAAY,MAAM,IAAI;EAAC;EAAY;EAAY;CAAoB,GAAG,QAAQ,GAAG;CAEvF,IAAI,UAAU,SAAS,GACrB,OAAO,EACL,SAAS,WACP,wBAAwB,UAAU,OAAO,KAAK,KAAK,QAAQ,UAAU,QACvE,EACF;CAQF,OAAO,EAAE,OALK,CAAC,mBAAG,IAAI,IAAI,CAAC,GAAG,MAAM,KAAK,MAAM,GAAG,GAAG,MAAM,UAAU,MAAM,CAAC,CAAC,CAAC,CAAC,CAC5E,KAAK,SAAS,KAAK,MAAM,KAAK,GAAG,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAC7C,QAAQ,SAAS,YAAY,KAAK,IAAI,KAAK,WAAW,KAAK,KAAK,QAAQ,KAAK,IAAI,CAAC,CAAC,CAAC,CACpF,KAEU,EAAE;AACjB"}
+{"version":3,"file":"changed-files.mjs","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/changed-files.ts"],"sourcesContent":["import { execFile } from \"node:child_process\";\nimport { existsSync } from \"node:fs\";\nimport path from \"node:path\";\nimport { createProblem, type ToolingProblem } from \"../problem\";\n\n/** The result of one git invocation. */\nexport type GitResult = { code: number; stdout: string; stderr: string };\n\n/** Runs `git <args>` in `cwd`. Injectable so tests need no repository. */\nexport type GitRunner = (args: readonly string[], cwd: string) => Promise<GitResult>;\n\n/** The default {@link GitRunner}: the `git` on PATH. */\nexport const runGit: GitRunner = (args, cwd) =>\n  new Promise((resolve) => {\n    execFile(\n      \"git\",\n      [...args],\n      { cwd, encoding: \"utf8\", windowsHide: true, maxBuffer: 16 * 1024 * 1024 },\n      (error, stdout, stderr) => {\n        const code = error === null ? 0 : typeof error.code === \"number\" ? error.code : 127;\n\n        resolve({ code, stdout, stderr: stderr || (error?.message ?? \"\") });\n      },\n    );\n  });\n\n/** Files `warlock check --changed` considers: JavaScript and TypeScript sources. */\nconst SOURCE_FILE = /\\.(?:[cm]?[jt]sx?)$/i;\n\nfunction lines(output: string): string[] {\n  return output\n    .split(/\\r?\\n/)\n    .map((line) => line.trim())\n    .filter((line) => line.length > 0);\n}\n\nfunction gitProblem(message: string): ToolingProblem {\n  return createProblem({ code: \"WL8006\", source: \"environment\", message });\n}\n\n/**\n * The source files changed in the working tree — tracked changes against\n * `base` (or HEAD) plus untracked files — relative to `cwd`, existing only.\n *\n * With `base`, the comparison point is `git merge-base <base> HEAD`, so a\n * branch is compared with where it left `base` rather than with `base`'s tip.\n * A git failure is a `WL8006` environment problem.\n */\nexport async function listChangedFiles(options: {\n  cwd: string;\n  base?: string;\n  git?: GitRunner;\n  /** Keep deleted paths as seeds for reverse dependency selection. */\n  includeDeleted?: boolean;\n}): Promise<{ files: string[] } | { problem: ToolingProblem }> {\n  const git = options.git ?? runGit;\n  let ref = \"HEAD\";\n\n  if (options.base) {\n    const mergeBase = await git([\"merge-base\", options.base, \"HEAD\"], options.cwd);\n\n    if (mergeBase.code !== 0) {\n      return {\n        problem: gitProblem(\n          `git merge-base ${options.base} HEAD failed: ${mergeBase.stderr.trim() || `exit ${mergeBase.code}`}`,\n        ),\n      };\n    }\n\n    ref = mergeBase.stdout.trim();\n  }\n\n  // `--relative` makes the paths relative to cwd and limits them to it.\n  const diff = await git(\n    [\n      \"diff\",\n      \"--name-only\",\n      \"--relative\",\n      options.includeDeleted ? \"--diff-filter=ACMRD\" : \"--diff-filter=ACMR\",\n      ref,\n    ],\n    options.cwd,\n  );\n\n  if (diff.code !== 0) {\n    return {\n      problem: gitProblem(\n        `git diff against ${ref} failed: ${diff.stderr.trim() || `exit ${diff.code}`}`,\n      ),\n    };\n  }\n\n  const untracked = await git([\"ls-files\", \"--others\", \"--exclude-standard\"], options.cwd);\n\n  if (untracked.code !== 0) {\n    return {\n      problem: gitProblem(\n        `git ls-files failed: ${untracked.stderr.trim() || `exit ${untracked.code}`}`,\n      ),\n    };\n  }\n\n  const files = [...new Set([...lines(diff.stdout), ...lines(untracked.stdout)])]\n    .map((file) => file.split(path.sep).join(\"/\"))\n    .filter(\n      (file) =>\n        SOURCE_FILE.test(file) &&\n        (options.includeDeleted || existsSync(path.join(options.cwd, file))),\n    )\n    .sort();\n\n  return { files };\n}\n"],"mappings":";;;;;;;AAYA,MAAa,UAAqB,MAAM,QACtC,IAAI,SAAS,YAAY;CACvB,SACE,OACA,CAAC,GAAG,IAAI,GACR;EAAE;EAAK,UAAU;EAAQ,aAAa;EAAM,WAAW;CAAiB,IACvE,OAAO,QAAQ,WAAW;EAGzB,QAAQ;GAAE,MAFG,UAAU,OAAO,IAAI,OAAO,MAAM,SAAS,WAAW,MAAM,OAAO;GAEhE;GAAQ,QAAQ,WAAW,OAAO,WAAW;EAAI,CAAC;CACpE,CACF;AACF,CAAC;;AAGH,MAAM,cAAc;AAEpB,SAAS,MAAM,QAA0B;CACvC,OAAO,OACJ,MAAM,OAAO,CAAC,CACd,KAAK,SAAS,KAAK,KAAK,CAAC,CAAC,CAC1B,QAAQ,SAAS,KAAK,SAAS,CAAC;AACrC;AAEA,SAAS,WAAW,SAAiC;CACnD,OAAO,cAAc;EAAE,MAAM;EAAU,QAAQ;EAAe;CAAQ,CAAC;AACzE;;;;;;;;;AAUA,eAAsB,iBAAiB,SAMwB;CAC7D,MAAM,MAAM,QAAQ,OAAO;CAC3B,IAAI,MAAM;CAEV,IAAI,QAAQ,MAAM;EAChB,MAAM,YAAY,MAAM,IAAI;GAAC;GAAc,QAAQ;GAAM;EAAM,GAAG,QAAQ,GAAG;EAE7E,IAAI,UAAU,SAAS,GACrB,OAAO,EACL,SAAS,WACP,kBAAkB,QAAQ,KAAK,gBAAgB,UAAU,OAAO,KAAK,KAAK,QAAQ,UAAU,QAC9F,EACF;EAGF,MAAM,UAAU,OAAO,KAAK;CAC9B;CAGA,MAAM,OAAO,MAAM,IACjB;EACE;EACA;EACA;EACA,QAAQ,iBAAiB,wBAAwB;EACjD;CACF,GACA,QAAQ,GACV;CAEA,IAAI,KAAK,SAAS,GAChB,OAAO,EACL,SAAS,WACP,oBAAoB,IAAI,WAAW,KAAK,OAAO,KAAK,KAAK,QAAQ,KAAK,QACxE,EACF;CAGF,MAAM,YAAY,MAAM,IAAI;EAAC;EAAY;EAAY;CAAoB,GAAG,QAAQ,GAAG;CAEvF,IAAI,UAAU,SAAS,GACrB,OAAO,EACL,SAAS,WACP,wBAAwB,UAAU,OAAO,KAAK,KAAK,QAAQ,UAAU,QACvE,EACF;CAYF,OAAO,EAAE,OATK,CAAC,mBAAG,IAAI,IAAI,CAAC,GAAG,MAAM,KAAK,MAAM,GAAG,GAAG,MAAM,UAAU,MAAM,CAAC,CAAC,CAAC,CAAC,CAC5E,KAAK,SAAS,KAAK,MAAM,KAAK,GAAG,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAC7C,QACE,SACC,YAAY,KAAK,IAAI,MACpB,QAAQ,kBAAkB,WAAW,KAAK,KAAK,QAAQ,KAAK,IAAI,CAAC,EACtE,CAAC,CACA,KAEU,EAAE;AACjB"}

package/esm/tooling/check/conventions-step.d.mts

@@ -1,4 +1,6 @@
 import { ToolingProblem } from "../problem.mjs";
+import { ConventionOptions } from "./web-conventions.mjs";
+import { ApiRouteFacts } from "./api-route-references.mjs";
 //#region ../../packages/core/src/tooling/check/conventions-step.d.ts
 /** Matches a project-relative path of a file the framework loads by itself. */
 export declare function isAutoLoadedFile(relativePath: string): boolean;
@@ -15,18 +17,20 @@ export declare function isAutoLoadedFile(relativePath: string): boolean;
  *   and no `public: true`, a redirect target from request input, sensitive
  *   values in logs, unbounded regexes on request input;
  * - `WL9010`–`WL9012` warnings: `await log.*()`, a second controller in one
- *   file, a `kind` discriminator.
+ *   file, a `kind` discriminator;
+ * - `WL9019` (warning) and `WL9020` (error) the browser route references
+ *   (see `apiRouteReferenceFindings`).
  *
  * A `// warlock-check-ignore WL9006` comment on the problem's line or the line
  * above silences that code there; without codes it silences every one.
  */
-export declare function checkFileConventions(relativePath: string, source: string, cwd: string): ToolingProblem[];
+export declare function checkFileConventions(relativePath: string, source: string, cwd: string, options?: ConventionOptions, apiRouteFacts?: ApiRouteFacts): ToolingProblem[];
 /** `WL9004` when the project tsconfig turns on legacy decorators. */
 export declare function checkTsconfigConventions(cwd: string, project?: string): ToolingProblem[];
 /**
  * Run every convention check over `files` (project-relative) plus the
  * project tsconfig.
  */
-export declare function runConventions(cwd: string, files: readonly string[], project?: string): ToolingProblem[];
+export declare function runConventions(cwd: string, files: readonly string[], project?: string, apiRouteFacts?: ApiRouteFacts): ToolingProblem[];
 //#endregion
 //# sourceMappingURL=conventions-step.d.mts.map

package/esm/tooling/check/conventions-step.d.mts.map

@@ -1 +1 @@
-{"version":3,"file":"conventions-step.d.mts","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/conventions-step.ts"],"mappings":";;;wBAiBgB,iBAAiB;;;;;;;;;;;;;;;;;;;wBAiKjB,qBACd,sBACA,gBACA,cACC;;wBAmBa,yBAAyB,aAAa,mBAA4B;;;;;wBAyClE,eACd,aACA,0BACA,mBACC"}
+{"version":3,"file":"conventions-step.d.mts","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/conventions-step.ts"],"mappings":";;;;;wBA2BgB,iBAAiB;;;;;;;;;;;;;;;;;;;;;wBAyKjB,qBACd,sBACA,gBACA,aACA,UAAS,mBACT,gBAAe,gBACd;;wBAkCa,yBAAyB,aAAa,mBAA4B;;;;;wBAyClE,eACd,aACA,0BACA,kBACA,gBAAe,gBACd"}

package/esm/tooling/check/conventions-step.mjs

@@ -1,9 +1,14 @@
+import { isWarlockErrorCode } from "../../errors/error-codes.mjs";
 import { parseSourceFile } from "../syntax/ast.mjs";
 import { importSpecifiers } from "../syntax/imports.mjs";
 import { createProblem } from "../problem.mjs";
+import { importFixes } from "./fixes.mjs";
 import { positionOf } from "./source-ast.mjs";
 import { codeStyleFindings } from "./code-style-conventions.mjs";
 import { securityFindings } from "./security-conventions.mjs";
+import { responseSchemaFindings, webFindings } from "./web-conventions.mjs";
+import { EMPTY_API_ROUTE_FACTS, apiRouteReferenceFindings, collectApiRouteFacts } from "./api-route-references.mjs";
+import { translationProblems } from "./translation-conventions.mjs";
 import path from "node:path";
 import { existsSync, readFileSync } from "node:fs";
 import { parse } from "jsonc-parser";
@@ -62,6 +67,7 @@ function isIgnored(lines, line, code) {
 /** The import conventions (`WL9001`–`WL9003`) of one parsed file. */
 function importProblems(sourceFile, relativePath, cwd) {
 	const problems = [];
+	const fixes = importFixes(relativePath, sourceFile.text);
 	const importerModule = moduleOf(relativePath);
 	for (const { specifier, node } of importsOf(sourceFile)) {
 		const at = {
@@ -70,6 +76,7 @@ function importProblems(sourceFile, relativePath, cwd) {
 		};
 		if (specifier.startsWith(".") && /\.jsx?$/.test(specifier)) problems.push(createProblem({
 			code: "WL9002",
+			fix: fixes.find((fix) => fix.end === node.end - 1),
 			source: "convention",
 			message: `Import "${specifier}" ends in a .js extension.`,
 			...at
@@ -118,16 +125,31 @@ function findingProblem(sourceFile, relativePath, finding) {
 *   and no `public: true`, a redirect target from request input, sensitive
 *   values in logs, unbounded regexes on request input;
 * - `WL9010`–`WL9012` warnings: `await log.*()`, a second controller in one
-*   file, a `kind` discriminator.
+*   file, a `kind` discriminator;
+* - `WL9019` (warning) and `WL9020` (error) the browser route references
+*   (see `apiRouteReferenceFindings`).
 *
 * A `// warlock-check-ignore WL9006` comment on the problem's line or the line
 * above silences that code there; without codes it silences every one.
 */
-function checkFileConventions(relativePath, source, cwd) {
+function checkFileConventions(relativePath, source, cwd, options = {}, apiRouteFacts = EMPTY_API_ROUTE_FACTS) {
 	const sourceFile = parseSourceFile(relativePath, source);
-	const problems = [...importProblems(sourceFile, relativePath, cwd), ...[...securityFindings(sourceFile), ...codeStyleFindings(sourceFile)].map((finding) => findingProblem(sourceFile, relativePath, finding))];
+	const problems = [
+		...importProblems(sourceFile, relativePath, cwd),
+		...translationProblems(relativePath, source),
+		...[
+			...securityFindings(sourceFile),
+			...codeStyleFindings(sourceFile),
+			...webFindings(sourceFile, options),
+			...responseSchemaFindings(sourceFile),
+			...apiRouteReferenceFindings(sourceFile, apiRouteFacts)
+		].map((finding) => findingProblem(sourceFile, relativePath, finding))
+	];
 	const lines = source.split(/\r?\n/);
-	return problems.filter((problem) => problem.line === null || !isIgnored(lines, problem.line, problem.code)).sort((first, second) => (first.line ?? 0) - (second.line ?? 0) || (first.column ?? 0) - (second.column ?? 0));
+	return problems.filter((problem) => options.rules?.[problem.code] !== "off").map((problem) => ({
+		...problem,
+		severity: options.rules?.[problem.code] === "error" ? "error" : options.rules?.[problem.code] === "warning" ? "warning" : problem.severity
+	})).filter((problem) => problem.line === null || !isIgnored(lines, problem.line, problem.code)).sort((first, second) => (first.line ?? 0) - (second.line ?? 0) || (first.column ?? 0) - (second.column ?? 0));
 }
 /** `WL9004` when the project tsconfig turns on legacy decorators. */
 function checkTsconfigConventions(cwd, project = "tsconfig.json") {
@@ -154,8 +176,15 @@ function checkTsconfigConventions(cwd, project = "tsconfig.json") {
 * Run every convention check over `files` (project-relative) plus the
 * project tsconfig.
 */
-function runConventions(cwd, files, project) {
-	const problems = checkTsconfigConventions(cwd, project);
+function runConventions(cwd, files, project, apiRouteFacts = collectApiRouteFacts(cwd, files)) {
+	const options = readConventionOptions(cwd);
+	const problems = checkTsconfigConventions(cwd, project).filter((problem) => options.rules?.[problem.code] !== "off").map((problem) => {
+		const severity = options.rules?.[problem.code];
+		return severity === "error" || severity === "warning" ? {
+			...problem,
+			severity
+		} : problem;
+	});
 	for (const file of files) {
 		let source;
 		try {
@@ -163,11 +192,33 @@ function runConventions(cwd, files, project) {
 		} catch {
 			continue;
 		}
-		problems.push(...checkFileConventions(file, source, cwd));
+		problems.push(...checkFileConventions(file, source, cwd, options, apiRouteFacts));
 	}
 	return problems;
 }
+/** Optional JSON configuration; malformed configuration is reported by the check step. */
+function readConventionOptions(cwd) {
+	const file = path.join(cwd, ".warlock-check.json");
+	if (!existsSync(file)) return {};
+	const value = JSON.parse(readFileSync(file, "utf8"));
+	if (!value || typeof value !== "object") throw new Error(".warlock-check.json must be an object.");
+	const limit = Reflect.get(value, "componentLineLimit");
+	if (limit !== void 0 && (typeof limit !== "number" || !Number.isInteger(limit) || limit < 1)) throw new Error("componentLineLimit must be a positive integer.");
+	const rules = Reflect.get(value, "rules");
+	const overrides = {};
+	if (rules !== void 0) {
+		if (!rules || typeof rules !== "object" || Array.isArray(rules)) throw new Error("rules must be an object.");
+		for (const [code, severity] of Object.entries(rules)) {
+			if (!isWarlockErrorCode(code) || severity !== "off" && severity !== "warning" && severity !== "error") throw new Error("rules must map registered WL codes to off, warning or error.");
+			overrides[code] = severity;
+		}
+	}
+	return {
+		...typeof limit === "number" ? { componentLineLimit: limit } : {},
+		rules: overrides
+	};
+}
 
 //#endregion
-export { checkFileConventions, checkTsconfigConventions, isAutoLoadedFile, runConventions };
+export { checkFileConventions, checkTsconfigConventions, isAutoLoadedFile, readConventionOptions, resolveSpecifier, runConventions };
 //# sourceMappingURL=conventions-step.mjs.map

package/esm/tooling/check/conventions-step.mjs.map

@@ -1 +1 @@
-{"version":3,"file":"conventions-step.mjs","names":["ast.parseSourceFile"],"sources":["../../../../../../../../../packages/core/src/tooling/check/conventions-step.ts"],"sourcesContent":["import { existsSync, readFileSync } from \"node:fs\";\nimport path from \"node:path\";\nimport { parse } from \"jsonc-parser\";\nimport { importSpecifiers } from \"../syntax/imports\";\nimport * as ast from \"../syntax/ast\";\nimport { createProblem, type ToolingProblem } from \"../problem\";\nimport { codeStyleFindings } from \"./code-style-conventions\";\nimport { securityFindings } from \"./security-conventions\";\nimport { positionOf, type SourceFinding } from \"./source-ast\";\n\n/** Extensions tried when resolving an extension-less import. */\nconst RESOLVE_EXTENSIONS = [\".ts\", \".tsx\", \".js\", \".jsx\", \"/index.ts\", \"/index.tsx\"];\n\n/** Test files may import anything on purpose. */\nconst TEST_FILE = /\\.(?:test|spec)\\.[cm]?[jt]sx?$/i;\n\n/** Matches a project-relative path of a file the framework loads by itself. */\nexport function isAutoLoadedFile(relativePath: string): boolean {\n  return (\n    /^src\\/app\\/main\\.tsx?$/.test(relativePath) ||\n    /^src\\/app\\/[^/]+\\/(?:routes|main|worker)\\.tsx?$/.test(relativePath) ||\n    /^src\\/app\\/.+\\/events\\/[^/]+\\.tsx?$/.test(relativePath)\n  );\n}\n\n/** The module folder (`src/app/<module>`) a project-relative path lives in, if any. */\nfunction moduleOf(relativePath: string): string | undefined {\n  return /^src\\/app\\/([^/]+)\\//.exec(relativePath)?.[1];\n}\n\nfunction toPosix(file: string): string {\n  return file.split(path.sep).join(\"/\");\n}\n\n/** Resolve a relative or `app/...` specifier to a project-relative file, if it exists. */\nfunction resolveSpecifier(specifier: string, importer: string, cwd: string): string | undefined {\n  let base: string;\n\n  if (specifier.startsWith(\".\")) {\n    base = path.resolve(path.dirname(path.join(cwd, importer)), specifier);\n  } else if (specifier.startsWith(\"app/\")) {\n    base = path.join(cwd, \"src\", specifier);\n  } else {\n    return undefined;\n  }\n\n  const candidates = [base, ...RESOLVE_EXTENSIONS.map((extension) => `${base}${extension}`)];\n  const withoutJs = base.replace(/\\.jsx?$/, \"\");\n\n  if (withoutJs !== base) {\n    candidates.push(...RESOLVE_EXTENSIONS.map((extension) => `${withoutJs}${extension}`));\n  }\n\n  const found = candidates.find(\n    (candidate) => existsSync(candidate) && path.extname(candidate) !== \"\",\n  );\n\n  return found ? toPosix(path.relative(cwd, found)) : undefined;\n}\n\n/** Every module specifier a source file imports, with its position. */\nfunction importsOf(sourceFile: ast.SourceFile): { specifier: string; node: ast.Node }[] {\n  return importSpecifiers(sourceFile.fileName, sourceFile.text);\n}\n\n/** `// warlock-check-ignore [WLxxxx …]` on a problem's line or the line above it. */\nconst IGNORE_COMMENT = /\\/[/*]\\s*warlock-check-ignore\\b(.*)/;\n\n/** Whether an ignore comment on `line` (1-based) or the line before covers `code`. */\nfunction isIgnored(lines: readonly string[], line: number, code: string): boolean {\n  for (const text of [lines[line - 1], lines[line - 2]]) {\n    const match = text === undefined ? null : IGNORE_COMMENT.exec(text);\n\n    if (!match) continue;\n\n    const codes = match[1]?.match(/WL\\d{4}/g);\n\n    if (!codes || codes.includes(code)) return true;\n  }\n\n  return false;\n}\n\n/** The import conventions (`WL9001`–`WL9003`) of one parsed file. */\nfunction importProblems(\n  sourceFile: ast.SourceFile,\n  relativePath: string,\n  cwd: string,\n): ToolingProblem[] {\n  const problems: ToolingProblem[] = [];\n  const importerModule = moduleOf(relativePath);\n\n  for (const { specifier, node } of importsOf(sourceFile)) {\n    const at = { file: relativePath, ...positionOf(sourceFile, node) };\n\n    if (specifier.startsWith(\".\") && /\\.jsx?$/.test(specifier)) {\n      problems.push(\n        createProblem({\n          code: \"WL9002\",\n          source: \"convention\",\n          message: `Import \"${specifier}\" ends in a .js extension.`,\n          ...at,\n        }),\n      );\n    }\n\n    const target = resolveSpecifier(specifier, relativePath, cwd);\n\n    if (!target) continue;\n\n    if (!TEST_FILE.test(relativePath) && isAutoLoadedFile(target)) {\n      problems.push(\n        createProblem({\n          code: \"WL9001\",\n          source: \"convention\",\n          message: `\"${specifier}\" imports ${target}, which the framework loads by itself.`,\n          ...at,\n        }),\n      );\n    }\n\n    const targetModule = moduleOf(target);\n\n    if (\n      specifier.startsWith(\".\") &&\n      importerModule !== undefined &&\n      targetModule !== undefined &&\n      importerModule !== targetModule\n    ) {\n      const alias = `app/${target.replace(/^src\\/app\\//, \"\").replace(/\\.[cm]?[jt]sx?$/, \"\")}`;\n\n      problems.push(\n        createProblem({\n          code: \"WL9003\",\n          source: \"convention\",\n          message: `\"${specifier}\" reaches from module \"${importerModule}\" into \"${targetModule}\"; import \"${alias}\" instead.`,\n          ...at,\n        }),\n      );\n    }\n  }\n\n  return problems;\n}\n\nfunction findingProblem(\n  sourceFile: ast.SourceFile,\n  relativePath: string,\n  finding: SourceFinding,\n): ToolingProblem {\n  return createProblem({\n    code: finding.code,\n    source: \"convention\",\n    severity: finding.severity ?? \"error\",\n    message: finding.message,\n    file: relativePath,\n    ...positionOf(sourceFile, finding.node),\n  });\n}\n\n/**\n * The convention problems in one source file, in source order:\n *\n * - `WL9001` an import of a file the framework loads by itself (`routes.ts`,\n *   `main.ts`, `worker.ts`, `events/*`) — importing it registers it twice;\n * - `WL9002` a relative import ending in `.js`;\n * - `WL9003` a relative import from one `src/app/<module>` into another, which\n *   should use the `app/<module>/...` alias;\n * - `WL9005`–`WL9009` the security conventions (see `securityFindings`):\n *   `request.all()` into a query or service, a mutating route with no guard\n *   and no `public: true`, a redirect target from request input, sensitive\n *   values in logs, unbounded regexes on request input;\n * - `WL9010`–`WL9012` warnings: `await log.*()`, a second controller in one\n *   file, a `kind` discriminator.\n *\n * A `// warlock-check-ignore WL9006` comment on the problem's line or the line\n * above silences that code there; without codes it silences every one.\n */\nexport function checkFileConventions(\n  relativePath: string,\n  source: string,\n  cwd: string,\n): ToolingProblem[] {\n  const sourceFile = ast.parseSourceFile(relativePath, source);\n  const problems = [\n    ...importProblems(sourceFile, relativePath, cwd),\n    ...[...securityFindings(sourceFile), ...codeStyleFindings(sourceFile)].map((finding) =>\n      findingProblem(sourceFile, relativePath, finding),\n    ),\n  ];\n  const lines = source.split(/\\r?\\n/);\n\n  return problems\n    .filter((problem) => problem.line === null || !isIgnored(lines, problem.line, problem.code))\n    .sort(\n      (first, second) =>\n        (first.line ?? 0) - (second.line ?? 0) || (first.column ?? 0) - (second.column ?? 0),\n    );\n}\n\n/** `WL9004` when the project tsconfig turns on legacy decorators. */\nexport function checkTsconfigConventions(cwd: string, project = \"tsconfig.json\"): ToolingProblem[] {\n  const file = path.join(cwd, project);\n\n  if (!existsSync(file)) return [];\n\n  const text = readFileSync(file, \"utf8\");\n  const config: unknown = parse(text);\n  const options: unknown =\n    typeof config === \"object\" && config !== null\n      ? Reflect.get(config, \"compilerOptions\")\n      : undefined;\n\n  if (\n    typeof options !== \"object\" ||\n    options === null ||\n    Reflect.get(options, \"experimentalDecorators\") !== true\n  ) {\n    return [];\n  }\n\n  const index = text.indexOf(\"experimentalDecorators\");\n  const before = index === -1 ? \"\" : text.slice(0, index);\n  const line = index === -1 ? null : before.split(\"\\n\").length;\n  const column = index === -1 ? null : index - before.lastIndexOf(\"\\n\");\n\n  return [\n    createProblem({\n      code: \"WL9004\",\n      source: \"convention\",\n      message: '\"experimentalDecorators\": true switches TypeScript to legacy decorators.',\n      file: toPosix(project),\n      line,\n      column,\n    }),\n  ];\n}\n\n/**\n * Run every convention check over `files` (project-relative) plus the\n * project tsconfig.\n */\nexport function runConventions(\n  cwd: string,\n  files: readonly string[],\n  project?: string,\n): ToolingProblem[] {\n  const problems = checkTsconfigConventions(cwd, project);\n\n  for (const file of files) {\n    let source: string;\n\n    try {\n      source = readFileSync(path.join(cwd, file), \"utf8\");\n    } catch {\n      continue;\n    }\n\n    problems.push(...checkFileConventions(file, source, cwd));\n  }\n\n  return problems;\n}\n"],"mappings":";;;;;;;;;;;;AAWA,MAAM,qBAAqB;CAAC;CAAO;CAAQ;CAAO;CAAQ;CAAa;AAAY;;AAGnF,MAAM,YAAY;;AAGlB,SAAgB,iBAAiB,cAA+B;CAC9D,OACE,yBAAyB,KAAK,YAAY,KAC1C,kDAAkD,KAAK,YAAY,KACnE,sCAAsC,KAAK,YAAY;AAE3D;;AAGA,SAAS,SAAS,cAA0C;CAC1D,OAAO,uBAAuB,KAAK,YAAY,CAAC,GAAG;AACrD;AAEA,SAAS,QAAQ,MAAsB;CACrC,OAAO,KAAK,MAAM,KAAK,GAAG,CAAC,CAAC,KAAK,GAAG;AACtC;;AAGA,SAAS,iBAAiB,WAAmB,UAAkB,KAAiC;CAC9F,IAAI;CAEJ,IAAI,UAAU,WAAW,GAAG,GAC1B,OAAO,KAAK,QAAQ,KAAK,QAAQ,KAAK,KAAK,KAAK,QAAQ,CAAC,GAAG,SAAS;MAChE,IAAI,UAAU,WAAW,MAAM,GACpC,OAAO,KAAK,KAAK,KAAK,OAAO,SAAS;MAEtC;CAGF,MAAM,aAAa,CAAC,MAAM,GAAG,mBAAmB,KAAK,cAAc,GAAG,OAAO,WAAW,CAAC;CACzF,MAAM,YAAY,KAAK,QAAQ,WAAW,EAAE;CAE5C,IAAI,cAAc,MAChB,WAAW,KAAK,GAAG,mBAAmB,KAAK,cAAc,GAAG,YAAY,WAAW,CAAC;CAGtF,MAAM,QAAQ,WAAW,MACtB,cAAc,WAAW,SAAS,KAAK,KAAK,QAAQ,SAAS,MAAM,EACtE;CAEA,OAAO,QAAQ,QAAQ,KAAK,SAAS,KAAK,KAAK,CAAC,IAAI;AACtD;;AAGA,SAAS,UAAU,YAAqE;CACtF,OAAO,iBAAiB,WAAW,UAAU,WAAW,IAAI;AAC9D;;AAGA,MAAM,iBAAiB;;AAGvB,SAAS,UAAU,OAA0B,MAAc,MAAuB;CAChF,KAAK,MAAM,QAAQ,CAAC,MAAM,OAAO,IAAI,MAAM,OAAO,EAAE,GAAG;EACrD,MAAM,QAAQ,SAAS,SAAY,OAAO,eAAe,KAAK,IAAI;EAElE,IAAI,CAAC,OAAO;EAEZ,MAAM,QAAQ,MAAM,EAAE,EAAE,MAAM,UAAU;EAExC,IAAI,CAAC,SAAS,MAAM,SAAS,IAAI,GAAG,OAAO;CAC7C;CAEA,OAAO;AACT;;AAGA,SAAS,eACP,YACA,cACA,KACkB;CAClB,MAAM,WAA6B,CAAC;CACpC,MAAM,iBAAiB,SAAS,YAAY;CAE5C,KAAK,MAAM,EAAE,WAAW,UAAU,UAAU,UAAU,GAAG;EACvD,MAAM,KAAK;GAAE,MAAM;GAAc,GAAG,WAAW,YAAY,IAAI;EAAE;EAEjE,IAAI,UAAU,WAAW,GAAG,KAAK,UAAU,KAAK,SAAS,GACvD,SAAS,KACP,cAAc;GACZ,MAAM;GACN,QAAQ;GACR,SAAS,WAAW,UAAU;GAC9B,GAAG;EACL,CAAC,CACH;EAGF,MAAM,SAAS,iBAAiB,WAAW,cAAc,GAAG;EAE5D,IAAI,CAAC,QAAQ;EAEb,IAAI,CAAC,UAAU,KAAK,YAAY,KAAK,iBAAiB,MAAM,GAC1D,SAAS,KACP,cAAc;GACZ,MAAM;GACN,QAAQ;GACR,SAAS,IAAI,UAAU,YAAY,OAAO;GAC1C,GAAG;EACL,CAAC,CACH;EAGF,MAAM,eAAe,SAAS,MAAM;EAEpC,IACE,UAAU,WAAW,GAAG,KACxB,mBAAmB,UACnB,iBAAiB,UACjB,mBAAmB,cACnB;GACA,MAAM,QAAQ,OAAO,OAAO,QAAQ,eAAe,EAAE,CAAC,CAAC,QAAQ,mBAAmB,EAAE;GAEpF,SAAS,KACP,cAAc;IACZ,MAAM;IACN,QAAQ;IACR,SAAS,IAAI,UAAU,yBAAyB,eAAe,UAAU,aAAa,aAAa,MAAM;IACzG,GAAG;GACL,CAAC,CACH;EACF;CACF;CAEA,OAAO;AACT;AAEA,SAAS,eACP,YACA,cACA,SACgB;CAChB,OAAO,cAAc;EACnB,MAAM,QAAQ;EACd,QAAQ;EACR,UAAU,QAAQ,YAAY;EAC9B,SAAS,QAAQ;EACjB,MAAM;EACN,GAAG,WAAW,YAAY,QAAQ,IAAI;CACxC,CAAC;AACH;;;;;;;;;;;;;;;;;;;AAoBA,SAAgB,qBACd,cACA,QACA,KACkB;CAClB,MAAM,aAAaA,gBAAoB,cAAc,MAAM;CAC3D,MAAM,WAAW,CACf,GAAG,eAAe,YAAY,cAAc,GAAG,GAC/C,GAAG,CAAC,GAAG,iBAAiB,UAAU,GAAG,GAAG,kBAAkB,UAAU,CAAC,CAAC,CAAC,KAAK,YAC1E,eAAe,YAAY,cAAc,OAAO,CAClD,CACF;CACA,MAAM,QAAQ,OAAO,MAAM,OAAO;CAElC,OAAO,SACJ,QAAQ,YAAY,QAAQ,SAAS,QAAQ,CAAC,UAAU,OAAO,QAAQ,MAAM,QAAQ,IAAI,CAAC,CAAC,CAC3F,MACE,OAAO,YACL,MAAM,QAAQ,MAAM,OAAO,QAAQ,OAAO,MAAM,UAAU,MAAM,OAAO,UAAU,EACtF;AACJ;;AAGA,SAAgB,yBAAyB,KAAa,UAAU,iBAAmC;CACjG,MAAM,OAAO,KAAK,KAAK,KAAK,OAAO;CAEnC,IAAI,CAAC,WAAW,IAAI,GAAG,OAAO,CAAC;CAE/B,MAAM,OAAO,aAAa,MAAM,MAAM;CACtC,MAAM,SAAkB,MAAM,IAAI;CAClC,MAAM,UACJ,OAAO,WAAW,YAAY,WAAW,OACrC,QAAQ,IAAI,QAAQ,iBAAiB,IACrC;CAEN,IACE,OAAO,YAAY,YACnB,YAAY,QACZ,QAAQ,IAAI,SAAS,wBAAwB,MAAM,MAEnD,OAAO,CAAC;CAGV,MAAM,QAAQ,KAAK,QAAQ,wBAAwB;CACnD,MAAM,SAAS,UAAU,KAAK,KAAK,KAAK,MAAM,GAAG,KAAK;CACtD,MAAM,OAAO,UAAU,KAAK,OAAO,OAAO,MAAM,IAAI,CAAC,CAAC;CACtD,MAAM,SAAS,UAAU,KAAK,OAAO,QAAQ,OAAO,YAAY,IAAI;CAEpE,OAAO,CACL,cAAc;EACZ,MAAM;EACN,QAAQ;EACR,SAAS;EACT,MAAM,QAAQ,OAAO;EACrB;EACA;CACF,CAAC,CACH;AACF;;;;;AAMA,SAAgB,eACd,KACA,OACA,SACkB;CAClB,MAAM,WAAW,yBAAyB,KAAK,OAAO;CAEtD,KAAK,MAAM,QAAQ,OAAO;EACxB,IAAI;EAEJ,IAAI;GACF,SAAS,aAAa,KAAK,KAAK,KAAK,IAAI,GAAG,MAAM;EACpD,QAAQ;GACN;EACF;EAEA,SAAS,KAAK,GAAG,qBAAqB,MAAM,QAAQ,GAAG,CAAC;CAC1D;CAEA,OAAO;AACT"}
+{"version":3,"file":"conventions-step.mjs","names":["ast.parseSourceFile"],"sources":["../../../../../../../../../packages/core/src/tooling/check/conventions-step.ts"],"sourcesContent":["import { isWarlockErrorCode } from \"../../errors/error-codes\";\nimport { existsSync, readFileSync } from \"node:fs\";\nimport path from \"node:path\";\nimport { parse } from \"jsonc-parser\";\nimport { importSpecifiers } from \"../syntax/imports\";\nimport * as ast from \"../syntax/ast\";\nimport { createProblem, type ToolingProblem } from \"../problem\";\nimport { importFixes } from \"./fixes\";\nimport { codeStyleFindings } from \"./code-style-conventions\";\nimport { securityFindings } from \"./security-conventions\";\nimport { webFindings, responseSchemaFindings, type ConventionOptions } from \"./web-conventions\";\nimport {\n  apiRouteReferenceFindings,\n  collectApiRouteFacts,\n  EMPTY_API_ROUTE_FACTS,\n  type ApiRouteFacts,\n} from \"./api-route-references\";\nimport { translationProblems } from \"./translation-conventions\";\nimport { positionOf, type SourceFinding } from \"./source-ast\";\n\n/** Extensions tried when resolving an extension-less import. */\nconst RESOLVE_EXTENSIONS = [\".ts\", \".tsx\", \".js\", \".jsx\", \"/index.ts\", \"/index.tsx\"];\n\n/** Test files may import anything on purpose. */\nconst TEST_FILE = /\\.(?:test|spec)\\.[cm]?[jt]sx?$/i;\n\n/** Matches a project-relative path of a file the framework loads by itself. */\nexport function isAutoLoadedFile(relativePath: string): boolean {\n  return (\n    /^src\\/app\\/main\\.tsx?$/.test(relativePath) ||\n    /^src\\/app\\/[^/]+\\/(?:routes|main|worker)\\.tsx?$/.test(relativePath) ||\n    /^src\\/app\\/.+\\/events\\/[^/]+\\.tsx?$/.test(relativePath)\n  );\n}\n\n/** The module folder (`src/app/<module>`) a project-relative path lives in, if any. */\nfunction moduleOf(relativePath: string): string | undefined {\n  return /^src\\/app\\/([^/]+)\\//.exec(relativePath)?.[1];\n}\n\nfunction toPosix(file: string): string {\n  return file.split(path.sep).join(\"/\");\n}\n\n/** Resolve a relative or `app/...` specifier to a project-relative file, if it exists. */\nexport function resolveSpecifier(\n  specifier: string,\n  importer: string,\n  cwd: string,\n): string | undefined {\n  let base: string;\n\n  if (specifier.startsWith(\".\")) {\n    base = path.resolve(path.dirname(path.join(cwd, importer)), specifier);\n  } else if (specifier.startsWith(\"app/\")) {\n    base = path.join(cwd, \"src\", specifier);\n  } else {\n    return undefined;\n  }\n\n  const candidates = [base, ...RESOLVE_EXTENSIONS.map((extension) => `${base}${extension}`)];\n  const withoutJs = base.replace(/\\.jsx?$/, \"\");\n\n  if (withoutJs !== base) {\n    candidates.push(...RESOLVE_EXTENSIONS.map((extension) => `${withoutJs}${extension}`));\n  }\n\n  const found = candidates.find(\n    (candidate) => existsSync(candidate) && path.extname(candidate) !== \"\",\n  );\n\n  return found ? toPosix(path.relative(cwd, found)) : undefined;\n}\n\n/** Every module specifier a source file imports, with its position. */\nfunction importsOf(sourceFile: ast.SourceFile): { specifier: string; node: ast.Node }[] {\n  return importSpecifiers(sourceFile.fileName, sourceFile.text);\n}\n\n/** `// warlock-check-ignore [WLxxxx …]` on a problem's line or the line above it. */\nconst IGNORE_COMMENT = /\\/[/*]\\s*warlock-check-ignore\\b(.*)/;\n\n/** Whether an ignore comment on `line` (1-based) or the line before covers `code`. */\nfunction isIgnored(lines: readonly string[], line: number, code: string): boolean {\n  for (const text of [lines[line - 1], lines[line - 2]]) {\n    const match = text === undefined ? null : IGNORE_COMMENT.exec(text);\n\n    if (!match) continue;\n\n    const codes = match[1]?.match(/WL\\d{4}/g);\n\n    if (!codes || codes.includes(code)) return true;\n  }\n\n  return false;\n}\n\n/** The import conventions (`WL9001`–`WL9003`) of one parsed file. */\nfunction importProblems(\n  sourceFile: ast.SourceFile,\n  relativePath: string,\n  cwd: string,\n): ToolingProblem[] {\n  const problems: ToolingProblem[] = [];\n  const fixes = importFixes(relativePath, sourceFile.text);\n  const importerModule = moduleOf(relativePath);\n\n  for (const { specifier, node } of importsOf(sourceFile)) {\n    const at = { file: relativePath, ...positionOf(sourceFile, node) };\n\n    if (specifier.startsWith(\".\") && /\\.jsx?$/.test(specifier)) {\n      problems.push(\n        createProblem({\n          code: \"WL9002\",\n          fix: fixes.find((fix) => fix.end === node.end - 1),\n          source: \"convention\",\n          message: `Import \"${specifier}\" ends in a .js extension.`,\n          ...at,\n        }),\n      );\n    }\n\n    const target = resolveSpecifier(specifier, relativePath, cwd);\n\n    if (!target) continue;\n\n    if (!TEST_FILE.test(relativePath) && isAutoLoadedFile(target)) {\n      problems.push(\n        createProblem({\n          code: \"WL9001\",\n          source: \"convention\",\n          message: `\"${specifier}\" imports ${target}, which the framework loads by itself.`,\n          ...at,\n        }),\n      );\n    }\n\n    const targetModule = moduleOf(target);\n\n    if (\n      specifier.startsWith(\".\") &&\n      importerModule !== undefined &&\n      targetModule !== undefined &&\n      importerModule !== targetModule\n    ) {\n      const alias = `app/${target.replace(/^src\\/app\\//, \"\").replace(/\\.[cm]?[jt]sx?$/, \"\")}`;\n\n      problems.push(\n        createProblem({\n          code: \"WL9003\",\n          source: \"convention\",\n          message: `\"${specifier}\" reaches from module \"${importerModule}\" into \"${targetModule}\"; import \"${alias}\" instead.`,\n          ...at,\n        }),\n      );\n    }\n  }\n\n  return problems;\n}\n\nfunction findingProblem(\n  sourceFile: ast.SourceFile,\n  relativePath: string,\n  finding: SourceFinding,\n): ToolingProblem {\n  return createProblem({\n    code: finding.code,\n    source: \"convention\",\n    severity: finding.severity ?? \"error\",\n    message: finding.message,\n    file: relativePath,\n    ...positionOf(sourceFile, finding.node),\n  });\n}\n\n/**\n * The convention problems in one source file, in source order:\n *\n * - `WL9001` an import of a file the framework loads by itself (`routes.ts`,\n *   `main.ts`, `worker.ts`, `events/*`) — importing it registers it twice;\n * - `WL9002` a relative import ending in `.js`;\n * - `WL9003` a relative import from one `src/app/<module>` into another, which\n *   should use the `app/<module>/...` alias;\n * - `WL9005`–`WL9009` the security conventions (see `securityFindings`):\n *   `request.all()` into a query or service, a mutating route with no guard\n *   and no `public: true`, a redirect target from request input, sensitive\n *   values in logs, unbounded regexes on request input;\n * - `WL9010`–`WL9012` warnings: `await log.*()`, a second controller in one\n *   file, a `kind` discriminator;\n * - `WL9019` (warning) and `WL9020` (error) the browser route references\n *   (see `apiRouteReferenceFindings`).\n *\n * A `// warlock-check-ignore WL9006` comment on the problem's line or the line\n * above silences that code there; without codes it silences every one.\n */\nexport function checkFileConventions(\n  relativePath: string,\n  source: string,\n  cwd: string,\n  options: ConventionOptions = {},\n  apiRouteFacts: ApiRouteFacts = EMPTY_API_ROUTE_FACTS,\n): ToolingProblem[] {\n  const sourceFile = ast.parseSourceFile(relativePath, source);\n  const problems = [\n    ...importProblems(sourceFile, relativePath, cwd),\n    ...translationProblems(relativePath, source),\n    ...[\n      ...securityFindings(sourceFile),\n      ...codeStyleFindings(sourceFile),\n      ...webFindings(sourceFile, options),\n      ...responseSchemaFindings(sourceFile),\n      ...apiRouteReferenceFindings(sourceFile, apiRouteFacts),\n    ].map((finding) => findingProblem(sourceFile, relativePath, finding)),\n  ];\n  const lines = source.split(/\\r?\\n/);\n\n  return problems\n    .filter((problem) => options.rules?.[problem.code] !== \"off\")\n    .map((problem) => ({\n      ...problem,\n      severity:\n        options.rules?.[problem.code] === \"error\"\n          ? (\"error\" as const)\n          : options.rules?.[problem.code] === \"warning\"\n            ? (\"warning\" as const)\n            : problem.severity,\n    }))\n    .filter((problem) => problem.line === null || !isIgnored(lines, problem.line, problem.code))\n    .sort(\n      (first, second) =>\n        (first.line ?? 0) - (second.line ?? 0) || (first.column ?? 0) - (second.column ?? 0),\n    );\n}\n\n/** `WL9004` when the project tsconfig turns on legacy decorators. */\nexport function checkTsconfigConventions(cwd: string, project = \"tsconfig.json\"): ToolingProblem[] {\n  const file = path.join(cwd, project);\n\n  if (!existsSync(file)) return [];\n\n  const text = readFileSync(file, \"utf8\");\n  const config: unknown = parse(text);\n  const options: unknown =\n    typeof config === \"object\" && config !== null\n      ? Reflect.get(config, \"compilerOptions\")\n      : undefined;\n\n  if (\n    typeof options !== \"object\" ||\n    options === null ||\n    Reflect.get(options, \"experimentalDecorators\") !== true\n  ) {\n    return [];\n  }\n\n  const index = text.indexOf(\"experimentalDecorators\");\n  const before = index === -1 ? \"\" : text.slice(0, index);\n  const line = index === -1 ? null : before.split(\"\\n\").length;\n  const column = index === -1 ? null : index - before.lastIndexOf(\"\\n\");\n\n  return [\n    createProblem({\n      code: \"WL9004\",\n      source: \"convention\",\n      message: '\"experimentalDecorators\": true switches TypeScript to legacy decorators.',\n      file: toPosix(project),\n      line,\n      column,\n    }),\n  ];\n}\n\n/**\n * Run every convention check over `files` (project-relative) plus the\n * project tsconfig.\n */\nexport function runConventions(\n  cwd: string,\n  files: readonly string[],\n  project?: string,\n  apiRouteFacts: ApiRouteFacts = collectApiRouteFacts(cwd, files),\n): ToolingProblem[] {\n  const options = readConventionOptions(cwd);\n  const problems = checkTsconfigConventions(cwd, project)\n    .filter((problem) => options.rules?.[problem.code] !== \"off\")\n    .map((problem) => {\n      const severity = options.rules?.[problem.code];\n      return severity === \"error\" || severity === \"warning\" ? { ...problem, severity } : problem;\n    });\n\n  for (const file of files) {\n    let source: string;\n\n    try {\n      source = readFileSync(path.join(cwd, file), \"utf8\");\n    } catch {\n      continue;\n    }\n\n    problems.push(...checkFileConventions(file, source, cwd, options, apiRouteFacts));\n  }\n\n  return problems;\n}\n\n/** Optional JSON configuration; malformed configuration is reported by the check step. */\nexport function readConventionOptions(cwd: string): ConventionOptions {\n  const file = path.join(cwd, \".warlock-check.json\");\n  if (!existsSync(file)) return {};\n  const value: unknown = JSON.parse(readFileSync(file, \"utf8\"));\n  if (!value || typeof value !== \"object\")\n    throw new Error(\".warlock-check.json must be an object.\");\n  const limit: unknown = Reflect.get(value, \"componentLineLimit\");\n  if (limit !== undefined && (typeof limit !== \"number\" || !Number.isInteger(limit) || limit < 1))\n    throw new Error(\"componentLineLimit must be a positive integer.\");\n  const rules: unknown = Reflect.get(value, \"rules\");\n  const overrides: NonNullable<ConventionOptions[\"rules\"]> = {};\n  if (rules !== undefined) {\n    if (!rules || typeof rules !== \"object\" || Array.isArray(rules))\n      throw new Error(\"rules must be an object.\");\n    for (const [code, severity] of Object.entries(rules)) {\n      if (\n        !isWarlockErrorCode(code) ||\n        (severity !== \"off\" && severity !== \"warning\" && severity !== \"error\")\n      )\n        throw new Error(\"rules must map registered WL codes to off, warning or error.\");\n      overrides[code] = severity;\n    }\n  }\n  return {\n    ...(typeof limit === \"number\" ? { componentLineLimit: limit } : {}),\n    rules: overrides,\n  };\n}\n"],"mappings":";;;;;;;;;;;;;;;;;AAqBA,MAAM,qBAAqB;CAAC;CAAO;CAAQ;CAAO;CAAQ;CAAa;AAAY;;AAGnF,MAAM,YAAY;;AAGlB,SAAgB,iBAAiB,cAA+B;CAC9D,OACE,yBAAyB,KAAK,YAAY,KAC1C,kDAAkD,KAAK,YAAY,KACnE,sCAAsC,KAAK,YAAY;AAE3D;;AAGA,SAAS,SAAS,cAA0C;CAC1D,OAAO,uBAAuB,KAAK,YAAY,CAAC,GAAG;AACrD;AAEA,SAAS,QAAQ,MAAsB;CACrC,OAAO,KAAK,MAAM,KAAK,GAAG,CAAC,CAAC,KAAK,GAAG;AACtC;;AAGA,SAAgB,iBACd,WACA,UACA,KACoB;CACpB,IAAI;CAEJ,IAAI,UAAU,WAAW,GAAG,GAC1B,OAAO,KAAK,QAAQ,KAAK,QAAQ,KAAK,KAAK,KAAK,QAAQ,CAAC,GAAG,SAAS;MAChE,IAAI,UAAU,WAAW,MAAM,GACpC,OAAO,KAAK,KAAK,KAAK,OAAO,SAAS;MAEtC;CAGF,MAAM,aAAa,CAAC,MAAM,GAAG,mBAAmB,KAAK,cAAc,GAAG,OAAO,WAAW,CAAC;CACzF,MAAM,YAAY,KAAK,QAAQ,WAAW,EAAE;CAE5C,IAAI,cAAc,MAChB,WAAW,KAAK,GAAG,mBAAmB,KAAK,cAAc,GAAG,YAAY,WAAW,CAAC;CAGtF,MAAM,QAAQ,WAAW,MACtB,cAAc,WAAW,SAAS,KAAK,KAAK,QAAQ,SAAS,MAAM,EACtE;CAEA,OAAO,QAAQ,QAAQ,KAAK,SAAS,KAAK,KAAK,CAAC,IAAI;AACtD;;AAGA,SAAS,UAAU,YAAqE;CACtF,OAAO,iBAAiB,WAAW,UAAU,WAAW,IAAI;AAC9D;;AAGA,MAAM,iBAAiB;;AAGvB,SAAS,UAAU,OAA0B,MAAc,MAAuB;CAChF,KAAK,MAAM,QAAQ,CAAC,MAAM,OAAO,IAAI,MAAM,OAAO,EAAE,GAAG;EACrD,MAAM,QAAQ,SAAS,SAAY,OAAO,eAAe,KAAK,IAAI;EAElE,IAAI,CAAC,OAAO;EAEZ,MAAM,QAAQ,MAAM,EAAE,EAAE,MAAM,UAAU;EAExC,IAAI,CAAC,SAAS,MAAM,SAAS,IAAI,GAAG,OAAO;CAC7C;CAEA,OAAO;AACT;;AAGA,SAAS,eACP,YACA,cACA,KACkB;CAClB,MAAM,WAA6B,CAAC;CACpC,MAAM,QAAQ,YAAY,cAAc,WAAW,IAAI;CACvD,MAAM,iBAAiB,SAAS,YAAY;CAE5C,KAAK,MAAM,EAAE,WAAW,UAAU,UAAU,UAAU,GAAG;EACvD,MAAM,KAAK;GAAE,MAAM;GAAc,GAAG,WAAW,YAAY,IAAI;EAAE;EAEjE,IAAI,UAAU,WAAW,GAAG,KAAK,UAAU,KAAK,SAAS,GACvD,SAAS,KACP,cAAc;GACZ,MAAM;GACN,KAAK,MAAM,MAAM,QAAQ,IAAI,QAAQ,KAAK,MAAM,CAAC;GACjD,QAAQ;GACR,SAAS,WAAW,UAAU;GAC9B,GAAG;EACL,CAAC,CACH;EAGF,MAAM,SAAS,iBAAiB,WAAW,cAAc,GAAG;EAE5D,IAAI,CAAC,QAAQ;EAEb,IAAI,CAAC,UAAU,KAAK,YAAY,KAAK,iBAAiB,MAAM,GAC1D,SAAS,KACP,cAAc;GACZ,MAAM;GACN,QAAQ;GACR,SAAS,IAAI,UAAU,YAAY,OAAO;GAC1C,GAAG;EACL,CAAC,CACH;EAGF,MAAM,eAAe,SAAS,MAAM;EAEpC,IACE,UAAU,WAAW,GAAG,KACxB,mBAAmB,UACnB,iBAAiB,UACjB,mBAAmB,cACnB;GACA,MAAM,QAAQ,OAAO,OAAO,QAAQ,eAAe,EAAE,CAAC,CAAC,QAAQ,mBAAmB,EAAE;GAEpF,SAAS,KACP,cAAc;IACZ,MAAM;IACN,QAAQ;IACR,SAAS,IAAI,UAAU,yBAAyB,eAAe,UAAU,aAAa,aAAa,MAAM;IACzG,GAAG;GACL,CAAC,CACH;EACF;CACF;CAEA,OAAO;AACT;AAEA,SAAS,eACP,YACA,cACA,SACgB;CAChB,OAAO,cAAc;EACnB,MAAM,QAAQ;EACd,QAAQ;EACR,UAAU,QAAQ,YAAY;EAC9B,SAAS,QAAQ;EACjB,MAAM;EACN,GAAG,WAAW,YAAY,QAAQ,IAAI;CACxC,CAAC;AACH;;;;;;;;;;;;;;;;;;;;;AAsBA,SAAgB,qBACd,cACA,QACA,KACA,UAA6B,CAAC,GAC9B,gBAA+B,uBACb;CAClB,MAAM,aAAaA,gBAAoB,cAAc,MAAM;CAC3D,MAAM,WAAW;EACf,GAAG,eAAe,YAAY,cAAc,GAAG;EAC/C,GAAG,oBAAoB,cAAc,MAAM;EAC3C,GAAG;GACD,GAAG,iBAAiB,UAAU;GAC9B,GAAG,kBAAkB,UAAU;GAC/B,GAAG,YAAY,YAAY,OAAO;GAClC,GAAG,uBAAuB,UAAU;GACpC,GAAG,0BAA0B,YAAY,aAAa;EACxD,CAAC,CAAC,KAAK,YAAY,eAAe,YAAY,cAAc,OAAO,CAAC;CACtE;CACA,MAAM,QAAQ,OAAO,MAAM,OAAO;CAElC,OAAO,SACJ,QAAQ,YAAY,QAAQ,QAAQ,QAAQ,UAAU,KAAK,CAAC,CAC5D,KAAK,aAAa;EACjB,GAAG;EACH,UACE,QAAQ,QAAQ,QAAQ,UAAU,UAC7B,UACD,QAAQ,QAAQ,QAAQ,UAAU,YAC/B,YACD,QAAQ;CAClB,EAAE,CAAC,CACF,QAAQ,YAAY,QAAQ,SAAS,QAAQ,CAAC,UAAU,OAAO,QAAQ,MAAM,QAAQ,IAAI,CAAC,CAAC,CAC3F,MACE,OAAO,YACL,MAAM,QAAQ,MAAM,OAAO,QAAQ,OAAO,MAAM,UAAU,MAAM,OAAO,UAAU,EACtF;AACJ;;AAGA,SAAgB,yBAAyB,KAAa,UAAU,iBAAmC;CACjG,MAAM,OAAO,KAAK,KAAK,KAAK,OAAO;CAEnC,IAAI,CAAC,WAAW,IAAI,GAAG,OAAO,CAAC;CAE/B,MAAM,OAAO,aAAa,MAAM,MAAM;CACtC,MAAM,SAAkB,MAAM,IAAI;CAClC,MAAM,UACJ,OAAO,WAAW,YAAY,WAAW,OACrC,QAAQ,IAAI,QAAQ,iBAAiB,IACrC;CAEN,IACE,OAAO,YAAY,YACnB,YAAY,QACZ,QAAQ,IAAI,SAAS,wBAAwB,MAAM,MAEnD,OAAO,CAAC;CAGV,MAAM,QAAQ,KAAK,QAAQ,wBAAwB;CACnD,MAAM,SAAS,UAAU,KAAK,KAAK,KAAK,MAAM,GAAG,KAAK;CACtD,MAAM,OAAO,UAAU,KAAK,OAAO,OAAO,MAAM,IAAI,CAAC,CAAC;CACtD,MAAM,SAAS,UAAU,KAAK,OAAO,QAAQ,OAAO,YAAY,IAAI;CAEpE,OAAO,CACL,cAAc;EACZ,MAAM;EACN,QAAQ;EACR,SAAS;EACT,MAAM,QAAQ,OAAO;EACrB;EACA;CACF,CAAC,CACH;AACF;;;;;AAMA,SAAgB,eACd,KACA,OACA,SACA,gBAA+B,qBAAqB,KAAK,KAAK,GAC5C;CAClB,MAAM,UAAU,sBAAsB,GAAG;CACzC,MAAM,WAAW,yBAAyB,KAAK,OAAO,CAAC,CACpD,QAAQ,YAAY,QAAQ,QAAQ,QAAQ,UAAU,KAAK,CAAC,CAC5D,KAAK,YAAY;EAChB,MAAM,WAAW,QAAQ,QAAQ,QAAQ;EACzC,OAAO,aAAa,WAAW,aAAa,YAAY;GAAE,GAAG;GAAS;EAAS,IAAI;CACrF,CAAC;CAEH,KAAK,MAAM,QAAQ,OAAO;EACxB,IAAI;EAEJ,IAAI;GACF,SAAS,aAAa,KAAK,KAAK,KAAK,IAAI,GAAG,MAAM;EACpD,QAAQ;GACN;EACF;EAEA,SAAS,KAAK,GAAG,qBAAqB,MAAM,QAAQ,KAAK,SAAS,aAAa,CAAC;CAClF;CAEA,OAAO;AACT;;AAGA,SAAgB,sBAAsB,KAAgC;CACpE,MAAM,OAAO,KAAK,KAAK,KAAK,qBAAqB;CACjD,IAAI,CAAC,WAAW,IAAI,GAAG,OAAO,CAAC;CAC/B,MAAM,QAAiB,KAAK,MAAM,aAAa,MAAM,MAAM,CAAC;CAC5D,IAAI,CAAC,SAAS,OAAO,UAAU,UAC7B,MAAM,IAAI,MAAM,wCAAwC;CAC1D,MAAM,QAAiB,QAAQ,IAAI,OAAO,oBAAoB;CAC9D,IAAI,UAAU,WAAc,OAAO,UAAU,YAAY,CAAC,OAAO,UAAU,KAAK,KAAK,QAAQ,IAC3F,MAAM,IAAI,MAAM,gDAAgD;CAClE,MAAM,QAAiB,QAAQ,IAAI,OAAO,OAAO;CACjD,MAAM,YAAqD,CAAC;CAC5D,IAAI,UAAU,QAAW;EACvB,IAAI,CAAC,SAAS,OAAO,UAAU,YAAY,MAAM,QAAQ,KAAK,GAC5D,MAAM,IAAI,MAAM,0BAA0B;EAC5C,KAAK,MAAM,CAAC,MAAM,aAAa,OAAO,QAAQ,KAAK,GAAG;GACpD,IACE,CAAC,mBAAmB,IAAI,KACvB,aAAa,SAAS,aAAa,aAAa,aAAa,SAE9D,MAAM,IAAI,MAAM,8DAA8D;GAChF,UAAU,QAAQ;EACpB;CACF;CACA,OAAO;EACL,GAAI,OAAO,UAAU,WAAW,EAAE,oBAAoB,MAAM,IAAI,CAAC;EACjE,OAAO;CACT;AACF"}

package/esm/tooling/check/fixes.d.mts

@@ -0,0 +1,8 @@
+//#region ../../packages/core/src/tooling/check/fixes.d.ts
+export type ProblemFix = {
+  start: number;
+  end: number;
+  text: string;
+};
+//#endregion
+//# sourceMappingURL=fixes.d.mts.map

package/esm/tooling/check/fixes.d.mts.map

@@ -0,0 +1 @@
+{"version":3,"file":"fixes.d.mts","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/fixes.ts"],"mappings":";YAKY;EAAe;EAAe;EAAa"}

package/esm/tooling/check/fixes.mjs

@@ -0,0 +1,31 @@
+import { importSpecifiers } from "../syntax/imports.mjs";
+import path from "node:path";
+import { readFileSync, writeFileSync } from "node:fs";
+
+//#region ../../packages/core/src/tooling/check/fixes.ts
+/** Only relative .js imports: removing the suffix preserves TypeScript resolution. */
+function importFixes(file, source) {
+	return importSpecifiers(file, source).filter(({ specifier, node }) => specifier.startsWith(".") && specifier.endsWith(".js") && /\.js['"]$/.test(source.slice(node.start, node.end))).map(({ node }) => ({
+		start: node.end - 4,
+		end: node.end - 1,
+		text: ""
+	}));
+}
+function applyConventionFixes(cwd, problems) {
+	const files = /* @__PURE__ */ new Map();
+	for (const problem of problems) if (problem.file && problem.fix) {
+		const fixes = files.get(problem.file) ?? [];
+		fixes.push(problem.fix);
+		files.set(problem.file, fixes);
+	}
+	for (const [file, fixes] of files) {
+		const target = path.join(cwd, file);
+		let source = readFileSync(target, "utf8");
+		for (const fix of [...fixes].sort((a, b) => b.start - a.start)) source = source.slice(0, fix.start) + fix.text + source.slice(fix.end);
+		writeFileSync(target, source);
+	}
+}
+
+//#endregion
+export { applyConventionFixes, importFixes };
+//# sourceMappingURL=fixes.mjs.map

package/esm/tooling/check/fixes.mjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"fixes.mjs","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/fixes.ts"],"sourcesContent":["import { readFileSync, writeFileSync } from \"node:fs\";\nimport path from \"node:path\";\nimport type { ToolingProblem } from \"../problem\";\nimport { importSpecifiers } from \"../syntax/imports\";\n\nexport type ProblemFix = { start: number; end: number; text: string };\n\n/** Only relative .js imports: removing the suffix preserves TypeScript resolution. */\nexport function importFixes(file: string, source: string): ProblemFix[] {\n  return importSpecifiers(file, source)\n    .filter(\n      ({ specifier, node }) =>\n        specifier.startsWith(\".\") &&\n        specifier.endsWith(\".js\") &&\n        /\\.js['\"]$/.test(source.slice(node.start, node.end)),\n    )\n    .map(({ node }) => ({ start: node.end - 4, end: node.end - 1, text: \"\" }));\n}\n\nexport function applyConventionFixes(cwd: string, problems: readonly ToolingProblem[]): void {\n  const files = new Map<string, ProblemFix[]>();\n  for (const problem of problems)\n    if (problem.file && problem.fix) {\n      const fixes = files.get(problem.file) ?? [];\n      fixes.push(problem.fix);\n      files.set(problem.file, fixes);\n    }\n  for (const [file, fixes] of files) {\n    const target = path.join(cwd, file);\n    let source = readFileSync(target, \"utf8\");\n    for (const fix of [...fixes].sort((a, b) => b.start - a.start))\n      source = source.slice(0, fix.start) + fix.text + source.slice(fix.end);\n    writeFileSync(target, source);\n  }\n}\n"],"mappings":";;;;;;AAQA,SAAgB,YAAY,MAAc,QAA8B;CACtE,OAAO,iBAAiB,MAAM,MAAM,CAAC,CAClC,QACE,EAAE,WAAW,WACZ,UAAU,WAAW,GAAG,KACxB,UAAU,SAAS,KAAK,KACxB,YAAY,KAAK,OAAO,MAAM,KAAK,OAAO,KAAK,GAAG,CAAC,CACvD,CAAC,CACA,KAAK,EAAE,YAAY;EAAE,OAAO,KAAK,MAAM;EAAG,KAAK,KAAK,MAAM;EAAG,MAAM;CAAG,EAAE;AAC7E;AAEA,SAAgB,qBAAqB,KAAa,UAA2C;CAC3F,MAAM,wBAAQ,IAAI,IAA0B;CAC5C,KAAK,MAAM,WAAW,UACpB,IAAI,QAAQ,QAAQ,QAAQ,KAAK;EAC/B,MAAM,QAAQ,MAAM,IAAI,QAAQ,IAAI,KAAK,CAAC;EAC1C,MAAM,KAAK,QAAQ,GAAG;EACtB,MAAM,IAAI,QAAQ,MAAM,KAAK;CAC/B;CACF,KAAK,MAAM,CAAC,MAAM,UAAU,OAAO;EACjC,MAAM,SAAS,KAAK,KAAK,KAAK,IAAI;EAClC,IAAI,SAAS,aAAa,QAAQ,MAAM;EACxC,KAAK,MAAM,OAAO,CAAC,GAAG,KAAK,CAAC,CAAC,MAAM,GAAG,MAAM,EAAE,QAAQ,EAAE,KAAK,GAC3D,SAAS,OAAO,MAAM,GAAG,IAAI,KAAK,IAAI,IAAI,OAAO,OAAO,MAAM,IAAI,GAAG;EACvE,cAAc,QAAQ,MAAM;CAC9B;AACF"}

package/esm/tooling/check/index.d.mts

@@ -0,0 +1,12 @@
+import { ProblemFix } from "./fixes.mjs";
+import { GitResult, GitRunner, listChangedFiles, runGit } from "./changed-files.mjs";
+import { CHECK_EXIT_CODES, CHECK_STEPS, CheckExitCode, CheckReport, CheckStep, CheckStepName, CheckStepStatus, checkExitCode } from "./check-report.types.mjs";
+import { ConventionOptions } from "./web-conventions.mjs";
+import { checkFileConventions, checkTsconfigConventions, isAutoLoadedFile, runConventions } from "./conventions-step.mjs";
+import { MIN_NODE_VERSION, checkNodeVersion } from "./environment-step.mjs";
+import { LintOutcome, runLint } from "./lint-step.mjs";
+import { VitestRun, VitestRunner, findVitest, parseVitestReport, runVitest } from "./tests-step.mjs";
+import { CheckOptions, TypecheckRunner, runCheck } from "./run-check.mjs";
+import { RunTestsOptions, TestRunReport, runTests, testRunArguments } from "./run-tests.mjs";
+import { affectedFiles } from "./affected-files.mjs";
+export { CHECK_EXIT_CODES, CHECK_STEPS, type CheckExitCode, type CheckOptions, type CheckReport, type CheckStep, type CheckStepName, type CheckStepStatus, type ConventionOptions, type GitResult, type GitRunner, type LintOutcome, MIN_NODE_VERSION, type ProblemFix, type RunTestsOptions, type TestRunReport, type TypecheckRunner, type VitestRun, type VitestRunner, affectedFiles, checkExitCode, checkFileConventions, checkNodeVersion, checkTsconfigConventions, findVitest, isAutoLoadedFile, listChangedFiles, parseVitestReport, runCheck, runConventions, runGit, runLint, runTests, runVitest, testRunArguments };

package/esm/tooling/check/index.mjs

@@ -1,10 +1,11 @@
 import { CHECK_EXIT_CODES, CHECK_STEPS, checkExitCode } from "./check-report.types.mjs";
-import { listChangedFiles, runGit } from "./changed-files.mjs";
 import { checkFileConventions, checkTsconfigConventions, isAutoLoadedFile, runConventions } from "./conventions-step.mjs";
+import { affectedFiles } from "./affected-files.mjs";
+import { listChangedFiles, runGit } from "./changed-files.mjs";
 import { MIN_NODE_VERSION, checkNodeVersion } from "./environment-step.mjs";
 import { runLint } from "./lint-step.mjs";
 import { findVitest, parseVitestReport, runVitest } from "./tests-step.mjs";
 import { runCheck } from "./run-check.mjs";
 import { runTests, testRunArguments } from "./run-tests.mjs";
 
-export { CHECK_EXIT_CODES, CHECK_STEPS, MIN_NODE_VERSION, checkExitCode, checkFileConventions, checkNodeVersion, checkTsconfigConventions, findVitest, isAutoLoadedFile, listChangedFiles, parseVitestReport, runCheck, runConventions, runGit, runLint, runTests, runVitest, testRunArguments };
+export { CHECK_EXIT_CODES, CHECK_STEPS, MIN_NODE_VERSION, affectedFiles, checkExitCode, checkFileConventions, checkNodeVersion, checkTsconfigConventions, findVitest, isAutoLoadedFile, listChangedFiles, parseVitestReport, runCheck, runConventions, runGit, runLint, runTests, runVitest, testRunArguments };

package/esm/tooling/check/run-check.d.mts

@@ -13,6 +13,8 @@ export type CheckOptions = {
   lint?: "oxlint";
   /** Limit every step to the files changed against `base` (default HEAD) plus untracked files. */
   changed?: boolean;
+  /** Apply safe convention fixes before checking. */
+  fix?: boolean;
   /** Compare with `git merge-base <base> HEAD` instead of HEAD. Implies `changed`. */
   base?: string;
   /** Steps not to run. */

package/esm/tooling/check/run-check.d.mts.map

@@ -1 +1 @@
-{"version":3,"file":"run-check.d.mts","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/run-check.ts"],"mappings":";;;;;;YA8BY,mBACV,SAAS,qBACN,QAAQ,mBAAmB;;YAGpB;;EAEV;;EAEA;;EAEA;;EAEA;;EAEA,gBAAgB;;EAEhB;;EAEA,MAAM;;EAEN,YAAY;;EAEZ,SAAS;;EAET;;EAEA;;;;;;;;;;;;wBA4BoB,SAAS,UAAS,eAAoB,QAAQ"}
+{"version":3,"file":"run-check.d.mts","names":[],"sources":["../../../../../../../../../packages/core/src/tooling/check/run-check.ts"],"mappings":";;;;;;YAiCY,mBACV,SAAS,qBACN,QAAQ,mBAAmB;;YAGpB;;EAEV;;EAEA;;EAEA;;EAEA;;EAEA;;EAEA,gBAAgB;;EAEhB;;EAEA,MAAM;;EAEN,YAAY;;EAEZ,SAAS;;EAET;;EAEA;;;;;;;;;;;;wBA4BoB,SAAS,UAAS,eAAoB,QAAQ"}

package/esm/tooling/check/run-check.mjs

@@ -1,11 +1,14 @@
 import { CHECK_STEPS, checkExitCode } from "./check-report.types.mjs";
 import { createProblem } from "../problem.mjs";
+import { applyConventionFixes } from "./fixes.mjs";
+import { collectApiRouteFacts } from "./api-route-references.mjs";
+import { runConventions } from "./conventions-step.mjs";
+import { listSourceFiles, listTestFiles } from "./source-files.mjs";
+import { affectedFiles } from "./affected-files.mjs";
 import { runTypecheck } from "../typecheck/run-typecheck.mjs";
 import { listChangedFiles } from "./changed-files.mjs";
-import { runConventions } from "./conventions-step.mjs";
 import { checkNodeVersion } from "./environment-step.mjs";
 import { runLint } from "./lint-step.mjs";
-import { listSourceFiles, listTestFiles } from "./source-files.mjs";
 import { findVitest, parseVitestReport, runVitest, vitestArguments, vitestRunFailure } from "./tests-step.mjs";
 import path from "node:path";
 
@@ -85,12 +88,17 @@ async function runCheck(options = {}) {
 			const listed = await listChangedFiles({
 				cwd,
 				base: options.base,
-				git: options.git
+				git: options.git,
+				includeDeleted: true
 			});
 			if ("problem" in listed) {
 				filesBlocked = true;
 				found.push(listed.problem);
-			} else files = listed.files;
+			} else {
+				filesBlocked = true;
+				files = await affectedFiles(cwd, listed.files, options.project);
+				filesBlocked = false;
+			}
 		}
 		return {
 			detail: `Node ${options.nodeVersion ?? process.versions.node}${files ? `, ${files.length} changed file(s)` : ""}`,
@@ -98,6 +106,16 @@ async function runCheck(options = {}) {
 		};
 	});
 	const targets = async () => files ?? await listSourceFiles(cwd);
+	if (options.fix && !filesBlocked && !skip.has("conventions")) try {
+		const fixTargets = await targets();
+		applyConventionFixes(cwd, runConventions(cwd, fixTargets, options.project, collectApiRouteFacts(cwd, await listSourceFiles(cwd))));
+	} catch (error) {
+		problems.push(createProblem({
+			code: "WL8007",
+			source: "environment",
+			message: `Could not apply fixes: ${errorMessage(error)}`
+		}));
+	}
 	await run("typecheck", async () => {
 		if (files !== null && files.length === 0) return {
 			status: "skipped",
@@ -166,7 +184,7 @@ async function runCheck(options = {}) {
 		const conventionFiles = await targets();
 		return {
 			detail: `${conventionFiles.length} file(s) checked`,
-			problems: runConventions(cwd, conventionFiles, options.project)
+			problems: runConventions(cwd, conventionFiles, options.project, collectApiRouteFacts(cwd, await listSourceFiles(cwd)))
 		};
 	});
 	steps.sort((first, second) => CHECK_STEPS.indexOf(first.name) - CHECK_STEPS.indexOf(second.name));