Skip to content

Commit 1ad583a

Browse files
kariyclaude
andcommitted
chore(sozo): clarify the account block id comment
PreConfirmed is the starknet-rs default; the comment now reflects that we're just being explicit rather than overriding it. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent ccf7d0c commit 1ad583a

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

  • bin/sozo/src/commands/options/account

bin/sozo/src/commands/options/account/mod.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -153,9 +153,8 @@ impl AccountOptions {
153153
let mut account =
154154
SingleOwnerAccount::new(provider, signer, account_address, chain_id, encoding);
155155

156-
// Default to the pre-confirmed block so nonce lookups and fee estimation
157-
// reflect the block the transaction will actually land in, rather than the
158-
// already-mined latest block (whose gas prices may be stale).
156+
// `PreConfirmed` is already the starknet-rs default; we set it
157+
// explicitly so the choice is visible and easy to override.
159158
let block_id = match &self.block_id {
160159
Some(s) => dojo_utils::parse_block_id(s.clone())?,
161160
None => BlockId::Tag(BlockTag::PreConfirmed),

0 commit comments

Comments
 (0)