Skip to content
This repository was archived by the owner on Sep 2, 2026. It is now read-only.

Protect JSON CLI stdout from PHP diagnostics - #1309

Merged
chubes4 merged 1 commit into
mainfrom
fix/1279-json-output-purity
Aug 30, 2026
Merged

chubes4 merged 1 commit into
mainfrom
fix/1279-json-output-purity

Conversation

@chubes4

@chubes4 chubes4 commented Aug 30, 2026

Copy link
Copy Markdown
Member

Summary

  • detect DMC JSON requests before Composer/runtime bootstrap
  • route PHP diagnostics to stderr while leaving human formats unchanged
  • prove stdout remains strict-parseable in a subprocess contract

Fixes #1279.

Verification

  • php tests/cli-json-stdout-guard.php
  • php tests/workspace-show-cli-format-contract.php
  • git diff --check

AI assistance

  • Model: openai/gpt-5.6-sol
  • Tool: OpenCode
  • Used for: Investigated the corrupted stream, implemented the early CLI guard and subprocess regression coverage, and ran focused verification.

@chubes4
chubes4 merged commit 7cbc1f4 into main Aug 30, 2026
11 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

--format=json stdout is corrupted by a PHP deprecation notice, so output is not parseable

1 participant