Repository navigation
Let the Version Packages PR run CI; install x402 peers in skills - #89
Merged
Merged
Conversation
The release workflow opens the Version Packages PR with the default GITHUB_TOKEN, whose pushes never trigger pull_request workflows, so the PR carries none of the checks the main ruleset requires and cannot be merged. The version job now opens it with a GitHub App token (CHANGESETS_APP_ID + CHANGESETS_APP_PRIVATE_KEY) or a fine-grained PAT (CHANGESETS_GITHUB_TOKEN) when either secret exists, and falls back to the default token otherwise. README documents the setup. radius-sdk 0.7.0 moves @x402/core and @x402/evm to peer dependencies. @x402/evm is an optional peer, which npm and pnpm do not install, so the buyer install commands in the x402 and radius-dev skills left radius-sdk/client unable to load. The skills now name both peers, the sell-api table names @x402/core for sellers, and the plugin README says which peer each entry point needs. Plugin 0.2.1 -> 0.2.2. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HfY52NzCySMNxcm6AknwNo
Contributor
|
erikzrekz
marked this pull request as ready for review
October 21, 2026 21:10
Contributor
kylecrawshaw
approved these changes
Oct 26, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Two fixes found while reviewing #81 (the radius-sdk 0.7.0 / radius-cli 0.6.0 Version Packages PR).
1. The Version Packages PR cannot merge
The
mainruleset requires one approval plus theNode 22,Node 24andChanged packages have a changesetchecks. The release workflow opens the Version Packages PR with the defaultGITHUB_TOKEN, and pushes made with that token never triggerpull_requestworkflows, so the PR has no checks at all and GitHub reports it as blocked.release.ymlnow opens the PR with a GitHub App token (CHANGESETS_APP_ID+CHANGESETS_APP_PRIVATE_KEY) or a fine-grained PAT (CHANGESETS_GITHUB_TOKEN) when either secret exists, and falls back to the default token otherwise, so nothing changes until a secret is added. The App is preferred: a PR opened with a person's PAT cannot be approved by that person. README "Releasing" documents the setup and the fallback (close and reopen the PR by hand to run CI).Order that unblocks #81 with no manual reopen: add the secret, merge this PR, and the release workflow refreshes #81 as the App, which runs CI on it. Without a secret, close and reopen #81 to get its checks.
2. Skills break on radius-sdk 0.7.0
0.7.0 moves
@x402/coreand@x402/evmto peer dependencies.@x402/evmis an optional peer, and neither npm nor pnpm installs an optional peer. I verified against the packed 0.7.0 tarball with both: after thenpm install radius-sdk viemthe x402 and radius-dev skills instruct,import 'radius-sdk/client'fails withERR_MODULE_NOT_FOUND: Cannot find package '@x402/evm'. The server entry still loads, since the required peer is auto-installed.x402/references/x402-client.mdandradius-dev/references/typescript-viem.md: buyer install is nowradius-sdk @x402/core @x402/evm viem, with a note on why.sell-api/SKILL.md: the install column names@x402/core, matching the docs reference page and the SDK README, so package managers that do not auto-install peers (Yarn) work too.plugins/radius/README.md: says which peer each entry point needs.No package code changes, so no changeset. No new API is named, so this can merge before or after #81 is published.
Checks run
python3 scripts/validate_plugin.py --base-ref origin/main,claude plugin validate plugins/radius --strict,claude plugin validate . --strict: passrelease.ymlparses; the App-token step is gated on a job-level env mirror of the secret becausesecretsis not available in a stepifclaude plugin eval) were not run here: this session has no eval credential. TheClaude plugin evalsworkflow runs on this PR since it touchesplugins/radius/**.Generated by Claude Code