comment

protocol/x/accountplus/authenticator/signature_authenticator.go

@@ -58,6 +58,7 @@ func (sva SignatureVerification) Initialize(config []byte) (types.Authenticator,
 func (sva SignatureVerification) Authenticate(ctx sdk.Context, request types.AuthenticationRequest) error {
 	// First consume gas for verifying the signature
 	params := sva.ak.GetParams(ctx)
+	// Signature verification only accepts secp256k1 signatures so consume static gas here.
 	ctx.GasMeter().ConsumeGas(params.SigVerifyCostSecp256k1, "secp256k1 signature verification")
 
 	// after gas consumption continue to verify signatures

protocol/x/clob/e2e/permissioned_keys_test.go

@@ -1238,6 +1238,12 @@ func TestPlaceOrder_PermissionedKeys_Success(t *testing.T) {
 				constants.Order_Alice_Num0_Id1_Clob1_Sell5_Price15_GTB20_IOC.OrderId: 0,
 			},
 		},
+		// Short term maker orders are explicitly removed when the permissioned key is removed. This
+		// is because short term orders go through another round of ante handler check during `DeliverTx`
+		// and we have to maintain the invariant that the operations queue is always valid.
+		//
+		// On contrast, stateful orders don't go through ante handlers and therefore we can allow these orders
+		// to be matched optimistically.
 		"Stateful maker order can be matched even if permissioned key is removed": {
 			smartAccountEnabled: true,
 			blocks: []TestBlockWithMsgs{