Skip to content

Integrate exact estimator results into proposal authority - #387

Merged
adrianwebb merged 1 commit into
stagingfrom
codex/estimator-proposal-integration-386
Sep 20, 2026
Merged

adrianwebb merged 1 commit into
stagingfrom
codex/estimator-proposal-integration-386

Conversation

@adrianwebb

@adrianwebb adrianwebb commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

Outcome

Bind modern graph estimator AssignmentResult TreeDX references to the proposal-owned execution plan before assignment completion. Merge one actor estimate or all six Reviewer review estimates into the current exact proposal without a second estimate store or legacy artifact manifest. The active SDK golden retry is underway; this PR is not yet acceptance evidence.

Work authority

  • Work item / Issue: Closes Integrate exact estimator results into proposal authority before graph completion #386; Platform #520
  • Proposal / decision: golden-sdk-decision-governed-workday-intent-retry-1 v1; decision pending
  • Assignment / checkpoint: stopped diagnostic workday workday-6a5bab5f-c5f9-4803-b7cf-9e078a6e614f; new simulation workday workday-7ec4fac3-8df3-4715-a44a-aa2b98062569
  • Actor: Codex assistant
  • Human authority: Adrian Webb (Platform #520 delivery request)
  • Agent / capacity provider: local dev2; no golden acceptance claimed
  • Exact base ref: staging ff170ff
  • Exact head ref: codex/estimator-proposal-integration-386 3415779

Contributor mode (select one):

  • Human-authored
  • Agent-assisted under human authority
  • Agent-authored under human authority

Plan

Use the validated general AssignmentResult and exact proposal source/workspace refs. Verify only the assigned estimate fields changed, preserve disjoint previous estimates, author one TreeDX proposal revision, bind it to governance, and only then complete the graph assignment. Reject changed authority, unrelated edits, missing estimates, and conflicts. Exercise in local development mode, then run the unchanged SDK golden campaign. No tag or RC.

Changes and commits

  • 3415779 - exact actor/Reviewer estimate merge and lifecycle gate, with focused regression tests.

Verification

  • Focused Vitest: four tests passed.

  • API build: passed.

  • Repository file-length and architecture policy: passed.

  • Local trsd dev rebuild api.service: passed; API targets report ready.

  • Touched-file TypeScript filter: no new errors; repository-wide tsc has unrelated existing errors.

  • Refrozen Stage 0 manifest sha256:2efa9196764029edfe779c1ce7697ec03fc0b9ff348d465234157d20aaf64050.

  • Live SDK golden: running, not yet accepted.

  • Staging Actions: pending.

  • I ran the narrowest relevant package verification and documented any checks that could not be run.

  • I have not claimed golden acceptance from partial evidence.

  • No routine progress comment, release tag, RC, or package install.

Risk and rollback

Risk: TreeDX authoring succeeds but governance version binding fails, leaving an explicit unbound revision; API #385 tracks atomicity. If the live estimator projection fails or the golden graph diverges, stop the workday through trsd, preserve exact TreeDX commits as diagnostic evidence, and revert commit 3415779 on this branch. No database migration, package install, tag, or RC is included.

Completion summary

The bounded implementation and focused tests are complete. Local development rebuild is ready. The refrozen SDK golden run is active; no integrated acceptance or staging merge is claimed. Next: verify real owner and Reviewer estimates, disposition, accepted decision, and Actor/Reviewer execution before marking this PR ready.

AGPL committer authorization

The provider-authenticated PR author must be present in the base-owned approved-committers policy. No per-PR grant is requested.

Submission checklist

  • The change is bounded to the stated work item and target repository.
  • Exact base and head refs are recorded and the branch is ready for review.
  • Verification and compatibility evidence are recorded above.
  • No plaintext secrets, credentials, machine state, or unrelated residue are included.
  • Plan, status, commits, and completion summary form a complete durable record.
  • Rollback or recovery steps are documented and executable.

@adrianwebb
adrianwebb marked this pull request as draft September 20, 2026 16:05
@adrianwebb
adrianwebb marked this pull request as ready for review September 20, 2026 17:22
@adrianwebb
adrianwebb merged commit 201ffac into staging Sep 20, 2026
5 of 8 checks passed
@adrianwebb
adrianwebb deleted the codex/estimator-proposal-integration-386 branch September 20, 2026 17:22
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