Skip to content

test: Pure Skill#38

Open
MigOKG wants to merge 10 commits intomainfrom
submit/test-pure-skill
Open

test: Pure Skill#38
MigOKG wants to merge 10 commits intomainfrom
submit/test-pure-skill

Conversation

@MigOKG
Copy link
Copy Markdown
Owner

@MigOKG MigOKG commented Apr 5, 2026

E2E - test-pure-skill

yz06276 and others added 5 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>
@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-pure-skill

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


Auto-injected Pre-flight
## Pre-flight Dependencies (auto-injected by Plugin Store CI)

> Run once per session before first use. These checks ensure required tools are installed.

### Install onchainos CLI + Skills (auto-injected)

```bash
# 1. Install onchainos CLI
onchainos --version 2>/dev/null || curl -fsSL https://raw.githubusercontent.com/okx/onchainos-skills/main/install.sh | sh

# 2. Install onchainos skills (enables AI agent to use onchainos commands)
npx skills add okx/onchainos-skills --yes --global

# 3. Install plugin-store skills (enables plugin discovery and management)
npx skills add okx/plugin-store --skill plugin-store --yes --global

Report install (auto-injected, runs once)

REPORT_FLAG="$HOME/.plugin-store/reported/test-pure-skill"
if [ ! -f "$REPORT_FLAG" ]; then
  mkdir -p "$HOME/.plugin-store/reported"
  # Generate device ID
  DEV_RAW="$(hostname):$(uname -s):$(uname -m):$HOME"
  DEV_ID=$(echo -n "$DEV_RAW" | shasum -a 256 | head -c 32)
  # Report to Vercel stats
  curl -s -X POST "https://plugin-store-dun.vercel.app/install" \
    -H "Content-Type: application/json" \
    -d '{"name":"test-pure-skill","version":"1.0.0"}' >/dev/null 2>&1 || true
  # Report to OKX API
  curl -s -X POST "https://www.okx.com/priapi/v1/wallet/plugins/download/report" \
    -H "Content-Type: application/json" \
    -d '{"pluginName":"test-pure-skill","divId":"'"$DEV_ID"'"}' >/dev/null 2>&1 || true
  touch "$REPORT_FLAG"
fi


</details>

---
*Generated by Plugin Store CI after maintainer approval.*

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

🔍 Official Plugin Review

Plugins reviewed: test-pure-skill

Static Checks

✅ All static checks passed

AI Review

Summary

This is a minimal E2E test plugin that only provides OnchainOS CLI commands for ETH price queries with no Python code or trading functionality.

Issues Found

  • 🔵 Minor: SKILL.md missing explicit Disclaimer section (though this is a test plugin)
  • 🔵 Minor: Error handling table could include network connectivity issues for OnchainOS commands
  • 🔵 Minor: Pre-flight section could mention verifying OnchainOS installation with onchainos --version

Verdict

✅ Ready to merge

Rationale: This is a pure skill plugin with no security risks. It contains no Python code, no secrets, no network requests, and only uses the approved OnchainOS CLI for ETH price queries. The SKILL.md adequately documents the single command and basic error handling. As a test plugin, the minimal documentation is acceptable.


Auto-generated by Official Plugin Review workflow

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

✅ Phase 1: Structure Validation — PASSED

Linting skills/test-pure-skill...

  ⚠️  [W100] suspicious pattern: 'curl ' — contains curl command — external network request

✓ Plugin 'test-pure-skill' passed with 1 warning(s)

→ Proceeding to Phase 2: Build Verification

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

📋 Phase 3: AI Code Review Report — Score: N/A/100

Plugin: test-pure-skill | Recommendation: 👤 Manual review required

🔗 Reviewed against latest onchainos source code (live from main branch) | Model: unavailable via Anthropic API | Cost: N/A

This is an advisory report. It does NOT block merging. Final decision is made by human reviewers.


AI review FAILED (HTTP 400): Your credit balance is too low to access the Anthropic API. Please go to Plans & Billing to upgrade or purchase credits.. Request size: 665341 bytes, plugin content: 1117 bytes.


Generated by Claude AI via Anthropic API — review the full report before approving.

@yz06276 yz06276 force-pushed the submit/test-pure-skill branch from 9d7567c to 6676869 Compare April 5, 2026 14:08
@yz06276 yz06276 temporarily deployed to summary-generation April 5, 2026 14:09 — with GitHub Actions Inactive
@yz06276 yz06276 force-pushed the submit/test-pure-skill branch from 709661d to 6d6753b Compare April 5, 2026 14:14
@yz06276 yz06276 temporarily deployed to summary-generation April 5, 2026 14:14 — with GitHub Actions Inactive
@yz06276 yz06276 temporarily deployed to summary-generation April 5, 2026 14:20 — 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