Skip to content

[command] Respect raw output when rendering DevTools banner#279

Closed
coisa wants to merge 9 commits intomainfrom
issue/277-no-logo-raw-changelog-output
Closed

[command] Respect raw output when rendering DevTools banner#279
coisa wants to merge 9 commits intomainfrom
issue/277-no-logo-raw-changelog-output

Conversation

@coisa
Copy link
Copy Markdown
Contributor

@coisa coisa commented Apr 28, 2026

Summary

Implemented a central logo rendering decision in the DevTools console entrypoint and applied it so command output uses the ASCII banner only for non-raw flows.

Changes

  • Added central shouldRenderLogo() logic in DevTools to compute logo visibility from command name and options.
  • Added explicit raw-output command suppression for changelog:next-version and changelog:show, which previously relied on raw output contracts and now do not print banner.
  • Added option-based suppression for --no-logo, --json, and --pretty-json before any output rendering.
  • Updated unit coverage around run-time behavior in tests/Console/DevToolsTest.php to validate that raw changelog commands skip the logo.

Testing

  • Focused tests: vendor/bin/phpunit tests/Console/DevToolsTest.php.
  • Additional verification: existing targeted command tests for changelog JSON/raw behaviors in the suite.

Review note

Logo gating is centralized in DevTools::doRun() via shouldRenderLogo(InputInterface $input).

Closes #277

Implementation note: raw/changelog-raw behavior is now controlled at the command-boot layer (isRawOutputCommand) so each command avoids per-command output flags and remains orchestrator-safe.

@coisa coisa closed this Apr 28, 2026
@coisa coisa deleted the issue/277-no-logo-raw-changelog-output branch April 28, 2026 22:10
@github-project-automation github-project-automation Bot moved this from Backlog to Merged in PHP Fast Forward Project Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

Add global banner for top-level dev-tools execution with orchestration-safe suppression

1 participant