Skip to content

Commit b802afc

Browse files
committed
Update CI and developer scripts for layout
1 parent 91aad8f commit b802afc

20 files changed

Lines changed: 476 additions & 1954 deletions

.github/workflows/platform-validation.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,4 +20,4 @@ jobs:
2020
node-version: "20"
2121

2222
- name: Run Platform Validation Suite
23-
run: node ./dev/scripts/run-platform-validation-suite.mjs
23+
run: npm run validate:platform

dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ Rules:
117117

118118
## Current Version/Date
119119

120-
- Project Instructions Version: 2026.06.28.011
120+
- Project Instructions Version: 2026.06.28.012
121121
- Date: 2026-06-28
122122
- Owner: OWNER
123123

dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS_VERSION.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Project Instructions Version
22

3-
Current Project Instructions Version: 2026.06.28.011
3+
Current Project Instructions Version: 2026.06.28.012
44

55
Last Updated: 2026-06-28
66

@@ -19,6 +19,7 @@ Last Updated: 2026-06-28
1919
- Moved server/API application runtime under `api/`; browser API clients remain outside `api/`.
2020
- Moved developer-only local runtime/bootstrap entrypoints under `dev/local-runtime/` and preserved the public npm command surface.
2121
- Finalized moved test and validation path assumptions for the `www/`, `api/`, and `dev/local-runtime/` layout.
22+
- Updated CI, package scripts, and developer utilities for the `www/`, `api/`, and `dev/` layout.
2223
- Codex must read this version file and the latest repository copy of `PROJECT_INSTRUCTIONS.md` before performing work.
2324
- Codex must discard previously remembered Project Instructions and treat the repository copy as authoritative.
2425
- Codex must validate canonical report and ZIP paths, branching model, and legacy path avoidance before work proceeds.

dev/build/ProjectInstructions/PROJECT_STATE.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Project State
22

33
```yaml
4-
project_state_version: "2026.06.28.011"
4+
project_state_version: "2026.06.28.012"
55
last_updated: "2026-06-28"
66
current_main_commit: "40de767476d70cadfd1292c916844c2f31b6f185"
77
repository_status:
@@ -10,18 +10,18 @@ repository_status:
1010
runtime_database: "Postgres"
1111
creator_metadata: "API to Postgres"
1212
creator_assets: "API to R2"
13-
project_instructions_version: "2026.06.28.011"
14-
repository_structure_version: "2026.06.28.011"
15-
canonical_layout_version: "2026.06.28.011"
13+
project_instructions_version: "2026.06.28.012"
14+
repository_structure_version: "2026.06.28.012"
15+
canonical_layout_version: "2026.06.28.012"
1616
active_teams:
1717
- "Owner"
1818
- "Alfa"
1919
- "Bravo"
2020
- "Charlie"
2121
- "Delta"
2222
- "Golf"
23-
latest_owner_pr: "PR_26180_OWNER_011-move-tests-and-validation"
24-
latest_structure_pr: "PR_26180_OWNER_011-move-tests-and-validation"
23+
latest_owner_pr: "PR_26180_OWNER_012-update-ci-and-scripts"
24+
latest_structure_pr: "PR_26180_OWNER_012-update-ci-and-scripts"
2525
valid_top_level_folders:
2626
- "api/"
2727
- "deploy/"

dev/build/ProjectInstructions/backlog/BACKLOG_MASTER.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,9 @@ The backlog is the authoritative source for determining the next logical PRs.
3939
- Team: Owner
4040
- Product Area: Repository Architecture Simplification
4141
- Status: Active
42-
- Percent Complete: 60%
43-
- Active PR: PR_26180_OWNER_011-move-tests-and-validation
44-
- Next Milestone: Update CI, package scripts, validation scripts, and developer utilities for the new layout
42+
- Percent Complete: 70%
43+
- Active PR: PR_26180_OWNER_012-update-ci-and-scripts
44+
- Next Milestone: Remove or retire obsolete legacy paths after all references are updated
4545
- Source / Reference: `dev/build/ProjectInstructions/repository/repository_layout_architecture_plan.md`; `dev/build/ProjectInstructions/repository/www_migration_map.md`
4646

4747
## Game Journey MVP

dev/build/ProjectInstructions/repository/repository_layout_architecture_plan.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,8 @@ No browser, API, server, test, script, or runtime files move in the scaffold or
3232

3333
`PR_26180_OWNER_011-move-tests-and-validation` finalizes moved test and validation path assumptions for the `www/`, `api/`, and `dev/local-runtime/` layout while keeping product behavior unchanged.
3434

35+
`PR_26180_OWNER_012-update-ci-and-scripts` updates CI, package scripts, and developer utilities for the `www/`, `api/`, and `dev/` layout while preserving the public developer command surface.
36+
3537
## Proposed Future Layout
3638

3739
```text
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
# PR_26180_OWNER_012 Branch Validation
2+
3+
## Result
4+
5+
PASS
6+
7+
## Details
8+
9+
- Current branch: `PR_26180_OWNER_012-update-ci-and-scripts`
10+
- Expected branch: `PR_26180_OWNER_012-update-ci-and-scripts`
11+
- Base branch: `PR_26180_OWNER_011-move-tests-and-validation`
12+
- PR model: Owner stacked PR
13+
- Previous dependency: `PR_26180_OWNER_011-move-tests-and-validation`
14+
- Worktree before report generation: contained only scoped PR012 changes
15+
16+
## Hard Stop Review
17+
18+
- Wrong branch: PASS
19+
- Dirty unrelated worktree: PASS
20+
- Missing Project Instructions: PASS
21+
- Missing branching policy: PASS
22+
- Runtime/product behavior change required: PASS
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
# PR_26180_OWNER_012 Manual Validation Notes
2+
3+
## Manual Review
4+
5+
- Confirmed `package.json` still exposes:
6+
- `npm run dev:bootstrap`
7+
- `npm run dev:api`
8+
- `npm run dev:web`
9+
- `npm run dev:local-api`
10+
- Confirmed GitHub Actions platform validation now calls a package script instead of hardcoding the runner path.
11+
- Confirmed developer scripts use current `www/games/` and `www/toolbox/` assumptions where they route or report moved browser-served paths.
12+
- Confirmed no `www/` or `api/` files were moved.
13+
- Confirmed no production page, API route, database, or product runtime behavior was modified.
14+
15+
## Notes For Reviewer
16+
17+
This PR is infrastructure-path maintenance for the layout migration stack. It should be reviewed after PR011 and before the legacy-layout retirement PR.
Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
# PR_26180_OWNER_012-update-ci-and-scripts Report
2+
3+
## Executive Summary
4+
5+
PR_26180_OWNER_012 updates CI, package scripts, and developer utilities for the current `www/`, `api/`, and `dev/` repository layout.
6+
7+
This PR is stacked on `PR_26180_OWNER_011-move-tests-and-validation`.
8+
9+
## Scope Decision
10+
11+
- Scope: CI, package scripts, and developer utilities only.
12+
- No `www/` files moved.
13+
- No `api/` files moved.
14+
- No runtime product behavior changed.
15+
- No production UI files changed.
16+
17+
## Changes
18+
19+
- Added `npm run validate:platform` as a package script for the platform validation suite.
20+
- Updated the GitHub platform validation workflow to call the package script.
21+
- Updated Node test runner absolute import aliases so `/assets/` and `/toolbox/` resolve through `www/` while `/src/` remains rooted at `src/`.
22+
- Updated developer validation scripts to recognize `www/games/` and `www/toolbox/` paths.
23+
- Updated tool registry validation messages to report current `www/toolbox/` locations.
24+
- Updated Project Instructions version/status and repository architecture backlog status for PR_26180_OWNER_012.
25+
26+
## Dependency Order
27+
28+
- Previous PR: `PR_26180_OWNER_011-move-tests-and-validation`
29+
- Current PR: `PR_26180_OWNER_012-update-ci-and-scripts`
30+
- Next PR: `PR_26180_OWNER_013-remove-legacy-layout`
31+
- Merge order: PR011, then PR012, then PR013.
32+
33+
## Owner Recommendation
34+
35+
Ready for review after validation passes and the draft PR is opened against `PR_26180_OWNER_011-move-tests-and-validation`.
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
# PR_26180_OWNER_012 Requirement Checklist
2+
3+
| Requirement | Status | Notes |
4+
| --- | --- | --- |
5+
| Update CI/path references for `www/`, `api/`, `dev/` | PASS | Platform workflow now uses `npm run validate:platform`; developer scripts recognize `www/` paths. |
6+
| Update package scripts only where needed | PASS | Added `validate:platform`; preserved existing command surface. |
7+
| Preserve `npm run dev:bootstrap` | PASS | Script remains present. |
8+
| Preserve `npm run dev:api` | PASS | Script remains present. |
9+
| Preserve `npm run dev:web` | PASS | Script remains present. |
10+
| Preserve `npm run dev:local-api` | PASS | Legacy alias remains present. |
11+
| Update developer scripts after moved paths | PASS | Node test runners, Playwright location audit, targeted lane routing, duplicate audit, and tool registry validator updated. |
12+
| Do not move `www` files | PASS | No `www/` files moved. |
13+
| Do not move `api` files | PASS | No `api/` files moved. |
14+
| Do not change product behavior | PASS | Changes are CI/script/path-governance only. |
15+
| Produce reports under `dev/reports/` | PASS | Required reports created. |
16+
| Produce ZIP under `dev/workspace/zips/` | PASS | ZIP created during closeout. |

0 commit comments

Comments
 (0)