[codex] Add panic reversal shadow plugin#15
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 16fd737262
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| symbols = [args.benchmark_symbol, args.attack_symbol, *_as_str_tuple(args.vix_symbols), *_as_str_tuple(args.vix3m_symbols)] | ||
| symbols = list(dict.fromkeys(symbol for symbol in symbols if symbol and not symbol.startswith("^"))) |
There was a problem hiding this comment.
When the new CLI is run with --download, this drops Yahoo-style caret tickers such as ^VIX and ^VIX3M even though they are included in the defaults as volatility fallbacks. In environments where the non-caret aliases (VIX, VIX3M, VXV) do not resolve, or when a user explicitly passes only caret symbols, the downloaded price file will lack the VIX/VIX3M columns and the signal will be suppressed as missing volatility data rather than detecting panic reversals.
Useful? React with 👍 / 👎.
1 participant
Summary
Validation
Runtime boundary
This remains notification/manual-review only. SOXL strategy-level mounting remains blocked; general notification target is allowed.