feat(TS): Integrate Zerion API for agentkit - #696

Closed
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main
Closed

feat(TS): Integrate Zerion API for agentkit#696
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main

Conversation

@pcminh0505

Copy link
Copy Markdown
Contributor

Description

  • Integrate 2 Zerion API endpoints for portfolio fetching
    • get_portfolio_overview: Get aggregated USD balance of a wallet address
    • get_fungible_positions: Get aggregated token positions (wallet and DeFi) of a wallet address

Requirement: Zerion API key

Example

  • get_portfolio_overview
Wallet Portfolio Overview:
- Total Value: $3267.53
- 24h Change: -0.63%
- Position Types: wallet: $497.17, deposited: $2138.66, staked: $631.65
- Top Chains: unichain: $2179.59, base: $815.98, ethereum: $270.11, polygon: $0.85, abstract: $0.58
  • get_fungible_positions
Total Value: $3762.01
Token Positions (>$1):
- ETH (Ethereum) on unichain — $1134.21 [deposit via Uniswap V4]
- USD₮0 (USD₮0) on unichain — $1004.22 [deposit via Uniswap V4]
- WELL (WELL) on base — $630.55 [staked via Moonwell]
- MORPHO (Morpho Token) on base — $497.49 [deposit via Moonwell]
- ETH (Ethereum) on ethereum — $268.30 [wallet]
- ETH (Ethereum) on base — $182.84 [wallet]
- ETH (Ethereum) on unichain — $40.94 [wallet]
- UNI (Uniswap) on ethereum — $1.92 [wallet]
- ALCH (Alchemist Accelerate) on base — $1.54 [wallet]

Tests

 PASS src/action-providers/zerion/zerionActionProvider.test.ts (8.381 s)
ZerionActionProvider
✓ should throw if no API key is provided (10 ms)
✓ should use provided API key from config
✓ should support the protocol family
✓ should not support other protocol families
✓ should handle invalid network objects
getPortfolioOverview
✓ returns error for invalid address
✓ returns formatted data for valid address (1 ms)
✓ returns error on fetch failure
getFungiblePositions
✓ returns error for invalid address (1 ms)
✓ returns formatted data for valid address
✓ returns error on fetch failure

Checklist

A couple of things to include in your PR for completeness:

  • Added documentation to all relevant README.md files
  • Added a changelog entry

* 0.1.0 initial scaffolding (#175)
* chore: fix lint (#177)
* chore: migrate basename action (#176)
* chore: CDP, Farcaster, ERC721 action providers (#181)
* CDP, Farcaster, ERC721 ActionProvider
* README erc721
* chore: migrate erc20 action (#182)
* 0.1.0(ts): morpho actions (#184)
* chore: migrate erc20 getBalance (#186)
* chore: fix npm scripts, linting and bump lint packages (#179)
* chore: migrate erc721 getBalance (#188)
* chore: migrate wrap eth action (#189)
* chore: langchain helper to convert actions to tools (#190)
* 0.1.0(ts): wow actions (#191)
* chore: migrate get wallet details (#192)
* 0.1.0(ts): twitter actions (#193)
* chore: eslint import order (#194)
* chore: map chainId to networkId (#196)
* chore: fix typo (#197)
* chore: migrate deploy token (#198)
* 0.1.0(ts): deploy contract (#201)
* first pass adding deploy contract cdp action
* linting
* the lost file
* tests
* 0.1.0(ts): adding eth balance to wallet details (#199)
* feat: Client side metrics (#195)
* 0.1.0(ts): deploy nft (#200)
* chore: rename typescript files and folders (#203)
* chore: split out cdp wallet and cdp api actions (#204)
* chore: migrate trade action (#208)
* twitter chatbot example (#205)
* 0.1.0(ts): farcaster example (#206)
* chore: cdpApiActionProvider usage (#211)
* chore: prefix action name (#210)
* chore: set default action provider to wallet action provider (#212)
* 0.1.0(ts): wallet native token transfer (#213)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* fix: use prefix notation that plays nicely with langchain (#217)
* feat: support custom action provider (#214)
* 0.1.0(ts): cdp wallet provider (#218)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* feat: CdpWalletProvider sendTransaction
* fix cdp provider config fields
* docs + dir structure refactor
* add language level readmes
* release ready!
* fix agentkit-langchain unit tests
* fix python unit test GHA poetry relative paths
* update poetry lock files
---------
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
Co-authored-by: Chris Gerber <csgerber@gmail.com>
disambuguates -> disambiguates
…d protocols (#256)
@murrlincoln Co-authored-by: Lincoln Murr <lincoln.murr@coinbase.com>
* add moonwell action providers
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* update: lint, update nomenclature, fix tests, add new constants
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
---------
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* Update README.md
* Update README.md
* Add files via upload
* Update README.md
* Update README.md
Co-authored-by: Christopher Gerber <christopher.gerber@coinbase.com>
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
@cb-heimdall

cb-heimdall commented May 1, 2025

Copy link
Copy Markdown

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation action provider New action provider typescript labels May 1, 2025
@chungquantin

Copy link
Copy Markdown

Looks good, hope this is merged!

@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the contribution! Looks pretty good, please rebase against main, then I will do some testing.

A few initial comments:

})
async getPortfolioOverview(args: z.infer<typeof GetWalletPortfolioSchema>): Promise<string> {
try {
const address = args.walletAddress || "";

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should fall back to walletProvider.address as states in schema. same for other action

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Hi @phdargen , imo it should be wallet-free tool (llke OpenSea). I already have an error check for Invalid wallet address below. My initial intention for Zerion API is portfolio check (with DeFi/fungible positions) for any wallet

@@ -0,0 +1,182 @@
/**

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove l1-9 to fit existing code style

* Replace or modify this with your actual action.
*
* @description
* This is a template action that demonstrates the basic structure.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

replace template description with actual action description

@github-actionsgithub-actionsBot added wallet provider New wallet provider example New example agent framework extension New framework extension python labels Aug 5, 2025
@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the update! Unfortunately the rebasing didnt work out as intended, it shows 71 commits now and hundreds of changed files. Could you please try to clean-up the commit history? Also your first 2 commits are still unsigned

@github-actionsgithub-actionsBot removed documentation Improvements or additions to documentation action provider New action provider wallet provider New wallet provider example New example agent framework extension New framework extension python typescript labels Aug 5, 2025
@pcminh0505

Copy link
Copy Markdown
ContributorAuthor

Hi @phdargen , my bad in rebasing the branch :( moved my changes to #816

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

20 participants

@pcminh0505@cb-heimdall@chungquantin@phdargen@John-peterson-coinbase@0xRAG@murrlincoln@thomasmetta@eltociear@stat@SahilAujla@kkomelin@debuggingfuture@Shelvak@ElliotFriedman@azf20@yuga-cb@Touutae-labs@ngundotra@CarsonRoscoe
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

feat(TS): Integrate Zerion API for agentkit - #696

Closed
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main
Closed

feat(TS): Integrate Zerion API for agentkit#696
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main

Conversation

@pcminh0505

Copy link
Copy Markdown
Contributor

Description

  • Integrate 2 Zerion API endpoints for portfolio fetching
    • get_portfolio_overview: Get aggregated USD balance of a wallet address
    • get_fungible_positions: Get aggregated token positions (wallet and DeFi) of a wallet address

Requirement: Zerion API key

Example

  • get_portfolio_overview
Wallet Portfolio Overview:
- Total Value: $3267.53
- 24h Change: -0.63%
- Position Types: wallet: $497.17, deposited: $2138.66, staked: $631.65
- Top Chains: unichain: $2179.59, base: $815.98, ethereum: $270.11, polygon: $0.85, abstract: $0.58
  • get_fungible_positions
Total Value: $3762.01
Token Positions (>$1):
- ETH (Ethereum) on unichain — $1134.21 [deposit via Uniswap V4]
- USD₮0 (USD₮0) on unichain — $1004.22 [deposit via Uniswap V4]
- WELL (WELL) on base — $630.55 [staked via Moonwell]
- MORPHO (Morpho Token) on base — $497.49 [deposit via Moonwell]
- ETH (Ethereum) on ethereum — $268.30 [wallet]
- ETH (Ethereum) on base — $182.84 [wallet]
- ETH (Ethereum) on unichain — $40.94 [wallet]
- UNI (Uniswap) on ethereum — $1.92 [wallet]
- ALCH (Alchemist Accelerate) on base — $1.54 [wallet]

Tests

 PASS src/action-providers/zerion/zerionActionProvider.test.ts (8.381 s)
ZerionActionProvider
✓ should throw if no API key is provided (10 ms)
✓ should use provided API key from config
✓ should support the protocol family
✓ should not support other protocol families
✓ should handle invalid network objects
getPortfolioOverview
✓ returns error for invalid address
✓ returns formatted data for valid address (1 ms)
✓ returns error on fetch failure
getFungiblePositions
✓ returns error for invalid address (1 ms)
✓ returns formatted data for valid address
✓ returns error on fetch failure

Checklist

A couple of things to include in your PR for completeness:

  • Added documentation to all relevant README.md files
  • Added a changelog entry

* 0.1.0 initial scaffolding (#175)
* chore: fix lint (#177)
* chore: migrate basename action (#176)
* chore: CDP, Farcaster, ERC721 action providers (#181)
* CDP, Farcaster, ERC721 ActionProvider
* README erc721
* chore: migrate erc20 action (#182)
* 0.1.0(ts): morpho actions (#184)
* chore: migrate erc20 getBalance (#186)
* chore: fix npm scripts, linting and bump lint packages (#179)
* chore: migrate erc721 getBalance (#188)
* chore: migrate wrap eth action (#189)
* chore: langchain helper to convert actions to tools (#190)
* 0.1.0(ts): wow actions (#191)
* chore: migrate get wallet details (#192)
* 0.1.0(ts): twitter actions (#193)
* chore: eslint import order (#194)
* chore: map chainId to networkId (#196)
* chore: fix typo (#197)
* chore: migrate deploy token (#198)
* 0.1.0(ts): deploy contract (#201)
* first pass adding deploy contract cdp action
* linting
* the lost file
* tests
* 0.1.0(ts): adding eth balance to wallet details (#199)
* feat: Client side metrics (#195)
* 0.1.0(ts): deploy nft (#200)
* chore: rename typescript files and folders (#203)
* chore: split out cdp wallet and cdp api actions (#204)
* chore: migrate trade action (#208)
* twitter chatbot example (#205)
* 0.1.0(ts): farcaster example (#206)
* chore: cdpApiActionProvider usage (#211)
* chore: prefix action name (#210)
* chore: set default action provider to wallet action provider (#212)
* 0.1.0(ts): wallet native token transfer (#213)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* fix: use prefix notation that plays nicely with langchain (#217)
* feat: support custom action provider (#214)
* 0.1.0(ts): cdp wallet provider (#218)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* feat: CdpWalletProvider sendTransaction
* fix cdp provider config fields
* docs + dir structure refactor
* add language level readmes
* release ready!
* fix agentkit-langchain unit tests
* fix python unit test GHA poetry relative paths
* update poetry lock files
---------
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
Co-authored-by: Chris Gerber <csgerber@gmail.com>
disambuguates -> disambiguates
…d protocols (#256)
@murrlincoln Co-authored-by: Lincoln Murr <lincoln.murr@coinbase.com>
* add moonwell action providers
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* update: lint, update nomenclature, fix tests, add new constants
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
---------
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* Update README.md
* Update README.md
* Add files via upload
* Update README.md
* Update README.md
Co-authored-by: Christopher Gerber <christopher.gerber@coinbase.com>
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
@cb-heimdall

cb-heimdall commented May 1, 2025

Copy link
Copy Markdown

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation action provider New action provider typescript labels May 1, 2025
@chungquantin

Copy link
Copy Markdown

Looks good, hope this is merged!

@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the contribution! Looks pretty good, please rebase against main, then I will do some testing.

A few initial comments:

})
async getPortfolioOverview(args: z.infer<typeof GetWalletPortfolioSchema>): Promise<string> {
try {
const address = args.walletAddress || "";

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should fall back to walletProvider.address as states in schema. same for other action

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Hi @phdargen , imo it should be wallet-free tool (llke OpenSea). I already have an error check for Invalid wallet address below. My initial intention for Zerion API is portfolio check (with DeFi/fungible positions) for any wallet

@@ -0,0 +1,182 @@
/**

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove l1-9 to fit existing code style

* Replace or modify this with your actual action.
*
* @description
* This is a template action that demonstrates the basic structure.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

replace template description with actual action description

@github-actionsgithub-actionsBot added wallet provider New wallet provider example New example agent framework extension New framework extension python labels Aug 5, 2025
@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the update! Unfortunately the rebasing didnt work out as intended, it shows 71 commits now and hundreds of changed files. Could you please try to clean-up the commit history? Also your first 2 commits are still unsigned

@github-actionsgithub-actionsBot removed documentation Improvements or additions to documentation action provider New action provider wallet provider New wallet provider example New example agent framework extension New framework extension python typescript labels Aug 5, 2025
@pcminh0505

Copy link
Copy Markdown
ContributorAuthor

Hi @phdargen , my bad in rebasing the branch :( moved my changes to #816

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

20 participants

@pcminh0505@cb-heimdall@chungquantin@phdargen@John-peterson-coinbase@0xRAG@murrlincoln@thomasmetta@eltociear@stat@SahilAujla@kkomelin@debuggingfuture@Shelvak@ElliotFriedman@azf20@yuga-cb@Touutae-labs@ngundotra@CarsonRoscoe
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(TS): Integrate Zerion API for agentkit - #696

Closed
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main
Closed

feat(TS): Integrate Zerion API for agentkit#696
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main

Conversation

@pcminh0505

Copy link
Copy Markdown
Contributor

Description

  • Integrate 2 Zerion API endpoints for portfolio fetching
    • get_portfolio_overview: Get aggregated USD balance of a wallet address
    • get_fungible_positions: Get aggregated token positions (wallet and DeFi) of a wallet address

Requirement: Zerion API key

Example

  • get_portfolio_overview
Wallet Portfolio Overview:
- Total Value: $3267.53
- 24h Change: -0.63%
- Position Types: wallet: $497.17, deposited: $2138.66, staked: $631.65
- Top Chains: unichain: $2179.59, base: $815.98, ethereum: $270.11, polygon: $0.85, abstract: $0.58
  • get_fungible_positions
Total Value: $3762.01
Token Positions (>$1):
- ETH (Ethereum) on unichain — $1134.21 [deposit via Uniswap V4]
- USD₮0 (USD₮0) on unichain — $1004.22 [deposit via Uniswap V4]
- WELL (WELL) on base — $630.55 [staked via Moonwell]
- MORPHO (Morpho Token) on base — $497.49 [deposit via Moonwell]
- ETH (Ethereum) on ethereum — $268.30 [wallet]
- ETH (Ethereum) on base — $182.84 [wallet]
- ETH (Ethereum) on unichain — $40.94 [wallet]
- UNI (Uniswap) on ethereum — $1.92 [wallet]
- ALCH (Alchemist Accelerate) on base — $1.54 [wallet]

Tests

 PASS src/action-providers/zerion/zerionActionProvider.test.ts (8.381 s)
ZerionActionProvider
✓ should throw if no API key is provided (10 ms)
✓ should use provided API key from config
✓ should support the protocol family
✓ should not support other protocol families
✓ should handle invalid network objects
getPortfolioOverview
✓ returns error for invalid address
✓ returns formatted data for valid address (1 ms)
✓ returns error on fetch failure
getFungiblePositions
✓ returns error for invalid address (1 ms)
✓ returns formatted data for valid address
✓ returns error on fetch failure

Checklist

A couple of things to include in your PR for completeness:

  • Added documentation to all relevant README.md files
  • Added a changelog entry

* 0.1.0 initial scaffolding (#175)
* chore: fix lint (#177)
* chore: migrate basename action (#176)
* chore: CDP, Farcaster, ERC721 action providers (#181)
* CDP, Farcaster, ERC721 ActionProvider
* README erc721
* chore: migrate erc20 action (#182)
* 0.1.0(ts): morpho actions (#184)
* chore: migrate erc20 getBalance (#186)
* chore: fix npm scripts, linting and bump lint packages (#179)
* chore: migrate erc721 getBalance (#188)
* chore: migrate wrap eth action (#189)
* chore: langchain helper to convert actions to tools (#190)
* 0.1.0(ts): wow actions (#191)
* chore: migrate get wallet details (#192)
* 0.1.0(ts): twitter actions (#193)
* chore: eslint import order (#194)
* chore: map chainId to networkId (#196)
* chore: fix typo (#197)
* chore: migrate deploy token (#198)
* 0.1.0(ts): deploy contract (#201)
* first pass adding deploy contract cdp action
* linting
* the lost file
* tests
* 0.1.0(ts): adding eth balance to wallet details (#199)
* feat: Client side metrics (#195)
* 0.1.0(ts): deploy nft (#200)
* chore: rename typescript files and folders (#203)
* chore: split out cdp wallet and cdp api actions (#204)
* chore: migrate trade action (#208)
* twitter chatbot example (#205)
* 0.1.0(ts): farcaster example (#206)
* chore: cdpApiActionProvider usage (#211)
* chore: prefix action name (#210)
* chore: set default action provider to wallet action provider (#212)
* 0.1.0(ts): wallet native token transfer (#213)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* fix: use prefix notation that plays nicely with langchain (#217)
* feat: support custom action provider (#214)
* 0.1.0(ts): cdp wallet provider (#218)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* feat: CdpWalletProvider sendTransaction
* fix cdp provider config fields
* docs + dir structure refactor
* add language level readmes
* release ready!
* fix agentkit-langchain unit tests
* fix python unit test GHA poetry relative paths
* update poetry lock files
---------
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
Co-authored-by: Chris Gerber <csgerber@gmail.com>
disambuguates -> disambiguates
…d protocols (#256)
@murrlincoln Co-authored-by: Lincoln Murr <lincoln.murr@coinbase.com>
* add moonwell action providers
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* update: lint, update nomenclature, fix tests, add new constants
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
---------
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* Update README.md
* Update README.md
* Add files via upload
* Update README.md
* Update README.md
Co-authored-by: Christopher Gerber <christopher.gerber@coinbase.com>
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
@cb-heimdall

cb-heimdall commented May 1, 2025

Copy link
Copy Markdown

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation action provider New action provider typescript labels May 1, 2025
@chungquantin

Copy link
Copy Markdown

Looks good, hope this is merged!

@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the contribution! Looks pretty good, please rebase against main, then I will do some testing.

A few initial comments:

})
async getPortfolioOverview(args: z.infer<typeof GetWalletPortfolioSchema>): Promise<string> {
try {
const address = args.walletAddress || "";

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should fall back to walletProvider.address as states in schema. same for other action

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Hi @phdargen , imo it should be wallet-free tool (llke OpenSea). I already have an error check for Invalid wallet address below. My initial intention for Zerion API is portfolio check (with DeFi/fungible positions) for any wallet

@@ -0,0 +1,182 @@
/**

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove l1-9 to fit existing code style

* Replace or modify this with your actual action.
*
* @description
* This is a template action that demonstrates the basic structure.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

replace template description with actual action description

@github-actionsgithub-actionsBot added wallet provider New wallet provider example New example agent framework extension New framework extension python labels Aug 5, 2025
@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the update! Unfortunately the rebasing didnt work out as intended, it shows 71 commits now and hundreds of changed files. Could you please try to clean-up the commit history? Also your first 2 commits are still unsigned

@github-actionsgithub-actionsBot removed documentation Improvements or additions to documentation action provider New action provider wallet provider New wallet provider example New example agent framework extension New framework extension python typescript labels Aug 5, 2025
@pcminh0505

Copy link
Copy Markdown
ContributorAuthor

Hi @phdargen , my bad in rebasing the branch :( moved my changes to #816

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

20 participants

@pcminh0505@cb-heimdall@chungquantin@phdargen@John-peterson-coinbase@0xRAG@murrlincoln@thomasmetta@eltociear@stat@SahilAujla@kkomelin@debuggingfuture@Shelvak@ElliotFriedman@azf20@yuga-cb@Touutae-labs@ngundotra@CarsonRoscoe
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(TS): Integrate Zerion API for agentkit - #696

Closed
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main
Closed

feat(TS): Integrate Zerion API for agentkit#696
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main

Conversation

@pcminh0505

Copy link
Copy Markdown
Contributor

Description

  • Integrate 2 Zerion API endpoints for portfolio fetching
    • get_portfolio_overview: Get aggregated USD balance of a wallet address
    • get_fungible_positions: Get aggregated token positions (wallet and DeFi) of a wallet address

Requirement: Zerion API key

Example

  • get_portfolio_overview
Wallet Portfolio Overview:
- Total Value: $3267.53
- 24h Change: -0.63%
- Position Types: wallet: $497.17, deposited: $2138.66, staked: $631.65
- Top Chains: unichain: $2179.59, base: $815.98, ethereum: $270.11, polygon: $0.85, abstract: $0.58
  • get_fungible_positions
Total Value: $3762.01
Token Positions (>$1):
- ETH (Ethereum) on unichain — $1134.21 [deposit via Uniswap V4]
- USD₮0 (USD₮0) on unichain — $1004.22 [deposit via Uniswap V4]
- WELL (WELL) on base — $630.55 [staked via Moonwell]
- MORPHO (Morpho Token) on base — $497.49 [deposit via Moonwell]
- ETH (Ethereum) on ethereum — $268.30 [wallet]
- ETH (Ethereum) on base — $182.84 [wallet]
- ETH (Ethereum) on unichain — $40.94 [wallet]
- UNI (Uniswap) on ethereum — $1.92 [wallet]
- ALCH (Alchemist Accelerate) on base — $1.54 [wallet]

Tests

 PASS src/action-providers/zerion/zerionActionProvider.test.ts (8.381 s)
ZerionActionProvider
✓ should throw if no API key is provided (10 ms)
✓ should use provided API key from config
✓ should support the protocol family
✓ should not support other protocol families
✓ should handle invalid network objects
getPortfolioOverview
✓ returns error for invalid address
✓ returns formatted data for valid address (1 ms)
✓ returns error on fetch failure
getFungiblePositions
✓ returns error for invalid address (1 ms)
✓ returns formatted data for valid address
✓ returns error on fetch failure

Checklist

A couple of things to include in your PR for completeness:

  • Added documentation to all relevant README.md files
  • Added a changelog entry

* 0.1.0 initial scaffolding (#175)
* chore: fix lint (#177)
* chore: migrate basename action (#176)
* chore: CDP, Farcaster, ERC721 action providers (#181)
* CDP, Farcaster, ERC721 ActionProvider
* README erc721
* chore: migrate erc20 action (#182)
* 0.1.0(ts): morpho actions (#184)
* chore: migrate erc20 getBalance (#186)
* chore: fix npm scripts, linting and bump lint packages (#179)
* chore: migrate erc721 getBalance (#188)
* chore: migrate wrap eth action (#189)
* chore: langchain helper to convert actions to tools (#190)
* 0.1.0(ts): wow actions (#191)
* chore: migrate get wallet details (#192)
* 0.1.0(ts): twitter actions (#193)
* chore: eslint import order (#194)
* chore: map chainId to networkId (#196)
* chore: fix typo (#197)
* chore: migrate deploy token (#198)
* 0.1.0(ts): deploy contract (#201)
* first pass adding deploy contract cdp action
* linting
* the lost file
* tests
* 0.1.0(ts): adding eth balance to wallet details (#199)
* feat: Client side metrics (#195)
* 0.1.0(ts): deploy nft (#200)
* chore: rename typescript files and folders (#203)
* chore: split out cdp wallet and cdp api actions (#204)
* chore: migrate trade action (#208)
* twitter chatbot example (#205)
* 0.1.0(ts): farcaster example (#206)
* chore: cdpApiActionProvider usage (#211)
* chore: prefix action name (#210)
* chore: set default action provider to wallet action provider (#212)
* 0.1.0(ts): wallet native token transfer (#213)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* fix: use prefix notation that plays nicely with langchain (#217)
* feat: support custom action provider (#214)
* 0.1.0(ts): cdp wallet provider (#218)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* feat: CdpWalletProvider sendTransaction
* fix cdp provider config fields
* docs + dir structure refactor
* add language level readmes
* release ready!
* fix agentkit-langchain unit tests
* fix python unit test GHA poetry relative paths
* update poetry lock files
---------
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
Co-authored-by: Chris Gerber <csgerber@gmail.com>
disambuguates -> disambiguates
…d protocols (#256)
@murrlincoln Co-authored-by: Lincoln Murr <lincoln.murr@coinbase.com>
* add moonwell action providers
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* update: lint, update nomenclature, fix tests, add new constants
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
---------
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* Update README.md
* Update README.md
* Add files via upload
* Update README.md
* Update README.md
Co-authored-by: Christopher Gerber <christopher.gerber@coinbase.com>
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
@cb-heimdall

cb-heimdall commented May 1, 2025

Copy link
Copy Markdown

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation action provider New action provider typescript labels May 1, 2025
@chungquantin

Copy link
Copy Markdown

Looks good, hope this is merged!

@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the contribution! Looks pretty good, please rebase against main, then I will do some testing.

A few initial comments:

})
async getPortfolioOverview(args: z.infer<typeof GetWalletPortfolioSchema>): Promise<string> {
try {
const address = args.walletAddress || "";

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should fall back to walletProvider.address as states in schema. same for other action

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Hi @phdargen , imo it should be wallet-free tool (llke OpenSea). I already have an error check for Invalid wallet address below. My initial intention for Zerion API is portfolio check (with DeFi/fungible positions) for any wallet

@@ -0,0 +1,182 @@
/**

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove l1-9 to fit existing code style

* Replace or modify this with your actual action.
*
* @description
* This is a template action that demonstrates the basic structure.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

replace template description with actual action description

@github-actionsgithub-actionsBot added wallet provider New wallet provider example New example agent framework extension New framework extension python labels Aug 5, 2025
@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the update! Unfortunately the rebasing didnt work out as intended, it shows 71 commits now and hundreds of changed files. Could you please try to clean-up the commit history? Also your first 2 commits are still unsigned

@github-actionsgithub-actionsBot removed documentation Improvements or additions to documentation action provider New action provider wallet provider New wallet provider example New example agent framework extension New framework extension python typescript labels Aug 5, 2025
@pcminh0505

Copy link
Copy Markdown
ContributorAuthor

Hi @phdargen , my bad in rebasing the branch :( moved my changes to #816

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

20 participants

@pcminh0505@cb-heimdall@chungquantin@phdargen@John-peterson-coinbase@0xRAG@murrlincoln@thomasmetta@eltociear@stat@SahilAujla@kkomelin@debuggingfuture@Shelvak@ElliotFriedman@azf20@yuga-cb@Touutae-labs@ngundotra@CarsonRoscoe
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

feat(TS): Integrate Zerion API for agentkit - #696

Closed
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main
Closed

feat(TS): Integrate Zerion API for agentkit#696
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main

Conversation

@pcminh0505

Copy link
Copy Markdown
Contributor

Description

  • Integrate 2 Zerion API endpoints for portfolio fetching
    • get_portfolio_overview: Get aggregated USD balance of a wallet address
    • get_fungible_positions: Get aggregated token positions (wallet and DeFi) of a wallet address

Requirement: Zerion API key

Example

  • get_portfolio_overview
Wallet Portfolio Overview:
- Total Value: $3267.53
- 24h Change: -0.63%
- Position Types: wallet: $497.17, deposited: $2138.66, staked: $631.65
- Top Chains: unichain: $2179.59, base: $815.98, ethereum: $270.11, polygon: $0.85, abstract: $0.58
  • get_fungible_positions
Total Value: $3762.01
Token Positions (>$1):
- ETH (Ethereum) on unichain — $1134.21 [deposit via Uniswap V4]
- USD₮0 (USD₮0) on unichain — $1004.22 [deposit via Uniswap V4]
- WELL (WELL) on base — $630.55 [staked via Moonwell]
- MORPHO (Morpho Token) on base — $497.49 [deposit via Moonwell]
- ETH (Ethereum) on ethereum — $268.30 [wallet]
- ETH (Ethereum) on base — $182.84 [wallet]
- ETH (Ethereum) on unichain — $40.94 [wallet]
- UNI (Uniswap) on ethereum — $1.92 [wallet]
- ALCH (Alchemist Accelerate) on base — $1.54 [wallet]

Tests

 PASS src/action-providers/zerion/zerionActionProvider.test.ts (8.381 s)
ZerionActionProvider
✓ should throw if no API key is provided (10 ms)
✓ should use provided API key from config
✓ should support the protocol family
✓ should not support other protocol families
✓ should handle invalid network objects
getPortfolioOverview
✓ returns error for invalid address
✓ returns formatted data for valid address (1 ms)
✓ returns error on fetch failure
getFungiblePositions
✓ returns error for invalid address (1 ms)
✓ returns formatted data for valid address
✓ returns error on fetch failure

Checklist

A couple of things to include in your PR for completeness:

  • Added documentation to all relevant README.md files
  • Added a changelog entry

* 0.1.0 initial scaffolding (#175)
* chore: fix lint (#177)
* chore: migrate basename action (#176)
* chore: CDP, Farcaster, ERC721 action providers (#181)
* CDP, Farcaster, ERC721 ActionProvider
* README erc721
* chore: migrate erc20 action (#182)
* 0.1.0(ts): morpho actions (#184)
* chore: migrate erc20 getBalance (#186)
* chore: fix npm scripts, linting and bump lint packages (#179)
* chore: migrate erc721 getBalance (#188)
* chore: migrate wrap eth action (#189)
* chore: langchain helper to convert actions to tools (#190)
* 0.1.0(ts): wow actions (#191)
* chore: migrate get wallet details (#192)
* 0.1.0(ts): twitter actions (#193)
* chore: eslint import order (#194)
* chore: map chainId to networkId (#196)
* chore: fix typo (#197)
* chore: migrate deploy token (#198)
* 0.1.0(ts): deploy contract (#201)
* first pass adding deploy contract cdp action
* linting
* the lost file
* tests
* 0.1.0(ts): adding eth balance to wallet details (#199)
* feat: Client side metrics (#195)
* 0.1.0(ts): deploy nft (#200)
* chore: rename typescript files and folders (#203)
* chore: split out cdp wallet and cdp api actions (#204)
* chore: migrate trade action (#208)
* twitter chatbot example (#205)
* 0.1.0(ts): farcaster example (#206)
* chore: cdpApiActionProvider usage (#211)
* chore: prefix action name (#210)
* chore: set default action provider to wallet action provider (#212)
* 0.1.0(ts): wallet native token transfer (#213)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* fix: use prefix notation that plays nicely with langchain (#217)
* feat: support custom action provider (#214)
* 0.1.0(ts): cdp wallet provider (#218)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* feat: CdpWalletProvider sendTransaction
* fix cdp provider config fields
* docs + dir structure refactor
* add language level readmes
* release ready!
* fix agentkit-langchain unit tests
* fix python unit test GHA poetry relative paths
* update poetry lock files
---------
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
Co-authored-by: Chris Gerber <csgerber@gmail.com>
disambuguates -> disambiguates
…d protocols (#256)
@murrlincoln Co-authored-by: Lincoln Murr <lincoln.murr@coinbase.com>
* add moonwell action providers
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* update: lint, update nomenclature, fix tests, add new constants
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
---------
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* Update README.md
* Update README.md
* Add files via upload
* Update README.md
* Update README.md
Co-authored-by: Christopher Gerber <christopher.gerber@coinbase.com>
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
@cb-heimdall

cb-heimdall commented May 1, 2025

Copy link
Copy Markdown

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation action provider New action provider typescript labels May 1, 2025
@chungquantin

Copy link
Copy Markdown

Looks good, hope this is merged!

@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the contribution! Looks pretty good, please rebase against main, then I will do some testing.

A few initial comments:

})
async getPortfolioOverview(args: z.infer<typeof GetWalletPortfolioSchema>): Promise<string> {
try {
const address = args.walletAddress || "";

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should fall back to walletProvider.address as states in schema. same for other action

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Hi @phdargen , imo it should be wallet-free tool (llke OpenSea). I already have an error check for Invalid wallet address below. My initial intention for Zerion API is portfolio check (with DeFi/fungible positions) for any wallet

@@ -0,0 +1,182 @@
/**

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove l1-9 to fit existing code style

* Replace or modify this with your actual action.
*
* @description
* This is a template action that demonstrates the basic structure.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

replace template description with actual action description

@github-actionsgithub-actionsBot added wallet provider New wallet provider example New example agent framework extension New framework extension python labels Aug 5, 2025
@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the update! Unfortunately the rebasing didnt work out as intended, it shows 71 commits now and hundreds of changed files. Could you please try to clean-up the commit history? Also your first 2 commits are still unsigned

@github-actionsgithub-actionsBot removed documentation Improvements or additions to documentation action provider New action provider wallet provider New wallet provider example New example agent framework extension New framework extension python typescript labels Aug 5, 2025
@pcminh0505

Copy link
Copy Markdown
ContributorAuthor

Hi @phdargen , my bad in rebasing the branch :( moved my changes to #816

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

20 participants

@pcminh0505@cb-heimdall@chungquantin@phdargen@John-peterson-coinbase@0xRAG@murrlincoln@thomasmetta@eltociear@stat@SahilAujla@kkomelin@debuggingfuture@Shelvak@ElliotFriedman@azf20@yuga-cb@Touutae-labs@ngundotra@CarsonRoscoe
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(TS): Integrate Zerion API for agentkit - #696

Closed
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main
Closed

feat(TS): Integrate Zerion API for agentkit#696
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main

Conversation

@pcminh0505

Copy link
Copy Markdown
Contributor

Description

  • Integrate 2 Zerion API endpoints for portfolio fetching
    • get_portfolio_overview: Get aggregated USD balance of a wallet address
    • get_fungible_positions: Get aggregated token positions (wallet and DeFi) of a wallet address

Requirement: Zerion API key

Example

  • get_portfolio_overview
Wallet Portfolio Overview:
- Total Value: $3267.53
- 24h Change: -0.63%
- Position Types: wallet: $497.17, deposited: $2138.66, staked: $631.65
- Top Chains: unichain: $2179.59, base: $815.98, ethereum: $270.11, polygon: $0.85, abstract: $0.58
  • get_fungible_positions
Total Value: $3762.01
Token Positions (>$1):
- ETH (Ethereum) on unichain — $1134.21 [deposit via Uniswap V4]
- USD₮0 (USD₮0) on unichain — $1004.22 [deposit via Uniswap V4]
- WELL (WELL) on base — $630.55 [staked via Moonwell]
- MORPHO (Morpho Token) on base — $497.49 [deposit via Moonwell]
- ETH (Ethereum) on ethereum — $268.30 [wallet]
- ETH (Ethereum) on base — $182.84 [wallet]
- ETH (Ethereum) on unichain — $40.94 [wallet]
- UNI (Uniswap) on ethereum — $1.92 [wallet]
- ALCH (Alchemist Accelerate) on base — $1.54 [wallet]

Tests

 PASS src/action-providers/zerion/zerionActionProvider.test.ts (8.381 s)
ZerionActionProvider
✓ should throw if no API key is provided (10 ms)
✓ should use provided API key from config
✓ should support the protocol family
✓ should not support other protocol families
✓ should handle invalid network objects
getPortfolioOverview
✓ returns error for invalid address
✓ returns formatted data for valid address (1 ms)
✓ returns error on fetch failure
getFungiblePositions
✓ returns error for invalid address (1 ms)
✓ returns formatted data for valid address
✓ returns error on fetch failure

Checklist

A couple of things to include in your PR for completeness:

  • Added documentation to all relevant README.md files
  • Added a changelog entry

* 0.1.0 initial scaffolding (#175)
* chore: fix lint (#177)
* chore: migrate basename action (#176)
* chore: CDP, Farcaster, ERC721 action providers (#181)
* CDP, Farcaster, ERC721 ActionProvider
* README erc721
* chore: migrate erc20 action (#182)
* 0.1.0(ts): morpho actions (#184)
* chore: migrate erc20 getBalance (#186)
* chore: fix npm scripts, linting and bump lint packages (#179)
* chore: migrate erc721 getBalance (#188)
* chore: migrate wrap eth action (#189)
* chore: langchain helper to convert actions to tools (#190)
* 0.1.0(ts): wow actions (#191)
* chore: migrate get wallet details (#192)
* 0.1.0(ts): twitter actions (#193)
* chore: eslint import order (#194)
* chore: map chainId to networkId (#196)
* chore: fix typo (#197)
* chore: migrate deploy token (#198)
* 0.1.0(ts): deploy contract (#201)
* first pass adding deploy contract cdp action
* linting
* the lost file
* tests
* 0.1.0(ts): adding eth balance to wallet details (#199)
* feat: Client side metrics (#195)
* 0.1.0(ts): deploy nft (#200)
* chore: rename typescript files and folders (#203)
* chore: split out cdp wallet and cdp api actions (#204)
* chore: migrate trade action (#208)
* twitter chatbot example (#205)
* 0.1.0(ts): farcaster example (#206)
* chore: cdpApiActionProvider usage (#211)
* chore: prefix action name (#210)
* chore: set default action provider to wallet action provider (#212)
* 0.1.0(ts): wallet native token transfer (#213)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* fix: use prefix notation that plays nicely with langchain (#217)
* feat: support custom action provider (#214)
* 0.1.0(ts): cdp wallet provider (#218)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* feat: CdpWalletProvider sendTransaction
* fix cdp provider config fields
* docs + dir structure refactor
* add language level readmes
* release ready!
* fix agentkit-langchain unit tests
* fix python unit test GHA poetry relative paths
* update poetry lock files
---------
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
Co-authored-by: Chris Gerber <csgerber@gmail.com>
disambuguates -> disambiguates
…d protocols (#256)
@murrlincoln Co-authored-by: Lincoln Murr <lincoln.murr@coinbase.com>
* add moonwell action providers
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* update: lint, update nomenclature, fix tests, add new constants
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
---------
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* Update README.md
* Update README.md
* Add files via upload
* Update README.md
* Update README.md
Co-authored-by: Christopher Gerber <christopher.gerber@coinbase.com>
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
@cb-heimdall

cb-heimdall commented May 1, 2025

Copy link
Copy Markdown

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation action provider New action provider typescript labels May 1, 2025
@chungquantin

Copy link
Copy Markdown

Looks good, hope this is merged!

@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the contribution! Looks pretty good, please rebase against main, then I will do some testing.

A few initial comments:

})
async getPortfolioOverview(args: z.infer<typeof GetWalletPortfolioSchema>): Promise<string> {
try {
const address = args.walletAddress || "";

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should fall back to walletProvider.address as states in schema. same for other action

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Hi @phdargen , imo it should be wallet-free tool (llke OpenSea). I already have an error check for Invalid wallet address below. My initial intention for Zerion API is portfolio check (with DeFi/fungible positions) for any wallet

@@ -0,0 +1,182 @@
/**

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove l1-9 to fit existing code style

* Replace or modify this with your actual action.
*
* @description
* This is a template action that demonstrates the basic structure.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

replace template description with actual action description

@github-actionsgithub-actionsBot added wallet provider New wallet provider example New example agent framework extension New framework extension python labels Aug 5, 2025
@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the update! Unfortunately the rebasing didnt work out as intended, it shows 71 commits now and hundreds of changed files. Could you please try to clean-up the commit history? Also your first 2 commits are still unsigned

@github-actionsgithub-actionsBot removed documentation Improvements or additions to documentation action provider New action provider wallet provider New wallet provider example New example agent framework extension New framework extension python typescript labels Aug 5, 2025
@pcminh0505

Copy link
Copy Markdown
ContributorAuthor

Hi @phdargen , my bad in rebasing the branch :( moved my changes to #816

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

20 participants

@pcminh0505@cb-heimdall@chungquantin@phdargen@John-peterson-coinbase@0xRAG@murrlincoln@thomasmetta@eltociear@stat@SahilAujla@kkomelin@debuggingfuture@Shelvak@ElliotFriedman@azf20@yuga-cb@Touutae-labs@ngundotra@CarsonRoscoe
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(TS): Integrate Zerion API for agentkit - #696

Closed
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main
Closed

feat(TS): Integrate Zerion API for agentkit#696
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main

Conversation

@pcminh0505

Copy link
Copy Markdown
Contributor

Description

  • Integrate 2 Zerion API endpoints for portfolio fetching
    • get_portfolio_overview: Get aggregated USD balance of a wallet address
    • get_fungible_positions: Get aggregated token positions (wallet and DeFi) of a wallet address

Requirement: Zerion API key

Example

  • get_portfolio_overview
Wallet Portfolio Overview:
- Total Value: $3267.53
- 24h Change: -0.63%
- Position Types: wallet: $497.17, deposited: $2138.66, staked: $631.65
- Top Chains: unichain: $2179.59, base: $815.98, ethereum: $270.11, polygon: $0.85, abstract: $0.58
  • get_fungible_positions
Total Value: $3762.01
Token Positions (>$1):
- ETH (Ethereum) on unichain — $1134.21 [deposit via Uniswap V4]
- USD₮0 (USD₮0) on unichain — $1004.22 [deposit via Uniswap V4]
- WELL (WELL) on base — $630.55 [staked via Moonwell]
- MORPHO (Morpho Token) on base — $497.49 [deposit via Moonwell]
- ETH (Ethereum) on ethereum — $268.30 [wallet]
- ETH (Ethereum) on base — $182.84 [wallet]
- ETH (Ethereum) on unichain — $40.94 [wallet]
- UNI (Uniswap) on ethereum — $1.92 [wallet]
- ALCH (Alchemist Accelerate) on base — $1.54 [wallet]

Tests

 PASS src/action-providers/zerion/zerionActionProvider.test.ts (8.381 s)
ZerionActionProvider
✓ should throw if no API key is provided (10 ms)
✓ should use provided API key from config
✓ should support the protocol family
✓ should not support other protocol families
✓ should handle invalid network objects
getPortfolioOverview
✓ returns error for invalid address
✓ returns formatted data for valid address (1 ms)
✓ returns error on fetch failure
getFungiblePositions
✓ returns error for invalid address (1 ms)
✓ returns formatted data for valid address
✓ returns error on fetch failure

Checklist

A couple of things to include in your PR for completeness:

  • Added documentation to all relevant README.md files
  • Added a changelog entry

* 0.1.0 initial scaffolding (#175)
* chore: fix lint (#177)
* chore: migrate basename action (#176)
* chore: CDP, Farcaster, ERC721 action providers (#181)
* CDP, Farcaster, ERC721 ActionProvider
* README erc721
* chore: migrate erc20 action (#182)
* 0.1.0(ts): morpho actions (#184)
* chore: migrate erc20 getBalance (#186)
* chore: fix npm scripts, linting and bump lint packages (#179)
* chore: migrate erc721 getBalance (#188)
* chore: migrate wrap eth action (#189)
* chore: langchain helper to convert actions to tools (#190)
* 0.1.0(ts): wow actions (#191)
* chore: migrate get wallet details (#192)
* 0.1.0(ts): twitter actions (#193)
* chore: eslint import order (#194)
* chore: map chainId to networkId (#196)
* chore: fix typo (#197)
* chore: migrate deploy token (#198)
* 0.1.0(ts): deploy contract (#201)
* first pass adding deploy contract cdp action
* linting
* the lost file
* tests
* 0.1.0(ts): adding eth balance to wallet details (#199)
* feat: Client side metrics (#195)
* 0.1.0(ts): deploy nft (#200)
* chore: rename typescript files and folders (#203)
* chore: split out cdp wallet and cdp api actions (#204)
* chore: migrate trade action (#208)
* twitter chatbot example (#205)
* 0.1.0(ts): farcaster example (#206)
* chore: cdpApiActionProvider usage (#211)
* chore: prefix action name (#210)
* chore: set default action provider to wallet action provider (#212)
* 0.1.0(ts): wallet native token transfer (#213)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* fix: use prefix notation that plays nicely with langchain (#217)
* feat: support custom action provider (#214)
* 0.1.0(ts): cdp wallet provider (#218)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* feat: CdpWalletProvider sendTransaction
* fix cdp provider config fields
* docs + dir structure refactor
* add language level readmes
* release ready!
* fix agentkit-langchain unit tests
* fix python unit test GHA poetry relative paths
* update poetry lock files
---------
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
Co-authored-by: Chris Gerber <csgerber@gmail.com>
disambuguates -> disambiguates
…d protocols (#256)
@murrlincoln Co-authored-by: Lincoln Murr <lincoln.murr@coinbase.com>
* add moonwell action providers
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* update: lint, update nomenclature, fix tests, add new constants
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
---------
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* Update README.md
* Update README.md
* Add files via upload
* Update README.md
* Update README.md
Co-authored-by: Christopher Gerber <christopher.gerber@coinbase.com>
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
@cb-heimdall

cb-heimdall commented May 1, 2025

Copy link
Copy Markdown

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation action provider New action provider typescript labels May 1, 2025
@chungquantin

Copy link
Copy Markdown

Looks good, hope this is merged!

@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the contribution! Looks pretty good, please rebase against main, then I will do some testing.

A few initial comments:

})
async getPortfolioOverview(args: z.infer<typeof GetWalletPortfolioSchema>): Promise<string> {
try {
const address = args.walletAddress || "";

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should fall back to walletProvider.address as states in schema. same for other action

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Hi @phdargen , imo it should be wallet-free tool (llke OpenSea). I already have an error check for Invalid wallet address below. My initial intention for Zerion API is portfolio check (with DeFi/fungible positions) for any wallet

@@ -0,0 +1,182 @@
/**

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove l1-9 to fit existing code style

* Replace or modify this with your actual action.
*
* @description
* This is a template action that demonstrates the basic structure.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

replace template description with actual action description

@github-actionsgithub-actionsBot added wallet provider New wallet provider example New example agent framework extension New framework extension python labels Aug 5, 2025
@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the update! Unfortunately the rebasing didnt work out as intended, it shows 71 commits now and hundreds of changed files. Could you please try to clean-up the commit history? Also your first 2 commits are still unsigned

@github-actionsgithub-actionsBot removed documentation Improvements or additions to documentation action provider New action provider wallet provider New wallet provider example New example agent framework extension New framework extension python typescript labels Aug 5, 2025
@pcminh0505

Copy link
Copy Markdown
ContributorAuthor

Hi @phdargen , my bad in rebasing the branch :( moved my changes to #816

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

20 participants

@pcminh0505@cb-heimdall@chungquantin@phdargen@John-peterson-coinbase@0xRAG@murrlincoln@thomasmetta@eltociear@stat@SahilAujla@kkomelin@debuggingfuture@Shelvak@ElliotFriedman@azf20@yuga-cb@Touutae-labs@ngundotra@CarsonRoscoe
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

feat(TS): Integrate Zerion API for agentkit - #696

Closed
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main
Closed

feat(TS): Integrate Zerion API for agentkit#696
pcminh0505 wants to merge 380 commits into
coinbase:mainfrom
growly-foundation:main

Conversation

@pcminh0505

Copy link
Copy Markdown
Contributor

Description

  • Integrate 2 Zerion API endpoints for portfolio fetching
    • get_portfolio_overview: Get aggregated USD balance of a wallet address
    • get_fungible_positions: Get aggregated token positions (wallet and DeFi) of a wallet address

Requirement: Zerion API key

Example

  • get_portfolio_overview
Wallet Portfolio Overview:
- Total Value: $3267.53
- 24h Change: -0.63%
- Position Types: wallet: $497.17, deposited: $2138.66, staked: $631.65
- Top Chains: unichain: $2179.59, base: $815.98, ethereum: $270.11, polygon: $0.85, abstract: $0.58
  • get_fungible_positions
Total Value: $3762.01
Token Positions (>$1):
- ETH (Ethereum) on unichain — $1134.21 [deposit via Uniswap V4]
- USD₮0 (USD₮0) on unichain — $1004.22 [deposit via Uniswap V4]
- WELL (WELL) on base — $630.55 [staked via Moonwell]
- MORPHO (Morpho Token) on base — $497.49 [deposit via Moonwell]
- ETH (Ethereum) on ethereum — $268.30 [wallet]
- ETH (Ethereum) on base — $182.84 [wallet]
- ETH (Ethereum) on unichain — $40.94 [wallet]
- UNI (Uniswap) on ethereum — $1.92 [wallet]
- ALCH (Alchemist Accelerate) on base — $1.54 [wallet]

Tests

 PASS src/action-providers/zerion/zerionActionProvider.test.ts (8.381 s)
ZerionActionProvider
✓ should throw if no API key is provided (10 ms)
✓ should use provided API key from config
✓ should support the protocol family
✓ should not support other protocol families
✓ should handle invalid network objects
getPortfolioOverview
✓ returns error for invalid address
✓ returns formatted data for valid address (1 ms)
✓ returns error on fetch failure
getFungiblePositions
✓ returns error for invalid address (1 ms)
✓ returns formatted data for valid address
✓ returns error on fetch failure

Checklist

A couple of things to include in your PR for completeness:

  • Added documentation to all relevant README.md files
  • Added a changelog entry

* 0.1.0 initial scaffolding (#175)
* chore: fix lint (#177)
* chore: migrate basename action (#176)
* chore: CDP, Farcaster, ERC721 action providers (#181)
* CDP, Farcaster, ERC721 ActionProvider
* README erc721
* chore: migrate erc20 action (#182)
* 0.1.0(ts): morpho actions (#184)
* chore: migrate erc20 getBalance (#186)
* chore: fix npm scripts, linting and bump lint packages (#179)
* chore: migrate erc721 getBalance (#188)
* chore: migrate wrap eth action (#189)
* chore: langchain helper to convert actions to tools (#190)
* 0.1.0(ts): wow actions (#191)
* chore: migrate get wallet details (#192)
* 0.1.0(ts): twitter actions (#193)
* chore: eslint import order (#194)
* chore: map chainId to networkId (#196)
* chore: fix typo (#197)
* chore: migrate deploy token (#198)
* 0.1.0(ts): deploy contract (#201)
* first pass adding deploy contract cdp action
* linting
* the lost file
* tests
* 0.1.0(ts): adding eth balance to wallet details (#199)
* feat: Client side metrics (#195)
* 0.1.0(ts): deploy nft (#200)
* chore: rename typescript files and folders (#203)
* chore: split out cdp wallet and cdp api actions (#204)
* chore: migrate trade action (#208)
* twitter chatbot example (#205)
* 0.1.0(ts): farcaster example (#206)
* chore: cdpApiActionProvider usage (#211)
* chore: prefix action name (#210)
* chore: set default action provider to wallet action provider (#212)
* 0.1.0(ts): wallet native token transfer (#213)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* fix: use prefix notation that plays nicely with langchain (#217)
* feat: support custom action provider (#214)
* 0.1.0(ts): cdp wallet provider (#218)
Co-authored-by: John Peterson <98187317+John-peterson-coinbase@users.noreply.github.com>
* feat: CdpWalletProvider sendTransaction
* fix cdp provider config fields
* docs + dir structure refactor
* add language level readmes
* release ready!
* fix agentkit-langchain unit tests
* fix python unit test GHA poetry relative paths
* update poetry lock files
---------
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
Co-authored-by: Chris Gerber <csgerber@gmail.com>
disambuguates -> disambiguates
…d protocols (#256)
@murrlincoln Co-authored-by: Lincoln Murr <lincoln.murr@coinbase.com>
* add moonwell action providers
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* update: lint, update nomenclature, fix tests, add new constants
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
---------
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
* Update README.md
* Update README.md
* Add files via upload
* Update README.md
* Update README.md
Co-authored-by: Christopher Gerber <christopher.gerber@coinbase.com>
Co-authored-by: Ryan Gilbert <ryangilbert7926@gmail.com>
@cb-heimdall

cb-heimdall commented May 1, 2025

Copy link
Copy Markdown

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation action provider New action provider typescript labels May 1, 2025
@chungquantin

Copy link
Copy Markdown

Looks good, hope this is merged!

@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the contribution! Looks pretty good, please rebase against main, then I will do some testing.

A few initial comments:

})
async getPortfolioOverview(args: z.infer<typeof GetWalletPortfolioSchema>): Promise<string> {
try {
const address = args.walletAddress || "";

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should fall back to walletProvider.address as states in schema. same for other action

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Hi @phdargen , imo it should be wallet-free tool (llke OpenSea). I already have an error check for Invalid wallet address below. My initial intention for Zerion API is portfolio check (with DeFi/fungible positions) for any wallet

@@ -0,0 +1,182 @@
/**

@phdargenphdargenJul 25, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove l1-9 to fit existing code style

* Replace or modify this with your actual action.
*
* @description
* This is a template action that demonstrates the basic structure.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

replace template description with actual action description

@github-actionsgithub-actionsBot added wallet provider New wallet provider example New example agent framework extension New framework extension python labels Aug 5, 2025
@phdargen

Copy link
Copy Markdown
Contributor

Hi @pcminh0505, thanks for the update! Unfortunately the rebasing didnt work out as intended, it shows 71 commits now and hundreds of changed files. Could you please try to clean-up the commit history? Also your first 2 commits are still unsigned

@github-actionsgithub-actionsBot removed documentation Improvements or additions to documentation action provider New action provider wallet provider New wallet provider example New example agent framework extension New framework extension python typescript labels Aug 5, 2025
@pcminh0505

Copy link
Copy Markdown
ContributorAuthor

Hi @phdargen , my bad in rebasing the branch :( moved my changes to #816

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

20 participants

@pcminh0505@cb-heimdall@chungquantin@phdargen@John-peterson-coinbase@0xRAG@murrlincoln@thomasmetta@eltociear@stat@SahilAujla@kkomelin@debuggingfuture@Shelvak@ElliotFriedman@azf20@yuga-cb@Touutae-labs@ngundotra@CarsonRoscoe