chore(release): 3.1.1 - #90
wyre-agent-fleet[bot] wants to merge 1 commit into
Conversation
Prepared by scripts/prepare-release.mjs. Merging this PR (any merge method) triggers this workflow's PUBLISH mode, which tags, publishes to npm, and creates the GitHub release — nothing publishes until this merges.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
Included review availability: Your plan provides up to 10 included reviews per hour; 5 remain after this review. 📝 WalkthroughWalkthroughThe package version changes from ChangesRelease metadata
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The release metadata is consistent and no actionable merge-blocking issue was established. 🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
asachs01
left a comment
There was a problem hiding this comment.
Code Review — PR #90 (chore(release): 3.1.1)
Verdict: Approve
Automated semantic-release PR. Diff is limited to:
CHANGELOG.md: new3.1.1entry documentingdeps: resolve js-yaml CVE-2026-84375, linked to issue #89 and the fix commit.package.json/package-lock.json: version bump3.1.0→3.1.1, consistent across both files.
Correctness
- Version bump is consistent between
package.jsonandpackage-lock.json. - Changelog entry matches the security-fix intent and title.
Security
- No hardcoded secrets/credentials. Release ships a js-yaml CVE fix.
Code Quality / Docs
- No source changes, consistent with "chore(release)" scope. Already has a coderabbit "Approved" review.
- Minor nit (non-blocking): the new changelog entry has a few extra blank lines before the next version header compared to the tighter spacing elsewhere in the file — purely cosmetic, no functional impact.
No blocking issues.
Code Review Summary — Claude CodeVerdict: Approve CriticalNone WarningsNone Suggestions
Looks Good
|
asachs01
left a comment
There was a problem hiding this comment.
Reviewed by Hermes Agent. Clean 3.1.0→3.1.1 bump (package.json/lock + CHANGELOG) for js-yaml CVE-2026-84375 fix; no unintended changes.
Automated release PR. Merging this (any merge method) publishes 3.1.1 to npm and creates the GitHub release — see CHANGELOG.md in this diff for the notes.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit