From ffb08152387aad9788f7e7325b6bdfdd3568b074 Mon Sep 17 00:00:00 2001 From: nfel Date: Sun, 30 Aug 2026 20:05:39 +0330 Subject: [PATCH] docs: start W009 internal wallet extraction --- REFACTORING-AUDIT.md | 4 ++++ REFACTORING-TODO.md | 3 ++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/REFACTORING-AUDIT.md b/REFACTORING-AUDIT.md index 69a9e05..b432747 100644 --- a/REFACTORING-AUDIT.md +++ b/REFACTORING-AUDIT.md @@ -455,3 +455,7 @@ Migration order implied by the map: isolate shared infrastructure constructors ( - Stellar payment stream lookup classification now uses `application/stream.IsNew`, preserving not-found and invalid empty-result behavior while keeping Horizon/database access in infrastructure. Focused Wallet tests pass in `6c3243c`. - Cron command now uses `SetupCronRepository`, avoiding unnecessary strict service peers, availability monitor, Stellar client, profiling server, and ledger dispatcher initialization; cron jobs receive only PostgreSQL-backed repository dependencies. Focused command tests pass in `08f3225`. - Stellar stream external-deposit transaction construction now uses `application/transaction.SuccessfulExternalDeposit`; network parsing, deduplication, locking, and repository insertion remain infrastructure-owned. Focused stream/transaction tests pass in `61768e7`. + +### `W009` — internal-wallet boundary checkpoint + +- Lock/release balance mutation policy now uses `application/walletlock.Lock` and `Release`, preserving available/frozen balance rules while keeping transaction scope, ledger journal creation, persistence, and gRPC error mapping in `core/walletImp`. Focused Wallet tests pass in `382fd79`. diff --git a/REFACTORING-TODO.md b/REFACTORING-TODO.md index f11212d..ce689b6 100644 --- a/REFACTORING-TODO.md +++ b/REFACTORING-TODO.md @@ -126,7 +126,7 @@ This is the authoritative execution tracker for the refactor. Work is performed | W006 | `STARTED` | Extract deposit, withdrawal, and transaction use cases. | Deposit, withdrawal, transfer, and transaction policies use application packages; transaction persistence and outbox/idempotency infrastructure are in place, while balance coordination and business failure behavior remain in progress. | | W007 | `STARTED` | Extract market use cases and adapters. | Agreement amount tolerance now uses `application/market`; pricing, order lifecycle, settlement, and adapter boundaries remain. | | W008 | `STARTED` | Extract alert use cases and adapters. | Alert level/source presentation now uses `application/alert`; delivery, retry, persistence, and adapter boundaries remain. | -| W009 | `TODO` | Extract internal-wallet use cases and adapters. | Internal RPC contracts remain compatible. | +| W009 | `STARTED` | Extract internal-wallet use cases and adapters. | Lock/release balance mutation policy now uses `application/walletlock`; transaction, ledger, persistence, and RPC adapter boundaries remain. | | W010 | `STARTED` | Separate cron and stream bootstrap from business operations. | Cron retry policy uses `application/cron`, and cron bootstrap now initializes only PostgreSQL; stream and remaining process composition work remain. | | W011 | `TODO` | Replace wallet bootstrap with explicit dependency composition. | All five sub-services work independently and together. | | W012 | `TODO` | Remove superseded `core/*Imp` packages and shims. | No duplicate business implementations remain. | @@ -329,3 +329,4 @@ Append one row whenever a task changes status. Existing rows are never rewritten | 2026-08-30 | W007 | `TODO` | `STARTED` | Added typed `application/market` policies for agreement tolerance, available amount, raw conversion, positive order inputs, ICO publisher validation, order status codes, and market pricing calculation; market ICO/order validation and calculation delegate through compatibility mapping. Synchronous settlement/stream locking fixed in `3004bb9`; order lifecycle, settlement, contract generation, and adapter extraction remain. | | 2026-08-30 | W008 | `TODO` | `STARTED` | Added `application/alert` level/source label and subject policies plus nil/incomplete-event validation; alert delivery keeps existing asynchronous email/SMS retry behavior. Focused Wallet tests pass in `b3e1dbc`. Delivery, retry, persistence, and adapter extraction remain. | | 2026-08-30 | W010 | `TODO` | `STARTED` | Added `application/cron.Retry`, `application/stream.IsNew`, and stream external-deposit transaction policy; `SetupCronRepository` initializes only PostgreSQL for cron jobs. Focused Wallet tests pass in `61768e7`. Stream and remaining process composition work remain. | +| 2026-08-30 | W009 | `TODO` | `STARTED` | Added `application/walletlock` lock/release mutation policies with frozen/available balance tests; internal RPC retains transaction scope, ledger journaling, persistence, and error mapping. Focused Wallet tests pass in `382fd79`. Remaining internal-wallet operations stay at the service boundary. |