Repository navigation
Release nansen-cli 2.0.0 - #689
Conversation
pr-reviewer Summary for #99caf27✅ No issues found The code review completed successfully with no findings. Review effort: 1/5 (Trivial) SummaryThis is a standard automated release PR generated by the Changesets GitHub Action. The diff is entirely bookkeeping: The major version bump is correct: PR #657 introduced a breaking change to Token usage: 283 input, 2,096 output, 234,555 cache read, 39,730 cache write | Usage Guide New pushes are reviewed automatically with a 10-minute cooldown between reviews. To request a review at any time, comment |
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
476bf0d
7330c3c to
476bf0d
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
872b574
476bf0d to
872b574
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
2b3530b
872b574 to
2b3530b
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
b6e5ebb
2b3530b to
b6e5ebb
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
f936f85
b6e5ebb to
f936f85
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
51ea198
f936f85 to
51ea198
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
51ea198 to
75cdaf1
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
aac4810
75cdaf1 to
aac4810
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
b09367b
aac4810 to
b09367b
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
e7b3415
b09367b to
e7b3415
Compare
d9dc8f7 to
861af78
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
56f6e31
861af78 to
56f6e31
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
2ca5610
56f6e31 to
2ca5610
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
b34f808
2ca5610 to
b34f808
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
70e01f4
b34f808 to
70e01f4
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
8e3ecff
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
- Review comment contains non-blocking feedback
If you have any concerns, please request a manual review.
43809cb
8e3ecff to
43809cb
Compare
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
Codier
left a comment
There was a problem hiding this comment.
Independent release review: clean at 99caf27818f7a128a1ac39ef9d708e5f8dd589c1, against source main fab0a94fad4c2a0cc10a2d4668393ad62a070e67.
Factory Droid / Anthropic claude-sonnet-5, high reasoning verified from runtime, reviewed in a separate checkout. Writer/coordinator is OpenAI Codex. No actionable findings.
The release consumes all 31 pending changesets, updates the changelog and sets version 2.0.0. Runtime source and workflows are unchanged from main. Dependency records and native platform metadata are unchanged. The notes include the corrected x402 payment behavior and bridge fee checks. The final validation commit has the same tree as the reviewed release-bot commit.
The reviewer inspected the actual 101-file archive, its installed version/help evidence, payment notes and platform qualification statements. Local checks passed 4,454 tests with nine skipped, plus lint. Coordinator separately verified fresh CI passing 4,461 tests with two skipped on each of Node 20/22/24, plus lint and security checks. Production dependency audit is clean; two existing development-tool advisories remain unchanged.
This is a code/package assessment, not authorization to publish. Live production login, refresh and two-session revocation acceptance remain outstanding. Keep this PR unmerged until those checks pass. No publication, live login or live payment was performed by this review.
There was a problem hiding this comment.
Auto-approved
This PR was automatically approved because:
- Claude recommends approval
- Claude assessed this as a minimal effort change
- The effort level is within the auto-approval threshold of 2
- No high or critical issues were detected
If you have any concerns, please request a manual review.
Codier
left a comment
There was a problem hiding this comment.
Approved for code readiness at 99caf27, based on the independent Factory Droid / Anthropic claude-sonnet-5 high review already posted in #689 (review). Head and base remain unchanged. Node 20/22/24 CI is green. The version/changelog-only release delta and 101-file archive were checked; no implementation or workflow changes are in this PR.
Publication remains held until the current production login/refresh/revocation validation is complete. Merging this PR triggers publication. This review does not claim fresh native-storage qualification or broaden the documented platform support.
Release
nansen-cli@2.0.0from mainfab0a94. Changesets regenerated the release with all 31 pending changesets, including the corrected x402 payment behavior and bridge fee checks. The changelog contains the full notes.Plain
nansen loginnow requests browser approval. Automation that previously used plain login to save an API key must use explicit key setup. Environment keys still take precedence for commands. Saved browser sessions renew automatically and use v2 authentication metadata; an in-place downgrade to a pre-v2 client is unsupported.API keys and browser sessions share automatic x402 payment for supported HTTP 402 challenges under the existing wallet policy and spending limits. Authentication, authorization and renewal failures never trigger signing. Login verification and account checks remain non-paying. Paid retries omit the account credential and leave saved authentication unchanged.
Browser-login qualification remains limited to the documented macOS arm64 local Terminal preview. Linux remote and Windows qualification are not claimed. Keep this PR unmerged until production readiness and live acceptance checks pass. Merging this PR can publish to npm and sync skills to ClawHub; the temporary publication gate has been removed.
Validation on the release tree:
99caf27818f7a128a1ac39ef9d708e5f8dd589c1. Production acceptance remains pending; do not merge for publication yet.AI execution metadata
43809cb. OpenAI Codex independently regenerated and compared the tree, preserved lockfile platform metadata locally, validated the archive, and added the empty validation commit ashulk-linus[bot]. No source-code change or identity fallback.claude-sonnet-5, high reasoning verified from runtime init, separate checkout. Clean verdict on99caf27818f7a128a1ac39ef9d708e5f8dd589c1, posted as Codier. Scope: release delta, actual archive and evidence; production rollout not approved.