Skip to content

Address cursorbot bugs#418

Merged
peachbits merged 1 commit intomasterfrom
matthew/cursor-fixes
Nov 4, 2025
Merged

Address cursorbot bugs#418
peachbits merged 1 commit intomasterfrom
matthew/cursor-fixes

Conversation

@peachbits
Copy link
Contributor

@peachbits peachbits commented Nov 4, 2025

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Description

none

Note

Send ERC-20 approvals to the recipient/spender address and safely handle non-standard token lists with optional chaining.

  • EVM token approvals (src/swap/defi/defiUtils.ts):
    • Direct approve calls to recipientAddress instead of tokenContractAddress when building approval data.
    • Use optional chaining in NON_STANDARD_APPROVAL_TOKENS[pluginId]?.includes(...) to avoid errors when a plugin ID key is absent.

Written by Cursor Bugbot for commit f0db3b4. This will update automatically on new commits. Configure here.


@peachbits peachbits merged commit fb83fec into master Nov 4, 2025
2 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.

2 participants