fix(cli): return txid directly from ForCli methods, eliminate LAST_BROADCAST_TX_ID ThreadLocal

  Replace the thread-local side-channel (LAST_BROADCAST_TX_ID) with direct
  return values — all *ForCli methods now return the txid String (null on
  failure) instead of void/boolean. deployContractForCli returns
  Pair<contractAddress, txid>; callContractForCli returns
  Triple<String, Long, Long> with txid as the left element.

  Other fixes in this commit:
  - Use lenient JSON parsing in Utils.formatMessageString to tolerate
    malformed on-chain data (e.g. asset #436 description on Nile)
  - Add QA manifest entry for the help command (noauth-help type)
  - Fix test compilation: update overrides to match new signatures

Author: gummy789j

docs/standard-cli-contract-spec.md

@@ -752,6 +752,22 @@ Rules:
 - future command output expansion should prefer additive changes inside `data` rather than changing the top-level
   envelope shape
 
+### Broadcast Command Payload
+
+On-chain broadcast commands (send-coin, transfer-asset, trigger-contract, deploy-contract,
+freeze-balance, vote-witness, etc.) include transaction-specific fields under `data` when the
+broadcast succeeds.
+
+Rules:
+
+- a successful single-sign broadcast must include `"txid"` under `data`
+- a successful multi-sign submission must not include `"txid"` under `data` because multi-sign
+  submits a partially-signed transaction to a coordinator, not directly to the network; no txid
+  is generated at broadcast time
+- deploy-contract must additionally include `"contract_address"` (base58Check) under `data`
+- callers that require a txid must check that `data.txid` is present; its absence means the
+  transaction was submitted for multi-sign coordination
+
 ### Text and JSON Consistency
 
 Text mode and JSON mode are two renderings of the same command outcome.

qa/lib/case_resolver.py

@@ -94,6 +94,7 @@ def smoke_expectation(case_type):
     mapping = {
         "offline-success": "success",
         "noauth-success": "success",
+        "noauth-help": "help_dual",
         "auth-success": "success",
         "stateful-success": "success",
         "stateful-replay-execution": "stateful_replay_execution",
@@ -143,7 +144,8 @@ def build_smoke_case(label, row, values):
         "json_path_absent": json_path_absent,
         "error_code": error_code,
         "text_contains": text_contains,
-        "text_absent": ["Error:", "Usage:"] if smoke_expectation(case_type) == "success" else [],
+        "text_absent": ["Error:", "Usage:"] if smoke_expectation(case_type) == "success"
+                       else ["Error:"] if smoke_expectation(case_type) == "help_dual" else [],
         "preflight": preflight,
         "unresolved_placeholders": unresolved,
         "excluded": case_type == "excluded-interactive",

qa/manifest.tsv

@@ -100,4 +100,4 @@ clear-wallet-keystore|offline-success|auth|private_key|--force
 reset-wallet|offline-success|auth|private_key|--confirm delete-all-wallets
 modify-wallet-name|auth-success|auth|private_key|--name renamed-wallet
 generate-sub-account|offline-success|auth-mnemonic|mnemonic|--index 1 --name qasubaccount
-help|help_dual|empty|none|
+help|noauth-help|empty|none|

src/main/java/org/tron/common/utils/Utils.java

@@ -186,8 +186,15 @@ public static Date strToDateLong(String strDate) {
 
   public static String formatMessageString(Message message) {
     String result = JsonFormat.printToString(message, true);
-    return new GsonBuilder().setPrettyPrinting().create()
-        .toJson(JsonParser.parseString(result));
+    try {
+      com.google.gson.stream.JsonReader reader =
+          new com.google.gson.stream.JsonReader(new java.io.StringReader(result));
+      reader.setLenient(true);
+      return new GsonBuilder().setPrettyPrinting().create()
+          .toJson(JsonParser.parseReader(reader));
+    } catch (com.google.gson.JsonSyntaxException e) {
+      return result;
+    }
   }
 
   public static String printTransactionExceptId(Chain.Transaction transaction)