chore: resolve withdrawToSeat TODO with comment

Agoric · Dec 9, 2024 · 4db6510 · 4db6510
1 parent e33f9e8
commit 4db6510
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/packages/fast-usdc/src/exos/settler.js b/packages/fast-usdc/src/exos/settler.js
@@ -230,13 +230,12 @@ export const prepareSettler = (
           const split = calculateSplit(received);
           log('disbursing', split);
 
-          // TODO: what if this throws?
-          // arguably, it cannot. Even if deposits
-          // and notifications get out of order,
-          // we don't ever withdraw more than has been deposited.
+          // If this throws, which arguably can't occur since we don't ever
+          // withdraw more than has been deposited, funds will remain in the
+          // `settlementAccount`. A remediation can occur in a future upgrade.
           await vowTools.when(
             withdrawToSeat(
-              // @ts-expect-error Vow vs. Promise stuff. TODO: is this OK???
+              // @ts-expect-error LocalAccountMethods vs OrchestrationAccount
               settlementAccount,
               settlingSeat,
               harden({ In: received }),