Skip to content

docs: add output format and error states to ERC20 action descriptions#1109

Open
abhicris wants to merge 1 commit intocoinbase:mainfrom
kcolbchain:fix/erc20-action-descriptions
Open

docs: add output format and error states to ERC20 action descriptions#1109
abhicris wants to merge 1 commit intocoinbase:mainfrom
kcolbchain:fix/erc20-action-descriptions

Conversation

@abhicris
Copy link
Copy Markdown

Summary

  • Adds Returns and Error cases sections to all five ERC20 action tool descriptions (get_balance, transfer, approve, get_allowance, get_erc20_token_address)
  • Updated in both TypeScript and Python implementations for parity
  • Adds context about approve-before-transfer workflow and gas fee requirements

This helps LLM agents understand what each action returns and how to handle error conditions, improving autonomous decision-making.

Addresses #1066

Test plan

  • No logic changes — description-only updates
  • Both TS and Python descriptions kept in sync
  • Verify descriptions render correctly in agent tool listings

🤖 Generated with Claude Code

Adds Returns and Error cases sections to all five ERC20 action tool
descriptions (get_balance, transfer, approve, get_allowance,
get_erc20_token_address) in both TypeScript and Python implementations.
This helps LLM agents understand what to expect from each action and
handle errors more reliably.

Addresses coinbase#1066

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@abhicris abhicris requested a review from murrlincoln as a code owner April 15, 2026 02:02
@cb-heimdall
Copy link
Copy Markdown

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants