docs: expand agent multichain safety guidance - #997
Closed
grayfoxd wants to merge 1 commit into
Closed
Conversation
gummy789j
reviewed
Sep 3, 2026
Comment on lines
+202
to
+208
| The catalog deliberately does not prescribe retry behavior. A transport `timeout` or `rpc_error` | ||
| only says that the CLI did not observe a successful response. For a read-only call, retrying the same | ||
| request is normally harmless. On a submit path, however, the node may have accepted the transaction | ||
| before the response was lost. Treat that outcome as unknown: reconcile by `txId` when available, or | ||
| by sender and nonce for EVM, before building and signing another transaction. Re-running a send does | ||
| not retry the same transaction; it creates a second transaction and can duplicate a payment. | ||
|
|
Collaborator
There was a problem hiding this comment.
There is a section more clearly to explain the same thing
retry describes the error, not the command. timeout and rpc_error are same because
for most calls that is correct — the node never acted, so resending is free. But a command that
may have already broadcast a transaction (tx send and anything else on the submit path) can hit
timeout or rpc_error after the node accepted the transaction and before the response made it
back. In that case the outcome is unknown, not failed, and resending does not retry the original
request — it builds and signs a new transaction, which on TRON is a second, distinct transfer.
retry: "same" is correct for a timeout/rpc_error that happens while resolving a network id or
reading a balance; it is not a license to resend a broadcast blind. Reconcile with
[tx status](https://github.com/tronprotocol/wallet-cli/pull/997/changes#script-safety-never-mistake-submitted-for-confirmed) before deciding whether to
retry, exactly as the four-state model below requires.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.