Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/platform-validation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,4 +20,4 @@ jobs:
node-version: "20"

- name: Run Platform Validation Suite
run: node ./dev/scripts/run-platform-validation-suite.mjs
run: npm run validate:platform
2 changes: 1 addition & 1 deletion dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS.md
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ Rules:

## Current Version/Date

- Project Instructions Version: 2026.06.28.011
- Project Instructions Version: 2026.06.28.012
- Date: 2026-06-28
- Owner: OWNER

Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Project Instructions Version

Current Project Instructions Version: 2026.06.28.011
Current Project Instructions Version: 2026.06.28.012

Last Updated: 2026-06-28

Expand All @@ -19,6 +19,7 @@ Last Updated: 2026-06-28
- Moved server/API application runtime under `api/`; browser API clients remain outside `api/`.
- Moved developer-only local runtime/bootstrap entrypoints under `dev/local-runtime/` and preserved the public npm command surface.
- Finalized moved test and validation path assumptions for the `www/`, `api/`, and `dev/local-runtime/` layout.
- Updated CI, package scripts, and developer utilities for the `www/`, `api/`, and `dev/` layout.
- Codex must read this version file and the latest repository copy of `PROJECT_INSTRUCTIONS.md` before performing work.
- Codex must discard previously remembered Project Instructions and treat the repository copy as authoritative.
- Codex must validate canonical report and ZIP paths, branching model, and legacy path avoidance before work proceeds.
12 changes: 6 additions & 6 deletions dev/build/ProjectInstructions/PROJECT_STATE.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Project State

```yaml
project_state_version: "2026.06.28.011"
project_state_version: "2026.06.28.012"
last_updated: "2026-06-28"
current_main_commit: "40de767476d70cadfd1292c916844c2f31b6f185"
repository_status:
Expand All @@ -10,18 +10,18 @@ repository_status:
runtime_database: "Postgres"
creator_metadata: "API to Postgres"
creator_assets: "API to R2"
project_instructions_version: "2026.06.28.011"
repository_structure_version: "2026.06.28.011"
canonical_layout_version: "2026.06.28.011"
project_instructions_version: "2026.06.28.012"
repository_structure_version: "2026.06.28.012"
canonical_layout_version: "2026.06.28.012"
active_teams:
- "Owner"
- "Alfa"
- "Bravo"
- "Charlie"
- "Delta"
- "Golf"
latest_owner_pr: "PR_26180_OWNER_011-move-tests-and-validation"
latest_structure_pr: "PR_26180_OWNER_011-move-tests-and-validation"
latest_owner_pr: "PR_26180_OWNER_012-update-ci-and-scripts"
latest_structure_pr: "PR_26180_OWNER_012-update-ci-and-scripts"
valid_top_level_folders:
- "api/"
- "deploy/"
Expand Down
6 changes: 3 additions & 3 deletions dev/build/ProjectInstructions/backlog/BACKLOG_MASTER.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,9 @@ The backlog is the authoritative source for determining the next logical PRs.
- Team: Owner
- Product Area: Repository Architecture Simplification
- Status: Active
- Percent Complete: 60%
- Active PR: PR_26180_OWNER_011-move-tests-and-validation
- Next Milestone: Update CI, package scripts, validation scripts, and developer utilities for the new layout
- Percent Complete: 70%
- Active PR: PR_26180_OWNER_012-update-ci-and-scripts
- Next Milestone: Remove or retire obsolete legacy paths after all references are updated
- Source / Reference: `dev/build/ProjectInstructions/repository/repository_layout_architecture_plan.md`; `dev/build/ProjectInstructions/repository/www_migration_map.md`

## Game Journey MVP
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@ No browser, API, server, test, script, or runtime files move in the scaffold or

`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.

`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.

## Proposed Future Layout

```text
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
# PR_26180_OWNER_012 Branch Validation

## Result

PASS

## Details

- Current branch: `PR_26180_OWNER_012-update-ci-and-scripts`
- Expected branch: `PR_26180_OWNER_012-update-ci-and-scripts`
- Base branch: `PR_26180_OWNER_011-move-tests-and-validation`
- PR model: Owner stacked PR
- Previous dependency: `PR_26180_OWNER_011-move-tests-and-validation`
- Worktree before report generation: contained only scoped PR012 changes

## Hard Stop Review

- Wrong branch: PASS
- Dirty unrelated worktree: PASS
- Missing Project Instructions: PASS
- Missing branching policy: PASS
- Runtime/product behavior change required: PASS
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
# PR_26180_OWNER_012 Manual Validation Notes

## Manual Review

- Confirmed `package.json` still exposes:
- `npm run dev:bootstrap`
- `npm run dev:api`
- `npm run dev:web`
- `npm run dev:local-api`
- Confirmed GitHub Actions platform validation now calls a package script instead of hardcoding the runner path.
- Confirmed developer scripts use current `www/games/` and `www/toolbox/` assumptions where they route or report moved browser-served paths.
- Confirmed no `www/` or `api/` files were moved.
- Confirmed no production page, API route, database, or product runtime behavior was modified.

## Notes For Reviewer

This PR is infrastructure-path maintenance for the layout migration stack. It should be reviewed after PR011 and before the legacy-layout retirement PR.
35 changes: 35 additions & 0 deletions dev/reports/PR_26180_OWNER_012-update-ci-and-scripts_report.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
# PR_26180_OWNER_012-update-ci-and-scripts Report

## Executive Summary

PR_26180_OWNER_012 updates CI, package scripts, and developer utilities for the current `www/`, `api/`, and `dev/` repository layout.

This PR is stacked on `PR_26180_OWNER_011-move-tests-and-validation`.

## Scope Decision

- Scope: CI, package scripts, and developer utilities only.
- No `www/` files moved.
- No `api/` files moved.
- No runtime product behavior changed.
- No production UI files changed.

## Changes

- Added `npm run validate:platform` as a package script for the platform validation suite.
- Updated the GitHub platform validation workflow to call the package script.
- Updated Node test runner absolute import aliases so `/assets/` and `/toolbox/` resolve through `www/` while `/src/` remains rooted at `src/`.
- Updated developer validation scripts to recognize `www/games/` and `www/toolbox/` paths.
- Updated tool registry validation messages to report current `www/toolbox/` locations.
- Updated Project Instructions version/status and repository architecture backlog status for PR_26180_OWNER_012.

## Dependency Order

- Previous PR: `PR_26180_OWNER_011-move-tests-and-validation`
- Current PR: `PR_26180_OWNER_012-update-ci-and-scripts`
- Next PR: `PR_26180_OWNER_013-remove-legacy-layout`
- Merge order: PR011, then PR012, then PR013.

## Owner Recommendation

Ready for review after validation passes and the draft PR is opened against `PR_26180_OWNER_011-move-tests-and-validation`.
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
# PR_26180_OWNER_012 Requirement Checklist

| Requirement | Status | Notes |
| --- | --- | --- |
| Update CI/path references for `www/`, `api/`, `dev/` | PASS | Platform workflow now uses `npm run validate:platform`; developer scripts recognize `www/` paths. |
| Update package scripts only where needed | PASS | Added `validate:platform`; preserved existing command surface. |
| Preserve `npm run dev:bootstrap` | PASS | Script remains present. |
| Preserve `npm run dev:api` | PASS | Script remains present. |
| Preserve `npm run dev:web` | PASS | Script remains present. |
| Preserve `npm run dev:local-api` | PASS | Legacy alias remains present. |
| Update developer scripts after moved paths | PASS | Node test runners, Playwright location audit, targeted lane routing, duplicate audit, and tool registry validator updated. |
| Do not move `www` files | PASS | No `www/` files moved. |
| Do not move `api` files | PASS | No `api/` files moved. |
| Do not change product behavior | PASS | Changes are CI/script/path-governance only. |
| Produce reports under `dev/reports/` | PASS | Required reports created. |
| Produce ZIP under `dev/workspace/zips/` | PASS | ZIP created during closeout. |
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
# PR_26180_OWNER_012 Validation Report

## Validation Summary

PASS

## Commands

| Command | Result | Notes |
| --- | --- | --- |
| `node --check dev/scripts/run-node-tests.mjs dev/scripts/run-node-test-files.mjs dev/scripts/audit-playwright-test-locations.mjs dev/scripts/run-targeted-test-lanes.mjs dev/scripts/validate-tool-registry.mjs dev/scripts/audit-duplicate-file-content.mjs` | PASS | Changed developer scripts parse successfully. |
| Targeted package script check | PASS | Required command names are preserved and `validate:platform` is present. |
| `npm run validate:canonical-structure` | PASS | Canonical structure validation passed. |
| `git diff --check` | PASS | Whitespace validation passed. |
| `npm run validate:platform` | PASS | Platform validation suite passed 8/8 scenarios. |
| `node --test dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs` | PASS | Team-aware bootstrap tests passed. |
| `node dev/scripts/audit-playwright-test-locations.mjs --target dev/tests/playwright/games/AsteroidsShipStateVisuals.spec.mjs --lanes games --report dev/workspace/tmp/pr012_playwright_structure_audit.md --discovery-report dev/workspace/tmp/pr012_playwright_discovery_ownership_report.md --scope-report dev/workspace/tmp/pr012_playwright_discovery_scope_report.md --scan-report dev/workspace/tmp/pr012_filesystem_scan_reduction_report.md` | PASS | Targeted Playwright route/location audit passed. |

## Playwright Decision

Browser-served routes were not changed by this PR. A targeted Playwright structure audit was run for route ownership/path assumptions; full Playwright browser execution was not required.
109 changes: 14 additions & 95 deletions dev/reports/codex_changed_files.txt
Original file line number Diff line number Diff line change
@@ -1,101 +1,20 @@
dev/build/ProjectInstructions/backlog/BACKLOG_MASTER.md
.github/workflows/platform-validation.yml
dev/build/ProjectInstructions/backlog/BACKLOG_MASTER.md
dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS.md
dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS_VERSION.md
dev/build/ProjectInstructions/PROJECT_STATE.md
dev/build/ProjectInstructions/repository/repository_layout_architecture_plan.md
dev/reports/codex_changed_files.txt
dev/reports/codex_review.diff
dev/reports/PR_26180_OWNER_011-move-tests-and-validation_branch-validation.md
dev/reports/PR_26180_OWNER_011-move-tests-and-validation_manual-validation-notes.md
dev/reports/PR_26180_OWNER_011-move-tests-and-validation_report.md
dev/reports/PR_26180_OWNER_011-move-tests-and-validation_requirement-checklist.md
dev/reports/PR_26180_OWNER_011-move-tests-and-validation_validation-report.md
dev/scripts/validate-canonical-repository-structure.mjs
dev/tests/dev-runtime/AdminNotesBoundary.test.mjs
dev/tests/dev-runtime/ApiMenuPathCleanup.test.mjs
dev/tests/dev-runtime/DbSeedIntegrity.test.mjs
dev/tests/final/ToolchainEngineIntegrationValidation.test.mjs
dev/tests/games/asteroidsManifestObjectGeometry.mjs
dev/tests/playwright/tools/AdminDbViewer.spec.mjs
dev/tests/playwright/tools/GameJourneyTool.spec.mjs
dev/tests/playwright/tools/LoginSessionMode.spec.mjs
dev/tests/playwright/tools/ThemeV2SvgIconRegistry.spec.mjs
dev/tests/runtime/GameManifestPayloadExpectations.test.mjs
dev/tests/runtime/GamesIndexWorkspaceManagerOpen.test.mjs
dev/tests/runtime/LaunchSmokeAllEntries.test.mjs
dev/tests/runtime/Phase18OverlayCssUiNormalizationSlice.test.mjs
dev/tests/runtime/V2AssetBrowserStrictJson.test.mjs
dev/tests/runtime/V2AssetManagerAddRemoveHardening.test.mjs
dev/tests/runtime/V2AssetManagerWorkspacePersistence.test.mjs
dev/tests/runtime/V2BackNav.test.mjs
dev/tests/runtime/V2BlockFakeSessionSave.test.mjs
dev/tests/runtime/V2ClearStaleMergePreview.test.mjs
dev/tests/runtime/V2ConfirmPreviewEnableState.test.mjs
dev/tests/runtime/V2CrossToolFlow.test.mjs
dev/tests/runtime/V2CrossToolMergeBlock.test.mjs
dev/tests/runtime/V2DeterministicStateTransitions.test.mjs
dev/tests/runtime/V2Diagnostics.test.mjs
dev/tests/runtime/V2DiffMergeButtonState.test.mjs
dev/tests/runtime/V2DiffViewerMessaging.test.mjs
dev/tests/runtime/V2DiffViewerSummaryCounts.test.mjs
dev/tests/runtime/V2EnableStateFeedback.test.mjs
dev/tests/runtime/V2EntryFlow.test.mjs
dev/tests/runtime/V2ErrorLogging.test.mjs
dev/tests/runtime/V2ErrorViewer.test.mjs
dev/tests/runtime/V2ImportExport.test.mjs
dev/tests/runtime/V2MergeApplyGuardAudit.test.mjs
dev/tests/runtime/V2MergeApplySessionGateFix.test.mjs
dev/tests/runtime/V2MergeConflictSummary.test.mjs
dev/tests/runtime/V2MergedRecentSessionRegistration.test.mjs
dev/tests/runtime/V2MergeOutputPersistence.test.mjs
dev/tests/runtime/V2MergePreviewOverlayFix.test.mjs
dev/tests/runtime/V2MergePreviewSelectionFix.test.mjs
dev/tests/runtime/V2MergeResultSummary.test.mjs
dev/tests/runtime/V2MergeSelectorPopulation.test.mjs
dev/tests/runtime/V2MergeStateSingleSourceOfTruth.test.mjs
dev/tests/runtime/V2MergeStateStatusReset.test.mjs
dev/tests/runtime/V2Performance.test.mjs
dev/tests/runtime/V2ProducerRender.test.mjs
dev/tests/runtime/V2RecentSessionDelete.test.mjs
dev/tests/runtime/V2RecentSessionSelectorBinding.test.mjs
dev/tests/runtime/V2ResetState.test.mjs
dev/tests/runtime/V2SavedSessionDeleteFeedback.test.mjs
dev/tests/runtime/V2SavedSessionRowActions.test.mjs
dev/tests/runtime/V2SaveLibraryFromRecentSession.test.mjs
dev/tests/runtime/V2SelectionFeedbackEnableState.test.mjs
dev/tests/runtime/V2SelectionPersistence.test.mjs
dev/tests/runtime/V2SelectionSyncRowActions.test.mjs
dev/tests/runtime/V2SessionDiff.test.mjs
dev/tests/runtime/V2SessionHistory.test.mjs
dev/tests/runtime/V2SessionIdUsability.test.mjs
dev/tests/runtime/V2SessionInventoryDiffMerge.test.mjs
dev/tests/runtime/V2SessionLibrary.test.mjs
dev/tests/runtime/V2SessionLibraryActionCleanup.test.mjs
dev/tests/runtime/V2SessionLibraryActionLabel.test.mjs
dev/tests/runtime/V2SessionLibraryActions.test.mjs
dev/tests/runtime/V2SessionLibraryCardOverwrite.test.mjs
dev/tests/runtime/V2SessionLibrarySaveGuard.test.mjs
dev/tests/runtime/V2SessionMerge.test.mjs
dev/tests/runtime/V2SessionMergePreview.test.mjs
dev/tests/runtime/V2SessionMigration.test.mjs
dev/tests/runtime/V2SessionPersistence.test.mjs
dev/tests/runtime/V2SessionProducer.test.mjs
dev/tests/runtime/V2SessionSize.test.mjs
dev/tests/runtime/V2SessionSource.test.mjs
dev/tests/runtime/V2SessionStateModelConsolidation.test.mjs
dev/tests/runtime/V2SessionStorage.test.mjs
dev/tests/runtime/V2SessionToolsCloseoutBundle.test.mjs
dev/tests/runtime/V2SessionUxStabilization.test.mjs
dev/tests/runtime/V2SessionValidation.test.mjs
dev/tests/runtime/V2SessionVersion.test.mjs
dev/tests/runtime/V2ShareLinks.test.mjs
dev/tests/runtime/V2Snapshot.test.mjs
dev/tests/runtime/V2ToolActionFlow.test.mjs
dev/tests/runtime/V2ToolLaunch.test.mjs
dev/tests/runtime/V2UndoButtonRender.test.mjs
dev/tests/runtime/V2UndoEnableStateActualAvailability.test.mjs
dev/tests/runtime/V2UndoEnableStateRefresh.test.mjs
dev/tests/runtime/V2UndoLastMerge.test.mjs
dev/tests/runtime/V2UrlState.test.mjs
dev/tests/runtime/V2WorkspaceAssetManagerLaunch.test.mjs
dev/tests/runtime/V2WorkspaceDefaultToolInitialization.test.mjs
dev/reports/PR_26180_OWNER_012-update-ci-and-scripts_branch-validation.md
dev/reports/PR_26180_OWNER_012-update-ci-and-scripts_manual-validation-notes.md
dev/reports/PR_26180_OWNER_012-update-ci-and-scripts_report.md
dev/reports/PR_26180_OWNER_012-update-ci-and-scripts_requirement-checklist.md
dev/reports/PR_26180_OWNER_012-update-ci-and-scripts_validation-report.md
dev/scripts/audit-duplicate-file-content.mjs
dev/scripts/audit-playwright-test-locations.mjs
dev/scripts/run-node-test-files.mjs
dev/scripts/run-node-tests.mjs
dev/scripts/run-targeted-test-lanes.mjs
dev/scripts/validate-tool-registry.mjs
package.json
Loading
Loading