Skip to content

Commit ac637b0

Browse files
sparrowDomclaude
andcommitted
Route Defender signer through the talos recorder, bump @talos/client
Defender's relayer manages nonce + gas + retries server-side, so the nonce-queue wrap is both broken (the SDK's signTransaction is a permanent stub) and unnecessary. Switch to the new wrapSignerWithDefenderRecorderV6 from @talos/client which forwards sendTransaction unchanged but inserts one nonce_queue_transactions row post-broadcast — the tx hash + run linkage still appear on talos's Transactions page. Bumps @talos/client to talos 060f99e for the new export. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 0c9a24c commit ac637b0

2 files changed

Lines changed: 16 additions & 6 deletions

File tree

pnpm-lock.yaml

Lines changed: 5 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/js/utils/signers.ts

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import * as hhHelpers from "@nomicfoundation/hardhat-network-helpers";
44
import {
55
createDb,
66
createPool,
7+
wrapSignerWithDefenderRecorderV6,
78
wrapSignerWithNonceQueueV6,
89
} from "@talos/client";
910

@@ -157,7 +158,16 @@ export async function getSigner(address?: string): Promise<Signer> {
157158
"Either uncheck the Defender option on this action or configure the env vars.",
158159
);
159160
}
160-
return maybeWrap(await getDefenderSigner());
161+
// Defender's relayer manages nonce + gas + retries server-side, so
162+
// the nonce-queue wrap is both broken (Defender can't sign offline)
163+
// and unnecessary. The recorder wrapper instead inserts a single row
164+
// into nonce_queue_transactions post-broadcast so the tx hash still
165+
// shows up on talos's Transactions page linked to its run.
166+
const signer = await getDefenderSigner();
167+
const db = getNonceDb();
168+
return db
169+
? wrapSignerWithDefenderRecorderV6(signer, { db })
170+
: signer;
161171
}
162172

163173
if (hasAwsKmsCredentials()) {

0 commit comments

Comments
 (0)