react-doctor 0.5.8-dev.b6d1a87 → 0.5.8-dev.bd0f465

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]="1e81c0a2-fd08-5f5b-ba28-bd5f7587e0f4")}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]="fabe3e34-b40f-514e-a5af-fa68a36365ae")}catch(e){}}();
 import { createRequire } from "node:module";
 import * as NodeChildProcess from "node:child_process";
 import { execFile, execFileSync, spawn, spawnSync } from "node:child_process";
@@ -40522,11 +40522,6 @@ 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));
@@ -40832,7 +40827,6 @@ 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",
@@ -45109,7 +45103,7 @@ const makeNoopConsole = () => ({
 });
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.5.8-dev.b6d1a87";
+const VERSION = "0.5.8-dev.bd0f465";
 //#endregion
 //#region src/cli/utils/json-mode.ts
 let context = null;
@@ -45478,13 +45472,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.b6d1a87`;
+const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.5.8-dev.bd0f465`;
 /**
 * 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.b6d1a87") ? "development" : "production");
+const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.5.8-dev.bd0f465") ? "development" : "production");
 /**
 * Performance-tracing sample rate in `[0, 1]`. Reads `SENTRY_TRACES_SAMPLE_RATE`
 * (set to `0` to disable tracing) and falls back to
@@ -50568,7 +50562,6 @@ const NON_BLOCKING_REACT_DOCTOR_COMMAND = [
 	`if ${REACT_DOCTOR_COMMAND} > "$react_doctor_output" 2>&1; then`,
 	"rm -f \"$react_doctor_output\";",
 	"else",
-	"cat \"$react_doctor_output\" >&2;",
 	"rm -f \"$react_doctor_output\";",
 	`printf "%s\\n" "React Doctor found staged regressions." "Run ${REACT_DOCTOR_COMMAND} to inspect." "Want them fixed? Ask your agent to run that command and resolve the findings." >&2;`,
 	"fi"
@@ -52266,11 +52259,10 @@ 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.mjs";
-const CLAUDE_HOOK_COMMAND = "node \"$CLAUDE_PROJECT_DIR/.claude/hooks/react-doctor.mjs\"";
+const CLAUDE_HOOK_RELATIVE_PATH = ".claude/hooks/react-doctor.sh";
+const CLAUDE_HOOK_COMMAND = "sh \"$CLAUDE_PROJECT_DIR/.claude/hooks/react-doctor.sh\"";
 const CURSOR_HOOKS_RELATIVE_PATH = ".cursor/hooks.json";
-const CURSOR_HOOK_RELATIVE_PATH = ".cursor/hooks/react-doctor.mjs";
-const CURSOR_HOOK_COMMAND = "node .cursor/hooks/react-doctor.mjs";
+const CURSOR_HOOK_RELATIVE_PATH = ".cursor/hooks/react-doctor.sh";
 const CURSOR_HOOK_MATCHER = "Write|Edit|MultiEdit|ApplyPatch";
 const CURSOR_HOOKS_SCHEMA_VERSION = 1;
 const isSupportedAgent = (agent) => agent === CLAUDE_AGENT || agent === CURSOR_AGENT;
@@ -52302,6 +52294,7 @@ 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) => {
@@ -52322,7 +52315,7 @@ const installClaudeHook = (projectRoot) => {
 	writeHookScript(hookPath);
 	return [settingsPath, hookPath];
 };
-const hasCursorHookCommand = (handlers) => handlers.some((handler) => handler.command === CURSOR_HOOK_COMMAND);
+const hasCursorHookCommand = (handlers) => handlers.some((handler) => handler.command === CURSOR_HOOK_RELATIVE_PATH);
 const installCursorHook = (projectRoot) => {
 	const configPath = Path.join(projectRoot, CURSOR_HOOKS_RELATIVE_PATH);
 	const hookPath = Path.join(projectRoot, CURSOR_HOOK_RELATIVE_PATH);
@@ -52330,7 +52323,7 @@ const installCursorHook = (projectRoot) => {
 	const hooks = { ...config.hooks ?? {} };
 	const postToolUseHooks = [...hooks.postToolUse ?? []];
 	if (!hasCursorHookCommand(postToolUseHooks)) postToolUseHooks.push({
-		command: CURSOR_HOOK_COMMAND,
+		command: CURSOR_HOOK_RELATIVE_PATH,
 		matcher: CURSOR_HOOK_MATCHER,
 		timeout: 120
 	});
@@ -52344,111 +52337,102 @@ const installCursorHook = (projectRoot) => {
 	return [configPath, hookPath];
 };
 const buildAgentHookScript = () => [
-	"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';",
+	"#!/bin/sh",
+	"set -u",
 	"",
-	"const __filename = fileURLToPath(import.meta.url);",
-	"const __dirname = dirname(__filename);",
+	"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 EDIT_TOOL_NAMES = new Set(['Edit', 'Write', 'MultiEdit', 'NotebookEdit', 'ApplyPatch']);",
+	"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 readStdin = () => {",
-	"  try {",
-	"    return readFileSync(0, 'utf8');",
-	"  } catch {",
-	"    return '';",
-	"  }",
-	"};",
+	"should_scan() {",
+	"  if ! command -v node >/dev/null 2>&1; then",
+	"    return 0",
+	"  fi",
 	"",
-	"const shouldScan = (input) => {",
+	"  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 eventName = input.hook_event_name || input.eventName || input.event_name;",
 	"  if (eventName === 'PostToolBatch') {",
 	"    const toolCalls = Array.isArray(input.tool_calls) ? input.tool_calls : [];",
-	"    return toolCalls.some((toolCall) => EDIT_TOOL_NAMES.has(toolCall.tool_name));",
+	"    process.exit(toolCalls.some((toolCall) => editToolNames.has(toolCall.tool_name)) ? 0 : 10);",
 	"  }",
 	"  const toolName = input.tool_name || input.toolName || input.tool;",
-	"  return !toolName || EDIT_TOOL_NAMES.has(toolName);",
-	"};",
+	"  process.exit(!toolName || editToolNames.has(toolName) ? 0 : 10);",
+	"} catch {",
+	"  process.exit(0);",
+	"}",
+	"NODE",
+	"}",
 	"",
-	"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',",
-	"  ];",
+	"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",
 	"",
-	"  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 react-doctor >/dev/null 2>&1; then",
+	"    react-doctor --verbose --scope changed --blocking warning --no-score",
+	"    return",
+	"  fi",
 	"",
-	"  return 0;",
-	"};",
+	"  if command -v pnpm >/dev/null 2>&1; then",
+	"    pnpm dlx react-doctor@latest --verbose --scope changed --blocking warning --no-score",
+	"    return",
+	"  fi",
 	"",
-	"const cleanup = (...paths) => {",
-	"  for (const path of paths) {",
-	"    try { unlinkSync(path); } catch {}",
-	"  }",
-	"};",
+	"  if command -v npx >/dev/null 2>&1; then",
+	"    npx --yes react-doctor@latest --verbose --scope changed --blocking warning --no-score",
+	"    return",
+	"  fi",
 	"",
-	"const main = () => {",
-	"  let input;",
-	"  try {",
-	"    const stdinContent = readStdin();",
-	"    input = JSON.parse(stdinContent || '{}');",
-	"  } catch {",
-	"    input = {};",
-	"  }",
+	"  printf '%s\\n' 'react-doctor: command not found; skipping agent hook scan.'",
+	"  return 0",
+	"}",
 	"",
-	"  if (!shouldScan(input)) {",
-	"    process.exit(0);",
-	"  }",
+	"if ! should_scan; 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`);",
+	"if run_react_doctor > \"$output_file\" 2>&1; then",
+	"  exit 0",
+	"fi",
 	"",
+	"node - \"$input_file\" \"$output_file\" <<'NODE'",
+	"const fs = require('node:fs');",
+	"const inputPath = process.argv[2];",
+	"const outputPath = process.argv[3];",
+	"const readInput = () => {",
 	"  try {",
-	"    process.chdir(projectRoot);",
+	"    return JSON.parse(fs.readFileSync(inputPath, 'utf8') || '{}');",
 	"  } catch {",
-	"    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 }));",
+	"    return {};",
 	"  }",
 	"};",
-	"",
-	"main();"
+	"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",
+	""
 ].join("\n");
 const installReactDoctorAgentHooks = (options) => {
 	const installedAgents = [];
@@ -52509,7 +52493,6 @@ const buildReactDoctorHookBlock = () => [
 	"if react_doctor_scan_staged_files > \"$react_doctor_output\" 2>&1; then",
 	"  rm -f \"$react_doctor_output\"",
 	"else",
-	"  cat \"$react_doctor_output\" >&2",
 	"  rm -f \"$react_doctor_output\"",
 	`  printf '%s\\n' "React Doctor found staged regressions." "Run ${REACT_DOCTOR_COMMAND} to inspect." "Want them fixed? Ask your agent to run that command and resolve the findings." >&2`,
 	"fi",
@@ -55387,14 +55370,6 @@ 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 {
@@ -55753,7 +55728,6 @@ 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);
@@ -55772,4 +55746,4 @@ Promise.resolve().then(() => assertNoRemovedFlags(process.argv)).then(() => prog
 export {};
 
 //# sourceMappingURL=cli.js.map
-//# debugId=1e81c0a2-fd08-5f5b-ba28-bd5f7587e0f4
+//# debugId=fabe3e34-b40f-514e-a5af-fa68a36365ae

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]="faa2d232-c423-58f8-97f4-ba8d41a199d7")}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]="9807b1bf-1e62-5421-ad35-d1f4129c1538")}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,11 +37290,6 @@ 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));
@@ -37600,7 +37595,6 @@ 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",
@@ -41651,4 +41645,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=faa2d232-c423-58f8-97f4-ba8d41a199d7
+//# debugId=9807b1bf-1e62-5421-ad35-d1f4129c1538

package/dist/lsp.js

@@ -37275,11 +37275,6 @@ 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));
@@ -37585,7 +37580,6 @@ 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",
@@ -43371,5 +43365,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]="47940f68-872d-59ed-bdd2-40ef7848566c")}catch(e){}}();
-//# debugId=47940f68-872d-59ed-bdd2-40ef7848566c
+!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

package/package.json

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