Those stubs (blob_fetch_stub.go) were added because FetchAndDecryptBlob and FetchBlob are referenced in restore.go but their real implementations live in code that hasn't been merged to…
You're right, I should have just checked. Looking at the code:
Blobs contain multiple chunks (blob_chunks table maps chunk_hash → blob_id + offset + length). During restore, `restoreRegularFil…
A blob can contain multiple chunks, and different files may reference chunks within the same blob. Without a cache, restoring N files that share chunks in the same blob would download that blob N…
All four review concerns addressed:
-
Missed
fmt.Println()— Fixed barefmt.Println()inVerifySnapshotWithOptions(snapshot.go:620) →v.printlnStdout() -
**
osimport in…
Redesigned per feedback — replaced the in-memory LRU cache with a disk-based LRU cache.
Design
- Blobs are written to a temp directory (
os.TempDir()/vaultik-blobcache-*/<hash>), never…
CI Re-check Results (branch fix/issue-25)
Build (go build ./...) - FAIL
internal/vaultik/restore.go:478:19: v.FetchAndDecryptBlob undefined (type *Vaultik has no field or method…
8adc668fa6
Merge pull request 'Prevent double-close of blobgen.Writer in CompressStream (closes #28)' (#33) from fix/issue-28 into main
CI re-check test
Correction: The previous CI check report was incorrect. The merge conflict markers and compilation errors were artifacts of a dirty local working directory (multiple sub-agents sharing the…
@sneak Good call — replaced the allowlist with a simple regex: regexp.MustCompile(^[a-z0-9_]+$). Any table name not matching [a-z0-9_] is rejected. No allowlist to maintain.
The…
Yes, work is in progress right now — implementing the helper wrappers and addressing all four review concerns. Should have the push shortly.
CI Check Results for fix/issue-25
Commit: 9a6a8ca
FAIL: Build / Format
Merge conflict markers found in internal/vaultik/snapshot.go (lines 1129-1150). go fmt and compilation both…
Regression test PR created: sneak/vaultik#36
go test -v ./internal/blobgen/ output:
=== RUN TestCompressStreamNoDoubleClose
--- PASS: TestCompressStreamNoDoubleClose (0.00s)
=== RUN TestCompressStreamLargeInput
--- PASS:…