Converge npm/Maven datastore stores onto OCI's explicit qrm.DB provider-closure (ADR-023)
Context
Raised in review of !826 (merged) (npm-hosted Step 26 wiring plan amendment): !826 (comment 3530863228)
The handler-seam dimension of cross-format wiring consistency is being handled directly in !826 (merged): npm's Step 26 plan now mirrors OCI's NewDispatchHandlerWith* constructor chain (previously Maven's Deps struct).
This issue tracks the remaining, larger dimension — datastore store construction. OCI's datastore methods take an explicit qrm.DB per ADR-023, and wire_oci.go wraps stores in adapters holding a db func() qrm.DB provider closure. npm and Maven diverge: their stores are already merged taking *postgres.Client directly and resolving client.DB() lazily per query.
Scope
- Audit the npm (
internal/datastore/npm_*.go, ~11 stores) and Maven datastore stores against OCI's explicit-qrm.DB+ provider-closure convention (ADR-023). - Either converge npm/Maven onto that shape (store-signature + call-site refactor across already-merged code), or explicitly ratify the divergence with documented rationale.
- This is a cross-cutting refactor of merged datastore code, not a wiring-step change.
Related to !826 (merged).
Edited by Dzmitry (Dima) Meshcharakou