Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
79 changes: 69 additions & 10 deletions docs_build/dev/PROJECT_INSTRUCTIONS.md
Original file line number Diff line number Diff line change
Expand Up @@ -91,30 +91,60 @@ User:
- Do not expand scope beyond the PR
- Do not modify `start_of_day` folders unless requested

## MAIN BRANCH EXECUTION GUARD
## EXECUTION BRANCH VALIDATION GUARD

Before any BUILD execution, Codex must verify the current git branch.

Rules:
- The required execution branch is:
- The default required execution branch is:
- `main`
- If the current branch is not `main`:
- Queue initialization must start from `main`.
- The first queue branch must be created from `main`.
- Approved Sequential Codex Queue Mode branches are allowed after queue initialization.
- Approved queue branch format:
- `team/<TEAM>/<workstream>`
- Queue branch examples:
- `team/ALPHA/game-hub`
- `team/BETA/messages`
- `team/GAMMA/admin`
- Sequential Queue Mode execution path:
- `main`
- create team queue branch
- queue PR 001
- queue PR 002
- queue PR 003
- owner review
- EOD merge
- Subsequent queued PRs may execute from the active approved team queue branch.
- Start gate passes in queue mode only when:
- current branch matches the approved queue branch
- TEAM ownership matches the queue branch
- repository is clean
- PR scope stays inside TEAM ownership
- Start gate fails when:
- branch is neither `main` nor the approved queue branch
- repository is dirty
- TEAM ownership mismatches
- cross-team work is attempted
- If the current branch is neither `main` nor an approved Sequential Codex Queue Mode branch:
- HARD STOP.
- Do not create code changes.
- Do not create implementation PRs.
- Do not create ZIP artifacts.
- Do not continue execution.
- Codex must report:
- current branch
- expected branch (`main`)
- expected branch (`main` by default, or approved `team/<TEAM>/<workstream>` in Sequential Codex Queue Mode)
- local branches found
- Codex may continue only after the user explicitly returns to `main`.
- Codex may continue only after the user explicitly returns to `main` or identifies the approved queue branch for Sequential Codex Queue Mode.

Exception:
- Explicit branch-audit or branch-comparison PRs may inspect non-main branches but must not perform implementation work on them.

Required report output:
- Current branch
- Expected branch
- Queue mode active/inactive
- Branch validation PASS/FAIL

## SLIDER VALUE VISIBILITY REQUIREMENT
Expand Down Expand Up @@ -1999,7 +2029,7 @@ Required workflow:
10. Create Pull Request automatically.
11. Resolve merge conflicts if encountered.
12. Re-run validation after conflict resolution.
13. Merge PR.
13. Merge PR only after explicit owner/EOD approval.
14. Return to main.
15. Pull latest main.
16. Continue to next approved PR.
Expand All @@ -2008,8 +2038,8 @@ Rules:
- Do not ask the user if a PR should be created.
- Do not ask the user if a branch should be pushed.
- Treat PR creation as required.
- Treat branch push as required.
- Treat merge as required after validation passes.
- Treat branch push as required for scoped PR branch publishing.
- Treat merge as required only after validation passes and explicit owner/EOD approval is provided.
- If GitHub prompts `Would you like to create a Pull Request?`, answer YES automatically.
- If merge conflicts occur:
- preserve latest main
Expand All @@ -2031,9 +2061,38 @@ Stable promotion and merge approval are owner-controlled.

Rules:
- Codex may prepare scoped changes, reports, validation evidence, ZIP artifacts, branches, and PRs.
- Codex must not merge a PR or mark a workstream stable without explicit approval from the assigned Team Alpha or Team Beta owner.
- Codex must not merge a PR or mark a workstream stable without explicit approval from the assigned Team Alpha, Team Beta, or Team Gamma owner.
- Master Control may recommend sequencing or assignment, but affected workstream owners control stable and merge approval.
- This targeted section supersedes older automatic-merge wording when approval ownership is in question.
- EOD merge and EOD push actions are owner-controlled and require explicit approval.

## SEQUENTIAL CODEX QUEUE BRANCH GOVERNANCE

Sequential Codex Queue Mode may use approved team queue branches only after initialization from `main`.

Rules:
- Queue initialization must start from `main`.
- The first queue branch must be created from `main`.
- Subsequent queued PRs may execute from the active approved team queue branch.
- Approved queue branch format is `team/<TEAM>/<workstream>`.
- Examples:
- `team/ALPHA/game-hub`
- `team/BETA/messages`
- `team/GAMMA/admin`
- Start gate PASS in queue mode requires:
- current branch matches the approved queue branch
- TEAM ownership matches the queue branch
- repository is clean
- PR scope stays inside TEAM ownership
- Start gate FAIL in queue mode when:
- branch is neither `main` nor the approved queue branch
- repository is dirty
- TEAM ownership mismatches
- cross-team work is attempted
- Cross-team work remains prohibited unless Master Control splits or assigns each PR to the correct TEAM token.
- Owner approval remains required.
- EOD merge remains required.
- Automatic merge wording must not override owner-controlled EOD approval.

## CODEX INSTRUCTION ENFORCEMENT START GATE

Expand All @@ -2051,7 +2110,7 @@ Required pre-change report:
- Any `FAIL` is a hard stop unless the PR explicitly scopes branch audit or recovery documentation without implementation.

Hard stops before changes:
- If the current branch is not `main`, HARD STOP.
- If the current branch is neither `main` nor an approved Sequential Codex Queue Mode branch, HARD STOP.
- If the repository is not clean before the PR branch is created, HARD STOP.
- If the PR name does not include a required TEAM token, HARD STOP.
- If the PR TEAM owner does not match the team ownership map in `PROJECT_MULTI_PC.txt`, HARD STOP.
Expand Down
24 changes: 24 additions & 0 deletions docs_build/dev/PROJECT_MULTI_PC.txt
Original file line number Diff line number Diff line change
Expand Up @@ -524,6 +524,30 @@ Governance, recovery, diagnostics, and instruction-hardening PRs:
- Must document TEAM ownership compliance in the PR report.
- Recovery reports must include TEAM ownership.

Sequential Codex Queue branch governance:
- Queue initialization must start from `main`.
- The first queue branch must be created from `main`.
- Subsequent queued PRs may execute from the active approved team queue branch.
- Approved queue branch format:
- `team/<TEAM>/<workstream>`
- Queue branch examples:
- `team/ALPHA/game-hub`
- `team/BETA/messages`
- `team/GAMMA/admin`
- Start gate PASS in queue mode requires:
- current branch matches the approved queue branch
- TEAM ownership matches the queue branch
- repository is clean
- PR scope stays inside TEAM ownership
- Start gate FAIL in queue mode when:
- branch is neither `main` nor the approved queue branch
- repository is dirty
- TEAM ownership mismatches
- cross-team work is attempted
- Cross-team work remains prohibited unless Master Control splits or assigns each PR to the correct TEAM token.
- Owner approval remains required.
- EOD merge remains required.

Stable and merge approval:
- Stable promotion and merge approval are controlled by the assigned Team Alpha, Team Beta, or Team Gamma owner.
- Master Control may recommend sequencing, but Codex must not merge or mark stable without explicit owner approval for the affected workstream.
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
# Instruction Compliance Checklist - PR_26171_GAMMA_008

- PR has one purpose only: PASS
- Exact PR name provided: PASS
- PR name includes TEAM token `GAMMA`: PASS
- Exact branch name provided: PASS
- Branch started from latest `main`: PASS
- `docs_build/dev/PROJECT_INSTRUCTIONS.md` read before changes: PASS
- `docs_build/dev/PROJECT_MULTI_PC.txt` read before changes: PASS
- Instruction enforcement start gate run before changes: PASS
- Repository clean before branch creation: PASS
- TEAM ownership verified as Gamma governance/instruction-hardening work: PASS
- Scope limited to requested governance docs: PASS
- MAIN BRANCH EXECUTION GUARD modified, not removed: PASS
- `main` remains valid/default execution branch: PASS
- Approved queue branches valid only in Sequential Codex Queue Mode: PASS
- TEAM ownership boundaries remain enforced: PASS
- Owner-controlled EOD merge approval remains enforced: PASS
- Required reports created: PASS
- Manual validation notes created: PASS
- ZIP required and pending final packaging: PASS
- Playwright skipped with reason documented: PASS
- Samples skipped with reason documented: PASS
- PR merge withheld until explicit EOD approval: PASS
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
# Manual Validation Notes - PR_26171_GAMMA_008

## Scope Review

Reviewed the governance docs after edit:
- `docs_build/dev/PROJECT_INSTRUCTIONS.md`
- `docs_build/dev/PROJECT_MULTI_PC.txt`

Confirmed:
- The branch guard was modified and renamed, not removed.
- `main` remains the default required execution branch.
- Queue initialization must start from `main`.
- The first queue branch must be created from `main`.
- Subsequent queued PRs may execute from the active approved team queue branch.
- Approved queue branch format is `team/<TEAM>/<workstream>`.
- Queue examples include:
- `team/ALPHA/game-hub`
- `team/BETA/messages`
- `team/GAMMA/admin`
- Queue-mode start gate PASS requires:
- current branch matches the approved queue branch
- TEAM ownership matches the queue branch
- repository is clean
- PR scope stays inside TEAM ownership
- Queue-mode start gate FAIL includes:
- branch is neither `main` nor the approved queue branch
- repository is dirty
- TEAM ownership mismatches
- cross-team work is attempted
- Owner approval and EOD merge approval remain required.
- Automatic merge wording does not override owner approval.

## Validation Notes

Static validation was sufficient because this PR updates governance documentation only.

Skipped lanes:
- Playwright was not run because no UI route, component, or browser behavior changed.
- Samples smoke was not run because samples are outside the requested docs/static validation scope.
Original file line number Diff line number Diff line change
@@ -0,0 +1,63 @@
# PR_26171_GAMMA_008-sequential-queue-branch-governance-alignment

## Summary

This PR aligns Sequential Codex Queue Mode with the branch execution guard.

Scope completed:
- Modified and renamed `MAIN BRANCH EXECUTION GUARD` to `EXECUTION BRANCH VALIDATION GUARD`.
- Preserved `main` as the default required execution branch.
- Added approved Sequential Codex Queue Mode branches as an allowed execution branch type after queue initialization.
- Added approved queue branch format `team/<TEAM>/<workstream>` with Alpha, Beta, and Gamma examples.
- Added queue-mode start gate PASS and FAIL criteria.
- Preserved TEAM ownership boundaries and cross-team prohibition.
- Preserved owner-controlled EOD merge approval.

## Start Gate

Instruction compliance start gate: PASS

- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`: PASS
- Read `docs_build/dev/PROJECT_MULTI_PC.txt`: PASS
- Stopped current branch work before starting this PR: PASS
- Checked out `main`: PASS
- Pulled latest `main`: PASS
- Verified current branch was `main` before creating PR branch: PASS
- Verified repository was clean before creating PR branch: PASS
- Created PR branch from `main`: PASS
- PR name includes TEAM token `GAMMA`: PASS
- TEAM ownership verified as Gamma governance/instruction-hardening scope: PASS
- Base `main` commit: `e8845dae6`

## Git Workflow

- PR branch: `pr/26171-GAMMA-008-sequential-queue-branch-governance-alignment`
- Branch created from: `main`
- Branch push: PASS, pushed to `origin/pr/26171-GAMMA-008-sequential-queue-branch-governance-alignment`
- Pull request: PASS, draft PR https://github.com/ToolboxAid/HTML-JavaScript-Gaming/pull/33
- Merge status: not merged; EOD merge requires explicit owner approval
- ZIP artifact path: `tmp/PR_26171_GAMMA_008-sequential-queue-branch-governance-alignment_delta.zip`

## Validation

Requested validation scope was docs/static only.

Executed:
- `git diff --check`: PASS
- Targeted text checks verifying the branch guard was modified and not removed.
- Targeted text checks verifying `main` remains the default required execution branch.
- Targeted text checks verifying approved queue branches are valid only in Sequential Codex Queue Mode.
- Targeted text checks verifying TEAM ownership and cross-team restrictions remain enforced.
- Targeted text checks verifying owner-controlled EOD merge approval remains enforced.

Skipped:
- Playwright: skipped by request; this PR modifies governance docs only.
- Samples smoke: skipped by request; this PR modifies governance docs only.

## Required Reports

- `docs_build/dev/reports/codex_review.diff`
- `docs_build/dev/reports/codex_changed_files.txt`
- `docs_build/dev/reports/PR_26171_GAMMA_008-sequential-queue-branch-governance-alignment.md`
- `docs_build/dev/reports/PR_26171_GAMMA_008-sequential-queue-branch-governance-alignment-manual-validation-notes.md`
- `docs_build/dev/reports/PR_26171_GAMMA_008-sequential-queue-branch-governance-alignment-instruction-compliance-checklist.md`
56 changes: 8 additions & 48 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line number Diff line number Diff line change
@@ -1,48 +1,8 @@
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
docs_build/dev/reports/coverage_changed_js_guardrail.txt
docs_build/dev/reports/dependency_gating_report.md
docs_build/dev/reports/dependency_hydration_reuse_report.md
docs_build/dev/reports/execution_graph_reuse_report.md
docs_build/dev/reports/failure_fingerprint_report.md
docs_build/dev/reports/filesystem_scan_reduction_report.md
docs_build/dev/reports/incremental_validation_report.md
docs_build/dev/reports/lane_compilation_report.md
docs_build/dev/reports/lane_deduplication_report.md
docs_build/dev/reports/lane_input_validation_report.md
docs_build/dev/reports/lane_manifests/workspace-contract.json
docs_build/dev/reports/lane_runtime_optimization_report.md
docs_build/dev/reports/lane_snapshot_report.md
docs_build/dev/reports/lane_snapshots/workspace-contract.json
docs_build/dev/reports/lane_warm_start_report.md
docs_build/dev/reports/lane_warm_starts/workspace-contract.json
docs_build/dev/reports/monolith_trigger_removal_report.md
docs_build/dev/reports/persistent_lane_manifest_report.md
docs_build/dev/reports/playwright_discovery_ownership_report.md
docs_build/dev/reports/playwright_discovery_scope_report.md
docs_build/dev/reports/playwright_structure_audit.md
docs_build/dev/reports/playwright_v8_coverage_report.txt
docs_build/dev/reports/PR_26171_BETA_073-message-tts-table-ui-correction-instruction-compliance-checklist.md
docs_build/dev/reports/PR_26171_BETA_073-message-tts-table-ui-correction-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_073-message-tts-table-ui-correction-message-parent-child-table-checklist.md
docs_build/dev/reports/PR_26171_BETA_073-message-tts-table-ui-correction-tts-summary-layout-checklist.md
docs_build/dev/reports/PR_26171_BETA_073-message-tts-table-ui-correction-tts-tools-registration-checklist.md
docs_build/dev/reports/PR_26171_BETA_073-message-tts-table-ui-correction-validation-report.md
docs_build/dev/reports/PR_26171_BETA_073-message-tts-table-ui-correction.md
docs_build/dev/reports/retry_suppression_report.md
docs_build/dev/reports/slow_path_pruning_report.md
docs_build/dev/reports/static_validation_report.md
docs_build/dev/reports/targeted_file_manifest_report.md
docs_build/dev/reports/test_cleanup_performance_report.md
docs_build/dev/reports/test_cleanup_routing_report.md
docs_build/dev/reports/testing_lane_execution_report.md
docs_build/dev/reports/validation_cache_report.md
docs_build/dev/reports/zero_browser_preflight_report.md
src/shared/toolbox/tool-metadata-inventory.js
tests/playwright/tools/MessagesTool.spec.mjs
tests/playwright/tools/RootToolsFutureState.spec.mjs
tests/playwright/tools/TextToSpeechFunctional.spec.mjs
tests/playwright/tools/ToolboxRoutePages.spec.mjs
toolbox/messages/index.html
toolbox/messages/messages.js
toolbox/text-to-speech/index.html
M docs_build/dev/reports/PR_26171_GAMMA_008-sequential-queue-branch-governance-alignment.md

docs_build/dev/PROJECT_INSTRUCTIONS.md | 79 +++++++++++++++++++---
docs_build/dev/PROJECT_MULTI_PC.txt | 24 +++++++
...e-alignment-instruction-compliance-checklist.md | 24 +++++++
...governance-alignment-manual-validation-notes.md | 39 +++++++++++
...sequential-queue-branch-governance-alignment.md | 63 +++++++++++++++++
5 files changed, 219 insertions(+), 10 deletions(-)
Loading
Loading