react-doctor 0.0.6 → 0.0.8

package/dist/cli.js

@@ -5,8 +5,8 @@ import path, { join } from "node:path";
 import { Command } from "commander";
 import pc from "picocolors";
 import { randomUUID } from "node:crypto";
-import fs, { mkdirSync, writeFileSync } from "node:fs";
-import os, { tmpdir } from "node:os";
+import fs, { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import os, { homedir, tmpdir } from "node:os";
 import { performance } from "node:perf_hooks";
 import { main } from "knip";
 import { createOptions } from "knip/session";
@@ -73,6 +73,7 @@ const SCORE_GOOD_THRESHOLD = 75;
 const SCORE_OK_THRESHOLD = 50;
 const SCORE_BAR_WIDTH_CHARS = 50;
 const SCORE_API_URL = "https://www.react.doctor/api/score";
+const SHARE_BASE_URL = "https://www.react.doctor/share";
 const OFFLINE_MESSAGE = "You are offline, could not calculate score. Reconnect to calculate.";
 
 //#endregion
@@ -493,7 +494,8 @@ const NEXTJS_RULES = {
 	"react-doctor/nextjs-no-font-link": "warn",
 	"react-doctor/nextjs-no-css-link": "warn",
 	"react-doctor/nextjs-no-polyfill-script": "warn",
-	"react-doctor/nextjs-no-head-import": "error"
+	"react-doctor/nextjs-no-head-import": "error",
+	"react-doctor/nextjs-no-side-effect-in-get-handler": "error"
 };
 const REACT_COMPILER_RULES = {
 	"react-hooks-js/set-state-in-render": "error",
@@ -670,6 +672,7 @@ const RULE_CATEGORY_MAP = {
 	"react-doctor/nextjs-no-css-link": "Next.js",
 	"react-doctor/nextjs-no-polyfill-script": "Next.js",
 	"react-doctor/nextjs-no-head-import": "Next.js",
+	"react-doctor/nextjs-no-side-effect-in-get-handler": "Security",
 	"react-doctor/server-auth-actions": "Server",
 	"react-doctor/server-after-nonblocking": "Server",
 	"react-doctor/client-passive-event-listeners": "Performance",
@@ -725,6 +728,7 @@ const RULE_HELP_MAP = {
 	"nextjs-no-css-link": "Import CSS directly: `import './styles.css'` or use CSS Modules: `import styles from './Button.module.css'`",
 	"nextjs-no-polyfill-script": "Next.js includes polyfills for fetch, Promise, Object.assign, Array.from, and 50+ others automatically",
 	"nextjs-no-head-import": "Use the Metadata API instead: `export const metadata = { title: '...' }` or `export async function generateMetadata()`",
+	"nextjs-no-side-effect-in-get-handler": "Move the side effect to a POST handler and use a <form> or fetch with method POST — GET requests can be triggered by prefetching and are vulnerable to CSRF",
 	"server-auth-actions": "Add `const session = await auth()` at the top and throw/redirect if unauthorized before any data access",
 	"server-after-nonblocking": "`import { after } from 'next/server'` then wrap: `after(() => analytics.track(...))` — response isn't blocked",
 	"client-passive-event-listeners": "Add `{ passive: true }` as the third argument: `addEventListener('scroll', handler, { passive: true })`",
@@ -971,6 +975,17 @@ const printBranding = (score) => {
 	logger.log(`  React Doctor ${highlighter.dim("(www.react.doctor)")}`);
 	logger.break();
 };
+const buildShareUrl = (diagnostics, scoreResult) => {
+	const errorCount = diagnostics.filter((diagnostic) => diagnostic.severity === "error").length;
+	const warningCount = diagnostics.filter((diagnostic) => diagnostic.severity === "warning").length;
+	const affectedFileCount = collectAffectedFiles(diagnostics).size;
+	const params = new URLSearchParams();
+	if (scoreResult) params.set("s", String(scoreResult.score));
+	if (errorCount > 0) params.set("e", String(errorCount));
+	if (warningCount > 0) params.set("w", String(warningCount));
+	if (affectedFileCount > 0) params.set("f", String(affectedFileCount));
+	return `${SHARE_BASE_URL}?${params.toString()}`;
+};
 const printSummary = (diagnostics, elapsedMilliseconds, scoreResult) => {
 	const errorCount = diagnostics.filter((diagnostic) => diagnostic.severity === "error").length;
 	const warningCount = diagnostics.filter((diagnostic) => diagnostic.severity === "warning").length;
@@ -997,6 +1012,9 @@ const printSummary = (diagnostics, elapsedMilliseconds, scoreResult) => {
 	} catch {
 		logger.break();
 	}
+	const shareUrl = buildShareUrl(diagnostics, scoreResult);
+	logger.break();
+	logger.dim(`  Share your results: ${highlighter.info(shareUrl)}`);
 };
 const scan = async (directory, options) => {
 	const startTime = performance.now();
@@ -1125,9 +1143,68 @@ 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 = "aidenybai/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" });
+		return true;
+	} catch {
+		return false;
+	}
+};
+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?",
+		initial: true
+	});
+	if (shouldInstall) {
+		logger.break();
+		if (installSkill()) {
+			logger.break();
+			logger.success("Skill installed!");
+		} else {
+			logger.break();
+			logger.dim("Skill install failed. You can install manually:");
+			logger.dim(`  npx skills add ${SKILL_REPO}`);
+		}
+	}
+	writeConfig({
+		...config,
+		skillPromptDismissed: true
+	});
+};
+
 //#endregion
 //#region src/cli.ts
-const VERSION = "0.0.6";
+const VERSION = "0.0.8";
 process.on("SIGINT", () => process.exit(0));
 process.on("SIGTERM", () => process.exit(0));
 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("--fix", "open Ami to auto-fix all issues").action(async (directory, flags) => {
@@ -1144,7 +1221,17 @@ const program = new Command().name("react-doctor").description("Diagnose React c
 			verbose: Boolean(flags.verbose),
 			scoreOnly: isScoreOnly
 		};
-		const shouldSkipPrompts = flags.yes || Boolean(process.env.CI) || Boolean(process.env.CLAUDECODE) || Boolean(process.env.AMI) || !process.stdin.isTTY;
+		const isAutomatedEnvironment = [
+			process.env.CI,
+			process.env.CLAUDECODE,
+			process.env.CURSOR_TRACE_ID,
+			process.env.CURSOR_AGENT,
+			process.env.CODEX_CI,
+			process.env.OPENCODE,
+			process.env.AMP_HOME,
+			process.env.AMI
+		].some(Boolean);
+		const shouldSkipPrompts = flags.yes || isAutomatedEnvironment || !process.stdin.isTTY;
 		const projectDirectories = await selectProjects(resolvedDirectory, flags.project, shouldSkipPrompts);
 		for (const projectDirectory of projectDirectories) {
 			if (!isScoreOnly) {
@@ -1155,6 +1242,10 @@ const program = new Command().name("react-doctor").description("Diagnose React c
 			if (!isScoreOnly) logger.break();
 		}
 		if (flags.fix) openAmiToFix(resolvedDirectory);
+		if (!isScoreOnly) {
+			await maybePromptSkillInstall(shouldSkipPrompts);
+			if (!shouldSkipPrompts && !flags.fix) await maybePromptAmiFix(resolvedDirectory);
+		}
 	} catch (error) {
 		handleError(error);
 	}
@@ -1164,9 +1255,6 @@ ${highlighter.dim("Learn more:")}
 `);
 const AMI_INSTALL_URL = "https://ami.dev/install.sh";
 const AMI_FIX_PROMPT = "Run npx -y react-doctor@latest . --verbose, read every diagnostic, then fix all issues one by one. After fixing, re-run react-doctor to verify the score improved.";
-const buildAmiDeeplink = (projectDirectory) => {
-	return `ami://new-chat?cwd=${encodeURIComponent(projectDirectory)}&prompt=${encodeURIComponent(AMI_FIX_PROMPT)}&mode=agent`;
-};
 const isAmiInstalled = () => {
 	try {
 		execSync("ls /Applications/Ami.app", { stdio: "ignore" });
@@ -1190,7 +1278,7 @@ const openAmiToFix = (directory) => {
 	const resolvedDirectory = path.resolve(directory);
 	if (!isAmiInstalled()) installAmi();
 	logger.log("Opening Ami to fix react-doctor issues...");
-	const deeplink = buildAmiDeeplink(resolvedDirectory);
+	const deeplink = `ami://open-project?cwd=${encodeURIComponent(resolvedDirectory)}&prompt=${encodeURIComponent(AMI_FIX_PROMPT)}&mode=agent`;
 	try {
 		execSync(`open "${deeplink}"`, { stdio: "ignore" });
 		logger.success("Opened Ami with react-doctor fix prompt.");
@@ -1200,6 +1288,20 @@ const openAmiToFix = (directory) => {
 		logger.info(deeplink);
 	}
 };
+const maybePromptAmiFix = async (directory) => {
+	logger.break();
+	logger.log(`Fix these issues with ${highlighter.info("Ami")}?`);
+	logger.dim("   Ami is a coding agent built to understand your codebase and fix issues");
+	logger.dim(`   automatically. Learn more at ${highlighter.info("https://ami.dev")}`);
+	logger.break();
+	const { shouldFix } = await prompts({
+		type: "confirm",
+		name: "shouldFix",
+		message: "Open Ami to fix?",
+		initial: true
+	});
+	if (shouldFix) openAmiToFix(directory);
+};
 const fixAction = (directory) => {
 	try {
 		openAmiToFix(directory);
@@ -1208,7 +1310,7 @@ const fixAction = (directory) => {
 	}
 };
 const fixCommand = new Command("fix").description("Open Ami to auto-fix react-doctor issues").argument("[directory]", "project directory", ".").action(fixAction);
-const installAmiCommand = new Command("install-ami").description("Open Ami to auto-fix react-doctor issues").argument("[directory]", "project directory", ".").action(fixAction);
+const installAmiCommand = new Command("install-ami").description("Install Ami and open it to auto-fix issues").argument("[directory]", "project directory", ".").action(fixAction);
 program.addCommand(fixCommand);
 program.addCommand(installAmiCommand);
 const main$1 = async () => {