Skip to content

Conversation

@crodas
Copy link
Collaborator

@crodas crodas commented Jan 4, 2026

Description

Split the large monolithic mod.rs file into separate files, each containing a database trait and its corresponding transaction trait:

  • proofs.rs: MintProofsTransaction + MintProofsDatabase
  • keys.rs: MintKeyDatabaseTransaction + MintKeysDatabase
  • quotes.rs: MintQuotesTransaction + MintQuotesDatabase
  • signatures.rs: MintSignatureTransaction + MintSignaturesDatabase
  • keyvalue.rs: KVStoreTransaction + KVStoreDatabase + KVStore
  • saga.rs: SagaTransaction + SagaDatabase
  • completed_operations.rs: CompletedOperationsTransaction + CompletedOperationsDatabase

The mod.rs now only contains struct definitions, migrations, and the main MintDatabase/MintTransaction/DbTransactionFinalizer impls.

closes: #1489


Notes to the reviewers


Suggested CHANGELOG Updates

CHANGED

ADDED

REMOVED

FIXED


Checklist

@crodas crodas requested a review from thesimplekid January 4, 2026 03:21
@crodas crodas self-assigned this Jan 4, 2026
Split the large monolithic mod.rs file into separate files, each containing a
database trait and its corresponding transaction trait:

- proofs.rs: MintProofsTransaction + MintProofsDatabase
- keys.rs: MintKeyDatabaseTransaction + MintKeysDatabase
- quotes.rs: MintQuotesTransaction + MintQuotesDatabase
- signatures.rs: MintSignatureTransaction + MintSignaturesDatabase
- keyvalue.rs: KVStoreTransaction + KVStoreDatabase + KVStore
- saga.rs: SagaTransaction + SagaDatabase
- completed_operations.rs: CompletedOperationsTransaction +
  CompletedOperationsDatabase

The mod.rs now only contains struct definitions, migrations, and the main
MintDatabase/MintTransaction/DbTransactionFinalizer impls.
@crodas crodas force-pushed the feature/split-mod-file-sql-common branch from 740ce27 to 216ecdd Compare January 4, 2026 03:22
@crodas crodas force-pushed the feature/split-mod-file-sql-common branch from fb52ac4 to 4e599d0 Compare January 4, 2026 04:51
@crodas crodas force-pushed the feature/split-mod-file-sql-common branch from 4e599d0 to cdfc407 Compare January 4, 2026 04:55
crodas and others added 2 commits January 4, 2026 12:28
Co-authored-by: tsk <tsk@thesimplekid.com>
Co-authored-by: tsk <tsk@thesimplekid.com>
@thesimplekid thesimplekid merged commit dd196a8 into cashubtc:main Jan 5, 2026
175 of 176 checks passed
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.

refactor: break cdk-sql-common mint mod.rs into multple mode

2 participants