Skip to content

feat: execute full Basic strategy catalog - #145

Merged
pjy008008 merged 1 commit into
developfrom
feature/basic-strategy-full-catalog
Aug 7, 2026
Merged

feat: execute full Basic strategy catalog#145
pjy008008 merged 1 commit into
developfrom
feature/basic-strategy-full-catalog

Conversation

@pjy008008

Copy link
Copy Markdown
Contributor

What changed

  • Implements every direct Basic strategy operation displayed by the editor, including price, volume, streak, SMA, RSI, MACD, Bollinger, position-return, holding-period, peak-return, drawdown, schedule, and terminal order candidates.
  • Aggregates normalized one-minute events into the supported editor resolutions and evaluates only completed bars.
  • Connects released direct plans to the existing event-driven candidate convergence and exactly-once order-intent path.
  • Loads durable active-position metrics for sell-side conditions and skips official-feature warmup for direct-only plans.

Why

The editor exposed a full sample catalog, but the live trading runtime only supported the RSI path. A READY strategy must remain server-executable after personal-bot launch and must react to ordered real-time market events without browser participation.

Impact

Every editor-visible Basic block can now participate in live evaluation. Missing history or partial candles fail closed and cannot emit orders.

Peak and completed-bar counters are intentionally fail-safe in-memory state and restart from current durable position state after worker restart. Extraordinary one-off exchange closures still require an approved exchange-status authority.

Validation

  • :modules:strategy-runtime:test
  • :apps:trading-worker:test
  • Full worker evaluation-loop E2E, including a direct RSI-free price strategy producing a PostgreSQL order intent
  • git diff --check origin/develop...HEAD

Governance

This is a draft implementation of the isolated Basic catalog proposal. Protected specs/** and contracts/** were not changed. Integration and release require fresh exact-commit product-authority approval.

@pjy008008
pjy008008 marked this pull request as ready for review August 7, 2026 13:13
@pjy008008
pjy008008 merged commit 84f556f into develop Aug 7, 2026
2 checks passed
@pjy008008
pjy008008 deleted the feature/basic-strategy-full-catalog branch August 7, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant