Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Improve ability to distinguish between networks, assets, and trade types #19

Merged
merged 10 commits into from
Dec 6, 2024

Conversation

derek-cb
Copy link
Contributor

@derek-cb derek-cb commented Dec 5, 2024

What changed? Why?

Improved the agent's ability to recognize assets (ie, USDC, DAI) without further prompting in specific cases whereby network was not explicitly queried beforehand by the agent. Also better ability to distinguish between WOW and non-WOW type asset trading.

  • Improved prompts for CDP actions (more specificity surrounding WOW vs non-WOW trades)
  • Formatted prompts for better legibility
  • Fixed issue whereby chatbot didn't know what network it was on by default
  • Added chatbot to local lib workspace for easier local development

Qualified Impact

…vs non-WOW trades

- Formatted prompts for better legibility
- Fixed issue whereby chatbot didn't know what network it was on by default
- Added chatbot to local lib workspace for easier local development
@derek-cb derek-cb marked this pull request as ready for review December 5, 2024 19:26
Copy link
Contributor

@John-peterson-coinbase John-peterson-coinbase left a comment

Choose a reason for hiding this comment

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

These look reasonable. Please update CHANGELOG.md's

- Remove chatbot example from workspace
- Add 'bonding curve token' to prompts for WOW actions
- Remove chatbot example from workspace
- Add 'bonding curve token' to prompts for WOW actions
@@ -1,5 +1,6 @@
{
"name": "cdp-agentkit-monorepo",
"private": true,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

added this to prevent accidental publishing of the root-level package

@derek-cb derek-cb merged commit 3d23153 into master Dec 6, 2024
6 checks passed
@derek-cb derek-cb deleted the improve-prompts branch December 6, 2024 21:10
stat pushed a commit that referenced this pull request Dec 6, 2024
…ade types (#19)

* - Improved prompts for CDP actions (more specificity surrounding WOW vs non-WOW trades
- Formatted prompts for better legibility
- Fixed issue whereby chatbot didn't know what network it was on by default
- Added chatbot to local lib workspace for easier local development

* Changelog update

* Added chatbot changelog, updated cdp-langchain changelog

* Remove unnecessary import from root-level package.json

* No need to increment chatbot version (not publishing at the moment)

* Implementing feedback:
- Remove chatbot example from workspace
- Add 'bonding curve token' to prompts for WOW actions

* Implementing feedback:
- Remove chatbot example from workspace
- Add 'bonding curve token' to prompts for WOW actions

* Stripping out AgentKit version upgrades

* Revert chatbot package-lock.json
stat pushed a commit that referenced this pull request Dec 6, 2024
…ade types (#19)

* - Improved prompts for CDP actions (more specificity surrounding WOW vs non-WOW trades
- Formatted prompts for better legibility
- Fixed issue whereby chatbot didn't know what network it was on by default
- Added chatbot to local lib workspace for easier local development

* Changelog update

* Added chatbot changelog, updated cdp-langchain changelog

* Remove unnecessary import from root-level package.json

* No need to increment chatbot version (not publishing at the moment)

* Implementing feedback:
- Remove chatbot example from workspace
- Add 'bonding curve token' to prompts for WOW actions

* Implementing feedback:
- Remove chatbot example from workspace
- Add 'bonding curve token' to prompts for WOW actions

* Stripping out AgentKit version upgrades

* Revert chatbot package-lock.json
Sign up for free to 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.

2 participants