From 44cf2f131eea0dbc5adb620a1162efff01d13018 Mon Sep 17 00:00:00 2001 From: nfel Date: Sun, 30 Aug 2026 19:22:14 +0330 Subject: [PATCH] docs: complete federation removal task --- REFACTORING-AUDIT.md | 2 +- REFACTORING-TODO.md | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/REFACTORING-AUDIT.md b/REFACTORING-AUDIT.md index 5d67430..f0ab4fb 100644 --- a/REFACTORING-AUDIT.md +++ b/REFACTORING-AUDIT.md @@ -404,4 +404,4 @@ Migration order implied by the map: isolate shared infrastructure constructors ( - Federation is not currently safe to delete outright: wallet persistence links `wallet.federation_id`, transaction records expose nullable `from_federation_id`/`to_federation_id`, generated wallet APIs expose federation messages, and wallet code still has federation lookup/creation paths. - Auth has no active federation implementation; its wallet federation client is commented/dead code. Auth’s identity service should remain the owner of identity and national-ID data. - The proposed target is reasonable only after consumers are migrated: `user_id → identity_id → wallet_id`, with `wallet_id` still scoped to `asset_id` and Stellar key derivation/custody explicitly preserved. W013 must first identify whether federation addresses or transaction routing are actually supported in production, then remove or retain the model based on evidence and a data/API compatibility plan. -- Per the requested migration, automatic federation creation is now removed from wallet initialization. New wallet creation uses user/asset identity and leaves federation assignment unset; Auth's dead federation client was deleted. Wallet federation tables, fields, generated API messages, and transaction routing fields still require the remaining W013 cleanup. +- Per the requested migration, federation is fully removed from active Wallet/Auth/API code and the shared wallet protobuf: no federation creation/lookup, persistence adapter/model, wallet field, transaction field/filter, generated message, stale API route, or runtime reference remains. New wallet creation uses user/asset identity; wallet records remain asset-scoped. Wallet/Auth/API contracts were regenerated and all service tests pass. The database must be migrated separately to drop legacy federation columns/tables in deployed environments. diff --git a/REFACTORING-TODO.md b/REFACTORING-TODO.md index e3626b6..86f6203 100644 --- a/REFACTORING-TODO.md +++ b/REFACTORING-TODO.md @@ -130,7 +130,7 @@ This is the authoritative execution tracker for the refactor. Work is performed | W010 | `TODO` | Separate cron and stream bootstrap from business operations. | Both modes start and stop correctly through explicit dependencies. | | 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. | -| W013 | `STARTED` | Remove redundant wallet federation creation/model. | Automatic federation lookup/generation is removed from wallet creation; new wallets are created from user/asset identity with no federation assignment. Remaining federation schema/API/transaction-field removal is still in progress. | +| 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 @@ -324,3 +324,4 @@ Append one row whenever a task changes status. Existing rows are never rewritten | 2026-08-30 | W005 | `TODO` | `STARTED` | Added `application/walletinit` for identity preconditions, trustline-limit calculation, key recovery, trustline submission, wallet-code generation, wallet-draft construction, repository-backed find-or-create orchestration, trustline transaction construction, and tested commit/rollback plus existing/create wallet paths; full Wallet tests pass in `15200dc`. | | 2026-08-30 | A009 | `TODO` | `DONE` | Removed Auth's commented/dead wallet federation client; Auth tests pass with writable cache in `508a239`. | | 2026-08-30 | W013 | `TODO` | `STARTED` | Removed automatic federation creation and federation lookup from wallet initialization; new wallet records no longer receive a federation assignment; full Wallet tests pass in `97dd9d9`. Remaining schema/API/transaction-field cleanup is pending. | +| 2026-08-30 | W013 | `STARTED` | `DONE` | Removed federation persistence/model code, wallet and transaction federation fields, federation protobuf messages, stale API route/docs, and remaining runtime references; regenerated Wallet contracts for Wallet/Auth/API; Wallet, Auth, and API tests pass. Wallet `a38501d`/`7ae90cc`, Auth `446e875`, API `c450264`/`c2c076c`. |