Skip to content

#150 - Require symfony/monolog-bundle and monolog/monolog - #151

Merged
njoubert-cleverage merged 1 commit into
mainfrom
150
Oct 14, 2026
Merged

njoubert-cleverage merged 1 commit into
mainfrom
150

Conversation

@njoubert-cleverage

Copy link
Copy Markdown
Member

Description

Fixes #150: require symfony/monolog-bundle (^3.11 || ^4, same range as cleverage/process-bundle) and monolog/monolog (^3: Monolog\Level and Monolog\LogRecord are Monolog 3 classes).

The bundle uses them directly but only got them through cleverage/process-bundle:

  • Monolog classes in src/Monolog/ (log handlers of the process executions), LogRecord entity, LogRecordCrudController, Configuration;
  • CleverAgeUiProcessExtension::prepend() configures the monolog extension (pb_ui_file / pb_ui_orm handlers and their channel filters), which fails without MonologBundle;
  • monolog.logger tags in config/services/*.yaml, MonologBundle in tests/App/TestKernel.php.

Prerequisite of cleverage/process-bundle#269, which replaces symfony/monolog-bundle by psr/log in process-bundle: to merge and release before it.

Checks

  • composer validate --strict: OK.
  • After composer update: composer why monolog/monolog lists the bundle itself; 397 tests OK, PHPStan OK (no PHP / Twig change).

Requirements

  • Documentation updates
    • Reference
    • Cookbooks
    • Changelog
  • Unit tests

Breaking changes

None: same packages as installed today, now declared.

🤖 Generated with Claude Code

Used directly by the bundle (Monolog handlers, "monolog" configuration
prepended, monolog.logger tags, TestKernel) but only installed through
cleverage/process-bundle, which may stop requiring MonologBundle
(cleverage/process-bundle#269).

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@njoubert-cleverage
njoubert-cleverage merged commit 534ac90 into main Oct 14, 2026
24 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.

Composer: require symfony/monolog-bundle and monolog/monolog, used directly but installed through process-bundle

1 participant