From dbb858078f53fa06700c908275e693e8bfdcf83f Mon Sep 17 00:00:00 2001 From: DavidQ Date: Mon, 22 Jun 2026 13:11:37 -0400 Subject: [PATCH] PR_26174_ALFA_000-projectinstructions-archive-ignore --- .gitignore | 1 + ...tions-archive-ignore-branch-validation.txt | 9 ++ ...archive-ignore-manual-validation-notes.txt | 6 + ...s-archive-ignore-requirement-checklist.txt | 10 ++ ...uctions-archive-ignore-validation-lane.txt | 11 ++ ..._000-projectinstructions-archive-ignore.md | 15 ++ .../dev/reports/codex_changed_files.txt | 12 +- docs_build/dev/reports/codex_review.diff | 143 +----------------- 8 files changed, 65 insertions(+), 142 deletions(-) create mode 100644 docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-branch-validation.txt create mode 100644 docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-manual-validation-notes.txt create mode 100644 docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-requirement-checklist.txt create mode 100644 docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-validation-lane.txt create mode 100644 docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore.md diff --git a/.gitignore b/.gitignore index dcf729395..ec5f65503 100644 --- a/.gitignore +++ b/.gitignore @@ -45,3 +45,4 @@ docs/dev/reports/playwright_v8_coverage_report.txt # Game Foundry Studio project files assets/*.gfsp +docs_build/dev/ProjectInstructions.zip diff --git a/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-branch-validation.txt b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-branch-validation.txt new file mode 100644 index 000000000..fde67a5b6 --- /dev/null +++ b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-branch-validation.txt @@ -0,0 +1,9 @@ +Branch Validation: PASS + +Expected starting branch: main +Actual starting branch: main +Working branch: pr/26174-ALFA-000-projectinstructions-archive-ignore +Base pull: origin/main already up to date before branch creation +Start dirty state: `.gitignore` only, matching this PR scope +Protected instruction changes: none +Scope mismatch: none diff --git a/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-manual-validation-notes.txt b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-manual-validation-notes.txt new file mode 100644 index 000000000..4d11010db --- /dev/null +++ b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-manual-validation-notes.txt @@ -0,0 +1,6 @@ +Manual Validation Notes + +PASS - `.gitignore` now ignores `docs_build/dev/ProjectInstructions.zip`. +PASS - The generated archive can remain in the workspace without appearing in `git status`. +PASS - No Project Instructions content was modified. +PASS - No source/runtime/test implementation files were modified. diff --git a/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-requirement-checklist.txt b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-requirement-checklist.txt new file mode 100644 index 000000000..6db03fd30 --- /dev/null +++ b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-requirement-checklist.txt @@ -0,0 +1,10 @@ +Requirement Checklist: PASS + +PASS - Started from main. +PASS - Hard stop rule checked: current branch was main before branch creation. +PASS - Hard stop rule checked: only dirty file was `.gitignore`, matching this PR. +PASS - Updated `.gitignore` only, plus required reports. +PASS - Added exact line: docs_build/dev/ProjectInstructions.zip +PASS - No other file changes except required reports. +PASS - Required reports generated. +PASS - Repo-structured ZIP produced under tmp/. diff --git a/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-validation-lane.txt b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-validation-lane.txt new file mode 100644 index 000000000..3bd69581d --- /dev/null +++ b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-validation-lane.txt @@ -0,0 +1,11 @@ +Validation Lane Report: PASS + +Commands/run checks: +- `git branch --show-current` -> main before branch creation. +- `git status --short` -> `.gitignore` only before branch creation. +- `git -c http.sslBackend=schannel pull --ff-only origin main` -> already up to date. +- `Select-String -Path .gitignore -Pattern '^docs_build/dev/ProjectInstructions\.zip$'` -> exact entry found. +- `git diff -- .gitignore` -> one-line ignore entry only. +- `git diff --check` -> PASS. + +No runtime tests were required for this `.gitignore`-only BUILD_PR. diff --git a/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore.md b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore.md new file mode 100644 index 000000000..2a84dc928 --- /dev/null +++ b/docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore.md @@ -0,0 +1,15 @@ +# PR_26174_ALFA_000-projectinstructions-archive-ignore + +## Purpose + +Add `docs_build/dev/ProjectInstructions.zip` to `.gitignore` so the generated Project Instructions archive does not dirty the worktree. + +## Scope + +- Updated `.gitignore` only for executable/repository behavior. +- Added required Codex report files under `docs_build/dev/reports/`. +- ZIP artifact path: `tmp/PR_26174_ALFA_000-projectinstructions-archive-ignore_delta.zip`. + +## Validation Result + +PASS. The exact ignore entry exists once, branch context is correct, and changed files are limited to `.gitignore` plus required reports. diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt index de36ff823..9c2ec60c8 100644 --- a/docs_build/dev/reports/codex_changed_files.txt +++ b/docs_build/dev/reports/codex_changed_files.txt @@ -1,6 +1,8 @@ -docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md -docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-governance-closeout-report.md -docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-manual-validation-notes.md -docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-instruction-compliance-checklist.md -docs_build/dev/reports/codex_changed_files.txt +.gitignore docs_build/dev/reports/codex_review.diff +docs_build/dev/reports/codex_changed_files.txt +docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore.md +docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-branch-validation.txt +docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-requirement-checklist.txt +docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-validation-lane.txt +docs_build/dev/reports/PR_26174_ALFA_000-projectinstructions-archive-ignore-manual-validation-notes.txt diff --git a/docs_build/dev/reports/codex_review.diff b/docs_build/dev/reports/codex_review.diff index 9391b9ac1..74ceae1af 100644 --- a/docs_build/dev/reports/codex_review.diff +++ b/docs_build/dev/reports/codex_review.diff @@ -1,138 +1,7 @@ -diff --git a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md -index ab10302ff..09cc34579 100644 ---- a/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md -+++ b/docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md -@@ -136,3 +136,26 @@ - - Tile overlay status rules installed. - - Deprecation workflow installed. - - Team start commands installed. -+ -+- [x] OWNER - Governance hygiene initiative complete -+ -+ Notes: -+ - Workstream Hygiene governance verified. -+ - PI Closeout governance verified. -+ - GitHub Hygiene Audit governance verified. -+ - EOD Workstream Closeout governance verified. -+ - GitHub-authoritative workstream enforcement verified through push, local/origin sync, and GitHub hygiene review requirements. -+ -+- [x] OWNER - Multi-team workflow governance complete -+ -+ Notes: -+ - Sequential Codex Queue governance verified through the all-team sequential PR execution model. -+ - OWNER merge approval and EOD merge governance verified. -+ - Team ownership governance verified through the authoritative ownership map. -+ -+- [x] OWNER - Repository hygiene governance complete -+ -+ Notes: -+ - Open PR, draft PR, local branch, and remote branch review requirements verified. -+ - Recommendation-only hygiene audit process verified. -+ - Branch deletion and PR closure remain prohibited without explicit owner approval. -diff --git a/docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-governance-closeout-report.md b/docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-governance-closeout-report.md -new file mode 100644 -index 0000000000000000000000000000000000000000..25afe742506df1b222ffa8f8e3056e6f1abb2f6f -GIT binary patch -literal 2315 -zcmd5;O>^5e5bZ59`V~9lt10GVrk$}*vZ^RbYl$W0NW$F&rwVJNXKIbH+86nuo?#y` -zK5}CX9r6(9PoZ+Nx_4X@ae=!yNf#d{kF^{!m(0{~VqFKDz!CBIazVG$)#PY4Q5dkl;);)TNHMy5^gKdrbmZ!q%CR -zL;>nnUIXEY={2;JZ{k6yoQarHZ@2_=kU9JUudkb9UpC%!4WiX#)VSlgudB)k -z4KbgSDvniwkMNg6_e@Z~CdG(oNR=nwG7}0UeMK-_ZUi>>86;6t67Ugo=LrmZZp&I@ -zw-jYM07+_y#6cxjq#Cp7{B|;5dR++UU!CTCUB^glOBUatW$~0%6@yL;&kW`TRl2?V -zb2(cjovJnUSj^xnBlXY7_t`sBI(CSYl&@KBG_S1 -fY%9q}vp;6ZW$>oBdrgs&;^2FiG2cfj=UmwT(`Pdp +PR_26174_ALFA_000-projectinstructions-archive-ignore review diff -literal 0 -HcmV?d00001 - -diff --git a/docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-instruction-compliance-checklist.md b/docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-instruction-compliance-checklist.md -new file mode 100644 -index 000000000..0c4b589e4 ---- /dev/null -+++ b/docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-instruction-compliance-checklist.md -@@ -0,0 +1,22 @@ -+# Instruction Compliance Checklist -+ -+PR: $pr -+ -+- [x] Read docs_build/dev/PROJECT_INSTRUCTIONS.md. -+- [x] Read docs_build/dev/PROJECT_MULTI_PC.txt. -+- [x] Verified current branch was the approved OWNER workstream branch before PR 033 branch creation. -+- [x] Verified repository was clean/synced before PR 033 edits. -+- [x] Verified PR TEAM token is OWNER. -+- [x] Verified OWNER governance closeout scope matches TEAM ownership. -+- [x] Updated only scoped governance backlog and required reports. -+- [x] Ran git diff --check. -+- [x] Verified Workstream Hygiene governance exists. -+- [x] Verified PI Closeout governance exists. -+- [x] Verified GitHub Hygiene Audit governance exists. -+- [x] Verified EOD Workstream Closeout governance exists. -+- [x] Verified GitHub-authoritative workstream controls exist. -+- [x] Verified Sequential Codex Queue governance exists. -+- [x] Verified Alpha/Beta/Gamma ownership governance exists. -+- [x] Documented skipped Playwright and samples lanes. -+- [x] Created required Codex reports. -+- [x] Created repo-structured delta ZIP under mp/. -diff --git a/docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-manual-validation-notes.md b/docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-manual-validation-notes.md -new file mode 100644 -index 000000000..4a3661d3e ---- /dev/null -+++ b/docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-manual-validation-notes.md -@@ -0,0 +1,25 @@ -+# Manual Validation Notes -+ -+PR: $pr -+ -+## Checks Performed -+ -+- Verified Workstream Hygiene governance exists. -+- Verified PI Closeout governance exists. -+- Verified GitHub Hygiene Audit governance exists. -+- Verified EOD Workstream Closeout governance exists. -+- Verified GitHub-authoritative workstream controls through push, local/origin sync, and GitHub hygiene audit requirements. -+- Verified Sequential Codex Queue governance through the all-team sequential PR execution model. -+- Verified Alpha/Beta/Gamma ownership governance in the authoritative team ownership map and NATO-normalized active ProjectInstructions team names. -+- Confirmed no runtime code changed. -+- Confirmed no feature work was added. -+- Confirmed no branch deletion or PR closure was performed. -+ -+## Manual Result -+ -+PASS -+ -+## Skipped Validation -+ -+- Playwright was not run because no UI or runtime behavior changed. -+- Samples were not run because no samples or sample-impacting runtime changed. -diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt -index bef983eeb..de36ff823 100644 ---- a/docs_build/dev/reports/codex_changed_files.txt -+++ b/docs_build/dev/reports/codex_changed_files.txt -@@ -1,13 +1,6 @@ --docs_build/dev/PROJECT_INSTRUCTIONS.md --docs_build/dev/ProjectInstructions/addendums/multi_team.md --docs_build/dev/reports/PR_26172_OWNER_030-workstream-hygiene-governance-instruction-compliance-checklist.md --docs_build/dev/reports/PR_26172_OWNER_030-workstream-hygiene-governance-manual-validation-notes.md --docs_build/dev/reports/PR_26172_OWNER_030-workstream-hygiene-governance.md --docs_build/dev/reports/PR_26172_OWNER_031-pi-closeout-governance-instruction-compliance-checklist.md --docs_build/dev/reports/PR_26172_OWNER_031-pi-closeout-governance-manual-validation-notes.md --docs_build/dev/reports/PR_26172_OWNER_031-pi-closeout-governance.md --docs_build/dev/reports/PR_26172_OWNER_032-github-hygiene-audit-template-instruction-compliance-checklist.md --docs_build/dev/reports/PR_26172_OWNER_032-github-hygiene-audit-template-manual-validation-notes.md --docs_build/dev/reports/PR_26172_OWNER_032-github-hygiene-audit-template.md -+docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md -+docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-governance-closeout-report.md -+docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-manual-validation-notes.md -+docs_build/dev/reports/PR_26172_OWNER_033-governance-hygiene-closeout-instruction-compliance-checklist.md - docs_build/dev/reports/codex_changed_files.txt - docs_build/dev/reports/codex_review.diff +.gitignore +@@ + # Game Foundry Studio project files + assets/*.gfsp ++docs_build/dev/ProjectInstructions.zip