bring back Cost field but mark for deprecation

cmd/goal/clerk.go

@@ -1211,6 +1211,9 @@ var dryrunRemoteCmd = &cobra.Command{
 						trace = *txnResult.LogicSigTrace
 					}
 				}
+				if txnResult.Cost != nil {
+					fmt.Fprintf(os.Stdout, "tx[%d] cost: %d\n", i, *txnResult.Cost)
+				}
 				if txnResult.BudgetConsumed != nil {
 					fmt.Fprintf(os.Stdout, "tx[%d] budget consumed: %d\n", i, *txnResult.BudgetConsumed)
 				}

daemon/algod/api/algod.oas2.json

@@ -2302,6 +2302,10 @@
         "budget-consumed": {
           "description": "Budget consumed during execution of app call transaction.",
           "type": "integer"
+        },
+        "cost": {
+          "description": "Net cost of app execution. Will be DEPRECATED.",
+          "type": "integer"
         }
       }
     },

daemon/algod/api/algod.oas3.yml

@@ -1336,6 +1336,10 @@
             "description": "Budget consumed during execution of app call transaction.",
             "type": "integer"
           },
+          "cost": {
+            "description": "Net cost of app execution. Will be DEPRECATED.",
+            "type": "integer"
+          },
           "disassembly": {
             "description": "Disassembled program line by line.",
             "items": {

daemon/algod/api/server/v2/dryrun.go

@@ -546,8 +546,10 @@ func doDryrunRequest(dr *DryrunRequest, response *generated.DryrunResponse) {
 				// integers, so a negative cost would underflow. The two fields also provide
 				// more information, which can be useful for testing purposes.
 				// cost = budgetConsumed - budgetAdded
+				netCost := uint64(cost)
 				budgetAdded := uint64(proto.MaxAppProgramCost * numInnerTxns(delta))
 				budgetConsumed := uint64(cost) + budgetAdded
+				result.Cost = &netCost
 				result.BudgetAdded = &budgetAdded
 				result.BudgetConsumed = &budgetConsumed
 				maxCurrentBudget = pooledAppBudget