Skip to content

Commit 8dff4d4

Browse files
committed
PR_26171_ALPHA_075 team based PR naming
1 parent b31b319 commit 8dff4d4

7 files changed

Lines changed: 186 additions & 43 deletions

docs_build/dev/PROJECT_INSTRUCTIONS.md

Lines changed: 25 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -11,22 +11,38 @@ PLAN_PR → BUILD_PR → APPLY_PR
1111

1212
PR names MUST follow:
1313

14-
`PR_<YYJJJ>_<###>-<short-description>`
14+
`PR_<YYJJJ>_<TEAM>_<###>-<short-description>`
1515

1616
Where:
1717
- `YY` = year (2 digit)
18-
- `JJJ` = Julian day (001–365)
18+
- `JJJ` = Julian day (001-365)
19+
- `TEAM` = required team ownership token from `docs_build/dev/PROJECT_MULTI_PC.txt`
1920
- `###` = sequence for the day (001+)
2021

2122
Example:
22-
- `PR_26124_001-palette-baseline`
23-
- `PR_26124_002-tool-fix-asset-manager`
23+
- `PR_26171_ALPHA_065-message-studio-parent-child-table-foundation`
24+
- `PR_26171_BETA_069-message-tts-profile-contract-alignment`
25+
- `PR_26171_GAMMA_071-main-merge-conflict-recovery`
26+
27+
Branch names MUST mirror PR ownership:
28+
29+
`pr/<YYJJJ>-<TEAM>-<###>-<short-description>`
30+
31+
Branch examples:
32+
- `pr/26171-ALPHA-065-message-studio-parent-child-table-foundation`
33+
- `pr/26171-BETA-069-message-tts-profile-contract-alignment`
34+
- `pr/26171-GAMMA-071-main-merge-conflict-recovery`
2435

2536
Rules:
2637
- Must be unique per day
2738
- Must be sortable
39+
- `TEAM` is required
40+
- `TEAM` ownership comes from `docs_build/dev/PROJECT_MULTI_PC.txt`
41+
- Team ownership is independent of machine, workspace, laptop, desktop, or environment
2842
- Description must be short and hyphenated
2943
- Do NOT reuse old `PR_11_*` format for new PRs
44+
- Existing PC/LAPTOP, desktop/laptop, workspace, environment, or machine-parity examples are historical only
45+
- Future PR reports, recovery reports, validation reports, and manual validation notes must include TEAM ownership
3046

3147
## CHATGPT EXECUTION ROLE
3248

@@ -817,7 +833,7 @@ If the user says `NEXT`:
817833

818834
Use the current naming standard:
819835

820-
`PR_<YYJJJ>_<###>-<short-description>`
836+
`PR_<YYJJJ>_<TEAM>_<###>-<short-description>`
821837

822838
Do NOT continue old `PR_11_*` naming for new work.
823839

@@ -2027,18 +2043,18 @@ Required instruction reads:
20272043
- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
20282044
- Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
20292045
- Treat the newest applicable section in `PROJECT_INSTRUCTIONS.md` as authoritative when rules overlap.
2030-
- Treat the current owner/parity section in `PROJECT_MULTI_PC.txt` as authoritative for Team Alpha / Team Beta routing.
2046+
- Treat the current team ownership section in `PROJECT_MULTI_PC.txt` as authoritative for TEAM routing.
20312047

20322048
Required pre-change report:
20332049
- Codex must report instruction compliance as `PASS` or `FAIL` before making file changes.
2034-
- The report must include branch, clean status, PR owner, PR parity, implementation path, validation scope, required report list, and ZIP requirement.
2050+
- The report must include branch, clean status, PR TEAM owner, implementation path, validation scope, required report list, and ZIP requirement.
20352051
- Any `FAIL` is a hard stop unless the PR explicitly scopes branch audit or recovery documentation without implementation.
20362052

20372053
Hard stops before changes:
20382054
- If the current branch is not `main`, HARD STOP.
20392055
- If the repository is not clean before the PR branch is created, HARD STOP.
2040-
- If the PR owner does not match the Team Alpha / Team Beta ownership map in `PROJECT_MULTI_PC.txt`, HARD STOP.
2041-
- If the PR number parity does not match the assigned machine in `PROJECT_MULTI_PC.txt`, HARD STOP.
2056+
- If the PR name does not include a required TEAM token, HARD STOP.
2057+
- If the PR TEAM owner does not match the team ownership map in `PROJECT_MULTI_PC.txt`, HARD STOP.
20422058
- If the PR asks for implementation and the implementation path is wrong, HARD STOP.
20432059
- If a PR asks for functional parity and only placeholder-only work is possible, HARD STOP and report the missing source or blocker.
20442060
- If scoped validation is skipped without a documented reason, HARD STOP.

docs_build/dev/PROJECT_MULTI_PC.txt

Lines changed: 41 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ Requirements
2222
Acceptance Criteria
2323
Dependencies
2424
Priority
25-
Owner (Team Alpha / Team Beta)
25+
Owner (Team Alpha / Team Beta / Team Gamma)
2626
Recommended Workstream Split
2727

2828
Instead of arbitrary splits, split by Creator journey.
@@ -451,23 +451,35 @@ That is probably the fastest path to doubling throughput without creating chaos.
451451

452452
----------------------------------------------------------------------------------------
453453

454-
Current Authoritative Multi-PC Gate
454+
Current Authoritative Team Ownership Gate
455455

456456
Codex must read this file before every PR execution.
457457

458-
Machine parity:
458+
PR naming:
459459

460-
Team Alpha / Environment 1:
461-
- Uses even-numbered PR sequence values.
462-
- Example: `PR_26171_064-*`.
460+
- PR names must include the owning TEAM token:
461+
- `PR_<YYJJJ>_<TEAM>_<###>-<description>`
462+
- Branch names must mirror PR ownership:
463+
- `pr/<YYJJJ>-<TEAM>-<###>-<description>`
464+
- `TEAM` is required.
465+
- Team ownership is independent of machine, workspace, laptop, desktop, or environment.
466+
- Do not infer PR ownership from PR number parity or current machine.
463467

464-
Team Beta / Environment 2:
465-
- Uses odd-numbered PR sequence values.
466-
- Example: `PR_26171_063-*`.
468+
Current examples:
469+
- `PR_26171_ALPHA_065-message-studio-parent-child-table-foundation`
470+
- `PR_26171_BETA_069-message-tts-profile-contract-alignment`
471+
- `PR_26171_GAMMA_071-main-merge-conflict-recovery`
472+
- `pr/26171-ALPHA-065-message-studio-parent-child-table-foundation`
473+
- `pr/26171-BETA-069-message-tts-profile-contract-alignment`
474+
- `pr/26171-GAMMA-071-main-merge-conflict-recovery`
475+
476+
Historical examples:
477+
- Older PC/LAPTOP, desktop/laptop, workspace, environment, and machine-parity examples are historical only.
478+
- Older parity-only examples such as `PR_26171_064-*` and `PR_26171_063-*` are historical only.
467479

468480
Owner map:
469481

470-
Team Alpha / Environment 1 owns Creator Journey work:
482+
Team Alpha owns Creator Journey work:
471483
- Game Journey
472484
- Game Hub
473485
- Idea
@@ -483,7 +495,7 @@ Team Alpha / Environment 1 owns Creator Journey work:
483495
- Game Design
484496
- Game Crew
485497

486-
Team Beta / Environment 2 owns Content Creation and asset/publishing work:
498+
Team Beta owns Content Creation and asset/publishing work:
487499
- Graphics
488500
- Toolbox images
489501
- Audio
@@ -496,17 +508,29 @@ Team Beta / Environment 2 owns Content Creation and asset/publishing work:
496508
- Community
497509
- Arcade
498510

511+
Team Gamma owns governance, recovery, diagnostics, and instruction-hardening work:
512+
- PR naming governance
513+
- Git workflow governance
514+
- Recovery reports
515+
- Workspace recovery
516+
- Main merge conflict recovery
517+
- Diagnostics
518+
- Instruction enforcement
519+
- Static docs governance
520+
499521
Governance, recovery, diagnostics, and instruction-hardening PRs:
500-
- Follow PR number parity unless Master Control explicitly assigns an owner.
522+
- Use the TEAM token assigned by Master Control.
501523
- Must not implement tool/runtime work from the opposite owner.
502-
- Must document owner/parity compliance in the PR report.
524+
- Must document TEAM ownership compliance in the PR report.
525+
- Recovery reports must include TEAM ownership.
503526

504527
Stable and merge approval:
505-
- Stable promotion and merge approval are controlled by the assigned Team Alpha or Team Beta owner.
528+
- Stable promotion and merge approval are controlled by the assigned Team Alpha, Team Beta, or Team Gamma owner.
506529
- Master Control may recommend sequencing, but Codex must not merge or mark stable without explicit owner approval for the affected workstream.
507530

508531
Hard stop rules:
509-
- If the PR number parity does not match the current machine, stop before changes.
510-
- If the PR scope belongs to the other machine owner, stop before changes.
511-
- If the PR crosses Team Alpha and Team Beta ownership, stop and require Master Control to split or assign the work.
532+
- 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.
534+
- 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.
512536
- If the requested implementation path conflicts with the active owner path, stop before changes.
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
# PR_26171_ALPHA_075-team-based-pr-naming Manual Validation Notes
2+
3+
Generated: 2026-06-20T21:34:35.616Z
4+
5+
## TEAM Ownership
6+
7+
- TEAM owner: ALPHA
8+
9+
## Manual Review
10+
11+
- Confirmed PROJECT_INSTRUCTIONS.md documents the required TEAM token in PR names.
12+
- Confirmed PROJECT_INSTRUCTIONS.md documents branch names mirroring PR ownership.
13+
- Confirmed PROJECT_MULTI_PC.txt makes team ownership independent of machine, workspace, laptop, desktop, and environment.
14+
- Confirmed older PC/LAPTOP and parity examples are marked historical only.
15+
- Confirmed recovery reports are required to include TEAM ownership.
16+
17+
No browser or runtime validation was required because this PR is docs/static only.
Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
# PR_26171_ALPHA_075-team-based-pr-naming Validation
2+
3+
Generated: 2026-06-20T21:34:35.616Z
4+
5+
## TEAM Ownership
6+
7+
- TEAM owner: ALPHA
8+
- Validation scope: docs/static validation only
9+
10+
## Commands
11+
12+
```text
13+
git diff --check
14+
```
15+
16+
## Results
17+
18+
- Initial docs diff check: PASS
19+
20+
Output:
21+
22+
```text
23+
warning: in the working copy of 'docs_build/dev/reports/codex_review.diff', LF will be replaced by CRLF the next time Git touches it
24+
```
25+
26+
## Final Validation
27+
28+
- Final full diff check after report generation: PASS
29+
30+
Output:
31+
32+
```text
33+
(no output)
34+
```
Lines changed: 62 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,62 @@
1+
# PR_26171_ALPHA_075-team-based-pr-naming
2+
3+
Generated: 2026-06-20T21:34:35.616Z
4+
5+
## Purpose
6+
7+
Replace machine-based PR naming examples with team-based ownership naming.
8+
9+
## TEAM Ownership
10+
11+
- TEAM owner: ALPHA
12+
- Ownership source: docs_build/dev/PROJECT_MULTI_PC.txt
13+
- Scope classification: static docs governance
14+
- Machine/workspace/environment ownership: not used for PR naming or branch naming
15+
16+
## Scope
17+
18+
Updated only:
19+
20+
- docs_build/dev/PROJECT_INSTRUCTIONS.md
21+
- docs_build/dev/PROJECT_MULTI_PC.txt
22+
23+
## Naming Contract
24+
25+
Authoritative PR format:
26+
27+
`PR_<YYJJJ>_<TEAM>_<###>-<description>`
28+
29+
Authoritative branch format:
30+
31+
`pr/<YYJJJ>-<TEAM>-<###>-<description>`
32+
33+
Required examples are documented for ALPHA, BETA, and GAMMA.
34+
35+
## Instruction Compliance
36+
37+
- Read PROJECT_INSTRUCTIONS.md first: PASS
38+
- Read PROJECT_MULTI_PC.txt before edits: PASS
39+
- Started from main: PASS
40+
- Pulled latest main before branch: PASS
41+
- Clean repo before branch: PASS
42+
- Created branch: pr/26171-ALPHA-075-team-based-pr-naming
43+
- TEAM token required: PASS
44+
- TEAM ownership comes from PROJECT_MULTI_PC.txt: PASS
45+
- Future reports include TEAM ownership: PASS
46+
47+
## Git Workflow Status
48+
49+
- Starting HEAD: b31b319983dc62a240ab7095b8430b7c6ed182bd
50+
- origin/main at start: b31b319983dc62a240ab7095b8430b7c6ed182bd
51+
- origin/main...HEAD after branch creation: 0 0
52+
- Push result: pending at report generation
53+
- PR URL: pending at report generation
54+
- Merge result: pending at report generation
55+
- Final main commit: pending at report generation
56+
57+
## Changed Files Before Report Generation
58+
59+
```text
60+
docs_build/dev/PROJECT_INSTRUCTIONS.md
61+
docs_build/dev/PROJECT_MULTI_PC.txt
62+
```
Lines changed: 7 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,7 @@
1-
A docs_build/dev/reports/PR_26171_069-manual-validation-notes.md
2-
A docs_build/dev/reports/PR_26171_069-message-tts-ownership-checklist.md
3-
A docs_build/dev/reports/PR_26171_069-message-tts-profile-contract-alignment.md
4-
A docs_build/dev/reports/PR_26171_069-parent-child-table-checklist.md
5-
A docs_build/dev/reports/PR_26171_069-validation.md
6-
A docs_build/pr/PR_26171_069-message-tts-profile-contract-alignment/APPLY_PR.md
7-
A docs_build/pr/PR_26171_069-message-tts-profile-contract-alignment/BUILD_PR.md
8-
A docs_build/pr/PR_26171_069-message-tts-profile-contract-alignment/PLAN_PR.md
9-
M tests/playwright/tools/MessagesTool.spec.mjs
10-
M tests/playwright/tools/TextToSpeechFunctional.spec.mjs
11-
M tests/tools/Text2SpeechShell.test.mjs
12-
M toolbox/messages/index.html
13-
M toolbox/messages/message-tts-service-registry.js
14-
M toolbox/messages/messages.js
15-
M toolbox/text-to-speech/text2speech.js
16-
M docs_build/dev/reports/codex_review.diff
17-
M docs_build/dev/reports/codex_changed_files.txt
1+
docs_build/dev/PROJECT_INSTRUCTIONS.md
2+
docs_build/dev/PROJECT_MULTI_PC.txt
3+
docs_build/dev/reports/PR_26171_ALPHA_075-team-based-pr-naming-manual-validation-notes.md
4+
docs_build/dev/reports/PR_26171_ALPHA_075-team-based-pr-naming-validation.md
5+
docs_build/dev/reports/PR_26171_ALPHA_075-team-based-pr-naming.md
6+
docs_build/dev/reports/codex_changed_files.txt
7+
docs_build/dev/reports/codex_review.diff
-65.1 KB
Binary file not shown.

0 commit comments

Comments
 (0)