Bump com.hedera.hashgraph:hedera-protobuf-java-api from 0.66.0 to 0.67.0

build.gradle.kts

@@ -50,9 +50,9 @@ dependencies {
         api("com.graphql-java-generator:graphql-java-client-runtime:3.0.1")
         api("com.graphql-java:graphql-java-extended-scalars:24.0")
         api("com.graphql-java:graphql-java-extended-validation:24.0")
-        api("com.hedera.hashgraph:app:0.66.0")
+        api("com.hedera.hashgraph:app:0.67.0")
         api("com.hedera.evm:hedera-evm:0.54.2")
-        api("com.hedera.hashgraph:hedera-protobuf-java-api:0.66.0")
+        api("com.hedera.hashgraph:hedera-protobuf-java-api:0.67.0")
         api("com.hedera.hashgraph:sdk:2.64.0")
         api("com.ongres.scram:client:2.1")
         api("com.salesforce.servicelibs:reactor-grpc-stub:$reactorGrpcVersion")

common/src/main/java/org/hiero/mirror/common/domain/transaction/StateChangeContext.java

@@ -21,7 +21,6 @@
 import java.util.ArrayList;
 import java.util.Comparator;
 import java.util.HashMap;
-import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
@@ -42,12 +41,12 @@ public final class StateChangeContext {
     private final Map<ByteString, ContractID> contractIds = new HashMap<>();
     private final Map<SlotKey, BytesValue> contractStorageChanges = new HashMap<>();
     private final Map<ContractID, List<SlotValue>> contractStorageChangesIndexed = new HashMap<>();
-    private final List<Long> nodeIds = new LinkedList<>();
-    private final List<FileID> fileIds = new LinkedList<>();
+    private final List<Long> nodeIds = new ArrayList<>();
+    private final List<FileID> fileIds = new ArrayList<>();
     private final Map<PendingAirdropId, Long> pendingFungibleAirdrops = new HashMap<>();
-    private final List<TokenID> tokenIds = new LinkedList<>();
+    private final List<TokenID> tokenIds = new ArrayList<>();
     private final Map<TokenID, Long> tokenTotalSupplies = new HashMap<>();
-    private final List<TopicID> topicIds = new LinkedList<>();
+    private final List<TopicID> topicIds = new ArrayList<>();
     private final Map<TopicID, TopicMessage> topicState = new HashMap<>();
 
     private StateChangeContext() {}
@@ -66,8 +65,8 @@ private StateChangeContext() {}
                     var mapUpdate = stateChange.getMapUpdate();
                     switch (stateChange.getStateId()) {
                         case StateIdentifier.STATE_ID_ACCOUNTS_VALUE -> processAccountStateChange(mapUpdate);
-                        case StateIdentifier.STATE_ID_CONTRACT_BYTECODE_VALUE -> processContractBytecode(mapUpdate);
-                        case StateIdentifier.STATE_ID_CONTRACT_STORAGE_VALUE -> processContractStorageChange(mapUpdate);
+                        case StateIdentifier.STATE_ID_BYTECODE_VALUE -> processContractBytecode(mapUpdate);
+                        case StateIdentifier.STATE_ID_STORAGE_VALUE -> processContractStorageChange(mapUpdate);
                         case StateIdentifier.STATE_ID_FILES_VALUE ->
                             fileIds.add(mapUpdate.getKey().getFileIdKey());
                         case StateIdentifier.STATE_ID_NODES_VALUE -> processNodeStateChange(mapUpdate);

common/src/test/java/org/hiero/mirror/common/domain/transaction/StateChangeContextTest.java

@@ -3,7 +3,7 @@
 package org.hiero.mirror.common.domain.transaction;
 
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_ACCOUNTS_VALUE;
-import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_CONTRACT_BYTECODE_VALUE;
+import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_BYTECODE_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_FILES_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_NFTS_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_NODES_VALUE;
@@ -121,7 +121,7 @@ void contractBytecode() {
         var bytecode = bytes(128);
         var stateChanges = StateChanges.newBuilder()
                 .addStateChanges(StateChange.newBuilder()
-                        .setStateId(STATE_ID_CONTRACT_BYTECODE_VALUE)
+                        .setStateId(STATE_ID_BYTECODE_VALUE)
                         .setMapUpdate(MapUpdateChange.newBuilder()
                                 .setKey(MapChangeKey.newBuilder().setContractIdKey(contractId))
                                 .setValue(MapChangeValue.newBuilder()

common/src/test/java/org/hiero/mirror/common/domain/transaction/StateChangeTestUtils.java

@@ -2,7 +2,7 @@
 
 package org.hiero.mirror.common.domain.transaction;
 
-import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_CONTRACT_STORAGE_VALUE;
+import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_STORAGE_VALUE;
 
 import com.google.protobuf.ByteString;
 import com.hedera.hapi.block.stream.output.protoc.MapChangeKey;
@@ -32,7 +32,7 @@ public static ByteString bytes(int size) {
 
     public static StateChange contractStorageMapDeleteChange(ContractID contractId, ByteString slot) {
         return StateChange.newBuilder()
-                .setStateId(STATE_ID_CONTRACT_STORAGE_VALUE)
+                .setStateId(STATE_ID_STORAGE_VALUE)
                 .setMapDelete(MapDeleteChange.newBuilder()
                         .setKey(MapChangeKey.newBuilder()
                                 .setSlotKeyKey(SlotKey.newBuilder()
@@ -43,7 +43,7 @@ public static StateChange contractStorageMapDeleteChange(ContractID contractId,
 
     public static StateChange contractStorageMapUpdateChange(ContractID contractId, ByteString slot, ByteString value) {
         return StateChange.newBuilder()
-                .setStateId(STATE_ID_CONTRACT_STORAGE_VALUE)
+                .setStateId(STATE_ID_STORAGE_VALUE)
                 .setMapUpdate(MapUpdateChange.newBuilder()
                         .setKey(MapChangeKey.newBuilder()
                                 .setSlotKeyKey(SlotKey.newBuilder()

importer/src/test/java/org/hiero/mirror/importer/downloader/block/transformer/ContractTransformerTest.java

@@ -3,7 +3,7 @@
 package org.hiero.mirror.importer.downloader.block.transformer;
 
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_ACCOUNTS_VALUE;
-import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_CONTRACT_BYTECODE_VALUE;
+import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_BYTECODE_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_PENDING_AIRDROPS_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_TOKENS_VALUE;
 import static org.assertj.core.api.Assertions.assertThat;
@@ -1091,7 +1091,7 @@ private Consumer<List<StateChanges>> stateChangesFromChildContractCreate(
                                                     .setAccountId(accountId)
                                                     .setSmartContract(true)))))
                     .addStateChanges(StateChange.newBuilder()
-                            .setStateId(STATE_ID_CONTRACT_BYTECODE_VALUE)
+                            .setStateId(STATE_ID_BYTECODE_VALUE)
                             .setMapUpdate(MapUpdateChange.newBuilder()
                                     .setKey(MapChangeKey.newBuilder().setContractIdKey(contractId))
                                     .setValue(MapChangeValue.newBuilder()

importer/src/test/java/org/hiero/mirror/importer/parser/domain/BlockTransactionBuilder.java

@@ -3,12 +3,12 @@
 package org.hiero.mirror.importer.parser.domain;
 
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_ACCOUNTS_VALUE;
-import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_CONTRACT_BYTECODE_VALUE;
-import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_CONTRACT_STORAGE_VALUE;
+import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_BYTECODE_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_NFTS_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_NODES_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_PENDING_AIRDROPS_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_SCHEDULES_BY_ID_VALUE;
+import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_STORAGE_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.StateIdentifier.STATE_ID_TOKENS_VALUE;
 import static com.hedera.hapi.block.stream.output.protoc.TransactionOutput.TransactionCase.ACCOUNT_CREATE;
 import static com.hedera.hapi.block.stream.output.protoc.TransactionOutput.TransactionCase.CONTRACT_CALL;
@@ -738,7 +738,7 @@ private void convertContractBytecode(
                         sidecarRecords, TransactionSidecarRecord::hasBytecode, TransactionSidecarRecord::getBytecode)
                 .ifPresent(bytecode -> {
                     stateChangesBuilder.addStateChanges(StateChange.newBuilder()
-                            .setStateId(STATE_ID_CONTRACT_BYTECODE_VALUE)
+                            .setStateId(STATE_ID_BYTECODE_VALUE)
                             .setMapUpdate(MapUpdateChange.newBuilder()
                                     .setKey(MapChangeKey.newBuilder().setContractIdKey(bytecode.getContractId()))
                                     .setValue(MapChangeValue.newBuilder()
@@ -780,7 +780,7 @@ private void convertContractStateChanges(
 
         // add an identical MapUpdateChange
         stateChangesBuilder.addStateChanges(StateChange.newBuilder()
-                .setStateId(STATE_ID_CONTRACT_STORAGE_VALUE)
+                .setStateId(STATE_ID_STORAGE_VALUE)
                 .setMapUpdate(MapUpdateChange.newBuilder()
                         .setIdentical(true)
                         .setKey(MapChangeKey.newBuilder()
@@ -816,7 +816,7 @@ private void convertContractStateChanges(
                             .build();
                     if (!BytesValue.getDefaultInstance().equals(storageChange.getValueWritten())) {
                         stateChangesBuilder.addStateChanges(StateChange.newBuilder()
-                                .setStateId(STATE_ID_CONTRACT_STORAGE_VALUE)
+                                .setStateId(STATE_ID_STORAGE_VALUE)
                                 .setMapUpdate(MapUpdateChange.newBuilder()
                                         .setKey(mapChangeKey)
                                         .setValue(MapChangeValue.newBuilder()
@@ -826,7 +826,7 @@ private void convertContractStateChanges(
                     } else {
                         // deleted
                         stateChangesBuilder.addStateChanges(StateChange.newBuilder()
-                                .setStateId(STATE_ID_CONTRACT_STORAGE_VALUE)
+                                .setStateId(STATE_ID_STORAGE_VALUE)
                                 .setMapDelete(MapDeleteChange.newBuilder().setKey(mapChangeKey)));
                     }
                 }

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

@@ -83,16 +83,20 @@
 
     private final HederaNativeOperations nativeOperations;
     private final ContractStateStore contractStateStore;
+    private final CodeFactory codeFactory;
 
     /**
-     * @param nativeOperations the Hedera native operation
+     * @param nativeOperations   the Hedera native operation
      * @param contractStateStore the contract store that manages the key/value states
+     * @param codeFactory the code factory to use
      */
     public DispatchingEvmFrameState(
             @NonNull final HederaNativeOperations nativeOperations,
-            @NonNull final ContractStateStore contractStateStore) {
+            @NonNull final ContractStateStore contractStateStore,
+            @NonNull final CodeFactory codeFactory) {
         this.nativeOperations = requireNonNull(nativeOperations);
         this.contractStateStore = requireNonNull(contractStateStore);
+        this.codeFactory = codeFactory;
     }
 
     /**
@@ -205,14 +209,15 @@
      * {@inheritDoc}
      */
     @Override
-    public @NonNull Hash getCodeHash(@NonNull final ContractID contractID) {
+    public @NonNull Hash getCodeHash(@NonNull final ContractID contractID, @NonNull final CodeFactory codeFactory) {
         requireNonNull(contractID);
 
         final var numberedBytecode = contractStateStore.getBytecode(contractID);
         if (numberedBytecode == null) {
             return Hash.EMPTY;
         } else {
-            return CodeFactory.createCode(pbjToTuweniBytes(numberedBytecode.code()), 0, false)
+            return codeFactory
+                    .createCode(pbjToTuweniBytes(numberedBytecode.code()), false)
                     .getCodeHash();
         }
     }
@@ -230,7 +235,8 @@
      */
     @Override
     public @NonNull Hash getTokenRedirectCodeHash(@NonNull final Address address) {
-        return CodeFactory.createCode(RedirectBytecodeUtils.tokenProxyBytecodeFor(address), 0, false)
+        return codeFactory
+                .createCode(RedirectBytecodeUtils.tokenProxyBytecodeFor(address), false)
                 .getCodeHash();
     }
 
@@ -247,7 +253,8 @@
      */
     @Override
     public @NonNull Hash getAccountRedirectCodeHash(@Nullable final Address address) {
-        return CodeFactory.createCode(RedirectBytecodeUtils.accountProxyBytecodeFor(address), 0, false)
+        return codeFactory
+                .createCode(RedirectBytecodeUtils.accountProxyBytecodeFor(address), false)
                 .getCodeHash();
     }
 
@@ -264,7 +271,8 @@
      */
     @Override
     public @NonNull Hash getScheduleRedirectCodeHash(@Nullable final Address address) {
-        return CodeFactory.createCode(RedirectBytecodeUtils.scheduleProxyBytecodeFor(address), 0, false)
+        return codeFactory
+                .createCode(RedirectBytecodeUtils.scheduleProxyBytecodeFor(address), false)
                 .getCodeHash();
     }
 
@@ -325,15 +333,15 @@
      * {@inheritDoc}
      */
     @Override
-    public Wei getBalance(AccountID accountID) {
+    public Wei getBalance(final AccountID accountID) {
         return Wei.of(validatedAccount(accountID).tinybarBalance());
     }
 
     /**
      * {@inheritDoc}
      */
     @Override
-    public long getIdNumber(@NonNull Address address) {
+    public long getIdNumber(@NonNull final Address address) {
         final var number = maybeMissingNumberOf(address, nativeOperations);
         if (number == MISSING_ENTITY_NUMBER) {
             throw new IllegalArgumentException("Address " + address + " has no associated Hedera id");
@@ -463,7 +471,8 @@
         }
         final var number = maybeMissingNumberOf(address, nativeOperations);
         if (number != MISSING_ENTITY_NUMBER) {
-            AccountID accountID = AccountID.newBuilder().accountNum(number).build();
+            final AccountID accountID =
+                    AccountID.newBuilder().accountNum(number).build();
             final var account = nativeOperations.getAccount(accountID);
             if (account != null) {
                 if (account.expiredAndPendingRemoval()) {
@@ -554,7 +563,7 @@
                 return null;
             }
             if (account.smartContract()) {
-                return new ProxyEvmContract(account.accountId(), this);
+                return new ProxyEvmContract(account.accountId(), this, codeFactory);
             } else {
                 return new ProxyEvmAccount(account.accountId(), this);
             }