react-doctor 0.0.44 → 0.0.46

package/README.md

@@ -48,7 +48,7 @@ npx -y react-doctor@latest install
 
 You'll be prompted to pick which detected agents to install for. Pass `--yes` to skip prompts and install for every detected agent.
 
-Supports Claude Code, Codex, GitHub Copilot, Gemini CLI, Cursor, OpenCode, Factory Droid, and Pi.
+Supports 50+ coding agents via [`agent-install`](https://www.npmjs.com/package/agent-install), including Claude Code, Codex, Cursor, Factory Droid, Gemini CLI, GitHub Copilot, Goose, OpenCode, Pi, Windsurf, Roo Code, Cline, Kilo Code, Warp, Replit, OpenHands, Continue, and many more. Detection is the union of CLI binaries on `$PATH` and config dirs in `$HOME` (`~/.claude`, `~/.cursor`, `~/.codex`, `~/.factory`, `~/.pi`, etc.).
 
 ## GitHub Actions
 

package/dist/cli.js

@@ -1,10 +1,11 @@
 import { createRequire } from "node:module";
-import fs, { accessSync, constants, cpSync, existsSync, mkdirSync, mkdtempSync, readdirSync, rmSync, statSync, writeFileSync } from "node:fs";
+import fs, { accessSync, constants, existsSync, mkdirSync, mkdtempSync, readdirSync, statSync, writeFileSync } from "node:fs";
 import os, { tmpdir } from "node:os";
 import path, { join } from "node:path";
 import { performance } from "node:perf_hooks";
 import { Command } from "commander";
 import { fileURLToPath } from "node:url";
+import { SKILL_MANIFEST_FILE, detectInstalledSkillAgents, getSkillAgentConfig, getSkillAgentTypes, installSkillsFromSource } from "agent-install";
 import pc from "picocolors";
 import basePrompts from "prompts";
 import ora from "ora";
@@ -43,54 +44,21 @@ const IGNORED_DIRECTORIES = new Set([
 	"coverage"
 ]);
 const CANONICAL_GITHUB_URL = "https://github.com/millionco/react-doctor";
+const SKILL_NAME = "react-doctor";
 const PROXY_OUTPUT_MAX_BYTES = 50 * 1024 * 1024;
 const buildNoReactDependencyError = (directory) => `No React dependency found in ${directory}/package.json. Add "react" to dependencies (or peerDependencies) and re-run.`;
 //#endregion
 //#region src/utils/detect-agents.ts
-const AGENTS_SKILL_DIR = ".agents/skills";
-const SUPPORTED_AGENTS = {
-	claude: {
-		binaries: ["claude"],
-		displayName: "Claude Code",
-		skillDir: ".claude/skills"
-	},
-	codex: {
-		binaries: ["codex"],
-		displayName: "Codex",
-		skillDir: AGENTS_SKILL_DIR
-	},
-	copilot: {
-		binaries: ["copilot"],
-		displayName: "GitHub Copilot",
-		skillDir: AGENTS_SKILL_DIR
-	},
-	gemini: {
-		binaries: ["gemini"],
-		displayName: "Gemini CLI",
-		skillDir: AGENTS_SKILL_DIR
-	},
-	cursor: {
-		binaries: ["cursor", "agent"],
-		displayName: "Cursor",
-		skillDir: AGENTS_SKILL_DIR
-	},
-	opencode: {
-		binaries: ["opencode"],
-		displayName: "OpenCode",
-		skillDir: AGENTS_SKILL_DIR
-	},
-	droid: {
-		binaries: ["droid"],
-		displayName: "Factory Droid",
-		skillDir: ".factory/skills"
-	},
-	pi: {
-		binaries: ["pi", "omegon"],
-		displayName: "Pi",
-		skillDir: AGENTS_SKILL_DIR
-	}
+const PATH_BINARIES = {
+	"claude-code": ["claude"],
+	codex: ["codex"],
+	cursor: ["cursor", "agent"],
+	droid: ["droid"],
+	"gemini-cli": ["gemini"],
+	"github-copilot": ["copilot"],
+	opencode: ["opencode"],
+	pi: ["pi", "omegon"]
 };
-const ALL_SUPPORTED_AGENTS = Object.keys(SUPPORTED_AGENTS);
 const isCommandAvailable = (command) => {
 	const pathDirectories = (process.env.PATH ?? "").split(path.delimiter).filter(Boolean);
 	for (const directory of pathDirectories) {
@@ -104,9 +72,15 @@ const isCommandAvailable = (command) => {
 	}
 	return false;
 };
-const detectAvailableAgents = () => ALL_SUPPORTED_AGENTS.filter((agent) => SUPPORTED_AGENTS[agent].binaries.some(isCommandAvailable));
-const toDisplayName = (agent) => SUPPORTED_AGENTS[agent].displayName;
-const toSkillDir = (agent) => SUPPORTED_AGENTS[agent].skillDir;
+const detectPathAvailableAgents = () => {
+	const detected = [];
+	for (const [agent, binaries] of Object.entries(PATH_BINARIES)) if (binaries.some(isCommandAvailable)) detected.push(agent);
+	return detected;
+};
+const detectAvailableAgents = async () => {
+	const detected = new Set([...detectPathAvailableAgents(), ...await detectInstalledSkillAgents()]);
+	return getSkillAgentTypes().filter((agent) => agent !== "universal" && detected.has(agent));
+};
 //#endregion
 //#region src/utils/highlighter.ts
 const highlighter = {
@@ -117,19 +91,6 @@ const highlighter = {
 	dim: pc.dim
 };
 //#endregion
-//#region src/utils/install-skill-for-agent.ts
-const installSkillForAgent = (projectRoot, agent, skillSourceDirectory, skillName, alreadyInstalledDirectories) => {
-	const installedSkillDirectory = path.join(projectRoot, toSkillDir(agent), skillName);
-	if (alreadyInstalledDirectories?.has(installedSkillDirectory)) return installedSkillDirectory;
-	rmSync(installedSkillDirectory, {
-		recursive: true,
-		force: true
-	});
-	mkdirSync(path.dirname(installedSkillDirectory), { recursive: true });
-	cpSync(skillSourceDirectory, installedSkillDirectory, { recursive: true });
-	return installedSkillDirectory;
-};
-//#endregion
 //#region src/utils/logger.ts
 let isSilent$1 = false;
 const setLoggerSilent = (silent) => {
@@ -274,31 +235,34 @@ const spinner = (text) => ({ start() {
 	return handle;
 } });
 //#endregion
+//#region src/utils/to-display-name.ts
+const toDisplayName = (agent) => getSkillAgentConfig(agent).displayName;
+//#endregion
 //#region src/install-skill.ts
-const SKILL_NAME = "react-doctor";
 const getSkillSourceDirectory = () => {
 	const distDirectory = path.dirname(fileURLToPath(import.meta.url));
 	return path.join(distDirectory, "skills", SKILL_NAME);
 };
 const runInstallSkill = async (options = {}) => {
-	const projectRoot = process.cwd();
-	const sourceDir = getSkillSourceDirectory();
-	if (!existsSync(path.join(sourceDir, "SKILL.md"))) {
+	const projectRoot = options.projectRoot ?? process.cwd();
+	const sourceDir = options.sourceDir ?? getSkillSourceDirectory();
+	if (!existsSync(path.join(sourceDir, SKILL_MANIFEST_FILE))) {
 		logger.error(`Could not locate the ${SKILL_NAME} skill bundled with this package.`);
 		process.exitCode = 1;
 		return;
 	}
-	const detectedAgents = detectAvailableAgents();
+	const detectedAgents = options.detectedAgents ?? await detectAvailableAgents();
 	if (detectedAgents.length === 0) {
-		logger.error("No supported coding agents detected on your PATH.");
-		logger.dim("  Supported: Claude Code, Codex, GitHub Copilot, Gemini CLI, Cursor, OpenCode, Factory Droid, Pi.");
+		logger.error("No supported coding agents detected.");
+		logger.dim("  Looked for binaries on PATH (claude, codex, cursor, droid, gemini, copilot, opencode, pi)");
+		logger.dim("  and config dirs in $HOME (~/.claude, ~/.cursor, ~/.codex, ~/.gemini, ...).");
 		process.exitCode = 1;
 		return;
 	}
 	const selectedAgents = Boolean(options.yes) || !process.stdin.isTTY ? detectedAgents : (await prompts({
 		type: "multiselect",
 		name: "agents",
-		message: `Install the ${highlighter.info(SKILL_NAME)} skill for:`,
+		message: `Install the ${highlighter.info("react-doctor")} skill for:`,
 		choices: detectedAgents.map((agent) => ({
 			title: toDisplayName(agent),
 			value: agent,
@@ -315,12 +279,20 @@ const runInstallSkill = async (options = {}) => {
 		return;
 	}
 	const installSpinner = spinner(`Installing ${SKILL_NAME} skill...`).start();
-	const installedDirectories = /* @__PURE__ */ new Set();
-	for (const agent of selectedAgents) {
-		const installedDirectory = installSkillForAgent(projectRoot, agent, sourceDir, SKILL_NAME, installedDirectories);
-		installedDirectories.add(installedDirectory);
+	try {
+		const installResult = await installSkillsFromSource({
+			source: sourceDir,
+			agents: selectedAgents,
+			cwd: projectRoot,
+			mode: "copy"
+		});
+		if (installResult.skills.length === 0) throw new Error(`Could not parse ${SKILL_MANIFEST_FILE} for ${SKILL_NAME} (missing or invalid frontmatter).`);
+		if (installResult.failed.length > 0) throw new Error(installResult.failed.map((failure) => `${toDisplayName(failure.agent)}: ${failure.error}`).join("\n"));
+		installSpinner.succeed(`${SKILL_NAME} skill installed for ${selectedAgents.map(toDisplayName).join(", ")}.`);
+	} catch (error) {
+		installSpinner.fail(`Failed to install ${SKILL_NAME} skill.`);
+		throw error;
 	}
-	installSpinner.succeed(`${SKILL_NAME} skill installed for ${selectedAgents.map(toDisplayName).join(", ")}.`);
 };
 //#endregion
 //#region src/core/calculate-score-locally.ts
@@ -1789,14 +1761,14 @@ const REACT_COMPILER_RULES = {
 	"react-hooks-js/todo": "warn"
 };
 const resolveReactHooksJsPlugin = (hasReactCompiler, customRulesOnly) => {
-	if (!hasReactCompiler || customRulesOnly) return [];
+	if (!hasReactCompiler || customRulesOnly) return null;
 	try {
-		return [{
+		return {
 			name: "react-hooks-js",
 			specifier: esmRequire$1.resolve("eslint-plugin-react-hooks")
-		}];
+		};
 	} catch {
-		return [];
+		return null;
 	}
 };
 const TANSTACK_QUERY_RULES = {
@@ -1943,29 +1915,32 @@ const ALL_REACT_DOCTOR_RULE_KEYS = new Set([
 	...Object.keys(TANSTACK_START_RULES),
 	...Object.keys(TANSTACK_QUERY_RULES)
 ]);
-const createOxlintConfig = ({ pluginPath, framework, hasReactCompiler, hasTanStackQuery, customRulesOnly = false }) => ({
-	categories: {
-		correctness: "off",
-		suspicious: "off",
-		pedantic: "off",
-		perf: "off",
-		restriction: "off",
-		style: "off",
-		nursery: "off"
-	},
-	plugins: customRulesOnly ? [] : ["react", "jsx-a11y"],
-	jsPlugins: [...resolveReactHooksJsPlugin(hasReactCompiler, customRulesOnly), pluginPath],
-	rules: {
-		...customRulesOnly ? {} : BUILTIN_REACT_RULES,
-		...customRulesOnly ? {} : BUILTIN_A11Y_RULES,
-		...hasReactCompiler && !customRulesOnly ? REACT_COMPILER_RULES : {},
-		...GLOBAL_REACT_DOCTOR_RULES,
-		...framework === "nextjs" ? NEXTJS_RULES : {},
-		...framework === "expo" || framework === "react-native" ? REACT_NATIVE_RULES : {},
-		...framework === "tanstack-start" ? TANSTACK_START_RULES : {},
-		...hasTanStackQuery ? TANSTACK_QUERY_RULES : {}
-	}
-});
+const createOxlintConfig = ({ pluginPath, framework, hasReactCompiler, hasTanStackQuery, customRulesOnly = false }) => {
+	const reactHooksJsPlugin = resolveReactHooksJsPlugin(hasReactCompiler, customRulesOnly);
+	return {
+		categories: {
+			correctness: "off",
+			suspicious: "off",
+			pedantic: "off",
+			perf: "off",
+			restriction: "off",
+			style: "off",
+			nursery: "off"
+		},
+		plugins: customRulesOnly ? [] : ["react", "jsx-a11y"],
+		jsPlugins: reactHooksJsPlugin ? [reactHooksJsPlugin, pluginPath] : [pluginPath],
+		rules: {
+			...customRulesOnly ? {} : BUILTIN_REACT_RULES,
+			...customRulesOnly ? {} : BUILTIN_A11Y_RULES,
+			...reactHooksJsPlugin ? REACT_COMPILER_RULES : {},
+			...GLOBAL_REACT_DOCTOR_RULES,
+			...framework === "nextjs" ? NEXTJS_RULES : {},
+			...framework === "expo" || framework === "react-native" ? REACT_NATIVE_RULES : {},
+			...framework === "tanstack-start" ? TANSTACK_START_RULES : {},
+			...hasTanStackQuery ? TANSTACK_QUERY_RULES : {}
+		}
+	};
+};
 //#endregion
 //#region src/utils/neutralize-disable-directives.ts
 const DISABLE_DIRECTIVE_PATTERN = /(eslint|oxlint)-disable/;
@@ -3358,7 +3333,7 @@ const promptProjectSelection = async (workspacePackages, rootDirectory) => {
 };
 //#endregion
 //#region src/cli.ts
-const VERSION = "0.0.44";
+const VERSION = "0.0.46";
 const VALID_FAIL_ON_LEVELS = new Set([
 	"error",
 	"warning",

package/dist/index.js

@@ -1513,14 +1513,14 @@ const REACT_COMPILER_RULES = {
 	"react-hooks-js/todo": "warn"
 };
 const resolveReactHooksJsPlugin = (hasReactCompiler, customRulesOnly) => {
-	if (!hasReactCompiler || customRulesOnly) return [];
+	if (!hasReactCompiler || customRulesOnly) return null;
 	try {
-		return [{
+		return {
 			name: "react-hooks-js",
 			specifier: esmRequire$1.resolve("eslint-plugin-react-hooks")
-		}];
+		};
 	} catch {
-		return [];
+		return null;
 	}
 };
 const TANSTACK_QUERY_RULES = {
@@ -1667,29 +1667,32 @@ const ALL_REACT_DOCTOR_RULE_KEYS = new Set([
 	...Object.keys(TANSTACK_START_RULES),
 	...Object.keys(TANSTACK_QUERY_RULES)
 ]);
-const createOxlintConfig = ({ pluginPath, framework, hasReactCompiler, hasTanStackQuery, customRulesOnly = false }) => ({
-	categories: {
-		correctness: "off",
-		suspicious: "off",
-		pedantic: "off",
-		perf: "off",
-		restriction: "off",
-		style: "off",
-		nursery: "off"
-	},
-	plugins: customRulesOnly ? [] : ["react", "jsx-a11y"],
-	jsPlugins: [...resolveReactHooksJsPlugin(hasReactCompiler, customRulesOnly), pluginPath],
-	rules: {
-		...customRulesOnly ? {} : BUILTIN_REACT_RULES,
-		...customRulesOnly ? {} : BUILTIN_A11Y_RULES,
-		...hasReactCompiler && !customRulesOnly ? REACT_COMPILER_RULES : {},
-		...GLOBAL_REACT_DOCTOR_RULES,
-		...framework === "nextjs" ? NEXTJS_RULES : {},
-		...framework === "expo" || framework === "react-native" ? REACT_NATIVE_RULES : {},
-		...framework === "tanstack-start" ? TANSTACK_START_RULES : {},
-		...hasTanStackQuery ? TANSTACK_QUERY_RULES : {}
-	}
-});
+const createOxlintConfig = ({ pluginPath, framework, hasReactCompiler, hasTanStackQuery, customRulesOnly = false }) => {
+	const reactHooksJsPlugin = resolveReactHooksJsPlugin(hasReactCompiler, customRulesOnly);
+	return {
+		categories: {
+			correctness: "off",
+			suspicious: "off",
+			pedantic: "off",
+			perf: "off",
+			restriction: "off",
+			style: "off",
+			nursery: "off"
+		},
+		plugins: customRulesOnly ? [] : ["react", "jsx-a11y"],
+		jsPlugins: reactHooksJsPlugin ? [reactHooksJsPlugin, pluginPath] : [pluginPath],
+		rules: {
+			...customRulesOnly ? {} : BUILTIN_REACT_RULES,
+			...customRulesOnly ? {} : BUILTIN_A11Y_RULES,
+			...reactHooksJsPlugin ? REACT_COMPILER_RULES : {},
+			...GLOBAL_REACT_DOCTOR_RULES,
+			...framework === "nextjs" ? NEXTJS_RULES : {},
+			...framework === "expo" || framework === "react-native" ? REACT_NATIVE_RULES : {},
+			...framework === "tanstack-start" ? TANSTACK_START_RULES : {},
+			...hasTanStackQuery ? TANSTACK_QUERY_RULES : {}
+		}
+	};
+};
 //#endregion
 //#region src/utils/neutralize-disable-directives.ts
 const DISABLE_DIRECTIVE_PATTERN = /(eslint|oxlint)-disable/;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.0.44",
+  "version": "0.0.46",
   "description": "Diagnose and fix React codebases for security, performance, correctness, accessibility, bundle-size, and architecture issues",
   "keywords": [
     "accessibility",
@@ -62,6 +62,7 @@
     }
   },
   "dependencies": {
+    "agent-install": "0.0.4",
     "commander": "^14.0.3",
     "knip": "^6.3.1",
     "ora": "^9.3.0",