Skip to content

Commit 090163b

Browse files
ToolboxAidToolboxAid
committed
Add Codex ZIP and next PR governance
1 parent ae1d838 commit 090163b

10 files changed

Lines changed: 436 additions & 380 deletions

‎docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md‎

Lines changed: 27 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -6,20 +6,31 @@ Standardize Codex deliverables, completion reporting, and artifact generation.
66

77
## ZIP Artifact Requirement
88

9-
Every Codex task must produce a ZIP artifact.
9+
Every Codex execution must produce a repo-structured ZIP artifact and the required reports.
1010

11-
Applies to:
12-
- Success
13-
- Failure
14-
- Stop Gate
15-
- Partial Completion
16-
- Review Deliverables
17-
- Governance Deliverables
11+
This applies regardless of result:
12+
13+
- success
14+
- completion
15+
- partial completion
16+
- hard stop
17+
- blocked
18+
- validation failure
19+
- new information discovered
20+
- no files changed
21+
- review deliverables
22+
- governance deliverables
23+
24+
No exceptions.
1825

1926
Minimum ZIP contents:
20-
- summary.md
27+
28+
- changed or preserved repo files from the run, stored at repo-relative paths
29+
- required reports under `docs_build/dev/reports/`
2130

2231
Optional:
32+
33+
- summary.md
2334
- changed-files.txt
2435
- findings.md
2536
- validation.txt
@@ -28,11 +39,16 @@ Optional:
2839
## Completion Reporting
2940

3041
Codex responses must include:
42+
3143
- ZIP filename
3244
- ZIP location
45+
- repo-structured ZIP path
46+
- reports
47+
- changed file list
3348
- PR number(s)
3449
- Merge commit(s)
3550
- Validation results
51+
- branch, worktree, and local/origin sync status when relevant
3652

3753
## Tool MVP PR Report Requirements
3854

@@ -82,3 +98,5 @@ unless explicitly requested.
8298
## No ZIP Means Incomplete
8399

84100
A task is not considered complete until the ZIP artifact is generated and reported.
101+
102+
If execution stops before implementation, validation, or commit, the stop result must still include the repo-structured ZIP and reports that document the hard stop, blocker, validation failure, or no-change result.

‎docs_build/dev/ProjectInstructions/addendums/pr_workflow.md‎

Lines changed: 33 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,10 +126,11 @@ Closed gates:
126126
- If validation fails, stop and report.
127127
- If conflict occurs, stop and report.
128128
- If OWNER decision is required, stop and report.
129-
- Every completed Codex PR run must produce a repo-structured ZIP under `tmp/`.
130-
- The ZIP rule applies to implementation, audit, report-only, validation-only, governance, and cleanup PRs.
129+
- Every Codex execution must produce a repo-structured ZIP under `tmp/`.
130+
- The ZIP rule applies to implementation, audit, report-only, validation-only, governance, cleanup, hard-stop, blocked, validation-failure, partial-completion, new-information, and no-change runs.
131131
- The ZIP must include all changed or preserved repo files from the run and must not replace required reports under `docs_build/dev/reports/`.
132-
- 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.
132+
- If no repo files changed, Codex must still create a ZIP containing the report that proves the no-change, hard-stop, blocked, validation-failure, or partial-completion result.
133+
- No exceptions.
133134

134135
## Batch Governance Mode
135136

@@ -240,6 +241,35 @@ Stop only for:
240241
- Validation failure
241242
- OWNER decision
242243

244+
## Completed PR ZIP Review And Next Logical PRs
245+
246+
Whenever the Product Owner provides a completed PR ZIP to ChatGPT, ChatGPT shall:
247+
248+
- review the completed work
249+
- identify completed scope
250+
- identify remaining scope
251+
- update the team backlog and completion percentages
252+
- determine the next logical PRs
253+
- recommend the execution order
254+
- generate each recommended PR with:
255+
- PR summary
256+
- Codex command
257+
- commit comment
258+
- Playwright scope
259+
- manual validation steps
260+
261+
Use plural wording: `next logical PRs`.
262+
263+
Unless blocked by dependencies or directed otherwise by the Product Owner, ChatGPT should generate enough sequential PRs to complete the current backlog item to a Product Owner testable state rather than stopping after a single PR.
264+
265+
The Product Owner should not have to ask:
266+
267+
- `What is next?`
268+
- `Create the next PR.`
269+
- `Continue.`
270+
271+
The backlog drives the next recommended PRs automatically.
272+
243273
## EOD Main Lock
244274

245275
End of Day:

‎docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@ The backlog is the authoritative source for determining the next PRs.
5353

5454
If the backlog and a generated report conflict, the backlog wins unless OWNER explicitly approves a newer governance decision.
5555

56+
When the Product Owner provides a completed PR ZIP, ChatGPT must review that ZIP, update the active team backlog and completion percentages, and recommend the next logical PRs needed to keep moving toward a Product Owner testable outcome.
57+
5658
## Start Of Day Team Briefing
5759

5860
When a team is assigned, ChatGPT/Codex must provide this briefing before implementation begins:

‎docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md‎

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,56 @@
1+
# PR_26177_OWNER_011-codex-zip-and-next-pr-standard
2+
3+
Date: 2026-06-27
4+
Team: OWNER
5+
Branch: PR_26177_OWNER_011-codex-zip-and-next-pr-standard
6+
7+
## Purpose
8+
9+
Add governance that every Codex execution produces reports and a repo-structured ZIP, and that completed PR ZIP review automatically drives the next logical PRs.
10+
11+
## Scope
12+
13+
Documentation and governance only.
14+
15+
No runtime code, UI code, API code, database code, `start_of_day` files, history snapshots, or unrelated cleanup were changed.
16+
17+
## Changes
18+
19+
- Strengthened the Codex artifact standard so every Codex execution produces required reports and a repo-structured ZIP.
20+
- Made the ZIP/report rule apply to success, completion, partial completion, hard stop, blocked, validation failure, new information, no-change, review, and governance results.
21+
- Removed the previous exception that allowed hard stops before outputs to skip ZIP/report delivery.
22+
- Required Codex responses to return ZIP path, reports, changed file list, validation results, and branch/worktree/sync status when relevant.
23+
- Added completed PR ZIP review governance:
24+
- review completed work
25+
- identify completed and remaining scope
26+
- update team backlog and completion percentages
27+
- determine the next logical PRs
28+
- recommend execution order
29+
- generate each recommended PR with summary, Codex command, commit comment, Playwright scope, and manual validation steps
30+
- Confirmed the backlog drives next recommended PRs automatically.
31+
- Preserved the existing SOD, active team branch, sequential workstream, EOD, Product Owner testable, and page-level Playwright workflow rules.
32+
33+
## Validation
34+
35+
- PASS: `git diff --cached --check -- . :(exclude)docs_build/dev/reports/codex_review.diff`
36+
- PASS: documentation/governance-only changed-file check.
37+
- PASS: ZIP-on-every-result rule exists.
38+
- PASS: hard stops require ZIP/report output.
39+
- PASS: `next logical PRs` plural wording exists.
40+
- PASS: automatic next PR planning is tied to completed PR ZIP review.
41+
- PASS: SOD starts from latest synchronized `main`.
42+
- PASS: SOD creates or uses the active team branch.
43+
- PASS: all commits go to the active team branch, not `main`.
44+
- PASS: sequential PRs stay on the active team branch/workstream during the day.
45+
- PASS: EOD returns to `main` and verifies clean sync `0 0`.
46+
- PASS: Product Owner testable completion rule remains present.
47+
- PASS: page-level Playwright completion gate remains present.
48+
- PASS: no runtime files changed.
49+
50+
## Artifact
51+
52+
- `tmp/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_delta.zip`
53+
54+
## Manual Validation Notes
55+
56+
Reviewers should confirm the active governance now makes ZIP/report delivery mandatory for every Codex result and makes completed PR ZIP review produce the next logical PRs without requiring the Product Owner to ask for continuation.

‎docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# PR_26177_OWNER_011 Branch Validation
2+
3+
Date: 2026-06-27
4+
Team: OWNER
5+
Branch: PR_26177_OWNER_011-codex-zip-and-next-pr-standard
6+
7+
## Result
8+
9+
PASS
10+
11+
## Checks
12+
13+
- PASS: Work was performed on the active OWNER PR branch, not `main`.
14+
- PASS: Branch is stacked on `PR_26177_OWNER_010-team-backlog-sod-eod-standard`.
15+
- PASS: Changes are limited to active Project Instructions and generated reports.
16+
- PASS: No runtime, UI, API, database, or `start_of_day` files changed.
17+
- PASS: Staged whitespace validation passed with generated `codex_review.diff` excluded.
18+
- PASS: Required report files were generated.
19+
- PASS: Repo-structured ZIP artifact path is defined under `tmp/`.
20+
21+
## Notes
22+
23+
This PR remains in the OWNER workstream and must not be merged to `main` until EOD closeout or explicit OWNER approval.

‎docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# PR_26177_OWNER_011 Manual Validation Notes
2+
3+
Date: 2026-06-27
4+
Team: OWNER
5+
6+
## Manual Review
7+
8+
Review these files:
9+
10+
- `docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md`
11+
- `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md`
12+
- `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md`
13+
14+
Confirm they require every Codex result to include reports and a repo-structured ZIP, including hard stops and no-change results.
15+
16+
Confirm completed PR ZIP review now drives the next logical PRs and produces enough sequential PRs to reach a Product Owner testable outcome unless blocked or directed otherwise.
17+
18+
## Expected Reviewer Outcome
19+
20+
The Product Owner should no longer need to ask for continuation after handing ChatGPT a completed PR ZIP; the backlog and ZIP review should automatically drive the next recommended PRs.

‎docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md‎

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
# PR_26177_OWNER_011 Requirements Checklist
2+
3+
Date: 2026-06-27
4+
Team: OWNER
5+
6+
## Requirement Results
7+
8+
- PASS: Documentation/governance only.
9+
- PASS: Active Project Instructions reviewed and updated.
10+
- PASS: Active instruction files updated only.
11+
- PASS: Historical snapshots were not rewritten.
12+
- PASS: No runtime changes.
13+
- PASS: ZIP-on-every-result governance added.
14+
- PASS: Rule applies to success, completion, partial completion, hard stop, blocked, validation failure, new information, and no files changed.
15+
- PASS: Every Codex execution must return repo-structured ZIP path, reports, changed file list, validation results, and branch/worktree/sync status when relevant.
16+
- PASS: `No exceptions` wording added.
17+
- PASS: Automatic next PR planning tied to Product Owner completed PR ZIP review.
18+
- PASS: `next logical PRs` plural wording exists.
19+
- PASS: Recommended PR output includes PR summary, Codex command, commit comment, Playwright scope, and manual validation steps.
20+
- PASS: Product Owner should not need to ask `What is next?`, `Create the next PR.`, or `Continue.`
21+
- PASS: Backlog drives next recommended PRs automatically.
22+
- PASS: Existing SOD/main, active team branch, stacked PR, EOD, Product Owner testable, and page-level Playwright rules remain present.
23+
- PASS: Required reports were generated.
24+
- PASS: Repo-structured ZIP artifact will be generated under `tmp/`.
25+
26+
## Restrictions
27+
28+
- PASS: No commit was made to `main`.
29+
- PASS: No `start_of_day` folders changed.
30+
- PASS: No unrelated cleanup.

‎docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md‎

Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
# PR_26177_OWNER_011 Validation Lane
2+
3+
Date: 2026-06-27
4+
Team: OWNER
5+
6+
## Commands
7+
8+
```powershell
9+
git diff --cached --check -- . ":(exclude)docs_build/dev/reports/codex_review.diff"
10+
```
11+
12+
Result: PASS
13+
14+
```powershell
15+
node -e "documentation/governance-only changed-file check"
16+
```
17+
18+
Result: PASS
19+
20+
```powershell
21+
node -e "required ZIP, next logical PRs, workflow preservation checks"
22+
```
23+
24+
Result: PASS
25+
26+
## Targeted Results
27+
28+
- PASS: ZIP-on-every-result rule exists.
29+
- PASS: hard stops require ZIP/report output.
30+
- PASS: `next logical PRs` plural wording exists.
31+
- PASS: automatic next PR planning is tied to completed PR ZIP review.
32+
- PASS: SOD/main active branch workflow remains present.
33+
- PASS: EOD clean main sync rule remains present.
34+
- PASS: Product Owner testable rule remains present.
35+
- PASS: page-level Playwright completion gate remains present.
36+
- PASS: no runtime files changed.
37+
- PASS: no UI files changed.
38+
- PASS: no API files changed.
39+
- PASS: no database files changed.
40+
41+
## Playwright
42+
43+
Not impacted. This PR is documentation/governance only.

‎docs_build/dev/reports/codex_changed_files.txt‎

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,10 @@
1-
docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md
2-
docs_build/dev/ProjectInstructions/README.txt
3-
docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md
4-
docs_build/dev/ProjectInstructions/addendums/naming_registry.md
1+
docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md
2+
docs_build/dev/ProjectInstructions/addendums/pr_workflow.md
53
docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md
6-
docs_build/dev/ProjectInstructions/addendums/team_start_and_release.md
7-
docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md
8-
docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md
9-
docs_build/dev/reports/PR_26177_OWNER_010-team-backlog-sod-eod-standard.md
10-
docs_build/dev/reports/PR_26177_OWNER_010-team-backlog-sod-eod-standard_branch-validation.md
11-
docs_build/dev/reports/PR_26177_OWNER_010-team-backlog-sod-eod-standard_manual-validation-notes.md
12-
docs_build/dev/reports/PR_26177_OWNER_010-team-backlog-sod-eod-standard_requirements-checklist.md
13-
docs_build/dev/reports/PR_26177_OWNER_010-team-backlog-sod-eod-standard_validation-lane.md
4+
docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md
5+
docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md
6+
docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md
7+
docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md
8+
docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md
149
docs_build/dev/reports/codex_changed_files.txt
1510
docs_build/dev/reports/codex_review.diff

0 commit comments

Comments
 (0)