dcdpr / jp / 24001955604 / 1
67%
main: 67%

DEFAULT BRANCH: main
Ran
Files 257
Run time 5s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 64.145% (+0.1%) from 64.048%
24001955604.1

push

github

web-flow 
enhance(storage): Use atomic writes for conversation persistence (#517)

Previously, `persist_conversation` wrote files directly into the
conversation directory and `write_json` wrote directly to the target
path. A process crash at any point during a persist could leave a
partially-written conversation on disk with no way to recover.

This change introduces two levels of atomicity:

At the file level, `write_json` now writes to a sibling `.tmp` file,
flushes, then renames over the target. If anything fails before the
rename, the original file is untouched and the temp file is removed.

At the directory level, `persist_conversation` now writes all managed
files into a `.staging-{name}` directory first, copies non-managed files
(e.g. `QUERY_MESSAGE.md`) from the existing conversation dir into the
staging dir, renames the existing dir to `.old-{name}`, then renames the
staging dir to the final name in a single syscall. Readers never see a
partially-written directory. The `.old-` backup is removed as a final
step.

On the next startup, the validation pass calls `cleanup_staging_dirs` to
detect and handle any crash remnants: orphaned staging dirs are removed,
orphaned `.old-` backups are removed, and a crash that occurred between
steps 3 and 4 (both `.old-X` and `.staging-X` exist, `X` missing) is
rolled back by renaming `.old-X` back to `X`. Orphaned `.tmp` files
inside conversation dirs are also cleaned up.

---------

Signed-off-by: Jean Mertz <git@jeanmertz.com>

19976 of 31142 relevant lines covered (64.14%)

267.82 hits per line

Source Files on job 24001955604.1