feat(API): refactor merge http servlets

.github/workflows/pr-build.yml

@@ -197,7 +197,7 @@ jobs:
             debian11-x86_64-gradle-
 
       - name: Build
-        run: ./gradlew clean build --no-daemon --no-build-cache
+        run: ./gradlew clean build --no-daemon
 
       - name: Toolkit jar smoke test
         run: |
@@ -209,7 +209,7 @@ jobs:
           java -jar "$JAR" keystore --help
 
       - name: Test with RocksDB engine
-        run: ./gradlew :framework:testWithRocksDb --no-daemon --no-build-cache
+        run: ./gradlew :framework:testWithRocksDb --no-daemon
 
       - name: Generate module coverage reports
         run: ./gradlew jacocoTestReport --no-daemon
@@ -265,11 +265,11 @@ jobs:
         # this PR.  The only output we need from this job is the jacoco XML for
         # coverage diffing, so we must not let a stale test failure block it.
         continue-on-error: true
-        run: ./gradlew clean build --no-daemon --no-build-cache
+        run: ./gradlew clean build --no-daemon
 
       - name: Test with RocksDB engine (base)
         continue-on-error: true
-        run: ./gradlew :framework:testWithRocksDb --no-daemon --no-build-cache
+        run: ./gradlew :framework:testWithRocksDb --no-daemon
 
       - name: Generate module coverage reports (base)
         run: ./gradlew jacocoTestReport --no-daemon

.github/workflows/pr-check.yml

@@ -43,9 +43,16 @@ jobs:
               errors.push(`PR title is too long (${title.length}/72 characters).`);
             }
 
-            // 2. Conventional format check
-            const conventionalRegex = /^(feat|fix|refactor|docs|style|test|chore|ci|perf|build|revert)(\([^)]+\))?:\s\S.*/;
-            if (title && !conventionalRegex.test(title)) {
+            // 2. Conventional format check (require a space after the colon)
+            const titlePrefix = '(?:feat|fix|refactor|docs|style|test|chore|ci|perf|build|revert)(?:[(][^)]+[)])?';
+            const missingSpaceAfterColonRegex = new RegExp(`^${titlePrefix}:[^ ]`);
+            const conventionalRegex = new RegExp(`^${titlePrefix}: [^ ].*`);
+            if (title && missingSpaceAfterColonRegex.test(title)) {
+              errors.push(
+                'PR title must include a space after the colon.\n' +
+                '  Example: `feat(tvm): add blob opcodes`'
+              );
+            } else if (title && !conventionalRegex.test(title)) {
               errors.push(
                 'PR title must follow conventional format: `type(scope): description`\n' +
                 '  Allowed types: ' + allowedTypes.map(t => `\`${t}\``).join(', ') + '\n' +
@@ -60,7 +67,7 @@ jobs:
 
             // 4. Description part should not start with a capital letter
             if (title) {
-              const descMatch = title.match(/^\w+(?:\([^)]+\))?:\s*(.+)/);
+              const descMatch = title.match(/^\w+(?:\([^)]+\))?: (.+)/);
               if (descMatch) {
                 const desc = descMatch[1];
                 if (/^[A-Z]/.test(desc)) {

.github/workflows/pr-reviewer.yml

@@ -59,13 +59,23 @@ jobs:
             const normalize = s => s.toLowerCase().replace(/[\s\-_]/g, '');
 
             // ── Extract scope from conventional commit title ──────────
-            // Format: type(scope): description
+            // Formats documented by CONTRIBUTING.md:
+            //   type(scope): description
+            //   type: description
             // Also supports: type(scope1,scope2): description
+            // Only bare "ci" currently has an equivalent reviewer scope.
             const scopeMatch = title.match(/^\w+\(([^)]+)\):/);
-            const rawScope = scopeMatch ? scopeMatch[1] : null;
+            const bareTypeMatch = title.match(/^(\w+):/);
+            const inferredScope = !scopeMatch && bareTypeMatch?.[1].toLowerCase() === 'ci'
+              ? 'ci'
+              : null;
+            const rawScope = scopeMatch ? scopeMatch[1] : inferredScope;
 
             core.info(`PR title : ${title}`);
             core.info(`Raw scope: ${rawScope || '(none)'}`);
+            if (inferredScope) {
+              core.info('Inferred scope "ci" from bare "ci" PR title type.');
+            }
 
             // ── Skip if reviewers already assigned ──────────────────
             const pr = await github.rest.pulls.get({

actuator/src/main/java/org/tron/core/actuator/VMActuator.java

@@ -299,6 +299,10 @@ public void execute(Object object) throws ContractExeException {
         result.setRuntimeError(result.getException().getMessage());
       }
       logger.info("runtime result is :{}", result.getException().getMessage());
+    } finally {
+      if (isConstantCall) {
+        VMConfig.clearLocalSnapshot();
+      }
     }
     //use program returned fill context
     context.setProgramResult(result);
@@ -309,10 +313,6 @@ public void execute(Object object) throws ContractExeException {
           .error(result.getException())
           .toString();
 
-      if (VMConfig.vmTraceCompressed()) {
-        traceContent = VMUtils.zipAndEncode(traceContent);
-      }
-
       String txHash = Hex.toHexString(rootInternalTx.getHash());
       VMUtils.saveProgramTraceFile(txHash, traceContent);
     }

actuator/src/main/java/org/tron/core/utils/ProposalUtil.java

@@ -941,6 +941,21 @@ public static void validator(DynamicPropertiesStore dynamicPropertiesStore,
         }
         break;
       }
+      case ALLOW_OPTIMIZE_TVM_STORAGE: {
+        if (!forkController.pass(ForkBlockVersionEnum.VERSION_4_8_2_3)) {
+          throw new ContractValidateException(
+              "Bad chain parameter id [ALLOW_OPTIMIZE_TVM_STORAGE]");
+        }
+        if (dynamicPropertiesStore.getAllowOptimizeTvmStorage() == 1) {
+          throw new ContractValidateException(
+              "[ALLOW_OPTIMIZE_TVM_STORAGE] has been valid, no need to propose again");
+        }
+        if (value != 1) {
+          throw new ContractValidateException(
+              "This value[ALLOW_OPTIMIZE_TVM_STORAGE] is only allowed to be 1");
+        }
+        break;
+      }
       default:
         break;
     }
@@ -1029,7 +1044,8 @@ public enum ProposalType {         // current value, value range
     ALLOW_TVM_PRAGUE(95), // 0, 1
     ALLOW_TVM_OSAKA(96), // 0, 1
     ALLOW_HARDEN_RESOURCE_CALCULATION(97), // 0, 1
-    ALLOW_HARDEN_EXCHANGE_CALCULATION(98); // 0, 1
+    ALLOW_HARDEN_EXCHANGE_CALCULATION(98), // 0, 1
+    ALLOW_OPTIMIZE_TVM_STORAGE(99); // 0, 1
     private long code;
 
     ProposalType(long code) {

actuator/src/main/java/org/tron/core/vm/PrecompiledContracts.java

@@ -982,9 +982,13 @@ public Pair<Boolean, byte[]> execute(byte[] data) {
       }
 
       PairingCheck check = PairingCheck.create();
+      long deadlineNs = pairingDeadlineNs();
 
       // iterating over all pairs
       for (int offset = 0; offset < data.length; offset += PAIR_SIZE) {
+        if (deadlineNs < System.nanoTime()) {
+          throw Program.Exception.notEnoughTime("precompiled contract pair");
+        }
 
         Pair<BN128G1, BN128G2> pair = decodePair(data, offset);
 
@@ -996,12 +1000,23 @@ public Pair<Boolean, byte[]> execute(byte[] data) {
         check.addPair(pair.getLeft(), pair.getRight());
       }
 
-      check.run();
+      if (!check.run(deadlineNs)) {
+        throw Program.Exception.notEnoughTime("precompiled contract pair");
+      }
       int result = check.result();
 
       return Pair.of(true, new DataWord(result).getData());
     }
 
+    /** Same skip as {@link Program#checkCPUTimeLimit}: debug and solidity nodes do not enforce it. */
+    private long pairingDeadlineNs() {
+      if (CommonParameter.getInstance().isDebug()
+          || CommonParameter.getInstance().isSolidityNode()) {
+        return Long.MAX_VALUE;
+      }
+      return getVmShouldEndInUs() * VMConstant.ONE_THOUSAND;
+    }
+
     private Pair<BN128G1, BN128G2> decodePair(byte[] in, int offset) {
 
       byte[] x = parseWord(in, offset, 0);

actuator/src/main/java/org/tron/core/vm/VMUtils.java

@@ -1,20 +1,14 @@
 package org.tron.core.vm;
 
 import static java.lang.String.format;
-import static org.apache.commons.codec.binary.Base64.encodeBase64String;
 import static org.tron.common.math.Maths.addExact;
 
-import java.io.ByteArrayInputStream;
-import java.io.ByteArrayOutputStream;
 import java.io.Closeable;
 import java.io.File;
 import java.io.FileOutputStream;
 import java.io.IOException;
-import java.io.InputStream;
 import java.io.OutputStream;
 import java.util.Arrays;
-import java.util.zip.Deflater;
-import java.util.zip.DeflaterOutputStream;
 import lombok.extern.slf4j.Slf4j;
 import org.tron.common.utils.ByteArray;
 import org.tron.common.utils.ByteUtil;
@@ -29,8 +23,6 @@
 @Slf4j(topic = "VM")
 public final class VMUtils {
 
-  private static final int BUF_SIZE = 4096;
-
   private VMUtils() {
   }
 
@@ -97,43 +89,6 @@ public static void saveProgramTraceFile(String txHash, String content) {
     }
   }
 
-  private static void write(InputStream in, OutputStream out, int bufSize) throws IOException {
-    try {
-      byte[] buf = new byte[bufSize];
-      for (int count = in.read(buf); count != -1; count = in.read(buf)) {
-        out.write(buf, 0, count);
-      }
-    } finally {
-      closeQuietly(in);
-      closeQuietly(out);
-    }
-  }
-
-  public static byte[] compress(byte[] bytes) throws IOException {
-    ByteArrayOutputStream baos = new ByteArrayOutputStream();
-
-    ByteArrayInputStream in = new ByteArrayInputStream(bytes);
-    DeflaterOutputStream out = new DeflaterOutputStream(baos, new Deflater(), BUF_SIZE);
-
-    write(in, out, BUF_SIZE);
-
-    return baos.toByteArray();
-  }
-
-  public static byte[] compress(String content) throws IOException {
-    return compress(content.getBytes("UTF-8"));
-  }
-
-  public static String zipAndEncode(String content) {
-    try {
-      return encodeBase64String(compress(content));
-    } catch (Exception e) {
-      logger.error("Cannot zip or encode: ", e);
-      return content;
-    }
-  }
-
-
   public static boolean validateForSmartContract(Repository deposit, byte[] ownerAddress,
       byte[] toAddress, long amount) throws ContractValidateException {
     if (!DecodeUtil.addressValid(ownerAddress)) {

actuator/src/main/java/org/tron/core/vm/config/ConfigLoader.java

@@ -50,6 +50,7 @@ public static void load(StoreFactory storeFactory, boolean isolate) {
         snapshot.allowTvmSelfdestructRestriction = ds.getAllowTvmSelfdestructRestriction() == 1;
         snapshot.allowTvmOsaka = ds.getAllowTvmOsaka() == 1;
         snapshot.allowHardenResourceCalculation = ds.getAllowHardenResourceCalculation() == 1;
+        snapshot.allowOptimizeTvmStorage = ds.getAllowOptimizeTvmStorage() == 1;
         if (isolate) {
           VMConfig.setLocalSnapshot(snapshot);
         } else {