Merge pull request #3393 from input-output-hk/check-all-wallet-TODOs

[CBR-227] Go over TODOs, remove stray, add tickets

Author: edsko

wallet-new/server/Main.hs

@@ -127,10 +127,6 @@ actionWithNewWallet pm txpConfig sscParams nodeParams params =
         sscParams
         (txpGlobalSettings pm txpConfig)
         (initNodeDBs pm epochSlots) $ \nr -> do
-      -- TODO: Will probably want to extract some parameters from the
-      -- 'NewWalletBackendParams' to construct or initialize the wallet
-
-      -- TODO(ks): Currently using non-implemented layer for wallet layer.
       userSecret <- readTVarIO (ncUserSecret $ nrContext nr)
       let nodeState = NodeStateAdaptor.newNodeStateAdaptor nr
       liftIO $ Keystore.bracketLegacyKeystore userSecret $ \keystore -> do

wallet-new/src/Cardano/Wallet/API/V1/Handlers/Transactions.hs

@@ -53,8 +53,6 @@ newTransaction :: ActiveWalletLayer IO
                -> Handler (WalletResponse Transaction)
 newTransaction aw payment@Payment{..} = do
 
-    -- TODO(adn) If the wallet is being restored, we need to disallow any @Payment@ from
-    -- being submitted.
     -- NOTE(adn) The 'SenderPaysFee' option will become configurable as part
     -- of CBR-291.
     res <- liftIO $ (WalletLayer.pay aw) (maybe mempty coerce pmtSpendingPassword)

wallet-new/src/Cardano/Wallet/Kernel.hs

@@ -103,7 +103,7 @@ data WalletHandles = Handles {
 handlesOpen :: IO WalletHandles
 handlesOpen = do
     db <- openMemoryState defDB
-    metadb <- openMetaDB ":memory:" -- TODO: Eventually :memory: should be replaced with the real path.
+    metadb <- openMetaDB ":memory:" -- TODO: CBR-378
     migrateMetaDB metadb            -- TODO: this will be run with asynchronous exceptions masked.
     return $ Handles db metadb
 
@@ -144,7 +144,7 @@ init PassiveWallet{..} = do
 
 -- | Prefilter the block for each esk in the `WalletESK` map.
 --   Return a unified Map of accountId and prefiltered blocks (representing multiple ESKs)
--- TODO(@uroboros/ryan) optimisation: we are prefiltering the block n times for n keys, change this to be a single pass
+-- TODO: Improve performance (CBR-379)
 prefilterBlock' :: PassiveWallet
                 -> ResolvedBlock
                 -> IO (Map HdAccountId PrefilteredBlock)

wallet-new/src/Cardano/Wallet/Kernel/Addresses.hs

@@ -52,8 +52,6 @@ data CreateAddressError =
       -- to find another random index would be too expensive
     deriving Eq
 
--- TODO(adn): This will be done as part of my work on the 'newTransaction'
--- endpoint, see [CBR-313].
 instance Arbitrary CreateAddressError where
     arbitrary = oneof []
 

wallet-new/src/Cardano/Wallet/Kernel/CoinSelection/FromGeneric.hs

@@ -250,7 +250,7 @@ runCoinSelT opts pickUtxo policy request utxo = do
             originalOuts = case outs of
                                []   -> error "runCoinSelT: empty list of outputs"
                                o:os -> o :| os
-        -- TODO: We should shuffle allOuts
+        -- TODO: We should shuffle allOuts CBR-380
         return . Right $ CoinSelFinalResult allInps
                                             originalOuts
                                             (concatMap coinSelChange css)

wallet-new/src/Cardano/Wallet/Kernel/DB/HdWallet.hs

@@ -239,8 +239,6 @@ instance Arbitrary HdAddressId where
 -- address indices.
 --
 -- NOTE: We do not store the encrypted key of the wallet.
---
--- TODO: synchronization state
 data HdRoot = HdRoot {
       -- | Wallet ID
       _hdRootId          :: HdRootId

wallet-new/src/Cardano/Wallet/Kernel/DB/HdWallet/Create.hs

@@ -148,9 +148,6 @@ initHdRoot rootId name hasPass assurance created = HdRoot {
 --
 -- It is the responsibility of the caller to check the wallet's spending
 -- password.
---
--- TODO: If any key derivation is happening when creating accounts, should we
--- store a public key or an address or something?
 initHdAccount :: HdAccountId
               -> Checkpoint
               -> HdAccount

wallet-new/src/Cardano/Wallet/Kernel/DB/HdWallet/Read.hs

@@ -2,11 +2,6 @@
 --
 -- NOTE: These are pure functions, which are intended to work on a snapshot
 -- of the database. They are intended to support the V1 wallet API.
---
--- TODO: We need to think about which layer will have the responsibility for
--- filtering and sorting. If we want the 'IxSet' stuff to be local to the
--- "Kernel.DB" namespace (which would be a good thing), then filtering and
--- sorting (and maybe even pagination) will need to happen here.
 module Cardano.Wallet.Kernel.DB.HdWallet.Read (
     -- | * Infrastructure
     HdQuery

wallet-new/src/Cardano/Wallet/Kernel/DB/Resolved.hs

@@ -71,10 +71,6 @@ data ResolvedBlock = ResolvedBlock {
 makeLenses ''ResolvedTx
 makeLenses ''ResolvedBlock
 
--- TODO: Why are these necessary? and if they're not, get rid of InDb
---deriveSafeCopy 1 'base ''ResolvedTx
---deriveSafeCopy 1 'base ''ResolvedBlock
-
 {-------------------------------------------------------------------------------
   Pretty-printing
 -------------------------------------------------------------------------------}

wallet-new/src/Cardano/Wallet/Kernel/DB/Sqlite.hs

@@ -355,7 +355,7 @@ newConnection path = Sqlite.open path
 -- | Closes an open 'Connection' to the @Sqlite@ database stored in the
 -- input 'MetaDBHandle'.
 -- Even if open failed with error, this function should be called http://www.sqlite.org/c3ref/open.html
--- TODO: provide a brucket style interface to ensure this.
+-- TODO: provide a bracket style interface to ensure this.
 closeMetaDB :: Sqlite.Connection -> IO ()
 closeMetaDB = Sqlite.close