@warlock.js/core 5.33.0 → 5.34.0

package/skills/test-browser/SKILL.md

@@ -0,0 +1,119 @@
+---
+name: test-browser
+description: Write or repair browser e2e specs using warlock e2e and @warlock.js/core/e2e, with owned app fixtures, direct CDP interaction, responsive screenshots and run evidence. Use for real browser flows, click races or viewport overflow; use test-http for API-only tests.
+---
+
+# Browser e2e
+
+Use `@warlock.js/core/e2e`, a Node-only subpath, and default-export `defineE2E` from
+`e2e/*.e2e.ts`. Run `npx warlock e2e`. The CLI loads TypeScript itself; no Vitest
+bootstrap, Playwright, Puppeteer or external browser server is needed.
+
+Every run copies the app into a temporary directory, excludes dotenv files and
+symlinks, links installed dependencies for reads, seeds its own environment and
+owns the app/browser process groups. Keep foreground commands in the fixture;
+never use a shared dev server or daemonize. Map the disposable `dataRoot` into the
+app's actual storage/database config. The harness cannot infer that mapping.
+`prepare` is where seeds and fake executables belong; `app.path` is relative to the
+copied app. Do not install dependencies through the shared `node_modules` link.
+
+## Complete standalone spec
+
+This spec creates its own small app; it can be pasted into an existing project
+with core installed and run with `npx warlock e2e`.
+
+```ts
+import { mkdir, writeFile } from "node:fs/promises";
+import { join } from "node:path";
+import { defineE2E } from "@warlock.js/core/e2e";
+
+export default defineE2E({
+  app: {
+    start: [process.execPath, "form-fixture.mjs"],
+    portFromFile: true,
+    path: ["fake-bin"],
+    env: ({ dataRoot }) => ({ FORM_DATA: dataRoot }),
+    prepare: async ({ directory, dataRoot }) => {
+      await mkdir(join(directory, "fake-bin"));
+      await writeFile(join(dataRoot, "prefix.txt"), "Saved");
+      await writeFile(
+        join(directory, "form-fixture.mjs"),
+        `
+        import { createServer } from 'node:http';
+        import { readFileSync, writeFileSync } from 'node:fs';
+        import { join } from 'node:path';
+        const prefix = readFileSync(join(process.env.FORM_DATA, 'prefix.txt'), 'utf8');
+        const server = createServer((request, response) => {
+          if (request.url === '/favicon.ico') { response.writeHead(204); response.end(); return; }
+          response.setHeader('Content-Type', 'text/html');
+          response.end('<!doctype html><meta name="viewport" content="width=device-width,initial-scale=1">'
+            + '<input id="name"><button id="save"><span>Save</span></button><h1 id="result">Ready</h1>'
+            + '<script>document.querySelector("#save").onclick=()=>{history.pushState({},"","/saved");document.querySelector("#result").textContent='
+            + JSON.stringify(prefix) + '+" "+document.querySelector("#name").value}</script>');
+        });
+        server.listen(0, '127.0.0.1', () => writeFileSync(process.env.WARLOCK_E2E_PORT_FILE, String(server.address().port)));
+      `,
+      );
+    },
+  },
+  run: async ({ page, step }) => {
+    await page.goto("/");
+    await step("save the form", async () => {
+      await page.fill("#name", "Ada");
+      await page.type("#name", " Lovelace");
+      await page.waitForNavigation(() => page.click("#save"), { url: "/saved" });
+      await page.waitFor({ selector: "#result", text: "Saved Ada Lovelace" });
+    });
+    await page.captureViewports("saved");
+  },
+});
+```
+
+For an existing app, replace `prepare` and `start`; default start is
+`[process.execPath, "dist/app.js"]`. Optional
+`build: [process.execPath, "node_modules/@warlock.js/core/bin/warlock.js", "build"]`
+builds inside the copy. Supply app-specific test service variables via `env` and
+exclude production data directories with `exclude: ["storage", "data"]`.
+By default the fixture supplies an OS-assigned `PORT`/`HTTP_PORT`. Prefer the
+port-file handshake above when the app supports binding `0` itself.
+
+## Interaction and evidence
+
+- Use `page.click` for scrolling, settled boxes and hit testing, including a fresh
+  check after hover. Do not replace it with `element.click()` or fixed sleeps.
+- `fill` clears editable content, then sends CDP text input; `type` appends Unicode
+  text. Values do not appear in step names. `type` does not simulate shortcuts.
+- `waitFor({ selector, text, url, timeoutMs })` combines conditions. Arm navigation
+  **before** clicking with `waitForNavigation(() => page.click(...))`.
+- `setViewport({ name, width, height, mobile })`, `assertViewport()` and
+  `screenshot(name)` are available independently. `captureViewports(name)` checks
+  and captures 1280×800 desktop and 390×844 phone by default, restoring the
+  original viewport. `viewports` overrides the list. Final captures are automatic.
+- Overflow compares document/body scrollWidth **and innerWidth** against the
+  requested width, so a phone that expands to 448 px fails. Keep the viewport meta
+  tag; inspect the reported widest element IDs, classes and dimensions.
+- Health requires zero console errors, exceptions, HTTP statuses >=400 and failed
+  loads (including cancellation). Narrow exceptions use
+  `allowConsoleErrors: [/^Expected warning$/]` and
+  `allowNetworkFailures: [{ url: "/missing", status: 404 }]`. Canceled-request rules
+  can match `errorText`. Strings match substrings; regexps are also accepted.
+- `allowPendingRequests: ["/events"]` exempts an intentional stream from network
+  idle, without allowing its errors. `waitForNetworkIdle()` remains available.
+- `page.evaluate<T>(expression)` is an escape hatch for assertions in the main
+  document. Selector helpers do not enter frames or closed shadow roots.
+
+Use `--browser /path/to/chrome` or `WARLOCK_E2E_BROWSER` if discovery fails.
+`browser: { args: ["--no-sandbox"] }` is an explicit option for controlled root
+containers; it is not a default. On Windows, command arrays should invoke JS tools
+through `process.execPath`, avoiding `.cmd` executables that require a shell.
+
+Read `.warlock/e2e/run-*/evidence.json` and the PNGs/logs on failure. Evidence keeps
+allowed errors, step timings, viewport dimensions and recorded process identities.
+`--out` changes the evidence parent; `--path ./fake-bin,./engine-bin` prepends fake
+binary paths. `runE2E(spec, { outputDirectory, browser, path, signal })` is the
+programmatic runner; `E2ERunError.result` exposes evidence on failure.
+
+Cleanup verifies the unique supervisor command line before stopping only its
+recorded process group (Windows: verified PID tree). It retains disposable files
+if ownership verification fails and reports the failure in evidence. Do not add
+process-name or port-based kills to a spec.

package/skills/upgrade-from-5-29/SKILL.md

@@ -441,51 +441,92 @@ The CLI preserves incremental state and changed-file filtering. Development
 health checks keep one native compiler session and update its snapshots. Syntax
 analysis uses OXC. `warlock upgrade` still parses/transforms with ts-morph's own
 bundled TypeScript, without using it as the app's compiler. New scaffolds default
-to Oxlint/Oxfmt (`--lint=oxc`). Explicit `--lint=eslint` pins the app's compiler to
-TypeScript **6.0.3** because typescript-eslint requires the old JavaScript API
-and supports `>=4.8.4 <6.1.0`. Keep ESLint apps on 6.0.3 until switching tools:
-
-```bash
-npm install --save-dev --save-exact typescript@6.0.3
-npx tsc --noEmit
-```
-
-The same tsconfig cleanups work with TS6. Starting in 5.33, ESLint mode prints
-a deprecation warning naming removal in 5.34.0 and the Oxc migration command.
+to Oxlint/Oxfmt (`--lint=oxc`) with TypeScript 7.0.2. ESLint support was
+removed in 5.34.0; migrate legacy apps using the steps below.
 
 ## 5.32 to 5.33
 
-**BREAKING:** ESLint background health checking and automatic ESLint selection
-in `warlock check` are disabled by default. Oxlint/Oxfmt replace ESLint/Prettier
-as the default toolchain. Legacy support is deprecated now and removed in
-5.34.0. Temporary opt-ins are `devServer.healthCheckers: { eslint: true }`,
-`warlock check --lint eslint`, and `create-warlock --lint=eslint`; each warns
-about removal and migration. The legacy scaffold stays on TypeScript 6.0.3;
-Oxc apps use TypeScript 7.0.2.
-
-ESLint + Prettier are deprecated in 5.33.0 and will be removed in **5.34.0**.
-New apps should use `--lint=oxc`. To migrate an existing app, install the new
-core release and `ts-morph`, then run:
+The original `oxc-toolchain` codemod remains available:
 
 ```bash
 npx warlock upgrade --from 5.32 --to 5.33.0 --dry-run
 npx warlock upgrade --from 5.32 --to 5.33.0
+```
+
+## 5.33 to 5.34 toolchain removal
+
+**BREAKING:** ESLint and Prettier support was removed in **5.34.0**. Oxlint/Oxfmt is the only
+supported toolchain. `--lint eslint` and `healthCheckers.eslint` are removed.
+Install the new core release and `ts-morph`, then migrate an existing app:
+
+```bash
+npm install --save-dev ts-morph
+npx warlock upgrade --from 5.33 --to 5.34.0 --dry-run
+npx warlock upgrade --from 5.33 --to 5.34.0
 npm install
 npm run format
 npm run lint:check
 npx warlock check
 ```
 
-Review every reported `file:line`: customized configs are preserved and need
-manual translation before removal. The codemod installs the template's Oxlint
-and Oxfmt versions and pins TypeScript to 7.0.2. It replaces standard lint/format
-scripts, removes legacy dependencies, and deletes only recognized generated
-configs. A generated `.prettierignore` becomes `.oxfmtignore`. Existing Oxc
-configs are preserved. `--dry-run --json` reports the same planned changes
-without writing files.
+The `oxc-toolchain-removal` step reapplies the preserved 5.33 `oxc-toolchain`
+migration when legacy configs, scripts or dependencies remain. Apps upgrading
+from 5.32 can still use `npx warlock upgrade --from 5.32 --to 5.33.0` first.
+It replaces standard scripts/dependencies, pins TypeScript 7.0.2, updates generated
+editor settings, removes `healthCheckers.eslint` and selects Oxlint. A generated
+`.prettierignore` becomes `.oxfmtignore`; Oxfmt scripts read it explicitly.
 
 The `oxc-toolchain` codemod also updates generated VS Code settings and literal
 ESLint/Oxlint health-check switches. Custom editor settings, checker arrays,
 computed options and nonstandard scripts require review. Oxc diagnostics keep
 framework conventions in `warlock check`. Re-run the migration after addressing
 manual findings if needed; an app with only Oxc tooling is a no-op for this codemod.
+
+## 5.33 to 5.34 plain object validation
+
+**BREAKING:** Seal `v.object()` rejects arrays, dates, maps, sets, class instances
+and objects with custom prototypes, including under `.allowUnknown()` and
+`.stripUnknown()`. Plain and null-prototype dictionaries remain accepted. There
+is no permissive opt-in: choose an array/instance validator or construct a plain
+DTO explicitly before validation.
+
+```ts
+import { v } from "@warlock.js/seal";
+
+class User {
+  name = "Ada";
+}
+const instance = new User();
+const user = v.object({ name: v.string() });
+await v.validate(user, { name: instance.name });
+const names = v.array(v.string());
+await v.validate(names, ["Ada"]);
+```
+
+`v.record(valueValidator)` remains supported. To validate dictionary keys, use
+`v.record(keyValidator, valueValidator)`; malformed arguments now fail while
+defining the schema. Literal-key records infer partial dictionaries because
+only present keys are checked.
+Literal custom rules are translated without executing the config. Per-folder
+`no-restricted-imports` and `no-restricted-exports` become Oxlint overrides with
+file/exclusion patterns. Supported `@typescript-eslint/*` rules become native
+`typescript/*` rules (or native base rules). Options are checked against the
+pinned Oxlint 1.88.0 schema. Type-aware equivalents enable `options.typeAware`
+and add `oxlint-tsgolint@7.0.2004` to the app's dev dependencies.
+`unused-imports/no-unused-imports` uses `no-unused-vars` with safe import fixes;
+Oxlint shares the severity for imports and variables, so differing active
+severities are reported for review.
+
+Only unsupported rules/options, computed or imported settings, selectors such
+as `no-restricted-syntax`, and conflicting existing Oxc rules need manual review.
+Each finding names its source `file:line`, rule and affected file patterns.
+Fully ported lint configs are removed; configs with findings remain for reference
+until those findings are addressed. Existing Oxc rules are kept on conflict.
+`--dry-run --json` reports the same plan without writes. Customized formatting,
+editor settings and nonstandard scripts can also require review.
+
+**Upgrading:** new and migrated template configs enable
+`"categories": { "correctness": "error" }`. Address the additional diagnostics;
+existing Oxc configs retain their categories. A real lockfile beside a workspace
+marker determines the installer: `package-lock.json` plus `pnpm-workspace.yaml`
+without `pnpm-lock.yaml` now produces `npm install`.

package/skills/warlock-check/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: warlock-check
-description: 'Verify a change with one command: `warlock check [--changed] [--json]` runs an incremental typecheck, the app''s ESLint/oxlint, the tests affected by the change and framework convention checks, and reports one JSON object per problem `{ code, category, severity, source, file, line, column, message, hint, skill, docs, rule }` with exit 1 for problems in your code and 2 for problems with this machine (missing Redis/DB, busy port, Node version, missing tool). Also `warlock typecheck [--changed]`, `warlock explain <CODE>` and the stable `WL` error codes (`errorCodeOf`, `WarlockError`). Triggers: `warlock check`, `warlock typecheck`, `warlock explain`, `--changed`, `--base`, `runCheck`, `runTypecheck`, `CHECK_EXIT_CODES`, `ToolingProblem`, `WL7101`, `WL8003`, `errorCodeOf`, `explainErrorCode`, `WarlockError`, "verify my change", "run the checks", "is this failure my code or the machine", "what does WL3001 mean", "typecheck only changed files". Skip: app diagnostics at boot — the `warlock-doctor` topic; reading the app''s routes/models — the `warlock-inspect` topic; writing tests — the `test-http` / `test-service` topics.'
+description: 'Verify a change with one command: `warlock check [--changed] [--json]` runs an incremental typecheck, the app''s Oxlint, the tests affected by the change and framework convention checks, and reports one JSON object per problem `{ code, category, severity, source, file, line, column, message, hint, skill, docs, rule }` with exit 1 for problems in your code and 2 for problems with this machine (missing Redis/DB, busy port, Node version, missing tool). Also `warlock typecheck [--changed]`, `warlock explain <CODE>` and the stable `WL` error codes (`errorCodeOf`, `WarlockError`). Triggers: `warlock check`, `warlock typecheck`, `warlock explain`, `--changed`, `--base`, `runCheck`, `runTypecheck`, `CHECK_EXIT_CODES`, `ToolingProblem`, `WL7101`, `WL8003`, `errorCodeOf`, `explainErrorCode`, `WarlockError`, "verify my change", "run the checks", "is this failure my code or the machine", "what does WL3001 mean", "typecheck only changed files". Skip: app diagnostics at boot — the `warlock-doctor` topic; reading the app''s routes/models — the `warlock-inspect` topic; writing tests — the `test-http` / `test-service` topics.'
 ---
 
 # Warlock — `warlock check`, `typecheck`, `explain`
@@ -120,10 +120,10 @@ const types = runTypecheck({ files: ["src/app/posts/services/publish.service.ts"
 ## Gotchas
 
 - `check` boots nothing — no config, no connection, no port — so it runs the same with or without services; only tests that need them fail, and those report `WL8003`/`WL8004`.
-- Lint defaults to Oxlint (an app-root config + the app-installed package). No Oxlint config skips lint. A missing package reports `WL8002`. ESLint requires `--lint eslint` (or `runCheck({ lint: "eslint" })`); its flat config alone never enables it. This deprecated selection prints one warning on stderr and is removed in 5.34.0.
+- Lint defaults to Oxlint (an app-root config + the app-installed package). No Oxlint config skips lint. A missing package reports `WL8002`. ESLint support was removed in 5.34.0; see `upgrade-from-5-29`.
 - Tests use the app's own vitest (`vitest run`, or `vitest related <files> --run` with `--changed`). Test files without vitest installed → `WL8002`.
 - The typecheck runs in a child `warlock typecheck` process; running out of memory is reported as `WL8007` instead of ending the run.
-- The convention checks are the framework's own and run the same with ESLint, Oxlint or no linter. A deliberate exception gets `// warlock-check-ignore WL9006 -- why` on the problem's line or the line above (no code = every code on that line).
+- The convention checks are the framework's own and run the same with Oxlint or no linter. A deliberate exception gets `// warlock-check-ignore WL9006 -- why` on the problem's line or the line above (no code = every code on that line).
 
 ## See also
 

package/skills/warlock-doctor/SKILL.md

@@ -174,7 +174,6 @@ A `⚠ routes: 0 routes registered` line is the tell that a route module threw o
 
 `warlock doctor` checks application wiring and HTTP health probes. Use
 `npx warlock check` for source lint diagnostics and `npm run format:check` for
-formatting. In 5.33, Oxlint/Oxfmt are the defaults; ESLint requires
-`warlock check --lint eslint` or `devServer.healthCheckers.eslint: true` for
-background checks. ESLint/Prettier are deprecated and removed in 5.34.0.
+formatting. Oxlint/Oxfmt are the only supported tools in 5.34.0. Use the
+`oxc-toolchain-removal` upgrade step to migrate remaining legacy configs.
 Migrate with `npx warlock upgrade --from 5.32 --to 5.33.0` (preview with `--dry-run`).

package/esm/dev-server/health-checker/checkers/eslint-health-checker.mjs

@@ -1,152 +0,0 @@
-import { hasEslint, warnEslintDeprecation } from "../eslint-support.mjs";
-import { BaseHealthChecker } from "./base-health-checker.mjs";
-import { colors } from "@mongez/copper";
-import { ESLint } from "eslint";
-
-//#region ../../packages/core/src/dev-server/health-checker/checkers/eslint-health-checker.ts
-/** @deprecated Use OxlintHealthChecker. ESLint support is removed in 5.34.0. */
-var EslintHealthChecker = class extends BaseHealthChecker {
-	constructor() {
-		super();
-		this.eslint = null;
-		this.name = "ESLint";
-		this.workerPath = "./workers/eslint-health.worker";
-		this.initialized = false;
-		warnEslintDeprecation();
-	}
-	/**
-	* Check if file is a lintable file
-	*/
-	isLintableFile(filePath) {
-		const ext = filePath.toLowerCase();
-		return ext.endsWith(".ts") || ext.endsWith(".tsx") || ext.endsWith(".js") || ext.endsWith(".jsx");
-	}
-	/**
-	* Initialize the health checker
-	*/
-	initialize() {
-		try {
-			if (!hasEslint(process.cwd())) {
-				this.initialized = true;
-				return this;
-			}
-			this.eslint = new ESLint({ cwd: process.cwd() });
-			this.initialized = true;
-		} catch (error) {
-			console.warn("ESLint Health Checker: Failed to initialize:", error);
-			this.initialized = true;
-		}
-		return this;
-	}
-	/**
-	* Display health check results in a pretty format
-	*/
-	displayResults(file, result) {
-		const stats = result.getStats();
-		if (stats.errors === 0 && stats.warnings === 0) return;
-		const fileName = file.relativePath.replace(/\\/g, "/");
-		const errorCount = stats.errors;
-		const warningCount = stats.warnings;
-		const sourceLines = file.source ? file.source.split("\n") : [];
-		if (errorCount > 0) {
-			const errorMessages = result.messages.filter((m) => m.type === "error");
-			for (const error of errorMessages) {
-				const icon = colors.redBright("✖");
-				const level = colors.redBright(colors.bold("ERROR"));
-				console.log(`\n${icon} ${level} ${colors.dim("in")} ${colors.cyanBright(fileName)}${colors.dim(`(${error.lineNumber},${error.columnNumber})`)}`);
-				const ruleId = error.ruleId || "eslint";
-				console.log(`  ${colors.magentaBright(ruleId)} ${colors.dim("→")} ${colors.red(error.message)}`);
-				if (sourceLines.length > 0 && error.lineNumber > 0 && error.lineNumber <= sourceLines.length) {
-					const lineContent = sourceLines[error.lineNumber - 1];
-					const lineNum = error.lineNumber.toString().padStart(4, " ");
-					const errorLength = error.length || 1;
-					const columnIndex = error.columnNumber - 1;
-					console.log(`  ${colors.dim(lineNum)} ${colors.dim("│")} ${lineContent || ""}`);
-					const prefixPadding = lineNum.length + 3;
-					const columnPadding = " ".repeat(columnIndex);
-					const underline = colors.redBright("~".repeat(Math.max(1, errorLength)));
-					console.log(`  ${colors.dim(" ".repeat(prefixPadding))}${columnPadding}${underline}`);
-				}
-			}
-		}
-		if (warningCount > 0) {
-			const warningMessages = result.messages.filter((m) => m.type === "warning");
-			for (const warning of warningMessages) {
-				const icon = colors.yellowBright("⚠");
-				const level = colors.yellowBright(colors.bold("WARNING"));
-				console.log(`\n${icon} ${level} ${colors.dim("in")} ${colors.cyanBright(fileName)}${colors.dim(`(${warning.lineNumber},${warning.columnNumber})`)}`);
-				const ruleId = warning.ruleId || "eslint";
-				console.log(`  ${colors.magentaBright(ruleId)} ${colors.dim("→")} ${colors.yellow(warning.message)}`);
-				if (sourceLines.length > 0 && warning.lineNumber > 0 && warning.lineNumber <= sourceLines.length) {
-					const lineContent = sourceLines[warning.lineNumber - 1];
-					const lineNum = warning.lineNumber.toString().padStart(4, " ");
-					const warningLength = warning.length || 1;
-					const columnIndex = warning.columnNumber - 1;
-					console.log(`  ${colors.dim(lineNum)} ${colors.dim("│")} ${lineContent || ""}`);
-					const prefixPadding = lineNum.length + 3;
-					const columnPadding = " ".repeat(columnIndex);
-					const underline = colors.yellowBright("~".repeat(Math.max(1, warningLength)));
-					console.log(`  ${colors.dim(" ".repeat(prefixPadding))}${columnPadding}${underline}`);
-				}
-			}
-		}
-		const summary = [];
-		if (errorCount > 0) summary.push(colors.red(`${errorCount} error${errorCount > 1 ? "s" : ""}`));
-		if (warningCount > 0) summary.push(colors.yellow(`${warningCount} warning${warningCount > 1 ? "s" : ""}`));
-	}
-	/**
-	* Validate the health of the file
-	*/
-	async validate(file, result) {
-		if (!this.isLintableFile(file.absolutePath)) {
-			result.markAsHealthy();
-			return result;
-		}
-		if (!this.eslint) {
-			result.markAsHealthy();
-			return result;
-		}
-		try {
-			const lintResult = (await this.eslint.lintText(file.source, { filePath: file.absolutePath }))[0];
-			if (!lintResult) {
-				console.log("No lint results", file.relativePath);
-				result.markAsHealthy();
-				return result;
-			}
-			const errors = [];
-			const warnings = [];
-			for (const message of lintResult.messages) {
-				const isError = message.severity === 2;
-				const isWarning = message.severity === 1;
-				if (isError) errors.push({
-					message: message.message,
-					type: "error",
-					lineNumber: message.line || 1,
-					columnNumber: message.column || 1,
-					length: message.endColumn && message.column ? message.endColumn - message.column : 1,
-					ruleId: message.ruleId || void 0
-				});
-				else if (isWarning) warnings.push({
-					message: message.message,
-					type: "warning",
-					lineNumber: message.line || 1,
-					columnNumber: message.column || 1,
-					length: message.endColumn && message.column ? message.endColumn - message.column : 1,
-					ruleId: message.ruleId || void 0
-				});
-			}
-			if (errors.length > 0) result.addErrors(errors);
-			if (warnings.length > 0) result.addWarnings(warnings);
-			if (errors.length === 0 && warnings.length === 0) result.markAsHealthy();
-			else this.displayResults(file, result);
-		} catch (error) {
-			console.log("ESlint Error:", error);
-			result.markAsHealthy();
-		}
-		return result;
-	}
-};
-
-//#endregion
-export { EslintHealthChecker };
-//# sourceMappingURL=eslint-health-checker.mjs.map

package/esm/dev-server/health-checker/checkers/eslint-health-checker.mjs.map

@@ -1 +0,0 @@
-{"version":3,"file":"eslint-health-checker.mjs","names":[],"sources":["../../../../../../../../../../packages/core/src/dev-server/health-checker/checkers/eslint-health-checker.ts"],"sourcesContent":["import { colors } from \"@mongez/copper\";\nimport { ESLint } from \"eslint\";\nimport { hasEslint, warnEslintDeprecation } from \"../eslint-support\";\nimport type { FileManager } from \"../../file-manager\";\nimport type { FileHealthCheckerContract } from \"../file-health-checker.contract\";\nimport { FileHealthResult } from \"../file-health-result\";\nimport { BaseHealthChecker } from \"./base-health-checker\";\n\n/** @deprecated Use OxlintHealthChecker. ESLint support is removed in 5.34.0. */\nexport class EslintHealthChecker extends BaseHealthChecker implements FileHealthCheckerContract {\n  public constructor() {\n    super();\n    // Construction happens on the main thread for both inline and worker checking.\n    warnEslintDeprecation();\n  }\n\n  /**\n   * ESLint instance (using new ESLint v9 API with flat config)\n   */\n  private eslint: ESLint | null = null;\n\n  /**\n   * Health checker name\n   */\n  public name: string = \"ESLint\";\n\n  /**\n   * Path to dedicated worker file for ESLint checking\n   * Runs in a separate thread to avoid blocking the main dev server\n   */\n  public workerPath: string = \"./workers/eslint-health.worker\";\n\n  /**\n   * Whether checker is initialized\n   */\n  private initialized: boolean = false;\n\n  /**\n   * Check if file is a lintable file\n   */\n  private isLintableFile(filePath: string): boolean {\n    const ext = filePath.toLowerCase();\n    return (\n      ext.endsWith(\".ts\") || ext.endsWith(\".tsx\") || ext.endsWith(\".js\") || ext.endsWith(\".jsx\")\n    );\n  }\n\n  /**\n   * Initialize the health checker\n   */\n  public initialize(): EslintHealthChecker {\n    try {\n      const hasFlatConfig = hasEslint(process.cwd());\n\n      if (!hasFlatConfig) {\n        // No flat config found, skip initialization\n        this.initialized = true;\n        return this;\n      }\n\n      // Use new ESLint v9 API with flat config\n      // ESLint will automatically find eslint.config.js in the project root\n      this.eslint = new ESLint({\n        cwd: process.cwd(),\n      });\n\n      this.initialized = true;\n    } catch (error) {\n      // Handle any errors during initialization gracefully\n      console.warn(\"ESLint Health Checker: Failed to initialize:\", error);\n      this.initialized = true; // Mark as initialized to prevent retries\n    }\n\n    return this;\n  }\n\n  /**\n   * Display health check results in a pretty format\n   */\n  private displayResults(file: FileManager, result: FileHealthResult): void {\n    const stats = result.getStats();\n\n    // Only display if there are errors or warnings\n    if (stats.errors === 0 && stats.warnings === 0) {\n      return;\n    }\n\n    const fileName = file.relativePath.replace(/\\\\/g, \"/\");\n    const errorCount = stats.errors;\n    const warningCount = stats.warnings;\n    const sourceLines = file.source ? file.source.split(\"\\n\") : [];\n\n    // Display errors\n    if (errorCount > 0) {\n      const errorMessages = result.messages.filter((m) => m.type === \"error\");\n      for (const error of errorMessages) {\n        const icon = colors.redBright(\"✖\");\n        const level = colors.redBright(colors.bold(\"ERROR\"));\n        console.log(\n          `\\n${icon} ${level} ${colors.dim(\"in\")} ${colors.cyanBright(fileName)}${colors.dim(`(${error.lineNumber},${error.columnNumber})`)}`,\n        );\n        // Get rule ID from the error if available\n        const ruleId = error.ruleId || \"eslint\";\n        console.log(\n          `  ${colors.magentaBright(ruleId)} ${colors.dim(\"→\")} ${colors.red(error.message)}`,\n        );\n\n        // Display code line with underline indicator\n        if (\n          sourceLines.length > 0 &&\n          error.lineNumber > 0 &&\n          error.lineNumber <= sourceLines.length\n        ) {\n          const lineIndex = error.lineNumber - 1; // Convert to 0-based index\n          const lineContent = sourceLines[lineIndex];\n          const lineNum = error.lineNumber.toString().padStart(4, \" \");\n          const errorLength = error.length || 1;\n          const columnIndex = error.columnNumber - 1; // Convert to 0-based index\n\n          // Display the line with normal color (not all red)\n          console.log(`  ${colors.dim(lineNum)} ${colors.dim(\"│\")} ${lineContent || \"\"}`);\n\n          // Display underline indicator\n          // Padding accounts for: lineNum.length + 1 space + 1 (│) + 1 space = lineNum.length + 3\n          // (The initial 2 spaces indent is already in the console.log)\n          const prefixPadding = lineNum.length + 3;\n          const columnPadding = \" \".repeat(columnIndex);\n          const underline = colors.redBright(\"~\".repeat(Math.max(1, errorLength)));\n          console.log(`  ${colors.dim(\" \".repeat(prefixPadding))}${columnPadding}${underline}`);\n        }\n      }\n    }\n\n    // Display warnings\n    if (warningCount > 0) {\n      const warningMessages = result.messages.filter((m) => m.type === \"warning\");\n      for (const warning of warningMessages) {\n        const icon = colors.yellowBright(\"⚠\");\n        const level = colors.yellowBright(colors.bold(\"WARNING\"));\n        console.log(\n          `\\n${icon} ${level} ${colors.dim(\"in\")} ${colors.cyanBright(fileName)}${colors.dim(`(${warning.lineNumber},${warning.columnNumber})`)}`,\n        );\n        // Get rule ID from the warning if available\n        const ruleId = warning.ruleId || \"eslint\";\n        console.log(\n          `  ${colors.magentaBright(ruleId)} ${colors.dim(\"→\")} ${colors.yellow(warning.message)}`,\n        );\n\n        // Display code line with underline indicator\n        if (\n          sourceLines.length > 0 &&\n          warning.lineNumber > 0 &&\n          warning.lineNumber <= sourceLines.length\n        ) {\n          const lineIndex = warning.lineNumber - 1; // Convert to 0-based index\n          const lineContent = sourceLines[lineIndex];\n          const lineNum = warning.lineNumber.toString().padStart(4, \" \");\n          const warningLength = warning.length || 1;\n          const columnIndex = warning.columnNumber - 1; // Convert to 0-based index\n\n          // Display the line with normal color (not all yellow)\n          console.log(`  ${colors.dim(lineNum)} ${colors.dim(\"│\")} ${lineContent || \"\"}`);\n\n          // Display underline indicator\n          // Padding accounts for: lineNum.length + 1 space + 1 (│) + 1 space = lineNum.length + 3\n          // (The initial 2 spaces indent is already in the console.log)\n          const prefixPadding = lineNum.length + 3;\n          const columnPadding = \" \".repeat(columnIndex);\n          const underline = colors.yellowBright(\"~\".repeat(Math.max(1, warningLength)));\n          console.log(`  ${colors.dim(\" \".repeat(prefixPadding))}${columnPadding}${underline}`);\n        }\n      }\n    }\n\n    // Display summary\n    const summary = [];\n    if (errorCount > 0) {\n      summary.push(colors.red(`${errorCount} error${errorCount > 1 ? \"s\" : \"\"}`));\n    }\n    if (warningCount > 0) {\n      summary.push(colors.yellow(`${warningCount} warning${warningCount > 1 ? \"s\" : \"\"}`));\n    }\n  }\n\n  /**\n   * Validate the health of the file\n   */\n  public async validate(file: FileManager, result: FileHealthResult): Promise<FileHealthResult> {\n    // Early exit: skip non-lintable files\n    if (!this.isLintableFile(file.absolutePath)) {\n      result.markAsHealthy();\n      return result;\n    }\n\n    // Early exit: check if ESLint is available\n    if (!this.eslint) {\n      result.markAsHealthy();\n      return result;\n    }\n\n    try {\n      // Use lintText to lint the source code directly from FileManager\n      // This ensures we always use the latest content\n      const lintResults = await this.eslint.lintText(file.source, {\n        filePath: file.absolutePath,\n      });\n\n      const lintResult = lintResults[0];\n      if (!lintResult) {\n        console.log(\"No lint results\", file.relativePath);\n        result.markAsHealthy();\n        return result;\n      }\n\n      // Convert ESLint messages to FileHealthResult format\n      const errors: Array<{\n        message: string;\n        type: \"error\";\n        lineNumber: number;\n        columnNumber: number;\n        length: number;\n        ruleId?: string;\n      }> = [];\n      const warnings: Array<{\n        message: string;\n        type: \"warning\";\n        lineNumber: number;\n        columnNumber: number;\n        length: number;\n        ruleId?: string;\n      }> = [];\n\n      for (const message of lintResult.messages) {\n        const isError = message.severity === 2;\n        const isWarning = message.severity === 1;\n\n        if (isError) {\n          errors.push({\n            message: message.message,\n            type: \"error\",\n            lineNumber: message.line || 1,\n            columnNumber: message.column || 1,\n            length: message.endColumn && message.column ? message.endColumn - message.column : 1,\n            ruleId: message.ruleId || undefined,\n          });\n        } else if (isWarning) {\n          warnings.push({\n            message: message.message,\n            type: \"warning\",\n            lineNumber: message.line || 1,\n            columnNumber: message.column || 1,\n            length: message.endColumn && message.column ? message.endColumn - message.column : 1,\n            ruleId: message.ruleId || undefined,\n          });\n        }\n      }\n\n      // Add errors and warnings to result\n      if (errors.length > 0) {\n        result.addErrors(errors);\n      }\n\n      if (warnings.length > 0) {\n        result.addWarnings(warnings);\n      }\n\n      // If no errors or warnings, mark as healthy\n      if (errors.length === 0 && warnings.length === 0) {\n        result.markAsHealthy();\n      } else {\n        // Display results if there are errors or warnings\n        this.displayResults(file, result);\n      }\n    } catch (error) {\n      // Handle any errors during validation gracefully\n      console.log(\"ESlint Error:\", error);\n      result.markAsHealthy(); // Return healthy on error to avoid blocking\n    }\n\n    return result;\n  }\n}\n"],"mappings":";;;;;;;AASA,IAAa,sBAAb,cAAyC,kBAAuD;CAC9F,AAAO,cAAc;EACnB,MAAM;gBAQwB;cAKV;oBAMM;qBAKG;EAtB7B,sBAAsB;CACxB;;;;CA0BA,AAAQ,eAAe,UAA2B;EAChD,MAAM,MAAM,SAAS,YAAY;EACjC,OACE,IAAI,SAAS,KAAK,KAAK,IAAI,SAAS,MAAM,KAAK,IAAI,SAAS,KAAK,KAAK,IAAI,SAAS,MAAM;CAE7F;;;;CAKA,AAAO,aAAkC;EACvC,IAAI;GAGF,IAAI,CAFkB,UAAU,QAAQ,IAAI,CAE3B,GAAG;IAElB,KAAK,cAAc;IACnB,OAAO;GACT;GAIA,KAAK,SAAS,IAAI,OAAO,EACvB,KAAK,QAAQ,IAAI,EACnB,CAAC;GAED,KAAK,cAAc;EACrB,SAAS,OAAO;GAEd,QAAQ,KAAK,gDAAgD,KAAK;GAClE,KAAK,cAAc;EACrB;EAEA,OAAO;CACT;;;;CAKA,AAAQ,eAAe,MAAmB,QAAgC;EACxE,MAAM,QAAQ,OAAO,SAAS;EAG9B,IAAI,MAAM,WAAW,KAAK,MAAM,aAAa,GAC3C;EAGF,MAAM,WAAW,KAAK,aAAa,QAAQ,OAAO,GAAG;EACrD,MAAM,aAAa,MAAM;EACzB,MAAM,eAAe,MAAM;EAC3B,MAAM,cAAc,KAAK,SAAS,KAAK,OAAO,MAAM,IAAI,IAAI,CAAC;EAG7D,IAAI,aAAa,GAAG;GAClB,MAAM,gBAAgB,OAAO,SAAS,QAAQ,MAAM,EAAE,SAAS,OAAO;GACtE,KAAK,MAAM,SAAS,eAAe;IACjC,MAAM,OAAO,OAAO,UAAU,GAAG;IACjC,MAAM,QAAQ,OAAO,UAAU,OAAO,KAAK,OAAO,CAAC;IACnD,QAAQ,IACN,KAAK,KAAK,GAAG,MAAM,GAAG,OAAO,IAAI,IAAI,EAAE,GAAG,OAAO,WAAW,QAAQ,IAAI,OAAO,IAAI,IAAI,MAAM,WAAW,GAAG,MAAM,aAAa,EAAE,GAClI;IAEA,MAAM,SAAS,MAAM,UAAU;IAC/B,QAAQ,IACN,KAAK,OAAO,cAAc,MAAM,EAAE,GAAG,OAAO,IAAI,GAAG,EAAE,GAAG,OAAO,IAAI,MAAM,OAAO,GAClF;IAGA,IACE,YAAY,SAAS,KACrB,MAAM,aAAa,KACnB,MAAM,cAAc,YAAY,QAChC;KAEA,MAAM,cAAc,YADF,MAAM,aAAa;KAErC,MAAM,UAAU,MAAM,WAAW,SAAS,CAAC,CAAC,SAAS,GAAG,GAAG;KAC3D,MAAM,cAAc,MAAM,UAAU;KACpC,MAAM,cAAc,MAAM,eAAe;KAGzC,QAAQ,IAAI,KAAK,OAAO,IAAI,OAAO,EAAE,GAAG,OAAO,IAAI,GAAG,EAAE,GAAG,eAAe,IAAI;KAK9E,MAAM,gBAAgB,QAAQ,SAAS;KACvC,MAAM,gBAAgB,IAAI,OAAO,WAAW;KAC5C,MAAM,YAAY,OAAO,UAAU,IAAI,OAAO,KAAK,IAAI,GAAG,WAAW,CAAC,CAAC;KACvE,QAAQ,IAAI,KAAK,OAAO,IAAI,IAAI,OAAO,aAAa,CAAC,IAAI,gBAAgB,WAAW;IACtF;GACF;EACF;EAGA,IAAI,eAAe,GAAG;GACpB,MAAM,kBAAkB,OAAO,SAAS,QAAQ,MAAM,EAAE,SAAS,SAAS;GAC1E,KAAK,MAAM,WAAW,iBAAiB;IACrC,MAAM,OAAO,OAAO,aAAa,GAAG;IACpC,MAAM,QAAQ,OAAO,aAAa,OAAO,KAAK,SAAS,CAAC;IACxD,QAAQ,IACN,KAAK,KAAK,GAAG,MAAM,GAAG,OAAO,IAAI,IAAI,EAAE,GAAG,OAAO,WAAW,QAAQ,IAAI,OAAO,IAAI,IAAI,QAAQ,WAAW,GAAG,QAAQ,aAAa,EAAE,GACtI;IAEA,MAAM,SAAS,QAAQ,UAAU;IACjC,QAAQ,IACN,KAAK,OAAO,cAAc,MAAM,EAAE,GAAG,OAAO,IAAI,GAAG,EAAE,GAAG,OAAO,OAAO,QAAQ,OAAO,GACvF;IAGA,IACE,YAAY,SAAS,KACrB,QAAQ,aAAa,KACrB,QAAQ,cAAc,YAAY,QAClC;KAEA,MAAM,cAAc,YADF,QAAQ,aAAa;KAEvC,MAAM,UAAU,QAAQ,WAAW,SAAS,CAAC,CAAC,SAAS,GAAG,GAAG;KAC7D,MAAM,gBAAgB,QAAQ,UAAU;KACxC,MAAM,cAAc,QAAQ,eAAe;KAG3C,QAAQ,IAAI,KAAK,OAAO,IAAI,OAAO,EAAE,GAAG,OAAO,IAAI,GAAG,EAAE,GAAG,eAAe,IAAI;KAK9E,MAAM,gBAAgB,QAAQ,SAAS;KACvC,MAAM,gBAAgB,IAAI,OAAO,WAAW;KAC5C,MAAM,YAAY,OAAO,aAAa,IAAI,OAAO,KAAK,IAAI,GAAG,aAAa,CAAC,CAAC;KAC5E,QAAQ,IAAI,KAAK,OAAO,IAAI,IAAI,OAAO,aAAa,CAAC,IAAI,gBAAgB,WAAW;IACtF;GACF;EACF;EAGA,MAAM,UAAU,CAAC;EACjB,IAAI,aAAa,GACf,QAAQ,KAAK,OAAO,IAAI,GAAG,WAAW,QAAQ,aAAa,IAAI,MAAM,IAAI,CAAC;EAE5E,IAAI,eAAe,GACjB,QAAQ,KAAK,OAAO,OAAO,GAAG,aAAa,UAAU,eAAe,IAAI,MAAM,IAAI,CAAC;CAEvF;;;;CAKA,MAAa,SAAS,MAAmB,QAAqD;EAE5F,IAAI,CAAC,KAAK,eAAe,KAAK,YAAY,GAAG;GAC3C,OAAO,cAAc;GACrB,OAAO;EACT;EAGA,IAAI,CAAC,KAAK,QAAQ;GAChB,OAAO,cAAc;GACrB,OAAO;EACT;EAEA,IAAI;GAOF,MAAM,cAAa,MAJO,KAAK,OAAO,SAAS,KAAK,QAAQ,EAC1D,UAAU,KAAK,aACjB,CAAC,EAE6B,CAAC;GAC/B,IAAI,CAAC,YAAY;IACf,QAAQ,IAAI,mBAAmB,KAAK,YAAY;IAChD,OAAO,cAAc;IACrB,OAAO;GACT;GAGA,MAAM,SAOD,CAAC;GACN,MAAM,WAOD,CAAC;GAEN,KAAK,MAAM,WAAW,WAAW,UAAU;IACzC,MAAM,UAAU,QAAQ,aAAa;IACrC,MAAM,YAAY,QAAQ,aAAa;IAEvC,IAAI,SACF,OAAO,KAAK;KACV,SAAS,QAAQ;KACjB,MAAM;KACN,YAAY,QAAQ,QAAQ;KAC5B,cAAc,QAAQ,UAAU;KAChC,QAAQ,QAAQ,aAAa,QAAQ,SAAS,QAAQ,YAAY,QAAQ,SAAS;KACnF,QAAQ,QAAQ,UAAU;IAC5B,CAAC;SACI,IAAI,WACT,SAAS,KAAK;KACZ,SAAS,QAAQ;KACjB,MAAM;KACN,YAAY,QAAQ,QAAQ;KAC5B,cAAc,QAAQ,UAAU;KAChC,QAAQ,QAAQ,aAAa,QAAQ,SAAS,QAAQ,YAAY,QAAQ,SAAS;KACnF,QAAQ,QAAQ,UAAU;IAC5B,CAAC;GAEL;GAGA,IAAI,OAAO,SAAS,GAClB,OAAO,UAAU,MAAM;GAGzB,IAAI,SAAS,SAAS,GACpB,OAAO,YAAY,QAAQ;GAI7B,IAAI,OAAO,WAAW,KAAK,SAAS,WAAW,GAC7C,OAAO,cAAc;QAGrB,KAAK,eAAe,MAAM,MAAM;EAEpC,SAAS,OAAO;GAEd,QAAQ,IAAI,iBAAiB,KAAK;GAClC,OAAO,cAAc;EACvB;EAEA,OAAO;CACT;AACF"}

package/esm/dev-server/health-checker/eslint-support.mjs

@@ -1,27 +0,0 @@
-import path from "node:path";
-import { existsSync } from "node:fs";
-
-//#region ../../packages/core/src/dev-server/health-checker/eslint-support.ts
-/** Whether the app has an ESLint flat config at its root. */
-function hasEslint(cwd) {
-	return [
-		"js",
-		"mjs",
-		"cjs",
-		"ts",
-		"mts",
-		"cts"
-	].some((extension) => existsSync(path.join(cwd, `eslint.config.${extension}`)));
-}
-const ESLINT_DEPRECATION = "Deprecated: ESLint + Prettier support will be removed in 5.34.0. Migrate to Oxlint + Oxfmt with npx warlock upgrade --from 5.32 --to 5.33.0; remove the ESLint opt-in afterwards.";
-let warned = false;
-/** One warning per process, on stderr so JSON reports stay parseable. */
-function warnEslintDeprecation() {
-	if (warned) return;
-	warned = true;
-	console.warn(ESLINT_DEPRECATION);
-}
-
-//#endregion
-export { ESLINT_DEPRECATION, hasEslint, warnEslintDeprecation };
-//# sourceMappingURL=eslint-support.mjs.map

package/esm/dev-server/health-checker/eslint-support.mjs.map

@@ -1 +0,0 @@
-{"version":3,"file":"eslint-support.mjs","names":[],"sources":["../../../../../../../../../packages/core/src/dev-server/health-checker/eslint-support.ts"],"sourcesContent":["import { existsSync } from \"node:fs\";\nimport path from \"node:path\";\n\n/** Whether the app has an ESLint flat config at its root. */\nexport function hasEslint(cwd: string): boolean {\n  return [\"js\", \"mjs\", \"cjs\", \"ts\", \"mts\", \"cts\"].some((extension) =>\n    existsSync(path.join(cwd, `eslint.config.${extension}`)),\n  );\n}\n\nexport const ESLINT_DEPRECATION =\n  \"Deprecated: ESLint + Prettier support will be removed in 5.34.0. Migrate to Oxlint + Oxfmt with npx warlock upgrade --from 5.32 --to 5.33.0; remove the ESLint opt-in afterwards.\";\n\nlet warned = false;\n\n/** One warning per process, on stderr so JSON reports stay parseable. */\nexport function warnEslintDeprecation(): void {\n  if (warned) return;\n  warned = true;\n  console.warn(ESLINT_DEPRECATION);\n}\n"],"mappings":";;;;;AAIA,SAAgB,UAAU,KAAsB;CAC9C,OAAO;EAAC;EAAM;EAAO;EAAO;EAAM;EAAO;CAAK,CAAC,CAAC,MAAM,cACpD,WAAW,KAAK,KAAK,KAAK,iBAAiB,WAAW,CAAC,CACzD;AACF;AAEA,MAAa,qBACX;AAEF,IAAI,SAAS;;AAGb,SAAgB,wBAA8B;CAC5C,IAAI,QAAQ;CACZ,SAAS;CACT,QAAQ,KAAK,kBAAkB;AACjC"}

package/esm/dev-server/health-checker/workers/eslint-health.worker.d.mts

@@ -1 +0,0 @@
-export {}