Skip to content

3.0.9#530

Merged
Dargon789 merged 0 commit into
Dargon789:mainfrom
0xsequence:master
May 8, 2026
Merged

3.0.9#530
Dargon789 merged 0 commit into
Dargon789:mainfrom
0xsequence:master

Conversation

@Dargon789
Copy link
Copy Markdown
Owner

No description provided.

@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @Dargon789, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented May 8, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@mergify
Copy link
Copy Markdown

mergify Bot commented May 8, 2026

⚠️ The sha of the head commit of this PR conflicts with #529. Mergify cannot evaluate rules on this PR. Once #529 is merged or closed, Mergify will resume processing this PR. ⚠️

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates several packages to version 3.0.9 and includes fixes for relayer fee options. Specifically, it replaces a hardcoded stub signature in wallet-core with a dynamically generated one based on the wallet's topology to improve fee estimation. Feedback suggests extending the stubFeeOptionsTopology function to support SapientSignerLeaf to ensure accurate gas limits for all wallet configurations.

Comment thread packages/wallet/core/src/wallet.ts
@Dargon789 Dargon789 merged commit 87cf349 into Dargon789:main May 8, 2026
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant