Repository navigation
Conversation
Derives an auditable per-source obligation inventory from context source coverage so native preflight can name truncation gaps before provider calls. Co-authored-by: Cursor <cursoragent@cursor.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Summary
|
Closes #3820
Current acceptance boundary (closer, 2026-10-11)
Raw capture byte indexing/reassembly and failure-to-FAIL guards are implemented.
Semantic cross-proof obligation packages, complete exact-judge partition execution,
missing-proof recovery and authentic complete native comparison remain outstanding.
Source #3820 stays open; generated completion checkboxes below are not acceptance
proof. Current evidence and next action: #3831 (comment)
Why
Expanded native verifier runs on merged #3828 now capture complete changed-code bytes, but native preflight still blocks with input/context overflow. Source #3820 needs an auditable obligation contract before prompt partitioning/recovery can claim completeness.
Tasks
tools/verifier_capture_obligation_manifest.pyto parse builder context source coverage and emitworkflows-verifier-capture-obligation-manifest/v1with explicit per-source gaps.Acceptance Criteria
Test plan
python3 -m pytest tests/tools/test_verifier_capture_obligation_manifest.py -qCloses #3820
Automated Status Summary
Scope
Workflows#3802 merged as
427798df1c0dd46a39d1f94e809b8959952d9ca4, but actual post-merge comparisons standard38059611776 and expanded38059805990 both returned NON_PASS/CONCERNS due incomplete inputs. Expanded included 127635/257099 changed-code characters (49.64%); comments exhausted shared character allowance, associated-run discovery exhausted its cap, and nine artifact records were individually truncated. Workflow SUCCESS is not acceptance. This blocks the source-owned dependency sync campaign, but campaign tracker#1836 is background coordination, not this implementation contract.The completed bounded Sol6.1Medium assessment reproduced the capture offline and identified no existing safe recovery. Full result and inputs are available to the local owner; there is no authorization to waive completeness, select a clean subset while claiming complete discovery, fabricate reviewer acceptance, or mutate generated PRs.
Context for Agent
Related Issues/PRs
Tasks
Acceptance criteria