Skip to content

🪲 BUG-#215: Fix StorageFile.get() returning empty list for missing keys#223

Merged
FernandoCelmer merged 1 commit intodevelopfrom
feature/215
Apr 8, 2026
Merged

🪲 BUG-#215: Fix StorageFile.get() returning empty list for missing keys#223
FernandoCelmer merged 1 commit intodevelopfrom
feature/215

Conversation

@FernandoCelmer
Copy link
Copy Markdown
Member

Description

Fix StorageFile.get() returning Context(storage=[]) instead of Context() when a key does not exist. This broke resume=True — all tasks were silently skipped because [] is not None evaluates True.

Issue: 📌 ISSUE-#215

Changes

  • Add early return Context() when task_context is empty in StorageFile.get()
  • Aligns with the pattern already used by StorageS3 and StorageGCS

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@FernandoCelmer FernandoCelmer added bug Something isn't working critical Critical severity - must fix labels Apr 8, 2026
@FernandoCelmer FernandoCelmer merged commit 7865c1a into develop Apr 8, 2026
10 checks passed
@FernandoCelmer FernandoCelmer deleted the feature/215 branch April 9, 2026 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working critical Critical severity - must fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant