Skip to content

Separate entry hive on SDK#761

Merged
feruzm merged 2 commits into
developfrom
sdkentry
Apr 19, 2026
Merged

Separate entry hive on SDK#761
feruzm merged 2 commits into
developfrom
sdkentry

Conversation

@feruzm
Copy link
Copy Markdown
Member

@feruzm feruzm commented Apr 19, 2026

Summary by CodeRabbit

  • New Features

    • Added a dedicated hive import path so hive functionality can be imported separately with optimized browser and Node support.
  • Chores

    • Bumped SDK and wallets package versions.
    • Updated changelogs to document the new SDK release and wallet dependency update.
  • Build

    • SDK build now emits artifacts for the added hive entry alongside the main SDK entry.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 19, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f3f00998-2794-43f6-857f-91b542be0b8a

📥 Commits

Reviewing files that changed from the base of the PR and between 15d0f5e and be4a8eb.

📒 Files selected for processing (4)
  • packages/sdk/CHANGELOG.md
  • packages/sdk/package.json
  • packages/wallets/CHANGELOG.md
  • packages/wallets/package.json

📝 Walkthrough

Walkthrough

Added a new SDK public subpath ./hive, implemented a new module that re-exports hive-tx, and updated the build config to emit artifacts for the new entry; package versions and changelogs updated accordingly.

Changes

Cohort / File(s) Summary
SDK package export
packages/sdk/package.json
Bumped SDK to 2.2.3 and added a new exports subpath ./hive with types, browser, import/default, and require targets pointing to dist artifacts.
New entry + build config
packages/sdk/src/hive.ts, packages/sdk/tsup.config.ts
Added src/hive.ts that re-exports ./hive-tx; updated tsup shared entries to include src/hive.ts and removed as const type assertions from external/shared.
Changelogs
packages/sdk/CHANGELOG.md, packages/wallets/CHANGELOG.md
Added patch changelog entries for @ecency/sdk 2.2.3 and @ecency/wallets 4.0.1 (noting dependency update).
Wallets package bump
packages/wallets/package.json
Bumped @ecency/wallets version 4.0.0 → 4.0.1 and updated dependency reference to @ecency/sdk@2.2.3.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 I hopped a new path called ./hive today,
Re-exported treasures from hive-tx to play.
Browser and Node get parcels so neat,
Builds now emit both—what a treat! 🌿✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Separate entry hive on SDK' clearly indicates the main change: creating a separate entry point for the hive module in the SDK package, which aligns with the actual modifications across package.json, hive.ts, and tsup.config.ts.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sdkentry

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.

@feruzm feruzm added the patch Bug fixes and patches (1.0.0 → 1.0.1), add this only if any packages/ have patch changes in PR label Apr 19, 2026
@feruzm feruzm merged commit a75d69d into develop Apr 19, 2026
1 check was pending
@feruzm feruzm deleted the sdkentry branch April 19, 2026 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch Bug fixes and patches (1.0.0 → 1.0.1), add this only if any packages/ have patch changes in PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant