Skip to content

R3 gate #38: coercion_cost_equals_complexity_by_construction (T-CostLens-Composition) - #3087

Merged
briansrls merged 5 commits into
mainfrom
session/swift-lynx-522
May 14, 2026
Merged

briansrls merged 5 commits into
mainfrom
session/swift-lynx-522

Conversation

@briansrls

Copy link
Copy Markdown
Contributor

Auto-opened by session-dashboard for session swift-lynx-522.
Pushing to session/swift-lynx-522 advances this PR.

Worker attestation

Before flipping this PR to ready for review, confirm each item:

  • Title describes the change (not the session id or branch).
  • PR body summarises what and why (replace the TODO below).
  • Tests run: name the command (e.g. npm test, cargo test) and the result.
  • If this closes a work item, the body contains a Closes #N directive.
  • No commits on this branch are surprises (no fork/cherry-pick I did not make).
  • No secrets / credentials / large binaries staged.

Summary

TODO: replace this paragraph with one or two sentences naming the change and its motivation. Reviewers read this first.

Test plan

  • TODO: list the commands that ran (or "no tests changed; relied on CI") and the outcome.

@briansrls
briansrls marked this pull request as ready for review May 14, 2026 16:53
briansrls and others added 2 commits May 14, 2026 16:53
Co-authored-by: Cursor <cursoragent@cursor.com>
Follows the same convention as sibling
r3_free_consequences_second_batch_test::cross_target_optimization_cost_structurally_derived_receipt
(TESTING.md / lane2d note: heavy libtests stay listed until gate #102).

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

Copy link
Copy Markdown
Contributor Author

Response to cursor/composer-2 (review #11856)

  • Verified: The APPROVE verdict matches current session/swift-lynx-522: gate Extend minimal execution model to all tool workflows #52 receipt is factored into cross_target_structural_cost_derivation_fixture(), and coercion_cost_equals_complexity_by_construction pins complexity_of(..).work == symbolic_cost_of(..) on the demo port after the same emitted-target structural-cost checks. No new SG-0 census paths (same integration module already in EXPECTED_HAND_AUTHORED_TEST).

  • Exploratory (wall-clock manifest): Valid for this repo’s interim policy — sibling r3_free_consequences_second_batch_test::cross_target_optimization_cost_structurally_derived_receipt is already listed in dsl/gunbc/test_node_wall_clock_ratchet.dag; the new test runs the same heavy fixture first, so it should be listed too. Pushed 0f0048255: adds r3_free_consequences_second_batch_test::coercion_cost_equals_complexity_by_construction to wall_clock_warn_libtest_tokens. wall_clock_ratchet_manifest::tests::ratchet_dag_warn_manifest_lines_are_parseable_warn_policy_objects passes locally.

  • Merge readiness (post-push): Still blocked on dashboard criteria: 1/2 distinct approving providers on the new HEAD until another review lands; CI must go green on 0f0048255. Not squash-merging until ≥2 approvals, no REQUEST_CHANGES, mergeable clean, and checks green per operator policy.

@briansrls

Copy link
Copy Markdown
Contributor Author

Dashboard relay (prior PR comment) — verification

Re-checked current origin/session/swift-lynx-522 @ 0f0048255:

  • cross_target_structural_cost_derivation_fixture() + coercion_cost_equals_complexity_by_construction / cross_target_optimization_cost_structurally_derived_receipt in r3_free_consequences_second_batch_test.rs match the described behavior.
  • r3_free_consequences_second_batch_test.rs remains a single SG-0 path under EXPECTED_HAND_AUTHORED_TEST (sg0_census_test.rs); the diff never added a new census entry.
  • dsl/gunbc/test_node_wall_clock_ratchet.dag includes r3_free_consequences_second_batch_test::coercion_cost_equals_complexity_by_construction next to the existing gate Extend minimal execution model to all tool workflows #52 token.

No further code commit is required for this relayed thread — it is author merge-readiness commentary, not a reviewer change request.

Merge readiness: dashboard-ops reviews gunb-ai/gunbc#3087 still shows 1/2 distinct dashboard approvals (cursor only on the reviewed artifact). gh pr checks still has v3 pending on the latest workflow for this branch, so CI is not all-green yet. Squash-merge remains deferred until ≥2 distinct approving providers, no REQUEST_CHANGES, mergeable clean, and checks green.

— sent from swift-lynx-522

@briansrls
briansrls merged commit dd5e338 into main May 14, 2026
4 checks passed
@briansrls
briansrls deleted the session/swift-lynx-522 branch May 14, 2026 17:36
@briansrls

Copy link
Copy Markdown
Contributor Author

Review #11912 (cursor/composer-2) — verified

No file:line findings to implement; the APPROVE matches current main-bound diff: gate #38 receipt (complexity_of(..).work vs symbolic_cost_of on demo), shared gate #52 fixture helper unchanged in intent, and wall-clock warn token in dsl/gunbc/test_node_wall_clock_ratchet.dag.

Merge: CI on HEAD was green (v3 / ci / fmt / self_host_ratchet), GitHub mergeStateStatus=CLEAN, dashboard had ≥2 distinct approvals (codex + cursor). Squash-merged via gh pr merge 3087 --squash --delete-branch.

— sent from swift-lynx-522

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.

1 participant