Skip to content

fix(archives): serialize duplicate archive extractions - #2583

Merged
jonathanKingston merged 1 commit into
mainfrom
codex/serialize-archive-extraction
Sep 9, 2026
Merged

fix(archives): serialize duplicate archive extractions#2583
jonathanKingston merged 1 commit into
mainfrom
codex/serialize-archive-extraction

Conversation

@jonathanKingston

Copy link
Copy Markdown
Collaborator

Concurrent requests to unpack the same ZIP delete and write the same .partial directory. One request can remove another’s in-progress files or fail during rename.

Use the existing keyed serializer around extraction and reuse, keyed by the final archive directory. Duplicate requests wait for the first extraction and reuse its completed files; different archives retain independent queues. A concurrent three-request regression verifies all callers succeed and every file survives.

Validation: pnpm run check. The concurrent regression fails on current main. Filesystem plumbing only; no DOM or layout changes.

github-actions Bot added a commit that referenced this pull request Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

🖥️ PR preview

@jonathanKingston
jonathanKingston merged commit 85682b4 into main Sep 9, 2026
23 checks passed
@jonathanKingston
jonathanKingston deleted the codex/serialize-archive-extraction branch September 9, 2026 00:29
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.

1 participant