react-doctor 0.5.8-dev.f69f216 → 0.5.8-dev.fff9466

package/dist/cli.js

@@ -1,5 +1,5 @@
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="fabe3e34-b40f-514e-a5af-fa68a36365ae")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="f1523493-79d1-56b7-9f5f-78f1dd741d1e")}catch(e){}}();
 import { createRequire } from "node:module";
 import * as NodeChildProcess from "node:child_process";
 import { execFile, execFileSync, spawn, spawnSync } from "node:child_process";
@@ -40522,6 +40522,11 @@ const toCanonicalPath = (filePath) => {
 };
 const DEAD_CODE_PLUGIN = "deslop";
 const DEAD_CODE_CATEGORY = "Maintainability";
+const REACT_DOCTOR_TOOLCHAIN_PACKAGES = new Set([
+	"react-doctor",
+	"eslint-plugin-react-doctor",
+	"oxlint-plugin-react-doctor"
+]);
 const DEAD_CODE_WORKER_SCRIPT = `
 const inputChunks = [];
 process.stdin.on("data", (chunk) => inputChunks.push(chunk));
@@ -40827,6 +40832,7 @@ const checkDeadCode = async (options) => {
 		});
 	}
 	for (const unusedDependency of result.unusedDependencies) {
+		if (REACT_DOCTOR_TOOLCHAIN_PACKAGES.has(unusedDependency.name)) continue;
 		const label = unusedDependency.isDevDependency ? "devDependency" : "dependency";
 		diagnostics.push({
 			filePath: "package.json",
@@ -45103,7 +45109,7 @@ const makeNoopConsole = () => ({
 });
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.5.8-dev.f69f216";
+const VERSION = "0.5.8-dev.fff9466";
 //#endregion
 //#region src/cli/utils/json-mode.ts
 let context = null;
@@ -45472,13 +45478,13 @@ const isDevVersion = (version) => version === "0.0.0" || version.includes("-");
 * uploads source-map artifacts under, so stack frames symbolicate. Honors the
 * standard `SENTRY_RELEASE` override.
 */
-const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.5.8-dev.f69f216`;
+const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.5.8-dev.fff9466`;
 /**
 * Deployment environment shown in Sentry's environment filter. Defaults to
 * `production` for tagged releases and `development` for dev/unbuilt versions,
 * overridable via the standard `SENTRY_ENVIRONMENT` env var.
 */
-const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.5.8-dev.f69f216") ? "development" : "production");
+const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.5.8-dev.fff9466") ? "development" : "production");
 /**
 * Performance-tracing sample rate in `[0, 1]`. Reads `SENTRY_TRACES_SAMPLE_RATE`
 * (set to `0` to disable tracing) and falls back to
@@ -52259,10 +52265,11 @@ const rememberInstallAgents = (agents, options = {}) => writePreference(INSTALL_
 const CLAUDE_AGENT = "claude-code";
 const CURSOR_AGENT = "cursor";
 const CLAUDE_SETTINGS_RELATIVE_PATH = ".claude/settings.json";
-const CLAUDE_HOOK_RELATIVE_PATH = ".claude/hooks/react-doctor.sh";
-const CLAUDE_HOOK_COMMAND = "sh \"$CLAUDE_PROJECT_DIR/.claude/hooks/react-doctor.sh\"";
+const CLAUDE_HOOK_RELATIVE_PATH = ".claude/hooks/react-doctor.mjs";
+const CLAUDE_HOOK_COMMAND = "node \"$CLAUDE_PROJECT_DIR/.claude/hooks/react-doctor.mjs\"";
 const CURSOR_HOOKS_RELATIVE_PATH = ".cursor/hooks.json";
-const CURSOR_HOOK_RELATIVE_PATH = ".cursor/hooks/react-doctor.sh";
+const CURSOR_HOOK_RELATIVE_PATH = ".cursor/hooks/react-doctor.mjs";
+const CURSOR_HOOK_COMMAND = "node .cursor/hooks/react-doctor.mjs";
 const CURSOR_HOOK_MATCHER = "Write|Edit|MultiEdit|ApplyPatch";
 const CURSOR_HOOKS_SCHEMA_VERSION = 1;
 const isSupportedAgent = (agent) => agent === CLAUDE_AGENT || agent === CURSOR_AGENT;
@@ -52294,7 +52301,6 @@ const writeJsonFileWithDirectoryCheck = (filePath, value) => {
 const writeHookScript = (filePath) => {
 	ensureDirectoryExists(Path.dirname(filePath));
 	NFS.writeFileSync(filePath, buildAgentHookScript());
-	NFS.chmodSync(filePath, 493);
 };
 const hasClaudeHookCommand = (groups) => groups.some((group) => (group.hooks ?? []).some((hook) => hook.command === CLAUDE_HOOK_COMMAND));
 const installClaudeHook = (projectRoot) => {
@@ -52315,7 +52321,7 @@ const installClaudeHook = (projectRoot) => {
 	writeHookScript(hookPath);
 	return [settingsPath, hookPath];
 };
-const hasCursorHookCommand = (handlers) => handlers.some((handler) => handler.command === CURSOR_HOOK_RELATIVE_PATH);
+const hasCursorHookCommand = (handlers) => handlers.some((handler) => handler.command === CURSOR_HOOK_COMMAND);
 const installCursorHook = (projectRoot) => {
 	const configPath = Path.join(projectRoot, CURSOR_HOOKS_RELATIVE_PATH);
 	const hookPath = Path.join(projectRoot, CURSOR_HOOK_RELATIVE_PATH);
@@ -52323,7 +52329,7 @@ const installCursorHook = (projectRoot) => {
 	const hooks = { ...config.hooks ?? {} };
 	const postToolUseHooks = [...hooks.postToolUse ?? []];
 	if (!hasCursorHookCommand(postToolUseHooks)) postToolUseHooks.push({
-		command: CURSOR_HOOK_RELATIVE_PATH,
+		command: CURSOR_HOOK_COMMAND,
 		matcher: CURSOR_HOOK_MATCHER,
 		timeout: 120
 	});
@@ -52337,102 +52343,111 @@ const installCursorHook = (projectRoot) => {
 	return [configPath, hookPath];
 };
 const buildAgentHookScript = () => [
-	"#!/bin/sh",
-	"set -u",
+	"import { readFileSync, writeFileSync, unlinkSync } from 'node:fs';",
+	"import { tmpdir } from 'node:os';",
+	"import { join, dirname } from 'node:path';",
+	"import { fileURLToPath } from 'node:url';",
+	"import { spawnSync } from 'node:child_process';",
 	"",
-	"input_file=$(mktemp \"${TMPDIR:-/tmp}/react-doctor-agent-hook.XXXXXX\")",
-	"output_file=$(mktemp \"${TMPDIR:-/tmp}/react-doctor-agent-hook-output.XXXXXX\")",
-	"trap 'rm -f \"$input_file\" \"$output_file\"' EXIT",
-	"cat > \"$input_file\"",
+	"const __filename = fileURLToPath(import.meta.url);",
+	"const __dirname = dirname(__filename);",
 	"",
-	"script_dir=$(CDPATH= cd \"$(dirname \"$0\")\" && pwd)",
-	"project_root=${CLAUDE_PROJECT_DIR:-}",
-	"if [ -z \"$project_root\" ]; then",
-	"  project_root=$(CDPATH= cd \"$script_dir/../..\" && pwd)",
-	"fi",
-	"if ! cd \"$project_root\"; then",
-	"  exit 0",
-	"fi",
+	"const EDIT_TOOL_NAMES = new Set(['Edit', 'Write', 'MultiEdit', 'NotebookEdit', 'ApplyPatch']);",
 	"",
-	"should_scan() {",
-	"  if ! command -v node >/dev/null 2>&1; then",
-	"    return 0",
-	"  fi",
+	"const readStdin = () => {",
+	"  try {",
+	"    return readFileSync(0, 'utf8');",
+	"  } catch {",
+	"    return '';",
+	"  }",
+	"};",
 	"",
-	"  node - \"$input_file\" <<'NODE'",
-	"const fs = require('node:fs');",
-	"const inputPath = process.argv[2];",
-	"const editToolNames = new Set(['Edit', 'Write', 'MultiEdit', 'NotebookEdit', 'ApplyPatch']);",
-	"try {",
-	"  const input = JSON.parse(fs.readFileSync(inputPath, 'utf8') || '{}');",
+	"const shouldScan = (input) => {",
 	"  const eventName = input.hook_event_name || input.eventName || input.event_name;",
 	"  if (eventName === 'PostToolBatch') {",
 	"    const toolCalls = Array.isArray(input.tool_calls) ? input.tool_calls : [];",
-	"    process.exit(toolCalls.some((toolCall) => editToolNames.has(toolCall.tool_name)) ? 0 : 10);",
+	"    return toolCalls.some((toolCall) => EDIT_TOOL_NAMES.has(toolCall.tool_name));",
 	"  }",
 	"  const toolName = input.tool_name || input.toolName || input.tool;",
-	"  process.exit(!toolName || editToolNames.has(toolName) ? 0 : 10);",
-	"} catch {",
-	"  process.exit(0);",
-	"}",
-	"NODE",
-	"}",
+	"  return !toolName || EDIT_TOOL_NAMES.has(toolName);",
+	"};",
 	"",
-	"run_react_doctor() {",
-	"  if [ -x ./node_modules/.bin/react-doctor ]; then",
-	"    ./node_modules/.bin/react-doctor --verbose --scope changed --blocking warning --no-score",
-	"    return",
-	"  fi",
+	"const runReactDoctor = (outputPath) => {",
+	"  // Each candidate is a single shell command string (not an args array):",
+	"  // `shell: true` is required to run the Windows `.cmd` shims, and an args",
+	"  // array with `shell: true` trips Node's DEP0190. A missing command exits",
+	"  // 127 via the shell (no ENOENT error), so fall through on that. With no",
+	"  // runner found, exit 0 silently — stdout is parsed as the hook's JSON.",
+	"  const commands = [",
+	"    './node_modules/.bin/react-doctor --verbose --scope changed --blocking warning --no-score',",
+	"    'react-doctor --verbose --scope changed --blocking warning --no-score',",
+	"    'pnpm dlx react-doctor@latest --verbose --scope changed --blocking warning --no-score',",
+	"    'npx --yes react-doctor@latest --verbose --scope changed --blocking warning --no-score',",
+	"  ];",
 	"",
-	"  if command -v react-doctor >/dev/null 2>&1; then",
-	"    react-doctor --verbose --scope changed --blocking warning --no-score",
-	"    return",
-	"  fi",
+	"  for (const command of commands) {",
+	"    const result = spawnSync(command, { encoding: 'utf8', shell: true });",
+	"    if (result.error?.code === 'ENOENT' || result.status === 127) continue;",
+	"    try {",
+	"      writeFileSync(outputPath, (result.stdout || '') + (result.stderr || ''));",
+	"    } catch {}",
+	"    return result.status;",
+	"  }",
 	"",
-	"  if command -v pnpm >/dev/null 2>&1; then",
-	"    pnpm dlx react-doctor@latest --verbose --scope changed --blocking warning --no-score",
-	"    return",
-	"  fi",
+	"  return 0;",
+	"};",
 	"",
-	"  if command -v npx >/dev/null 2>&1; then",
-	"    npx --yes react-doctor@latest --verbose --scope changed --blocking warning --no-score",
-	"    return",
-	"  fi",
+	"const cleanup = (...paths) => {",
+	"  for (const path of paths) {",
+	"    try { unlinkSync(path); } catch {}",
+	"  }",
+	"};",
 	"",
-	"  printf '%s\\n' 'react-doctor: command not found; skipping agent hook scan.'",
-	"  return 0",
-	"}",
+	"const main = () => {",
+	"  let input;",
+	"  try {",
+	"    const stdinContent = readStdin();",
+	"    input = JSON.parse(stdinContent || '{}');",
+	"  } catch {",
+	"    input = {};",
+	"  }",
 	"",
-	"if ! should_scan; then",
-	"  exit 0",
-	"fi",
+	"  if (!shouldScan(input)) {",
+	"    process.exit(0);",
+	"  }",
 	"",
-	"if run_react_doctor > \"$output_file\" 2>&1; then",
-	"  exit 0",
-	"fi",
+	"  const projectRoot = process.env.CLAUDE_PROJECT_DIR || join(__dirname, '../..');",
+	"  const outputPath = join(tmpdir(), `react-doctor-agent-hook-output-${process.pid}.txt`);",
 	"",
-	"node - \"$input_file\" \"$output_file\" <<'NODE'",
-	"const fs = require('node:fs');",
-	"const inputPath = process.argv[2];",
-	"const outputPath = process.argv[3];",
-	"const readInput = () => {",
 	"  try {",
-	"    return JSON.parse(fs.readFileSync(inputPath, 'utf8') || '{}');",
+	"    process.chdir(projectRoot);",
 	"  } catch {",
-	"    return {};",
+	"    process.exit(0);",
+	"  }",
+	"",
+	"  const scanResult = runReactDoctor(outputPath);",
+	"  if (scanResult === 0) {",
+	"    cleanup(outputPath);",
+	"    process.exit(0);",
+	"  }",
+	"",
+	"  const scanOutput = readFileSync(outputPath, 'utf8').trim();",
+	"  cleanup(outputPath);",
+	"",
+	"  if (!scanOutput) {",
+	"    process.exit(0);",
+	"  }",
+	"",
+	"  const message = `React Doctor found issues in the changed files. Review this output and fix the regressions before finishing. For confirmed issues that cannot be fixed now, create GitHub issues with the rule, file/line, confidence, impact, and proposed fix.\\n\\n${scanOutput}`;",
+	"",
+	"  if (input.hook_event_name === 'PostToolBatch') {",
+	"    console.log(JSON.stringify({ hookSpecificOutput: { hookEventName: 'PostToolBatch', additionalContext: message } }));",
+	"  } else {",
+	"    console.log(JSON.stringify({ additional_context: message }));",
 	"  }",
 	"};",
-	"const input = readInput();",
-	"const scanOutput = fs.readFileSync(outputPath, 'utf8').trim();",
-	"if (!scanOutput) process.exit(0);",
-	"const message = `React Doctor found issues in the changed files. Review this output and fix the regressions before finishing. For confirmed issues that cannot be fixed now, create GitHub issues with the rule, file/line, confidence, impact, and proposed fix.\\n\\n${scanOutput}`;",
-	"if (input.hook_event_name === 'PostToolBatch') {",
-	"  console.log(JSON.stringify({ hookSpecificOutput: { hookEventName: 'PostToolBatch', additionalContext: message } }));",
-	"} else {",
-	"  console.log(JSON.stringify({ additional_context: message }));",
-	"}",
-	"NODE",
-	""
+	"",
+	"main();"
 ].join("\n");
 const installReactDoctorAgentHooks = (options) => {
 	const installedAgents = [];
@@ -55370,6 +55385,14 @@ const applyColorPreference = (argv, env = process.env) => {
 	setColorEnabled(enabled);
 };
 //#endregion
+//#region src/cli/utils/ensure-windows-utf8-console.ts
+const ensureWindowsUtf8Console = () => {
+	if (process.platform !== "win32" || !process.stdout.isTTY) return;
+	try {
+		execFileSync("chcp.com", ["65001"], { stdio: "ignore" });
+	} catch {}
+};
+//#endregion
 //#region src/cli/utils/exit-gracefully.ts
 const exitGracefully = () => {
 	try {
@@ -55728,6 +55751,7 @@ if (process.argv.includes("-V") && !strippedArgv.includes("-V")) {
 	process.exit(0);
 }
 applyColorPreference(strippedArgv);
+ensureWindowsUtf8Console();
 const argv = normalizeHelpInvocation(strippedArgv, knownCommands);
 Promise.resolve().then(() => assertNoRemovedFlags(process.argv)).then(() => program.parseAsync(argv)).then(() => flushSentry()).catch(async (error) => {
 	const isUserError = isExpectedUserError(error);
@@ -55746,4 +55770,4 @@ Promise.resolve().then(() => assertNoRemovedFlags(process.argv)).then(() => prog
 export {};
 
 //# sourceMappingURL=cli.js.map
-//# debugId=fabe3e34-b40f-514e-a5af-fa68a36365ae
+//# debugId=f1523493-79d1-56b7-9f5f-78f1dd741d1e

package/dist/index.js

@@ -1,5 +1,5 @@
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="9807b1bf-1e62-5421-ad35-d1f4129c1538")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="faa2d232-c423-58f8-97f4-ba8d41a199d7")}catch(e){}}();
 import { r as __toESM$1, t as __commonJSMin$1 } from "./chunk-N93fKeF6.js";
 import { createRequire } from "node:module";
 import * as NFS from "node:fs";
@@ -37290,6 +37290,11 @@ const toCanonicalPath = (filePath) => {
 };
 const DEAD_CODE_PLUGIN = "deslop";
 const DEAD_CODE_CATEGORY = "Maintainability";
+const REACT_DOCTOR_TOOLCHAIN_PACKAGES = new Set([
+	"react-doctor",
+	"eslint-plugin-react-doctor",
+	"oxlint-plugin-react-doctor"
+]);
 const DEAD_CODE_WORKER_SCRIPT = `
 const inputChunks = [];
 process.stdin.on("data", (chunk) => inputChunks.push(chunk));
@@ -37595,6 +37600,7 @@ const checkDeadCode = async (options) => {
 		});
 	}
 	for (const unusedDependency of result.unusedDependencies) {
+		if (REACT_DOCTOR_TOOLCHAIN_PACKAGES.has(unusedDependency.name)) continue;
 		const label = unusedDependency.isDevDependency ? "devDependency" : "dependency";
 		diagnostics.push({
 			filePath: "package.json",
@@ -41645,4 +41651,4 @@ const toJsonReport = (result, options) => buildJsonReport({
 export { AmbiguousProjectError, NoReactDependencyError, NotADirectoryError, PackageJsonNotFoundError, ProjectNotFoundError, ReactDoctorError, buildJsonReport, buildJsonReportError, clearCaches, defineConfig, diagnose, filterSourceFiles, getDiffInfo, isProjectDiscoveryError, isReactDoctorError, summarizeDiagnostics, toJsonReport };
 
 //# sourceMappingURL=index.js.map
-//# debugId=9807b1bf-1e62-5421-ad35-d1f4129c1538
+//# debugId=faa2d232-c423-58f8-97f4-ba8d41a199d7

package/dist/lsp.js

@@ -37275,6 +37275,11 @@ const toCanonicalPath = (filePath) => {
 };
 const DEAD_CODE_PLUGIN = "deslop";
 const DEAD_CODE_CATEGORY = "Maintainability";
+const REACT_DOCTOR_TOOLCHAIN_PACKAGES = new Set([
+	"react-doctor",
+	"eslint-plugin-react-doctor",
+	"oxlint-plugin-react-doctor"
+]);
 const DEAD_CODE_WORKER_SCRIPT = `
 const inputChunks = [];
 process.stdin.on("data", (chunk) => inputChunks.push(chunk));
@@ -37580,6 +37585,7 @@ const checkDeadCode = async (options) => {
 		});
 	}
 	for (const unusedDependency of result.unusedDependencies) {
+		if (REACT_DOCTOR_TOOLCHAIN_PACKAGES.has(unusedDependency.name)) continue;
 		const label = unusedDependency.isDevDependency ? "devDependency" : "dependency";
 		diagnostics.push({
 			filePath: "package.json",
@@ -43365,5 +43371,5 @@ const startLanguageServer = () => {
 };
 //#endregion
 export { startLanguageServer };
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="c5b6c740-461d-5f65-a993-9e6d35ad9113")}catch(e){}}();
-//# debugId=c5b6c740-461d-5f65-a993-9e6d35ad9113
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="47940f68-872d-59ed-bdd2-40ef7848566c")}catch(e){}}();
+//# debugId=47940f68-872d-59ed-bdd2-40ef7848566c

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.5.8-dev.f69f216",
+  "version": "0.5.8-dev.fff9466",
   "description": "Your agent writes bad React. This catches it",
   "keywords": [
     "accessibility",
@@ -65,7 +65,7 @@
     "vscode-languageserver-textdocument": "^1.0.12",
     "vscode-uri": "^3.1.0",
     "deslop-js": "0.5.8",
-    "oxlint-plugin-react-doctor": "0.5.8-dev.f69f216"
+    "oxlint-plugin-react-doctor": "0.5.8-dev.fff9466"
   },
   "devDependencies": {
     "@types/babel__code-frame": "^7.27.0",