diff --git a/REFACTORING-AUDIT.md b/REFACTORING-AUDIT.md index 35464f3..14fa8ba 100644 --- a/REFACTORING-AUDIT.md +++ b/REFACTORING-AUDIT.md @@ -467,3 +467,7 @@ Migration order implied by the map: isolate shared infrastructure constructors ( - Market command now uses `SetupMarketRepository`, retaining PostgreSQL/Redis/services/availability/Stellar while skipping ledger-dispatch and profiling startup. Command and market tests pass in `0132734`. - Internal-wallet command now uses `SetupInternalWalletRepository`, retaining PostgreSQL/Redis/services/Stellar while skipping availability, ledger-dispatch, and profiling startup. Command and wallet tests pass in `f46df4c`. - Main wallet command now uses the explicit `SetupWalletRepository` entry point for its full dependency profile, completing named setup entry points for all five service modes. Command and wallet tests pass in `cf13a58`. + +### `W012` — superseded implementation cleanup checkpoint + +- Removed the now-redundant `core/marketImp/amount.go` shim and routed market settlement directly to `application/market.RawAmount`; focused market tests pass in `085140c`. diff --git a/REFACTORING-TODO.md b/REFACTORING-TODO.md index d3ecd2d..9ffceca 100644 --- a/REFACTORING-TODO.md +++ b/REFACTORING-TODO.md @@ -129,7 +129,7 @@ This is the authoritative execution tracker for the refactor. Work is performed | 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 and stream now have dedicated repository setup paths; cron uses PostgreSQL only and stream skips ledger-dispatcher/profiling startup. Remaining process composition work remains. | | W011 | `STARTED` | Replace wallet bootstrap with explicit dependency composition. | All five service modes have named setup paths; dependency internals and superseded bootstrap cleanup remain. | -| W012 | `TODO` | Remove superseded `core/*Imp` packages and shims. | No duplicate business implementations remain. | +| W012 | `STARTED` | Remove superseded `core/*Imp` packages and shims. | Removed the superseded market raw-amount shim; broader duplicate implementation cleanup remains. | | W013 | `DONE` | Remove redundant wallet federation creation/model. | Removed federation creation and lookup, wallet federation fields, federation persistence/repository adapters, transaction federation fields/filters, federation protobuf messages, and API documentation/routes. Wallet/Auth/API contracts regenerate and tests pass. | ## Phase 4 — `api` architecture @@ -331,3 +331,4 @@ Append one row whenever a task changes status. Existing rows are never rewritten | 2026-08-30 | W010 | `TODO` | `STARTED` | Added `application/cron.Retry`, `application/stream.IsNew`, stream external-deposit transaction policy, and dedicated `SetupCronRepository`/`SetupStreamRepository` paths. Focused Wallet tests pass in `db52f89`. Remaining process composition work remains. | | 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. | | 2026-08-30 | W011 | `TODO` | `STARTED` | Added dedicated alert, market, internal-wallet, stream, cron, and wallet repository setup entry points; each mode now declares its bootstrap profile. Command/core tests pass in `cf13a58`. Dependency internals and superseded bootstrap cleanup remain. | +| 2026-08-30 | W012 | `TODO` | `STARTED` | Removed `core/marketImp/amount.go` delegation shim and routed settlement callers/tests directly to `application/market.RawAmount`. Focused market tests pass in `085140c`. Remaining core implementation cleanup is pending. |