Skip to content

Commit df1d175

Browse files
committed
Add Sequential Codex Queue Mode governance - PR_26171_GAMMA_004-sequential-gamma-queue-governance
1 parent 78b642f commit df1d175

7 files changed

Lines changed: 432 additions & 1556 deletions

docs_build/dev/PROJECT_INSTRUCTIONS.md

Lines changed: 58 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1999,7 +1999,7 @@ Required workflow:
19991999
10. Create Pull Request automatically.
20002000
11. Resolve merge conflicts if encountered.
20012001
12. Re-run validation after conflict resolution.
2002-
13. Merge PR.
2002+
13. Merge PR only after explicit owner/EOD approval.
20032003
14. Return to main.
20042004
15. Pull latest main.
20052005
16. Continue to next approved PR.
@@ -2008,8 +2008,8 @@ Rules:
20082008
- Do not ask the user if a PR should be created.
20092009
- Do not ask the user if a branch should be pushed.
20102010
- Treat PR creation as required.
2011-
- Treat branch push as required.
2012-
- Treat merge as required after validation passes.
2011+
- Treat branch push as required for scoped PR branch publishing.
2012+
- Treat merge as required only after validation passes and explicit owner/EOD approval is provided.
20132013
- If GitHub prompts `Would you like to create a Pull Request?`, answer YES automatically.
20142014
- If merge conflicts occur:
20152015
- preserve latest main
@@ -2031,9 +2031,63 @@ Stable promotion and merge approval are owner-controlled.
20312031

20322032
Rules:
20332033
- Codex may prepare scoped changes, reports, validation evidence, ZIP artifacts, branches, and PRs.
2034-
- Codex must not merge a PR or mark a workstream stable without explicit approval from the assigned Team Alpha or Team Beta owner.
2034+
- 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.
20352035
- Master Control may recommend sequencing or assignment, but affected workstream owners control stable and merge approval.
20362036
- This targeted section supersedes older automatic-merge wording when approval ownership is in question.
2037+
- EOD merge and EOD push actions are owner-controlled and require explicit approval.
2038+
2039+
## SEQUENTIAL CODEX QUEUE MODE
2040+
2041+
Team Gamma may use Sequential Codex Queue Mode for governance, diagnostics, admin, recovery, and instruction-hardening work.
2042+
2043+
Sequential Codex Queue Mode means Codex receives one queue command containing multiple ordered PR scopes.
2044+
2045+
Each queued PR must still have:
2046+
- exact PR name
2047+
- TEAM token
2048+
- branch name
2049+
- exact scope
2050+
- validation scope
2051+
- required reports
2052+
- manual validation notes
2053+
- ZIP requirement
2054+
2055+
Rules:
2056+
- A queue command is not itself a PR unless explicitly named as one.
2057+
- Missing exact scope for any queued PR is a hard stop for that PR only.
2058+
- Codex may continue to the next queued PR only if the next PR has exact scope and does not depend on the stopped PR.
2059+
- Codex must not create empty ZIPs or package partial PRs.
2060+
- EOD merge/push is owner-controlled and requires explicit approval.
2061+
- This section supersedes older wording that implies every PR must be independently prompted when Master Control provides a fully scoped sequential queue.
2062+
2063+
Exact Gamma queue scope examples:
2064+
2065+
1. `PR_26171_GAMMA_001-sqlite-deprecation-audit`
2066+
- TEAM token: `GAMMA`
2067+
- Branch: `pr/26171-GAMMA-001-sqlite-deprecation-audit`
2068+
- Exact scope: audit active repository documentation and runtime references for SQLite deprecation status; document existing SQLite references as legacy technical debt only; do not add SQLite services, DDL, seed data, or runtime persistence.
2069+
- Exact target paths: `docs_build/dev/PROJECT_INSTRUCTIONS.md`, `docs_build/dev/reports/PR_26171_GAMMA_001-sqlite-deprecation-audit.md`, `docs_build/dev/reports/PR_26171_GAMMA_001-sqlite-deprecation-audit-manual-validation-notes.md`, `docs_build/dev/reports/PR_26171_GAMMA_001-sqlite-deprecation-audit-instruction-compliance-checklist.md`, `docs_build/dev/reports/codex_review.diff`, `docs_build/dev/reports/codex_changed_files.txt`.
2070+
- Validation scope: docs/static validation only, including `git diff --check` and targeted SQLite reference review; do not run Playwright; do not run samples.
2071+
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`.
2072+
- ZIP requirement: `tmp/PR_26171_GAMMA_001-sqlite-deprecation-audit_delta.zip`.
2073+
2074+
2. `PR_26171_GAMMA_002-admin-system-health-foundation`
2075+
- TEAM token: `GAMMA`
2076+
- Branch: `pr/26171-GAMMA-002-admin-system-health-foundation`
2077+
- Exact scope: document the Admin system-health foundation contract for diagnostics ownership, visible failure reporting, and Postgres-first health checks; do not implement admin UI, runtime APIs, auth behavior, or database migrations.
2078+
- Exact target paths: `docs_build/dev/PROJECT_INSTRUCTIONS.md`, `docs_build/dev/reports/PR_26171_GAMMA_002-admin-system-health-foundation.md`, `docs_build/dev/reports/PR_26171_GAMMA_002-admin-system-health-foundation-manual-validation-notes.md`, `docs_build/dev/reports/PR_26171_GAMMA_002-admin-system-health-foundation-instruction-compliance-checklist.md`, `docs_build/dev/reports/codex_review.diff`, `docs_build/dev/reports/codex_changed_files.txt`.
2079+
- Validation scope: docs/static validation only, including `git diff --check` and targeted ownership wording review; do not run Playwright; do not run samples.
2080+
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`.
2081+
- ZIP requirement: `tmp/PR_26171_GAMMA_002-admin-system-health-foundation_delta.zip`.
2082+
2083+
3. `PR_26171_GAMMA_003-admin-postgres-r2-diagnostics`
2084+
- TEAM token: `GAMMA`
2085+
- Branch: `pr/26171-GAMMA-003-admin-postgres-r2-diagnostics`
2086+
- Exact scope: document Admin diagnostics expectations for Postgres and R2 readiness, credential-presence checks, actionable failure messages, and skipped-lane reporting; do not implement R2 storage, database connections, migrations, or admin runtime behavior.
2087+
- Exact target paths: `docs_build/dev/PROJECT_INSTRUCTIONS.md`, `docs_build/dev/reports/PR_26171_GAMMA_003-admin-postgres-r2-diagnostics.md`, `docs_build/dev/reports/PR_26171_GAMMA_003-admin-postgres-r2-diagnostics-manual-validation-notes.md`, `docs_build/dev/reports/PR_26171_GAMMA_003-admin-postgres-r2-diagnostics-instruction-compliance-checklist.md`, `docs_build/dev/reports/codex_review.diff`, `docs_build/dev/reports/codex_changed_files.txt`.
2088+
- Validation scope: docs/static validation only, including `git diff --check` and targeted Postgres/R2 diagnostics wording review; do not run Playwright; do not run samples.
2089+
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`.
2090+
- ZIP requirement: `tmp/PR_26171_GAMMA_003-admin-postgres-r2-diagnostics_delta.zip`.
20372091

20382092
## CODEX INSTRUCTION ENFORCEMENT START GATE
20392093

docs_build/dev/PROJECT_MULTI_PC.txt

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -524,6 +524,34 @@ Governance, recovery, diagnostics, and instruction-hardening PRs:
524524
- Must document TEAM ownership compliance in the PR report.
525525
- Recovery reports must include TEAM ownership.
526526

527+
Team Gamma Sequential Codex Queue Mode:
528+
- Team Gamma may use Sequential Codex Queue Mode for governance, diagnostics, admin, recovery, and instruction-hardening work.
529+
- Sequential Codex Queue Mode means Codex receives one queue command containing multiple ordered PR scopes.
530+
- Each queued PR must still have exact PR name, TEAM token, branch name, exact scope, validation scope, required reports, manual validation notes, and ZIP requirement.
531+
- A queue command is not itself a PR unless explicitly named as one.
532+
- Missing exact scope for any queued PR is a hard stop for that PR only.
533+
- Codex may continue to the next queued PR only if the next PR has exact scope and does not depend on the stopped PR.
534+
- Codex must not create empty ZIPs or package partial PRs.
535+
- EOD merge/push is owner-controlled and requires explicit approval.
536+
- This section supersedes older wording that implies every PR must be independently prompted when Master Control provides a fully scoped sequential queue.
537+
538+
Exact Gamma queue scope examples:
539+
- `PR_26171_GAMMA_001-sqlite-deprecation-audit`
540+
- Branch: `pr/26171-GAMMA-001-sqlite-deprecation-audit`
541+
- Exact scope: audit active repository documentation and runtime references for SQLite deprecation status; document existing SQLite references as legacy technical debt only; do not add SQLite services, DDL, seed data, or runtime persistence.
542+
- Validation scope: docs/static validation only; do not run Playwright; do not run samples.
543+
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`, and repo-structured ZIP.
544+
- `PR_26171_GAMMA_002-admin-system-health-foundation`
545+
- Branch: `pr/26171-GAMMA-002-admin-system-health-foundation`
546+
- Exact scope: document the Admin system-health foundation contract for diagnostics ownership, visible failure reporting, and Postgres-first health checks; do not implement admin UI, runtime APIs, auth behavior, or database migrations.
547+
- Validation scope: docs/static validation only; do not run Playwright; do not run samples.
548+
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`, and repo-structured ZIP.
549+
- `PR_26171_GAMMA_003-admin-postgres-r2-diagnostics`
550+
- Branch: `pr/26171-GAMMA-003-admin-postgres-r2-diagnostics`
551+
- Exact scope: document Admin diagnostics expectations for Postgres and R2 readiness, credential-presence checks, actionable failure messages, and skipped-lane reporting; do not implement R2 storage, database connections, migrations, or admin runtime behavior.
552+
- Validation scope: docs/static validation only; do not run Playwright; do not run samples.
553+
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`, and repo-structured ZIP.
554+
527555
Stable and merge approval:
528556
- Stable promotion and merge approval are controlled by the assigned Team Alpha, Team Beta, or Team Gamma owner.
529557
- Master Control may recommend sequencing, but Codex must not merge or mark stable without explicit owner approval for the affected workstream.
Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
# PR_26171_GAMMA_004 Instruction Compliance Checklist
2+
3+
- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`: PASS.
4+
- Read `docs_build/dev/PROJECT_MULTI_PC.txt`: PASS.
5+
- Start from `main`: PASS.
6+
- Pull latest `main`: PASS.
7+
- Verify clean repository before branch creation: PASS.
8+
- Verify PR name includes TEAM token: PASS.
9+
- Verify TEAM owner matches Team Gamma governance/diagnostics/instruction-hardening ownership: PASS.
10+
- Create requested branch `pr/26171-GAMMA-004-sequential-gamma-queue-governance`: PASS.
11+
- Keep scope limited to requested docs and reports: PASS.
12+
- Run docs/static validation only: PASS.
13+
- Run `git diff --check`: PASS.
14+
- Verify PR naming examples use TEAM token: PASS.
15+
- Verify branch examples use TEAM token: PASS.
16+
- Verify automatic merge wording is constrained by owner-controlled EOD approval: PASS.
17+
- Do not run Playwright: PASS.
18+
- Do not run samples: PASS.
19+
- Create required PR-specific report: PASS.
20+
- Create manual validation notes: PASS.
21+
- Create instruction compliance checklist: PASS.
22+
- Create `codex_review.diff`: PASS.
23+
- Create `codex_changed_files.txt`: PASS.
24+
- Create repo-structured ZIP under `tmp/`: PASS.
25+
- Do not merge before explicit EOD approval: PASS.
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# PR_26171_GAMMA_004 Manual Validation Notes
2+
3+
Manual validation status: PASS.
4+
5+
Manual checks performed:
6+
- Reviewed `docs_build/dev/PROJECT_INSTRUCTIONS.md` Sequential Codex Queue Mode section for required Team Gamma queue rules.
7+
- Reviewed `docs_build/dev/PROJECT_MULTI_PC.txt` Team Gamma ownership section for matching queue-mode routing guidance.
8+
- Confirmed exact Gamma queue examples include TEAM tokens and branch names with TEAM tokens.
9+
- Confirmed Playwright and samples are explicitly skipped because this PR changes docs/workflow governance only.
10+
- Confirmed no merge was performed and EOD approval remains owner-controlled.
11+
12+
Expected outcome:
13+
- Future Team Gamma sequential queue commands may contain multiple ordered, exact-scoped PRs.
14+
- Missing exact scope blocks only the affected queued PR when later queued PRs are independent and exact-scoped.
15+
- Codex must not package empty ZIPs or partial PRs.
16+
17+
Out of scope:
18+
- Runtime behavior.
19+
- Admin UI implementation.
20+
- Database migration or connection behavior.
21+
- R2 storage implementation.
22+
- Playwright validation.
23+
- Samples validation.
Lines changed: 69 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,69 @@
1+
# PR_26171_GAMMA_004-sequential-gamma-queue-governance
2+
3+
## Summary
4+
5+
Team ownership: GAMMA.
6+
7+
Purpose: add Sequential Codex Queue Mode governance for ordered Team Gamma queues while preserving exact-scope, report, validation, ZIP, and owner-controlled EOD approval requirements.
8+
9+
## Scope
10+
11+
Changed files:
12+
- `docs_build/dev/PROJECT_INSTRUCTIONS.md`
13+
- `docs_build/dev/PROJECT_MULTI_PC.txt`
14+
- `docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance.md`
15+
- `docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance-manual-validation-notes.md`
16+
- `docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance-instruction-compliance-checklist.md`
17+
- `docs_build/dev/reports/codex_review.diff`
18+
- `docs_build/dev/reports/codex_changed_files.txt`
19+
20+
## Instruction Compliance
21+
22+
Start gate: PASS.
23+
24+
- Required instruction reads: PASS. Read `docs_build/dev/PROJECT_INSTRUCTIONS.md` and `docs_build/dev/PROJECT_MULTI_PC.txt`.
25+
- Branch before changes: `main`.
26+
- Clean status before branch: PASS.
27+
- Created branch: `pr/26171-GAMMA-004-sequential-gamma-queue-governance`.
28+
- PR TEAM owner: GAMMA.
29+
- Implementation path: docs-only governance update in `docs_build/dev/PROJECT_INSTRUCTIONS.md` and `docs_build/dev/PROJECT_MULTI_PC.txt`.
30+
- Validation scope: docs/static only.
31+
- Required reports: PASS. PR report, manual validation notes, instruction compliance checklist, `codex_review.diff`, and `codex_changed_files.txt`.
32+
- ZIP requirement: PASS. Required final ZIP path is `tmp/PR_26171_GAMMA_004-sequential-gamma-queue-governance_delta.zip`.
33+
34+
## Requirement Checklist
35+
36+
- Add Team Gamma Sequential Codex Queue Mode governance: PASS.
37+
- State that queue commands can contain multiple ordered PR scopes: PASS.
38+
- Require exact PR name, TEAM token, branch name, exact scope, validation scope, reports, manual notes, and ZIP requirement per queued PR: PASS.
39+
- State a queue command is not itself a PR unless explicitly named as one: PASS.
40+
- State missing exact scope is a hard stop for that PR only: PASS.
41+
- Allow Codex to continue only when the next PR is exact-scoped and independent: PASS.
42+
- Prohibit empty ZIPs and partial PR packaging: PASS.
43+
- Preserve owner-controlled EOD merge/push approval: PASS.
44+
- Supersede older independent-prompt wording for fully scoped Master Control queues: PASS.
45+
- Add exact Gamma queue examples for PRs 001, 002, and 003: PASS.
46+
47+
## Validation
48+
49+
Lanes executed:
50+
- docs/static - governance docs only.
51+
52+
Commands run:
53+
- `git diff --check` - PASS.
54+
- `rg -n "PR_26171_GAMMA_00[1-4]|pr/26171-GAMMA-00[1-4]" docs_build/dev/PROJECT_INSTRUCTIONS.md docs_build/dev/PROJECT_MULTI_PC.txt` - PASS.
55+
- `rg -n "Merge PR only after explicit owner/EOD approval|Treat merge as required only after validation passes and explicit owner/EOD approval is provided|EOD merge/push is owner-controlled and requires explicit approval|Codex must not merge a PR or mark a workstream stable without explicit approval" docs_build/dev/PROJECT_INSTRUCTIONS.md docs_build/dev/PROJECT_MULTI_PC.txt` - PASS.
56+
57+
Skipped lanes:
58+
- Playwright: SKIP. Docs/workflow-only PR with no UI, tool runtime, toolState, or workspace behavior changes.
59+
- Samples: SKIP. Docs/workflow-only PR with no sample loader, sample JSON, or sample runtime changes.
60+
- Runtime, integration, and engine: SKIP. No runtime, handoff, shared parser, or engine behavior changed.
61+
62+
## Git Workflow
63+
64+
- Current branch: `pr/26171-GAMMA-004-sequential-gamma-queue-governance`.
65+
- Created branch: `pr/26171-GAMMA-004-sequential-gamma-queue-governance`.
66+
- Push result: pending final publish.
67+
- PR URL: pending final publish.
68+
- Merge result: not merged; EOD approval required.
69+
- Final main commit before branch: `78b642f70`.
Lines changed: 14 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -1,42 +1,14 @@
1-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-manual-validation-notes.md
2-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-message-tts-ownership-checklist.md
3-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-parent-child-table-checklist.md
4-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-validation.md
5-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment.md
6-
docs_build/dev/reports/codex_changed_files.txt
7-
docs_build/dev/reports/codex_review.diff
8-
docs_build/dev/reports/coverage_changed_js_guardrail.txt
9-
docs_build/dev/reports/dependency_gating_report.md
10-
docs_build/dev/reports/dependency_hydration_reuse_report.md
11-
docs_build/dev/reports/execution_graph_reuse_report.md
12-
docs_build/dev/reports/failure_fingerprint_report.md
13-
docs_build/dev/reports/filesystem_scan_reduction_report.md
14-
docs_build/dev/reports/incremental_validation_report.md
15-
docs_build/dev/reports/lane_compilation_report.md
16-
docs_build/dev/reports/lane_deduplication_report.md
17-
docs_build/dev/reports/lane_input_validation_report.md
18-
docs_build/dev/reports/lane_manifests/workspace-contract.json
19-
docs_build/dev/reports/lane_runtime_optimization_report.md
20-
docs_build/dev/reports/lane_snapshot_report.md
21-
docs_build/dev/reports/lane_snapshots/workspace-contract.json
22-
docs_build/dev/reports/lane_warm_start_report.md
23-
docs_build/dev/reports/lane_warm_starts/workspace-contract.json
24-
docs_build/dev/reports/monolith_trigger_removal_report.md
25-
docs_build/dev/reports/persistent_lane_manifest_report.md
26-
docs_build/dev/reports/playwright_discovery_ownership_report.md
27-
docs_build/dev/reports/playwright_discovery_scope_report.md
28-
docs_build/dev/reports/playwright_structure_audit.md
29-
docs_build/dev/reports/playwright_v8_coverage_report.txt
30-
docs_build/dev/reports/retry_suppression_report.md
31-
docs_build/dev/reports/slow_path_pruning_report.md
32-
docs_build/dev/reports/static_validation_report.md
33-
docs_build/dev/reports/targeted_file_manifest_report.md
34-
docs_build/dev/reports/test_cleanup_performance_report.md
35-
docs_build/dev/reports/test_cleanup_routing_report.md
36-
docs_build/dev/reports/testing_lane_execution_report.md
37-
docs_build/dev/reports/validation_cache_report.md
38-
docs_build/dev/reports/zero_browser_preflight_report.md
39-
src/dev-runtime/messages/messages-sqlite-service.mjs
40-
tests/playwright/tools/MessagesTool.spec.mjs
41-
toolbox/messages/index.html
42-
toolbox/messages/messages.js
1+
M docs_build/dev/PROJECT_INSTRUCTIONS.md
2+
M docs_build/dev/PROJECT_MULTI_PC.txt
3+
A docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance-instruction-compliance-checklist.md
4+
A docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance-manual-validation-notes.md
5+
A docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance.md
6+
M docs_build/dev/reports/codex_changed_files.txt
7+
M docs_build/dev/reports/codex_review.diff
8+
9+
docs_build/dev/PROJECT_INSTRUCTIONS.md | 62 +++++++++++++++++--
10+
docs_build/dev/PROJECT_MULTI_PC.txt | 28 +++++++++
11+
...-governance-instruction-compliance-checklist.md | 25 ++++++++
12+
...mma-queue-governance-manual-validation-notes.md | 23 ++++++++
13+
..._GAMMA_004-sequential-gamma-queue-governance.md | 69 ++++++++++++++++++++++
14+
5 files changed, 203 insertions(+), 4 deletions(-)

0 commit comments

Comments
 (0)