Skip to content

chore: alpha to main#382

Merged
fedellen merged 4 commits into
mainfrom
alpha
Jan 23, 2026
Merged

chore: alpha to main#382
fedellen merged 4 commits into
mainfrom
alpha

Conversation

@fedellen

@fedellen fedellen commented Jan 22, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • New Features

    • Added an optional abort signal to upload inputs, allowing uploads to be cancelled.
  • Chores

    • Updated gas price settings for three Kyve networks (kyve-1, kaon-1, korellia-2) to match current network parameters.

✏️ Tip: You can customize this high-level summary in your review settings.

dtfiedler and others added 3 commits January 21, 2026 16:57
chore: main back to alpha
following KYVE upgrade, gas price changed
Update gas prices for KYVE chains
@fedellen fedellen requested a review from a team as a code owner January 22, 2026 23:25
@coderabbitai

coderabbitai Bot commented Jan 22, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Updated Kyve chain gasPrice values for three networks and added an optional signal?: AbortSignal to the UploadDataInput type. Changes are configuration and type surface only; no control-flow or runtime logic modifications.

Changes

Cohort / File(s) Summary
Kyve Chain Configuration
packages/turbo-sdk/src/common/token/kyve.ts
Updated gasPrice values: kyve-1 (2 → 62.5), kaon-1 (2 → 0.02), korellia-2 (2 → 62.5).
Type definition
packages/turbo-sdk/src/types.ts
Added optional signal?: AbortSignal to UploadDataInput type; minor formatting adjustment.

Sequence Diagram(s)

(omitted — changes are configuration/type-only and do not alter control flow)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

released

Suggested reviewers

  • dtfiedler

Poem

🐰 I hopped through code with gentle paws,
Tuned gas values and added a clause,
A signal for uploads, quiet and neat,
Kyve chains hum with a livelier beat. 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'chore: alpha to main' is vague and does not clearly communicate the actual changes made in the pull request, which involve updating gas prices for Kyve networks and adding an AbortSignal parameter to the UploadDataInput type. Provide a more descriptive title that summarizes the main changes, such as 'chore: update Kyve gas prices and add abort signal support' or similar.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
packages/turbo-sdk/src/common/token/kyve.ts (1)

282-311: Correct kaon-1 gasPrice to align with official KYVE documentation.
The gasPrice for kaon-1 is set to 0.02 tkyve, but official KYVE setup guides recommend 0.001 tkyve as the minimum-gas-prices value for kaon testnet. The values for kyve-1 (62.5 ukyve) and korellia-2 (62.5 tkyve) are confirmed correct. Update kaon-1 from 0.02 to 0.001 to match documented standards.

@codecov

codecov Bot commented Jan 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.52%. Comparing base (ecded57) to head (4a45661).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #382      +/-   ##
==========================================
- Coverage   89.62%   89.52%   -0.10%     
==========================================
  Files          26       26              
  Lines        6562     6561       -1     
  Branches      422      422              
==========================================
- Hits         5881     5874       -7     
- Misses        680      686       +6     
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dtfiedler

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.40.2-alpha.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@fedellen fedellen merged commit 261d1e1 into main Jan 23, 2026
30 checks passed
@dtfiedler

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.40.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.41.0-alpha.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants