Skip to content

Commit 7d7e09c

Browse files
committed
PR_26172_MASTER_004 team assignments governance
1 parent 0ccaf0e commit 7d7e09c

7 files changed

Lines changed: 115 additions & 138 deletions
Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
# Team Assignment Governance
2+
3+
## Assignment Source
4+
5+
Teams pull work from `docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md`.
6+
7+
## Assignment Flow
8+
9+
1. Select one backlog item.
10+
2. Change the backlog status to `[.]` Building.
11+
3. Add the item under the owning team in `TEAM_ASSIGNMENTS.md`.
12+
4. Create or continue the team's active branch.
13+
5. Work only on that assignment.
14+
15+
## Limits
16+
17+
- One active assignment per team.
18+
- One active branch per team unless a MASTER override approves otherwise.
19+
- A team must not work outside its active assignment.
20+
21+
## MASTER Override
22+
23+
The required override wording is:
24+
25+
`MASTER override approved: <reason>`
26+
27+
The reason must identify the assignment, branch, scope, or ownership exception being approved.
Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
# Team Assignments
2+
3+
## Purpose
4+
5+
This file records active team assignments pulled from `BACKLOG_MASTER.md`.
6+
7+
## Rules
8+
9+
- A team pulls one item from `BACKLOG_MASTER.md`.
10+
- The backlog item is marked `[.]` when assigned.
11+
- The assigned item is placed under the owning team below.
12+
- A team may only work on its assigned item.
13+
- Each team may have one active assignment.
14+
- Each team may have one active branch unless a MASTER override explicitly approves more.
15+
16+
## Team Alpha
17+
18+
Active assignment: none.
19+
20+
## Team Beta
21+
22+
Active assignment: none.
23+
24+
## Team Gamma
25+
26+
Active assignment: none.
27+
28+
## Team MASTER
29+
30+
Active assignment: Project Instructions operating system bootstrap.
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# PR_26172_MASTER_004 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-004-team-assignments-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_004-team-assignments-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_004 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 team assignment flow and one-active-assignment/branch rules are documented.
7+
- Did not run Playwright because this is documentation-only.
8+
- Did not run samples.
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
# PR_26172_MASTER_004-team-assignments-governance
2+
3+
## Scope
4+
5+
Create team assignment governance under `docs_build/dev/ProjectInstructions/`.
6+
7+
## Changes
8+
9+
- Added `team_assignments/TEAM_ASSIGNMENTS.md`.
10+
- Added `addendums/team_assignments.md`.
11+
- Documented backlog pull, building status, team placement, one active assignment, one active branch, and MASTER override rules.
12+
13+
## Validation
14+
15+
- `git diff --check`
16+
- Markdown/text review for requested assignment rules.
17+
- Playwright skipped: no runtime UI files changed.
18+
- Samples skipped: not requested.
19+
20+
## ZIP
21+
22+
`tmp/PR_26172_MASTER_004-team-assignments-governance_delta.zip`
Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
1-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
2-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
3-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
1+
docs_build/dev/ProjectInstructions/addendums/team_assignments.md
2+
docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md
43
docs_build/dev/reports/codex_changed_files.txt
54
docs_build/dev/reports/codex_review.diff
6-
src/dev-runtime/persistence/mock-db-store.js
5+
docs_build/dev/reports/PR_26172_MASTER_004-team-assignments-governance.md
6+
docs_build/dev/reports/PR_26172_MASTER_004-team-assignments-governance-instruction-compliance-checklist.md
7+
docs_build/dev/reports/PR_26172_MASTER_004-team-assignments-governance-manual-validation-notes.md
Lines changed: 12 additions & 134 deletions
Original file line numberDiff line numberDiff line change
@@ -1,138 +1,16 @@
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..cc436c9c3 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,7 @@
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/team_assignments.md
10+
+docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md
11811
docs_build/dev/reports/codex_changed_files.txt
11912
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",
13+
-src/dev-runtime/persistence/mock-db-store.js
14+
+docs_build/dev/reports/PR_26172_MASTER_004-team-assignments-governance.md
15+
+docs_build/dev/reports/PR_26172_MASTER_004-team-assignments-governance-instruction-compliance-checklist.md
16+
+docs_build/dev/reports/PR_26172_MASTER_004-team-assignments-governance-manual-validation-notes.md

0 commit comments

Comments
 (0)