Bump hedera.app to 0.71.0 (#12936)

This PR fixes the compilation issues when bumping hedera.app to 0.71.0-rc3.
Also CustomMessageCallProcessor and RootProxyWorldUpdater overriden classes are removed.
The OpcodeActionTracer now tracks the gasUsage for precompiles using a field in the ContractCallContext.
Precompile calls with null output now have tracked gasUsage. Previously if the output was null, the gasCost was directly set to 0.

---------

Signed-off-by: Ivan Kavaldzhiev <ivankavaldzhiev@gmail.com>

Author: IvanKavaldzhiev

.github/workflows/releases-comparator.yml

@@ -94,8 +94,6 @@ jobs:
             "com/swirlds/state/spi/ReadableKVStateBase.java"
             "com/swirlds/state/spi/WritableKVStateBase.java"
             "com/swirlds/state/spi/WritableSingletonStateBase.java"
-            "com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java"
-            "com/hedera/node/app/service/contract/impl/state/RootProxyWorldUpdater.java"
           )
 
           # Compare the schemas - in case there is a new version we might need to register a new state.

build.gradle.kts

@@ -19,7 +19,7 @@ extra.apply {
     set("assertj.version", "3.27.7") // Temporary until next Spring Boot
     set("besuVersion", "25.2.2")
     set("blockNodeVersion", "0.24.2")
-    set("consensusNodeVersion", "0.70.0")
+    set("consensusNodeVersion", "0.71.0")
     set("grpcVersion", "1.79.0")
     set("jooq.version", "3.20.11") // Must match buildSrc/build.gradle.kts
     set("mapStructVersion", "1.6.3")
@@ -55,7 +55,7 @@ dependencies {
         api("com.graphql-java:graphql-java-extended-validation:24.0")
         api("com.hedera.hashgraph:app:$consensusNodeVersion")
         api("com.hedera.hashgraph:app-service-entity-id-impl:$consensusNodeVersion")
-        api("com.hedera.hashgraph:hedera-protobuf-java-api:0.71.0-rc.3") // verify no issues
+        api("com.hedera.hashgraph:hedera-protobuf-java-api:$consensusNodeVersion")
         api("com.hedera.hashgraph:sdk:2.66.0")
         api("com.ongres.scram:client:2.1")
         api("commons-beanutils:commons-beanutils:1.11.0")

web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java

@@ -61,6 +61,9 @@ public class ContractCallContext {
     @Setter
     private boolean isBalanceCall;
 
+    @Setter
+    private long gasRemaining;
+
     @Setter
     private long gasRequirement;
 

web3/src/main/java/com/hedera/node/app/service/contract/impl/state/RootProxyWorldUpdater.java

@@ -2,15 +2,20 @@
 
 package org.hiero.mirror.web3.evm.contracts.execution.traceability;
 
+import com.hedera.node.app.service.contract.impl.state.RootProxyWorldUpdater;
 import com.hederahashgraph.api.proto.java.ResponseCodeEnum;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
+import java.util.Map;
 import java.util.Optional;
+import java.util.TreeMap;
+import java.util.stream.Collectors;
 import org.apache.tuweni.bytes.Bytes;
 import org.hiero.mirror.common.domain.contract.ContractAction;
 import org.hiero.mirror.web3.common.ContractCallContext;
 import org.hiero.mirror.web3.convert.BytesDecoder;
+import org.hyperledger.besu.evm.ModificationNotAllowedException;
 import org.hyperledger.besu.evm.frame.MessageFrame;
 import org.springframework.util.CollectionUtils;
 
@@ -44,6 +49,45 @@ protected final List<Bytes> captureStack(final MessageFrame frame, final OpcodeT
         return stack;
     }
 
+    protected Map<Bytes, Bytes> captureStorage(final MessageFrame frame, final OpcodeTracerOptions options) {
+        if (!options.isStorage()) {
+            return Collections.emptyMap();
+        }
+
+        try {
+            var worldUpdater = frame.getWorldUpdater();
+            while (worldUpdater.parentUpdater().isPresent()) {
+                worldUpdater = worldUpdater.parentUpdater().get();
+            }
+
+            if (!(worldUpdater instanceof RootProxyWorldUpdater rootProxyWorldUpdater)) {
+                // The storage updates are kept only in the RootProxyWorldUpdater.
+                // If we don't have one -> something unexpected happened and an attempt to
+                // get the storage changes from a ProxyWorldUpdater would result in a
+                // NullPointerException, so in this case just return an empty map.
+                return Collections.emptyMap();
+            }
+            final var updates = rootProxyWorldUpdater
+                    .getEvmFrameState()
+                    .getTxStorageUsage(true)
+                    .accesses();
+            return updates.stream()
+                    .flatMap(storageAccesses ->
+                            storageAccesses.accesses().stream()) // Properly flatten the nested structure
+                    .collect(Collectors.toMap(
+                            e -> Bytes.wrap(e.key().toArray()),
+                            e -> Bytes.wrap(
+                                    e.writtenValue() != null
+                                            ? e.writtenValue().toArray()
+                                            : e.value().toArray()),
+                            (v1, v2) -> v1, // in case of duplicates, keep the first value
+                            TreeMap::new));
+
+        } catch (final ModificationNotAllowedException e) {
+            return Collections.emptyMap();
+        }
+    }
+
     protected final Optional<Bytes> getRevertReasonFromContractActions(final ContractCallContext context) {
         final var contractActions = context.getContractActions();
 

web3/src/main/java/org/hiero/mirror/web3/common/ContractCallContext.java

@@ -5,7 +5,11 @@
 import static org.hiero.mirror.common.util.DomainUtils.toEvmAddress;
 import static org.hiero.mirror.web3.utils.Constants.BALANCE_OPERATION_NAME;
 
+import com.hedera.hapi.streams.ContractAction;
+import com.hedera.hapi.streams.ContractActionType;
+import com.hedera.node.app.service.contract.impl.exec.ActionSidecarContentTracer;
 import jakarta.inject.Named;
+import java.util.List;
 import java.util.Optional;
 import lombok.CustomLog;
 import lombok.RequiredArgsConstructor;
@@ -17,13 +21,12 @@
 import org.hiero.mirror.web3.state.CommonEntityAccessor;
 import org.hyperledger.besu.evm.frame.MessageFrame;
 import org.hyperledger.besu.evm.operation.Operation;
-import org.hyperledger.besu.evm.tracing.OperationTracer;
 import org.jspecify.annotations.NonNull;
 
 @Named
 @CustomLog
 @RequiredArgsConstructor
-public class MirrorOperationActionTracer implements OperationTracer {
+public class MirrorOperationActionTracer implements ActionSidecarContentTracer {
 
     private final TraceProperties traceProperties;
     private final CommonEntityAccessor commonEntityAccessor;
@@ -76,4 +79,24 @@ public void tracePostExecution(
                 frame.getOutputData().toShortHexString(),
                 frame.getReturnData().toShortHexString());
     }
+
+    @Override
+    public void traceOriginAction(@NonNull MessageFrame frame) {
+        // NO-OP
+    }
+
+    @Override
+    public void sanitizeTracedActions(@NonNull MessageFrame frame) {
+        // NO-OP
+    }
+
+    @Override
+    public void tracePrecompileResult(@NonNull MessageFrame frame, @NonNull ContractActionType type) {
+        // NO-OP
+    }
+
+    @Override
+    public List<ContractAction> contractActions() {
+        return List.of();
+    }
 }