Skip to content

fix(task) #15 ZipTask and UnzipTask resolve their options on every execution - #19

Merged
njoubert-cleverage merged 1 commit into
mainfrom
15
Sep 30, 2026
Merged

njoubert-cleverage merged 1 commit into
mainfrom
15

Conversation

@njoubert-cleverage

@njoubert-cleverage njoubert-cleverage commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

Description

Fixes #15.

ZipTask and UnzipTask merge their input into their options, but kept the options resolved on the first execution: with several inputs in the same process execution (e.g. after an iterable task), the following inputs were silently ignored and the first archive was re-created / re-extracted.

  • ZipTask::getOptions(), UnzipTask::getOptions(): resolve the options on every execution
  • ZipTaskTest, UnzipTaskTest (new): nominal case, and two inputs create / extract two archives (fails without the fix); symfony/filesystem added to require-dev for the temporary files cleanup
  • zip_task.md, unzip_task.md: replace the note describing the limitation

Checked with demo.archive.zip_multiple_inputs in process-bundle-demo: two archives are now created, demo_zip_multiple_inputs_1.zip with sample.txt and demo_zip_multiple_inputs_2.zip with sample.csv (only the first one, twice, before).

This is the first PR of a stack: #20 → #21 → #22 are based on it (they share the new test classes and the CHANGELOG Latest section).

Requirements

  • Documentation updates
    • Reference
    • Changelog
  • Unit tests

Breaking changes

None: each input is now used as documented. A configuration relying on the first input being reused already produced wrong results.

🤖 Generated with Claude Code

…ecution, so that each input is used

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ZipTask and UnzipTask reuse the options of the first input for the following ones

1 participant