Skip to content

#83 #84 #85 Remove the broken UploadAndExecuteAction, LogRecord::hasContext(), ProcessHandler default level, declare symfony/ux-twig-component - #86

Merged
njoubert-cleverage merged 3 commits into
mainfrom
83
Oct 2, 2026
Merged

njoubert-cleverage merged 3 commits into
mainfrom
83

Conversation

@njoubert-cleverage

@njoubert-cleverage njoubert-cleverage commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

Description

Fixes #83.
Fixes #84.
Fixes #85.

Tests: LogRecordTest (hasContext(), deprecated contextIsEmpty()), ProcessHandlerTest (default and configured report level, no write without filename). Without the fix, the two tests on hasContext() and the default level fail. The 2 PHPUnit deprecations come from phpunit.xml.dist (executionOrder="depends", schema), as in the other bridges.

Checked in process-bundle-demo with this branch installed (logged in): the container compiles, the process_upload_and_execute route is gone, and the process list, process launch (file upload form of demo.upload_and_run), executions, logs (with the "Has context info ?" column) and users pages answer 200.

Also documents, with a comment on ProcessScheduleCrudController::schedulerIsRunning(), that the scheduler worker detection (ps -faux) only finds workers running on the same host as the web server (limitation already described in 05-scheduler.md).

Also fixes the v3.0.2 CHANGELOG entry (separate commit): broken GHSA link (…/security/advisories/...) and hard line wrapping.

Not in this PR: the full test coverage of #81.

Requirements

  • Documentation updates
    • Reference
    • Changelog
  • Unit tests

Breaking changes

  • The process_upload_and_execute route and the UploadAndExecuteAction, ProcessUploadFileType and ProcessConfigurationValueResolver classes are removed (the route always failed).
  • LogRecord::contextIsEmpty() is deprecated.

🤖 Generated with Claude Code

njoubert-cleverage and others added 2 commits October 2, 2026 15:47
…hasContext(), ProcessHandler default report level, declare symfony/ux-twig-component

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…cheduleCrudController

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

1 participant