Skip to content

#94 - flysystem_list_content: read the listing item by item - #95

Merged
njoubert-cleverage merged 1 commit into
mainfrom
94
Nov 6, 2026
Merged

njoubert-cleverage merged 1 commit into
mainfrom
94

Conversation

@njoubert-cleverage

Copy link
Copy Markdown
Member

Description

Fixes #94:

  • ListContentTask::getFilteredFilesystemContents() is a generator iterating the Flysystem listing; same output and order, same behaviour for an empty listing or no matching item; each new execution lists again;
  • list_content_task.md: with an adapter listing lazily, files removed during the iteration (e.g. by a flysystem_remove_file task downstream) may change the listing.

Checked locally: highest dependencies (PHP 8.5.12), with pcov coverage: all tests OK, no risky; PHPStan, PHP-CS-Fixer and Rector OK.

Requirements

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

Breaking changes

The protected $fsContent and getFilteredFilesystemContents() (class @final since 3.3) now hold / return a Generator.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@njoubert-cleverage
njoubert-cleverage requested a review from a team as a code owner November 6, 2026 13:16
@njoubert-cleverage
njoubert-cleverage merged commit f493908 into main Nov 6, 2026
17 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.

flysystem_list_content loads the whole listing in memory

1 participant