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: add plugin-sui #934

Merged
merged 1 commit into from
Dec 14, 2024
Merged

Conversation

jnaulty
Copy link
Contributor

@jnaulty jnaulty commented Dec 9, 2024

Relates to:

#868

Risks

  • no new risks presented

Background

What does this PR do?

Introduce new Eliza Agent Plugin to manage a Sui Wallet

What kind of change is this?

Features (non-breaking change which adds functionality)

Why are we doing this? Any context or related work?

Add the ability for Eliza agents to interface with the Sui Blockchain.

Documentation changes needed?

Testing

Where should a reviewer start?

  • Goto packages/plugin-sui directory
  • Run pnpm i
  • Run pnpm test
stdout | src/tests/wallet.test.ts > WalletProvider > Wallet Integration > should check wallet address
Cache miss for fetchPortfolioValue
Cache miss for fetchPrices

stdout | src/tests/wallet.test.ts > WalletProvider > Wallet Integration > should check wallet address
Fetched portfolio: { totalUsd: '0.90126', totalSui: '0.9' }
Cache miss for fetchPrices

 ✓ src/tests/wallet.test.ts (1) 778ms
   ✓ WalletProvider (1) 777ms
     ✓ Wallet Integration (1) 777ms
       ✓ should check wallet address 776ms

 Test Files  1 passed (1)
      Tests  1 passed (1)
   Start at  00:37:43
   Duration  1.98s (transform 203ms, setup 0ms, collect 845ms, tests 778ms, environment 0ms, prepare 76ms)

Detailed testing steps

For e2e Test:

  • Replace Default Character Here
    // Replace the existing character initialization with custom setup
    const character = defaultCharacter;

    character.plugins = [suiPlugin];
    character.modelProvider = ModelProviderName.OPENAI;
    let characters = [character];

Discord Username

@crypto__john

@jnaulty jnaulty force-pushed the jnaulty/add-sui-plugin branch 2 times, most recently from 962edae to af04019 Compare December 9, 2024 08:51
@jnaulty jnaulty changed the title add plugin-sui feat: add plugin-sui Dec 9, 2024
@jnaulty jnaulty marked this pull request as ready for review December 9, 2024 09:19
@jnaulty
Copy link
Contributor Author

jnaulty commented Dec 10, 2024

Updated to generateObjectV2

Tested out and transferred 1 SUI ALmxao2hvCRoxHc6x8vMw9JxbUceBLruR98W9bhzXJx

image

odilitime
odilitime previously approved these changes Dec 10, 2024
@jnaulty jnaulty force-pushed the jnaulty/add-sui-plugin branch from d0fb057 to 00767d3 Compare December 10, 2024 17:46
@odilitime odilitime added the Plugin_new Mark PRs that are a new plugin label Dec 10, 2024
@jnaulty jnaulty force-pushed the jnaulty/add-sui-plugin branch from 00767d3 to 0275c5e Compare December 11, 2024 01:18
@jnaulty
Copy link
Contributor Author

jnaulty commented Dec 11, 2024

Looks like there's a hold on merging new plugins due to high-volume:
image

https://discord.com/channels/1253563208833433701/1316266980118106133/1316267190697070593

@jnaulty jnaulty force-pushed the jnaulty/add-sui-plugin branch 3 times, most recently from 8c47475 to 5032281 Compare December 12, 2024 07:20
@jnaulty jnaulty changed the base branch from main to develop December 12, 2024 07:22
@jnaulty jnaulty force-pushed the jnaulty/add-sui-plugin branch 2 times, most recently from 4260c45 to 687aecc Compare December 12, 2024 07:25
@jnaulty jnaulty requested a review from odilitime December 12, 2024 07:26
@odilitime odilitime deleted the branch elizaOS:develop December 13, 2024 02:37
@odilitime odilitime closed this Dec 13, 2024
@odilitime odilitime reopened this Dec 13, 2024
@jnaulty jnaulty force-pushed the jnaulty/add-sui-plugin branch from 687aecc to 9bcd489 Compare December 13, 2024 05:56
@odilitime odilitime deleted the branch elizaOS:develop December 13, 2024 19:07
@odilitime odilitime closed this Dec 13, 2024
@odilitime odilitime reopened this Dec 13, 2024
- add walletProvider for Sui
- add transferSui action
@jnaulty jnaulty force-pushed the jnaulty/add-sui-plugin branch from 9bcd489 to b7bf5c1 Compare December 14, 2024 06:08
@lalalune lalalune merged commit 8f767c1 into elizaOS:develop Dec 14, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Plugin_new Mark PRs that are a new plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants