react-doctor 0.9.12 → 0.9.13-dev.fa72869

package/dist/{install-react-doctor-Bq7dfEBI.js → install-react-doctor-0EPCiZSM.js}

@@ -1,10 +1,10 @@
 
-!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]="d678e544-bf9c-5a23-bf1c-c11caae57e45")}catch(e){}}();
-import { R as CliInputError, S as CI_TRUST_COMPANIES, cn as highlighter, ot as GITHUB_ACTIONS_SETUP_URL, t as recordCount, un as isErrnoException, w as METRIC, wt as SKILL_NAME } from "./record-metric-CsQXJewH.js";
-import { A as packageHasNestedRecordKey, D as getPackageJsonPath, E as ensureTrailingNewline, I as writeJsonFile, L as cliLogger, M as readPackageJson, N as resolveGitPath, O as isRecord, P as runGit, S as REACT_DOCTOR_COMMAND, T as VITE_PLUS_HOOKS_PATH, _ as LEFTHOOK_CONFIG_FILES, a as installReactDoctorScriptStep, b as OVERCOMMIT_CONFIG_FILE, c as spinner, g as HUSKY_HOOKS_PATH, h as HOOK_RELATIVE_PATH, j as packageHasRecordKey, k as packageHasDependency, m as HOOK_FILE_NAME, n as findNearestPackageDirectory, r as hasDoctorDependency, t as DOCTOR_PACKAGE_NAME, v as LEGACY_HOOK_RUNNER_RELATIVE_PATH, x as PRE_COMMIT_CONFIG_FILE, y as NON_BLOCKING_REACT_DOCTOR_COMMAND } from "./install-doctor-script-DLayXS7q.js";
-import { a as readReactDoctorWorkflow, c as workflowUsesV1Action, d as detectDefaultBranch, l as prompts, n as getReactDoctorWorkflowPath, o as upgradeReactDoctorWorkflowInPlace, r as installReactDoctorWorkflow } from "./install-github-workflow-Goz-KFqb.js";
-import { t as shouldSkipPrompts } from "./should-skip-prompts-D1GWG2qN.js";
-import { a as recordGate, c as ACTION_UPGRADE_EVENT, d as INSTALL_AGENTS_PREFERENCE_ID, h as detectAvailableAgents, i as readPreference, l as CI_PITCH_EVENT, m as computeDefaultSelectedAgents, n as openUrl, r as isGatePending, s as writePreference, t as reportWorkflowResult } from "./report-workflow-result-6n_wSqzA.js";
+!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]="d43042c4-c766-54b9-a30f-944a55426467")}catch(e){}}();
+import { C as CI_TRUST_COMPANIES, Kt as highlighter, T as METRIC, br as isErrnoException, cr as GITHUB_ACTIONS_SETUP_URL, gr as SKILL_NAME, t as recordCount, z as CliInputError } from "./record-metric-BcJN2xOB.js";
+import { C as isRecord, D as readPackageJson, E as packageHasRecordKey, M as cliLogger, O as resolveGitPath, S as getPackageJsonPath, T as packageHasNestedRecordKey, _ as REACT_DOCTOR_COMMAND, a as installReactDoctorScriptStep, b as VITE_PLUS_HOOKS_PATH, c as spinner, d as HUSKY_HOOKS_PATH, f as LEFTHOOK_CONFIG_FILES, g as PRE_COMMIT_CONFIG_FILE, h as OVERCOMMIT_CONFIG_FILE, j as writeJsonFile, k as runGit, l as HOOK_FILE_NAME, m as NON_BLOCKING_REACT_DOCTOR_COMMAND, n as findNearestPackageDirectory, p as LEGACY_HOOK_RUNNER_RELATIVE_PATH, r as hasDoctorDependency, t as DOCTOR_PACKAGE_NAME, u as HOOK_RELATIVE_PATH, w as packageHasDependency, x as ensureTrailingNewline } from "./install-doctor-script-Czibd55T.js";
+import { a as readReactDoctorWorkflow, c as workflowUsesV1Action, d as detectDefaultBranch, l as prompts, n as getReactDoctorWorkflowPath, o as upgradeReactDoctorWorkflowInPlace, r as installReactDoctorWorkflow } from "./install-github-workflow-DAW5HCoZ.js";
+import { t as shouldSkipPrompts } from "./should-skip-prompts-DkasjzpQ.js";
+import { a as recordGate, c as ACTION_UPGRADE_EVENT, d as INSTALL_AGENTS_PREFERENCE_ID, h as detectAvailableAgents, i as readPreference, l as CI_PITCH_EVENT, m as computeDefaultSelectedAgents, n as openUrl, r as isGatePending, s as writePreference, t as reportWorkflowResult } from "./report-workflow-result-C4EMosul.js";
 import { execFile } from "node:child_process";
 import * as path$1 from "node:path";
 import * as fs$1 from "node:fs";
@@ -124,10 +124,10 @@ const CLAUDE_HOOK_COMMAND = "node \"$CLAUDE_PROJECT_DIR/.claude/hooks/react-doct
 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_MATCHER = "Write|Edit|MultiEdit|ApplyPatch";
 const CURSOR_HOOKS_SCHEMA_VERSION = 1;
 const LEGACY_HOOK_SCRIPT_PATHS = [".claude/hooks/react-doctor.sh", ".cursor/hooks/react-doctor.sh"];
 const isLegacyHookCommand = (command) => typeof command === "string" && LEGACY_HOOK_SCRIPT_PATHS.some((legacyPath) => command.includes(legacyPath));
+const isManagedHookCommand = (command, installedCommand) => command === installedCommand || isLegacyHookCommand(command);
 const isSupportedAgent = (agent) => agent === CLAUDE_AGENT || agent === CURSOR_AGENT;
 const readJsonFile = (filePath, fallback) => {
 	if (!fs$1.existsSync(filePath)) return fallback;
@@ -147,10 +147,12 @@ const readJsonFileSafely = (filePath, fallback) => {
 		return fallback;
 	}
 };
-const findAgentsWithLegacyShellHooks = (projectRoot) => {
+const findAgentsWithOutdatedReactDoctorHooks = (projectRoot) => {
 	const agents = [];
-	if ((readJsonFileSafely(path$1.join(projectRoot, CLAUDE_SETTINGS_RELATIVE_PATH), {}).hooks?.PostToolBatch ?? []).some((group) => (group.hooks ?? []).some((hook) => isLegacyHookCommand(hook.command)))) agents.push(CLAUDE_AGENT);
-	if ((readJsonFileSafely(path$1.join(projectRoot, CURSOR_HOOKS_RELATIVE_PATH), {}).hooks?.postToolUse ?? []).some((handler) => isLegacyHookCommand(handler.command))) agents.push(CURSOR_AGENT);
+	const settings = readJsonFileSafely(path$1.join(projectRoot, CLAUDE_SETTINGS_RELATIVE_PATH), {});
+	if ([...settings.hooks?.PostToolBatch ?? [], ...settings.hooks?.Stop ?? []].some((group) => (group.hooks ?? []).some((hook) => isLegacyHookCommand(hook.command))) || (settings.hooks?.PostToolBatch ?? []).some((group) => (group.hooks ?? []).some((hook) => hook.command === CLAUDE_HOOK_COMMAND))) agents.push(CLAUDE_AGENT);
+	const config = readJsonFileSafely(path$1.join(projectRoot, CURSOR_HOOKS_RELATIVE_PATH), {});
+	if ([...config.hooks?.postToolUse ?? [], ...config.hooks?.stop ?? []].some((handler) => isLegacyHookCommand(handler.command)) || (config.hooks?.postToolUse ?? []).some((handler) => handler.command === CURSOR_HOOK_COMMAND)) agents.push(CURSOR_AGENT);
 	return agents;
 };
 const ensureDirectoryExists = (directoryPath) => {
@@ -167,62 +169,62 @@ const writeJsonFileWithDirectoryCheck = (filePath, value) => {
 	ensureDirectoryExists(path$1.dirname(filePath));
 	writeJsonFile(filePath, value);
 };
-const writeHookScript = (filePath) => {
+const writeHookScript = (filePath, hookEventName) => {
 	ensureDirectoryExists(path$1.dirname(filePath));
-	fs$1.writeFileSync(filePath, buildAgentHookScript());
+	fs$1.writeFileSync(filePath, buildAgentHookScript(hookEventName));
 	try {
 		fs$1.rmSync(filePath.replace(/\.mjs$/, ".sh"), { force: true });
 	} catch {}
 };
-const hasClaudeHookCommand = (groups) => groups.some((group) => (group.hooks ?? []).some((hook) => hook.command === CLAUDE_HOOK_COMMAND));
 const installClaudeHook = (projectRoot) => {
 	const settingsPath = path$1.join(projectRoot, CLAUDE_SETTINGS_RELATIVE_PATH);
 	const hookPath = path$1.join(projectRoot, CLAUDE_HOOK_RELATIVE_PATH);
 	const settings = readJsonFile(settingsPath, {});
 	const hooks = { ...settings.hooks ?? {} };
-	const postToolBatchHooks = (hooks.PostToolBatch ?? []).flatMap((group) => {
+	const stripManagedHooks = (groups) => groups.flatMap((group) => {
 		const groupHooks = group.hooks ?? [];
-		const keptHooks = groupHooks.filter((hook) => !isLegacyHookCommand(hook.command));
+		const keptHooks = groupHooks.filter((hook) => !isManagedHookCommand(hook.command, CLAUDE_HOOK_COMMAND));
 		if (keptHooks.length === groupHooks.length) return [group];
 		return keptHooks.length > 0 ? [{
 			...group,
 			hooks: keptHooks
 		}] : [];
 	});
-	if (!hasClaudeHookCommand(postToolBatchHooks)) postToolBatchHooks.push({ hooks: [{
+	const stopHooks = stripManagedHooks(hooks.Stop ?? []);
+	stopHooks.push({ hooks: [{
 		type: "command",
 		command: CLAUDE_HOOK_COMMAND
 	}] });
-	hooks.PostToolBatch = postToolBatchHooks;
+	hooks.PostToolBatch = stripManagedHooks(hooks.PostToolBatch ?? []);
+	hooks.Stop = stopHooks;
 	writeJsonFileWithDirectoryCheck(settingsPath, {
 		...settings,
 		hooks
 	});
-	writeHookScript(hookPath);
+	writeHookScript(hookPath, "Stop");
 	return [settingsPath, hookPath];
 };
-const hasCursorHookCommand = (handlers) => handlers.some((handler) => handler.command === CURSOR_HOOK_COMMAND);
 const installCursorHook = (projectRoot) => {
 	const configPath = path$1.join(projectRoot, CURSOR_HOOKS_RELATIVE_PATH);
 	const hookPath = path$1.join(projectRoot, CURSOR_HOOK_RELATIVE_PATH);
 	const config = readJsonFile(configPath, {});
 	const hooks = { ...config.hooks ?? {} };
-	const postToolUseHooks = (hooks.postToolUse ?? []).filter((handler) => !isLegacyHookCommand(handler.command));
-	if (!hasCursorHookCommand(postToolUseHooks)) postToolUseHooks.push({
+	const stopHooks = (hooks.stop ?? []).filter((handler) => !isManagedHookCommand(handler.command, CURSOR_HOOK_COMMAND));
+	stopHooks.push({
 		command: CURSOR_HOOK_COMMAND,
-		matcher: CURSOR_HOOK_MATCHER,
 		timeout: 120
 	});
-	hooks.postToolUse = postToolUseHooks;
+	hooks.postToolUse = (hooks.postToolUse ?? []).filter((handler) => !isManagedHookCommand(handler.command, CURSOR_HOOK_COMMAND));
+	hooks.stop = stopHooks;
 	writeJsonFileWithDirectoryCheck(configPath, {
 		...config,
 		version: config.version ?? CURSOR_HOOKS_SCHEMA_VERSION,
 		hooks
 	});
-	writeHookScript(hookPath);
+	writeHookScript(hookPath, "stop");
 	return [configPath, hookPath];
 };
-const buildAgentHookScript = () => [
+const buildAgentHookScript = (hookEventName) => [
 	"import { existsSync, readFileSync, writeFileSync, unlinkSync } from 'node:fs';",
 	"import { tmpdir } from 'node:os';",
 	"import { join, dirname } from 'node:path';",
@@ -231,12 +233,12 @@ const buildAgentHookScript = () => [
 	"",
 	"const __filename = fileURLToPath(import.meta.url);",
 	"const __dirname = dirname(__filename);",
+	`const HOOK_EVENT_NAME = ${JSON.stringify(hookEventName)};`,
+	`const MAX_CONTINUATIONS = 1;`,
 	"",
 	"// --verbose scans on large diffs can exceed spawnSync's 1 MiB default.",
 	"const SPAWN_MAX_BUFFER_BYTES = 16 * 1024 * 1024;",
 	"",
-	"const EDIT_TOOL_NAMES = new Set(['Edit', 'Write', 'MultiEdit', 'NotebookEdit', 'ApplyPatch']);",
-	"",
 	"const readFileOrEmpty = (source) => {",
 	"  try {",
 	"    return readFileSync(source, 'utf8');",
@@ -246,13 +248,10 @@ const buildAgentHookScript = () => [
 	"};",
 	"",
 	"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 : [];",
-	"    return toolCalls.some((toolCall) => EDIT_TOOL_NAMES.has(toolCall.tool_name));",
-	"  }",
-	"  const toolName = input.tool_name || input.toolName || input.tool;",
-	"  return !toolName || EDIT_TOOL_NAMES.has(toolName);",
+	"  if (HOOK_EVENT_NAME === 'Stop') return !input.stop_hook_active;",
+	"  if (input.status && input.status !== 'completed') return false;",
+	"  const loopCount = Number(input.loop_count);",
+	"  return !Number.isFinite(loopCount) || loopCount < MAX_CONTINUATIONS;",
 	"};",
 	"",
 	"const runReactDoctor = (outputPath) => {",
@@ -268,11 +267,11 @@ const buildAgentHookScript = () => [
 	"    : './node_modules/.bin/react-doctor';",
 	"  const commands = [",
 	"    ...(existsSync(localBin)",
-	"      ? [localBin + ' --verbose --scope changed --blocking warning --no-score']",
+	"      ? [localBin + ' --verbose --scope changed --include-untracked --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',",
+	"    'react-doctor --verbose --scope changed --include-untracked --blocking warning --no-score',",
+	"    'pnpm dlx react-doctor@latest --verbose --scope changed --include-untracked --blocking warning --no-score',",
+	"    'npx --yes react-doctor@latest --verbose --scope changed --include-untracked --blocking warning --no-score',",
 	"  ];",
 	"",
 	"  for (const command of commands) {",
@@ -331,10 +330,10 @@ const buildAgentHookScript = () => [
 	"",
 	"  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 } }));",
+	"  if (HOOK_EVENT_NAME === 'Stop') {",
+	"    console.log(JSON.stringify({ decision: 'block', reason: message }));",
 	"  } else {",
-	"    console.log(JSON.stringify({ additional_context: message }));",
+	"    console.log(JSON.stringify({ followup_message: message }));",
 	"  }",
 	"};",
 	"",
@@ -1005,7 +1004,10 @@ const installReactDoctorAgentHooksStep = (projectRoot, selectedAgents) => {
 		if (hookResult.installedAgents.length === 0) hookSpinner.succeed("No supported native agent hook targets selected.");
 		else {
 			hookSpinner.succeed(`React Doctor agent hooks installed for ${hookResult.installedAgents.map((agent) => getSkillAgentConfig(agent).displayName).join(", ")}.`);
-			recordCount(METRIC.installAgentHooks, 1, { agentsCount: hookResult.installedAgents.length });
+			recordCount(METRIC.installAgentHooks, 1, {
+				agentsCount: hookResult.installedAgents.length,
+				hookEvent: "stop"
+			});
 		}
 	} catch (error) {
 		hookSpinner.fail("Failed to install React Doctor agent hooks.");
@@ -1148,7 +1150,7 @@ const runInstallReactDoctor = async (options = {}) => {
 	for (const agent of selectedAgents) recordCount(METRIC.installAgent, 1, { agent });
 };
 //#endregion
-export { askUpgradeActionVersion as a, recordCiPromptDecision as c, installReactDoctorAgentHooks as i, hasHandledActionUpgrade as l, runInstallReactDoctor as n, askAddToGitHubActions as o, findAgentsWithLegacyShellHooks as r, hasHandledCiPrompt as s, getSkillSourceDirectory as t, recordActionUpgradeDecision as u };
+export { askUpgradeActionVersion as a, recordCiPromptDecision as c, installReactDoctorAgentHooks as i, hasHandledActionUpgrade as l, runInstallReactDoctor as n, askAddToGitHubActions as o, findAgentsWithOutdatedReactDoctorHooks as r, hasHandledCiPrompt as s, getSkillSourceDirectory as t, recordActionUpgradeDecision as u };
 
-//# sourceMappingURL=install-react-doctor-Bq7dfEBI.js.map
-//# debugId=d678e544-bf9c-5a23-bf1c-c11caae57e45
+//# sourceMappingURL=install-react-doctor-0EPCiZSM.js.map
+//# debugId=d43042c4-c766-54b9-a30f-944a55426467