Skip to content

Version Packages #5955

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

Merged
merged 1 commit into from
Jan 18, 2025
Merged

Version Packages #5955

merged 1 commit into from
Jan 18, 2025

Conversation

jnsdls
Copy link
Member

@jnsdls jnsdls commented Jan 15, 2025

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.

Releases

@thirdweb-dev/service-utils@0.6.0

Minor Changes

thirdweb@5.85.0

Minor Changes

  • #5972 0b62397 Thanks @joaquim-verges! - Support multiple messages for Nebula API, updated input props.

    Some prop names have been updated:

    prompt -> messsage
    context -> contextFilter

    Nebula.chat({
      client,
      // prompt is now message
      message:
        "What's the total supply of this contract: 0xe2cb0eb5147b42095c2FfA6F7ec953bb0bE347D8",
      // context is now contextFilter
      contextFilter: {
        chains: [sepolia],
      },
    });

    The Nebula.chat and Nebula.execute functions now support multiple input messages, and the input properties have been updated to match the http API.

    Nebula.chat({
      client,
      // multi message format
      messages: [
        {
          role: "user",
          content:
            "Tell me the name of this contract: 0xe2cb0eb5147b42095c2FfA6F7ec953bb0bE347D8",
        },
        {
          role: "assistant",
          content: "The name of the contract is My NFT Collection",
        },
        {
          role: "user",
          content: "What's the symbol of this contract?",
        },
      ],
      contextFilter: {
        chains: [sepolia],
      },
    });

    Same changes apply to Nebula.execute.

    Nebula.execute({
      client,
      account,
      messages: [
        { role: "user", content: "What's the address of vitalik.eth" },
        {
          role: "assistant",
          content:
            "The address of vitalik.eth is 0xd8dA6BF26964aF8E437eEa5e3616511D7G3a3298",
        },
        { role: "user", content: "Send them 0.0001 ETH" },
      ],
      contextFilter: {
        chains: [sepolia],
      },
    });

Patch Changes

@thirdweb-dev/wagmi-adapter@0.1.9


PR-Codex overview

This PR focuses on updating version numbers for several packages, enhancing the Nebula API functionality, and fixing issues related to NFT components. It also includes changes to the CHANGELOG.md files to reflect these updates.

Detailed summary

  • Updated version in packages/thirdweb/package.json from 5.84.0 to 5.85.0.
  • Updated version in packages/service-utils/package.json from 0.5.1 to 0.6.0.
  • Updated version in packages/wagmi-adapter/package.json from 0.1.8 to 0.1.9.
  • Added new entries in CHANGELOG.md for 0.6.0 in packages/service-utils.
  • Added new entries in CHANGELOG.md for 5.85.0 in packages/thirdweb.
  • Enhanced Nebula.chat and Nebula.execute to support multiple input messages.
  • Updated input properties for Nebula.chat from prompt to message and context to contextFilter.
  • Fixed NFT components to correctly display metadata when multiple contracts with the same token ID are rendered.
  • Added mode as a predefined chain in the API.
  • Implemented overrides for Lumia Testnet to use pre-EIP1559 gas values.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@jnsdls jnsdls requested a review from a team as a code owner January 15, 2025 07:18
Copy link

vercel bot commented Jan 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 18, 2025 8:36am
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 18, 2025 8:36am
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 18, 2025 8:36am
wallet-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 18, 2025 8:36am

Copy link
Contributor

graphite-app bot commented Jan 15, 2025

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge-queue - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

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.

Copy link
Contributor

github-actions bot commented Jan 15, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 46.66 KB (0%) 934 ms (0%) 4.2 s (+18.78% 🔺) 5.1 s
thirdweb (cjs) 116.84 KB (0%) 2.4 s (0%) 6.6 s (+15.52% 🔺) 8.9 s
thirdweb (minimal + tree-shaking) 5.59 KB (0%) 112 ms (0%) 307 ms (+65.99% 🔺) 419 ms
thirdweb/chains (tree-shaking) 506 B (0%) 10 ms (0%) 43 ms (-84.07% 🔽) 54 ms
thirdweb/react (minimal + tree-shaking) 19.24 KB (0%) 385 ms (0%) 682 ms (-35.08% 🔽) 1.1 s

@joaquim-verges joaquim-verges requested a review from a team as a code owner January 18, 2025 08:27
@github-actions github-actions bot added the SDK Involves changes to the thirdweb SDK label Jan 18, 2025

Some prop names have been updated:

`prompt -> messsage`
Copy link
Contributor

Choose a reason for hiding this comment

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

There appears to be a typo in the documentation - the property name is written as messsage (with 3 s's) but is correctly shown as message in all the code examples below. This should be corrected to maintain consistency and prevent confusion.

Spotted by Graphite Reviewer

Is this helpful? React 👍 or 👎 to let us know.

// prompt is now message
message:
"What's the total supply of this contract: 0xe2cb0eb5147b42095c2FfA6F7ec953bb0bE347D8",
// contextFilter is now contextFilter
Copy link
Contributor

Choose a reason for hiding this comment

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

The comment // contextFilter is now contextFilter appears to be incorrect - it should be // context is now contextFilter to accurately reflect the property rename described in the changelog. This would better document the breaking change for developers.

Spotted by Graphite Reviewer

Is this helpful? React 👍 or 👎 to let us know.

Copy link

codecov bot commented Jan 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.77%. Comparing base (b6d65cf) to head (c4a0d1c).
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5955      +/-   ##
==========================================
- Coverage   54.80%   54.77%   -0.03%     
==========================================
  Files        1151     1151              
  Lines       61257    61257              
  Branches     5151     5151              
==========================================
- Hits        33571    33556      -15     
- Misses      26959    26975      +16     
+ Partials      727      726       -1     
Flag Coverage Δ *Carryforward flag
legacy_packages 65.68% <ø> (ø) Carriedforward from b6d65cf
packages 52.42% <ø> (-0.03%) ⬇️

*This pull request uses carry forward flags. Click here to find out more.

see 8 files with indirect coverage changes

@joaquim-verges joaquim-verges merged commit eded4ff into main Jan 18, 2025
30 of 33 checks passed
@joaquim-verges joaquim-verges deleted the changeset-release/main branch January 18, 2025 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages SDK Involves changes to the thirdweb SDK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants