Skip to content

Commit b499f7f

Browse files
committed
Canonicalize Project Instructions backlog governance
1 parent 090163b commit b499f7f

22 files changed

Lines changed: 922 additions & 333 deletions

docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,20 @@ Existing Project Instructions outside `docs_build/dev/ProjectInstructions/` rema
5858

5959
`docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` defines required Start of Day team briefings, End of Day team summaries, active team backlog fields, completion percentage update points, backlog-driven next PR selection, and official military team-name spelling.
6060

61+
## Canonical Governance Owners
62+
63+
When active guidance overlaps, use these canonical owner documents:
64+
65+
- Workflow and Product Owner testable completion: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md`
66+
- START / WORK / END lifecycle, branch gates, mandatory hard stops, and EOD main lock: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`
67+
- Page-level Playwright organization and completion coverage: `docs_build/dev/ProjectInstructions/addendums/test_structure_standardization.md`
68+
- API/environment model and `Browser -> API -> Database` rule: `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md`
69+
- Environment variable, URL, R2 prefix, and feature flag configuration: `docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md`
70+
- Team backlog fields, completion percentages, and next logical PR ownership: `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md`
71+
- Team ownership and assignment routing: `docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md`
72+
73+
Other active addendums may summarize these rules, but they must point back to the canonical owner document and must not create a competing active rule.
74+
6175
## Environment Governance
6276

6377
`docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md` defines the official environment model, environment invariance rule, shared API/service contract rule, required Supabase/Postgres/R2 services, required R2 prefixes, and SQLite retired status.
@@ -97,7 +111,7 @@ OWNER override wording:
97111
`OWNER override approved: <reason>`
98112

99113
OWNER follows the same safety rules:
100-
- One active Team OWNER branch at a time.
114+
- Team OWNER follows the same one-active-branch discipline as every team.
101115
- One active OWNER assignment at a time.
102116
- OWNER may override team locks, but may not silently delete, rewrite, or remove protected instructions.
103117
- OWNER override must be explicitly documented.

docs_build/dev/ProjectInstructions/README.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ No direct commits to main:
1616
Do not commit directly to main. Normal work must use PR branches, draft PRs, validation evidence, and owner-controlled merge approval.
1717

1818
Branch lifecycle:
19-
Every PR follows exactly three phases: START, WORK, END. The canonical lifecycle is `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`.
19+
The canonical START / WORK / END lifecycle is `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`.
2020

2121
OWNER override rule:
2222
An OWNER override must use this wording:

docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,7 @@ Use `docs_build/dev/ProjectInstructions/` as the only active Project Instruction
1313
Read `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` before implementation.
1414

1515
Branch Lifecycle (Canonical):
16-
- Every PR follows exactly three phases: START, WORK, END.
17-
- Follow `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`.
18-
- START begins on synchronized `main` and creates the PR branch only after all gates pass.
19-
- WORK remains on the PR branch. Never checkout `main`.
20-
- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work.
21-
- No commits on `main`.
22-
- No implementation on `main`.
23-
- No validation on `main` except start validation.
16+
- Follow `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md` for START / WORK / END lifecycle, branch gates, mandatory hard stops, and EOD main lock.
2417

2518
## Start Team Alfa
2619

docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ Owner: OWNER
77

88
Require Codex and teams to confirm branch context before changing files.
99

10+
Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`.
11+
12+
This file owns branch context confirmation and stop conditions; it must not create a competing START / WORK / END lifecycle rule.
13+
1014
## Session Start Context
1115

1216
At the start of work, report or validate:

docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ Owner: OWNER
77

88
Keep active work attached to the correct assigned team, branch, and OWNER decision.
99

10+
Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`.
11+
12+
This file owns branch lock enforcement and OWNER override handling; it must not create a competing START / WORK / END lifecycle rule.
13+
1014
## Active Work Lock
1115

1216
- Work must occur on the active team branch.
@@ -39,21 +43,13 @@ Keep active work attached to the correct assigned team, branch, and OWNER decisi
3943

4044
## Branch Lifecycle (Canonical)
4145

42-
Every PR follows exactly three phases:
46+
The canonical branch lifecycle lives in:
4347

4448
```text
45-
START
46-
WORK
47-
END
49+
docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md
4850
```
4951

50-
The canonical lifecycle lives in `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`.
51-
52-
Branch lock governance enforces:
53-
- START on synchronized `main`.
54-
- WORK only on the PR branch.
55-
- END by merging, returning to synchronized `main`, publishing branch, HEAD SHA, and date/time, then stopping all work.
56-
- Mandatory hard stops before commits on `main`, dirty branch creation, non-`0 0` main sync, baseline SHA mismatch, unvalidated merge, or new unrelated workstream before synchronized main return.
52+
Branch lock governance enforces ownership and branch-lock compliance with that canonical lifecycle.
5753

5854
## OWNER Override
5955

docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,15 +55,15 @@ Codex responses must include:
5555
Every tool MVP PR report must include:
5656
- Product Owner testable outcome
5757
- What Playwright tests
58-
- What Mr. Q should manually test
58+
- What the Product Owner should manually test
5959
- Whether the PR is part of a stacked MVP sequence
6060
- Previous PR dependency
6161
- Next PR dependency
6262

6363
The report must answer:
6464

6565
```text
66-
What can Mr. Q test after applying this ZIP?
66+
What can the Product Owner test after applying this ZIP?
6767
```
6868

6969
If a tool MVP PR has no Playwright lane, the report must state why and list the manual Product Owner validation instead.

docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -23,25 +23,13 @@ Codex must use this as the only active source of truth for:
2323

2424
## Branch Lifecycle Start Gate
2525

26-
Every PR follows exactly three phases:
26+
Codex must follow the canonical lifecycle in:
2727

2828
```text
29-
START
30-
WORK
31-
END
29+
docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md
3230
```
3331

34-
Codex must follow the canonical lifecycle in:
35-
36-
`docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`
37-
38-
Startup enforcement:
39-
- START begins on synchronized `main`.
40-
- WORK remains on the PR branch. Never checkout `main`.
41-
- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work.
42-
- STOP if current branch is `main` before commit.
43-
- STOP if attempting to push `main`.
44-
- STOP if a new PR starts before returning to synchronized `main`.
32+
This startup addendum only requires Codex to read and apply that canonical lifecycle; it must not define a competing lifecycle rule.
4533

4634
Deprecated Project Instructions material outside `docs_build/dev/ProjectInstructions/` is reference-only and must not override active governance.
4735

docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,10 @@ This addendum is governance/documentation only. It does not change runtime behav
1111

1212
## Source Model
1313

14+
Canonical environment/API model reference: `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md`.
15+
16+
This file owns environment variable names, URL configuration, R2 prefix configuration, and feature flag configuration only.
17+
1418
This standard builds on:
1519

1620
```text
@@ -88,12 +92,10 @@ R2 project, backup, export, import, or future storage paths must stay under the
8892

8993
## API/Service Contract Configuration
9094

91-
One shared API/service contract is required across Local (VS Code), DEV, IST, UAT, and PROD.
95+
The canonical shared API/service contract rule lives in `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md`.
9296

93-
Rules:
97+
This section owns configuration-only API URL requirements:
9498

95-
- Browser/UI/runtime code must follow `Browser -> API -> Database` for authoritative product data.
96-
- `Local API` means the same shared API/service contract running locally, not a separate local-only API implementation.
9799
- API URLs may differ by `.env` only.
98100
- Do not split Local API and Public API contracts.
99101
- Do not create environment-specific API/service contracts.

docs_build/dev/ProjectInstructions/addendums/multi_team.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
# Multi-Team Codex Execution Governance
22

3+
Canonical workflow reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md`.
4+
5+
Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`.
6+
7+
This file owns multi-team execution coordination and must not create competing workflow or branch lifecycle rules.
8+
39
## Four Active Delivery Teams
410

511
The single authoritative four-team ownership definition is:
@@ -65,7 +71,7 @@ Rules:
6571
## Day Work / EOD Merge Rule
6672

6773
During active work:
68-
- Work happens on assigned team branches, OWNER branches, or scoped PR branches.
74+
- Work happens on active non-main team branches or scoped PR branches.
6975
- Commits are allowed only on assigned non-main branches.
7076
- Pushes are allowed and expected.
7177
- Draft PRs are allowed and expected.

docs_build/dev/ProjectInstructions/addendums/pr_workflow.md

Lines changed: 4 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -46,31 +46,11 @@ This section supersedes older active wording that implies returning to `main` be
4646

4747
## Branch Lifecycle (Canonical)
4848

49-
Every PR follows exactly three phases:
50-
51-
```text
52-
START
53-
WORK
54-
END
55-
```
56-
5749
The canonical START, WORK, END, Daily Synchronization, and Mandatory Hard Stops rules live in:
5850

5951
`docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`
6052

61-
PR workflow must follow that lifecycle exactly.
62-
63-
Summary:
64-
- START happens on synchronized `main` only and creates the PR branch only after all required gates pass.
65-
- WORK happens only on the PR branch.
66-
- END validates, commits, pushes, opens/updates the PR, merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work.
67-
- No commits on `main`.
68-
- No implementation on `main`.
69-
- No validation on `main` except start validation.
70-
- Never checkout `main` during WORK.
71-
- STOP before commit if current branch is `main`.
72-
- STOP if current branch changes unexpectedly.
73-
- STOP if attempting to push `main`.
53+
PR workflow must follow that lifecycle exactly and must not create a competing lifecycle rule.
7454

7555
## PR Lifecycle States
7656

@@ -153,7 +133,7 @@ Do not stop after every small PR unless blocked by branch state, failed validati
153133
Each tool MVP PR plan or template must include:
154134
- Product Owner testable outcome
155135
- What Playwright tests
156-
- What Mr. Q should manually test
136+
- What the Product Owner should manually test
157137
- Whether the PR is part of a stacked MVP sequence
158138
- Previous PR dependency
159139
- Next PR dependency
@@ -162,6 +142,8 @@ Visible acceptance must be Creator-facing first. Architecture can be handled und
162142

163143
## Product Owner Testable Definition
164144

145+
Canonical owner: this section is the active canonical Product Owner testable completion rule.
146+
165147
A request to complete a page, tool, MVP, or testable experience means Product Owner testable by default. Codex must deliver a working Product Owner testable feature, not a shell or foundation page, unless the Product Owner explicitly requests a shell/foundation PR.
166148

167149
A Product Owner testable outcome means the Product Owner can:

0 commit comments

Comments
 (0)