Skip to content

docs(plugin): align Radius skills with CLI and SDK 0.3.0 - #46

Open
erikzrekz wants to merge 1 commit into
mainfrom
feat/enterprise-ready-radius-skills
Open

erikzrekz wants to merge 1 commit into
mainfrom
feat/enterprise-ready-radius-skills

Conversation

@erikzrekz

@erikzrekz erikzrekz commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor
  • Adds a changeset entry for the skill
  • Updates skills to be less viem-centric, and more radius-sdk centric

I also made a few comments throughout for further consideration.

"name": "Radius Technology Systems"
}
},
"homepage": "https://github.com/radiustechsystems/radius-cli/tree/main/plugins/radius",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably should have a page either on the docs or www that acts as this projects homepage.

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

⚠️ No Changeset found

Latest commit: 0289963

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Comment thread plugins/radius/README.md

```sh
claude plugin marketplace add radiustechsystems/radius-cli
claude plugin install radius-dev@radius-cli

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still don't love this naming convention.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any other naming ideas?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/plugin marketplace add radiustechsystems/tools
/plugin install radius@skills

It would require this monorepo be renamed to "tools"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm supportive of this rename. Is @radius available in skills marketplaces?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Within this setting "radius" is not part of a global namespace so should be fine

@erikzrekz
erikzrekz marked this pull request as draft October 5, 2026 15:19
@erikzrekz
erikzrekz marked this pull request as ready for review October 5, 2026 15:19
const key = process.env.RADIUS_PRIVATE_KEY;
if (!key) throw new Error('Configure RADIUS_PRIVATE_KEY through your secrets manager');
const signer = privateKeyToAccount(key as `0x${string}`);
const wallet = createRadiusFetch({

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We may want a generic fetch that can just answer whatever the seller provides. I started down a path of making the SDK more multi-EVM compatible #40 but am holding off until we can discuss.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants