@elyracode/doctor 0.4.4 → 0.4.5

package/README.md

@@ -11,6 +11,7 @@ elyra install npm:@elyracode/doctor
 ## Commands
 
 - `/doctor` -- Run all health checks and send the report to the agent for analysis
+- `/doctor --heal` -- Run checks and auto-fix issues (the agent works through each finding step by step)
 
 ## Tools
 
@@ -41,6 +42,20 @@ elyra install npm:@elyracode/doctor
 
 The agent runs the checks automatically and suggests fixes based on findings.
 
+### Auto-Heal Mode
+```
+/doctor --heal
+```
+The agent works through each error and warning step by step:
+- **Security**: runs `npm audit fix` / `composer update`
+- **Dependencies**: updates outdated packages one by one
+- **Configuration**: fills in missing .env keys
+- **Code debt**: implements TODOs, resolves FIXMEs, cleans up HACKs
+- **Code quality**: refactors large files into smaller modules
+- **Project**: creates missing essential files
+
+After each fix, the agent verifies the change didn't break anything.
+
 ## Report Format
 
 ```

package/extensions/index.ts

@@ -5,17 +5,37 @@ import { type Finding, runAllChecks } from "./checks.js";
 export default function (elyra: ExtensionAPI): void {
 	// ── Command: /doctor ──
 	elyra.registerCommand("doctor", {
-		description: "Run project health analysis",
-		handler: async (_args: string, ctx) => {
+		description: "Run project health analysis. Use --heal to auto-fix issues.",
+		handler: async (args: string, ctx) => {
+			const healMode = args.trim() === "--heal" || args.trim() === "heal";
+
 			ctx.ui.notify("Running health checks...");
 
 			const cwd = process.cwd();
 			const findings = runAllChecks(cwd);
 			const report = formatReport(findings);
 
-			elyra.sendUserMessage(
-				`Here is the project health report. Analyze the findings and suggest the most important fixes:\n\n${report}`,
-			);
+			if (healMode) {
+				const actionable = findings.filter((f) => f.severity === "error" || f.severity === "warning");
+				if (actionable.length === 0) {
+					elyra.sendUserMessage(
+						`Health check complete. No issues to fix.\n\n${report}`,
+					);
+					return;
+				}
+
+				const healInstructions = formatHealInstructions(actionable);
+				elyra.sendUserMessage(
+					`Health check found ${actionable.length} issue${actionable.length > 1 ? "s" : ""} to fix. ` +
+					`Work through each one systematically. For each issue: analyze it, fix it if possible, and report what you did.\n\n` +
+					`${report}\n\n` +
+					`## Auto-Heal Instructions\n\n${healInstructions}`,
+				);
+			} else {
+				elyra.sendUserMessage(
+					`Here is the project health report. Analyze the findings and suggest the most important fixes:\n\n${report}`,
+				);
+			}
 		},
 	});
 
@@ -109,3 +129,67 @@ function formatReport(findings: Finding[]): string {
 
 	return lines.join("\n");
 }
+
+function formatHealInstructions(findings: Finding[]): string {
+	const instructions: string[] = [];
+	let step = 1;
+
+	for (const f of findings) {
+		const action = getHealAction(f);
+		if (action) {
+			instructions.push(`### Step ${step}: ${f.category} -- ${f.message}`);
+			instructions.push(action);
+			instructions.push("");
+			step++;
+		}
+	}
+
+	if (instructions.length === 0) {
+		return "No auto-healable issues found.";
+	}
+
+	instructions.push(`### Step ${step}: Verify`);
+	instructions.push("Run the health check again to verify all issues are resolved.");
+
+	return instructions.join("\n");
+}
+
+function getHealAction(finding: Finding): string | undefined {
+	const msg = finding.message.toLowerCase();
+	const cat = finding.category.toLowerCase();
+
+	if (cat === "security" && msg.includes("npm")) {
+		return "Run `npm audit fix` to fix vulnerabilities automatically. If that doesn't resolve all issues, run `npm audit` to see which packages need manual updates, then update them with `npm install <package>@latest`.";
+	}
+
+	if (cat === "security" && msg.includes("composer")) {
+		return "Run `composer audit` to see affected packages, then `composer update <package>` for each one. Check that tests still pass after updating.";
+	}
+
+	if (cat === "dependencies" && msg.includes("outdated")) {
+		return "Run `npm outdated` to see the full list. Update packages one by one with `npm install <package>@latest`, running tests after each update to catch breaking changes.";
+	}
+
+	if (cat === "configuration" && msg.includes(".env file is missing")) {
+		return "Copy `.env.example` to `.env` with `cp .env.example .env`. Then read the example file and fill in the values that need customization (database credentials, API keys, etc.).";
+	}
+
+	if (cat === "configuration" && msg.includes("keys in .env.example missing")) {
+		return "Read `.env.example` to find the missing keys. For each one: check the codebase to understand what it controls, determine a sensible default value, and add it to `.env`.";
+	}
+
+	if (cat === "code debt") {
+		return "Read each TODO/FIXME/HACK comment. For each one: if the task is small, implement it now and remove the comment. If it's large, create a GitHub issue and update the comment with the issue number. Remove stale comments that no longer apply.";
+	}
+
+	if (cat === "code quality" && msg.includes("files over")) {
+		return "For each large file: read it and identify responsibilities that can be extracted into separate classes/functions/files. Refactor one file at a time, running tests after each change.";
+	}
+
+	if (cat === "project" && msg.includes("missing")) {
+		const file = finding.message.replace("Missing ", "");
+		return `Create the missing ${file}. Read similar files in the project for conventions, or use standard templates.`;
+	}
+
+	return undefined;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elyracode/doctor",
-  "version": "0.4.4",
+  "version": "0.4.5",
   "description": "Elyra extension for project health analysis -- dependency audit, security checks, code quality, and more",
   "type": "module",
   "keywords": [