Skip to content

Fix simulation proposal estimates after workday discussion - #392

Merged
adrianwebb merged 1 commit into
stagingfrom
codex/workday-simulation-proposal-head-391
Sep 20, 2026
Merged

adrianwebb merged 1 commit into
stagingfrom
codex/workday-simulation-proposal-head-391

Conversation

@adrianwebb

Copy link
Copy Markdown
Contributor

Outcome

Simulation proposal estimate publication no longer collides with the workday discussion branch. Proposal revisions use a deterministic proposal-specific local branch rooted in exact proposal provenance; no upstream publication occurs.

Work authority

  • Work item / Issue: Closes Use current simulation workday branch for proposal estimate publication #391; Platform #520 golden SDK acceptance
  • Proposal / decision: golden-sdk-decision-governed-workday-intent-retry-5 (draft; no decision yet)
  • Assignment / checkpoint: assignment_3ta4IUALf6W4_QGk1mW4EuJTiUD8mVmU, workday workday-0a888dd7-aaec-4203-af58-7319d9bf9d06 (failed, stopped, not acceptance)
  • Actor: Codex implementation agent
  • Human authority: Adrian Webb
  • Agent / capacity provider: SDK Architect estimator; local dev2 provider
  • Exact base ref: staging at be1e9f46613545fa7287a817435c6ca5d7b9bfbc
  • Exact head ref: codex/workday-simulation-proposal-head-391 at fd54549353228f78a7fa81063c6a821fd1732f09

Contributor mode (select one):

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

Plan

Trace the actual TreeDX branch/base mismatch; isolate proposal version lineage from the workday discussion branch; test the conflicting branch and sequential-estimate case; verify and retry the unchanged golden proposal from a new freeze.

Changes and commits

  • fd54549353228f78a7fa81063c6a821fd1732f09 gives each proposal a deterministic workday-local branch rooted in its exact current provenance. This preserves strict TreeDX branch-start conflict detection and avoids adding a compatibility path.
  • Existing discussion branch is untouched. No release or tag.

Verification

  • I ran the narrowest relevant package verification and documented any checks that could not be run.
  • npx vitest run --config ./vitest.control-plane.config.ts tests/unit/control-plane/governance tests/unit/control-plane/capacity/planning/workday-content-source.test.ts: 13 files, 83 tests passed.
  • npm run build: passed.
  • npm run check:file-lengths: passed.
  • git diff --check: passed.
  • Fresh live estimator read-back: pending new golden freeze, tracked in Platform #520; not represented as PR CI success.

Risk and rollback

The proposal branch name changes for new simulation authoring only. Exact proposal source commit remains the workspace base and TreeDX rejects stale branch heads. Revert this commit before a new attempt if it fails; retry-5 was stopped, terminalized, and cannot count as acceptance. No database migration or upstream content mutation.

Completion summary

Code and focused tests ready for Actions; live golden acceptance remains pending after merge and fresh freeze. No claim that SDK/API golden passed.

AGPL committer authorization

The base-owned workflow checks the provider-authenticated pull-request author against .github/approved-committers.json. There is no per-PR grant checkbox. If the check reports that the GitHub username is unapproved, follow the linked one-time approval form and re-run the check after the reviewed allowlist change reaches this target branch.

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 merged commit 08bdac8 into staging Sep 20, 2026
3 checks passed
@adrianwebb
adrianwebb deleted the codex/workday-simulation-proposal-head-391 branch September 20, 2026 19:34
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