chore(release): v1.3.0 - #447
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughThis PR prepares the v1.3.0 release by bumping the package version and adding a comprehensive CHANGELOG entry. All the actual feature and bug-fix work was merged in prior PRs (#433, #435, #437 and related); this PR is purely the release bookkeeping step that precedes tagging. Changes
Review Notes
🤖 Generated with Claude Code |
There was a problem hiding this comment.
Review: chore(release): v1.3.0
This is a clean, minimal release prep PR — two files changed, no code modifications. The version bump (1.2.1 → 1.3.0) is correctly a minor rather than patch bump given the feature additions (local server support, ABLY_URL, chat integration rules). Semver is right.
One issue to fix before merging:
CHANGELOG: Incorrect vulnerability range for tar
In the Fixed section: Bumped tar to 7.5.21 to cover a vulnerability in the 7.5.11-7.5.21 range.
The range 7.5.11-7.5.21 includes the fixed version itself, implying 7.5.21 is still vulnerable. This should almost certainly be 7.5.11-7.5.20 (the affected versions) fixed by bumping to 7.5.21.
Suggested fix: Bumped tar to 7.5.21 to cover a vulnerability in the 7.5.11-7.5.20 range.
Everything else looks good: changelog format follows Keep a Changelog, date matches the release date, entries accurately describe what is in the included PRs (#437, #433, #435), and the format is consistent with prior release entries.
|
Note: The review above (posted via API) has unformatted text due to sandbox restrictions. Please see the intended formatted version below. |
|
Formatted review (re-post): |
|
Formatted review content (markdown): Review: chore(release): v1.3.0 This is a clean, minimal release prep PR. Two files changed, no code modifications. Version bump 1.2.1 to 1.3.0 is correctly minor not patch given feature additions. Semver is right. Issue: CHANGELOG vulnerability range for tar is wrong. "7.5.11-7.5.21" includes the fix itself. Should be "7.5.11-7.5.20". Fix: change to "in the 7.5.11-7.5.20 range". Otherwise looks good - Keep a Changelog format, correct date, PRs #437 #433 #435 described correctly. |
Prepares the v1.3.0 release: bumps
package.jsonto 1.3.0 and adds the CHANGELOG entry for everything merged since v1.2.1.Minor, not patch — this release adds features (local server support,
ABLY_URL/--urlrouting, Ably Chat integration rule types), matching how 1.1.0 and 1.2.0 were cut for feature additions.What's in it
Local server support (#437)
ably accounts login --localstores a locally-running server as an ordinary account profileABLY_URL/--urlset host, port and TLS from a single value for one-off targetingaccounts list/switchreport endpoint and control plane URLsAbly Chat integration rules (#433 and predecessors, DX-1546)
integrations createsupports the chat rule types (http/before-publish, the moderation vendors,aws/lambda/before-publish)--chat-room-filteroncreate/update, displayed onget/list/deleteintegrations update/deleteSecurity
tarbumped to 7.5.21 (fix(deps): bump tar from 7.5.15 to 7.5.21 #435)Validation
pnpm prepare— cleanpnpm exec eslint src test scripts packages— 0 errors (10 pre-existing warnings)pnpm test:unit— 2644 passed, 1 todopnpm test:tty— 5 passedAfter merge
Tag the merge commit to trigger the release workflow:
git tag -a v1.3.0 -m "Release v1.3.0" git push origin v1.3.0🤖 Generated with Claude Code