Add FileWorker.delete() method #9

New issue

Closed

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

gravityfargo commented

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

Owner

Context

Filed in support of athena-archive-api #81 (couple BE tightly with athena-file). The BE currently bypasses athena-file for file deletion — `Path.unlink()` directly in `routers/media/delete.py` and a custom `_remove_dir()` recursive helper in `routers/media/utils.py`. Routing those through athena-file would let the library maintain hash/index consistency on delete.

Plan

Add `FileWorker.delete()` that:

Removes the underlying file
Hash properties become invalid afterwards (document this clearly)
Returns `bool` (True if file existed and was removed)
Optionally: notifies `FileArchiver` / `ShardedJSONStorage` so any indices stay consistent (or leave that to a higher-level wrapper)

Acceptance criteria

`FileWorker.delete()` exists and is tested
Documented that calling `delete()` invalidates the worker

## Context Filed in support of athena-archive-api #81 (couple BE tightly with athena-file). The BE currently bypasses athena-file for file deletion — \`Path.unlink()\` directly in \`routers/media/delete.py\` and a custom \`_remove_dir()\` recursive helper in \`routers/media/utils.py\`. Routing those through athena-file would let the library maintain hash/index consistency on delete. ## Plan Add \`FileWorker.delete()\` that: - Removes the underlying file - Hash properties become invalid afterwards (document this clearly) - Returns \`bool\` (True if file existed and was removed) - Optionally: notifies \`FileArchiver\` / \`ShardedJSONStorage\` so any indices stay consistent (or leave that to a higher-level wrapper) ## Acceptance criteria - [ ] \`FileWorker.delete()\` exists and is tested - [ ] Documented that calling \`delete()\` invalidates the worker

gravityfargo added the

enhancement

label

2026-05-03 10:09:11 +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