Skip to content

fix: harden raw storage and split storage modules - #57

Merged
davidrobertson merged 25 commits into
mainfrom
fix/review-findings
Aug 4, 2026
Merged

fix: harden raw storage and split storage modules#57
davidrobertson merged 25 commits into
mainfrom
fix/review-findings

Conversation

@davidrobertson

Copy link
Copy Markdown
Contributor

Summary

  • harden raw-log locking, durability, and failure reporting across hook and command paths
  • split the storage facade into focused modules while preserving its public API
  • update retrieval guidance, package docs, smoke coverage, and contract tests

Why

The review found cases where concurrent writers and failed lock publication could strand or lose access to durable raw events. This change keeps events.jsonl authoritative, makes lock recovery explicit, and separates derived storage work from the raw append path.

Impact

Capture and recovery are safer under concurrent or failed writes. The public storage surface stays intact, while the smaller modules make each storage responsibility easier to test and review.

Validation

  • npm run typecheck
  • npm test
  • npm run smoke
  • npm pack --dry-run --cache /tmp/codex-lcm-npm-cache

@davidrobertson
davidrobertson marked this pull request as ready for review August 4, 2026 21:42
@davidrobertson
davidrobertson merged commit 752d29d into main Aug 4, 2026
3 checks passed
@davidrobertson
davidrobertson deleted the fix/review-findings branch August 4, 2026 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant