PR_26175_OWNER_047 require Codex ZIP for all runs

Author: ToolboxAid

docs_build/dev/PROJECT_INSTRUCTIONS.md

@@ -784,12 +784,13 @@ The full samples smoke test rule remains separate and runs only when broadly imp
 
 ## CODEX REVIEW DIFF REQUIREMENT
 
-Every Codex PR must produce review artifacts so ChatGPT can review the exact code changes.
+Every completed Codex PR or completed Codex run must produce review artifacts so ChatGPT can review the exact repo changes or the no-change evidence.
 
 Codex must create:
 
 - `docs_build/dev/reports/codex_review.diff`
 - `docs_build/dev/reports/codex_changed_files.txt`
+- a PR-specific or EOD-specific report under `docs_build/dev/reports/`
 
 `codex_review.diff` must contain:
 - `git diff --cached`
@@ -804,6 +805,9 @@ Rules:
 - Do not pause commits
 - Do not add dependencies
 - Do not change runtime behavior just to create review artifacts
+- Required reports remain required even when a ZIP is created.
+- Required report evidence must include branch validation PASS/FAIL, a requirement-by-requirement PASS/FAIL checklist, a validation lane report, and manual validation notes.
+- A ZIP artifact must not replace or move required report files out of `docs_build/dev/reports/`.
 
 When user asks for code review, they should upload:
 - PR delta ZIP
@@ -812,7 +816,7 @@ When user asks for code review, they should upload:
 
 ## MANUAL TEST REQUIREMENT
 
-Every PR must include:
+Every completed PR, audit, report-only, validation-only, or EOD closeout run must include:
 - exact manual validation steps
 - expected outcome
 - any known out-of-scope checks
@@ -870,8 +874,15 @@ Codex ZIPs must:
 - be repo-structured
 - preserve exact repo-relative paths
 - be placed under `<project folder>/tmp/`
-- use the PR name in the ZIP filename
+- use the PR or EOD run name in the ZIP filename
 - contain no extra files outside the defined structure
+- include all changed or preserved repo files from the completed run
+- not include or depend on generated loose files under `tmp/`
+- not replace required report files under `docs_build/dev/reports/`
+
+If a completed run changes no repo files, Codex must still create a ZIP containing the PR or EOD report that proves no repo files changed.
+
+The no-change ZIP rule does not apply when the run hard-stopped before producing outputs.
 
 Before Codex returns any ZIP, Codex must:
 1. Physically create the ZIP file.
@@ -1584,11 +1595,21 @@ Required reports:
 
 ## REQUIRED ZIP OUTPUT
 
-Codex must ALWAYS produce a repo-structured ZIP for every PR.
+Codex must ALWAYS produce a repo-structured ZIP for every completed Codex run.
+
+This applies to:
+- implementation PRs
+- audit PRs
+- report-only PRs
+- validation-only PRs
+- EOD closeout runs
+- governance and cleanup runs
 
 The ZIP must follow the existing CODEX ZIP STANDARD.
 
-The ZIP is required output, not optional.
+The ZIP must include all changed or preserved repo files from the run.
+
+The ZIP is required output, not optional, and it does not replace required reports under `docs_build/dev/reports/`.
 
 ## HTML FILE RESTRICTIONS
 
@@ -1823,10 +1844,12 @@ Do NOT require:
 
 ## CODEX ZIP RETURN CONTRACT
 
-Codex must include the repo-structured ZIP in returned artifacts for user and ChatGPT review.
+Codex must include the repo-structured ZIP path in returned artifacts for user and ChatGPT review for every completed run.
 
 The ZIP must still follow the CODEX ZIP STANDARD.
 
+If no repo files changed, the returned ZIP must contain the PR or EOD report proving the no-change result unless the run hard-stopped before outputs were produced.
+
 ## CODE REVIEW EVIDENCE RULE
 
 ChatGPT must not claim code review was completed unless it inspected uploaded source, ZIP contents, or `codex_review.diff`.
@@ -2044,6 +2067,7 @@ Required steps:
    - local/origin sync = 0 0
 5. Record final main commit.
 6. Report final repository state.
+7. Produce the required repo-structured ZIP under `tmp/` containing the EOD report and any changed or preserved repo files from the closeout.
 
 Required final state:
 
@@ -2235,6 +2259,9 @@ Completion hard stops:
 - If required reports are missing, HARD STOP.
 - If `docs_build/dev/reports/codex_review.diff` is missing, HARD STOP.
 - If `docs_build/dev/reports/codex_changed_files.txt` is missing, HARD STOP.
+- If branch validation PASS/FAIL is missing, HARD STOP.
+- If the requirement-by-requirement PASS/FAIL checklist is missing, HARD STOP.
+- If the validation lane report is missing, HARD STOP.
 - If manual validation notes are missing, HARD STOP.
 - If the PR-specific report is missing, HARD STOP.
 - If an instruction compliance checklist is required and missing, HARD STOP.

docs_build/dev/ProjectInstructions/addendums/multi_team.md

@@ -121,6 +121,7 @@ Required gate:
    - blocked PRs with blockers
    - next review queue
    - final branch/worktree/local-origin sync
+   - repo-structured ZIP path under `tmp/`
 
 OWNER_049 lesson:
 - PRs #129, #132, and #134 were merge-approved and still required merge execution.
@@ -148,6 +149,7 @@ Required steps:
    - local/origin sync = 0 0
 5. Record final main commit.
 6. Report final repository state.
+7. Produce a repo-structured ZIP under `tmp/` that includes the EOD report and all changed or preserved repo files from the closeout.
 
 Required final state:
 
@@ -165,6 +167,8 @@ Rules:
 - A workstream is not considered closed until the repository is returned to main.
 - A PI is not considered complete until main is current and synchronized.
 - Do not leave Codex on a feature, team, workstream, recovery, governance, or owner branch after successful merge.
+- A completed EOD closeout must produce the required ZIP even when the closeout changed no repo files; in that case, the ZIP must contain the EOD report proving the no-change result.
+- The EOD ZIP does not replace the EOD report or other required reports under `docs_build/dev/reports/`.
 - If merge succeeds but repository is not returned to main:
   closeout status = FAIL.
 

docs_build/dev/ProjectInstructions/addendums/pr_workflow.md

@@ -34,6 +34,10 @@ Define the standard pull request workflow for Game Foundry Studio.
 - If validation fails, stop and report.
 - If conflict occurs, stop and report.
 - If OWNER decision is required, stop and report.
+- Every completed Codex PR run must produce a repo-structured ZIP under `tmp/`.
+- The ZIP rule applies to implementation, audit, report-only, validation-only, governance, and cleanup PRs.
+- The ZIP must include all changed or preserved repo files from the run and must not replace required reports under `docs_build/dev/reports/`.
+- If no repo files changed, Codex must still create a ZIP containing the PR report that proves the no-change result, unless the run hard-stopped before producing outputs.
 
 ## Batch Governance Mode
 

docs_build/dev/reports/PR_26175_OWNER_047-codex-zip-required-for-all-runs.md

@@ -0,0 +1,67 @@
+# PR_26175_OWNER_047 Codex ZIP Required For All Runs
+
+## Summary
+This governance-only PR clarifies that every completed Codex run must produce a repo-structured ZIP under `tmp/`, including implementation PRs, audit PRs, report-only PRs, validation-only PRs, governance/cleanup runs, and EOD closeouts.
+
+No runtime code changed.
+
+## Located Governance Files
+| File | Role | Change |
+| --- | --- | --- |
+| `docs_build/dev/PROJECT_INSTRUCTIONS.md` | Authoritative root project instructions for Codex outputs, report requirements, ZIP standard, PR completion, and EOD hard stops. | Updated |
+| `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` | PR workflow addendum for standard PR execution rules. | Updated |
+| `docs_build/dev/ProjectInstructions/addendums/multi_team.md` | Multi-team and EOD closeout addendum. | Updated |
+| `docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md` | Index for the append-first Project Instructions operating system. | Reviewed; no output rules to update |
+| `docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md` | Team start and EOD reminder text. | Reviewed; no output rules to update |
+
+## Requirement Checklist
+| Requirement | Result | Evidence |
+| --- | --- | --- |
+| Use main branch only | PASS | Branch validation confirmed current branch is `main`. |
+| Scope governance docs only | PASS | Changed files are governance docs and required reports only. |
+| No runtime code changes | PASS | No `src/`, `assets/`, `toolbox/`, or test runtime files changed. |
+| Locate active governance files | PASS | Active output rules found in root instructions, PR workflow addendum, and EOD multi-team addendum. |
+| Require ZIP for every completed Codex run | PASS | Root instructions now require ZIPs for every completed Codex run. |
+| Cover implementation, audit, report-only, validation-only, and EOD runs | PASS | Root instructions and PR/EOD addendums list those run types. |
+| ZIP includes changed/preserved repo files | PASS | Root ZIP standard and PR/EOD addendums require inclusion of all changed or preserved repo files. |
+| ZIP does not replace reports | PASS | Root instructions and addendums state ZIPs do not replace `docs_build/dev/reports/` files. |
+| Existing reports remain required | PASS | Root report rules explicitly preserve `codex_review.diff`, `codex_changed_files.txt`, PR/EOD report, branch validation, requirement checklist, validation lane report, and manual validation notes. |
+| No-change runs still produce ZIP | PASS | Root instructions and PR/EOD addendums require a ZIP containing the PR/EOD no-change report unless the run hard-stopped before outputs. |
+| Required reports created | PASS | This report, `codex_review.diff`, and `codex_changed_files.txt` were created/updated. |
+| Repo-structured ZIP under `tmp/` | PASS | ZIP created at `tmp/PR_26175_OWNER_047-codex-zip-required-for-all-runs_delta.zip`. |
+
+## Branch Validation
+| Check | Result | Evidence |
+| --- | --- | --- |
+| Current branch is `main` | PASS | `git branch --show-current` returned `main`. |
+| Branch validation recorded | PASS | This section records PASS/FAIL branch evidence. |
+
+## Validation Lane Report
+| Lane | Result | Evidence |
+| --- | --- | --- |
+| Docs-only validation | PASS | Changed paths are limited to `docs_build/dev/` governance and report files. |
+| Playwright impacted | PASS | No; governance docs only. |
+| Runtime validation | PASS | Not applicable; no runtime files changed. |
+
+## Manual Validation Notes
+| Step | Result | Notes |
+| --- | --- | --- |
+| Reviewed root output rules | PASS | Confirmed required report and ZIP sections remain present and now apply to all completed Codex runs. |
+| Reviewed PR workflow addendum | PASS | Confirmed completed PR runs require `tmp/` ZIP output. |
+| Reviewed EOD addendum | PASS | Confirmed EOD reports must include ZIP path and closeouts must produce a ZIP. |
+| Verified report preservation rule | PASS | ZIP language states reports remain required under `docs_build/dev/reports/`. |
+
+## Changed Files
+| File | Purpose |
+| --- | --- |
+| `docs_build/dev/PROJECT_INSTRUCTIONS.md` | Clarifies required reports and universal completed-run ZIP output. |
+| `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` | Adds PR workflow ZIP output rule for completed Codex PR runs. |
+| `docs_build/dev/ProjectInstructions/addendums/multi_team.md` | Adds EOD ZIP path/report expectations and closeout ZIP rule. |
+| `docs_build/dev/reports/PR_26175_OWNER_047-codex-zip-required-for-all-runs.md` | PR-specific report with branch validation, checklist, validation lane, and manual notes. |
+| `docs_build/dev/reports/codex_review.diff` | Required review diff artifact. |
+| `docs_build/dev/reports/codex_changed_files.txt` | Required changed-files artifact. |
+
+## ZIP
+| Artifact | Result |
+| --- | --- |
+| `tmp/PR_26175_OWNER_047-codex-zip-required-for-all-runs_delta.zip` | PASS - repo-structured ZIP containing changed repo files from this run. |

docs_build/dev/reports/codex_changed_files.txt

@@ -1,9 +1,22 @@
-docs_build/dev/BUILD_PR.md
-assets/theme-v2/js/toolbox-status-bar.js
-assets/theme-v2/css/status.css
-tests/playwright/tools/ToolboxSelectedGameStatusBar.spec.mjs
-docs_build/dev/reports/PR_26175_ALFA_011-status-bar-journey-progress-context_report.md
-docs_build/dev/reports/PR_26175_ALFA_011-status-bar-journey-progress-context_validation-lane.md
-docs_build/dev/reports/PR_26175_ALFA_011-status-bar-journey-progress-context_requirements-checklist.md
-docs_build/dev/reports/codex_review.diff
-docs_build/dev/reports/codex_changed_files.txt
+# git status --short
+M  docs_build/dev/PROJECT_INSTRUCTIONS.md
+M  docs_build/dev/ProjectInstructions/addendums/multi_team.md
+M  docs_build/dev/ProjectInstructions/addendums/pr_workflow.md
+A  docs_build/dev/reports/PR_26175_OWNER_047-codex-zip-required-for-all-runs.md
+M  docs_build/dev/reports/codex_changed_files.txt
+M  docs_build/dev/reports/codex_review.diff
+
+# git ls-files --others --exclude-standard
+
+
+# git diff --stat
+
+
+# git diff --cached --stat
+ docs_build/dev/PROJECT_INSTRUCTIONS.md             |  39 +-
+ .../ProjectInstructions/addendums/multi_team.md    |   4 +
+ .../ProjectInstructions/addendums/pr_workflow.md   |   4 +
+ ...75_OWNER_047-codex-zip-required-for-all-runs.md |  67 ++
+ docs_build/dev/reports/codex_changed_files.txt     |  27 +-
+ docs_build/dev/reports/codex_review.diff           | 818 +++++----------------
+ 6 files changed, 307 insertions(+), 652 deletions(-)