Add eliza-plugin-insumer#278
Add eliza-plugin-insumer#278douglasborthwick-crypto wants to merge 3 commits intoelizaos-plugins:mainfrom
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
WalkthroughAdded a new public plugin entry Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required. |
index.json
Outdated
| "@elizaos/plugin-zytron": "github:zypher-network/plugin-zytron", | ||
| "@erdgecrawl/plugin-base-signals": "github:erdGeclaw/plugin-base-signals", | ||
| "@esscrypt/plugin-polkadot": "github:Esscrypt/plugin-polkadot", | ||
| "@insumer/plugin-insumer": "github:douglasborthwick-crypto/eliza-plugin-insumer", |
There was a problem hiding this comment.
npm package name mismatch
The registry key @insumer/plugin-insumer is a scoped npm package name, but the PR description links to the npm package as eliza-plugin-insumer (unscoped: https://www.npmjs.com/package/eliza-plugin-insumer).
The registry maps npm package names to GitHub repos, so if the package is only published under the unscoped name eliza-plugin-insumer, users installing via this registry entry using the scoped name @insumer/plugin-insumer will encounter a 404. The entry should match exactly the name under which the package is published on npm.
If the package is published under the @insumer scope, please confirm that @insumer/plugin-insumer is the correct npm package name. Otherwise, consider:
| "@insumer/plugin-insumer": "github:douglasborthwick-crypto/eliza-plugin-insumer", | |
| "eliza-plugin-insumer": "github:douglasborthwick-crypto/eliza-plugin-insumer", |
There was a problem hiding this comment.
Fixed in 37f33b7 — registry key is now eliza-plugin-insumer (unscoped), matching the published npm package.
index.json
Outdated
| "@elizaos/plugin-arkham": "github:elizaos-plugins/plugin-arkham", | ||
| "@elizaos/plugin-arthera": "github:elizaos-plugins/plugin-arthera", | ||
| "@elizaos/plugin-asterai": "github:elizaos-plugins/plugin-asterai", | ||
| "@elizaos/plugin-ATTPs": "github:APRO-com/plugin-ATTPs", |
There was a problem hiding this comment.
Unrelated change included
This PR repositions @elizaos/plugin-ATTPs from its previous location (between plugin-8004 and plugin-abstract) to here (between plugin-asterai and plugin-autocoder). While the new position is alphabetically correct, this change is not mentioned in the PR description and is unrelated to the addition of @insumer/plugin-insumer. Could you clarify if this reordering is intentional as part of this PR?
There was a problem hiding this comment.
Fixed in 042c008 — plugin-ATTPs restored to its original position. The only diff vs upstream is the eliza-plugin-insumer addition.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@index.json`:
- Line 227: The registry entry for "@insumer/plugin-insumer" points to
"github:douglasborthwick-crypto/eliza-plugin-insumer" which cannot be found;
verify and correct the GitHub repo reference (confirm owner/name spelling and
public accessibility) or remove the entry if the repo is private or nonexistent,
and if corrected ensure the target repository contains a valid package.json and
plugin structure before updating the value in index.json (replace the
"github:..." string with the correct GitHub path or a validated package source).
|
Both issues flagged by the bots have been addressed:
Ready for review when you get a chance. |
042c008 to
0ba410e
Compare
0ba410e to
7f59a65
Compare
Registry Update Checklist
Registry:
If not an eliza-plugins official repo, i.e. new plugin:
The plugin repo has:
elizaos-pluginsin the topics in the GitHub repo settings. If the plugin is related toAIorcrypto, please add those as topics as well.images/banner.jpgandimages/logo.jpgand theypluginType: "elizaos:plugin:1.0.0"andpluginParametersPlugin summary
eliza-plugin-insumer — On-chain wallet verification across 32 blockchains via InsumerAPI.
Summary by CodeRabbit