chore: release v5.8.5 - #342
Conversation
There was a problem hiding this comment.
Pull request overview
Patch release prep for v5.8.5 by stamping the new version across all published manifests and documenting the release notes.
Changes:
- Bumped version from
5.8.4→5.8.5in npm/package and plugin/site manifests. - Updated
package-lock.jsonroot/package versions to5.8.5. - Added the
[5.8.5]section toCHANGELOG.mdwith notes for #333/#334/#335/#339.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
package.json |
Updates npm package version to 5.8.5. |
package-lock.json |
Syncs lockfile version fields to 5.8.5. |
.claude-plugin/plugin.json |
Updates plugin manifest version to 5.8.5. |
.claude-plugin/marketplace.json |
Updates marketplace manifest version to 5.8.5. |
site/content.json |
Updates website metadata version to 5.8.5. |
CHANGELOG.md |
Adds release notes for 5.8.5 dated 2026-04-23. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Code Review
This pull request updates the project version to 5.8.5 and introduces several fixes and improvements. Key changes include replacing hardcoded developer paths with portable ones in web-ctl skills, moving git hook installation from the npm prepare hook to a dedicated setup script, and refining the npm version lifecycle to use an explicit file allowlist for staging instead of a broad git add. A review comment suggests that CHANGELOG.md should be added to this allowlist to ensure repository consistency during the release process.
Bot review caught that CHANGELOG.md is part of the release prep flow (developer manually edits it before running `npm version`), so it must be in the lifecycle script's `git add` allowlist - otherwise `npm version`'s auto-commit drops the changelog entry, leaving the version bump committed but the changelog entry stranded as unstaged. Updates the CHANGELOG #339/#342 description to reflect the new file.
Summary
Patch release v5.8.5 - all 4 changes since v5.8.4 are NLPM-audit follow-ups (xiaolai's PRs #333, #334, #335, #339) addressing portability and dev-experience papercuts. No behavior change to plugin functionality.
Highlights
Fixed:
/Users/avifen/.agentsys/paths in web-ctl skills (76 sites) replaced with~/.agentsys/preparelifecycle hook moved to explicitsetup-hooksscript (no more auto-installing git hooks onnpm install); also removed the no-op pre-commit placeholdernpm versionlifecycle now uses an explicit allowlist (wasgit add -A) covering all 5 filesstamp-version.jswritesChanged:
^4.1.1to~4.1.1(allow 4.1.x patches, block 4.2.x minors)Full notes in CHANGELOG.md
[5.8.5]section.Files touched
package.json(5.8.4 -> 5.8.5)package-lock.json.claude-plugin/plugin.json.claude-plugin/marketplace.jsonsite/content.jsonCHANGELOG.mdTest plan
npm testfrom main: 3458 tests passing on the merge basenode scripts/preflight.js --release: 18/19 checks pass; therelease:npm-testfailure is a known artifact (jest config ignores/worktrees/paths so tests don't run from the release worktree). Code-only diff is version strings + CHANGELOG, so test outcome is identical to main.npm version 5.8.5v5.8.5-> release.yml publishes to npm + creates GitHub release with CHANGELOG section