feat: Add Nebula AI chat and transaction execution API #5948
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: 19cbb18 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
| import { getContract } from "../contract/contract.js"; | ||
| import * as Nebula from "./index.js"; | ||
|
|
||
| describe("chat", () => { |
There was a problem hiding this comment.
The test file execute.test.ts has a describe block labeled as "chat", but since these tests are validating the execute functionality, the block should be labeled as "execute" to accurately reflect its purpose.
Spotted by Graphite Reviewer
Is this helpful? React 👍 or 👎 to let us know.
packages/thirdweb/src/ai/common.ts
Outdated
| transactions = data.actions.map((action) => { | ||
| const tx = JSON.parse(action.data) as { | ||
| chainId: number; | ||
| to: Address | undefined; | ||
| value: Hex; | ||
| data: Hex; | ||
| }; | ||
| return prepareTransaction({ |
There was a problem hiding this comment.
The action.type check is missing before parsing transaction data. Since action.type can be 'init', 'presence', or 'sign_transaction', and each type may have different data structures, add a guard to ensure only 'sign_transaction' actions are processed as transactions. This will prevent potential runtime errors from parsing incompatible data formats.
Spotted by Graphite Reviewer
Is this helpful? React 👍 or 👎 to let us know.
joaquim-verges
force-pushed
the
feat_Add_Nebula_AI_chat_and_transaction_execution_API
branch
from
543f9ec to
07a59a2
Compare
joaquim-verges
force-pushed
the
feat_Add_Nebula_AI_chat_and_transaction_execution_API
branch
from
07a59a2 to
1017315
Compare
size-limit report 📦
|
joaquim-verges
force-pushed
the
feat_Add_Nebula_AI_chat_and_transaction_execution_API
branch
from
1017315 to
19cbb18
Compare
github-actions
bot
added
the
Portal
joaquim-verges
deleted the
feat_Add_Nebula_AI_chat_and_transaction_execution_API
branch
Fixes TOOL-3063
How to test
pnpm test:dev chat.test.tspnpm test:dev execute.test.tsPR-Codex overview
This PR introduces the
NebulaAPI, enabling users to interact with it for chat and transaction execution functionalities. It enhances thethirdwebpackage with new methods and updates documentation for better usage examples.Detailed summary
Nebulaexport inpackages/thirdweb/src/exports/ai.ts.chatandexecutefunctions inpackages/thirdweb/src/ai/chat.tsandpackages/thirdweb/src/ai/execute.ts.packages/thirdweb/src/ai/common.tswithInputandOutputtypes.chatandexecuteinpackages/thirdweb/src/ai/chat.test.tsandpackages/thirdweb/src/ai/execute.test.ts.typedoc.mjsto include the newsrc/ai/index.ts.packages/thirdweb/package.jsonto includeaitypes and imports.NebulaAPI usage examples inchangeset/warm-dodos-destroy.md.