Skip to content

#31 - Give a code to the tasks, class name as service id - #32

Merged
njoubert-cleverage merged 2 commits into
mainfrom
31
Oct 15, 2026
Merged

njoubert-cleverage merged 2 commits into
mainfrom
31

Conversation

@njoubert-cleverage

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

Copy link
Copy Markdown
Member

Description

Fixes #31: propagate cleverage/process-bundle#91 (PR cleverage/process-bundle#294) to this bridge. Deprecations introduced in 2.3, removal in 3.0.

Changes

  • Codes: archive_zip (ZipTask) and archive_unzip (UnzipTask), cleverage.task tag, so that the tasks are referenced with task: archive_zip (process-bundle 5.4 or later, where referencing a task by its service is deprecated).
  • Service ids: the class names, as in process-bundle (definitions non-shared, as before). They stay public, but fetching them directly from the service container is deprecated since 2.3 ({ name: container.private, package: cleverage/archive-process-bundle, version: '2.3' }): Accessing the "…" service directly from the container is deprecated, use dependency injection instead.; private in 3.0. The references of the container (process-bundle task locator) use the private service, without deprecation.
  • Former ids cleverage_archive_process.task.zip / cleverage_archive_process.task.unzip (released since 1.0, private): deprecated aliases of the class names since 2.3 (The "cleverage_archive_process.task.zip" service alias is deprecated, use the "archive_zip" task code or the class name of the task instead. It will be removed in 3.0.), removed in 3.0. They stay private, as before.
  • Docs: the code on the reference page of each task (and in the template), all the examples use the codes (archive tasks and process-bundle tasks, 19 references); the doctrine / flysystem tasks of the cookbooks keep their service until those bridges get their codes (Give a code to the tasks (process-bundle#91), class name as service id doctrine-process-bundle#48, Give a code to the tasks (process-bundle#91), class name as service id flysystem-process-bundle#51); index.md (codes, service ids, deprecations).
  • CHANGELOG (Latest).

Requirement not changed yet: cleverage/process-bundle stays ^5.0, because 5.4 is not released (Composer could not resolve ^5.4, the CI would fail). The change is harmless with process-bundle ≤ 5.3 (the cleverage.task tag is ignored there; container.private and deprecated aliases are Symfony features). To raise to ^5.4 before releasing 2.3 (the task: key of the documentation requires it).

Checks

  • Demo project (process-bundle main and this branch copied in vendor/, then reverted): lint:container OK; demo.archive_unzip / demo.archive_zip (still service: '@<class>', i.e. the service id now) executed; a temporary process using task: constant_output → task: archive_unzip → task: input_folder_browser executed; no deprecation of this bundle in the container build log (the former ids are not used).
  • Tests: CleverAgeArchiveProcessExtensionTest (class name as id, non-shared, code, public + container.private, former ids private deprecated aliases with their message). After composer update: 31 tests OK, PHPStan, PHP-CS-Fixer, Rector OK, composer validate --strict OK.

Requirements

  • Documentation updates
    • Reference
    • Changelog
  • Unit tests

Breaking changes

None: the former ids and the public access are only deprecated (2.3), removed in 3.0.

🤖 Generated with Claude Code

njoubert-cleverage and others added 2 commits October 15, 2026 11:25
- Codes (cleverage/process-bundle#91): archive.zip, archive.unzip
  (cleverage.task tag), to reference the tasks with "task: <code>" (process-bundle
  5.4 or later, "service" being deprecated there).
- Service ids: the class names (as in process-bundle). Still public, but
  fetching the tasks directly from the service container is deprecated since
  2.3 (container.private tag): private in 3.0.
- Former ids cleverage_archive_process.task.zip / .unzip: deprecated aliases
  since 2.3, removed in 3.0.
- Documentation: codes on the reference pages, examples switched to the task
  codes (archive and process-bundle tasks), index. Tests.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
archive_zip / archive_unzip instead of archive.zip / archive.unzip, to avoid
name conflicts (convention of the bridge codes: "<bridge>_<name>").

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

Development

Successfully merging this pull request may close these issues.

Give a code to the tasks (process-bundle#91), class name as service id

1 participant