Skip to content

Split notification targets from strategy plugin consumers#7

Merged
Pigbibi merged 1 commit into
mainfrom
codex/split-notification-targets-20260529
May 28, 2026
Merged

Split notification targets from strategy plugin consumers#7
Pigbibi merged 1 commit into
mainfrom
codex/split-notification-targets-20260529

Conversation

@Pigbibi
Copy link
Copy Markdown
Contributor

@Pigbibi Pigbibi commented May 28, 2026

Summary

  • split general market-regime notifications into notification_targets instead of a synthetic strategy consumer
  • emit notification target artifacts with target_type=notification_target and no strategy field
  • update runner docs, examples, and bilingual notification/log wording

Verification

  • PYTHONPATH=src .venv/bin/pytest tests/test_strategy_plugin_runner.py -q
  • PYTHONPATH=src .venv/bin/pytest -q
  • PYTHONPATH=src .venv/bin/ruff check .
  • git diff --check

@Pigbibi Pigbibi merged commit 254c05e into main May 28, 2026
1 check passed
@Pigbibi Pigbibi deleted the codex/split-notification-targets-20260529 branch May 28, 2026 19:25
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