Put and PutFile were the same function twice: cap check, path build,
blob description, budget claim, sidecar first. Extract reserve, release,
bucket and newBlob so each caller reads as its own difference. List and
pruneOrphans walked a kind's tree with the same skeleton, so that is now
walkSidecars and walkBlobFiles over one walkKind.
Name what was a literal in five places: 0o700, 0o600, ".json", "spool",
the two-character bucket and the 64-character id. The kind list is one
var rather than four copies.
Rename the stat result in both writers. It was called "already", and
"already != nil" meant the blob was NOT already there, which reads
backwards at every use. It is now statErr with a fresh bool beside it.
No behaviour change. The reservation leak on a failed writeMeta is
preserved on purpose and filed as V-584.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
claude
ecaf3407fa