feat: add dawn skills — local Polymarket trading strategy platform#38
feat: add dawn skills — local Polymarket trading strategy platform#38tonyagents wants to merge 4 commits intomoonpay:mainfrom
Conversation
- dawn, dawn-auth, dawn-sdk-tools, dawn-strategy: full workflow skills for the @dawnai/cli trading strategy platform (MoonPay-built, Polymarket execution) - perp-arb: funding rate arbitrage scout across Hyperliquid, Pacifica, Lighter, Aster - perp-portfolio: multi-exchange perpetual futures portfolio aggregator - perp-trade: perp order execution with pre-trade checks, TWAP/DCA/grid/trailing-stop bots Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds dawn-skills plugin block (dawn, dawn-auth, dawn-strategy, dawn-sdk-tools). Skill files were added in c89b1ae. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
kevarifin14
left a comment
There was a problem hiding this comment.
Grade: C — Needs work
The 4 Dawn skills are well-structured and close to mergeable, but this PR also includes 3 undisclosed perp skills with significant issues. The diff must be split or the perp skills must be brought up to standard.
Dawn skills (dawn, dawn-auth, dawn-strategy, dawn-sdk-tools)
Verified:
@dawnai/cliis a real npm package (v1.2.5, published by njdawn) — confirmed vianpm viewdawnbinary is installed and all subcommands in the skills (auth login/status/logout,wallet list/use/current/create,strategy launch/list/logs/stop/positions,template list/launch,tool list/run/docs,portfolio current/sell/redeem,skill list/install) matchdawn --helpoutput exactlyows(Open Wallet Standard) is real —@open-wallet-standard/coreon npm, binary installed at~/.ows/bin/owsmoonpay buyandmoonpay token bridgeare real commands- marketplace.json: correctly placed in a separate
dawn-skillsplugin block - Frontmatter: all 4 skills have
name,description,tags— descriptions are trigger-specific - Cross-references: dawn skills only reference each other (all within this PR) — valid
- Security: execution tools clearly marked, explicit user confirmation required before trades, budget enforcement pattern documented
Issues to fix:
-
Naming convention violation —
skills/dawn/is a bare partner name. The convention isskills/{partner}-{name}/(see CONTRIBUTING.md "Common Mistakes" #1). Rename toskills/dawn-workflow/orskills/dawn-operations/or similar. -
moonpay buy --token pol_polygonis not a valid token code — Indawn-auth/SKILL.md, the wallet funding section instructs:moonpay buy --token pol_polygon --amount 2 --wallet <your-address>pol_polygonis not in the supported token list (moonpay buy --helplists: sol, eth, btc, trx, usdc, usdc_sol, usdc_base, usdc_arbitrum, usdc_optimism, usdc_polygon, usdc_cchain, usdt, usdt_sol, usdt_trx, usdt_ton, usdt_bsc, usdt_arbitrum, usdt_optimism, usdt_polygon, pyusd, pyusd_sol, rlusd). Either remove this line or note that POL must be acquired elsewhere (bridge, swap, or exchange withdrawal).
Perp skills (perp-arb, perp-portfolio, perp-trade) — NOT mergeable
These 3 skills are included in the diff but not mentioned in the PR description (which says "Skills added (4)"). They have multiple rubric failures:
-
Not registered in marketplace.json — The diff only adds a
dawn-skillsblock. The 3 perp skills need their own plugin block. -
No Prerequisites section — None of the 3 perp skills list setup requirements (CLI installs, auth, API keys, wallet config).
-
Unverifiable CLI commands — The skills reference
perp-cliMCP tools (arb scan,trade,account,portfolio,bot) without specifying how to install or configure them. Whileperp-cliexists on npm (binary name:perp), the specific MCP tool invocations (arb scan,tradeMCP tool,botMCP tools) are not verifiable from the skill text alone. Each skill must include install instructions and verifiable command syntax. -
No install instructions —
perp-cliis never mentioned as a dependency to install. The skills assume it exists without telling agents how to set it up. -
Template-style output, not executable commands — The perp skills describe output formats with placeholders (
[exchange],[asset],[bot-id]) but don't provide actual CLI commands an agent can run. They read more like prompt templates than skill instructions.
Recommendation: Split this PR. Submit the 4 Dawn skills separately (after fixing the 2 issues above). The perp skills need a rewrite with real, verifiable perp-cli commands, install instructions, prerequisites, and marketplace.json registration.
…out perp skills Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
kevarifin14
left a comment
There was a problem hiding this comment.
A- — One open question before merge
Prior feedback fully addressed:
- Renamed
skills/dawn/→skills/dawn-workflow/✓ - Removed
pol_polygonreference, clarified POL sourcing ✓ - Removed the perp skills (now split into PR #39) ✓
- Frontmatter updated to match directory name ✓
Verified: @dawnai/cli v1.4.0 on npm (published under MoonPay), all dawn commands match dawn --help, cross-refs (dawn-auth, dawn-strategy, dawn-sdk-tools) resolve.
Open question (Kevin)
I have local dawn* skills in my ~/.claude/skills/ directory that may conflict — specifically a dawn skill (bare name) that overlaps with this PR's dawn-workflow. Before merging, I want to confirm:
- Should my local
dawn/skill be retired in favor of this PR'sdawn-workflow/? - Or is the local version still canonical and this PR should align?
@tonyagents — once Kevin confirms the canonical version, we can merge. Otherwise structurally this is ready.
Minor nit (not blocking)
dawn-strategy: --name is required for dawn strategy launch but only shown in examples, not called out explicitly in the workflow text.
|
Re: the local |
kevarifin14
left a comment
There was a problem hiding this comment.
A+ — Ready to merge
@tonyagents confirmed dawn-workflow is canonical; my local dawn/ is superseded. --name requirement is now explicit across launch/checklist/error-handling sections.
Verified: @dawnai/cli v1.4.0 on npm, all dawn commands match dawn --help, cross-refs to dawn-auth/dawn-strategy/dawn-sdk-tools resolve, dawn-skills plugin block correct.
Merging.
|
Superseded by #59 (cherry-pick onto current main, marketplace.json conflict resolved). Closing. Credit preserved via Co-Authored-By. Thanks @tonyagents! |
Skills added (4)
Dawn is MoonPay's local prediction market trading platform. It runs Python strategy scripts as background processes against Polymarket, with a full SDK for market research, portfolio management, and trade execution.
@dawnai/cli, authenticate, post-login onboarding (templates / build from scratch / wallet setup + funding via MoonPay)Checklist
skills/(committed in c89b1ae).claude-plugin/marketplace.jsonunder separatedawn-skillsplugin blockdawn-{name}/@dawnai/clion npm, binarydawn