Skip to content

Fix DMC JSON parsing with WP-CLI diagnostics - #460

Merged
chubes4 merged 1 commit into
mainfrom
fix/459-dmc-json-diagnostics
Aug 25, 2026
Merged

Fix DMC JSON parsing with WP-CLI diagnostics#460
chubes4 merged 1 commit into
mainfrom
fix/459-dmc-json-diagnostics

Conversation

@chubes4

@chubes4 chubes4 commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

  • isolate one complete JSON document after recognized leading PHP diagnostics
  • reuse the decoder across DMC provider JSON boundaries
  • retain fail-closed behavior for arbitrary prefixes and malformed output
  • add task-resolution regression coverage for both accepted and rejected prefixes

Closes #459.

Verification

  • php -l scripts/homeboy-dmc-provider.php
  • bash -n tests/homeboy-dmc-provider.sh
  • bash tests/homeboy-dmc-provider.sh
  • git diff --check

AI Assistance

GPT-5.6 Sol via OpenCode diagnosed the runtime failure, implemented the focused repair and regression coverage, and ran the verification under human direction.

@chubes4
chubes4 merged commit 64b196f into main Aug 25, 2026
61 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.

DMC provider should isolate JSON from WP-CLI diagnostic prefixes

1 participant