react-doctor 0.0.24 → 0.0.25

package/README.md

@@ -38,15 +38,15 @@ Use `--verbose` to see affected files and line numbers:
 npx -y react-doctor@latest . --verbose
 ```
 
-## Install as a skill
+## Install for your coding agent
 
-Add React Doctor's rules as a [skill](https://skills.sh) for your coding agent:
+Teach your coding agent all 47+ React best practice rules:
 
 ```bash
-npx skills add millionco/react-doctor
+curl -fsSL https://react.doctor/install-skill.sh | bash
 ```
 
-This gives agents like Cursor, Claude Code, Copilot, and others access to all 47+ React best practice rules. The CLI will also prompt to install the skill on first run.
+Supports Cursor, Claude Code, Amp Code, Codex, Gemini CLI, OpenCode, Windsurf, and Antigravity.
 
 ## Options
 
@@ -64,7 +64,6 @@ Options:
   --diff [base]     scan only files changed vs base branch
   --no-ami          skip Ami-related prompts
   --fix             open Ami to auto-fix all issues
-  --prompt          copy latest scan output to clipboard
   -h, --help        display help for command
 ```
 

package/dist/cli.js

@@ -1,8 +1,8 @@
 #!/usr/bin/env node
 import { createRequire } from "node:module";
 import { execSync, spawn, spawnSync } from "node:child_process";
-import fs, { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
-import os, { homedir, tmpdir } from "node:os";
+import fs, { existsSync, mkdirSync, writeFileSync } from "node:fs";
+import os, { tmpdir } from "node:os";
 import path, { join } from "node:path";
 import { Command } from "commander";
 import { randomUUID } from "node:crypto";
@@ -23,7 +23,6 @@ const PERFECT_SCORE = 100;
 const SCORE_GOOD_THRESHOLD = 75;
 const SCORE_OK_THRESHOLD = 50;
 const SCORE_BAR_WIDTH_CHARS = 50;
-const SEPARATOR_LENGTH_CHARS = 40;
 const SUMMARY_BOX_HORIZONTAL_PADDING_CHARS = 1;
 const SUMMARY_BOX_OUTER_INDENT_CHARS = 2;
 const SCORE_API_URL = "https://www.react.doctor/api/score";
@@ -41,11 +40,6 @@ const ERROR_RULE_PENALTY = 1.5;
 const WARNING_RULE_PENALTY = .75;
 const ERROR_ESTIMATED_FIX_RATE = .85;
 const WARNING_ESTIMATED_FIX_RATE = .8;
-const buildDiagnosticPayload = (diagnostics) => diagnostics.map((diagnostic) => ({
-	plugin: diagnostic.plugin,
-	rule: diagnostic.rule,
-	severity: diagnostic.severity
-}));
 const getScoreLabel = (score) => {
 	if (score >= SCORE_GOOD_THRESHOLD) return "Great";
 	if (score >= SCORE_OK_THRESHOLD) return "Needs work";
@@ -84,7 +78,7 @@ const calculateScore = async (diagnostics) => {
 		const response = await fetch(SCORE_API_URL, {
 			method: "POST",
 			headers: { "Content-Type": "application/json" },
-			body: JSON.stringify({ diagnostics: buildDiagnosticPayload(diagnostics) })
+			body: JSON.stringify({ diagnostics })
 		});
 		if (!response.ok) return null;
 		return await response.json();
@@ -97,7 +91,7 @@ const fetchEstimatedScore = async (diagnostics) => {
 		const response = await fetch(ESTIMATE_SCORE_API_URL, {
 			method: "POST",
 			headers: { "Content-Type": "application/json" },
-			body: JSON.stringify({ diagnostics: buildDiagnosticPayload(diagnostics) })
+			body: JSON.stringify({ diagnostics })
 		});
 		if (!response.ok) return estimateScoreLocally(diagnostics);
 		return await response.json();
@@ -468,57 +462,29 @@ const highlighter = {
 	dim: pc.dim
 };
 
-//#endregion
-//#region src/utils/strip-ansi.ts
-const ANSI_ESCAPE_SEQUENCE = String.raw`\u001B\[[0-9;]*m`;
-const ANSI_ESCAPE_PATTERN = new RegExp(ANSI_ESCAPE_SEQUENCE, "g");
-const stripAnsi = (text) => text.replace(ANSI_ESCAPE_PATTERN, "");
-
 //#endregion
 //#region src/utils/logger.ts
-const loggerCaptureState = {
-	isEnabled: false,
-	lines: []
-};
-const captureLogLine = (text) => {
-	if (!loggerCaptureState.isEnabled) return;
-	loggerCaptureState.lines.push(stripAnsi(text));
-};
-const writeLogLine = (text) => {
-	console.log(text);
-	captureLogLine(text);
-};
-const startLoggerCapture = () => {
-	loggerCaptureState.isEnabled = true;
-	loggerCaptureState.lines = [];
-};
-const stopLoggerCapture = () => {
-	const capturedOutput = loggerCaptureState.lines.join("\n");
-	loggerCaptureState.isEnabled = false;
-	loggerCaptureState.lines = [];
-	return capturedOutput;
-};
 const logger = {
 	error(...args) {
-		writeLogLine(highlighter.error(args.join(" ")));
+		console.log(highlighter.error(args.join(" ")));
 	},
 	warn(...args) {
-		writeLogLine(highlighter.warn(args.join(" ")));
+		console.log(highlighter.warn(args.join(" ")));
 	},
 	info(...args) {
-		writeLogLine(highlighter.info(args.join(" ")));
+		console.log(highlighter.info(args.join(" ")));
 	},
 	success(...args) {
-		writeLogLine(highlighter.success(args.join(" ")));
+		console.log(highlighter.success(args.join(" ")));
 	},
 	dim(...args) {
-		writeLogLine(highlighter.dim(args.join(" ")));
+		console.log(highlighter.dim(args.join(" ")));
 	},
 	log(...args) {
-		writeLogLine(args.join(" "));
+		console.log(args.join(" "));
 	},
 	break() {
-		writeLogLine("");
+		console.log("");
 	}
 };
 
@@ -1435,46 +1401,6 @@ const scan = async (directory, inputOptions = {}) => {
 	};
 };
 
-//#endregion
-//#region src/utils/copy-to-clipboard.ts
-const getClipboardCommands = () => {
-	if (process.platform === "darwin") return [{
-		command: "pbcopy",
-		args: []
-	}];
-	if (process.platform === "win32") return [{
-		command: "clip",
-		args: []
-	}];
-	return [
-		{
-			command: "wl-copy",
-			args: []
-		},
-		{
-			command: "xclip",
-			args: ["-selection", "clipboard"]
-		},
-		{
-			command: "xsel",
-			args: ["--clipboard", "--input"]
-		}
-	];
-};
-const copyToClipboard = (text) => {
-	const clipboardCommands = getClipboardCommands();
-	for (const clipboardCommand of clipboardCommands) if (spawnSync(clipboardCommand.command, clipboardCommand.args, {
-		input: text,
-		stdio: [
-			"pipe",
-			"ignore",
-			"ignore"
-		],
-		encoding: "utf8"
-	}).status === 0) return true;
-	return false;
-};
-
 //#endregion
 //#region src/utils/get-diff-files.ts
 const getCurrentBranch = (directory) => {
@@ -1520,12 +1446,33 @@ const getChangedFilesSinceBranch = (directory, baseBranch) => {
 		return [];
 	}
 };
+const getUncommittedChangedFiles = (directory) => {
+	try {
+		const output = execSync("git diff --name-only --diff-filter=ACMR --relative HEAD", {
+			cwd: directory,
+			stdio: "pipe"
+		}).toString().trim();
+		if (!output) return [];
+		return output.split("\n").filter(Boolean);
+	} catch {
+		return [];
+	}
+};
 const getDiffInfo = (directory, explicitBaseBranch) => {
 	const currentBranch = getCurrentBranch(directory);
 	if (!currentBranch) return null;
 	const baseBranch = explicitBaseBranch ?? detectDefaultBranch(directory);
 	if (!baseBranch) return null;
-	if (currentBranch === baseBranch) return null;
+	if (currentBranch === baseBranch) {
+		const uncommittedFiles = getUncommittedChangedFiles(directory);
+		if (uncommittedFiles.length === 0) return null;
+		return {
+			currentBranch,
+			baseBranch,
+			changedFiles: uncommittedFiles,
+			isCurrentChanges: true
+		};
+	}
 	return {
 		currentBranch,
 		baseBranch,
@@ -1534,34 +1481,6 @@ const getDiffInfo = (directory, explicitBaseBranch) => {
 };
 const filterSourceFiles = (filePaths) => filePaths.filter((filePath) => SOURCE_FILE_PATTERN.test(filePath));
 
-//#endregion
-//#region src/utils/global-install.ts
-const isGloballyInstalled = () => {
-	try {
-		return !execSync("which react-doctor", {
-			stdio: "pipe",
-			encoding: "utf-8"
-		}).trim().includes("/_npx/");
-	} catch {
-		return false;
-	}
-};
-const maybeInstallGlobally = () => {
-	try {
-		if (isGloballyInstalled()) return;
-		const child = spawn("npm", [
-			"install",
-			"-g",
-			"react-doctor@latest"
-		], {
-			detached: true,
-			stdio: "ignore"
-		});
-		child.on("error", () => {});
-		child.unref();
-	} catch {}
-};
-
 //#endregion
 //#region src/utils/handle-error.ts
 const DEFAULT_HANDLE_ERROR_OPTIONS = { shouldExit: true };
@@ -1710,62 +1629,9 @@ const promptProjectSelection = async (workspacePackages, rootDirectory) => {
 	return selectedDirectories;
 };
 
-//#endregion
-//#region src/utils/skill-prompt.ts
-const CONFIG_DIRECTORY = join(homedir(), ".react-doctor");
-const CONFIG_FILE = join(CONFIG_DIRECTORY, "config.json");
-const SKILL_REPO = "millionco/react-doctor";
-const readConfig = () => {
-	try {
-		if (!existsSync(CONFIG_FILE)) return {};
-		return JSON.parse(readFileSync(CONFIG_FILE, "utf-8"));
-	} catch {
-		return {};
-	}
-};
-const writeConfig = (config) => {
-	try {
-		if (!existsSync(CONFIG_DIRECTORY)) mkdirSync(CONFIG_DIRECTORY, { recursive: true });
-		writeFileSync(CONFIG_FILE, JSON.stringify(config, null, 2));
-	} catch {}
-};
-const installSkill = () => {
-	try {
-		execSync(`npx -y skills add ${SKILL_REPO}`, { stdio: "inherit" });
-	} catch {
-		logger.break();
-		logger.dim("Skill install failed. You can install manually:");
-		logger.dim(`  npx skills add ${SKILL_REPO}`);
-	}
-};
-const maybePromptSkillInstall = async (shouldSkipPrompts) => {
-	const config = readConfig();
-	if (config.skillPromptDismissed) return;
-	if (shouldSkipPrompts) return;
-	logger.break();
-	logger.log(`${highlighter.info("💡")} Have your coding agent fix these issues automatically?`);
-	logger.dim(`   Install the ${highlighter.info("react-doctor")} skill to teach Cursor, Claude Code, Copilot,`);
-	logger.dim("   Ami, and other AI agents how to diagnose and fix these React issues.");
-	logger.break();
-	const { shouldInstall } = await prompts({
-		type: "confirm",
-		name: "shouldInstall",
-		message: "Install skill? (recommended)",
-		initial: true
-	});
-	if (shouldInstall) {
-		logger.break();
-		installSkill();
-		writeConfig({
-			...config,
-			skillPromptDismissed: true
-		});
-	}
-};
-
 //#endregion
 //#region src/cli.ts
-const VERSION = "0.0.24";
+const VERSION = "0.0.25";
 const exitWithFixHint = () => {
 	logger.break();
 	logger.log("Cancelled.");
@@ -1779,7 +1645,7 @@ const resolveDiffMode = async (diffInfo, effectiveDiff, shouldSkipPrompts, isSco
 	if (effectiveDiff !== void 0 && effectiveDiff !== false) {
 		if (diffInfo) return true;
 		if (!isScoreOnly) {
-			logger.warn("Not on a feature branch or could not determine base branch. Running full scan.");
+			logger.warn("No feature branch or uncommitted changes detected. Running full scan.");
 			logger.break();
 		}
 		return false;
@@ -1789,18 +1655,16 @@ const resolveDiffMode = async (diffInfo, effectiveDiff, shouldSkipPrompts, isSco
 	if (changedSourceFiles.length === 0) return false;
 	if (shouldSkipPrompts) return true;
 	if (isScoreOnly) return false;
-	const { shouldScanBranchOnly } = await prompts({
+	const { shouldScanChangedOnly } = await prompts({
 		type: "confirm",
-		name: "shouldScanBranchOnly",
-		message: `On branch ${diffInfo.currentBranch} (${changedSourceFiles.length} changed files vs ${diffInfo.baseBranch}). Only scan this branch?`,
+		name: "shouldScanChangedOnly",
+		message: diffInfo.isCurrentChanges ? `Found ${changedSourceFiles.length} uncommitted changed files. Only scan current changes?` : `On branch ${diffInfo.currentBranch} (${changedSourceFiles.length} changed files vs ${diffInfo.baseBranch}). Only scan this branch?`,
 		initial: true
 	});
-	return Boolean(shouldScanBranchOnly);
+	return Boolean(shouldScanChangedOnly);
 };
-const program = new Command().name("react-doctor").description("Diagnose React codebase health").version(VERSION, "-v, --version", "display the version number").argument("[directory]", "project directory to scan", ".").option("--no-lint", "skip linting").option("--no-dead-code", "skip dead code detection").option("--verbose", "show file details per rule").option("--score", "output only the score").option("-y, --yes", "skip prompts, scan all workspace projects").option("--project <name>", "select workspace project (comma-separated for multiple)").option("--diff [base]", "scan only files changed vs base branch").option("--offline", "skip telemetry (anonymous, not stored, only used to calculate score)").option("--no-ami", "skip Ami-related prompts").option("--fix", "open Ami to auto-fix all issues").option("--prompt", "copy latest scan output to clipboard").action(async (directory, flags) => {
-	const isScoreOnly = flags.score && !flags.prompt;
-	const shouldCopyPromptOutput = flags.prompt;
-	startLoggerCapture();
+const program = new Command().name("react-doctor").description("Diagnose React codebase health").version(VERSION, "-v, --version", "display the version number").argument("[directory]", "project directory to scan", ".").option("--no-lint", "skip linting").option("--no-dead-code", "skip dead code detection").option("--verbose", "show file details per rule").option("--score", "output only the score").option("-y, --yes", "skip prompts, scan all workspace projects").option("--project <name>", "select workspace project (comma-separated for multiple)").option("--diff [base]", "scan only files changed vs base branch").option("--offline", "skip telemetry (anonymous, not stored, only used to calculate score)").option("--no-ami", "skip Ami-related prompts").option("--fix", "open Ami to auto-fix all issues").action(async (directory, flags) => {
+	const isScoreOnly = flags.score;
 	try {
 		const resolvedDirectory = path.resolve(directory);
 		const userConfig = loadConfig(resolvedDirectory);
@@ -1812,7 +1676,7 @@ const program = new Command().name("react-doctor").description("Diagnose React c
 		const scanOptions = {
 			lint: isCliOverride("lint") ? flags.lint : userConfig?.lint ?? flags.lint,
 			deadCode: isCliOverride("deadCode") ? flags.deadCode : userConfig?.deadCode ?? flags.deadCode,
-			verbose: flags.prompt || (isCliOverride("verbose") ? Boolean(flags.verbose) : userConfig?.verbose ?? false),
+			verbose: isCliOverride("verbose") ? Boolean(flags.verbose) : userConfig?.verbose ?? false,
 			scoreOnly: isScoreOnly,
 			offline: flags.offline
 		};
@@ -1833,7 +1697,8 @@ const program = new Command().name("react-doctor").description("Diagnose React c
 		const diffInfo = getDiffInfo(resolvedDirectory, explicitBaseBranch);
 		const isDiffMode = await resolveDiffMode(diffInfo, effectiveDiff, shouldSkipPrompts, isScoreOnly);
 		if (isDiffMode && diffInfo && !isScoreOnly) {
-			logger.log(`Scanning changes: ${highlighter.info(diffInfo.currentBranch)} → ${highlighter.info(diffInfo.baseBranch)}`);
+			if (diffInfo.isCurrentChanges) logger.log("Scanning uncommitted changes");
+			else logger.log(`Scanning changes: ${highlighter.info(diffInfo.currentBranch)} → ${highlighter.info(diffInfo.baseBranch)}`);
 			logger.break();
 		}
 		const allDiagnostics = [];
@@ -1864,18 +1729,10 @@ const program = new Command().name("react-doctor").description("Diagnose React c
 			allDiagnostics.push(...scanResult.diagnostics);
 			if (!isScoreOnly) logger.break();
 		}
-		const capturedScanOutput = stopLoggerCapture();
 		if (flags.fix) openAmiToFix(resolvedDirectory);
-		if (shouldCopyPromptOutput) copyPromptToClipboard(capturedScanOutput, !isScoreOnly);
-		else if (!isScoreOnly) {
-			await maybePromptSkillInstall(shouldSkipAmiPrompts);
-			if (!shouldSkipAmiPrompts && !flags.fix) await maybePromptFix(resolvedDirectory, allDiagnostics, flags.offline ? null : await fetchEstimatedScore(allDiagnostics), capturedScanOutput);
-		}
+		if (!isScoreOnly && !shouldSkipAmiPrompts && !flags.fix) await maybePromptFix(resolvedDirectory, allDiagnostics, flags.offline ? null : await fetchEstimatedScore(allDiagnostics));
 	} catch (error) {
-		handleError(error, { shouldExit: !shouldCopyPromptOutput });
-	} finally {
-		const remainingOutput = stopLoggerCapture();
-		if (shouldCopyPromptOutput && remainingOutput) copyPromptToClipboard(remainingOutput, !isScoreOnly);
+		handleError(error);
 	}
 }).addHelpText("after", `
 ${highlighter.dim("Learn more:")}
@@ -1890,9 +1747,6 @@ const colorizeByScore = (text, score) => {
 	return highlighter.error(text);
 };
 const DEEPLINK_FIX_PROMPT = "Run `npx -y react-doctor@latest .` to diagnose issues, then fix all reported issues one by one. After applying fixes, run it again to verify the results improved.";
-const CLIPBOARD_FIX_PROMPT = "Fix all issues reported in the react-doctor diagnostics below, one by one. After applying fixes, run `npx -y react-doctor@latest .` again to verify the results improved.";
-const REACT_DOCTOR_OUTPUT_LABEL = "react-doctor output";
-const SCAN_SUMMARY_SEPARATOR = "─".repeat(SEPARATOR_LENGTH_CHARS);
 const isAmiInstalled = () => {
 	if (process.platform === "darwin") return existsSync("/Applications/Ami.app") || existsSync(path.join(os.homedir(), "Applications", "Ami.app"));
 	if (process.platform === "win32") {
@@ -1961,24 +1815,7 @@ const openAmiToFix = (directory) => {
 		logger.info(webDeeplink);
 	}
 };
-const buildPromptWithOutput = (reactDoctorOutput) => {
-	const summaryStartIndex = reactDoctorOutput.indexOf(SCAN_SUMMARY_SEPARATOR);
-	const normalizedReactDoctorOutput = (summaryStartIndex === -1 ? reactDoctorOutput : reactDoctorOutput.slice(0, summaryStartIndex).trimEnd()).trim();
-	return `${CLIPBOARD_FIX_PROMPT}\n\n${REACT_DOCTOR_OUTPUT_LABEL}:\n\`\`\`\n${normalizedReactDoctorOutput.length > 0 ? normalizedReactDoctorOutput : "No output captured."}\n\`\`\``;
-};
-const copyPromptToClipboard = (reactDoctorOutput, shouldLogResult) => {
-	const promptWithOutput = buildPromptWithOutput(reactDoctorOutput);
-	const didCopyPromptToClipboard = copyToClipboard(promptWithOutput);
-	if (!shouldLogResult) return;
-	if (didCopyPromptToClipboard) {
-		logger.success("Copied latest scan output to clipboard");
-		return;
-	}
-	logger.warn("Could not copy prompt to clipboard automatically. Use this prompt:");
-	logger.info(promptWithOutput);
-};
 const FIX_METHOD_AMI = "ami";
-const FIX_METHOD_CLIPBOARD = "clipboard";
 const FIX_COMMAND_HINT = "npx react-doctor@latest --fix";
 const buildAmiBanner = (issueCount, currentScore, estimatedScore) => {
 	const currentScoreDisplay = colorizeByScore(String(currentScore), currentScore);
@@ -1995,13 +1832,6 @@ const buildAmiBanner = (issueCount, currentScore, estimatedScore) => {
 		createFramedLine(`Free to use. ${AMI_WEBSITE_URL}`, `Free to use. ${highlighter.info(AMI_WEBSITE_URL)}`)
 	]);
 };
-const buildClipboardWarningBanner = () => renderFramedBoxString([
-	createFramedLine("⚠ Other agents may not fix these issues well.", `${highlighter.warn("⚠")} Other agents may not fix these issues well.`),
-	createFramedLine(""),
-	createFramedLine("react-doctor diagnostics require React-specific context"),
-	createFramedLine("that general-purpose agents often miss, leading to"),
-	createFramedLine("incomplete or incorrect fixes.")
-]);
 const buildSkipBanner = (issueCount, estimatedScore) => {
 	const issueLabel = issueCount === 1 ? "issue" : "issues";
 	const estimatedScoreDisplay = colorizeByScore(`~${estimatedScore}`, estimatedScore);
@@ -2014,10 +1844,9 @@ const buildSkipBanner = (issueCount, estimatedScore) => {
 const configureFixBanners = (issueCount, estimatedScoreResult) => {
 	const { currentScore, estimatedScore } = estimatedScoreResult;
 	setSelectBanner(buildAmiBanner(issueCount, currentScore, estimatedScore), 0);
-	setSelectBanner(buildClipboardWarningBanner(), 1);
-	setSelectBanner(buildSkipBanner(issueCount, estimatedScore), 2);
+	setSelectBanner(buildSkipBanner(issueCount, estimatedScore), 1);
 };
-const maybePromptFix = async (directory, diagnostics, estimatedScoreResult, capturedScanOutput) => {
+const maybePromptFix = async (directory, diagnostics, estimatedScoreResult) => {
 	if (diagnostics.length === 0) return;
 	logger.break();
 	if (estimatedScoreResult) configureFixBanners(diagnostics.length, estimatedScoreResult);
@@ -2025,26 +1854,17 @@ const maybePromptFix = async (directory, diagnostics, estimatedScoreResult, capt
 		type: "select",
 		name: "fixMethod",
 		message: "Fix issues?",
-		choices: [
-			{
-				title: "Use ami.dev (recommended)",
-				description: "Optimized coding agent for React Doctor",
-				value: FIX_METHOD_AMI
-			},
-			{
-				title: "Copy to clipboard",
-				description: "Other agents may lack context for react-doctor fixes",
-				value: FIX_METHOD_CLIPBOARD
-			},
-			{
-				title: "Skip",
-				value: "skip"
-			}
-		]
+		choices: [{
+			title: "Use ami.dev (recommended)",
+			description: "Optimized coding agent for React Doctor",
+			value: FIX_METHOD_AMI
+		}, {
+			title: "Skip",
+			value: "skip"
+		}]
 	});
 	clearSelectBanner();
 	if (fixMethod === FIX_METHOD_AMI) openAmiToFix(directory);
-	else if (fixMethod === FIX_METHOD_CLIPBOARD) copyPromptToClipboard(capturedScanOutput, true);
 	else {
 		logger.break();
 		logger.dim(`  Run ${highlighter.info(FIX_COMMAND_HINT)} anytime to fix issues.`);
@@ -2062,7 +1882,6 @@ const installAmiCommand = new Command("install-ami").description("Install Ami an
 program.addCommand(fixCommand);
 program.addCommand(installAmiCommand);
 const main$1 = async () => {
-	maybeInstallGlobally();
 	await program.parseAsync();
 };
 main$1();