Repository navigation
fix(sdk): accept @agent-relay/sdk and harness-driver 16 as optional peers - #700
Conversation
…eers agent-relay 16.0.0's release gate (relay packages/cli relayflows-peer-compat test) installs the pinned @relayflows/sdk beside the released Relay packages; 2.0.60's peer range stops at <16, so the 16.0.0 publish fails with ERESOLVE. @agent-relay/sdk and @agent-relay/harness-driver are unchanged between v15.1.1 and relay main (the 16.0.0 major is the desktop probe), so 16 is compatible. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
[Low impact] This metadata-only change appears safe to merge.
|
maintainability lens — PASSMaintainability review — PR #700Scope: peer dependency upper bounds widened from BlockersNone. The change is a one-token bump in a well-scoped file and the lockfile mirrors it exactly. Concerns
Notes
REVIEW_PASSED |
history lens — PASSBlockers: none. PR #700 does not establish any of the three rejection conditions.
Concerns: The commit’s assertion that the upstream SDK and harness-driver are unchanged between Relay 15.1.1 and the proposed 16 release was not independently established here. The PR body reports test failures without captured command output; those reports are insufficient as reproducible compatibility evidence, but do not demonstrate a false statement. Notes: Publishing the SDK patch and updating Relay’s consumer pins are explicitly documented follow-ups. The older gate brief in REVIEW_PASSED |
structure lens — MISSING |
|
🎯 review-swarm: FAILED (M:pass H:pass S:missing) Lens transcripts posted as sibling comments above. |
Why
The agent-relay 16.0.0 release (publish run 41807496212) failed in
Build & Version. Relay'spackages/cli/src/cli/relayflows-peer-compat.test.tsinstalls the pinned@relayflows/sdk@2.0.60beside@agent-relay/sdkand@agent-relay/harness-driverat the version being released. 2.0.60's optional peer range is>=12.3.1 <13 || >=15.1.0 <16, so npm rejects 16.0.0 with ERESOLVE. Nothing was published.Change
packages/sdkpeers@agent-relay/sdkand@agent-relay/harness-driver:>=12.3.1 <13 || >=15.1.0 <17(the lockfile's root entry is updated to match).Compatibility
git diff v15.1.1 origin/main -- packages/sdk packages/harness-driverin relay is empty. The 16.0.0 major is the desktop core (agent-relay-probe) contract, so the SDK surface flows uses is the same as 15.1.x. The relay test still requires the next major (17) to be rejected, and<17does that.Tests
packages/sdkvitest on this Mac: the same 18 files fail on origin/main without this change (live, kernel and CLI tests that need a local relayflowd or cloud sign-in). With the change, one more live test failed once (authored-parallel-run-live, "f.run after done()", a local runtime flake). A peer range inpackage.jsoncan't change runtime behavior.After merge
@relayflows/sdk(patch: 2.0.61).@relayflows/sdkpin in the rootpackage.json(dependency andoverrides) and inpackages/cli/package.json.publish.ymlwithcustom_version=16.0.0.🤖 Generated with Claude Code
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.