Skip to content

Conversation

@phdargen
Copy link
Contributor

create-onchain-agent next template (smart wallet) build fails with:

./app/api/agent/prepare-agentkit.ts:105:7
Type error: Object literal may only specify known properties, and 'smartWalletAddress' does not exist in type 'CdpSmartWalletProviderConfig'.

  103 |       // eslint-disable-next-line @typescript-eslint/no-explicit-any
  104 |       owner: owner as any,
 105 |       smartWalletAddress: walletData?.smartWalletAddress,
  106 |       paymasterUrl: undefined, // Sponsor transactions: https://docs.cdp.coinbase.com/paymaster/docs/welcome
  107 |     });
  108 |

This PR fixes the config following the implementation in the corresponding chatbot example: https://github.com/coinbase/agentkit/blob/main/typescript/examples/langchain-cdp-smart-wallet-chatbot/chatbot.ts

@phdargen phdargen requested a review from murrlincoln as a code owner July 30, 2025 18:51
@cb-heimdall
Copy link

cb-heimdall commented Jul 30, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/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 0
Sum 1

@phdargen phdargen marked this pull request as draft July 30, 2025 18:51
@github-actions github-actions bot added documentation Improvements or additions to documentation typescript labels Jul 30, 2025
@CarsonRoscoe CarsonRoscoe marked this pull request as ready for review July 31, 2025 00:00
Copy link
Contributor

@CarsonRoscoe CarsonRoscoe left a comment

Choose a reason for hiding this comment

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

Great catch, ty. LGTM!

@CarsonRoscoe CarsonRoscoe merged commit 082163f into coinbase:main Jul 31, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation typescript

Development

Successfully merging this pull request may close these issues.

3 participants