Skip to content

feat(risk): stamp consecutive_losses before strategy evaluate - #322

Merged
Pigbibi merged 2 commits into
mainfrom
feat/wire-consecutive-losses-metadata
Jul 9, 2026
Merged

feat(risk): stamp consecutive_losses before strategy evaluate#322
Pigbibi merged 2 commits into
mainfrom
feat/wire-consecutive-losses-metadata

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Before evaluate, stamp portfolio.metadata.consecutive_losses from QPK live-run equity history
  • Entrypoint risk gates can now trip the >5 consecutive-loss circuit breaker
  • Telemetry falls back to portfolio_total_equity when writing live-run equity
  • Bump QPK pin to f6d5c2c (QPK #210)

Test plan

  • pytest consecutive-loss unit tests (3 passed)
  • CI green
  • After merge, confirm a strategy cycle with ≥2 equity snapshots populates consecutive_losses

Made with Cursor

Pigbibi and others added 2 commits July 10, 2026 06:30
Resolve trailing loss streak from live equity history into portfolio
metadata so entrypoint risk gates can trip the circuit breaker.

Co-Authored-By: Claude <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@cursor

cursor Bot commented Jul 9, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Pigbibi
Pigbibi merged commit 6407bb1 into main Jul 9, 2026
2 checks passed
@Pigbibi
Pigbibi deleted the feat/wire-consecutive-losses-metadata branch July 9, 2026 22:45
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