Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: ac70c93 The changes in this PR will be included in the next version bump. This PR includes changesets to release 24 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 |
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (19)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Greptile SummaryThis PR refactors existing multi-package changeset entries into per-package files to improve readability of generated changelogs. It also adds a new spec document (
Confidence Score: 3/5Safe to merge once the ENSv1DomainId delimiter change in unified-domain-model.md is confirmed to match the actual implementation. The changeset splitting is straightforward and well-executed, but the edit to
Important Files Changed
Reviews (1): Last reviewed commit: "fix: remove spec" | Re-trigger Greptile |
There was a problem hiding this comment.
Pull request overview
This PR refactors the repo’s Changesets metadata by splitting previously “multi-package” changesets into per-package entries, improving generated PR/release-note readability.
Changes:
- Split combined changesets into multiple per-package changesets (e.g., ENSRainbow bootstrap work, bigint helpers adoption, indexing metadata context refactor, docker-compose move).
- Updated the unified domain model changeset to reflect the current dash-delimited ENSv1 domain id format.
- Removed older aggregated changeset files that bundled multiple packages together.
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| .changeset/unified-domain-model.md | Updates release note text for ENSv1DomainId formatting. |
| .changeset/replace-bigints-migration.md | New per-package changeset for ensapi/ensindexer migration to SDK bigint/json helpers. |
| .changeset/replace-bigints-in-sdk.md | Removes the previous combined changeset (superseded by split entries). |
| .changeset/replace-bigints-helpers.md | New per-package changeset for adding bigint/json helpers to @ensnode/ensnode-sdk. |
| .changeset/ready-endpoint-bg-bootstrap-ensrainbow.md | Restricts this changeset to ensrainbow only (other packages split out). |
| .changeset/ready-endpoint-bg-bootstrap-ensrainbow-sdk.md | New per-package changeset for @ensnode/ensrainbow-sdk client/type updates. |
| .changeset/ready-endpoint-bg-bootstrap-ensindexer.md | New per-package changeset for ensindexer readiness polling behavior. |
| .changeset/label-set-rename-ensrainbow-sdk.md | New per-package breaking changeset for @ensnode/ensrainbow-sdk model/options renames. |
| .changeset/label-set-rename-ensnode-sdk.md | New per-package breaking changeset for @ensnode/ensnode-sdk labelSet rename. |
| .changeset/indexing-metadata-context-integration-test-env.md | New per-package changeset for integration-test-env adapting to indexing metadata context changes. |
| .changeset/indexing-metadata-context-ensnode-sdk.md | New per-package changeset for ensnode-sdk stack info type refactor. |
| .changeset/indexing-metadata-context-ensindexer.md | New per-package changeset summarizing ensindexer startup/config/indexing-status refactor. |
| .changeset/indexing-metadata-context-ensdb-sdk.md | New per-package changeset for ensdb-sdk IndexingMetadataContext API changes. |
| .changeset/indexing-metadata-context-ensapi.md | New per-package changeset for ensapi caches/schema consuming IndexingMetadataContext. |
| .changeset/hip-poems-shave.md | Removes the previous combined docker-compose move changeset (superseded by split entries). |
| .changeset/docker-compose-move-integration-test-env.md | New per-package changeset for integration-test-env docker-compose path move. |
| .changeset/docker-compose-move-docs.md | New per-package changeset for docs/CI updates due to docker-compose path move. |
| .changeset/clean-rivers-buy.md | Removes the previous combined indexing-metadata-context changeset (superseded by split entries). |
| .changeset/all-kids-smash.md | Removes the previous combined label-set rename changeset (superseded by split entries). |
Comments suppressed due to low confidence (1)
.changeset/ready-endpoint-bg-bootstrap-ensrainbow.md:9
- The changeset claims
/v1/configreturns503 Service Unavailableduring bootstrap, but the ENSRainbow API serves/v1/configeagerly (it always returns200withpublicConfigeven before the DB is attached). Only DB-backed routes like/v1/heal/:labelhashand/v1/labels/countreturn 503 while bootstrapping. Please update this bullet to avoid misleading release notes.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The generated PR descriptions duplicate change sets between all affected packages, so I split them to be per package, which should vastly improve readability.