Repository navigation
Hoist three body-scope annotation blocks so the required floor can parse again - #11874
Closed
gunbai-bot[bot] wants to merge 1 commit into
Closed
gunbai-bot[bot] wants to merge 1 commit into
gunbai-bot[bot] wants to merge 1 commit into
Conversation
…rse again DESIGN 4c admits only standalone leading // blocks at module-item grain. These three landed while the required floor was fail-open (#11829), so their 13 parse errors refused every floor run while the lane still reported success: the floor evaluates ZERO claims until they are fixed. Each block is hoisted to its enclosing declaration with a lead line naming the arm or variant it is about, so the rationale keeps its subject. No assertion, declaration or program text changes. Verified by execution at this head with gunbc run per file: each now reaches CallContractMismatch on its entry function, which is resolution, not parse. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Hoists three body-scope // blocks (DESIGN 4c admits only module-item grain): gunbc.floor_memory_demand qualify_floor_memory_demand, gunbc.target_binding TargetProducer, gunbc.namespace.namespace_reference_derived_residency_qualification qualify_bounded_realization. Each keeps its rationale with a lead line naming the arm or variant it is about. No assertion or program text changes; same repair as #11831.
WHY NOW: these 13 parse errors refuse the WHOLE floor (ArmSetConsumerPlanningUnavailable -- no declaration index is lent to it), so it evaluates ZERO claims. They landed while the required floor lane was fail-open (the defect #11829 repairs), so the lane reported SUCCESS anyway. Executed receipt on the #11829 branch with main merged: run 35519863429 job floor -- parse (13 error(s)), floor refused, phases_run=2 phases_failed=2, D0-ADJUDICATE adjudication REFUSED blockers=2. That adjudicator exists only on #11829; on main the identical refusal still greens.
EVIDENCE: per file at this head, gunbc run --source-root dag --source-root src/v2 --entry --function now reaches CallContractMismatch on the entry function -- the module parsed and resolved and evaluation refused on missing arguments. Before the change each refused at parse.
LAND ORDER: this PR, then #11829 (merge main for its positive control: adjudication PASSED blockers=0 with claims actually evaluated).
🤖 Generated with Claude Code