Skip to content

Add environment and writable path diagnostics - #53

Merged
josiahking merged 1 commit into
2.xfrom
feature/phase-6-4b-environment-writable-diagnostics
Aug 28, 2026
Merged

Add environment and writable path diagnostics#53
josiahking merged 1 commit into
2.xfrom
feature/phase-6-4b-environment-writable-diagnostics

Conversation

@josiahking

Copy link
Copy Markdown
Owner

Summary

  • add a configurable Doctor check for required environment variables
  • report environment-variable presence without exposing values
  • add a configurable Doctor check for writable local filesystem paths
  • keep writable-path inspection read-only with no automatic filesystem changes
  • document both checks as programmatic project diagnostics
  • update the accepted Core source inventory

Behavior

Environment variable diagnostics treat only a missing process variable as absent; explicitly defined empty-string values remain present. Required names are validated, duplicate definitions are rejected, lookup order is deterministic, and environment values are never included in findings or remediation.

Writable-path diagnostics inspect explicitly supplied local paths, preserve their supplied order, reject invalid or duplicate definitions, and report non-writable paths without creating directories or changing permissions.

These checks are not automatically added to the package-owned shell Doctor. Existing shell behavior remains unchanged.

Validation

  • Core test suite passes
  • full quality suite passes on PHP 8.4
  • static analysis passes with no errors
  • coding-style validation passes
  • architecture and documentation tests pass
  • release package validation passes
  • prerelease consumer validation passes
  • deterministic package split validation passes
  • Composer manifest, security, supply-chain, and license checks pass

@josiahking josiahking self-assigned this Aug 28, 2026
@josiahking
josiahking merged commit 7bfd9f4 into 2.x Aug 28, 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