Skip to content

Commit c993ed7

Browse files
committed
PR_26172_MASTER_005 branch main scope governance
1 parent 0ccaf0e commit c993ed7

6 files changed

Lines changed: 101 additions & 138 deletions
Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
# Multi-Team Branch And Scope Governance
2+
3+
## Purpose
4+
5+
This addendum keeps work routed through the backlog, team assignment, and branch controls before implementation begins.
6+
7+
## Main Branch
8+
9+
Do not commit directly to `main` unless the owner explicitly says to do so.
10+
11+
Normal execution uses:
12+
1. latest `main`
13+
2. scoped branch
14+
3. validation evidence
15+
4. draft PR
16+
5. owner-controlled merge approval
17+
18+
## Backlog And Assignment Required
19+
20+
No work starts without a backlog item.
21+
22+
No work starts without a team assignment.
23+
24+
If a request asks a team to work outside its current assignment, stop and report:
25+
- assigned team
26+
- backlog location
27+
- current assignment
28+
- requested scope
29+
- recommended action
30+
31+
## Team Scope
32+
33+
Teams may work only inside their assigned ownership area unless MASTER explicitly overrides the assignment.
34+
35+
Cross-team work must be split into team-owned PRs unless MASTER explicitly approves a cross-team exception.
36+
37+
## MASTER Override
38+
39+
The required override wording is:
40+
41+
`MASTER override approved: <reason>`
42+
43+
The reason must explain the branch, assignment, backlog, or scope exception.
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# PR_26172_MASTER_005 Instruction Compliance Checklist
2+
3+
- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
4+
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
5+
- PASS: Explicit MASTER override accepted Team MASTER for this operating-system bootstrap.
6+
- PASS: Branch name mirrors PR ownership: `pr/26172-MASTER-005-branch-main-scope-governance`.
7+
- PASS: Implementation path is `docs_build/dev/ProjectInstructions/`.
8+
- PASS: Existing Project Instructions were preserved.
9+
- PASS: No direct commit to `main`.
10+
- PASS: Required review artifacts are produced.
11+
- PASS: Required ZIP path is `tmp/PR_26172_MASTER_005-branch-main-scope-governance_delta.zip`.
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
# PR_26172_MASTER_005 Manual Validation Notes
2+
3+
- Confirmed branch was created from latest `main`.
4+
- Confirmed only new files were added.
5+
- Confirmed existing Project Instructions files were not deleted or rewritten.
6+
- Confirmed no-direct-main, backlog-required, assignment-required, out-of-assignment stop, and MASTER override rules are documented.
7+
- Did not run Playwright because this is documentation-only.
8+
- Did not run samples.
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
# PR_26172_MASTER_005-branch-main-scope-governance
2+
3+
## Scope
4+
5+
Create multi-team branch and scope governance under `docs_build/dev/ProjectInstructions/addendums/`.
6+
7+
## Changes
8+
9+
- Added `addendums/multi_team.md`.
10+
- Documented no direct commits to `main` unless explicitly told.
11+
- Documented no work without backlog item and team assignment.
12+
- Documented out-of-assignment stop report fields.
13+
- Documented MASTER override wording.
14+
15+
## Validation
16+
17+
- `git diff --check`
18+
- Markdown/text review for requested branch, backlog, assignment, and override rules.
19+
- Playwright skipped: no runtime UI files changed.
20+
- Samples skipped: not requested.
21+
22+
## ZIP
23+
24+
`tmp/PR_26172_MASTER_005-branch-main-scope-governance_delta.zip`
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
2-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
3-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
1+
docs_build/dev/ProjectInstructions/addendums/multi_team.md
42
docs_build/dev/reports/codex_changed_files.txt
53
docs_build/dev/reports/codex_review.diff
6-
src/dev-runtime/persistence/mock-db-store.js
4+
docs_build/dev/reports/PR_26172_MASTER_005-branch-main-scope-governance.md
5+
docs_build/dev/reports/PR_26172_MASTER_005-branch-main-scope-governance-instruction-compliance-checklist.md
6+
docs_build/dev/reports/PR_26172_MASTER_005-branch-main-scope-governance-manual-validation-notes.md
Lines changed: 11 additions & 134 deletions
Original file line numberDiff line numberDiff line change
@@ -1,138 +1,15 @@
1-
diff --git a/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
2-
new file mode 100644
3-
index 000000000..a07705200
4-
--- /dev/null
5-
+++ b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
6-
@@ -0,0 +1,28 @@
7-
+# PR_26171_GAMMA_027 Instruction Compliance Checklist
8-
+
9-
+## Start Gate
10-
+
11-
+- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
12-
+- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
13-
+- PASS: PR name includes TEAM token `GAMMA`.
14-
+- PASS: Branch name mirrors TEAM token: `pr/26171-GAMMA-027-sqlite-doc-reference-cleanup`.
15-
+- PASS: Scope is diagnostics/cleanup and belongs to Team Gamma.
16-
+- PASS: Started from fresh `main` after PR 026 merged.
17-
+- PASS: Repository was clean before branch creation.
18-
+
19-
+## Scope Guard
20-
+
21-
+- PASS: One PR purpose only.
22-
+- PASS: Cleaned active docs/seed wording only.
23-
+- PASS: Preserved `PROJECT_INSTRUCTIONS.md` SQLite deprecation rules.
24-
+- PASS: Did not rewrite historical PR/report archives.
25-
+- PASS: Did not run samples.
26-
+
27-
+## Completion Gate
28-
+
29-
+- PASS: PR-specific report exists.
30-
+- PASS: Manual validation notes exist.
31-
+- PASS: Instruction compliance checklist exists.
32-
+- PASS: `docs_build/dev/reports/codex_review.diff` exists.
33-
+- PASS: `docs_build/dev/reports/codex_changed_files.txt` exists.
34-
+- PASS: Repo-structured ZIP path is documented: `tmp/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup_delta.zip`.
35-
diff --git a/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
36-
new file mode 100644
37-
index 000000000..415d83d27
38-
--- /dev/null
39-
+++ b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
40-
@@ -0,0 +1,17 @@
41-
+# PR_26171_GAMMA_027 Manual Validation Notes
42-
+
43-
+Manual validation focused on active docs/seed wording.
44-
+
45-
+## Notes
46-
+
47-
+- Confirmed PR 027 branch was created from fresh `main` after PR 026 merged.
48-
+- Confirmed `PROJECT_INSTRUCTIONS.md` deprecation rules were preserved.
49-
+- Confirmed historical report and PR docs were not modified.
50-
+- Confirmed the only active seed wording change was the Local DB session-mode description.
51-
+- Confirmed remaining SQLite references in active paths are governance, validation guards, negative tests, or legacy data-preservation guard text.
52-
+
53-
+## Skipped
54-
+
55-
+- Playwright was skipped because no UI behavior changed.
56-
+- Samples were skipped by request.
57-
+- Broad tests were skipped because this is a wording-only cleanup.
58-
diff --git a/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
59-
new file mode 100644
60-
index 000000000..ecb1a233d
61-
--- /dev/null
62-
+++ b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
63-
@@ -0,0 +1,43 @@
64-
+# PR_26171_GAMMA_027-sqlite-doc-reference-cleanup
65-
+
66-
+TEAM ownership: GAMMA.
67-
+
68-
+## Scope
69-
+
70-
+Clean active docs/seed wording that implied SQLite is still an active persistence target.
71-
+
72-
+## Changes
73-
+
74-
+- Updated `src/dev-runtime/persistence/mock-db-store.js` session-mode description from SQLite-backed wording to neutral Local DB adapter wording.
75-
+- Preserved `PROJECT_INSTRUCTIONS.md` SQLite deprecation rules.
76-
+- Did not rewrite historical PR/report archives.
77-
+- Did not remove governance, validation guard, negative test, or legacy data-preservation references.
78-
+
79-
+## Validation
80-
+
81-
+Passed:
82-
+- `git diff --check`
83-
+- `node --check src/dev-runtime/persistence/mock-db-store.js`
84-
+- Targeted text check confirmed no active docs/seed wording still says `SQLite-backed`, `SQLite persistence`, `SQLite service`, or `backed by server SQLite storage`.
85-
+- Scoped active-source search confirmed remaining non-archive SQLite references are limited to:
86-
+ - governance rules in `PROJECT_INSTRUCTIONS.md`
87-
+ - validation guard patterns in `scripts/validate-browser-env-agnostic.mjs`
88-
+ - negative test assertions that SQLite is not exposed
89-
+ - Game Journey legacy SQLite data-preservation guard
90-
+
91-
+Skipped:
92-
+- Playwright: no UI/runtime behavior changed.
93-
+- Samples: skipped by request.
94-
+- Historical reports and PR docs: explicitly out of scope.
95-
+
96-
+## Reports
97-
+
98-
+- `docs_build/dev/reports/codex_review.diff`
99-
+- `docs_build/dev/reports/codex_changed_files.txt`
100-
+- `docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md`
101-
+- `docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md`
102-
+- `docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md`
103-
+
104-
+## ZIP
105-
+
106-
+`tmp/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup_delta.zip`
107-
diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt
108-
index 4e23ae82e..5ebd4f6a7 100644
1+
diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt
2+
index 5ebd4f6a7..89de5eb7d 100644
1093
--- a/docs_build/dev/reports/codex_changed_files.txt
1104
+++ b/docs_build/dev/reports/codex_changed_files.txt
111-
@@ -1,10 +1,6 @@
112-
-docs_build/dev/reports/PR_26171_GAMMA_026-sqlite-test-reference-cleanup-instruction-compliance-checklist.md
113-
-docs_build/dev/reports/PR_26171_GAMMA_026-sqlite-test-reference-cleanup-manual-validation-notes.md
114-
-docs_build/dev/reports/PR_26171_GAMMA_026-sqlite-test-reference-cleanup.md
115-
+docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
116-
+docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
117-
+docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
5+
@@ -1,6 +1,6 @@
6+
-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
7+
-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
8+
-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
9+
+docs_build/dev/ProjectInstructions/addendums/multi_team.md
11810
docs_build/dev/reports/codex_changed_files.txt
11911
docs_build/dev/reports/codex_review.diff
120-
-tests/dev-runtime/DbSeedIntegrity.test.mjs
121-
-tests/playwright/tools/AdminDbViewer.spec.mjs
122-
-tests/playwright/tools/BrowserApiUrlConfig.spec.mjs
123-
-tests/playwright/tools/GameJourneyTool.spec.mjs
124-
-tests/playwright/tools/LoginSessionMode.spec.mjs
125-
+src/dev-runtime/persistence/mock-db-store.js
126-
diff --git a/src/dev-runtime/persistence/mock-db-store.js b/src/dev-runtime/persistence/mock-db-store.js
127-
index 2305910bc..c98c915b5 100644
128-
--- a/src/dev-runtime/persistence/mock-db-store.js
129-
+++ b/src/dev-runtime/persistence/mock-db-store.js
130-
@@ -23,7 +23,7 @@ export const MOCK_DB_SESSION_MODES = Object.freeze([
131-
adapterId: "local-db",
132-
adapterName: "LocalDbAdapter",
133-
configured: true,
134-
- description: "Uses LocalDbAdapter backed by server SQLite storage.",
135-
+ description: "Uses the Local DB adapter behind the server API boundary.",
136-
environment: "Local DB",
137-
id: "local-db",
138-
label: "Local DB",
12+
-src/dev-runtime/persistence/mock-db-store.js
13+
+docs_build/dev/reports/PR_26172_MASTER_005-branch-main-scope-governance.md
14+
+docs_build/dev/reports/PR_26172_MASTER_005-branch-main-scope-governance-instruction-compliance-checklist.md
15+
+docs_build/dev/reports/PR_26172_MASTER_005-branch-main-scope-governance-manual-validation-notes.md

0 commit comments

Comments
 (0)