refactor: change raydium provider to query committed blocks instead of finalized

providers/apis/defi/raydium/README.md

@@ -0,0 +1,17 @@
+# Raydium Provider
+
+The Raydium provider fetches prices from the Raydium dex via JSON-RPC requests to Solana nodes. 
+
+## How It Works
+
+For each ticker (i.e. RAY/SOL), we query 4 accounts:
+
+* BaseTokenVault
+* QuoteTokenVault
+* AMMInfo
+* OpenOrders
+
+To calculate the price, we need to get the base and quote token balances, subtract PNL feels, and add the value of open orders.
+
+With the above values, we calculate the price by dividing quote / base and multiplying by the scaling factor.
+

providers/apis/defi/raydium/price_fetcher.go

@@ -159,7 +159,11 @@ func (pf *APIPriceFetcher) Fetch(
 	tickers []oracletypes.ProviderTicker,
 ) oracletypes.PriceResponse {
 	// get the accounts to query in order of the tickers given
-	expectedNumAccounts := len(tickers) * 4
+	expectedNumAccounts := len(tickers) * 4 // for each ticker, we query 4 accounts (base, quote, amm, open orders)
+
+	// accounts is a contiguous slice of solana.PublicKey.
+	// each ticker takes up 4 slots. this is functionally equivalent to [][]solana.PubKey,
+	// however, storing in one slice allows us query without rearranging the request data (i.e. converting [][] to []).
 	accounts := make([]solana.PublicKey, expectedNumAccounts)
 
 	for i, ticker := range tickers {
@@ -188,7 +192,7 @@ func (pf *APIPriceFetcher) Fetch(
 	defer cancel()
 
 	accountsResp, err := pf.client.GetMultipleAccountsWithOpts(ctx, accounts, &rpc.GetMultipleAccountsOpts{
-		Commitment: rpc.CommitmentFinalized,
+		Commitment: rpc.CommitmentConfirmed,
 		// TODO(nikhil): Keep track of latest height queried as well?
 	})
 	if err != nil {