thesimplekid/satsettle
1
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

refactor(storage)!: leave database coordination to applications #22

Merged
thesimplekid merged 1 commit from simplify/application-storage into main 2026-10-09 16:37:56 +00:00
Conversation 0 Commits 1 Files changed 21 +45 -2668
thesimplekid commented 2026-10-09 16:36:00 +00:00
Owner
Copy link

Remove the bundled PostgreSQL and lease implementation so applications own database setup, migrations, and worker coordination. Keep the PaymentStore, CashuJournal, and LightningJournal traits and embedded redb implementation, with the coordination and wallet-ownership contracts documented.

This removes the PostgreSQL feature, dependencies, migration, test harness, and CI job. Applications supplying storage must coordinate both journal and wallet writes during ownership transfer.

Validation: workspace formatting, Clippy, tests, documentation, cargo-deny, and native Nix checks passed. The complete dependent stack also passed 78 tests and the real Bitcoin/LDK regtest.

Remove the bundled PostgreSQL and lease implementation so applications own database setup, migrations, and worker coordination. Keep the PaymentStore, CashuJournal, and LightningJournal traits and embedded redb implementation, with the coordination and wallet-ownership contracts documented. This removes the PostgreSQL feature, dependencies, migration, test harness, and CI job. Applications supplying storage must coordinate both journal and wallet writes during ownership transfer. Validation: workspace formatting, Clippy, tests, documentation, cargo-deny, and native Nix checks passed. The complete dependent stack also passed 78 tests and the real Bitcoin/LDK regtest.
refactor(storage)!: leave database coordination to applications
Some checks failed
Checks / format (push) Successful in 1m10s
Details
Checks / cdk-compatible (push) Failing after 12m47s
Details
Checks / checks (push) Failing after 17m45s
Details
Checks / dependencies (push) Successful in 1m18s
Details
Checks / dependencies (pull_request) Successful in 1m4s
Details
Checks / checks (pull_request) Failing after 17m58s
Details
Checks / format (pull_request) Successful in 1m12s
Details
Checks / cdk-compatible (pull_request) Successful in 13m34s
Details
71c0f5b77a 
Keep the storage traits, embedded redb adapter and reusable contract tests.
Remove the bundled PostgreSQL backend, lease workers, wallet fencing hooks,
and their dependencies, development scripts and CI jobs.

Document application ownership of database setup and exclusive wallet access,
including fencing of journal and wallet writes during distributed failover.

BREAKING CHANGE: Remove the postgres feature, PostgresStore, lease errors,
and PaymentStore wallet validation hooks. Applications needing PostgreSQL
must supply their own storage implementation and worker coordination.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
thesimplekid/satsettle!22
No description provided.