Skip to content

Fit identity_cast_emission_route claims in the new-witness eval-step budget - #13489

Merged
gunbai-bot[bot] merged 10 commits into
mainfrom
session/lively-boar-148
Oct 8, 2026
Merged

gunbai-bot[bot] merged 10 commits into
mainfrom
session/lively-boar-148

Conversation

@gunbai-bot

@gunbai-bot gunbai-bot Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Gated identity-cast claims now include supplied-atom coercion_cast_crossing rows (Identity vs NoTargetCandidate at the diagnostic head) plus inhabitance that still runs emit_closure_from_ingest_located on every required run.
  • Native host-run evidence stays off-gate; the duplicate native refusal inhabitance is gone.
  • No cost-drop member remains for this subject: gunbc#13489 required floor run 37522243686 (job 112483273531, artifact required-floor-claim-cost) billed every gated identity under the 72300 new-witness budget, which is identity_cast_route_new_witness_eval_step_cost's restoration trigger. That drop and floor_eval_step_cost_drop_identity_cast_route_rows are deleted.

Per-identity billed eval_steps (run 37522243686, required_floor_claim_cost.tsv, head b2ef02e6d7)

identity eval_steps vs 72300
an_admitted_identity_cast_emits_through_the_closure_route_holds 11 under — no cost-drop row
a_cast_with_no_witness_refuses_before_realization_holds 11 under — no cost-drop row
an_int_identity_cast_is_an_identity_crossing_holds 302 under — no cost-drop row
an_int_to_bool_cast_has_no_witness_holds 388 under — no cost-drop row

emit_host_identity_cast_native was not planned on that run (unchanged-witness / off-gate).

Test plan

  • Required floor on b2ef02e6d7: all four gated identities pass; none over 72300 (run 37522243686).
  • Required floor on this head after the cost-drop retirement.
  • Native module as a changed witness still builds fn f(x: i32) -> i32.

@gunbai-bot

gunbai-bot Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor Author

review 77130 (on 8c664ae) asked for one of two things: keep a real-path claim on required runs with an operator-approved budget, or declare this as a §4b(3) drop. The brief forbids a cost-debt row and forbids raising the new-witness budget; the emit door measured 657428 eval steps / 1835 cpu_ms on gunbc#13480 run 37486290875, which is also over the 100ms new-witness envelope, so it cannot stay enrolled as a required-run claim.

c4ee5b1 takes the second arm the review named:

  • Gated tests are renamed to what they judge (an_int_identity_cast_is_an_identity_crossing_holds, an_int_to_bool_cast_has_no_witness_holds) so the old names are not a meaning-fork.
  • emit_closure_from_ingest_located inhabitance stays on v2.test.execution.emit_host_identity_cast_native.
  • That merge-time loss is gunbc.rung_drop identity_cast_closure_inhabitance_off_the_required_gate, with population in v2.workflow.floor_eval_step_cost_drop floor_eval_step_cost_drop_identity_cast_closure_inhabitance_rows. When those native claims run as changed witnesses they still execute; a semantic red or wall-clock crossing still blocks; only the eval-step overrun is reported.

That is the same shape as python_to_typescript_compile_inhabitance_off_the_required_gate, not a silent hole.

— sent from lively-boar-148

@gunbai-bot

gunbai-bot Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor Author

review 77172 is right: identity_cast_route_new_witness_eval_step_cost is retired only by its trigger, and relocating the door off the required gate is one of the moves that trigger names as not restoration (DESIGN §4b). 85bb90b restores an_admitted_identity_cast_emits_through_the_closure_route_holds and a_cast_with_no_witness_refuses_before_realization_holds on the gate under that drop (measured at the admitted identity on runs 37486290875 / 37493248052). The supplied coercion_cast_crossing rows stay. Native remains host-run evidence, not the last merge-time execution of emit_closure_from_ingest_located.

— sent from lively-boar-148

@gunbai-bot

gunbai-bot Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor Author

review 77178: deleted a_cast_with_no_witness_is_refused_on_the_closure_route_holds from the native module (42029c7). It was the same call and assertion as the gated a_cast_with_no_witness_refuses_before_realization_holds, and it paid refused_cast_route_verdict again without a host build.

— sent from lively-boar-148

@gunbai-bot

gunbai-bot Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

review 77377: regenerated docs/design-rung-drops.md so identity_cast_route_new_witness_eval_step_cost and floor_eval_step_cost_drop_identity_cast_route_rows are gone from the committed roster.

— sent from lively-boar-148

gunbc-ci-auto-heal and others added 8 commits October 7, 2026 19:36
The required-floor rows now judge coercion_cast_crossing over supplied Int/Bool atoms so they fit the new-witness eval-step budget; emit_closure_from_ingest_located stays the inhabitance path in the native partner.

Co-authored-by: Cursor <cursoragent@cursor.com>
Review 77130 was right that supplying coercion inputs had removed merge-time inhabitance. The gated tests are renamed to the crossing they actually judge; emit_closure_from_ingest_located stays on the native partner under a §4b(3) drop, not a cost-debt row.

Co-authored-by: Cursor <cursoragent@cursor.com>
… cost roster.

The safety set is the admitted-route and refused-route inhabitance obligations.
Leftover v2.std.optional imports after the #13388 re-home blocked this PR's
floor before any native claim-cost row could be written.

Co-authored-by: Cursor <cursoragent@cursor.com>
…s its trigger.

The gated claim that row named is gone. The remaining loss is merge-time
closure-door inhabitance, already declared on identity_cast_closure_inhabitance_off_the_required_gate.

Co-authored-by: Cursor <cursoragent@cursor.com>
review 77172: relocating emit_closure_from_ingest_located off the gate does
not retire identity_cast_route_new_witness_eval_step_cost (DESIGN §4b).
Supplied coercion rows stay; the door still runs every required run.

Co-authored-by: Cursor <cursoragent@cursor.com>
…nce.

review 77178: a_cast_with_no_witness_is_refused_on_the_closure_route_holds
paid refused_cast_route_verdict again without building or running anything.

Co-authored-by: Cursor <cursoragent@cursor.com>
The floor failed an_int_to_bool_cast_has_no_witness_holds: wrap puts the
typed mismatch first and find_witness_reason_no_candidate last, so
diagnostics_fatal_reason is not discriminant(NoTargetCandidate).

Co-authored-by: Cursor <cursoragent@cursor.com>
gunbc#13489 floor run 37522243686 required-floor-claim-cost: none of the four
gated identities exceeded NewWitnessTier, which is identity_cast_route_new_witness_eval_step_cost's restoration trigger.

Co-authored-by: Cursor <cursoragent@cursor.com>
@gunbai-bot
gunbai-bot Bot force-pushed the session/lively-boar-148 branch from 4b03faf to 9f695aa Compare October 7, 2026 19:39
…p drop is gone.

review 77691: the committed projection still named
floor_eval_step_cost_drop_identity_cast_route_rows after that list was deleted.

Co-authored-by: Cursor <cursoragent@cursor.com>
@gunbai-bot

gunbai-bot Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

review 77691: regenerated and committed docs/design-rung-drops.md from gunbc.rung_drop. The standing roster no longer lists identity_cast_route_new_witness_eval_step_cost or floor_eval_step_cost_drop_identity_cast_route_rows.

The rebase skip of the earlier regen commit was the generated-artifact concurrent-divergence driver; this commit is a post-rebase derivation on the merged authorities, not a hand-resolution of the merge.

— sent from lively-boar-148

Integration-only: pick up fleet-converge r2 cache mint modes (#13542).
@gunbai-bot
gunbai-bot Bot added this pull request to the merge queue Oct 7, 2026
Merged via the queue into main with commit 3522283 Oct 8, 2026
5 checks passed
@gunbai-bot
gunbai-bot Bot deleted the session/lively-boar-148 branch October 8, 2026 12:04
@briansrls
briansrls restored the session/lively-boar-148 branch October 8, 2026 12:06
gunbai-bot Bot pushed a commit that referenced this pull request Oct 8, 2026
Main already retired the eval-step drop (#13489). Native no longer demands identity_cast_route_verdict, so ActiveFillDebt on the admitted inhabitance does not go BecameSharedByDemand.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

0 participants