brainstorming
brainstorming
You MUST use this before any creative work - creating features, building components, adding functionality, modifying behavior, or planning anything new, in software or out of it (a talk, a business, a renovation).
trade-performance-coachReview closed trades, partial exits, and monthly trade aggregates for process adherence, risk discipline, execution quality, and evidence-based trading behavior patterns. Use after trader-memory-core and signal-postmortem have produced records, or when the user asks for a post-trade coach, risk-manager style review, rule-adherence review, next-session opera…
Independent GitHub snapshot · Automated classification · Original documentation governs
Trade Performance Coach reviews recorded trade outcomes and journal evidence to help a human trader improve their decision process. It converts closed-trade records, postmortem findings, risk rules, and optional market-regime context into an evidence-based coaching report covering:
This skill is intended to fill the support role that a risk manager, desk lead, or trading coach might provide in a professional trading environment. It is strictly a process-review skill: it never recommends entering, exiting, buying, selling, shorting, holding, or sizing a specific security.
Use this skill when any of the following are true:
trader-memory-core thesis records and signal-postmortem findings and wants next-session operating rules.Do not use this skill to:
trader-memory-core; this skill consumes journal/thesis records and produces coaching findings.If the input is incomplete, default to REVIEW_REQUIRED or journal_only mode and ask for missing records rather than inventing evidence.
Recommended upstream records:
trader-memory-core closed thesis record or journal entrysignal-postmortem postmortem findingsmarket-regime-daily / exposure-coach contextNo paid API key is required. The deterministic script works from local JSON/YAML-like records.
Minimum useful input is one recorded trade or one monthly aggregate.
Preferred fields:
review_type: single_trade | partial_close | monthly_aggregate
trade_id: string
ticker: string
outcome: win | loss | breakeven | mixed
planned:
thesis: string
entry: number
stop: number
target: number
risk_r: number
thesis_recorded_before_entry: boolean
setup_confirmed: boolean
market_regime: allowed | restrictive | cash_priority | unknown
actual:
entry: number
exit: number
risk_r: number
portfolio_heat_r: number
stop_moved: boolean
stop_move_planned: boolean
entry_before_confirmation: boolean
traded_against_regime: boolean
risk_plan:
max_risk_per_trade_r: number
max_portfolio_heat_r: number
max_weekly_loss_r: number
postmortem:
root_cause: thesis_quality | execution | risk_sizing | market_environment | rule_violation | randomness | unknown
notes: [string]
journal:
reflection: string
emotions: [string]
monthly:
trades: [object]
consecutive_losses: number
rule_violations: number
The script tolerates partial records. Missing evidence is marked as unclear.
For the numeric fields actually evaluated (planned.risk_r, actual.risk_r,
risk_plan.max_risk_per_trade_r, actual.portfolio_heat_r,
risk_plan.max_portfolio_heat_r, and monthly.consecutive_losses), supplied
non-null values must be finite and nonnegative. Numeric strings and zero are
accepted; consecutive losses must be a whole number. Booleans, negative values,
NaN, infinity, malformed strings, and conversion overflow are rejected. An
explicitly invalid maximum never falls back to planned risk. Missing/null fields
retain the partial-record behavior. The CLI validates every source record,
including multiple inputs, and returns exit code 2 with a field-specific error
before creating or modifying reports when a numeric value is invalid.
This skill remains beta. Numeric validation does not establish production readiness; report-ID path safety and the documented shallow multi-input wrapper still require separate assessment.
Collect the most recent closed trade record, postmortem, risk plan, and journal notes.
python3 skills/trade-performance-coach/scripts/review_trade_performance.py \
--input reports/trade_memory/closed_thesis_EXMPL.json \
--output-dir reports/trade-performance-coach
Compare actual actions against the user's documented plan and rules. Check for:
Compare actual risk and heat against the risk plan. Check for:
Classify entry, stop, exit, add, trim, and review behavior. Separate clean-process losses from execution mistakes.
Use evidence from journal notes and action flags to tag possible trading behavior patterns. Always tie a tag to evidence and use non-diagnostic language.
Supported MVP tags:
fomo_entryrevenge_tradepremature_exitoverconfidence_after_winnerstop_movedsize_creephesitationrule_driftno_pattern_detectedConvert findings into temporary, concrete guardrails. Examples:
End every report with a human decision gate. The default action is journal_only.
Allowed actions:
accept_rules / modify_rules / defer / journal_only
The skill produces a JSON report and optionally a Markdown report.
Required top-level JSON fields:
schema_versionreview_typereview_idoverall_verdictsummaryscoresprocess_adherence_findingsrisk_manager_notesexecution_quality_assessmentbehavioral_pattern_tagsnext_session_operating_rulescoach_questionshuman_decision_gatedisclaimerVerdicts:
| Verdict | Meaning |
|---|---|
OK | No material process violation found. Outcome appears compatible with the plan. |
WARN | Minor process or record-quality concern. |
REVIEW_REQUIRED | Meaningful process, risk, or behavior finding before next similar trade. |
RULE_VIOLATION | Explicit user rule appears to have been broken. |
COOL_DOWN | Repeated violations, drawdown/revenge pattern, or escalation suggests review-only mode. |
python3 skills/trade-performance-coach/scripts/review_trade_performance.py \
--input skills/trade-performance-coach/scripts/tests/fixtures/single_trade_rule_violation_loss.json \
--output-dir reports/trade-performance-coach \
--markdown
Read these selectively when invoked:
references/review-framework.md — five-axis review model, scoring, verdictsreferences/behavior-tags.md — behavior tag definitions and evidence rulesreferences/risk-review-checklist.md — risk manager checklist and severity rulesreferences/output-contract.md — JSON output contract and schema notesreferences/hermes-integration.md — suggested Hermes /post-trade-coach and monthly coaching integrationassets/performance_coach_report.schema.json — machine-readable output schemascripts/review_trade_performance.py — deterministic local reviewer