veeso / ic-dbms / 21728590178

95%

main: 96%

Build Type

Pull #40

github

Committed by veeso

Commit Message

refactor(canister): clean up dbms.rs with multiple improvements

- Extract duplicated record collection pattern from update/delete into
  collect_matching_records helper method
- Fix with_transaction to use immutable borrow (get_transaction)
  instead of unnecessarily getting a mutable reference
- Extract values_to_schema_entity as a standalone module-level function
  instead of an inner function inside update
- Wrap insert oneshot path in atomic for consistency with update/delete
- Hoist PK lookup outside inner loop in delete_foreign_keys_cascade
- Use pks.len() for update transaction count instead of re-querying
- Select only PK column in existing_primary_keys_for_filter
- Re-export NextRecord from memory module for crate-wide use

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Pull Request Pull Request #40: refactor(canister): clean up dbms.rs with multiple improvements

Coverage Stats

47 of 48 new or added lines in 1 file covered. (97.92%)

10 existing lines in 1 file now uncovered.

9770 of 10215 relevant lines covered (95.64%)

735000.27 hits per line