trader-memory-core

Track investment theses across their lifecycle — from screening idea to closed position with postmortem. Register theses from screener outputs, manage state transitions, attach position sizing, review due dates, and generate postmortem reports with P&L and MAE/MFE analysis. Trigger when user says "register thesis", "track this idea", "thesis status", "review due", "close position", "postmortem", or "trading journal".

Quality

91%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

Securityby

Low

Low-risk findings worth noting

Quality

Content

82%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A strong, highly actionable body with concrete commands throughout and validation embedded in the tooling. It loses a little on conciseness (length and rationale prose) and progressive disclosure (two non-bundle resource links, inlined detail).

Suggestions

Tighten the prerequisites/CLI-launcher troubleshooting prose and the futures re-attach rationale to the essential one-liners; the operational commands already carry the signal.

Either move schemas/thesis.schema.json into the skill bundle or relabel it as a repo-relative path, and verify the ../../examples/workflows/... sample-run link resolves from the skill's location.

Consider splitting the long Futures positions subsection (contracts/multiplier/direction, P&L formula, attach guards) into a references/futures_positions.md so the main workflow stays a concise overview.

Dimension	Reasoning	Score
Conciseness	Largely lean and assumes Claude's competence, but the ~345-line body carries some prose that could be trimmed — the uv-routing troubleshooting block and the multi-paragraph rationale for why futures re-attach is stricter — pushing it just below the lean-anchor 5.	4 / 5
Actionability	Dense with copy-paste-ready bash commands, concrete subcommands and flags, a full manual-ingest JSON example, and Python function signatures covering the common register/query/update/review/postmortem cases.	5 / 5
Workflow Clarity	Five clearly sequenced phases each with concrete commands, and validation is baked into the tooling (schema validation on save, fail-closed ingest, atomic writes, status invariants); it stops short of explicit validate-then-proceed checkpoints in the lifecycle flow, leaving a minor gap below anchor 5.	4 / 5
Progressive Disclosure	Well-organized into Overview/When-to-Use/Prerequisites/Workflow/Output Format/Resources with one-level-deep references (thesis_lifecycle.md, field_mapping.md both present), but two Resources links point outside the bundle (schemas/thesis.schema.json and the ../../examples path) and a fair amount of operational detail is inlined rather than split out.	4 / 5
	Total	17 / 20 Passed

Description

100%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

An exemplar description: it states a concrete capability set, gives an explicit trigger clause with natural user phrasing, and carves out a distinct niche with minimal conflict risk. No vague fluff or over-claims.

Dimension	Reasoning	Score
Specificity	Lists multiple concrete actions — "register theses from screener outputs", "manage state transitions", "attach position sizing", "review due dates", "generate postmortem reports with P&L and MAE/MFE analysis" — giving comprehensive coverage of the skill's capabilities.	5 / 5
Completeness	Explicitly answers both what (track theses across their lifecycle, register, transition, size, review, postmortem) and when ("Trigger when user says ..."), with concrete trigger phrases.	5 / 5
Trigger Term Quality	Provides comprehensive natural trigger phrases a user would actually say ("register thesis", "track this idea", "thesis status", "review due", "close position", "postmortem", "trading journal"), including synonyms like "trading journal"/"thesis status".	5 / 5
Distinctiveness Conflict Risk	Occupies a clear niche (investment-thesis lifecycle / trading journal) with distinctive triggers like "register thesis" and "track this idea" that are unlikely to fire for the surrounding screener/position-sizer skills.	5 / 5
	Total	20 / 20 Passed

Validation

100%

Warnings & errors only

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository: tradermonty/claude-trading-skills
Path: skills/trader-memory-core/SKILL.md
Commit: 51c7907

Reviewed: ~6 hours ago

Table of Contents

Discovery Implementation Validation

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.