@ankhorage/doctor 0.2.0 → 0.3.0

package/src/commands.ts

@@ -1,6 +1,6 @@
 import type { AnkhCapabilityId, AnkhCommandDescriptor } from '@ankhorage/contracts/cli';
 
-import type { DoctorAnalysisResult, DoctorTargetMode } from './analysis.js';
+import type { DoctorAnalysisResult, DoctorPlannedChange, DoctorTargetMode } from './analysis.js';
 import { analyzeDoctorTarget } from './analysis.js';
 import type { DoctorCommandContext, DoctorCommandRunResult } from './commandContext.js';
 import {
@@ -64,7 +64,7 @@ export const DOCTOR_COMMANDS = [
     path: ['validate'],
     capability: COMMAND_CAPABILITIES.validate,
     mode: 'validate',
-    summary: 'Auto-detect lightweight repo and package diagnostics for a local path.',
+    summary: 'Auto-detect the applicable doctor policy profile and validate a local path.',
     run: runValidateCommand,
   },
   {
@@ -72,7 +72,7 @@ export const DOCTOR_COMMANDS = [
     path: ['fix'],
     capability: COMMAND_CAPABILITIES.fix,
     mode: 'fix',
-    summary: 'Run the same lightweight checks and report that no automatic fixes exist yet.',
+    summary: 'Build a non-mutating fix plan for deterministic doctor policy violations.',
     run: runFixCommand,
   },
   {
@@ -226,7 +226,6 @@ async function runFixCommand(
   });
 
   request.context.writeStdout(renderAnalysisReport('fix', result));
-  request.context.writeStdout('No automatic fixes available.\n');
 
   return {
     exitCode: countErrorDiagnostics(result.diagnostics) > 0 ? 1 : 0,
@@ -274,6 +273,7 @@ function renderAnalysisReport(
   const lines = [
     `doctor ${commandName}`,
     `target: ${result.targetPath}`,
+    `profile: ${result.profile}`,
     `checks: ${result.appliedChecks.length > 0 ? result.appliedChecks.join(', ') : 'none'}`,
     `repoMarkers: ${result.repoMarkers.length > 0 ? result.repoMarkers.join(', ') : 'none'}`,
     '',
@@ -288,6 +288,19 @@ function renderAnalysisReport(
     }
   }
 
+  if (commandName === 'fix') {
+    lines.push('');
+    lines.push('Planned changes:');
+
+    if (result.plannedChanges.length === 0) {
+      lines.push('  none');
+    } else {
+      for (const change of result.plannedChanges) {
+        lines.push(renderPlannedChangeLine(change));
+      }
+    }
+  }
+
   const errorCount = countErrorDiagnostics(result.diagnostics);
   const warningCount = countWarningDiagnostics(result.diagnostics);
 
@@ -299,7 +312,11 @@ function renderAnalysisReport(
 }
 
 function renderDiagnosticLine(diagnostic: DoctorDiagnostic): string {
-  return `  - ${diagnostic.severity.toUpperCase()} ${diagnostic.code}: ${diagnostic.message}`;
+  return `  - ${diagnostic.severity.toUpperCase()} ${diagnostic.ruleId} (${diagnostic.code}): ${diagnostic.message}`;
+}
+
+function renderPlannedChangeLine(change: DoctorPlannedChange): string {
+  return `  - ${change.ruleId} [${change.kind}]: ${change.description}`;
 }
 
 function renderCommandFailure(commandName: DoctorCommandName, error: unknown): string {

package/src/diagnostics.ts

@@ -1,26 +1,83 @@
 export type DoctorDiagnosticCode =
+  | 'field-invalid'
+  | 'field-missing'
   | 'invalid-ankh-metadata'
   | 'invalid-package-json'
-  | 'missing-changelog'
-  | 'missing-changeset-directory'
-  | 'missing-license'
-  | 'missing-readme'
-  | 'missing-workflows-directory'
-  | 'package-json-missing'
-  | 'package-name-missing'
-  | 'package-type-missing'
-  | 'package-version-missing'
-  | 'repo-markers-missing'
+  | 'missing-dependency'
+  | 'missing-path'
+  | 'missing-script'
+  | 'provider-source-import-failed'
+  | 'provider-source-missing'
   | 'target-not-directory'
   | 'target-not-found'
   | 'unsupported-target';
 
 export type DoctorDiagnosticSeverity = 'error' | 'warning';
+export type DoctorPolicyProfile = 'integration-monorepo' | 'public-package' | 'unknown';
+export type DoctorRuleId =
+  | 'integration.package.ankh.disallowed'
+  | 'integration.package.private.required'
+  | 'integration.package.workspaces.required'
+  | 'package.ankh.capabilities.match-provider'
+  | 'package.ankh.present.valid-shape'
+  | 'package.ankh.provider-path.required'
+  | 'package.ankh.required-for-provider'
+  | 'package.dependencies.changesets.required'
+  | 'package.dependencies.devtools.required'
+  | 'package.dependencies.paradox.required'
+  | 'package.dependencies.types-bun.required'
+  | 'package.dependencies.types-node.required'
+  | 'package.dependencies.typescript.required'
+  | 'package.json.bugs.required'
+  | 'package.json.description.required'
+  | 'package.json.exports.required'
+  | 'package.json.files.required'
+  | 'package.json.homepage.required'
+  | 'package.json.keywords.required'
+  | 'package.json.license.required'
+  | 'package.json.name.required'
+  | 'package.json.package-manager.bun'
+  | 'package.json.package-manager.required'
+  | 'package.json.private.public-package-disallowed'
+  | 'package.json.publish-config.public'
+  | 'package.json.publish-config.required'
+  | 'package.json.repository.required'
+  | 'package.json.type.module'
+  | 'package.json.type.required'
+  | 'package.json.version.required'
+  | 'package.scripts.build.required'
+  | 'package.scripts.changeset-status.required'
+  | 'package.scripts.changeset.required'
+  | 'package.scripts.docs.required'
+  | 'package.scripts.format-check.required'
+  | 'package.scripts.format.required'
+  | 'package.scripts.knip.required'
+  | 'package.scripts.lint-fix.required'
+  | 'package.scripts.lint.required'
+  | 'package.scripts.test.required'
+  | 'package.scripts.typecheck.required'
+  | 'package.scripts.version-packages.required'
+  | 'provider.commands.match-capabilities'
+  | 'provider.commands.match-handlers'
+  | 'provider.source.importable'
+  | 'provider.source.required'
+  | 'repo.changelog.required'
+  | 'repo.changeset.required'
+  | 'repo.license.required'
+  | 'repo.readme.required'
+  | 'repo.workflows.required'
+  | 'target.package-json.required'
+  | 'target.path.directory'
+  | 'target.path.exists'
+  | 'target.profile.supported'
+  | 'target.repo-markers.required';
 
 export interface DoctorDiagnostic {
   readonly code: DoctorDiagnosticCode;
   readonly message: string;
   readonly path: string;
+  readonly profile: DoctorPolicyProfile;
+  readonly ruleId: DoctorRuleId;
   readonly severity: DoctorDiagnosticSeverity;
 }
 

package/src/index.ts

@@ -1,6 +1,9 @@
 export type {
   DoctorAnalysisRequest,
   DoctorAnalysisResult,
+  DoctorFixPlan,
+  DoctorPlannedChange,
+  DoctorPlannedChangeKind,
   DoctorTargetCheck,
   DoctorTargetMode,
 } from './analysis.js';
@@ -16,6 +19,8 @@ export type {
   DoctorDiagnostic,
   DoctorDiagnosticCode,
   DoctorDiagnosticSeverity,
+  DoctorPolicyProfile,
+  DoctorRuleId,
 } from './diagnostics.js';
 export {
   DOCTOR_CAPABILITIES,

package/src/readme-usage.ts

@@ -17,10 +17,18 @@ import { runCli } from './cli.js';
  * - `repo`
  * - `package`
  *
- * `doctor#1` intentionally stays lightweight:
+ * `doctor#2` adds a profile-based policy engine:
  *
- * - path-aware local repo/package diagnostics are implemented now
- * - GitHub checks, CI checks, full policy enforcement, and autofix rules are deferred
+ * - strict `public-package` validation for extracted public package repos
+ * - light recognized `integration-monorepo` validation for `ankhorage4`
+ * - non-mutating `fix` plans for deterministic mechanical changes only
+ *
+ * Still deferred:
+ *
+ * - GitHub checks
+ * - CI checks
+ * - on-disk `fix --apply`
+ * - deeper cross-repo policy enforcement
  *
  * Path handling:
  *