Repository navigation
Refuse intake policy promotion without paired wins over every baseline - #98
Merged
Merged
Conversation
…com/unbraind/pm-rl into feat/rsi-beats-baseline
…ons' into feat/rsi-objective
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
unbraind
marked this pull request as ready for review
October 25, 2026 03:12
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
unbraind
commented
Oct 25, 2026
unbraind
left a comment
Owner
Author
There was a problem hiding this comment.
Orchestrator review (built-in Claude review agent, verified by the orchestrator; bots are not waited for since 2026-10-22):
VERDICT: MERGE
FINDINGS:
- none
SUMMARY: The PR adds balanced intake scoring, paired bootstrap gates against every baseline, and refusal recovery while preserving main’s history bytes. The inspected tests cover gate decisions, replay, crash boundaries, and Git-free trackers; I did not execute them.
STOP.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Deliver the balanced intake objective, mandatory baseline promotion gate and durable evidence receipt. This PR includes repaired PR #95 commit
2ecb61fand current main0025c8b; it can be reviewed against main without waiting for either PR to merge during this run. No measured win is claimed: the fresh public experiment again has zero promotions.Before: closed-status recognition is dominated by the majority constant. After: creation-title Issue routing uses stochastic balanced accuracy, so constants score exactly 0.5, and every vector candidate must clear paired stratified item-bootstrap lower effects strictly above 0.01 against the incumbent, both constants and a fixed non-learning keyword router. 8,192 replicates, four-comparison Bonferroni adjustment, at least ten held-out items per class. Original exact/sampled/gap/contamination/checkpoint/approval/resource gates remain required. Later edits, identities, authors and timestamps are not model features.
Conflicts resolved: epic/baseline/lintruntime tracker items and append-only history unions, controller imports and objective dispatch, packaging, corpus regressions and source snapshot ordering. Main's newer evaluation/card/CAS/decoder behavior survives; intake sealing now uses the same deterministic ordering. CHANGELOG.md was regenerated inside the merge. The continuation instead restores main-based chains and re-records branch intent, keeping main records byte-identical.
pm test --run --only-last --progress, snapshot workspace, PM context nonenpm run typecheck,npm run lint,npm run build, regenerated changelog checkFresh real-data run: public MIT
unbraind/pm-opsateba7d5262fd75e67afce0caad29dae7dc7dfbaeb; 300 unchanged source files; 104 training (68 Issue / 36 other), 42 held-out (22 Issue / 20 other).All three refuse. Three jobs charge 256 steps each once, collect 3,072 actions, preserve every checkpoint on resume and launch no new jobs. Execution uses a plain tracker and a PATH without Git. See the fresh and historical receipt and objective/cycle report. These are conditional validation results, not an independent final-test, forecasting or transfer win.
The documented installed cycle is:
A separately declared Decision bounds promotions and resources. Run executes collect → train → evaluate → gate → promote-or-refuse → record on Generation/process/policy-card items and the terminal seed comment. Only the retained promoted incumbent collects the next generation; refused proposals never do. Resume uses the original programme and allowances.
Tracker health is unblocked. This branch merges repaired PR #95. The epic history extends main exactly. Because the baseline item is absent on main, its healthy PR #95 item/history were retained as the branch base, then twelve objective-specific comments, three files, two docs, both complete test links and the acceptance criterion were re-recorded with pinned pm commands. Both prefix checks pass. Only this merge's stale clone-local receipt was quarantined, as recommended by the CLI; normal reconciliation had written no audit history. Normal pinned strict health reports
ok:true, with the unchanged main provenance advisory. No main history, force, history-repair or re-anchor operation was used. Runtime source, tests, thresholds and dependency pins are unchanged. All complete current-head variants now pass. Each completed log was independently downloaded and checked: 1,023 tests per runtime, all 54 authored sources at unchanged four-metric 100% thresholds, 408 documented declarations, plus successful health, lint, duplication, identity, audit, attestation, packaging and changelog checks. CodeQL passes. The gate/objective/receipt, paired randomized properties, foreign-record/owner interleavings and crash/recovery suites meet this infrastructure acceptance. Zero real-data promotions remain the honest measured result. Earlier health-failing runs remain historical failures.PM: pm-rl-baseline, parent pm-rl-apvf; existing health follow-up pm-rl-9uz4. Claim released, items unclosed. Commits:
8105fcb(merge/source/evidence),ad27b2d(stronger lower-bound properties and CI receipt). Health merge commit:f35df8f. Ready for review after full CI and acceptance verification. No PR merge or release.