Skip to content

test: Rust CLI#39

Open
MigOKG wants to merge 76 commits intomainfrom
submit/test-rust-cli
Open

test: Rust CLI#39
MigOKG wants to merge 76 commits intomainfrom
submit/test-rust-cli

Conversation

@MigOKG
Copy link
Copy Markdown
Owner

@MigOKG MigOKG commented Apr 5, 2026

E2E - test-rust-cli

yz06276 and others added 27 commits April 5, 2026 19:52
- Move CLI source to cli/ directory
- Merge 14 community plugins into skills/ (single path)
- Add CODEOWNERS for permission control
- Migrate 5 community CI workflows with path updates (submissions/ → skills/)
- Update CLI Rust code: remove community repo special logic, update paths
- Update registry.json: repo refs, add trust_source/risk_level/category fields
- Rewrite marketplace.json: all 15 plugins with ./skills/<name> source paths
- Rewrite README.md: 4-screen layout for all user roles
- Add docs/FOR-USERS.md: non-technical user guide (677 lines)
- Add docs/FOR-DEVELOPERS.md: developer guide with 3 submission modes (818 lines)
- Add docs/FOR-PARTNERS.md: partner guide with high-risk policy
- Add docs/REVIEW-GUIDELINES.md: public review standards
- Add .github/security-rules, prompts, scripts, PR template
- Full npx skills add compatibility preserved
- Backward compatible with published plugin-store CLI

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Update .gitignore: add /cli/target, remove docs/ and skills/okx-* exclusions
- Remove 7071 build artifact files from git tracking
- Add docs/ (FOR-USERS, FOR-DEVELOPERS, FOR-PARTNERS, REVIEW-GUIDELINES)
- Add skills/okx-buildx-hackathon-agent-track/

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Remove plugin directory table (per PM feedback)
- Replace plugin-store install/list/search with npx skills add commands
- Keep category, risk level, and trust indicator tables

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- test-pure-skill: Pure SKILL.md, no binary
- test-rust-cli: Rust CLI + OnchainOS ETH price query
- test-go-cli: Go CLI + OnchainOS ETH price query
- test-ts-cli: TypeScript/Bun CLI + OnchainOS ETH price query
- test-node-cli: Node.js CLI + OnchainOS ETH price query
- test-python-cli: Python script + OnchainOS ETH price query
- test-stub-external: Mode B external repo pointer (MigOKG/test-stub-skill)

All pass plugin-store lint. All CLIs compile and run successfully.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
onchainos uses `token price-info --address <addr> --chain <chain>` not `token price ETH`

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Change checkout condition from source_mode=='local' to source_mode!='external'
  (handles case where source_mode output is empty)
- Add fallback: local_source_dir → plugin_dir when source_mode is not external
- Fix all 5 build jobs (Rust, Go, TypeScript, Node.js, Python)
- Fix plugin-summary.yml: use head.sha instead of head.ref

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Scripts are now in the same repo at .github/scripts/, no need to
sparse-checkout from okx/plugin-store anymore.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Checkout uses SHA (detached HEAD), so git push needs explicit branch target.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…mary.yml

plugin-summary.yml was incorrectly adding 'ai-reviewed' label.
Changed to 'summary-generated' to distinguish from AI code review.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Generated from plugin.yaml metadata for all plugins missing this file.
Required by skill-review.yml and follows Claude Skill architecture standard.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…RS.md

- Add to directory layout as required
- Add full field reference table
- Add Step 4 in Quick Start for creating plugin.json
- Note that name/description/version must match plugin.yaml

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Install onchainos CLI + onchainos skills (npx skills add okx/onchainos-skills)
- Install plugin-store skill (npx skills add okx/plugin-store --skill plugin-store)
- Fix binary download URL: plugin-store-community → plugin-store

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Moves install count reporting from CLI-only (plugin-store install) to
SKILL.md pre-flight, so npx-installed plugins also report usage.

- Reports to Vercel stats API (name + version)
- Reports to OKX API (pluginName + device ID)
- Runs once per plugin using ~/.plugin-store/reported/<name> flag file
- Fire-and-forget: curl errors silenced, never blocks the user

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@MigOKG MigOKG temporarily deployed to summary-generation April 5, 2026 14:06 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

Phase 4: Summary + Pre-flight for test-go-cli

Review below. AI Code Review is in a separate check.



Generated by Plugin Store CI after maintainer approval.

@yz06276 yz06276 temporarily deployed to summary-generation April 5, 2026 14:20 — with GitHub Actions Inactive
MigOKG and others added 18 commits April 5, 2026 22:26
…st plugins

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Remove 7 test plugins (test-pure-skill, test-rust-cli, test-go-cli,
test-ts-cli, test-node-cli, test-python-cli, test-stub-external) and
their entries from registry.json and marketplace.json.

These were used for E2E verification of the merged repo CI pipeline.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
feat: merge plugin-store-community + PRD optimization
@yz06276 yz06276 force-pushed the submit/test-rust-cli branch from 6dfa651 to 7401c7f Compare April 5, 2026 14:49
@yz06276 yz06276 temporarily deployed to summary-generation April 5, 2026 14:49 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

🔍 Official Plugin Review

Plugins reviewed: test-go-cli test-node-cli test-pure-skill test-python-cli test-rust-cli test-stub-external test-ts-cli

Static Checks


❌ [test-go-cli] SKILL.md not found
❌ [test-node-cli] SKILL.md not found
❌ [test-pure-skill] SKILL.md not found
❌ [test-python-cli] SKILL.md not found
❌ [test-stub-external] SKILL.md not found
❌ [test-ts-cli] SKILL.md not found

AI Review

AI review unavailable (HTTP 400)


Auto-generated by Official Plugin Review workflow

E2E test: Rust CLI querying ETH price via onchainos token price-info.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@yz06276 yz06276 force-pushed the submit/test-rust-cli branch from 7401c7f to eaf5d3a Compare April 5, 2026 14:55
@yz06276 yz06276 temporarily deployed to summary-generation April 5, 2026 14:56 — with GitHub Actions Inactive
@MigOKG MigOKG force-pushed the main branch 2 times, most recently from db25e01 to 9030baa Compare April 7, 2026 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants