Skip to content

Add Doctor console command adapter - #50

Merged
josiahking merged 1 commit into
2.xfrom
feature/phase-6-2-doctor-console-command
Aug 26, 2026
Merged

Add Doctor console command adapter#50
josiahking merged 1 commit into
2.xfrom
feature/phase-6-2-doctor-console-command

Conversation

@josiahking

Copy link
Copy Markdown
Owner

Summary

Adds a runtime-neutral doctor command adapter that connects the existing Doctor diagnostics to Core's console command abstraction.

Changes

  • adds DoctorCommand backed by an injected DoctorRunner
  • renders PASS, WARNING, and FAIL findings in deterministic report order
  • emits remediation immediately after its finding when available
  • returns exit code 0 for successful reports, 1 for diagnostic failures, and 2 for unsupported arguments
  • keeps diagnostic failures on normal command output rather than the error channel
  • preserves programming and malformed-definition failures as throwables
  • documents the console adapter and its current limitations
  • updates the Core source inventory and adds focused command coverage

Validation

  • 548 tests / 2064 assertions
  • architecture suite: 63 tests / 5575 assertions
  • documentation suite: 130 tests / 2124 assertions
  • dependency analysis: 0 violations
  • PHP CS Fixer: 186 files checked, 0 fixable
  • release package validation passed
  • package split validation passed
  • prerelease consumer validation passed
  • security audit and licence policy passed

Package split

Core:
e0eb08cf23fa393050210954cbd050963d582f95

Scope

4 files changed, 380 insertions.

No shell executable, argv parser, JSON output, Composer probing, environment inspection, route inspection, writable-path inspection, dependency changes, or package-manifest changes are included.

@josiahking josiahking self-assigned this Aug 26, 2026
@josiahking
josiahking merged commit 6761ed4 into 2.x Aug 26, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant