Split monolithic onboarding skill into focused, composable skills#2
Merged
matmilbury merged 3 commits intomainfrom Apr 24, 2026
Merged
Split monolithic onboarding skill into focused, composable skills#2matmilbury merged 3 commits intomainfrom
matmilbury merged 3 commits intomainfrom
Conversation
When openclaw detects a missing binary dependency, it now has structured metadata to tell users how to install it. Install hints are declared at both the plugin level (openclaw.plugin.json) and the skill level (SKILL.md frontmatter). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…skills The 894-line onboarding SKILL.md was too large for LLMs to follow reliably — during testing, the agent used getBalance() instead of getDepositRecords() to monitor deposits, skipped the funding-to-trading transfer, and jumped phases without confirming completion. Replace with 6 lean skills: - trade (router): always-active, detects user state, auto-onboards new users - market-scan: TA scanning, trade ideas, symbol deep-dives - deposit: deposit monitoring, funding-to-trading transfer (the broken flow) - open-position: trade execution with proper sizing and confirmation - close-position: close positions, cancel orders, modify TP/SL - withdraw: withdrawal flow Each skill includes only the SDK methods it uses and emphasizes the critical steps that were missed (e.g. deposit skill explicitly warns against using getBalance() to check deposits). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace references to removed onboard skill with the 6 new skills. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
onboardSKILL.md with 6 focused skills (trade,market-scan,deposit,open-position,close-position,withdraw), each containing only the SDK methods it needsgetBalance()instead ofgetDepositRecords()for deposit monitoring and skip the funding-to-trading transfer — the new skills explicitly guard against these mistakesTest plan
tradeskill correctly routes new users through onboarding flowgetDepositRecords(), notgetBalance()Generated with Claude Code