Skip to content

refactor(quota): keep execution preparation presentation with its owner - #8042

Merged
huangruiteng merged 1 commit into
codex/auto-lease-preparation-20261103from
codex/execution-preparation-presentation-20261104
Nov 4, 2026
Merged

huangruiteng merged 1 commit into
codex/auto-lease-preparation-20261103from
codex/execution-preparation-presentation-20261104

Conversation

@huangruiteng

Copy link
Copy Markdown
Collaborator

Goal And Delivered Outcome

Integrating #7987 with current main (b4216d8c7) makes interaction_contract.py exceed its existing module metric ceiling. Move the execution-preparation channel projection into the existing quota/execution_preparation.py owner and retain its invocation at the same point. The integrated candidate passes the unchanged maintainability gate after this extraction.

This is a stacked follow-up targeting codex/auto-lease-preparation-20261103, based on c10b100c049e8298622e5bc96e6756631dd3d8b5. It does not duplicate the selection-admission fix in #7987.

Author Declaration

  • Written by: model_agent, OpenAI Codex.
  • Specification: the concrete maintenance defect above is the basis; repository scope-fit and real-path validation guidance at b4216d8c7 applies.
Criterion Disposition Implementation Validation
Keep preparation presentation with its existing owner implemented quota/execution_preparation.py::attach_interaction_execution_preparation unchanged maintainability gate
Preserve binding, claim, permission and settlement behavior implemented same function body and invocation order File/SQLite CLI preparation and host tests

Scope And Continuation

Complete within this extraction. No vocabulary, storage schema, permission, command or UI change. The parent feature remains in #7987. No new tests were added: the existing real CLI cases cover the moved function, including explicit selection, missing/expired/foreign leases and same-Turn recovery.

Validation

  • Tested revision: 73d10d1c5.
  • Run state: finished.
  • Input classes: synthetic, public_fixture.
Check kind Result Evidence / limitation
integration passed 24 tests in test_quota_execution_preparation.py and test_codex_execution_preparation.py, through real File/SQLite command paths on the PR head
static passed Ruff on both changed files and diff hygiene
static passed Exact-base premerge: 6 direct checks and 18 selected checks; zero failures, warnings or manual holds

The same extraction was also tested after integrating the parent feature with main: 24 affected tests passed, and the previously failing maintainability gate passed. This is local verification, not CI, native desktop, PostgreSQL or model-performance qualification.

Frontend / Visual Evidence

UI impact: none; this preserves the existing machine-facing projection. Screenshots are not applicable.

Type of Change

  • Refactoring (no functional changes)

LoopX Area

  • Control plane (goals, todos, quota, scheduler, registry, runtime)

Shared-authority RFC fixture impact

N/A: no authority contract, provider routing, storage or fixture dimension changes.

Boundary Checklist

  • Public-safe source and synthetic test evidence only; no private state, traces, credentials or local paths.
  • Scoped follow-up to Fix automatic task lease preparation before execution #7987; no benchmark work or paired RFC changes.
  • UI impact is none.
  • Every commit includes a DCO Signed-off-by trailer.

Signed-off-by: huangruiteng <huangrt01@163.com>
@huangruiteng
huangruiteng merged commit 388fdcc into codex/auto-lease-preparation-20261103 Nov 4, 2026
16 of 32 checks passed
@huangruiteng
huangruiteng deleted the codex/execution-preparation-presentation-20261104 branch November 4, 2026 15:05
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