feat: add nansen skills — 29 on-chain analytics skills from nansen-ai/nansen-cli#32
feat: add nansen skills — 29 on-chain analytics skills from nansen-ai/nansen-cli#32tonyagents wants to merge 35 commits intomoonpay:mainfrom
Conversation
kevarifin14
left a comment
There was a problem hiding this comment.
PR Review: 29 Nansen Skills
Verification Results
CLI package: nansen-cli@1.21.0 on npm — real package, published by asvanevik <alex@nansen.ai>. GitHub repo nansen-ai/nansen-cli exists. All verified.
Commands verified against nansen --help and subcommand help:
nansen research smart-moneysubcommands (netflow, dex-trades, perp-trades, dcas, holdings, historical-holdings) — all confirmednansen research profilersubcommands (transactions, pnl-summary, balance, counterparties, historical-balances, related-wallets, pnl, labels, perp-positions, perp-trades, search, batch, trace, compare) — all confirmednansen research tokensubcommands (flows, who-bought-sold, dex-trades, transfers, holders, pnl, perp-pnl-leaderboard, perp-positions, perp-trades, flow-intelligence, info, indicators, ohlcv, jup-dca, screener) — all confirmednansen research perp(screener, leaderboard) — confirmednansen research prediction-market(ohlcv, orderbook, top-holders, trades-by-market, market-screener, event-screener, pnl-by-market, pnl-by-address, position-detail, trades-by-address, categories) — confirmednansen research portfolio(defi) — confirmednansen research search— confirmednansen web(search, fetch) — confirmednansen wallet(create, list, show, send, export, delete, default) — confirmednansen wallet secure,nansen wallet forget-password— confirmed working (unlisted in help but functional)nansen trade(quote, execute) — confirmednansen alerts(list, create, update, toggle, delete) — confirmednansen research pmas alias forprediction-market— confirmed
All CLI commands in the 29 skills are real and verifiable. No hallucinated commands.
Rubric Assessment
Naming: All 29 skills follow skills/nansen-{name}/ convention. Correct.
Frontmatter: All skills have name and description. Descriptions are specific about when an agent should trigger the skill (e.g., "Is smart money exiting a token I hold?" rather than generic taglines). The metadata.openclaw block with requires, install, and allowed-tools is well-structured. No tags field, but that is optional per the template.
marketplace.json: Properly placed in a separate nansen-skills plugin block, not inside moonpay-skills. Correct.
No code: All skills are instructional guides using CLI commands. No embedded Python/TypeScript code. Correct.
Real commands only: Every CLI command verified. All real.
Prerequisites: Each skill lists NANSEN_API_KEY as required env var and nansen binary in the metadata.openclaw block. Install instructions (npm: nansen-cli) are present. However, none of the skills have a traditional "## Prerequisites" section in the body text like the template suggests — the metadata block covers it, but it's less human-readable.
Clean diff: Only touches marketplace.json and adds new skill directories. No unrelated changes.
Security: nansen-wallet-manager and nansen-wallet-keychain-migration have strong security guidance (never generate passwords, never store in logs, never use --human). nansen-trading warns about irreversible trades and base-unit amounts.
Issues Requiring Changes
1. Phantom reference: nansen-prediction-market skill (does not exist)
In nansen-polymarket-deep-dive/SKILL.md:
Use
market_idfrom the screener (nansen-prediction-marketskill).
The actual skill is named nansen-prediction-markets (plural). This should be corrected.
2. Phantom reference: nansen-wallet-migration skill (does not exist)
In nansen-wallet-manager/SKILL.md:
For detailed migration steps [...] see the nansen-wallet-migration skill.
The actual skill is nansen-wallet-keychain-migration. This should be corrected.
3. Phantom reference: REFERENCE.md (does not exist)
In nansen-wallet-clustering/SKILL.md:
Full attribution rules in REFERENCE.md.
There is no REFERENCE.md in this PR or repo. Either include it or remove the reference.
4. Missing heading/overview in two skills
nansen-sm-cross-chain-flows/SKILL.md and nansen-smart-money-trend/SKILL.md jump straight from frontmatter into a bash code block with no heading or overview text. Every other skill has at least a heading and a one-line "Answers:" prompt. These two should be consistent with the rest.
Minor Notes (non-blocking)
- Skills use
nansen research pmas an alias fornansen research prediction-marketin several places. The alias works but is not documented in the main CLI help. The skills themselves document this, which is fine. - The
nansen-smart-alertsskill notes alerts are "internal-only" (requires Nansen internal API key). This is clearly documented but worth confirming this is intentional for a public skill repo. - The lack of a formal
## Prerequisitesbody section across all 29 skills is a style deviation from the template, but themetadata.openclawblock serves the same purpose for machine consumption.
Verdict
Grade: B — Close to A+, but the three phantom references are rubric violations ("No phantom references: Don't reference skills that don't exist in the repo"). These are straightforward fixes. The two missing headings are minor structural inconsistencies. Fix these five items and this is ready to merge.
|
All issues from the review have been addressed in the commits pushed on Mar 26:
Ready for re-review when you have a moment — thanks! |
e496f0c to
26325d0
Compare
…n-prediction-markets
…-wallet-keychain-migration
26325d0 to
75ef5cb
Compare
|
All 5 items from the review have been addressed:
The fixes were included in the rebase commits. Ready for re-review. |
kevarifin14
left a comment
There was a problem hiding this comment.
A+ — Ready to merge
All 5 items from my 2026-03-25 review addressed:
nansen-polymarket-deep-dive→ correctnansen-prediction-marketsref ✓nansen-wallet-manager→ correctnansen-wallet-keychain-migrationref ✓nansen-wallet-clustering→ phantomREFERENCE.mdremoved ✓nansen-sm-cross-chain-flows→ heading + overview added ✓nansen-smart-money-trend→ heading + overview added ✓
Verified:
nansen-cliv1.27.0 published on npmnansen-ai/nansen-clirepo exists, MIT license- All 29 skill CLI commands cross-referenced against repo README
Rubric: naming nansen-{name} consistent across all 29; own nansen-skills plugin block; no code; prerequisites (NANSEN_API_KEY + install) documented; security guidance on keychain, trading base units, and internal alerts all present.
Merging.
|
Superseded by #55 (cherry-pick onto current main with marketplace.json conflict resolved + signed commits). Closing. Credit preserved via Co-Authored-By on the new commit. Thanks @tonyagents! |
Skill name
29 Nansen skills (nansen-defi-positions, nansen-exit-signals, nansen-fund-tracker, nansen-general-search, nansen-holder-analysis, nansen-perp-screener, nansen-perp-trader-profile, nansen-polymarket-deep-dive, nansen-polymarket-insider-scan, nansen-polymarket-trader-profile, nansen-portfolio-tracker, nansen-prediction-markets, nansen-sm-cross-chain-flows, nansen-smart-alerts, nansen-smart-money-alpha, nansen-smart-money-tracker, nansen-smart-money-trend, nansen-token-research, nansen-token-screener, nansen-token-transfer-analysis, nansen-trading, nansen-wallet-batch, nansen-wallet-clustering, nansen-wallet-deep-dive, nansen-wallet-keychain-migration, nansen-wallet-manager, nansen-wallet-profiler, nansen-web-fetcher, nansen-web-searcher)
Note: nansen-dca-tracker is in a separate PR (#31).
Description
Complete Nansen skill suite sourced directly from https://github.com/nansen-ai/nansen-cli/tree/main/skills — the official Nansen CLI skills repo. Covers:
Primary chain
Multi-chain: Ethereum, Solana, Base, Arbitrum, Polygon, and more
Primary token
NANSEN_API_KEY required (env var)
Checklist
MoonPay Integration
Act on Nansen signals with MoonPay — fund wallets and execute trades based on smart money intelligence:
Example Usage