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
62 changes: 58 additions & 4 deletions docs_build/dev/PROJECT_INSTRUCTIONS.md
Original file line number Diff line number Diff line change
Expand Up @@ -1999,7 +1999,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 +2008,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 +2031,63 @@ 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 MODE

Team Gamma may use Sequential Codex Queue Mode for governance, diagnostics, admin, recovery, and instruction-hardening work.

Sequential Codex Queue Mode means Codex receives one queue command containing multiple ordered PR scopes.

Each queued PR must still have:
- exact PR name
- TEAM token
- branch name
- exact scope
- validation scope
- required reports
- manual validation notes
- ZIP requirement

Rules:
- A queue command is not itself a PR unless explicitly named as one.
- Missing exact scope for any queued PR is a hard stop for that PR only.
- Codex may continue to the next queued PR only if the next PR has exact scope and does not depend on the stopped PR.
- Codex must not create empty ZIPs or package partial PRs.
- EOD merge/push is owner-controlled and requires explicit approval.
- This section supersedes older wording that implies every PR must be independently prompted when Master Control provides a fully scoped sequential queue.

Exact Gamma queue scope examples:

1. `PR_26171_GAMMA_001-sqlite-deprecation-audit`
- TEAM token: `GAMMA`
- Branch: `pr/26171-GAMMA-001-sqlite-deprecation-audit`
- 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.
- 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`.
- Validation scope: docs/static validation only, including `git diff --check` and targeted SQLite reference review; do not run Playwright; do not run samples.
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`.
- ZIP requirement: `tmp/PR_26171_GAMMA_001-sqlite-deprecation-audit_delta.zip`.

2. `PR_26171_GAMMA_002-admin-system-health-foundation`
- TEAM token: `GAMMA`
- Branch: `pr/26171-GAMMA-002-admin-system-health-foundation`
- 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.
- 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`.
- Validation scope: docs/static validation only, including `git diff --check` and targeted ownership wording review; do not run Playwright; do not run samples.
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`.
- ZIP requirement: `tmp/PR_26171_GAMMA_002-admin-system-health-foundation_delta.zip`.

3. `PR_26171_GAMMA_003-admin-postgres-r2-diagnostics`
- TEAM token: `GAMMA`
- Branch: `pr/26171-GAMMA-003-admin-postgres-r2-diagnostics`
- 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.
- 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`.
- 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.
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`.
- ZIP requirement: `tmp/PR_26171_GAMMA_003-admin-postgres-r2-diagnostics_delta.zip`.

## CODEX INSTRUCTION ENFORCEMENT START GATE

Expand Down
28 changes: 28 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,34 @@ Governance, recovery, diagnostics, and instruction-hardening PRs:
- Must document TEAM ownership compliance in the PR report.
- Recovery reports must include TEAM ownership.

Team Gamma Sequential Codex Queue Mode:
- Team Gamma may use Sequential Codex Queue Mode for governance, diagnostics, admin, recovery, and instruction-hardening work.
- Sequential Codex Queue Mode means Codex receives one queue command containing multiple ordered PR scopes.
- 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.
- A queue command is not itself a PR unless explicitly named as one.
- Missing exact scope for any queued PR is a hard stop for that PR only.
- Codex may continue to the next queued PR only if the next PR has exact scope and does not depend on the stopped PR.
- Codex must not create empty ZIPs or package partial PRs.
- EOD merge/push is owner-controlled and requires explicit approval.
- This section supersedes older wording that implies every PR must be independently prompted when Master Control provides a fully scoped sequential queue.

Exact Gamma queue scope examples:
- `PR_26171_GAMMA_001-sqlite-deprecation-audit`
- Branch: `pr/26171-GAMMA-001-sqlite-deprecation-audit`
- 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.
- Validation scope: docs/static validation only; do not run Playwright; do not run samples.
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`, and repo-structured ZIP.
- `PR_26171_GAMMA_002-admin-system-health-foundation`
- Branch: `pr/26171-GAMMA-002-admin-system-health-foundation`
- 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.
- Validation scope: docs/static validation only; do not run Playwright; do not run samples.
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`, and repo-structured ZIP.
- `PR_26171_GAMMA_003-admin-postgres-r2-diagnostics`
- Branch: `pr/26171-GAMMA-003-admin-postgres-r2-diagnostics`
- 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.
- Validation scope: docs/static validation only; do not run Playwright; do not run samples.
- Required reports: PR-specific report, manual validation notes, instruction compliance checklist, `codex_review.diff`, `codex_changed_files.txt`, and repo-structured ZIP.

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,25 @@
# PR_26171_GAMMA_004 Instruction Compliance Checklist

- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`: PASS.
- Read `docs_build/dev/PROJECT_MULTI_PC.txt`: PASS.
- Start from `main`: PASS.
- Pull latest `main`: PASS.
- Verify clean repository before branch creation: PASS.
- Verify PR name includes TEAM token: PASS.
- Verify TEAM owner matches Team Gamma governance/diagnostics/instruction-hardening ownership: PASS.
- Create requested branch `pr/26171-GAMMA-004-sequential-gamma-queue-governance`: PASS.
- Keep scope limited to requested docs and reports: PASS.
- Run docs/static validation only: PASS.
- Run `git diff --check`: PASS.
- Verify PR naming examples use TEAM token: PASS.
- Verify branch examples use TEAM token: PASS.
- Verify automatic merge wording is constrained by owner-controlled EOD approval: PASS.
- Do not run Playwright: PASS.
- Do not run samples: PASS.
- Create required PR-specific report: PASS.
- Create manual validation notes: PASS.
- Create instruction compliance checklist: PASS.
- Create `codex_review.diff`: PASS.
- Create `codex_changed_files.txt`: PASS.
- Create repo-structured ZIP under `tmp/`: PASS.
- Do not merge before explicit EOD approval: PASS.
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# PR_26171_GAMMA_004 Manual Validation Notes

Manual validation status: PASS.

Manual checks performed:
- Reviewed `docs_build/dev/PROJECT_INSTRUCTIONS.md` Sequential Codex Queue Mode section for required Team Gamma queue rules.
- Reviewed `docs_build/dev/PROJECT_MULTI_PC.txt` Team Gamma ownership section for matching queue-mode routing guidance.
- Confirmed exact Gamma queue examples include TEAM tokens and branch names with TEAM tokens.
- Confirmed Playwright and samples are explicitly skipped because this PR changes docs/workflow governance only.
- Confirmed no merge was performed and EOD approval remains owner-controlled.

Expected outcome:
- Future Team Gamma sequential queue commands may contain multiple ordered, exact-scoped PRs.
- Missing exact scope blocks only the affected queued PR when later queued PRs are independent and exact-scoped.
- Codex must not package empty ZIPs or partial PRs.

Out of scope:
- Runtime behavior.
- Admin UI implementation.
- Database migration or connection behavior.
- R2 storage implementation.
- Playwright validation.
- Samples validation.
Original file line number Diff line number Diff line change
@@ -0,0 +1,69 @@
# PR_26171_GAMMA_004-sequential-gamma-queue-governance

## Summary

Team ownership: GAMMA.

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.

## Scope

Changed files:
- `docs_build/dev/PROJECT_INSTRUCTIONS.md`
- `docs_build/dev/PROJECT_MULTI_PC.txt`
- `docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance.md`
- `docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance-manual-validation-notes.md`
- `docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance-instruction-compliance-checklist.md`
- `docs_build/dev/reports/codex_review.diff`
- `docs_build/dev/reports/codex_changed_files.txt`

## Instruction Compliance

Start gate: PASS.

- Required instruction reads: PASS. Read `docs_build/dev/PROJECT_INSTRUCTIONS.md` and `docs_build/dev/PROJECT_MULTI_PC.txt`.
- Branch before changes: `main`.
- Clean status before branch: PASS.
- Created branch: `pr/26171-GAMMA-004-sequential-gamma-queue-governance`.
- PR TEAM owner: GAMMA.
- Implementation path: docs-only governance update in `docs_build/dev/PROJECT_INSTRUCTIONS.md` and `docs_build/dev/PROJECT_MULTI_PC.txt`.
- Validation scope: docs/static only.
- Required reports: PASS. PR report, manual validation notes, instruction compliance checklist, `codex_review.diff`, and `codex_changed_files.txt`.
- ZIP requirement: PASS. Required final ZIP path is `tmp/PR_26171_GAMMA_004-sequential-gamma-queue-governance_delta.zip`.

## Requirement Checklist

- Add Team Gamma Sequential Codex Queue Mode governance: PASS.
- State that queue commands can contain multiple ordered PR scopes: PASS.
- Require exact PR name, TEAM token, branch name, exact scope, validation scope, reports, manual notes, and ZIP requirement per queued PR: PASS.
- State a queue command is not itself a PR unless explicitly named as one: PASS.
- State missing exact scope is a hard stop for that PR only: PASS.
- Allow Codex to continue only when the next PR is exact-scoped and independent: PASS.
- Prohibit empty ZIPs and partial PR packaging: PASS.
- Preserve owner-controlled EOD merge/push approval: PASS.
- Supersede older independent-prompt wording for fully scoped Master Control queues: PASS.
- Add exact Gamma queue examples for PRs 001, 002, and 003: PASS.

## Validation

Lanes executed:
- docs/static - governance docs only.

Commands run:
- `git diff --check` - PASS.
- `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.
- `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.

Skipped lanes:
- Playwright: SKIP. Docs/workflow-only PR with no UI, tool runtime, toolState, or workspace behavior changes.
- Samples: SKIP. Docs/workflow-only PR with no sample loader, sample JSON, or sample runtime changes.
- Runtime, integration, and engine: SKIP. No runtime, handoff, shared parser, or engine behavior changed.

## Git Workflow

- Current branch: `pr/26171-GAMMA-004-sequential-gamma-queue-governance`.
- Created branch: `pr/26171-GAMMA-004-sequential-gamma-queue-governance`.
- Push result: PASS. Pushed `pr/26171-GAMMA-004-sequential-gamma-queue-governance` to `origin`.
- PR URL: `https://github.com/ToolboxAid/HTML-JavaScript-Gaming/pull/27`.
- Merge result: not merged; EOD approval required.
- Final main commit before branch: `78b642f70`.
50 changes: 8 additions & 42 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line number Diff line number Diff line change
@@ -1,42 +1,8 @@
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-message-tts-ownership-checklist.md
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-parent-child-table-checklist.md
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-validation.md
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment.md
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/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/dev-runtime/messages/messages-sqlite-service.mjs
tests/playwright/tools/MessagesTool.spec.mjs
toolbox/messages/index.html
toolbox/messages/messages.js
M docs_build/dev/reports/PR_26171_GAMMA_004-sequential-gamma-queue-governance.md

docs_build/dev/PROJECT_INSTRUCTIONS.md | 62 +++++++++++++++++--
docs_build/dev/PROJECT_MULTI_PC.txt | 28 +++++++++
...-governance-instruction-compliance-checklist.md | 25 ++++++++
...mma-queue-governance-manual-validation-notes.md | 23 ++++++++
..._GAMMA_004-sequential-gamma-queue-governance.md | 69 ++++++++++++++++++++++
5 files changed, 203 insertions(+), 4 deletions(-)
Loading
Loading