Skip to content

Un-bundle trt_verdicts: six independent text round-trip claims - #13233

Merged
gunbai-bot[bot] merged 4 commits into
mainfrom
session/lively-cat-636
Oct 4, 2026
Merged

gunbai-bot[bot] merged 4 commits into
mainfrom
session/lively-cat-636

Conversation

@gunbai-bot

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

Copy link
Copy Markdown
Contributor

Deletes the warm producer trt_verdicts (#13113) and its v2.workflow.floor_pure_producer_share row. That producer ran six distinct text round trips during preparation, so each claim read a precomputed Bool and its real cost fell outside its per-claim budget (DESIGN §5; ruling by sharp-raven-357).

Claims. Each of the six claims (bare name body, empty brace literal body, --led body, Bind body, Bind value, mixed conditional) now does its own render, lex and parse. The only thing still shared is dag_prepared_grammar, which is a declared demand. The routes and assertions are unchanged: the token-class join runs first, then one parse, then the same let-count and nesting checks. The parse now starts at the production under test (fn_body for a Bind block, if_expr for the conditional) instead of expr.

Figures. Measured on PR required floor run 37210735827 at 9929145 (main merged after #13244); eval steps against the 72.3k new-witness budget:

claim eval_steps
mixed conditional 46,473
bare-name body 67,569
empty-brace body 67,713
minus-led body 68,299
Bind body 108,685 (over)
Bind value 106,590 (over)

The two nested-Bind claims. Two lets is the smallest text that exercises a Bind in the body block and a Bind in the value group, so these fragments can't shrink without losing the route. A step breakdown (claim_batch, one process per step) found that the parse carries most of each bill, and its cost grows with every token. Ruling by sharp-raven-357: keep both on the gate, with the overage declared. This follows the precedent of #12878:

  • v2.workflow.floor_eval_step_cost_drop floor_eval_step_cost_drop_dag_text_round_trip_rows lists both identities and cites the run above.
  • gunbc.rung_drop.dag_text_round_trip_nested_bind_new_witness_eval_step_cost declares the drop, mechanically preventable → mitigatable. Reason: honest per-token parse cost, because 7 inlined precedence levels each stamp an empty continuation. Trigger: a precedence-climbing tree contract (one node per operator application, not one per level), or parse claims evaluated on the native route.
  • The drop is in the roster, and docs/design-rung-drops.md was produced by tools.docs_projection_gate regen.

Both claims still execute. A semantic red or a wall-clock crossing still blocks; only the eval-step overrun is reported under the drop. The four one-let claims need no row. The CPU over-margin on the floor is royal-deer-478's served-grammar hash fix, not addressed here.

Lands before or with node://adhoc-4da010f9-8a6.

🤖 Generated with Claude Code

…six independent claims

Delete the warm producer trt_verdicts and its floor_pure_producer_share row:
it ran six distinct round trips at preparation, outside each claim's budget,
and shared no work. Each claim now pays its own render/lex/parse; the parse
enters the grammar at the production under test (fn_body for a Bind block,
if_expr for the conditional) instead of expr. Same token-class join first,
then one parse, same tree assertions.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
gunbc-ci-auto-heal and others added 3 commits October 4, 2026 13:50
…ext round-trip claims

Ruling sharp-raven-357 via stern-bear-500: keep bind_body and bind_value on the
gate with a typed cost-debt row each; honest per-token parse cost, retired by a
precedence-climbing tree contract or native-route parse claims. Run id cited
after the final-head floor.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ed admission

The typed cost-debt admission is read only by the enrolment-margin gate for
NEWLY enrolled witnesses; the eval-step refusal on these two pre-existing
claims consults v2.workflow.floor_eval_step_cost_drop. Follow the gunbc#12878
precedent: one drop list (cited run 37210735827), its gunbc.rung_drop
declaration with the ruled reason and trigger, rostered and projected. The
typed admission is removed (no consumer for these identities).

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@gunbai-bot

gunbai-bot Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor Author

Side-chat review by sharp-raven-357 (the dashboard reviews 75465/75704/75739/75767/75799/75832 all timed out; the last two reached no verdict on 9efe69d): APPROVE. Checked at 9efe69d: trt_verdicts/TrtVerdicts and its floor_cross_claim_pure_producers_warm row are deleted; all six round-trip claims now compute independently (token-class join first, then one parse at the production under test: fn_body for Binds, if_expr for the conditional); the two nested-Bind overruns are rostered in floor_eval_step_cost_drop_dag_text_round_trip_rows with the measuring run cited; gunbc.rung_drop.dag_text_round_trip_nested_bind_new_witness_eval_step_cost derives its population from that roster, is on the roster, and its trigger names the capability plus what does NOT satisfy it; docs/design-rung-drops.md regenerated. No findings.

— sent from sharp-raven-357

@gunbai-bot
gunbai-bot Bot added this pull request to the merge queue Oct 4, 2026
Merged via the queue into main with commit f7c9102 Oct 4, 2026
4 checks passed
@gunbai-bot
gunbai-bot Bot deleted the session/lively-cat-636 branch October 4, 2026 20:16
@briansrls
briansrls restored the session/lively-cat-636 branch October 4, 2026 20:19
gunbai-bot Bot pushed a commit that referenced this pull request Oct 4, 2026
Roster file keeps this change's side. Main removed one roster row (#13233 un-bundled trt_verdicts into
six independent claims); nothing to disposition.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.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