Add atomic write_bytes helper for upload temp files #11

New issue

Closed

opened 2026-05-03 10:09:14 +00:00 by gravityfargo · 0 comments

gravityfargo commented

2026-05-03 10:09:14 +00:00

Owner

Context

Filed in support of athena-archive-api #81. The BE writes uploaded bytes to a temp file with raw `Path.write_bytes(file_content)` (utils.py:166), which is non-atomic — a crash mid-write leaves a half-written file.

Plan

Add a helper that takes `(target: Path, data: bytes)`, writes to `target.with_suffix(target.suffix + ".part")`, then `os.replace()` to the final path. Returns a `FileWorker` for the new file.

Could live as `FileWorker.from_bytes(target, data)` factory or as a standalone `atomic_write_bytes()` helper.

Acceptance criteria

Helper exists; tests cover happy path + interrupt-mid-write (file should not exist at target)

## Context Filed in support of athena-archive-api #81. The BE writes uploaded bytes to a temp file with raw \`Path.write_bytes(file_content)\` (utils.py:166), which is non-atomic — a crash mid-write leaves a half-written file. ## Plan Add a helper that takes \`(target: Path, data: bytes)\`, writes to \`target.with_suffix(target.suffix + ".part")\`, then \`os.replace()\` to the final path. Returns a \`FileWorker\` for the new file. Could live as \`FileWorker.from_bytes(target, data)\` factory or as a standalone \`atomic_write_bytes()\` helper. ## Acceptance criteria - [ ] Helper exists; tests cover happy path + interrupt-mid-write (file should not exist at target)

gravityfargo added the

enhancement

label

2026-05-03 10:09:16 +00:00

gravityfargo referenced this issue from a commit

2026-05-03 14:46:33 +00:00

feat: file ops helpers for archive coupling (#9, #10, #11)

gravityfargo referenced this issue from a pull request that will close it,

2026-05-03 14:46:47 +00:00

feat: file ops helpers for archive coupling (#9, #10, #11) #12

gravityfargo closed this issue

2026-05-03 14:47:49 +00:00