Skip to content

Commit 91aad8f

Browse files
committed
Finalize tests and validation layout paths
1 parent 5b237c2 commit 91aad8f

101 files changed

Lines changed: 2416 additions & 526 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

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.010
120+
- Project Instructions Version: 2026.06.28.011
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.010
3+
Current Project Instructions Version: 2026.06.28.011
44

55
Last Updated: 2026-06-28
66

@@ -18,6 +18,7 @@ Last Updated: 2026-06-28
1818
- Moved browser-served application files under `www/` and made local static serving prefer `www/` by default.
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.
21+
- Finalized moved test and validation path assumptions for the `www/`, `api/`, and `dev/local-runtime/` layout.
2122
- Codex must read this version file and the latest repository copy of `PROJECT_INSTRUCTIONS.md` before performing work.
2223
- Codex must discard previously remembered Project Instructions and treat the repository copy as authoritative.
2324
- 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.010"
4+
project_state_version: "2026.06.28.011"
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.010"
14-
repository_structure_version: "2026.06.28.010"
15-
canonical_layout_version: "2026.06.28.010"
13+
project_instructions_version: "2026.06.28.011"
14+
repository_structure_version: "2026.06.28.011"
15+
canonical_layout_version: "2026.06.28.011"
1616
active_teams:
1717
- "Owner"
1818
- "Alfa"
1919
- "Bravo"
2020
- "Charlie"
2121
- "Delta"
2222
- "Golf"
23-
latest_owner_pr: "PR_26180_OWNER_010-move-dev-local-runtime"
24-
latest_structure_pr: "PR_26180_OWNER_010-move-dev-local-runtime"
23+
latest_owner_pr: "PR_26180_OWNER_011-move-tests-and-validation"
24+
latest_structure_pr: "PR_26180_OWNER_011-move-tests-and-validation"
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: 50%
43-
- Active PR: PR_26180_OWNER_010-move-dev-local-runtime
44-
- Next Milestone: Move tests and validation path references for the new layout
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
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
@@ -30,6 +30,8 @@ No browser, API, server, test, script, or runtime files move in the scaffold or
3030

3131
`PR_26180_OWNER_010-move-dev-local-runtime` moves developer-only local runtime/bootstrap entrypoints under `dev/local-runtime/` while preserving `npm run dev:bootstrap`, `npm run dev:api`, `npm run dev:web`, and `npm run dev:local-api`.
3232

33+
`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.
34+
3335
## Proposed Future Layout
3436

3537
```text
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# PR_26180_OWNER_011-move-tests-and-validation Branch Validation
2+
3+
Generated: 2026-06-28T22:57:50.322Z
4+
5+
| Check | Result | Evidence |
6+
| --- | --- | --- |
7+
| Current branch | PASS | PR_26180_OWNER_011-move-tests-and-validation |
8+
| Stacked base | PASS | PR_26180_OWNER_010-move-dev-local-runtime is an ancestor of HEAD |
9+
| PR model | PASS | Owner stacked PR under Repository Architecture Simplification workstream |
10+
| Scope | PASS | dev tests, validation script, and governance/status docs only |
11+
| Product/runtime files | PASS | No www/, api/, or product runtime behavior files changed |
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# PR_26180_OWNER_011-move-tests-and-validation Manual Validation Notes
2+
3+
Generated: 2026-06-28T22:57:50.322Z
4+
5+
- Confirmed branch is PR_26180_OWNER_011-move-tests-and-validation.
6+
- Confirmed PR011 is based on PR_26180_OWNER_010-move-dev-local-runtime.
7+
- Confirmed no www/ or api/ files were moved or edited.
8+
- Confirmed local static route smoke preserves public URLs while resolving files from the current web root.
9+
- Confirmed Project Instructions version/status metadata now records 2026.06.28.011 and PR011 as the active structure PR.
10+
- No manual browser session was required because route smoke was covered by targeted Playwright.
Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
# PR_26180_OWNER_011-move-tests-and-validation Report
2+
3+
Generated: 2026-06-28T22:57:50.322Z
4+
5+
## Purpose
6+
7+
Finalize tests and validation paths for the www/api/dev layout without moving www or api files and without changing product behavior.
8+
9+
## Summary
10+
11+
- Updated moved dev test path assumptions from old root tests layout to dev/tests depth.
12+
- Updated test-generated output/local-db paths from root tmp/ to dev/workspace/tmp/.
13+
- Updated browser-served test fixture paths from root toolbox/assets/games to www/toolbox, www/assets, and www/games.
14+
- Updated canonical structure validation to reject tracked legacy root tests/scripts/tmp/docs_build paths and old dev/local-runtime entrypoints.
15+
- Bumped Project Instructions/status metadata to 2026.06.28.011 for PR011.
16+
17+
## Scope Confirmation
18+
19+
- Runtime/product behavior changed: No.
20+
- www or api files moved: No.
21+
- Product UI files changed: No.
22+
- API/server behavior changed: No.
23+
- Test and validation path assumptions updated: Yes.
24+
25+
## Stack Dependency
26+
27+
- Previous PR dependency: PR_26180_OWNER_010-move-dev-local-runtime.
28+
- Current PR: PR_26180_OWNER_011-move-tests-and-validation.
29+
- Next PR dependency: PR_26180_OWNER_012-update-ci-and-scripts.
30+
- Merge order: PR010, then PR011, then PR012.
31+
32+
## Notes
33+
34+
The formal validation lane is the moved-path dev-runtime Node test set plus the static web-root Playwright route smoke. Direct legacy V2 runtime scripts still reference retired workspace-v2/asset-manager-v2 fixtures; this PR corrected their repository-root and artifact paths only and does not recreate retired tool fixtures.
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
# PR_26180_OWNER_011-move-tests-and-validation Requirement Checklist
2+
3+
Generated: 2026-06-28T22:57:50.322Z
4+
5+
| Requirement | Result | Notes |
6+
| --- | --- | --- |
7+
| Ensure tests live under dev/tests/ | PASS | No tracked root tests/ path remains; moved tests now resolve repo root from dev/tests. |
8+
| Update validation scripts for www/api/dev/local-runtime | PASS | Canonical validation now rejects legacy root paths and old dev/local-runtime entrypoints. |
9+
| Fix remaining path assumptions | PASS | Updated old root tmp/, toolbox/, assets/, games/, and two-level repoRoot assumptions in tests. |
10+
| Do not move www or api files | PASS | No www/api file moves were made. |
11+
| Do not change product behavior | PASS | Changes are limited to tests, validation guardrails, and governance/status docs. |
12+
| Required reports | PASS | Reports generated under dev/reports/. |
13+
| Required ZIP | PASS | Repo-structured ZIP generated under dev/workspace/zips/. |
Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
# PR_26180_OWNER_011-move-tests-and-validation Validation Report
2+
3+
Generated: 2026-06-28T22:57:50.322Z
4+
5+
| Command | Result |
6+
| --- | --- |
7+
| git diff --check | PASS |
8+
| npm run validate:canonical-structure | PASS |
9+
| node --check dev/scripts/validate-canonical-repository-structure.mjs | PASS |
10+
| node --test dev/tests/dev-runtime/ApiMenuPathCleanup.test.mjs dev/tests/dev-runtime/AdminNotesBoundary.test.mjs dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs dev/tests/dev-runtime/StaticWebRootCompatibility.test.mjs dev/tests/dev-runtime/LocalApiStartupLogging.test.mjs | PASS, 32 tests |
11+
| npx playwright test --config=dev/config/playwright.config.cjs --project=playwright dev/tests/playwright/tools/StaticWebRootCompatibility.spec.mjs | PASS, 2 tests |
12+
13+
## Non-Blocking Legacy Observation
14+
15+
Exploratory direct runtime scripts that target retired V2 workspace fixtures remain out of the required validation lane. Their repository-root and artifact paths were corrected, but this PR does not restore retired workspace-v2 or asset-manager-v2 fixtures.

0 commit comments

Comments
 (0)