chore: release packages - #59
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This Changesets-generated release PR prepares @btravstack/entity for publication by bumping the package version and promoting the pending changeset into the package changelog (notably: Entity.union(...) now returns a value rather than supporting a class-extends form).
Changes:
- Bump
@btravstack/entityfrom0.6.0to0.7.0. - Add
0.7.0release notes topackages/entity/CHANGELOG.md, documenting the removal of theEntity.unionclass form and__base. - Remove the consumed changeset file for this release.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/entity/package.json | Version bump to 0.7.0 for the release. |
| packages/entity/CHANGELOG.md | Adds 0.7.0 release notes describing the union API change and related breaking details. |
| .changeset/union-no-class-form.md | Removed as part of the Changesets release process (changeset has been applied to the changelog). |
Suppressed comments (1)
packages/entity/CHANGELOG.md:33
- This uses an H2 heading (
##) inside the 0.7.0 section, which conflicts with the changelog’s established heading hierarchy (##for versions,###for subsections). Consider downgrading this to####(or using bold text) so the document outline stays consistent.
## Breaking: `__base` is removed
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
|
||
| - 5844dd4: `Entity.union(...)` returns a value, not a class. | ||
|
|
||
| ## Breaking: the class form is gone |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
@btravstack/entity@0.7.0
Minor Changes
5844dd4:
Entity.union(...)returns a value, not a class.Breaking: the class form is gone
The class form typed as the members' shared root, not as the member union, so
it could not narrow — and it failed late, at the first call site that touched a
member-only field. A class's instance type cannot be a union (
TS2509), so noversion of it could have narrowed; and its type was always redundant with the
root the author had already named.
class X extends Entity.union(...) {}is nowTS2507at the declaration.Statics that lived in the class body become plain functions:
Breaking:
__baseis removedThe phantom
__basecarrier is gone fromEntityStaticandUnionMember. Itexisted only to compute the class form's root type. Nothing writes it by hand;
it is listed because it is part of the emitted public surface.