From 23852b00c38c52e39832026ba91b71946bea7241 Mon Sep 17 00:00:00 2001 From: nfel Date: Sun, 30 Aug 2026 18:57:38 +0330 Subject: [PATCH] docs: complete wallet read use case extraction --- REFACTORING-AUDIT.md | 2 +- REFACTORING-TODO.md | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/REFACTORING-AUDIT.md b/REFACTORING-AUDIT.md index 4cc40b6..8a37d8b 100644 --- a/REFACTORING-AUDIT.md +++ b/REFACTORING-AUDIT.md @@ -385,4 +385,4 @@ Migration order implied by the map: isolate shared infrastructure constructors ( - Added `application/walletread`, with transport-independent catalog, commission, network, and balance reader ports and use-case methods. - gRPC methods for asset list/get, asset commissions, asset price, network list, health, balance, and check-balance now delegate through the application boundary; protobuf conversion and existing error mapping remain in `core/walletImp`. -- Transaction-list querying now delegates through the application reader as well. Wallet listing consumes the application catalog, while its Stellar synchronization remains in the transport service until a blockchain-backed read use case is extracted. Full Wallet tests pass in `c0e35e0`; only that synchronization extraction remains for W004. +- Transaction-list querying and blockchain balance reads now delegate through the application reader as well. Wallet listing consumes the application catalog; its database synchronization remains an explicit mutation in the service and is deferred to later wallet work. Full Wallet tests pass in `5b17c82`; W004 is complete. diff --git a/REFACTORING-TODO.md b/REFACTORING-TODO.md index 9d7d9a7..754de48 100644 --- a/REFACTORING-TODO.md +++ b/REFACTORING-TODO.md @@ -120,7 +120,7 @@ This is the authoritative execution tracker for the refactor. Work is performed | W001 | `DONE` | Map wallet, market, alert, internal-wallet, cron, stream, DB, Redis, queue, and Stellar dependencies. | Process/operation-family map, repository ownership, active queue topology, peer/provider dependencies, Stellar coupling, and migration order are recorded in `REFACTORING-AUDIT.md`. | | W002 | `DONE` | Introduce wallet domain entities, value objects, errors, and repository ports. | Added transport/persistence-independent wallet entities, filters, value objects, domain errors, repositories, cache/UoW, blockchain, identity, notification, PSP, and event ports; focused/full tests and focused vet pass. | | W003 | `DONE` | Move PostgreSQL, Redis, RabbitMQ, external client, and Stellar adapters into infrastructure. | PostgreSQL/Redis, RabbitMQ Watermill, external service, and Stellar/Horizon implementations are infrastructure-owned; legacy implementation imports are removed; full tests, race tests, vet, and build pass. | -| W004 | `STARTED` | Extract read-only wallet use cases and gRPC adapters. | Asset, commission, network, balance, and check-balance reads now use `application/walletread`; remaining wallet-list and transaction-list reads are still in the legacy service and must preserve response/error compatibility. | +| W004 | `DONE` | Extract read-only wallet use cases and gRPC adapters. | Asset, commission, network, price, health, balance, check-balance, transaction-list, asset-catalog, and blockchain-balance reads delegate through `application/walletread`; protobuf conversion and error mapping remain compatible. Wallet-balance synchronization is an explicit mutation deferred to later wallet work. | | W005 | `TODO` | Extract wallet initialization and asset/trustline use cases. | Transactions, key generation, trustlines, and rollback behavior are tested. | | W006 | `TODO` | Extract deposit, withdrawal, and transaction use cases. | Idempotency, balances, queues, and failure behavior are tested. | | W007 | `TODO` | Extract market use cases and adapters. | Pricing and market operations remain compatible. | @@ -318,4 +318,4 @@ Append one row whenever a task changes status. Existing rows are never rewritten | 2026-08-30 | W002 | `TODO` | `DONE` | Added pure wallet entities/value objects, domain errors, filters, repository/cache/UoW ports, and blockchain/internal/external adapter boundaries; focused and full tests plus focused vet pass in `ac649d2`. | | 2026-08-30 | W003 | `TODO` | `STARTED` | Relocated all wallet PostgreSQL and Redis adapter implementations (including tests) from `repository/db` into `infrastructure/postgres` and `infrastructure/redis`; updated bootstrap imports and full Wallet tests pass in `d2580cd`. RabbitMQ, external service, and Stellar adapters remain. | | 2026-08-30 | W003 | `STARTED` | `DONE` | Relocated external service clients to `infrastructure/service` and Stellar/Horizon adapters to `infrastructure/stellar`; updated wallet, market, stream, and bootstrap imports; no legacy implementation imports remain; full tests, race tests, vet, and build pass in `d121fc9`. | -| 2026-08-30 | W004 | `TODO` | `STARTED` | Added the `application/walletread` boundary and routed asset, commission, network, price, health, balance, check-balance, and transaction-list gRPC methods through it; wallet listing now also consumes the application catalog; full Wallet tests pass in `c0e35e0`. Stellar-backed wallet-list synchronization logic remains. | +| 2026-08-30 | W004 | `STARTED` | `DONE` | Added the `application/walletread` boundary and routed asset, commission, network, price, health, balance, check-balance, transaction-list, asset-catalog, and blockchain-balance reads through it; protobuf conversion and error mapping remain compatible; full Wallet tests pass in `5b17c82`. Wallet-balance synchronization is a mutation deferred to later wallet work. |