You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Review package.json / package-lock.json version bump
Post inline comments if needed (none needed)
Post summary review
Summary
Posted the full review as a top-level comment via gh pr comment (no inline comments — see below). Verdict: clean, mechanical release PR, no blocking issues.
Verdict: LGTM. This follows the repo's release process (see CLAUDE.md) exactly — no inline comments, nothing to fix.
What I checked:
Diff is limited to CHANGELOG.md, package.json, package-lock.json — no code, matching the "release PRs contain no code" rule.
Version bump is consistent across all three files: 1.7.0 → 1.7.1, and package-lock.json's root version was updated too (the exact thing npm version --no-git-tag-version is required for, vs. a hand-edit that would leave the lockfile stale).
Date (2026-08-23) matches the merge date of the referenced PRs.
Formatting (intro paragraph + ### Added/### Changed sections) is consistent with prior entries in the file (e.g. 1.7.0, 1.6.1).
One judgment call worth a sanity-check (not a blocker): this is a patch bump (1.7.0 → 1.7.1) even though the changelog has an ### Added section — new CLI command aliases (#175) and a newly-documented export-json command (#176) are new, backward-compatible functionality, which strict semver would usually call a minor bump. The PR body does explicitly justify the choice ("CLI ergonomics and docs; no tool or schema change"), which is exactly what CLAUDE.md asks the author to state — so process-wise this is satisfied. Worth double-checking that "no MCP tool/schema change" is really the bar this project uses to distinguish patch vs. minor (as opposed to "no new capability"), since 1.7.0 itself was bumped to minor for an analogous ### Added entry. If that's the established convention here, no change needed.
No test coverage concerns — this PR has no code to test, and tag-release.sh's pre-flight checks (CHANGELOG section present, lockfile version match, on main, clean tree) will catch anything structurally wrong at tag time.
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
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.
Summary
CHANGELOG.md: 1.7.1 section written from the Changelog sections of feat(cli): accept commands as words, and lay the help out as options and commands #175, feat(cli): document export-json and take an output path #176, docs: document the command line in docs/cli.md #177, docs: slim the README to the high level #178npm version patch --no-git-tag-version: 1.7.0 → 1.7.1Patch bump: CLI ergonomics and docs; no tool or schema change.
Test plan
scripts/tag-release.sh --yesonmain