Skip to content

feat(v21): Learned Policy decision outcome closed loop successor - #379

Closed
laiqian0239-glitch wants to merge 17 commits into
mainfrom
product/v21-learning-policy-p1-decision-outcome-closed-loop-v2-successor
Closed

feat(v21): Learned Policy decision outcome closed loop successor#379
laiqian0239-glitch wants to merge 17 commits into
mainfrom
product/v21-learning-policy-p1-decision-outcome-closed-loop-v2-successor

Conversation

@laiqian0239-glitch

Copy link
Copy Markdown
Owner

Work package

V21-LEARNING-POLICY-P1-DECISION-OUTCOME-CLOSED-LOOP-V2-SUCCESSOR

Authorized by ordinary two-parent merge c5b7d23dcd6a7aafef6f1b8749369ab008606c8b from #378.

Fresh failure-first topology

Exact current Head: 4d19aff5dae72b906d47276811ff8ad7b096c69d

It is the direct child of the fresh authorization merge and adds exactly the six authorization-frozen WP0 tests, with zero product/runtime/dependency implementation.

  • changed paths: exactly 6
  • six-test path-set SHA-256: 9aff346a55b16f1ae54a743daf15bb18cffc4c1471581556be1eea22acab3142
  • additions: 317
  • production code changed: false

The six test blobs are byte-identical to immutable historical #372 RED evidence, but this commit has new identity and fresh parentage from the post-ACV2 trusted main authorization. No historical commit was cherry-picked or rebased.

Expected fresh causal RED

Only the bounded missing Learned Policy contracts count:

  • canonical DecisionRecord identity/persona/feature/action provenance;
  • immutable raw OutcomeVector and eligible candidate_sent source binding;
  • read-only projectPolicy over eligible immutable evidence;
  • Vowpal Wabbit 9.11.2 sealed runtime and supply-chain closure;
  • production consumption of only candidateStrategyBranch before existing frontier generation.

ACV2 Schema 23 and all existing authorities must remain GREEN on this test-only Head.

Authorized GREEN boundary

Exactly the original 22 paths, SHA-256 930bf654a0088c6729ca265443a30e77e895ea0f7a69517df51ac4f207008ff6. Vowpal Wabbit 9.11.2 (122bae254a5b8bc2b774d13b33d53e6dbc2cfba7, BSD-3-Clause) owns the bounded contextual-bandit action head. Existing Learning/Growth Brain, Person/Persona, immutable learning ledger, Langfuse reward authority and Model Brain/LiteLLM remain authoritative.

P1 remains actionProbability=1.0, exploration=false; final reply remains frontier-generated.

No product implementation may be added until this exact Head records fresh causal RED. No workflow/root npm mutation, live randomized exploration, local reply generator, second reward/model gateway/feature-store infrastructure, release, publish, rebase, squash or force push.

@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@laiqian0239-glitch, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 62 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e0840b39-a8e2-4f39-aa57-5d3757bead06

📥 Commits

Reviewing files that changed from the base of the PR and between a45b11d and f431223.

⛔ Files ignored due to path filters (1)
  • runtime/learning-growth/python/uv.lock is excluded by !**/*.lock
📒 Files selected for processing (19)
  • THIRD_PARTY_NOTICES.md
  • backend/services/contextAwareReplyBrain.js
  • backend/services/learningDeepTrainingContract.js
  • backend/services/learningOutcomeAttributionService.js
  • backend/services/learningPolicyDecisionContract.js
  • backend/services/learningPolicyRuntimeAdapter.js
  • backend/services/replyFeedbackLearningService.js
  • backend/services/storeManagerService.js
  • config/upstreams/v21-learning-growth-brain-p0.json
  • runtime/learning-growth/python/learning_entrypoint.py
  • runtime/learning-growth/python/pyproject.toml
  • tests/wp0/v21-learning-policy-p1-decision-record.test.js
  • tests/wp0/v21-learning-policy-p1-outcome-binding.test.js
  • tests/wp0/v21-learning-policy-p1-production-consumption.test.js
  • tests/wp0/v21-learning-policy-p1-projection.test.js
  • tests/wp0/v21-learning-policy-p1-supply-chain.test.js
  • tests/wp0/v21-learning-policy-p1-vw-runtime.test.js
  • third_party/licenses/vowpal-wabbit-BSD-3-Clause.txt
  • tools/uat/v21LearningPolicyClosedLoopEvidence.js

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

chore(v21): refresh Learned Policy successor from repaired main

Copy link
Copy Markdown
Owner Author

Superseded by the ordinary-merged V3 production-closure authority and implementation. The merged V3 authorization explicitly records PR #379 / head f431223dcd44e42dd7f1f81e02bd4d2f2949919c as immutable audit evidence that is not mergeable, and forbids reuse of this history as executable authority. V3 implementation PR #384 was ordinary-merged to trusted main as 9252ebba53d0e6d4bd0388a88ede2d0e74c7164c. Closing this stale V2 PR without rebasing, rewriting, cherry-picking, or touching its audit history.

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