From b499f7f2a89a30eadc2be149336d91a5b2044564 Mon Sep 17 00:00:00 2001 From: Charlie Team <97194984+ToolboxAid@users.noreply.github.com> Date: Fri, 26 Jun 2026 21:42:15 -0400 Subject: [PATCH 1/2] Canonicalize Project Instructions backlog governance --- .../PROJECT_INSTRUCTIONS.md | 16 +- docs_build/dev/ProjectInstructions/README.txt | 2 +- .../TEAM_START_COMMANDS.md | 9 +- .../addendums/branch_context_governance.md | 4 + .../addendums/branch_lock_governance.md | 18 +- .../codex_artifact_and_reporting_standard.md | 4 +- .../codex_project_instructions_startup.md | 18 +- .../environment_configuration_standards.md | 10 +- .../addendums/multi_team.md | 8 +- .../addendums/pr_workflow.md | 26 +- .../team_backlog_sod_eod_standard.md | 5 +- .../addendums/tool_mvp_stacked_pr_standard.md | 41 +- .../backlog/BACKLOG_MASTER.md | 19 +- .../team_assignments/TEAM_ASSIGNMENTS.md | 27 +- .../team_assignments/team_ownership.md | 15 +- ...ctions-cleanup-backlog-canonicalization.md | 62 ++ ...klog-canonicalization_branch-validation.md | 23 + ...anonicalization_manual-validation-notes.md | 24 + ...canonicalization_requirements-checklist.md | 33 + ...acklog-canonicalization_validation-lane.md | 51 ++ .../dev/reports/codex_changed_files.txt | 22 +- docs_build/dev/reports/codex_review.diff | 818 +++++++++++++----- 22 files changed, 922 insertions(+), 333 deletions(-) create mode 100644 docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md create mode 100644 docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md create mode 100644 docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md create mode 100644 docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md create mode 100644 docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md diff --git a/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md b/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md index d6617c986..ffd6f2438 100644 --- a/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md +++ b/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md @@ -58,6 +58,20 @@ Existing Project Instructions outside `docs_build/dev/ProjectInstructions/` rema `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. +## Canonical Governance Owners + +When active guidance overlaps, use these canonical owner documents: + +- Workflow and Product Owner testable completion: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` +- 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` +- Page-level Playwright organization and completion coverage: `docs_build/dev/ProjectInstructions/addendums/test_structure_standardization.md` +- API/environment model and `Browser -> API -> Database` rule: `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md` +- Environment variable, URL, R2 prefix, and feature flag configuration: `docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md` +- Team backlog fields, completion percentages, and next logical PR ownership: `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` +- Team ownership and assignment routing: `docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md` + +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. + ## Environment Governance `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: `OWNER override approved: ` OWNER follows the same safety rules: -- One active Team OWNER branch at a time. +- Team OWNER follows the same one-active-branch discipline as every team. - One active OWNER assignment at a time. - OWNER may override team locks, but may not silently delete, rewrite, or remove protected instructions. - OWNER override must be explicitly documented. diff --git a/docs_build/dev/ProjectInstructions/README.txt b/docs_build/dev/ProjectInstructions/README.txt index 1b647c6fd..bbfff9812 100644 --- a/docs_build/dev/ProjectInstructions/README.txt +++ b/docs_build/dev/ProjectInstructions/README.txt @@ -16,7 +16,7 @@ No direct commits to main: Do not commit directly to main. Normal work must use PR branches, draft PRs, validation evidence, and owner-controlled merge approval. Branch lifecycle: -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`. +The canonical START / WORK / END lifecycle is `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. OWNER override rule: An OWNER override must use this wording: diff --git a/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md b/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md index 5fe1c43e4..7f9bcad29 100644 --- a/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md +++ b/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md @@ -13,14 +13,7 @@ Use `docs_build/dev/ProjectInstructions/` as the only active Project Instruction Read `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` before implementation. Branch Lifecycle (Canonical): -- Every PR follows exactly three phases: START, WORK, END. -- Follow `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. -- START begins on synchronized `main` and creates the PR branch only after all gates pass. -- WORK remains on the PR branch. Never checkout `main`. -- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. -- No commits on `main`. -- No implementation on `main`. -- No validation on `main` except start validation. +- 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. ## Start Team Alfa diff --git a/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md b/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md index 798bb2cba..a829b0a15 100644 --- a/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md +++ b/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md @@ -7,6 +7,10 @@ Owner: OWNER Require Codex and teams to confirm branch context before changing files. +Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. + +This file owns branch context confirmation and stop conditions; it must not create a competing START / WORK / END lifecycle rule. + ## Session Start Context At the start of work, report or validate: diff --git a/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md b/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md index 6f797306d..1851906ab 100644 --- a/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md +++ b/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md @@ -7,6 +7,10 @@ Owner: OWNER Keep active work attached to the correct assigned team, branch, and OWNER decision. +Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. + +This file owns branch lock enforcement and OWNER override handling; it must not create a competing START / WORK / END lifecycle rule. + ## Active Work Lock - 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 ## Branch Lifecycle (Canonical) -Every PR follows exactly three phases: +The canonical branch lifecycle lives in: ```text -START -WORK -END +docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md ``` -The canonical lifecycle lives in `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. - -Branch lock governance enforces: -- START on synchronized `main`. -- WORK only on the PR branch. -- END by merging, returning to synchronized `main`, publishing branch, HEAD SHA, and date/time, then stopping all work. -- 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. +Branch lock governance enforces ownership and branch-lock compliance with that canonical lifecycle. ## OWNER Override diff --git a/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md b/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md index 17be32ada..a382a65df 100644 --- a/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md +++ b/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md @@ -55,7 +55,7 @@ Codex responses must include: Every tool MVP PR report must include: - Product Owner testable outcome - What Playwright tests -- What Mr. Q should manually test +- What the Product Owner should manually test - Whether the PR is part of a stacked MVP sequence - Previous PR dependency - Next PR dependency @@ -63,7 +63,7 @@ Every tool MVP PR report must include: The report must answer: ```text -What can Mr. Q test after applying this ZIP? +What can the Product Owner test after applying this ZIP? ``` If a tool MVP PR has no Playwright lane, the report must state why and list the manual Product Owner validation instead. diff --git a/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md b/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md index c6d8f192c..daf35f1d3 100644 --- a/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md +++ b/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md @@ -23,25 +23,13 @@ Codex must use this as the only active source of truth for: ## Branch Lifecycle Start Gate -Every PR follows exactly three phases: +Codex must follow the canonical lifecycle in: ```text -START -WORK -END +docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md ``` -Codex must follow the canonical lifecycle in: - -`docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md` - -Startup enforcement: -- START begins on synchronized `main`. -- WORK remains on the PR branch. Never checkout `main`. -- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. -- STOP if current branch is `main` before commit. -- STOP if attempting to push `main`. -- STOP if a new PR starts before returning to synchronized `main`. +This startup addendum only requires Codex to read and apply that canonical lifecycle; it must not define a competing lifecycle rule. Deprecated Project Instructions material outside `docs_build/dev/ProjectInstructions/` is reference-only and must not override active governance. diff --git a/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md b/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md index 5a91f904c..414bf9758 100644 --- a/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md +++ b/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md @@ -11,6 +11,10 @@ This addendum is governance/documentation only. It does not change runtime behav ## Source Model +Canonical environment/API model reference: `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md`. + +This file owns environment variable names, URL configuration, R2 prefix configuration, and feature flag configuration only. + This standard builds on: ```text @@ -88,12 +92,10 @@ R2 project, backup, export, import, or future storage paths must stay under the ## API/Service Contract Configuration -One shared API/service contract is required across Local (VS Code), DEV, IST, UAT, and PROD. +The canonical shared API/service contract rule lives in `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md`. -Rules: +This section owns configuration-only API URL requirements: -- Browser/UI/runtime code must follow `Browser -> API -> Database` for authoritative product data. -- `Local API` means the same shared API/service contract running locally, not a separate local-only API implementation. - API URLs may differ by `.env` only. - Do not split Local API and Public API contracts. - Do not create environment-specific API/service contracts. diff --git a/docs_build/dev/ProjectInstructions/addendums/multi_team.md b/docs_build/dev/ProjectInstructions/addendums/multi_team.md index 7ba2cae8d..763dccc8a 100644 --- a/docs_build/dev/ProjectInstructions/addendums/multi_team.md +++ b/docs_build/dev/ProjectInstructions/addendums/multi_team.md @@ -1,5 +1,11 @@ # Multi-Team Codex Execution Governance +Canonical workflow reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md`. + +Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. + +This file owns multi-team execution coordination and must not create competing workflow or branch lifecycle rules. + ## Four Active Delivery Teams The single authoritative four-team ownership definition is: @@ -65,7 +71,7 @@ Rules: ## Day Work / EOD Merge Rule During active work: -- Work happens on assigned team branches, OWNER branches, or scoped PR branches. +- Work happens on active non-main team branches or scoped PR branches. - Commits are allowed only on assigned non-main branches. - Pushes are allowed and expected. - Draft PRs are allowed and expected. diff --git a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md index bee8bdd75..97a73ca30 100644 --- a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md +++ b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md @@ -46,31 +46,11 @@ This section supersedes older active wording that implies returning to `main` be ## Branch Lifecycle (Canonical) -Every PR follows exactly three phases: - -```text -START -WORK -END -``` - The canonical START, WORK, END, Daily Synchronization, and Mandatory Hard Stops rules live in: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md` -PR workflow must follow that lifecycle exactly. - -Summary: -- START happens on synchronized `main` only and creates the PR branch only after all required gates pass. -- WORK happens only on the PR branch. -- END validates, commits, pushes, opens/updates the PR, merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. -- No commits on `main`. -- No implementation on `main`. -- No validation on `main` except start validation. -- Never checkout `main` during WORK. -- STOP before commit if current branch is `main`. -- STOP if current branch changes unexpectedly. -- STOP if attempting to push `main`. +PR workflow must follow that lifecycle exactly and must not create a competing lifecycle rule. ## PR Lifecycle States @@ -153,7 +133,7 @@ Do not stop after every small PR unless blocked by branch state, failed validati Each tool MVP PR plan or template must include: - Product Owner testable outcome - What Playwright tests -- What Mr. Q should manually test +- What the Product Owner should manually test - Whether the PR is part of a stacked MVP sequence - Previous PR dependency - Next PR dependency @@ -162,6 +142,8 @@ Visible acceptance must be Creator-facing first. Architecture can be handled und ## Product Owner Testable Definition +Canonical owner: this section is the active canonical Product Owner testable completion rule. + 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. A Product Owner testable outcome means the Product Owner can: diff --git a/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md b/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md index be3555469..e6d6c7ce8 100644 --- a/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md +++ b/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md @@ -42,14 +42,15 @@ Each backlog item must track: - current completion percentage - remaining work - blocking dependencies +- owning team Completion percentages are updated: - at SOD -- after each accepted PR +- after every accepted PR - at EOD -The backlog is the authoritative source for determining the next PRs. +The backlog is the authoritative source for determining the next logical PRs. If the backlog and a generated report conflict, the backlog wins unless OWNER explicitly approves a newer governance decision. diff --git a/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md b/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md index 2bcc52a60..5da2f9952 100644 --- a/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md +++ b/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md @@ -24,7 +24,7 @@ One large Codex command -> multiple focused stacked PRs -> each PR has a Product - Each PR must answer: ```text -What can Mr. Q test after applying this ZIP? +What can the Product Owner test after applying this ZIP? ``` - Codex must continue through the stacked PR sequence unless blocked by: @@ -40,45 +40,14 @@ For tool MVP PR planning, visible acceptance must be Creator-facing first. Architecture can be handled under the covers, but PR purpose must be user-testable. -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. - -A Product Owner testable outcome means the Product Owner can: - -- open the page/tool -- perform the primary workflow -- save/load data where applicable -- observe expected results -- validate success and failure states -- follow manual validation steps from the PR report - -Not acceptable as complete/testable: - -- shell-only page -- route-only page -- navigation-only PR -- template-only page -- placeholder controls -- static table with no workflow -- `Not implemented yet` -- `coming soon` -- placeholder-only workspace, inspector, or output sections -- planned-only tile -- route that loads but cannot be used - -Required for Product Owner testable completion: - -- visible working controls -- API-backed data where required -- validation and error states -- empty states -- save/load behavior where applicable -- manual validation steps for Product Owner -- targeted Playwright coverage where impacted. +Canonical reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` owns the Product Owner testable definition and no-shell completion rule. + +Canonical Playwright reference: `docs_build/dev/ProjectInstructions/addendums/test_structure_standardization.md` owns page-level Playwright organization and minimum completion coverage. Each tool MVP PR must state: - Product Owner testable outcome - What Playwright tests -- What Mr. Q should manually test +- What the Product Owner should manually test - Whether the PR is part of a stacked MVP sequence - Previous PR dependency - Next PR dependency diff --git a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md index cc9ddc88c..c0e3336a9 100644 --- a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md +++ b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md @@ -2,19 +2,22 @@ ## Backlog Item Tracking Standard +Canonical backlog tracking owner: `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md`. + Every active team owns an active backlog when it has assigned work. -Each backlog item must track: +Backlog entries in this file follow the canonical field list, including: - Name - Description - Current completion percentage - Remaining work - Blocking dependencies +- Owning team -Completion percentages are updated at SOD, after each accepted PR, and at EOD. +Completion percentages are updated at SOD, after every accepted PR, and at EOD. -The backlog is the authoritative source for determining the next PRs. +The backlog is the authoritative source for determining the next logical PRs. ## Game Journey MVP @@ -44,7 +47,7 @@ The backlog is the authoritative source for determining the next PRs. - [ ] Bravo - Asset Studio V2 - [ ] Charlie - Sprites canvas editor MVP - Sprites is a creator tool, not only an asset metadata library. - - MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only reusable colors, color selection from Palette/Colors, pixel painting, save/load sprite grid data through the API/database, and Product Owner manual validation. + - MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. - Category is removed from Sprites MVP planning. - [ ] Bravo - Animation Studio V2 - [ ] Bravo - Palette Manager @@ -65,7 +68,8 @@ The backlog is the authoritative source for determining the next PRs. 0% Complete — Create the things players interact with -- [ ] Alfa - Objects +- [ ] Charlie - Objects + - Objects includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. - [ ] Alfa - Characters - [ ] Alfa - Object Behaviors @@ -226,6 +230,7 @@ Current OWNER clarification: ### Team Charlie +- Active ownership: Runtime, System Health, Environment Management, Sprites, Objects. - [ ] Charlie - Guardrail hardening - [ ] Charlie - Browser validation hardening - [ ] Charlie - Remaining test relocation audit @@ -257,6 +262,10 @@ Current OWNER clarification: - observability integrations - [ ] Charlie - Infrastructure dashboard - [ ] Charlie - Environment validation +- [ ] Charlie - Sprites canvas editor MVP + - Scope: canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. +- [ ] Charlie - Objects + - Scope: object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. ### Team Delta diff --git a/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md b/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md index 4253971d9..e7826a808 100644 --- a/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md +++ b/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md @@ -1,5 +1,11 @@ # TEAM_ASSIGNMENTS +Canonical workflow reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md`. + +Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. + +This file records active assignments and registry state. Workflow summaries here are subordinate to the canonical workflow and branch lifecycle documents. + # Active Team Registry | Team | Assignment | Branch | PR | Status | @@ -197,7 +203,7 @@ Conflict note: ## Day Work / EOD Merge Rule During active work: -- Work happens on assigned team branches, OWNER branches, or scoped PR branches. +- Work happens on active non-main team branches or scoped PR branches. - Commits are allowed only on assigned non-main branches. - Pushes are allowed and expected. - Draft PRs are allowed and expected. @@ -263,23 +269,10 @@ Teams must use only `docs_build/dev/ProjectInstructions/` as the active Project ## Branch Lifecycle (Canonical) -Every PR follows exactly three phases: +Teams must follow the canonical branch lifecycle in: ```text -START -WORK -END +docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md ``` -Teams must follow `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. - -Assignment governance enforces: -- START begins on synchronized `main`. -- WORK remains on the PR branch. Never checkout `main`. -- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. -- No commits on `main`. -- No implementation on `main`. -- No validation on `main` except start validation. -- STOP if current branch is `main` before commit. -- STOP if attempting to push `main`. -- STOP if new PR work starts before returning to synchronized `main`. +Assignment governance records active team and branch state; it must not create a competing lifecycle rule. diff --git a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md index 30b4f2dfe..fb8497c1f 100644 --- a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md +++ b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md @@ -30,11 +30,13 @@ This section is the current OWNER-approved active ownership alignment. - Repository compliance - Validation - Infrastructure +- Runtime - Storage -- Environment management +- Environment Management - System Health - Operations - Sprites canvas editor MVP +- Objects Team Charlie System Health owns: - Environment Summary @@ -45,9 +47,18 @@ Team Charlie System Health owns: Team Charlie Sprites ownership: - Sprites is a creator tool, not only an asset metadata library. -- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only reusable colors, color selection from Palette/Colors, pixel painting, save/load sprite grid data through the API/database, and Product Owner manual validation. +- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. - Sprites must not own reusable color definitions. +Team Charlie Objects ownership: +- Object library +- Object editor +- Sprite assignment +- Object properties +- Collision configuration +- Runtime object metadata +- Object API/database contracts + ## Team Delta - Engine diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md new file mode 100644 index 000000000..cbbefda41 --- /dev/null +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md @@ -0,0 +1,62 @@ +# PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + +Date: 2026-06-27 +Team: OWNER +Branch: PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + +## Purpose + +Finalize active Project Instructions cleanup, canonicalization, and backlog ownership governance. + +## Scope + +Documentation and governance only. + +No runtime code, UI code, API code, database code, `start_of_day` files, history snapshots, or unrelated cleanup were changed. + +## Changes + +- Added a canonical governance owner map to the Project Instructions root. +- Made overlap documents point to canonical owners instead of creating competing active rules: + - workflow and Product Owner testable completion: `pr_workflow.md` + - branch lifecycle and EOD main lock: `project_instructions_single_source_eod_lock.md` + - page-level Playwright: `test_structure_standardization.md` + - API/environment model: `environment_governance_model.md` + - environment configuration: `environment_configuration_standards.md` + - team backlog: `team_backlog_sod_eod_standard.md` + - team ownership: `team_ownership.md` +- Replaced active `Mr. Q` manual validation wording with Product Owner wording. +- Expanded backlog item requirements with owning team. +- Updated backlog percentage cadence to SOD, after every accepted PR, and EOD. +- Confirmed the backlog drives the next logical PRs. +- Updated active Team Charlie ownership for Runtime, System Health, Environment Management, Sprites, and Objects. +- Updated Sprites MVP scope to include canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. +- Added Objects scope for object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. + +## Validation + +- PASS: documentation/governance-only changed-file check. +- PASS: `git diff --check` +- PASS: no runtime files changed. +- PASS: no UI files changed. +- PASS: no API files changed. +- PASS: no database files changed. +- PASS: no `start_of_day` files changed. +- PASS: no active `Mr. Q` manual validation wording remains. +- PASS: no OWNER-only branch workflow wording remains. +- PASS: active `Alpha` references are limited to the preserved non-team cancellation phrase. +- PASS: canonical owner references exist. +- PASS: Product Owner testable canonical owner remains `pr_workflow.md`. +- PASS: page-level Playwright canonical owner remains `test_structure_standardization.md`. +- PASS: API/environment canonical owner remains `environment_governance_model.md`. +- PASS: branch lifecycle canonical owner remains `project_instructions_single_source_eod_lock.md`. +- PASS: Team Charlie owns both Sprites and Objects. +- PASS: repo-structured ZIP produced for documentation-only changes. + +## Artifact + +- `tmp/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_delta.zip` + +## Next Logical PRs + +No additional Project Instructions cleanup PR is required from this pass. Future cleanup should wait until after the OWNER governance stack is reviewed, unless the Product Owner identifies a new conflict. diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md new file mode 100644 index 000000000..995e565ba --- /dev/null +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md @@ -0,0 +1,23 @@ +# PR_26177_OWNER_012 Branch Validation + +Date: 2026-06-27 +Team: OWNER +Branch: PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + +## Result + +PASS + +## Checks + +- PASS: Work was performed on the active OWNER PR branch, not `main`. +- PASS: Branch is stacked on `PR_26177_OWNER_011-codex-zip-and-next-pr-standard`. +- PASS: Changes are limited to active Project Instructions and generated reports. +- PASS: No runtime, UI, API, database, or `start_of_day` files changed. +- PASS: `git diff --check` passed before report generation. +- PASS: Required report files were generated. +- PASS: Repo-structured ZIP artifact path is defined under `tmp/`. + +## Notes + +This PR remains in the OWNER workstream and must not be merged to `main` until EOD closeout or explicit OWNER approval. diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md new file mode 100644 index 000000000..2b111a71f --- /dev/null +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md @@ -0,0 +1,24 @@ +# PR_26177_OWNER_012 Manual Validation Notes + +Date: 2026-06-27 +Team: OWNER + +## Manual Review + +Review these files: + +- `docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md` +- `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` +- `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` +- `docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md` +- `docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md` + +Confirm the canonical governance owner map is clear and that overlap files point back to canonical owner documents. + +Confirm Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. + +Confirm active backlog wording uses Product Owner validation language and drives the next logical PRs. + +## Expected Reviewer Outcome + +The Product Owner should be able to identify the canonical owner document for workflow, branch rules, Product Owner testable completion, Playwright organization, API/environment rules, team backlog, and team ownership without reading competing active rules. diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md new file mode 100644 index 000000000..d9a12c54e --- /dev/null +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md @@ -0,0 +1,33 @@ +# PR_26177_OWNER_012 Requirements Checklist + +Date: 2026-06-27 +Team: OWNER + +## Requirement Results + +- PASS: Documentation/governance only. +- PASS: Active Project Instructions reviewed. +- PASS: Active instruction files updated only. +- PASS: Historical snapshots were not modified. +- PASS: No runtime changes. +- PASS: Canonical governance owner map added. +- PASS: Duplicated active guidance was removed where safe. +- PASS: Append-style overlap files now point to canonical owner documents. +- PASS: Military team spelling retained: OWNER, ALFA, BRAVO, CHARLIE, DELTA. +- PASS: No new Greek team spelling introduced. +- PASS: Active `Mr. Q` validation wording replaced with Product Owner wording. +- PASS: Backlog item fields include owning team. +- PASS: Backlog percentages update at SOD, after every accepted PR, and at EOD. +- PASS: Backlog is authoritative for determining the next logical PRs. +- PASS: Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. +- PASS: Sprites MVP scope includes canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. +- PASS: Objects scope includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. +- PASS: Existing SOD, active team branch, sequential PR, no-return-to-main, and EOD clean main sync workflow remains present. +- PASS: Required reports were generated. +- PASS: Repo-structured ZIP artifact will be generated under `tmp/`. + +## Restrictions + +- PASS: No commit was made to `main`. +- PASS: No `start_of_day` folders changed. +- PASS: No unrelated cleanup. diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md new file mode 100644 index 000000000..1f8990d29 --- /dev/null +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md @@ -0,0 +1,51 @@ +# PR_26177_OWNER_012 Validation Lane + +Date: 2026-06-27 +Team: OWNER + +## Commands + +```powershell +git diff --check +``` + +Result: PASS + +```powershell +node -e "documentation/governance-only changed-file check" +``` + +Result: PASS + +```powershell +rg -n "Mr\\. Q|What can Mr\\. Q|What Mr\\. Q" docs_build/dev/ProjectInstructions +``` + +Result: PASS, no active matches. + +```powershell +rg -n "Work must be committed only to the active OWNER branch|PR branches/commits stay on the active OWNER branch|active OWNER branch|OWNER branches|OWNER branch" docs_build/dev/ProjectInstructions +``` + +Result: PASS, no active OWNER-only branch workflow wording. + +```powershell +node -e "canonical owner, backlog, and Team Charlie ownership checks" +``` + +Result: PASS + +## Targeted Results + +- PASS: canonical governance owner map exists. +- PASS: Product Owner testable owner is canonicalized. +- PASS: page-level Playwright owner is canonicalized. +- PASS: API/environment owner is canonicalized. +- PASS: branch lifecycle owner is canonicalized. +- PASS: backlog includes owning team and next logical PR wording. +- PASS: Team Charlie owns Sprites and Objects. +- PASS: no runtime files changed. + +## Playwright + +Not impacted. This PR is documentation/governance only. diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt index 85d74e87a..4c7d9fba2 100644 --- a/docs_build/dev/reports/codex_changed_files.txt +++ b/docs_build/dev/reports/codex_changed_files.txt @@ -1,10 +1,22 @@ +docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md +docs_build/dev/ProjectInstructions/README.txt +docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md +docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md +docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md +docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md +docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md +docs_build/dev/ProjectInstructions/addendums/multi_team.md docs_build/dev/ProjectInstructions/addendums/pr_workflow.md docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md -docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md -docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md -docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md -docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md -docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md +docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md +docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md +docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md +docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md +docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md +docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md +docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md +docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md +docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md docs_build/dev/reports/codex_changed_files.txt docs_build/dev/reports/codex_review.diff diff --git a/docs_build/dev/reports/codex_review.diff b/docs_build/dev/reports/codex_review.diff index 0ea4e6438..17dfb5afa 100644 --- a/docs_build/dev/reports/codex_review.diff +++ b/docs_build/dev/reports/codex_review.diff @@ -1,154 +1,549 @@ +diff --git a/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md b/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md +index d6617c986..ffd6f2438 100644 +--- a/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md ++++ b/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md +@@ -58,6 +58,20 @@ Existing Project Instructions outside `docs_build/dev/ProjectInstructions/` rema + + `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. + ++## Canonical Governance Owners ++ ++When active guidance overlaps, use these canonical owner documents: ++ ++- Workflow and Product Owner testable completion: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` ++- 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` ++- Page-level Playwright organization and completion coverage: `docs_build/dev/ProjectInstructions/addendums/test_structure_standardization.md` ++- API/environment model and `Browser -> API -> Database` rule: `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md` ++- Environment variable, URL, R2 prefix, and feature flag configuration: `docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md` ++- Team backlog fields, completion percentages, and next logical PR ownership: `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` ++- Team ownership and assignment routing: `docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md` ++ ++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. ++ + ## Environment Governance + + `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: + `OWNER override approved: ` + + OWNER follows the same safety rules: +-- One active Team OWNER branch at a time. ++- Team OWNER follows the same one-active-branch discipline as every team. + - One active OWNER assignment at a time. + - OWNER may override team locks, but may not silently delete, rewrite, or remove protected instructions. + - OWNER override must be explicitly documented. +diff --git a/docs_build/dev/ProjectInstructions/README.txt b/docs_build/dev/ProjectInstructions/README.txt +index 1b647c6fd..bbfff9812 100644 +--- a/docs_build/dev/ProjectInstructions/README.txt ++++ b/docs_build/dev/ProjectInstructions/README.txt +@@ -16,7 +16,7 @@ No direct commits to main: + Do not commit directly to main. Normal work must use PR branches, draft PRs, validation evidence, and owner-controlled merge approval. + + Branch lifecycle: +-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`. ++The canonical START / WORK / END lifecycle is `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. + + OWNER override rule: + An OWNER override must use this wording: +diff --git a/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md b/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md +index 5fe1c43e4..7f9bcad29 100644 +--- a/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md ++++ b/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md +@@ -13,14 +13,7 @@ Use `docs_build/dev/ProjectInstructions/` as the only active Project Instruction + Read `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` before implementation. + + Branch Lifecycle (Canonical): +-- Every PR follows exactly three phases: START, WORK, END. +-- Follow `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. +-- START begins on synchronized `main` and creates the PR branch only after all gates pass. +-- WORK remains on the PR branch. Never checkout `main`. +-- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. +-- No commits on `main`. +-- No implementation on `main`. +-- No validation on `main` except start validation. ++- 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. + + ## Start Team Alfa + +diff --git a/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md b/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md +index 798bb2cba..a829b0a15 100644 +--- a/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md ++++ b/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md +@@ -7,6 +7,10 @@ Owner: OWNER + + Require Codex and teams to confirm branch context before changing files. + ++Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. ++ ++This file owns branch context confirmation and stop conditions; it must not create a competing START / WORK / END lifecycle rule. ++ + ## Session Start Context + + At the start of work, report or validate: +diff --git a/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md b/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md +index 6f797306d..1851906ab 100644 +--- a/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md ++++ b/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md +@@ -7,6 +7,10 @@ Owner: OWNER + + Keep active work attached to the correct assigned team, branch, and OWNER decision. + ++Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. ++ ++This file owns branch lock enforcement and OWNER override handling; it must not create a competing START / WORK / END lifecycle rule. ++ + ## Active Work Lock + + - 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 + + ## Branch Lifecycle (Canonical) + +-Every PR follows exactly three phases: ++The canonical branch lifecycle lives in: + + ```text +-START +-WORK +-END ++docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md + ``` + +-The canonical lifecycle lives in `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. +- +-Branch lock governance enforces: +-- START on synchronized `main`. +-- WORK only on the PR branch. +-- END by merging, returning to synchronized `main`, publishing branch, HEAD SHA, and date/time, then stopping all work. +-- 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. ++Branch lock governance enforces ownership and branch-lock compliance with that canonical lifecycle. + + ## OWNER Override + diff --git a/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md b/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md -index 1c3671363..17be32ada 100644 +index 17be32ada..a382a65df 100644 --- a/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md +++ b/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md -@@ -6,20 +6,31 @@ Standardize Codex deliverables, completion reporting, and artifact generation. - - ## ZIP Artifact Requirement - --Every Codex task must produce a ZIP artifact. -+Every Codex execution must produce a repo-structured ZIP artifact and the required reports. - --Applies to: --- Success --- Failure --- Stop Gate --- Partial Completion --- Review Deliverables --- Governance Deliverables -+This applies regardless of result: -+ -+- success -+- completion -+- partial completion -+- hard stop -+- blocked -+- validation failure -+- new information discovered -+- no files changed -+- review deliverables -+- governance deliverables -+ -+No exceptions. - - Minimum ZIP contents: --- summary.md -+ -+- changed or preserved repo files from the run, stored at repo-relative paths -+- required reports under `docs_build/dev/reports/` - - Optional: -+ -+- summary.md - - changed-files.txt - - findings.md - - validation.txt -@@ -28,11 +39,16 @@ Optional: - ## Completion Reporting - - Codex responses must include: -+ - - ZIP filename - - ZIP location -+- repo-structured ZIP path -+- reports -+- changed file list - - PR number(s) - - Merge commit(s) - - Validation results -+- branch, worktree, and local/origin sync status when relevant - - ## Tool MVP PR Report Requirements - -@@ -82,3 +98,5 @@ unless explicitly requested. - ## No ZIP Means Incomplete - - A task is not considered complete until the ZIP artifact is generated and reported. -+ -+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. -diff --git a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md -index 145d08536..bee8bdd75 100644 ---- a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md -+++ b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md -@@ -126,10 +126,11 @@ Closed gates: - - 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. -+- Every Codex execution must produce a repo-structured ZIP under `tmp/`. -+- 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. - - 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. -+- 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. -+- No exceptions. +@@ -55,7 +55,7 @@ Codex responses must include: + Every tool MVP PR report must include: + - Product Owner testable outcome + - What Playwright tests +-- What Mr. Q should manually test ++- What the Product Owner should manually test + - Whether the PR is part of a stacked MVP sequence + - Previous PR dependency + - Next PR dependency +@@ -63,7 +63,7 @@ Every tool MVP PR report must include: + The report must answer: - ## Batch Governance Mode + ```text +-What can Mr. Q test after applying this ZIP? ++What can the Product Owner test after applying this ZIP? + ``` -@@ -240,6 +241,35 @@ Stop only for: - - Validation failure - - OWNER decision + If a tool MVP PR has no Playwright lane, the report must state why and list the manual Product Owner validation instead. +diff --git a/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md b/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md +index c6d8f192c..daf35f1d3 100644 +--- a/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md ++++ b/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md +@@ -23,25 +23,13 @@ Codex must use this as the only active source of truth for: -+## Completed PR ZIP Review And Next Logical PRs -+ -+Whenever the Product Owner provides a completed PR ZIP to ChatGPT, ChatGPT shall: -+ -+- review the completed work -+- identify completed scope -+- identify remaining scope -+- update the team backlog and completion percentages -+- determine the next logical PRs -+- recommend the execution order -+- generate each recommended PR with: -+ - PR summary -+ - Codex command -+ - commit comment -+ - Playwright scope -+ - manual validation steps + ## Branch Lifecycle Start Gate + +-Every PR follows exactly three phases: ++Codex must follow the canonical lifecycle in: + + ```text +-START +-WORK +-END ++docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md + ``` + +-Codex must follow the canonical lifecycle in: +- +-`docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md` +- +-Startup enforcement: +-- START begins on synchronized `main`. +-- WORK remains on the PR branch. Never checkout `main`. +-- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. +-- STOP if current branch is `main` before commit. +-- STOP if attempting to push `main`. +-- STOP if a new PR starts before returning to synchronized `main`. ++This startup addendum only requires Codex to read and apply that canonical lifecycle; it must not define a competing lifecycle rule. + + Deprecated Project Instructions material outside `docs_build/dev/ProjectInstructions/` is reference-only and must not override active governance. + +diff --git a/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md b/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md +index 5a91f904c..414bf9758 100644 +--- a/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md ++++ b/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md +@@ -11,6 +11,10 @@ This addendum is governance/documentation only. It does not change runtime behav + + ## Source Model + ++Canonical environment/API model reference: `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md`. + -+Use plural wording: `next logical PRs`. ++This file owns environment variable names, URL configuration, R2 prefix configuration, and feature flag configuration only. + -+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. + This standard builds on: + + ```text +@@ -88,12 +92,10 @@ R2 project, backup, export, import, or future storage paths must stay under the + + ## API/Service Contract Configuration + +-One shared API/service contract is required across Local (VS Code), DEV, IST, UAT, and PROD. ++The canonical shared API/service contract rule lives in `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md`. + +-Rules: ++This section owns configuration-only API URL requirements: + +-- Browser/UI/runtime code must follow `Browser -> API -> Database` for authoritative product data. +-- `Local API` means the same shared API/service contract running locally, not a separate local-only API implementation. + - API URLs may differ by `.env` only. + - Do not split Local API and Public API contracts. + - Do not create environment-specific API/service contracts. +diff --git a/docs_build/dev/ProjectInstructions/addendums/multi_team.md b/docs_build/dev/ProjectInstructions/addendums/multi_team.md +index 7ba2cae8d..763dccc8a 100644 +--- a/docs_build/dev/ProjectInstructions/addendums/multi_team.md ++++ b/docs_build/dev/ProjectInstructions/addendums/multi_team.md +@@ -1,5 +1,11 @@ + # Multi-Team Codex Execution Governance + ++Canonical workflow reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md`. + -+The Product Owner should not have to ask: ++Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. + -+- `What is next?` -+- `Create the next PR.` -+- `Continue.` ++This file owns multi-team execution coordination and must not create competing workflow or branch lifecycle rules. + -+The backlog drives the next recommended PRs automatically. + ## Four Active Delivery Teams + + The single authoritative four-team ownership definition is: +@@ -65,7 +71,7 @@ Rules: + ## Day Work / EOD Merge Rule + + During active work: +-- Work happens on assigned team branches, OWNER branches, or scoped PR branches. ++- Work happens on active non-main team branches or scoped PR branches. + - Commits are allowed only on assigned non-main branches. + - Pushes are allowed and expected. + - Draft PRs are allowed and expected. +diff --git a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md +index bee8bdd75..97a73ca30 100644 +--- a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md ++++ b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md +@@ -46,31 +46,11 @@ This section supersedes older active wording that implies returning to `main` be + + ## Branch Lifecycle (Canonical) + +-Every PR follows exactly three phases: +- +-```text +-START +-WORK +-END +-``` +- + The canonical START, WORK, END, Daily Synchronization, and Mandatory Hard Stops rules live in: + + `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md` + +-PR workflow must follow that lifecycle exactly. +- +-Summary: +-- START happens on synchronized `main` only and creates the PR branch only after all required gates pass. +-- WORK happens only on the PR branch. +-- END validates, commits, pushes, opens/updates the PR, merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. +-- No commits on `main`. +-- No implementation on `main`. +-- No validation on `main` except start validation. +-- Never checkout `main` during WORK. +-- STOP before commit if current branch is `main`. +-- STOP if current branch changes unexpectedly. +-- STOP if attempting to push `main`. ++PR workflow must follow that lifecycle exactly and must not create a competing lifecycle rule. + + ## PR Lifecycle States + +@@ -153,7 +133,7 @@ Do not stop after every small PR unless blocked by branch state, failed validati + Each tool MVP PR plan or template must include: + - Product Owner testable outcome + - What Playwright tests +-- What Mr. Q should manually test ++- What the Product Owner should manually test + - Whether the PR is part of a stacked MVP sequence + - Previous PR dependency + - Next PR dependency +@@ -162,6 +142,8 @@ Visible acceptance must be Creator-facing first. Architecture can be handled und + + ## Product Owner Testable Definition + ++Canonical owner: this section is the active canonical Product Owner testable completion rule. + - ## EOD Main Lock + 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. - End of Day: + A Product Owner testable outcome means the Product Owner can: diff --git a/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md b/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md -index 437c01800..be3555469 100644 +index be3555469..e6d6c7ce8 100644 --- a/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md +++ b/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md -@@ -53,6 +53,8 @@ The backlog is the authoritative source for determining the next PRs. +@@ -42,14 +42,15 @@ Each backlog item must track: + - current completion percentage + - remaining work + - blocking dependencies ++- owning team + + Completion percentages are updated: + + - at SOD +-- after each accepted PR ++- after every accepted PR + - at EOD + +-The backlog is the authoritative source for determining the next PRs. ++The backlog is the authoritative source for determining the next logical PRs. If the backlog and a generated report conflict, the backlog wins unless OWNER explicitly approves a newer governance decision. -+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. +diff --git a/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md b/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md +index 2bcc52a60..5da2f9952 100644 +--- a/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md ++++ b/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md +@@ -24,7 +24,7 @@ One large Codex command -> multiple focused stacked PRs -> each PR has a Product + - Each PR must answer: + + ```text +-What can Mr. Q test after applying this ZIP? ++What can the Product Owner test after applying this ZIP? + ``` + + - Codex must continue through the stacked PR sequence unless blocked by: +@@ -40,45 +40,14 @@ For tool MVP PR planning, visible acceptance must be Creator-facing first. + + Architecture can be handled under the covers, but PR purpose must be user-testable. + +-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. +- +-A Product Owner testable outcome means the Product Owner can: +- +-- open the page/tool +-- perform the primary workflow +-- save/load data where applicable +-- observe expected results +-- validate success and failure states +-- follow manual validation steps from the PR report +- +-Not acceptable as complete/testable: +- +-- shell-only page +-- route-only page +-- navigation-only PR +-- template-only page +-- placeholder controls +-- static table with no workflow +-- `Not implemented yet` +-- `coming soon` +-- placeholder-only workspace, inspector, or output sections +-- planned-only tile +-- route that loads but cannot be used +- +-Required for Product Owner testable completion: +- +-- visible working controls +-- API-backed data where required +-- validation and error states +-- empty states +-- save/load behavior where applicable +-- manual validation steps for Product Owner +-- targeted Playwright coverage where impacted. ++Canonical reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` owns the Product Owner testable definition and no-shell completion rule. ++ ++Canonical Playwright reference: `docs_build/dev/ProjectInstructions/addendums/test_structure_standardization.md` owns page-level Playwright organization and minimum completion coverage. + + Each tool MVP PR must state: + - Product Owner testable outcome + - What Playwright tests +-- What Mr. Q should manually test ++- What the Product Owner should manually test + - Whether the PR is part of a stacked MVP sequence + - Previous PR dependency + - Next PR dependency +diff --git a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md +index cc9ddc88c..c0e3336a9 100644 +--- a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md ++++ b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md +@@ -2,19 +2,22 @@ + + ## Backlog Item Tracking Standard + ++Canonical backlog tracking owner: `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md`. + - ## Start Of Day Team Briefing + Every active team owns an active backlog when it has assigned work. - When a team is assigned, ChatGPT/Codex must provide this briefing before implementation begins: -diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md +-Each backlog item must track: ++Backlog entries in this file follow the canonical field list, including: + + - Name + - Description + - Current completion percentage + - Remaining work + - Blocking dependencies ++- Owning team + +-Completion percentages are updated at SOD, after each accepted PR, and at EOD. ++Completion percentages are updated at SOD, after every accepted PR, and at EOD. + +-The backlog is the authoritative source for determining the next PRs. ++The backlog is the authoritative source for determining the next logical PRs. + + ## Game Journey MVP + +@@ -44,7 +47,7 @@ The backlog is the authoritative source for determining the next PRs. + - [ ] Bravo - Asset Studio V2 + - [ ] Charlie - Sprites canvas editor MVP + - Sprites is a creator tool, not only an asset metadata library. +- - MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only reusable colors, color selection from Palette/Colors, pixel painting, save/load sprite grid data through the API/database, and Product Owner manual validation. ++ - MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. + - Category is removed from Sprites MVP planning. + - [ ] Bravo - Animation Studio V2 + - [ ] Bravo - Palette Manager +@@ -65,7 +68,8 @@ The backlog is the authoritative source for determining the next PRs. + + 0% Complete — Create the things players interact with + +-- [ ] Alfa - Objects ++- [ ] Charlie - Objects ++ - Objects includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. + - [ ] Alfa - Characters + - [ ] Alfa - Object Behaviors + +@@ -226,6 +230,7 @@ Current OWNER clarification: + + ### Team Charlie + ++- Active ownership: Runtime, System Health, Environment Management, Sprites, Objects. + - [ ] Charlie - Guardrail hardening + - [ ] Charlie - Browser validation hardening + - [ ] Charlie - Remaining test relocation audit +@@ -257,6 +262,10 @@ Current OWNER clarification: + - observability integrations + - [ ] Charlie - Infrastructure dashboard + - [ ] Charlie - Environment validation ++- [ ] Charlie - Sprites canvas editor MVP ++ - Scope: canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. ++- [ ] Charlie - Objects ++ - Scope: object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. + + ### Team Delta + +diff --git a/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md b/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md +index 4253971d9..e7826a808 100644 +--- a/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md ++++ b/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md +@@ -1,5 +1,11 @@ + # TEAM_ASSIGNMENTS + ++Canonical workflow reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md`. ++ ++Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. ++ ++This file records active assignments and registry state. Workflow summaries here are subordinate to the canonical workflow and branch lifecycle documents. ++ + # Active Team Registry + + | Team | Assignment | Branch | PR | Status | +@@ -197,7 +203,7 @@ Conflict note: + ## Day Work / EOD Merge Rule + + During active work: +-- Work happens on assigned team branches, OWNER branches, or scoped PR branches. ++- Work happens on active non-main team branches or scoped PR branches. + - Commits are allowed only on assigned non-main branches. + - Pushes are allowed and expected. + - Draft PRs are allowed and expected. +@@ -263,23 +269,10 @@ Teams must use only `docs_build/dev/ProjectInstructions/` as the active Project + + ## Branch Lifecycle (Canonical) + +-Every PR follows exactly three phases: ++Teams must follow the canonical branch lifecycle in: + + ```text +-START +-WORK +-END ++docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md + ``` + +-Teams must follow `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. +- +-Assignment governance enforces: +-- START begins on synchronized `main`. +-- WORK remains on the PR branch. Never checkout `main`. +-- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. +-- No commits on `main`. +-- No implementation on `main`. +-- No validation on `main` except start validation. +-- STOP if current branch is `main` before commit. +-- STOP if attempting to push `main`. +-- STOP if new PR work starts before returning to synchronized `main`. ++Assignment governance records active team and branch state; it must not create a competing lifecycle rule. +diff --git a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md +index 30b4f2dfe..fb8497c1f 100644 +--- a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md ++++ b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md +@@ -30,11 +30,13 @@ This section is the current OWNER-approved active ownership alignment. + - Repository compliance + - Validation + - Infrastructure ++- Runtime + - Storage +-- Environment management ++- Environment Management + - System Health + - Operations + - Sprites canvas editor MVP ++- Objects + + Team Charlie System Health owns: + - Environment Summary +@@ -45,9 +47,18 @@ Team Charlie System Health owns: + + Team Charlie Sprites ownership: + - Sprites is a creator tool, not only an asset metadata library. +-- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only reusable colors, color selection from Palette/Colors, pixel painting, save/load sprite grid data through the API/database, and Product Owner manual validation. ++- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. + - Sprites must not own reusable color definitions. + ++Team Charlie Objects ownership: ++- Object library ++- Object editor ++- Sprite assignment ++- Object properties ++- Collision configuration ++- Runtime object metadata ++- Object API/database contracts ++ + ## Team Delta + + - Engine +diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md new file mode 100644 -index 000000000..bc4213d30 +index 000000000..cbbefda41 --- /dev/null -+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md -@@ -0,0 +1,56 @@ -+# PR_26177_OWNER_011-codex-zip-and-next-pr-standard ++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md +@@ -0,0 +1,62 @@ ++# PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + +Date: 2026-06-27 +Team: OWNER -+Branch: PR_26177_OWNER_011-codex-zip-and-next-pr-standard ++Branch: PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + +## Purpose + -+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. ++Finalize active Project Instructions cleanup, canonicalization, and backlog ownership governance. + +## Scope + @@ -158,55 +553,61 @@ index 000000000..bc4213d30 + +## Changes + -+- Strengthened the Codex artifact standard so every Codex execution produces required reports and a repo-structured ZIP. -+- Made the ZIP/report rule apply to success, completion, partial completion, hard stop, blocked, validation failure, new information, no-change, review, and governance results. -+- Removed the previous exception that allowed hard stops before outputs to skip ZIP/report delivery. -+- Required Codex responses to return ZIP path, reports, changed file list, validation results, and branch/worktree/sync status when relevant. -+- Added completed PR ZIP review governance: -+ - review completed work -+ - identify completed and remaining scope -+ - update team backlog and completion percentages -+ - determine the next logical PRs -+ - recommend execution order -+ - generate each recommended PR with summary, Codex command, commit comment, Playwright scope, and manual validation steps -+- Confirmed the backlog drives next recommended PRs automatically. -+- Preserved the existing SOD, active team branch, sequential workstream, EOD, Product Owner testable, and page-level Playwright workflow rules. ++- Added a canonical governance owner map to the Project Instructions root. ++- Made overlap documents point to canonical owners instead of creating competing active rules: ++ - workflow and Product Owner testable completion: `pr_workflow.md` ++ - branch lifecycle and EOD main lock: `project_instructions_single_source_eod_lock.md` ++ - page-level Playwright: `test_structure_standardization.md` ++ - API/environment model: `environment_governance_model.md` ++ - environment configuration: `environment_configuration_standards.md` ++ - team backlog: `team_backlog_sod_eod_standard.md` ++ - team ownership: `team_ownership.md` ++- Replaced active `Mr. Q` manual validation wording with Product Owner wording. ++- Expanded backlog item requirements with owning team. ++- Updated backlog percentage cadence to SOD, after every accepted PR, and EOD. ++- Confirmed the backlog drives the next logical PRs. ++- Updated active Team Charlie ownership for Runtime, System Health, Environment Management, Sprites, and Objects. ++- Updated Sprites MVP scope to include canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. ++- Added Objects scope for object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. + +## Validation + -+- PASS: `git diff --cached --check -- . :(exclude)docs_build/dev/reports/codex_review.diff` +- PASS: documentation/governance-only changed-file check. -+- PASS: ZIP-on-every-result rule exists. -+- PASS: hard stops require ZIP/report output. -+- PASS: `next logical PRs` plural wording exists. -+- PASS: automatic next PR planning is tied to completed PR ZIP review. -+- PASS: SOD starts from latest synchronized `main`. -+- PASS: SOD creates or uses the active team branch. -+- PASS: all commits go to the active team branch, not `main`. -+- PASS: sequential PRs stay on the active team branch/workstream during the day. -+- PASS: EOD returns to `main` and verifies clean sync `0 0`. -+- PASS: Product Owner testable completion rule remains present. -+- PASS: page-level Playwright completion gate remains present. ++- PASS: `git diff --check` +- PASS: no runtime files changed. ++- PASS: no UI files changed. ++- PASS: no API files changed. ++- PASS: no database files changed. ++- PASS: no `start_of_day` files changed. ++- PASS: no active `Mr. Q` manual validation wording remains. ++- PASS: no OWNER-only branch workflow wording remains. ++- PASS: active `Alpha` references are limited to the preserved non-team cancellation phrase. ++- PASS: canonical owner references exist. ++- PASS: Product Owner testable canonical owner remains `pr_workflow.md`. ++- PASS: page-level Playwright canonical owner remains `test_structure_standardization.md`. ++- PASS: API/environment canonical owner remains `environment_governance_model.md`. ++- PASS: branch lifecycle canonical owner remains `project_instructions_single_source_eod_lock.md`. ++- PASS: Team Charlie owns both Sprites and Objects. ++- PASS: repo-structured ZIP produced for documentation-only changes. + +## Artifact + -+- `tmp/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_delta.zip` ++- `tmp/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_delta.zip` + -+## Manual Validation Notes ++## Next Logical PRs + -+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. -diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md ++No additional Project Instructions cleanup PR is required from this pass. Future cleanup should wait until after the OWNER governance stack is reviewed, unless the Product Owner identifies a new conflict. +diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md new file mode 100644 -index 000000000..cd2754f9f +index 000000000..995e565ba --- /dev/null -+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md ++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md @@ -0,0 +1,23 @@ -+# PR_26177_OWNER_011 Branch Validation ++# PR_26177_OWNER_012 Branch Validation + +Date: 2026-06-27 +Team: OWNER -+Branch: PR_26177_OWNER_011-codex-zip-and-next-pr-standard ++Branch: PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + +## Result + @@ -215,23 +616,23 @@ index 000000000..cd2754f9f +## Checks + +- PASS: Work was performed on the active OWNER PR branch, not `main`. -+- PASS: Branch is stacked on `PR_26177_OWNER_010-team-backlog-sod-eod-standard`. ++- PASS: Branch is stacked on `PR_26177_OWNER_011-codex-zip-and-next-pr-standard`. +- PASS: Changes are limited to active Project Instructions and generated reports. +- PASS: No runtime, UI, API, database, or `start_of_day` files changed. -+- PASS: Staged whitespace validation passed with generated `codex_review.diff` excluded. ++- PASS: `git diff --check` passed before report generation. +- PASS: Required report files were generated. +- PASS: Repo-structured ZIP artifact path is defined under `tmp/`. + +## Notes + +This PR remains in the OWNER workstream and must not be merged to `main` until EOD closeout or explicit OWNER approval. -diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md +diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md new file mode 100644 -index 000000000..af329a4cb +index 000000000..2b111a71f --- /dev/null -+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md -@@ -0,0 +1,20 @@ -+# PR_26177_OWNER_011 Manual Validation Notes ++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md +@@ -0,0 +1,24 @@ ++# PR_26177_OWNER_012 Manual Validation Notes + +Date: 2026-06-27 +Team: OWNER @@ -240,24 +641,28 @@ index 000000000..af329a4cb + +Review these files: + -+- `docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md` ++- `docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md` +- `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` +- `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` ++- `docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md` ++- `docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md` ++ ++Confirm the canonical governance owner map is clear and that overlap files point back to canonical owner documents. + -+Confirm they require every Codex result to include reports and a repo-structured ZIP, including hard stops and no-change results. ++Confirm Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. + -+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. ++Confirm active backlog wording uses Product Owner validation language and drives the next logical PRs. + +## Expected Reviewer Outcome + -+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. -diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md ++The Product Owner should be able to identify the canonical owner document for workflow, branch rules, Product Owner testable completion, Playwright organization, API/environment rules, team backlog, and team ownership without reading competing active rules. +diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md new file mode 100644 -index 000000000..0dc03d042 +index 000000000..d9a12c54e --- /dev/null -+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md -@@ -0,0 +1,30 @@ -+# PR_26177_OWNER_011 Requirements Checklist ++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md +@@ -0,0 +1,33 @@ ++# PR_26177_OWNER_012 Requirements Checklist + +Date: 2026-06-27 +Team: OWNER @@ -265,20 +670,23 @@ index 000000000..0dc03d042 +## Requirement Results + +- PASS: Documentation/governance only. -+- PASS: Active Project Instructions reviewed and updated. ++- PASS: Active Project Instructions reviewed. +- PASS: Active instruction files updated only. -+- PASS: Historical snapshots were not rewritten. ++- PASS: Historical snapshots were not modified. +- PASS: No runtime changes. -+- PASS: ZIP-on-every-result governance added. -+- PASS: Rule applies to success, completion, partial completion, hard stop, blocked, validation failure, new information, and no files changed. -+- PASS: Every Codex execution must return repo-structured ZIP path, reports, changed file list, validation results, and branch/worktree/sync status when relevant. -+- PASS: `No exceptions` wording added. -+- PASS: Automatic next PR planning tied to Product Owner completed PR ZIP review. -+- PASS: `next logical PRs` plural wording exists. -+- PASS: Recommended PR output includes PR summary, Codex command, commit comment, Playwright scope, and manual validation steps. -+- PASS: Product Owner should not need to ask `What is next?`, `Create the next PR.`, or `Continue.` -+- PASS: Backlog drives next recommended PRs automatically. -+- PASS: Existing SOD/main, active team branch, stacked PR, EOD, Product Owner testable, and page-level Playwright rules remain present. ++- PASS: Canonical governance owner map added. ++- PASS: Duplicated active guidance was removed where safe. ++- PASS: Append-style overlap files now point to canonical owner documents. ++- PASS: Military team spelling retained: OWNER, ALFA, BRAVO, CHARLIE, DELTA. ++- PASS: No new Greek team spelling introduced. ++- PASS: Active `Mr. Q` validation wording replaced with Product Owner wording. ++- PASS: Backlog item fields include owning team. ++- PASS: Backlog percentages update at SOD, after every accepted PR, and at EOD. ++- PASS: Backlog is authoritative for determining the next logical PRs. ++- PASS: Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. ++- PASS: Sprites MVP scope includes canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. ++- PASS: Objects scope includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. ++- PASS: Existing SOD, active team branch, sequential PR, no-return-to-main, and EOD clean main sync workflow remains present. +- PASS: Required reports were generated. +- PASS: Repo-structured ZIP artifact will be generated under `tmp/`. + @@ -287,13 +695,13 @@ index 000000000..0dc03d042 +- PASS: No commit was made to `main`. +- PASS: No `start_of_day` folders changed. +- PASS: No unrelated cleanup. -diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md +diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md new file mode 100644 -index 000000000..0eae5bc5a +index 000000000..1f8990d29 --- /dev/null -+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md -@@ -0,0 +1,43 @@ -+# PR_26177_OWNER_011 Validation Lane ++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md +@@ -0,0 +1,51 @@ ++# PR_26177_OWNER_012 Validation Lane + +Date: 2026-06-27 +Team: OWNER @@ -301,7 +709,7 @@ index 000000000..0eae5bc5a +## Commands + +```powershell -+git diff --cached --check -- . ":(exclude)docs_build/dev/reports/codex_review.diff" ++git diff --check +``` + +Result: PASS @@ -313,25 +721,33 @@ index 000000000..0eae5bc5a +Result: PASS + +```powershell -+node -e "required ZIP, next logical PRs, workflow preservation checks" ++rg -n "Mr\\. Q|What can Mr\\. Q|What Mr\\. Q" docs_build/dev/ProjectInstructions ++``` ++ ++Result: PASS, no active matches. ++ ++```powershell ++rg -n "Work must be committed only to the active OWNER branch|PR branches/commits stay on the active OWNER branch|active OWNER branch|OWNER branches|OWNER branch" docs_build/dev/ProjectInstructions ++``` ++ ++Result: PASS, no active OWNER-only branch workflow wording. ++ ++```powershell ++node -e "canonical owner, backlog, and Team Charlie ownership checks" +``` + +Result: PASS + +## Targeted Results + -+- PASS: ZIP-on-every-result rule exists. -+- PASS: hard stops require ZIP/report output. -+- PASS: `next logical PRs` plural wording exists. -+- PASS: automatic next PR planning is tied to completed PR ZIP review. -+- PASS: SOD/main active branch workflow remains present. -+- PASS: EOD clean main sync rule remains present. -+- PASS: Product Owner testable rule remains present. -+- PASS: page-level Playwright completion gate remains present. ++- PASS: canonical governance owner map exists. ++- PASS: Product Owner testable owner is canonicalized. ++- PASS: page-level Playwright owner is canonicalized. ++- PASS: API/environment owner is canonicalized. ++- PASS: branch lifecycle owner is canonicalized. ++- PASS: backlog includes owning team and next logical PR wording. ++- PASS: Team Charlie owns Sprites and Objects. +- PASS: no runtime files changed. -+- PASS: no UI files changed. -+- PASS: no API files changed. -+- PASS: no database files changed. + +## Playwright + From 31313ed914bbee2d1f462f3b0ddd1c910b0223c5 Mon Sep 17 00:00:00 2001 From: Charlie Team <97194984+ToolboxAid@users.noreply.github.com> Date: Fri, 26 Jun 2026 22:09:50 -0400 Subject: [PATCH 2/2] Update Charlie backlog ownership canonicalization --- .../backlog/BACKLOG_MASTER.md | 26 +- .../team_assignments/team_ownership.md | 10 +- ...ctions-cleanup-backlog-canonicalization.md | 10 +- ...anonicalization_manual-validation-notes.md | 2 +- ...canonicalization_requirements-checklist.md | 8 +- ...acklog-canonicalization_validation-lane.md | 2 +- docs_build/dev/reports/codex_review.diff | 1072 ++++++++++++++++- 7 files changed, 1086 insertions(+), 44 deletions(-) diff --git a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md index c0e3336a9..99b7388e3 100644 --- a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md +++ b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md @@ -47,10 +47,13 @@ The backlog is the authoritative source for determining the next logical PRs. - [ ] Bravo - Asset Studio V2 - [ ] Charlie - Sprites canvas editor MVP - Sprites is a creator tool, not only an asset metadata library. - - MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. + - 5% Complete. + - Remaining work: canvas/grid editor, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, Product Owner testable workflow, and remove Category from Sprites planning. - Category is removed from Sprites MVP planning. - [ ] Bravo - Animation Studio V2 -- [ ] Bravo - Palette Manager +- [ ] Charlie - Palette / Colors + - 40% Complete. + - Remaining work: color management, API/database integration, reusable color source of truth, Palette references for Sprites and Objects, and Product Owner testable workflow. - [ ] Bravo - Video Studio ### Audio @@ -69,7 +72,9 @@ The backlog is the authoritative source for determining the next logical PRs. 0% Complete — Create the things players interact with - [ ] Charlie - Objects + - 0% Complete. - Objects includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. + - Remaining work: object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, Object API/database contracts, and Product Owner testable workflow. - [ ] Alfa - Characters - [ ] Alfa - Object Behaviors @@ -230,7 +235,13 @@ Current OWNER clarification: ### Team Charlie -- Active ownership: Runtime, System Health, Environment Management, Sprites, Objects. +- Active ownership: Runtime, System Health, Environment Management, Palette / Colors, Sprites, Objects. +- Runtime: 100% Complete. +- System Health: 100% Complete. +- Environment Management: 100% Complete. +- Palette / Colors: 40% Complete. +- Sprites: 5% Complete. +- Objects: 0% Complete. - [ ] Charlie - Guardrail hardening - [ ] Charlie - Browser validation hardening - [ ] Charlie - Remaining test relocation audit @@ -262,10 +273,15 @@ Current OWNER clarification: - observability integrations - [ ] Charlie - Infrastructure dashboard - [ ] Charlie - Environment validation +- [ ] Charlie - Palette / Colors + - Current completion percentage: 40%. + - Remaining work: color management, API/database integration, reusable color source of truth, Palette references for Sprites and Objects, and Product Owner testable workflow. - [ ] Charlie - Sprites canvas editor MVP - - Scope: canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. + - Current completion percentage: 5%. + - Scope: canvas/grid editor, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, Product Owner testable workflow, and remove Category from Sprites planning. - [ ] Charlie - Objects - - Scope: object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. + - Current completion percentage: 0%. + - Scope: object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, Object API/database contracts, and Product Owner testable workflow. ### Team Delta diff --git a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md index fb8497c1f..0655d6a95 100644 --- a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md +++ b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md @@ -35,6 +35,7 @@ This section is the current OWNER-approved active ownership alignment. - Environment Management - System Health - Operations +- Palette / Colors - Sprites canvas editor MVP - Objects @@ -47,9 +48,16 @@ Team Charlie System Health owns: Team Charlie Sprites ownership: - Sprites is a creator tool, not only an asset metadata library. -- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. +- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, and a Product Owner testable workflow. - Sprites must not own reusable color definitions. +Team Charlie Palette / Colors ownership: +- Color management +- API/database integration +- Reusable color source of truth +- Palette references for Sprites and Objects +- Product Owner testable workflow + Team Charlie Objects ownership: - Object library - Object editor diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md index cbbefda41..a23272845 100644 --- a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md @@ -29,9 +29,11 @@ No runtime code, UI code, API code, database code, `start_of_day` files, history - Expanded backlog item requirements with owning team. - Updated backlog percentage cadence to SOD, after every accepted PR, and EOD. - Confirmed the backlog drives the next logical PRs. -- Updated active Team Charlie ownership for Runtime, System Health, Environment Management, Sprites, and Objects. -- Updated Sprites MVP scope to include canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. -- Added Objects scope for object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. +- Updated active Team Charlie ownership for Runtime, System Health, Environment Management, Palette / Colors, Sprites, and Objects. +- Added Charlie backlog status: Runtime 100%, System Health 100%, Environment Management 100%, Palette / Colors 40%, Sprites 5%, and Objects 0%. +- Updated Palette / Colors remaining work: color management, API/database integration, reusable color source of truth, Palette references for Sprites and Objects, and Product Owner testable workflow. +- Updated Sprites MVP scope to include canvas/grid editor, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, Product Owner testable workflow, and remove Category from Sprites planning. +- Added Objects scope for object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, Object API/database contracts, and Product Owner testable workflow. ## Validation @@ -50,7 +52,7 @@ No runtime code, UI code, API code, database code, `start_of_day` files, history - PASS: page-level Playwright canonical owner remains `test_structure_standardization.md`. - PASS: API/environment canonical owner remains `environment_governance_model.md`. - PASS: branch lifecycle canonical owner remains `project_instructions_single_source_eod_lock.md`. -- PASS: Team Charlie owns both Sprites and Objects. +- PASS: Team Charlie owns Palette / Colors, Sprites, and Objects. - PASS: repo-structured ZIP produced for documentation-only changes. ## Artifact diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md index 2b111a71f..7de18e4a0 100644 --- a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md @@ -15,7 +15,7 @@ Review these files: Confirm the canonical governance owner map is clear and that overlap files point back to canonical owner documents. -Confirm Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. +Confirm Team Charlie owns Runtime, System Health, Environment Management, Palette / Colors, Sprites, and Objects with the requested completion percentages and remaining work. Confirm active backlog wording uses Product Owner validation language and drives the next logical PRs. diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md index d9a12c54e..c08a85d44 100644 --- a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md @@ -19,9 +19,11 @@ Team: OWNER - PASS: Backlog item fields include owning team. - PASS: Backlog percentages update at SOD, after every accepted PR, and at EOD. - PASS: Backlog is authoritative for determining the next logical PRs. -- PASS: Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. -- PASS: Sprites MVP scope includes canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. -- PASS: Objects scope includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. +- PASS: Team Charlie owns Runtime, System Health, Environment Management, Palette / Colors, Sprites, and Objects. +- PASS: Charlie backlog status records Runtime 100%, System Health 100%, Environment Management 100%, Palette / Colors 40%, Sprites 5%, and Objects 0%. +- PASS: Palette / Colors remaining work includes color management, API/database integration, reusable color source of truth, Palette references for Sprites and Objects, and Product Owner testable workflow. +- PASS: Sprites remaining work includes canvas/grid editor, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, Product Owner testable workflow, and remove Category from Sprites planning. +- PASS: Objects scope includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, Object API/database contracts, and Product Owner testable workflow. - PASS: Existing SOD, active team branch, sequential PR, no-return-to-main, and EOD clean main sync workflow remains present. - PASS: Required reports were generated. - PASS: Repo-structured ZIP artifact will be generated under `tmp/`. diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md index 1f8990d29..d8729578c 100644 --- a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md @@ -43,7 +43,7 @@ Result: PASS - PASS: API/environment owner is canonicalized. - PASS: branch lifecycle owner is canonicalized. - PASS: backlog includes owning team and next logical PR wording. -- PASS: Team Charlie owns Sprites and Objects. +- PASS: Team Charlie owns Palette / Colors, Sprites, and Objects. - PASS: no runtime files changed. ## Playwright diff --git a/docs_build/dev/reports/codex_review.diff b/docs_build/dev/reports/codex_review.diff index 17dfb5afa..d3746d2fa 100644 --- a/docs_build/dev/reports/codex_review.diff +++ b/docs_build/dev/reports/codex_review.diff @@ -371,7 +371,7 @@ index 2bcc52a60..5da2f9952 100644 - Previous PR dependency - Next PR dependency diff --git a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md -index cc9ddc88c..c0e3336a9 100644 +index cc9ddc88c..99b7388e3 100644 --- a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md +++ b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md @@ -2,19 +2,22 @@ @@ -400,41 +400,61 @@ index cc9ddc88c..c0e3336a9 100644 ## Game Journey MVP -@@ -44,7 +47,7 @@ The backlog is the authoritative source for determining the next PRs. +@@ -44,10 +47,13 @@ The backlog is the authoritative source for determining the next PRs. - [ ] Bravo - Asset Studio V2 - [ ] Charlie - Sprites canvas editor MVP - Sprites is a creator tool, not only an asset metadata library. - - MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only reusable colors, color selection from Palette/Colors, pixel painting, save/load sprite grid data through the API/database, and Product Owner manual validation. -+ - MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. ++ - 5% Complete. ++ - Remaining work: canvas/grid editor, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, Product Owner testable workflow, and remove Category from Sprites planning. - Category is removed from Sprites MVP planning. - [ ] Bravo - Animation Studio V2 - - [ ] Bravo - Palette Manager -@@ -65,7 +68,8 @@ The backlog is the authoritative source for determining the next PRs. +-- [ ] Bravo - Palette Manager ++- [ ] Charlie - Palette / Colors ++ - 40% Complete. ++ - Remaining work: color management, API/database integration, reusable color source of truth, Palette references for Sprites and Objects, and Product Owner testable workflow. + - [ ] Bravo - Video Studio + + ### Audio +@@ -65,7 +71,10 @@ The backlog is the authoritative source for determining the next PRs. 0% Complete — Create the things players interact with -- [ ] Alfa - Objects +- [ ] Charlie - Objects ++ - 0% Complete. + - Objects includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. ++ - Remaining work: object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, Object API/database contracts, and Product Owner testable workflow. - [ ] Alfa - Characters - [ ] Alfa - Object Behaviors -@@ -226,6 +230,7 @@ Current OWNER clarification: +@@ -226,6 +235,13 @@ Current OWNER clarification: ### Team Charlie -+- Active ownership: Runtime, System Health, Environment Management, Sprites, Objects. ++- Active ownership: Runtime, System Health, Environment Management, Palette / Colors, Sprites, Objects. ++- Runtime: 100% Complete. ++- System Health: 100% Complete. ++- Environment Management: 100% Complete. ++- Palette / Colors: 40% Complete. ++- Sprites: 5% Complete. ++- Objects: 0% Complete. - [ ] Charlie - Guardrail hardening - [ ] Charlie - Browser validation hardening - [ ] Charlie - Remaining test relocation audit -@@ -257,6 +262,10 @@ Current OWNER clarification: +@@ -257,6 +273,15 @@ Current OWNER clarification: - observability integrations - [ ] Charlie - Infrastructure dashboard - [ ] Charlie - Environment validation ++- [ ] Charlie - Palette / Colors ++ - Current completion percentage: 40%. ++ - Remaining work: color management, API/database integration, reusable color source of truth, Palette references for Sprites and Objects, and Product Owner testable workflow. +- [ ] Charlie - Sprites canvas editor MVP -+ - Scope: canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. ++ - Current completion percentage: 5%. ++ - Scope: canvas/grid editor, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, Product Owner testable workflow, and remove Category from Sprites planning. +- [ ] Charlie - Objects -+ - Scope: object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. ++ - Current completion percentage: 0%. ++ - Scope: object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, Object API/database contracts, and Product Owner testable workflow. ### Team Delta @@ -491,10 +511,10 @@ index 4253971d9..e7826a808 100644 -- STOP if new PR work starts before returning to synchronized `main`. +Assignment governance records active team and branch state; it must not create a competing lifecycle rule. diff --git a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md -index 30b4f2dfe..fb8497c1f 100644 +index 30b4f2dfe..0655d6a95 100644 --- a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md +++ b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md -@@ -30,11 +30,13 @@ This section is the current OWNER-approved active ownership alignment. +@@ -30,11 +30,14 @@ This section is the current OWNER-approved active ownership alignment. - Repository compliance - Validation - Infrastructure @@ -504,19 +524,27 @@ index 30b4f2dfe..fb8497c1f 100644 +- Environment Management - System Health - Operations ++- Palette / Colors - Sprites canvas editor MVP +- Objects Team Charlie System Health owns: - Environment Summary -@@ -45,9 +47,18 @@ Team Charlie System Health owns: +@@ -45,9 +48,25 @@ Team Charlie System Health owns: Team Charlie Sprites ownership: - Sprites is a creator tool, not only an asset metadata library. -- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only reusable colors, color selection from Palette/Colors, pixel painting, save/load sprite grid data through the API/database, and Product Owner manual validation. -+- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. ++- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, and a Product Owner testable workflow. - Sprites must not own reusable color definitions. ++Team Charlie Palette / Colors ownership: ++- Color management ++- API/database integration ++- Reusable color source of truth ++- Palette references for Sprites and Objects ++- Product Owner testable workflow ++ +Team Charlie Objects ownership: +- Object library +- Object editor @@ -531,10 +559,10 @@ index 30b4f2dfe..fb8497c1f 100644 - Engine diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md new file mode 100644 -index 000000000..cbbefda41 +index 000000000..a23272845 --- /dev/null +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md -@@ -0,0 +1,62 @@ +@@ -0,0 +1,64 @@ +# PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + +Date: 2026-06-27 @@ -566,9 +594,11 @@ index 000000000..cbbefda41 +- Expanded backlog item requirements with owning team. +- Updated backlog percentage cadence to SOD, after every accepted PR, and EOD. +- Confirmed the backlog drives the next logical PRs. -+- Updated active Team Charlie ownership for Runtime, System Health, Environment Management, Sprites, and Objects. -+- Updated Sprites MVP scope to include canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. -+- Added Objects scope for object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. ++- Updated active Team Charlie ownership for Runtime, System Health, Environment Management, Palette / Colors, Sprites, and Objects. ++- Added Charlie backlog status: Runtime 100%, System Health 100%, Environment Management 100%, Palette / Colors 40%, Sprites 5%, and Objects 0%. ++- Updated Palette / Colors remaining work: color management, API/database integration, reusable color source of truth, Palette references for Sprites and Objects, and Product Owner testable workflow. ++- Updated Sprites MVP scope to include canvas/grid editor, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, Product Owner testable workflow, and remove Category from Sprites planning. ++- Added Objects scope for object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, Object API/database contracts, and Product Owner testable workflow. + +## Validation + @@ -587,7 +617,7 @@ index 000000000..cbbefda41 +- PASS: page-level Playwright canonical owner remains `test_structure_standardization.md`. +- PASS: API/environment canonical owner remains `environment_governance_model.md`. +- PASS: branch lifecycle canonical owner remains `project_instructions_single_source_eod_lock.md`. -+- PASS: Team Charlie owns both Sprites and Objects. ++- PASS: Team Charlie owns Palette / Colors, Sprites, and Objects. +- PASS: repo-structured ZIP produced for documentation-only changes. + +## Artifact @@ -628,7 +658,7 @@ index 000000000..995e565ba +This PR remains in the OWNER workstream and must not be merged to `main` until EOD closeout or explicit OWNER approval. diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md new file mode 100644 -index 000000000..2b111a71f +index 000000000..7de18e4a0 --- /dev/null +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md @@ -0,0 +1,24 @@ @@ -649,7 +679,7 @@ index 000000000..2b111a71f + +Confirm the canonical governance owner map is clear and that overlap files point back to canonical owner documents. + -+Confirm Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. ++Confirm Team Charlie owns Runtime, System Health, Environment Management, Palette / Colors, Sprites, and Objects with the requested completion percentages and remaining work. + +Confirm active backlog wording uses Product Owner validation language and drives the next logical PRs. + @@ -658,10 +688,10 @@ index 000000000..2b111a71f +The Product Owner should be able to identify the canonical owner document for workflow, branch rules, Product Owner testable completion, Playwright organization, API/environment rules, team backlog, and team ownership without reading competing active rules. diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md new file mode 100644 -index 000000000..d9a12c54e +index 000000000..c08a85d44 --- /dev/null +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md -@@ -0,0 +1,33 @@ +@@ -0,0 +1,35 @@ +# PR_26177_OWNER_012 Requirements Checklist + +Date: 2026-06-27 @@ -683,9 +713,11 @@ index 000000000..d9a12c54e +- PASS: Backlog item fields include owning team. +- PASS: Backlog percentages update at SOD, after every accepted PR, and at EOD. +- PASS: Backlog is authoritative for determining the next logical PRs. -+- PASS: Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. -+- PASS: Sprites MVP scope includes canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. -+- PASS: Objects scope includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. ++- PASS: Team Charlie owns Runtime, System Health, Environment Management, Palette / Colors, Sprites, and Objects. ++- PASS: Charlie backlog status records Runtime 100%, System Health 100%, Environment Management 100%, Palette / Colors 40%, Sprites 5%, and Objects 0%. ++- PASS: Palette / Colors remaining work includes color management, API/database integration, reusable color source of truth, Palette references for Sprites and Objects, and Product Owner testable workflow. ++- PASS: Sprites remaining work includes canvas/grid editor, width/height controls, Palette/Colors-only color selection, pixel painting, save/load sprite grid data through API/database, Product Owner testable workflow, and remove Category from Sprites planning. ++- PASS: Objects scope includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, Object API/database contracts, and Product Owner testable workflow. +- PASS: Existing SOD, active team branch, sequential PR, no-return-to-main, and EOD clean main sync workflow remains present. +- PASS: Required reports were generated. +- PASS: Repo-structured ZIP artifact will be generated under `tmp/`. @@ -697,7 +729,7 @@ index 000000000..d9a12c54e +- PASS: No unrelated cleanup. diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md new file mode 100644 -index 000000000..1f8990d29 +index 000000000..d8729578c --- /dev/null +++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md @@ -0,0 +1,51 @@ @@ -746,9 +778,991 @@ index 000000000..1f8990d29 +- PASS: API/environment owner is canonicalized. +- PASS: branch lifecycle owner is canonicalized. +- PASS: backlog includes owning team and next logical PR wording. -+- PASS: Team Charlie owns Sprites and Objects. ++- PASS: Team Charlie owns Palette / Colors, Sprites, and Objects. +- PASS: no runtime files changed. + +## Playwright + +Not impacted. This PR is documentation/governance only. +diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt +index 85d74e87a..4c7d9fba2 100644 +--- a/docs_build/dev/reports/codex_changed_files.txt ++++ b/docs_build/dev/reports/codex_changed_files.txt +@@ -1,10 +1,22 @@ ++docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md ++docs_build/dev/ProjectInstructions/README.txt ++docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md ++docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md ++docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md + docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md ++docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md ++docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md ++docs_build/dev/ProjectInstructions/addendums/multi_team.md + docs_build/dev/ProjectInstructions/addendums/pr_workflow.md + docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md +-docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md +-docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md +-docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md +-docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md +-docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md ++docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md ++docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md ++docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md ++docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md ++docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md ++docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md ++docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md ++docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md ++docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md + docs_build/dev/reports/codex_changed_files.txt + docs_build/dev/reports/codex_review.diff +diff --git a/docs_build/dev/reports/codex_review.diff b/docs_build/dev/reports/codex_review.diff +index 0ea4e6438..17dfb5afa 100644 +--- a/docs_build/dev/reports/codex_review.diff ++++ b/docs_build/dev/reports/codex_review.diff +@@ -1,154 +1,549 @@ ++diff --git a/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md b/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md ++index d6617c986..ffd6f2438 100644 ++--- a/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md +++++ b/docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md ++@@ -58,6 +58,20 @@ Existing Project Instructions outside `docs_build/dev/ProjectInstructions/` rema ++ ++ `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. ++ +++## Canonical Governance Owners +++ +++When active guidance overlaps, use these canonical owner documents: +++ +++- Workflow and Product Owner testable completion: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` +++- 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` +++- Page-level Playwright organization and completion coverage: `docs_build/dev/ProjectInstructions/addendums/test_structure_standardization.md` +++- API/environment model and `Browser -> API -> Database` rule: `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md` +++- Environment variable, URL, R2 prefix, and feature flag configuration: `docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md` +++- Team backlog fields, completion percentages, and next logical PR ownership: `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` +++- Team ownership and assignment routing: `docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md` +++ +++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. +++ ++ ## Environment Governance ++ ++ `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: ++ `OWNER override approved: ` ++ ++ OWNER follows the same safety rules: ++-- One active Team OWNER branch at a time. +++- Team OWNER follows the same one-active-branch discipline as every team. ++ - One active OWNER assignment at a time. ++ - OWNER may override team locks, but may not silently delete, rewrite, or remove protected instructions. ++ - OWNER override must be explicitly documented. ++diff --git a/docs_build/dev/ProjectInstructions/README.txt b/docs_build/dev/ProjectInstructions/README.txt ++index 1b647c6fd..bbfff9812 100644 ++--- a/docs_build/dev/ProjectInstructions/README.txt +++++ b/docs_build/dev/ProjectInstructions/README.txt ++@@ -16,7 +16,7 @@ No direct commits to main: ++ Do not commit directly to main. Normal work must use PR branches, draft PRs, validation evidence, and owner-controlled merge approval. ++ ++ Branch lifecycle: ++-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`. +++The canonical START / WORK / END lifecycle is `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. ++ ++ OWNER override rule: ++ An OWNER override must use this wording: ++diff --git a/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md b/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md ++index 5fe1c43e4..7f9bcad29 100644 ++--- a/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md +++++ b/docs_build/dev/ProjectInstructions/TEAM_START_COMMANDS.md ++@@ -13,14 +13,7 @@ Use `docs_build/dev/ProjectInstructions/` as the only active Project Instruction ++ Read `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` before implementation. ++ ++ Branch Lifecycle (Canonical): ++-- Every PR follows exactly three phases: START, WORK, END. ++-- Follow `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. ++-- START begins on synchronized `main` and creates the PR branch only after all gates pass. ++-- WORK remains on the PR branch. Never checkout `main`. ++-- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. ++-- No commits on `main`. ++-- No implementation on `main`. ++-- No validation on `main` except start validation. +++- 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. ++ ++ ## Start Team Alfa ++ ++diff --git a/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md b/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md ++index 798bb2cba..a829b0a15 100644 ++--- a/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md +++++ b/docs_build/dev/ProjectInstructions/addendums/branch_context_governance.md ++@@ -7,6 +7,10 @@ Owner: OWNER ++ ++ Require Codex and teams to confirm branch context before changing files. ++ +++Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. +++ +++This file owns branch context confirmation and stop conditions; it must not create a competing START / WORK / END lifecycle rule. +++ ++ ## Session Start Context ++ ++ At the start of work, report or validate: ++diff --git a/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md b/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md ++index 6f797306d..1851906ab 100644 ++--- a/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md +++++ b/docs_build/dev/ProjectInstructions/addendums/branch_lock_governance.md ++@@ -7,6 +7,10 @@ Owner: OWNER ++ ++ Keep active work attached to the correct assigned team, branch, and OWNER decision. ++ +++Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. +++ +++This file owns branch lock enforcement and OWNER override handling; it must not create a competing START / WORK / END lifecycle rule. +++ ++ ## Active Work Lock ++ ++ - 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 ++ ++ ## Branch Lifecycle (Canonical) ++ ++-Every PR follows exactly three phases: +++The canonical branch lifecycle lives in: ++ ++ ```text ++-START ++-WORK ++-END +++docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md ++ ``` ++ ++-The canonical lifecycle lives in `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. ++- ++-Branch lock governance enforces: ++-- START on synchronized `main`. ++-- WORK only on the PR branch. ++-- END by merging, returning to synchronized `main`, publishing branch, HEAD SHA, and date/time, then stopping all work. ++-- 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. +++Branch lock governance enforces ownership and branch-lock compliance with that canonical lifecycle. ++ ++ ## OWNER Override ++ + diff --git a/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md b/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md +-index 1c3671363..17be32ada 100644 ++index 17be32ada..a382a65df 100644 + --- a/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md + +++ b/docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md +-@@ -6,20 +6,31 @@ Standardize Codex deliverables, completion reporting, and artifact generation. +- +- ## ZIP Artifact Requirement +- +--Every Codex task must produce a ZIP artifact. +-+Every Codex execution must produce a repo-structured ZIP artifact and the required reports. +- +--Applies to: +--- Success +--- Failure +--- Stop Gate +--- Partial Completion +--- Review Deliverables +--- Governance Deliverables +-+This applies regardless of result: +-+ +-+- success +-+- completion +-+- partial completion +-+- hard stop +-+- blocked +-+- validation failure +-+- new information discovered +-+- no files changed +-+- review deliverables +-+- governance deliverables +-+ +-+No exceptions. +- +- Minimum ZIP contents: +--- summary.md +-+ +-+- changed or preserved repo files from the run, stored at repo-relative paths +-+- required reports under `docs_build/dev/reports/` +- +- Optional: +-+ +-+- summary.md +- - changed-files.txt +- - findings.md +- - validation.txt +-@@ -28,11 +39,16 @@ Optional: +- ## Completion Reporting +- +- Codex responses must include: +-+ +- - ZIP filename +- - ZIP location +-+- repo-structured ZIP path +-+- reports +-+- changed file list +- - PR number(s) +- - Merge commit(s) +- - Validation results +-+- branch, worktree, and local/origin sync status when relevant +- +- ## Tool MVP PR Report Requirements +- +-@@ -82,3 +98,5 @@ unless explicitly requested. +- ## No ZIP Means Incomplete +- +- A task is not considered complete until the ZIP artifact is generated and reported. +-+ +-+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. +-diff --git a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md +-index 145d08536..bee8bdd75 100644 +---- a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md +-+++ b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md +-@@ -126,10 +126,11 @@ Closed gates: +- - 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. +-+- Every Codex execution must produce a repo-structured ZIP under `tmp/`. +-+- 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. +- - 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. +-+- 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. +-+- No exceptions. ++@@ -55,7 +55,7 @@ Codex responses must include: ++ Every tool MVP PR report must include: ++ - Product Owner testable outcome ++ - What Playwright tests ++-- What Mr. Q should manually test +++- What the Product Owner should manually test ++ - Whether the PR is part of a stacked MVP sequence ++ - Previous PR dependency ++ - Next PR dependency ++@@ -63,7 +63,7 @@ Every tool MVP PR report must include: ++ The report must answer: + +- ## Batch Governance Mode ++ ```text ++-What can Mr. Q test after applying this ZIP? +++What can the Product Owner test after applying this ZIP? ++ ``` + +-@@ -240,6 +241,35 @@ Stop only for: +- - Validation failure +- - OWNER decision ++ If a tool MVP PR has no Playwright lane, the report must state why and list the manual Product Owner validation instead. ++diff --git a/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md b/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md ++index c6d8f192c..daf35f1d3 100644 ++--- a/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md +++++ b/docs_build/dev/ProjectInstructions/addendums/codex_project_instructions_startup.md ++@@ -23,25 +23,13 @@ Codex must use this as the only active source of truth for: + +-+## Completed PR ZIP Review And Next Logical PRs +-+ +-+Whenever the Product Owner provides a completed PR ZIP to ChatGPT, ChatGPT shall: +-+ +-+- review the completed work +-+- identify completed scope +-+- identify remaining scope +-+- update the team backlog and completion percentages +-+- determine the next logical PRs +-+- recommend the execution order +-+- generate each recommended PR with: +-+ - PR summary +-+ - Codex command +-+ - commit comment +-+ - Playwright scope +-+ - manual validation steps ++ ## Branch Lifecycle Start Gate ++ ++-Every PR follows exactly three phases: +++Codex must follow the canonical lifecycle in: ++ ++ ```text ++-START ++-WORK ++-END +++docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md ++ ``` ++ ++-Codex must follow the canonical lifecycle in: ++- ++-`docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md` ++- ++-Startup enforcement: ++-- START begins on synchronized `main`. ++-- WORK remains on the PR branch. Never checkout `main`. ++-- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. ++-- STOP if current branch is `main` before commit. ++-- STOP if attempting to push `main`. ++-- STOP if a new PR starts before returning to synchronized `main`. +++This startup addendum only requires Codex to read and apply that canonical lifecycle; it must not define a competing lifecycle rule. ++ ++ Deprecated Project Instructions material outside `docs_build/dev/ProjectInstructions/` is reference-only and must not override active governance. ++ ++diff --git a/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md b/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md ++index 5a91f904c..414bf9758 100644 ++--- a/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md +++++ b/docs_build/dev/ProjectInstructions/addendums/environment_configuration_standards.md ++@@ -11,6 +11,10 @@ This addendum is governance/documentation only. It does not change runtime behav ++ ++ ## Source Model ++ +++Canonical environment/API model reference: `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md`. + + +-+Use plural wording: `next logical PRs`. +++This file owns environment variable names, URL configuration, R2 prefix configuration, and feature flag configuration only. + + +-+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. ++ This standard builds on: ++ ++ ```text ++@@ -88,12 +92,10 @@ R2 project, backup, export, import, or future storage paths must stay under the ++ ++ ## API/Service Contract Configuration ++ ++-One shared API/service contract is required across Local (VS Code), DEV, IST, UAT, and PROD. +++The canonical shared API/service contract rule lives in `docs_build/dev/ProjectInstructions/addendums/environment_governance_model.md`. ++ ++-Rules: +++This section owns configuration-only API URL requirements: ++ ++-- Browser/UI/runtime code must follow `Browser -> API -> Database` for authoritative product data. ++-- `Local API` means the same shared API/service contract running locally, not a separate local-only API implementation. ++ - API URLs may differ by `.env` only. ++ - Do not split Local API and Public API contracts. ++ - Do not create environment-specific API/service contracts. ++diff --git a/docs_build/dev/ProjectInstructions/addendums/multi_team.md b/docs_build/dev/ProjectInstructions/addendums/multi_team.md ++index 7ba2cae8d..763dccc8a 100644 ++--- a/docs_build/dev/ProjectInstructions/addendums/multi_team.md +++++ b/docs_build/dev/ProjectInstructions/addendums/multi_team.md ++@@ -1,5 +1,11 @@ ++ # Multi-Team Codex Execution Governance ++ +++Canonical workflow reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md`. + + +-+The Product Owner should not have to ask: +++Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. + + +-+- `What is next?` +-+- `Create the next PR.` +-+- `Continue.` +++This file owns multi-team execution coordination and must not create competing workflow or branch lifecycle rules. + + +-+The backlog drives the next recommended PRs automatically. ++ ## Four Active Delivery Teams ++ ++ The single authoritative four-team ownership definition is: ++@@ -65,7 +71,7 @@ Rules: ++ ## Day Work / EOD Merge Rule ++ ++ During active work: ++-- Work happens on assigned team branches, OWNER branches, or scoped PR branches. +++- Work happens on active non-main team branches or scoped PR branches. ++ - Commits are allowed only on assigned non-main branches. ++ - Pushes are allowed and expected. ++ - Draft PRs are allowed and expected. ++diff --git a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md ++index bee8bdd75..97a73ca30 100644 ++--- a/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md +++++ b/docs_build/dev/ProjectInstructions/addendums/pr_workflow.md ++@@ -46,31 +46,11 @@ This section supersedes older active wording that implies returning to `main` be ++ ++ ## Branch Lifecycle (Canonical) ++ ++-Every PR follows exactly three phases: ++- ++-```text ++-START ++-WORK ++-END ++-``` ++- ++ The canonical START, WORK, END, Daily Synchronization, and Mandatory Hard Stops rules live in: ++ ++ `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md` ++ ++-PR workflow must follow that lifecycle exactly. ++- ++-Summary: ++-- START happens on synchronized `main` only and creates the PR branch only after all required gates pass. ++-- WORK happens only on the PR branch. ++-- END validates, commits, pushes, opens/updates the PR, merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. ++-- No commits on `main`. ++-- No implementation on `main`. ++-- No validation on `main` except start validation. ++-- Never checkout `main` during WORK. ++-- STOP before commit if current branch is `main`. ++-- STOP if current branch changes unexpectedly. ++-- STOP if attempting to push `main`. +++PR workflow must follow that lifecycle exactly and must not create a competing lifecycle rule. ++ ++ ## PR Lifecycle States ++ ++@@ -153,7 +133,7 @@ Do not stop after every small PR unless blocked by branch state, failed validati ++ Each tool MVP PR plan or template must include: ++ - Product Owner testable outcome ++ - What Playwright tests ++-- What Mr. Q should manually test +++- What the Product Owner should manually test ++ - Whether the PR is part of a stacked MVP sequence ++ - Previous PR dependency ++ - Next PR dependency ++@@ -162,6 +142,8 @@ Visible acceptance must be Creator-facing first. Architecture can be handled und ++ ++ ## Product Owner Testable Definition ++ +++Canonical owner: this section is the active canonical Product Owner testable completion rule. + + +- ## EOD Main Lock ++ 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. + +- End of Day: ++ A Product Owner testable outcome means the Product Owner can: + diff --git a/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md b/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md +-index 437c01800..be3555469 100644 ++index be3555469..e6d6c7ce8 100644 + --- a/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md + +++ b/docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md +-@@ -53,6 +53,8 @@ The backlog is the authoritative source for determining the next PRs. ++@@ -42,14 +42,15 @@ Each backlog item must track: ++ - current completion percentage ++ - remaining work ++ - blocking dependencies +++- owning team ++ ++ Completion percentages are updated: ++ ++ - at SOD ++-- after each accepted PR +++- after every accepted PR ++ - at EOD ++ ++-The backlog is the authoritative source for determining the next PRs. +++The backlog is the authoritative source for determining the next logical PRs. + + If the backlog and a generated report conflict, the backlog wins unless OWNER explicitly approves a newer governance decision. + +-+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. ++diff --git a/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md b/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md ++index 2bcc52a60..5da2f9952 100644 ++--- a/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md +++++ b/docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md ++@@ -24,7 +24,7 @@ One large Codex command -> multiple focused stacked PRs -> each PR has a Product ++ - Each PR must answer: ++ ++ ```text ++-What can Mr. Q test after applying this ZIP? +++What can the Product Owner test after applying this ZIP? ++ ``` ++ ++ - Codex must continue through the stacked PR sequence unless blocked by: ++@@ -40,45 +40,14 @@ For tool MVP PR planning, visible acceptance must be Creator-facing first. ++ ++ Architecture can be handled under the covers, but PR purpose must be user-testable. ++ ++-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. ++- ++-A Product Owner testable outcome means the Product Owner can: ++- ++-- open the page/tool ++-- perform the primary workflow ++-- save/load data where applicable ++-- observe expected results ++-- validate success and failure states ++-- follow manual validation steps from the PR report ++- ++-Not acceptable as complete/testable: ++- ++-- shell-only page ++-- route-only page ++-- navigation-only PR ++-- template-only page ++-- placeholder controls ++-- static table with no workflow ++-- `Not implemented yet` ++-- `coming soon` ++-- placeholder-only workspace, inspector, or output sections ++-- planned-only tile ++-- route that loads but cannot be used ++- ++-Required for Product Owner testable completion: ++- ++-- visible working controls ++-- API-backed data where required ++-- validation and error states ++-- empty states ++-- save/load behavior where applicable ++-- manual validation steps for Product Owner ++-- targeted Playwright coverage where impacted. +++Canonical reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` owns the Product Owner testable definition and no-shell completion rule. +++ +++Canonical Playwright reference: `docs_build/dev/ProjectInstructions/addendums/test_structure_standardization.md` owns page-level Playwright organization and minimum completion coverage. ++ ++ Each tool MVP PR must state: ++ - Product Owner testable outcome ++ - What Playwright tests ++-- What Mr. Q should manually test +++- What the Product Owner should manually test ++ - Whether the PR is part of a stacked MVP sequence ++ - Previous PR dependency ++ - Next PR dependency ++diff --git a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md ++index cc9ddc88c..c0e3336a9 100644 ++--- a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md +++++ b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md ++@@ -2,19 +2,22 @@ ++ ++ ## Backlog Item Tracking Standard ++ +++Canonical backlog tracking owner: `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md`. + + +- ## Start Of Day Team Briefing ++ Every active team owns an active backlog when it has assigned work. + +- When a team is assigned, ChatGPT/Codex must provide this briefing before implementation begins: +-diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md ++-Each backlog item must track: +++Backlog entries in this file follow the canonical field list, including: ++ ++ - Name ++ - Description ++ - Current completion percentage ++ - Remaining work ++ - Blocking dependencies +++- Owning team ++ ++-Completion percentages are updated at SOD, after each accepted PR, and at EOD. +++Completion percentages are updated at SOD, after every accepted PR, and at EOD. ++ ++-The backlog is the authoritative source for determining the next PRs. +++The backlog is the authoritative source for determining the next logical PRs. ++ ++ ## Game Journey MVP ++ ++@@ -44,7 +47,7 @@ The backlog is the authoritative source for determining the next PRs. ++ - [ ] Bravo - Asset Studio V2 ++ - [ ] Charlie - Sprites canvas editor MVP ++ - Sprites is a creator tool, not only an asset metadata library. ++- - MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only reusable colors, color selection from Palette/Colors, pixel painting, save/load sprite grid data through the API/database, and Product Owner manual validation. +++ - MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. ++ - Category is removed from Sprites MVP planning. ++ - [ ] Bravo - Animation Studio V2 ++ - [ ] Bravo - Palette Manager ++@@ -65,7 +68,8 @@ The backlog is the authoritative source for determining the next PRs. ++ ++ 0% Complete — Create the things players interact with ++ ++-- [ ] Alfa - Objects +++- [ ] Charlie - Objects +++ - Objects includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. ++ - [ ] Alfa - Characters ++ - [ ] Alfa - Object Behaviors ++ ++@@ -226,6 +230,7 @@ Current OWNER clarification: ++ ++ ### Team Charlie ++ +++- Active ownership: Runtime, System Health, Environment Management, Sprites, Objects. ++ - [ ] Charlie - Guardrail hardening ++ - [ ] Charlie - Browser validation hardening ++ - [ ] Charlie - Remaining test relocation audit ++@@ -257,6 +262,10 @@ Current OWNER clarification: ++ - observability integrations ++ - [ ] Charlie - Infrastructure dashboard ++ - [ ] Charlie - Environment validation +++- [ ] Charlie - Sprites canvas editor MVP +++ - Scope: canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. +++- [ ] Charlie - Objects +++ - Scope: object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. ++ ++ ### Team Delta ++ ++diff --git a/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md b/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md ++index 4253971d9..e7826a808 100644 ++--- a/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md +++++ b/docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md ++@@ -1,5 +1,11 @@ ++ # TEAM_ASSIGNMENTS ++ +++Canonical workflow reference: `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md`. +++ +++Canonical branch lifecycle reference: `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. +++ +++This file records active assignments and registry state. Workflow summaries here are subordinate to the canonical workflow and branch lifecycle documents. +++ ++ # Active Team Registry ++ ++ | Team | Assignment | Branch | PR | Status | ++@@ -197,7 +203,7 @@ Conflict note: ++ ## Day Work / EOD Merge Rule ++ ++ During active work: ++-- Work happens on assigned team branches, OWNER branches, or scoped PR branches. +++- Work happens on active non-main team branches or scoped PR branches. ++ - Commits are allowed only on assigned non-main branches. ++ - Pushes are allowed and expected. ++ - Draft PRs are allowed and expected. ++@@ -263,23 +269,10 @@ Teams must use only `docs_build/dev/ProjectInstructions/` as the active Project ++ ++ ## Branch Lifecycle (Canonical) ++ ++-Every PR follows exactly three phases: +++Teams must follow the canonical branch lifecycle in: ++ ++ ```text ++-START ++-WORK ++-END +++docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md ++ ``` ++ ++-Teams must follow `docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md`. ++- ++-Assignment governance enforces: ++-- START begins on synchronized `main`. ++-- WORK remains on the PR branch. Never checkout `main`. ++-- END merges, returns to synchronized `main`, publishes branch, HEAD SHA, and date/time, then stops all work. ++-- No commits on `main`. ++-- No implementation on `main`. ++-- No validation on `main` except start validation. ++-- STOP if current branch is `main` before commit. ++-- STOP if attempting to push `main`. ++-- STOP if new PR work starts before returning to synchronized `main`. +++Assignment governance records active team and branch state; it must not create a competing lifecycle rule. ++diff --git a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md ++index 30b4f2dfe..fb8497c1f 100644 ++--- a/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md +++++ b/docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md ++@@ -30,11 +30,13 @@ This section is the current OWNER-approved active ownership alignment. ++ - Repository compliance ++ - Validation ++ - Infrastructure +++- Runtime ++ - Storage ++-- Environment management +++- Environment Management ++ - System Health ++ - Operations ++ - Sprites canvas editor MVP +++- Objects ++ ++ Team Charlie System Health owns: ++ - Environment Summary ++@@ -45,9 +47,18 @@ Team Charlie System Health owns: ++ ++ Team Charlie Sprites ownership: ++ - Sprites is a creator tool, not only an asset metadata library. ++-- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors-only reusable colors, color selection from Palette/Colors, pixel painting, save/load sprite grid data through the API/database, and Product Owner manual validation. +++- Sprites MVP requires canvas/grid editor behavior, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data through the API/database, and a Product Owner testable workflow. ++ - Sprites must not own reusable color definitions. ++ +++Team Charlie Objects ownership: +++- Object library +++- Object editor +++- Sprite assignment +++- Object properties +++- Collision configuration +++- Runtime object metadata +++- Object API/database contracts +++ ++ ## Team Delta ++ ++ - Engine ++diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md + new file mode 100644 +-index 000000000..bc4213d30 ++index 000000000..cbbefda41 + --- /dev/null +-+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard.md +-@@ -0,0 +1,56 @@ +-+# PR_26177_OWNER_011-codex-zip-and-next-pr-standard +++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization.md ++@@ -0,0 +1,62 @@ +++# PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + + + +Date: 2026-06-27 + +Team: OWNER +-+Branch: PR_26177_OWNER_011-codex-zip-and-next-pr-standard +++Branch: PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + + + +## Purpose + + +-+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. +++Finalize active Project Instructions cleanup, canonicalization, and backlog ownership governance. + + + +## Scope + + +@@ -158,55 +553,61 @@ index 000000000..bc4213d30 + + + +## Changes + + +-+- Strengthened the Codex artifact standard so every Codex execution produces required reports and a repo-structured ZIP. +-+- Made the ZIP/report rule apply to success, completion, partial completion, hard stop, blocked, validation failure, new information, no-change, review, and governance results. +-+- Removed the previous exception that allowed hard stops before outputs to skip ZIP/report delivery. +-+- Required Codex responses to return ZIP path, reports, changed file list, validation results, and branch/worktree/sync status when relevant. +-+- Added completed PR ZIP review governance: +-+ - review completed work +-+ - identify completed and remaining scope +-+ - update team backlog and completion percentages +-+ - determine the next logical PRs +-+ - recommend execution order +-+ - generate each recommended PR with summary, Codex command, commit comment, Playwright scope, and manual validation steps +-+- Confirmed the backlog drives next recommended PRs automatically. +-+- Preserved the existing SOD, active team branch, sequential workstream, EOD, Product Owner testable, and page-level Playwright workflow rules. +++- Added a canonical governance owner map to the Project Instructions root. +++- Made overlap documents point to canonical owners instead of creating competing active rules: +++ - workflow and Product Owner testable completion: `pr_workflow.md` +++ - branch lifecycle and EOD main lock: `project_instructions_single_source_eod_lock.md` +++ - page-level Playwright: `test_structure_standardization.md` +++ - API/environment model: `environment_governance_model.md` +++ - environment configuration: `environment_configuration_standards.md` +++ - team backlog: `team_backlog_sod_eod_standard.md` +++ - team ownership: `team_ownership.md` +++- Replaced active `Mr. Q` manual validation wording with Product Owner wording. +++- Expanded backlog item requirements with owning team. +++- Updated backlog percentage cadence to SOD, after every accepted PR, and EOD. +++- Confirmed the backlog drives the next logical PRs. +++- Updated active Team Charlie ownership for Runtime, System Health, Environment Management, Sprites, and Objects. +++- Updated Sprites MVP scope to include canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. +++- Added Objects scope for object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. + + + +## Validation + + +-+- PASS: `git diff --cached --check -- . :(exclude)docs_build/dev/reports/codex_review.diff` + +- PASS: documentation/governance-only changed-file check. +-+- PASS: ZIP-on-every-result rule exists. +-+- PASS: hard stops require ZIP/report output. +-+- PASS: `next logical PRs` plural wording exists. +-+- PASS: automatic next PR planning is tied to completed PR ZIP review. +-+- PASS: SOD starts from latest synchronized `main`. +-+- PASS: SOD creates or uses the active team branch. +-+- PASS: all commits go to the active team branch, not `main`. +-+- PASS: sequential PRs stay on the active team branch/workstream during the day. +-+- PASS: EOD returns to `main` and verifies clean sync `0 0`. +-+- PASS: Product Owner testable completion rule remains present. +-+- PASS: page-level Playwright completion gate remains present. +++- PASS: `git diff --check` + +- PASS: no runtime files changed. +++- PASS: no UI files changed. +++- PASS: no API files changed. +++- PASS: no database files changed. +++- PASS: no `start_of_day` files changed. +++- PASS: no active `Mr. Q` manual validation wording remains. +++- PASS: no OWNER-only branch workflow wording remains. +++- PASS: active `Alpha` references are limited to the preserved non-team cancellation phrase. +++- PASS: canonical owner references exist. +++- PASS: Product Owner testable canonical owner remains `pr_workflow.md`. +++- PASS: page-level Playwright canonical owner remains `test_structure_standardization.md`. +++- PASS: API/environment canonical owner remains `environment_governance_model.md`. +++- PASS: branch lifecycle canonical owner remains `project_instructions_single_source_eod_lock.md`. +++- PASS: Team Charlie owns both Sprites and Objects. +++- PASS: repo-structured ZIP produced for documentation-only changes. + + + +## Artifact + + +-+- `tmp/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_delta.zip` +++- `tmp/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_delta.zip` + + +-+## Manual Validation Notes +++## Next Logical PRs + + +-+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. +-diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md +++No additional Project Instructions cleanup PR is required from this pass. Future cleanup should wait until after the OWNER governance stack is reviewed, unless the Product Owner identifies a new conflict. ++diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md + new file mode 100644 +-index 000000000..cd2754f9f ++index 000000000..995e565ba + --- /dev/null +-+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_branch-validation.md +++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_branch-validation.md + @@ -0,0 +1,23 @@ +-+# PR_26177_OWNER_011 Branch Validation +++# PR_26177_OWNER_012 Branch Validation + + + +Date: 2026-06-27 + +Team: OWNER +-+Branch: PR_26177_OWNER_011-codex-zip-and-next-pr-standard +++Branch: PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization + + + +## Result + + +@@ -215,23 +616,23 @@ index 000000000..cd2754f9f + +## Checks + + + +- PASS: Work was performed on the active OWNER PR branch, not `main`. +-+- PASS: Branch is stacked on `PR_26177_OWNER_010-team-backlog-sod-eod-standard`. +++- PASS: Branch is stacked on `PR_26177_OWNER_011-codex-zip-and-next-pr-standard`. + +- PASS: Changes are limited to active Project Instructions and generated reports. + +- PASS: No runtime, UI, API, database, or `start_of_day` files changed. +-+- PASS: Staged whitespace validation passed with generated `codex_review.diff` excluded. +++- PASS: `git diff --check` passed before report generation. + +- PASS: Required report files were generated. + +- PASS: Repo-structured ZIP artifact path is defined under `tmp/`. + + + +## Notes + + + +This PR remains in the OWNER workstream and must not be merged to `main` until EOD closeout or explicit OWNER approval. +-diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md ++diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md + new file mode 100644 +-index 000000000..af329a4cb ++index 000000000..2b111a71f + --- /dev/null +-+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_manual-validation-notes.md +-@@ -0,0 +1,20 @@ +-+# PR_26177_OWNER_011 Manual Validation Notes +++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_manual-validation-notes.md ++@@ -0,0 +1,24 @@ +++# PR_26177_OWNER_012 Manual Validation Notes + + + +Date: 2026-06-27 + +Team: OWNER +@@ -240,24 +641,28 @@ index 000000000..af329a4cb + + + +Review these files: + + +-+- `docs_build/dev/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md` +++- `docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md` + +- `docs_build/dev/ProjectInstructions/addendums/pr_workflow.md` + +- `docs_build/dev/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md` +++- `docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md` +++- `docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md` +++ +++Confirm the canonical governance owner map is clear and that overlap files point back to canonical owner documents. + + +-+Confirm they require every Codex result to include reports and a repo-structured ZIP, including hard stops and no-change results. +++Confirm Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. + + +-+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. +++Confirm active backlog wording uses Product Owner validation language and drives the next logical PRs. + + + +## Expected Reviewer Outcome + + +-+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. +-diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md +++The Product Owner should be able to identify the canonical owner document for workflow, branch rules, Product Owner testable completion, Playwright organization, API/environment rules, team backlog, and team ownership without reading competing active rules. ++diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md + new file mode 100644 +-index 000000000..0dc03d042 ++index 000000000..d9a12c54e + --- /dev/null +-+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_requirements-checklist.md +-@@ -0,0 +1,30 @@ +-+# PR_26177_OWNER_011 Requirements Checklist +++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_requirements-checklist.md ++@@ -0,0 +1,33 @@ +++# PR_26177_OWNER_012 Requirements Checklist + + + +Date: 2026-06-27 + +Team: OWNER +@@ -265,20 +670,23 @@ index 000000000..0dc03d042 + +## Requirement Results + + + +- PASS: Documentation/governance only. +-+- PASS: Active Project Instructions reviewed and updated. +++- PASS: Active Project Instructions reviewed. + +- PASS: Active instruction files updated only. +-+- PASS: Historical snapshots were not rewritten. +++- PASS: Historical snapshots were not modified. + +- PASS: No runtime changes. +-+- PASS: ZIP-on-every-result governance added. +-+- PASS: Rule applies to success, completion, partial completion, hard stop, blocked, validation failure, new information, and no files changed. +-+- PASS: Every Codex execution must return repo-structured ZIP path, reports, changed file list, validation results, and branch/worktree/sync status when relevant. +-+- PASS: `No exceptions` wording added. +-+- PASS: Automatic next PR planning tied to Product Owner completed PR ZIP review. +-+- PASS: `next logical PRs` plural wording exists. +-+- PASS: Recommended PR output includes PR summary, Codex command, commit comment, Playwright scope, and manual validation steps. +-+- PASS: Product Owner should not need to ask `What is next?`, `Create the next PR.`, or `Continue.` +-+- PASS: Backlog drives next recommended PRs automatically. +-+- PASS: Existing SOD/main, active team branch, stacked PR, EOD, Product Owner testable, and page-level Playwright rules remain present. +++- PASS: Canonical governance owner map added. +++- PASS: Duplicated active guidance was removed where safe. +++- PASS: Append-style overlap files now point to canonical owner documents. +++- PASS: Military team spelling retained: OWNER, ALFA, BRAVO, CHARLIE, DELTA. +++- PASS: No new Greek team spelling introduced. +++- PASS: Active `Mr. Q` validation wording replaced with Product Owner wording. +++- PASS: Backlog item fields include owning team. +++- PASS: Backlog percentages update at SOD, after every accepted PR, and at EOD. +++- PASS: Backlog is authoritative for determining the next logical PRs. +++- PASS: Team Charlie owns Runtime, System Health, Environment Management, Sprites, and Objects. +++- PASS: Sprites MVP scope includes canvas/grid editor, width/height controls, Palette/Colors references only, pixel painting, save/load sprite grid data, and Product Owner testable workflow. +++- PASS: Objects scope includes object library, object editor, Sprite assignment, object properties, collision configuration, runtime object metadata, and Object API/database contracts. +++- PASS: Existing SOD, active team branch, sequential PR, no-return-to-main, and EOD clean main sync workflow remains present. + +- PASS: Required reports were generated. + +- PASS: Repo-structured ZIP artifact will be generated under `tmp/`. + + +@@ -287,13 +695,13 @@ index 000000000..0dc03d042 + +- PASS: No commit was made to `main`. + +- PASS: No `start_of_day` folders changed. + +- PASS: No unrelated cleanup. +-diff --git a/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md ++diff --git a/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md + new file mode 100644 +-index 000000000..0eae5bc5a ++index 000000000..1f8990d29 + --- /dev/null +-+++ b/docs_build/dev/reports/PR_26177_OWNER_011-codex-zip-and-next-pr-standard_validation-lane.md +-@@ -0,0 +1,43 @@ +-+# PR_26177_OWNER_011 Validation Lane +++++ b/docs_build/dev/reports/PR_26177_OWNER_012-project-instructions-cleanup-backlog-canonicalization_validation-lane.md ++@@ -0,0 +1,51 @@ +++# PR_26177_OWNER_012 Validation Lane + + + +Date: 2026-06-27 + +Team: OWNER +@@ -301,7 +709,7 @@ index 000000000..0eae5bc5a + +## Commands + + + +```powershell +-+git diff --cached --check -- . ":(exclude)docs_build/dev/reports/codex_review.diff" +++git diff --check + +``` + + + +Result: PASS +@@ -313,25 +721,33 @@ index 000000000..0eae5bc5a + +Result: PASS + + + +```powershell +-+node -e "required ZIP, next logical PRs, workflow preservation checks" +++rg -n "Mr\\. Q|What can Mr\\. Q|What Mr\\. Q" docs_build/dev/ProjectInstructions +++``` +++ +++Result: PASS, no active matches. +++ +++```powershell +++rg -n "Work must be committed only to the active OWNER branch|PR branches/commits stay on the active OWNER branch|active OWNER branch|OWNER branches|OWNER branch" docs_build/dev/ProjectInstructions +++``` +++ +++Result: PASS, no active OWNER-only branch workflow wording. +++ +++```powershell +++node -e "canonical owner, backlog, and Team Charlie ownership checks" + +``` + + + +Result: PASS + + + +## Targeted Results + + +-+- PASS: ZIP-on-every-result rule exists. +-+- PASS: hard stops require ZIP/report output. +-+- PASS: `next logical PRs` plural wording exists. +-+- PASS: automatic next PR planning is tied to completed PR ZIP review. +-+- PASS: SOD/main active branch workflow remains present. +-+- PASS: EOD clean main sync rule remains present. +-+- PASS: Product Owner testable rule remains present. +-+- PASS: page-level Playwright completion gate remains present. +++- PASS: canonical governance owner map exists. +++- PASS: Product Owner testable owner is canonicalized. +++- PASS: page-level Playwright owner is canonicalized. +++- PASS: API/environment owner is canonicalized. +++- PASS: branch lifecycle owner is canonicalized. +++- PASS: backlog includes owning team and next logical PR wording. +++- PASS: Team Charlie owns Sprites and Objects. + +- PASS: no runtime files changed. +-+- PASS: no UI files changed. +-+- PASS: no API files changed. +-+- PASS: no database files changed. + + + +## Playwright + +