Skip to content

Commit 5837df3

Browse files
committed
PR_26172_OWNER_010 owner governance rename
1 parent 0ccaf0e commit 5837df3

7 files changed

Lines changed: 312 additions & 153 deletions

docs_build/dev/PROJECT_INSTRUCTIONS.md

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2032,9 +2032,29 @@ Stable promotion and merge approval are owner-controlled.
20322032
Rules:
20332033
- Codex may prepare scoped changes, reports, validation evidence, ZIP artifacts, branches, and PRs.
20342034
- Codex must not merge a PR or mark a workstream stable without explicit approval from the assigned Team Alpha or Team Beta owner.
2035-
- Master Control may recommend sequencing or assignment, but affected workstream owners control stable and merge approval.
2035+
- OWNER 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.
20372037

2038+
## OWNER GOVERNANCE LANE
2039+
2040+
The governance lane formerly referred to by the prior governance token is now OWNER.
2041+
2042+
Rules:
2043+
- Use Team OWNER for owner-governance PRs going forward.
2044+
- PR names use the OWNER token going forward:
2045+
- `PR_<YYJJJ>_OWNER_<###>-<short-description>`
2046+
- Branch names mirror OWNER ownership going forward:
2047+
- `pr/<YYJJJ>-OWNER-<###>-<short-description>`
2048+
- Existing PRs, branches, reports, and ZIPs already uploaded with the prior governance token are historical and must not be renamed unless separately scoped.
2049+
- Required override wording is:
2050+
- `OWNER override approved: <reason>`
2051+
2052+
OWNER follows the same safety rules:
2053+
- One active OWNER branch at a time.
2054+
- One active OWNER assignment at a time.
2055+
- OWNER may override team locks, but may not silently delete, rewrite, or remove protected instructions.
2056+
- OWNER override must be explicitly documented.
2057+
20382058
## CODEX INSTRUCTION ENFORCEMENT START GATE
20392059

20402060
Codex must run this gate before every PR execution and before any file changes.

docs_build/dev/PROJECT_MULTI_PC.txt

Lines changed: 42 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
I think this is the right operating model for GFS at your current stage, with a few modifications to fit how we've already organized the project.
22

3-
Master Control Responsibilities
3+
OWNER Responsibilities
44

5-
Master Control should own:
5+
OWNER should own:
66

77
Product roadmap
88
Tool priority
@@ -189,7 +189,7 @@ These become valuable after creators can build complete games.
189189

190190
Immediate Queue
191191

192-
If we start using Master Control today, my queue would be:
192+
If we start using OWNER today, my queue would be:
193193

194194
Team Alpha
195195
PR_26171_001-game-journey-completion-model
@@ -232,7 +232,7 @@ No TTS generation yet.
232232

233233
This establishes the foundation for dialog, narration, and speech consistency across games.
234234

235-
Master Control Rule
235+
OWNER Rule
236236

237237
Before any new PR is created, ask:
238238

@@ -257,7 +257,7 @@ until a complete game can be built start-to-finish entirely inside Game Foundry
257257
----------------------------------------------------------------------------------------
258258
If your goal is **maximum throughput**, I would have the three chats play different roles.
259259

260-
## Master Control
260+
## OWNER
261261

262262
Don't build anything here.
263263

@@ -336,7 +336,7 @@ it keeps building there.
336336

337337
---
338338

339-
## What should Master Control focus on?
339+
## What should OWNER focus on?
340340

341341
Right now, for GFS, I would prioritize:
342342

@@ -368,7 +368,7 @@ So I'd keep IST mostly as infrastructure work and focus development on tool comp
368368

369369
### 2. Create Tool Ownership Map
370370

371-
Master Control should assign ownership:
371+
OWNER should assign ownership:
372372

373373
```text
374374
Team Alpha
@@ -411,9 +411,9 @@ Never have both teams working in the same tool at the same time.
411411

412412
## What I can do
413413

414-
I can help you turn Master Control into a scheduler.
414+
I can help you turn OWNER into a scheduler.
415415

416-
For example, in Master Control you can ask:
416+
For example, in OWNER you can ask:
417417

418418
```text
419419
Given everything we know about GFS:
@@ -424,7 +424,7 @@ What should Team Beta build next?
424424
Generate the next PR for each.
425425
```
426426

427-
Then Master Control outputs:
427+
Then OWNER outputs:
428428

429429
```text
430430
Team Alpha:
@@ -438,7 +438,7 @@ You paste those into the two development chats.
438438

439439
Both Codex sessions work simultaneously.
440440

441-
Then when PRs come back, Master Control decides:
441+
Then when PRs come back, OWNER decides:
442442

443443
```text
444444
Merge
@@ -479,6 +479,25 @@ Historical examples:
479479

480480
Owner map:
481481

482+
Team OWNER owns owner-governance routing and explicit override decisions:
483+
- Product roadmap
484+
- Tool priority
485+
- Architecture
486+
- Naming governance
487+
- Workflow governance
488+
- Environment strategy
489+
- PR sequencing
490+
- Stable and merge approval recommendations
491+
- Team lock override decisions
492+
493+
Team OWNER follows the same safety rules:
494+
- One active OWNER branch at a time.
495+
- One active OWNER assignment at a time.
496+
- OWNER may override team locks, but may not silently delete, rewrite, or remove protected instructions.
497+
- OWNER override must be explicitly documented.
498+
- Required override wording:
499+
- `OWNER override approved: <reason>`
500+
482501
Team Alpha owns Creator Journey work:
483502
- Game Journey
484503
- Game Hub
@@ -519,18 +538,26 @@ Team Gamma owns governance, recovery, diagnostics, and instruction-hardening wor
519538
- Static docs governance
520539

521540
Governance, recovery, diagnostics, and instruction-hardening PRs:
522-
- Use the TEAM token assigned by Master Control.
541+
- Use the TEAM token assigned by OWNER.
542+
- OWNER governance PRs use the OWNER token going forward.
523543
- Must not implement tool/runtime work from the opposite owner.
524544
- Must document TEAM ownership compliance in the PR report.
525545
- Recovery reports must include TEAM ownership.
526546

527547
Stable and merge approval:
528548
- Stable promotion and merge approval are controlled by the assigned Team Alpha, Team Beta, or Team Gamma owner.
529-
- Master Control may recommend sequencing, but Codex must not merge or mark stable without explicit owner approval for the affected workstream.
549+
- OWNER may recommend sequencing, but Codex must not merge or mark stable without explicit owner approval for the affected workstream.
550+
551+
OWNER naming rule:
552+
- Use Team OWNER for owner-governance work going forward.
553+
- Use OWNER override wording going forward.
554+
- Required override wording is:
555+
- `OWNER override approved: <reason>`
556+
- Existing uploaded PRs, branches, reports, and ZIPs with the prior governance token are historical and are not renamed by this rule unless separately scoped.
530557

531558
Hard stop rules:
532559
- If the PR name does not include a TEAM token, stop before changes.
533-
- If the TEAM token does not match the owner map or explicit Master Control assignment, stop before changes.
560+
- If the TEAM token does not match the owner map or explicit OWNER assignment, stop before changes.
534561
- If the PR scope belongs to another TEAM owner, stop before changes.
535-
- If the PR crosses multiple team ownership areas, stop and require Master Control to split or assign the work.
562+
- If the PR crosses multiple team ownership areas, stop and require OWNER to split or assign the work.
536563
- If the requested implementation path conflicts with the active owner path, stop before changes.
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
# PR_26172_OWNER_010 Instruction Compliance Checklist
2+
3+
- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
4+
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
5+
- PASS: Owner decision explicitly approved OWNER as the governance lane token going forward.
6+
- PASS: Branch name mirrors PR ownership: `pr/26172-OWNER-010-owner-governance-rename`.
7+
- PASS: Implementation path is active documentation only.
8+
- PASS: Existing uploaded MASTER-named ZIPs were not renamed.
9+
- PASS: Existing Project Instructions were updated only for the requested OWNER governance rename.
10+
- PASS: No runtime code changed.
11+
- PASS: No direct commit to `main`.
12+
- PASS: Required review artifacts are produced.
13+
- PASS: Required ZIP path is `tmp/PR_26172_OWNER_010-owner-governance-rename_delta.zip`.
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# PR_26172_OWNER_010 Manual Validation Notes
2+
3+
- Confirmed branch was created from latest `main`.
4+
- Confirmed existing uploaded MASTER-named ZIPs were not renamed.
5+
- Confirmed active docs now define Team OWNER for owner-governance work going forward.
6+
- Confirmed required override wording is `OWNER override approved: <reason>`.
7+
- Confirmed OWNER safety rules are documented.
8+
- Confirmed no runtime/UI files changed.
9+
- Did not run Playwright because this is documentation-only.
10+
- Did not run samples.
Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
# PR_26172_OWNER_010-owner-governance-rename
2+
3+
## Scope
4+
5+
Rename the owner-governance lane from the previous governance token to OWNER in active documentation.
6+
7+
## Changes
8+
9+
- Updated `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
10+
- Updated `docs_build/dev/PROJECT_MULTI_PC.txt`.
11+
- Replaced active `Master Control` wording with OWNER wording.
12+
- Added OWNER governance lane rules.
13+
- Added OWNER PR and branch naming rules going forward.
14+
- Added required override wording: `OWNER override approved: <reason>`.
15+
- Added OWNER safety rules for active branch, active assignment, protected instructions, and explicit documentation.
16+
- Preserved existing uploaded PRs, branches, reports, and ZIPs with prior names as historical artifacts.
17+
18+
## Validation
19+
20+
- `git diff --check`
21+
- Text check confirmed no active `Master Control`, `Team MASTER`, or `MASTER override` wording remains in active instruction files.
22+
- Text check confirmed `Team OWNER`, `OWNER override approved`, and OWNER safety rules exist.
23+
- Playwright skipped: documentation-only.
24+
- Samples skipped: not requested.
25+
26+
## ZIP
27+
28+
`tmp/PR_26172_OWNER_010-owner-governance-rename_delta.zip`
Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
1-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
2-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
3-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
1+
docs_build/dev/PROJECT_INSTRUCTIONS.md
2+
docs_build/dev/PROJECT_MULTI_PC.txt
43
docs_build/dev/reports/codex_changed_files.txt
54
docs_build/dev/reports/codex_review.diff
6-
src/dev-runtime/persistence/mock-db-store.js
5+
docs_build/dev/reports/PR_26172_OWNER_010-owner-governance-rename.md
6+
docs_build/dev/reports/PR_26172_OWNER_010-owner-governance-rename-instruction-compliance-checklist.md
7+
docs_build/dev/reports/PR_26172_OWNER_010-owner-governance-rename-manual-validation-notes.md

0 commit comments

Comments
 (0)