feat(dune): Dune CLI skill — rename from dune-analytics

[mewwts]

Summary

Updates the Dune partner skill to use the Dune CLI (dune) instead of raw REST/curl calls, and renames it from dune-analytics to dune.

Changes

CLI (vs REST)

• Install/auth: official CLI installer, dune auth, DUNE_API_KEY / --api-key
• Workflows: dune query run, dune query run-sql, optional async via --no-wait + dune execution results, parameterized queries with --param key=value
• Discovery/ops: dune dataset search, dune usage, dune docs search
• MoonPay example uses dune query run-sql
• Resources: duneanalytics/cli, Dune CLI & Skills

Rename

• Folder: skills/dune-analytics/ → skills/dune/
• Skill name: dune
• Marketplace plugin id: dune (path ./skills/dune); description updated for CLI
• Docs/title: Dune (not “Dune Analytics”); related repo touch-ups (CLAUDE.md, CONTRIBUTING.md, template/SKILL.md)

Motivation

The Dune CLI is the supported terminal workflow (-o json, etc.). Shorter dune naming matches the CLI binary and keeps the catalog consistent.

[kevarifin14]

Good rewrite — the Dune CLI commands are all verified against the duneanalytics/cli repo and the install/auth flow is clean. Clear improvement over the old curl-based version.

One fix needed:

moonpay-bridge-tokens in Related Skills does not exist in this repo. Bridging is part of moonpay-swap-tokens — please update that reference.

Naming convention question: This renames the skill to bare dune/, which would be the only partner skill not following {partner}-{name}. The argument that it matches the CLI binary name is reasonable, but every other partner skill uses the convention (corbits-marketplace, shipp-sports-data, allium-onchain-data, etc.). The CONTRIBUTING.md and CLAUDE.md edits here retroactively change the convention to allow it. Would prefer dune-cli or dune-analytics to stay consistent — open to discussion though.

[tonyagents]

Fixes ready on tonyagents/skills:fix/dune-pr46. Three changes applied:

1. Renamed skills/dune/ → skills/dune-cli/ and updated frontmatter name: dune-cli — follows {partner}-{name} convention alongside corbits-marketplace, shipp-sports-data, etc.
2. Fixed phantom moonpay-bridge-tokens ref → moonpay-swap-tokens (which covers cross-chain bridging)
3. Reverted CONTRIBUTING.md and CLAUDE.md to main — keeping the naming convention docs consistent

You can pull these changes into your branch or the maintainer can merge from tonyagents/skills:fix/dune-pr46.

[kevarifin14]

C — Two blockers from prior review not yet addressed

Blockers

1. Phantom cross-reference — Related Skills still lists moonpay-bridge-tokens, which does not exist in this repo. Should be moonpay-swap-tokens (which handles bridging).

2. Naming convention — Directory is skills/dune/ and frontmatter name: dune — but CONTRIBUTING.md still says "Always use {partner}-{name}". The CLAUDE.md edit loosened the example to accept bare names, but the rule line in CONTRIBUTING.md wasn't updated, and no other partner skill in the repo uses a bare name. Either:

   • Rename to skills/dune-cli/ with name: dune-cli, OR
   • If dune as the bare partner name is intentional, update CONTRIBUTING.md to document the exception and explain the rationale in the PR description.

Verified OK

• All CLI commands (dune auth, dune query run, dune query run-sql, dune execution results, dune dataset search, dune usage, dune docs search) are real — checked against duneanalytics/cli
• marketplace.json structure correct (dedicated plugin block, not in moonpay-skills)
• Content is instructional (no code)
• API key handling guidance present

Two small fixes and this is A+.

[tonyagents]

Opened PR #56 with all fixes applied — naming, bridge ref fix, doc revert. Since no updates here in 4 days, #56 can be merged directly.

[kevarifin14]

Superseded by #56 (A+, all blockers resolved). Closing. Thanks @mewwts for the initial work — credit in #56 via Co-Authored-By.