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 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.012
- Project Instructions Version: 2026.06.28.013
- 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.012
Current Project Instructions Version: 2026.06.28.013

Last Updated: 2026-06-28

Expand All @@ -20,6 +20,7 @@ Last Updated: 2026-06-28
- 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.
- Removed active references to obsolete legacy layout paths after the `www/`, `api/`, and `dev/` migration steps.
- 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.012"
project_state_version: "2026.06.28.013"
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.012"
repository_structure_version: "2026.06.28.012"
canonical_layout_version: "2026.06.28.012"
project_instructions_version: "2026.06.28.013"
repository_structure_version: "2026.06.28.013"
canonical_layout_version: "2026.06.28.013"
active_teams:
- "Owner"
- "Alfa"
- "Bravo"
- "Charlie"
- "Delta"
- "Golf"
latest_owner_pr: "PR_26180_OWNER_012-update-ci-and-scripts"
latest_structure_pr: "PR_26180_OWNER_012-update-ci-and-scripts"
latest_owner_pr: "PR_26180_OWNER_013-remove-legacy-layout"
latest_structure_pr: "PR_26180_OWNER_013-remove-legacy-layout"
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: 70%
- Active PR: PR_26180_OWNER_012-update-ci-and-scripts
- Next Milestone: Remove or retire obsolete legacy paths after all references are updated
- Percent Complete: 80%
- Active PR: PR_26180_OWNER_013-remove-legacy-layout
- Next Milestone: Final layout validation and completion report
- 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 @@ -34,6 +34,8 @@ No browser, API, server, test, script, or runtime files move in the scaffold or

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

`PR_26180_OWNER_013-remove-legacy-layout` removes active stale references to obsolete legacy paths after the tracked `www/`, `api/`, and `dev/` migration steps.

## Proposed Future Layout

```text
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,74 @@
# PR_26180_OWNER_013-remove-legacy-layout Branch Validation

Generated: 2026-06-28T23:33:01.682Z

| Check | Result | Evidence |
|---|---|---|
| Current branch | PASS | PR_26180_OWNER_013-remove-legacy-layout |
| Expected branch | PASS | PR_26180_OWNER_013-remove-legacy-layout |
| Stacked base | PASS | PR_26180_OWNER_012-update-ci-and-scripts is an ancestor of HEAD |
| Worktree reviewed | PASS | Only governance, dev test/reference cleanup, reports, and one path-copy Owner page update are in scope |
| Runtime behavior change | PASS | No product behavior change intended |
| New www/api/dev files moved | PASS | No move of new www/api/dev application files in this PR |
| Legacy folders deleted | PASS | No tracked legacy folders remained to delete; untracked/ignored local files left untouched |

HEAD at start of report: b802afcd33b892d33610a577b160cb096fe2e721

## Current Status Snapshot

```text
M dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS.md
M dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS_VERSION.md
M dev/build/ProjectInstructions/PROJECT_STATE.md
M dev/build/ProjectInstructions/backlog/BACKLOG_MASTER.md
M dev/build/ProjectInstructions/repository/repository_layout_architecture_plan.md
M dev/tests/dev-runtime/AdminNotesBoundary.test.mjs
M dev/tests/dev-runtime/ApiMenuPathCleanup.test.mjs
M dev/tests/final/ToolchainEngineIntegrationValidation.test.mjs
M dev/tests/fixtures/games/AsteroidsValidation.test.mjs.patch
M dev/tests/fixtures/tools/VectorNativeTemplate.test.mjs.patch
M dev/tests/fixtures/workspace-v2/uat.manifest.json
M dev/tests/helpers/playwrightV8CoverageReporter.mjs
M dev/tests/helpers/testCoverageCatalog.mjs
M dev/tests/playwright_installation.txt
M dev/tests/production/TestsValidationCombinedPass.test.mjs
M dev/tests/runtime/V2BlockFakeSessionSave.test.mjs
M dev/tests/runtime/V2ClearStaleMergePreview.test.mjs
M dev/tests/runtime/V2ConfirmPreviewEnableState.test.mjs
M dev/tests/runtime/V2CrossToolFlow.test.mjs
M dev/tests/runtime/V2CrossToolMergeBlock.test.mjs
M dev/tests/runtime/V2DeterministicStateTransitions.test.mjs
M dev/tests/runtime/V2DiffViewerMessaging.test.mjs
M dev/tests/runtime/V2DiffViewerSummaryCounts.test.mjs
M dev/tests/runtime/V2EnableStateFeedback.test.mjs
M dev/tests/runtime/V2MergeConflictSummary.test.mjs
M dev/tests/runtime/V2MergeOutputPersistence.test.mjs
M dev/tests/runtime/V2MergePreviewOverlayFix.test.mjs
M dev/tests/runtime/V2MergeResultSummary.test.mjs
M dev/tests/runtime/V2MergeStateSingleSourceOfTruth.test.mjs
M dev/tests/runtime/V2MergeStateStatusReset.test.mjs
M dev/tests/runtime/V2MergedRecentSessionRegistration.test.mjs
M dev/tests/runtime/V2SaveLibraryFromRecentSession.test.mjs
M dev/tests/runtime/V2SavedSessionRowActions.test.mjs
M dev/tests/runtime/V2SelectionSyncRowActions.test.mjs
M dev/tests/runtime/V2SessionLibraryActionCleanup.test.mjs
M dev/tests/runtime/V2SessionLibraryActionLabel.test.mjs
M dev/tests/runtime/V2SessionLibraryActions.test.mjs
M dev/tests/runtime/V2SessionLibraryCardOverwrite.test.mjs
M dev/tests/runtime/V2SessionLibrarySaveGuard.test.mjs
M dev/tests/runtime/V2SessionStateModelConsolidation.test.mjs
M dev/tests/runtime/V2SessionToolsCloseoutBundle.test.mjs
M dev/tests/runtime/V2SessionUxStabilization.test.mjs
M dev/tests/runtime/V2ToolLaunch.test.mjs
M dev/tests/runtime/V2UndoButtonRender.test.mjs
M dev/tests/runtime/V2UndoEnableStateActualAvailability.test.mjs
M dev/tests/runtime/V2UndoEnableStateRefresh.test.mjs
M dev/tests/runtime/V2UndoLastMerge.test.mjs
M dev/tests/runtime/V2WorkspaceDefaultToolInitialization.test.mjs
M dev/tests/schemas/tool.manifest.schema.json
M dev/tests/shared/ContractFixtureIsolationValidation.test.mjs
M dev/tests/shared/ContractIndexValidation.test.mjs
M dev/tests/shared/ContractNegativeCaseCoverage.test.mjs
M dev/tests/tools/ToolManifestBoundary.test.mjs
M www/owner/notes.html
```
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
# PR_26180_OWNER_013-remove-legacy-layout Manual Validation Notes

Generated: 2026-06-28T23:33:01.682Z

- Reviewed changed files for scope: governance/status docs, dev test/reference cleanup, validation evidence, and one Owner notes archived path reference.
- Confirmed no UI redesign, API behavior change, database change, or product feature work is included.
- Confirmed public URL references remain intentionally preserved for route compatibility.
- Confirmed tracked obsolete root legacy folders were not present for deletion in this PR.
- Confirmed untracked/ignored local legacy-looking files were not staged or committed.
- Outcome ZIP is expected at dev/workspace/zips/PR_26180_OWNER_013-remove-legacy-layout_delta.zip.
104 changes: 104 additions & 0 deletions dev/reports/PR_26180_OWNER_013-remove-legacy-layout_report.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,104 @@
# PR_26180_OWNER_013-remove-legacy-layout Report

Generated: 2026-06-28T23:33:01.682Z
Team: Owner
Mode: BUILD_PR
Branch: PR_26180_OWNER_013-remove-legacy-layout
Base dependency: PR_26180_OWNER_012-update-ci-and-scripts
Stack order: after PR_26180_OWNER_012, before final layout validation

## Purpose

Remove active obsolete legacy layout references after the www/api/dev migration stack, without changing product behavior or moving the new www/api/dev files.

## Scope Summary

- Updated Project Instructions, project state, backlog, and repository layout plan to version 2026.06.28.013 and this PR's migration status.
- Retired active references to obsolete filesystem paths such as root tests/, docs_build/, root toolbox filesystem references, and dev/build/dev contract spec paths.
- Updated dev test helpers and validation evidence to use canonical dev/tests, www/, and dev/build/ProjectInstructions paths.
- Kept public URL compatibility references such as /toolbox, /assets, and /games intentionally intact.
- Updated www/owner/notes.html only to point its Admin Notes status/help text at the archived dev location already used by the service.

## Files Changed

- dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS.md
- dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS_VERSION.md
- dev/build/ProjectInstructions/PROJECT_STATE.md
- dev/build/ProjectInstructions/backlog/BACKLOG_MASTER.md
- dev/build/ProjectInstructions/repository/repository_layout_architecture_plan.md
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_branch-validation.md
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_manual-validation-notes.md
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_report.md
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_requirement-checklist.md
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_validation-report.md
- dev/reports/codex_changed_files.txt
- dev/reports/codex_review.diff
- dev/tests/dev-runtime/AdminNotesBoundary.test.mjs
- dev/tests/dev-runtime/ApiMenuPathCleanup.test.mjs
- dev/tests/final/ToolchainEngineIntegrationValidation.test.mjs
- dev/tests/fixtures/games/AsteroidsValidation.test.mjs.patch
- dev/tests/fixtures/tools/VectorNativeTemplate.test.mjs.patch
- dev/tests/fixtures/workspace-v2/uat.manifest.json
- dev/tests/helpers/playwrightV8CoverageReporter.mjs
- dev/tests/helpers/testCoverageCatalog.mjs
- dev/tests/playwright_installation.txt
- dev/tests/production/TestsValidationCombinedPass.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/V2DiffViewerMessaging.test.mjs
- dev/tests/runtime/V2DiffViewerSummaryCounts.test.mjs
- dev/tests/runtime/V2EnableStateFeedback.test.mjs
- dev/tests/runtime/V2MergeConflictSummary.test.mjs
- dev/tests/runtime/V2MergeOutputPersistence.test.mjs
- dev/tests/runtime/V2MergePreviewOverlayFix.test.mjs
- dev/tests/runtime/V2MergeResultSummary.test.mjs
- dev/tests/runtime/V2MergeStateSingleSourceOfTruth.test.mjs
- dev/tests/runtime/V2MergeStateStatusReset.test.mjs
- dev/tests/runtime/V2MergedRecentSessionRegistration.test.mjs
- dev/tests/runtime/V2SaveLibraryFromRecentSession.test.mjs
- dev/tests/runtime/V2SavedSessionRowActions.test.mjs
- dev/tests/runtime/V2SelectionSyncRowActions.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/V2SessionStateModelConsolidation.test.mjs
- dev/tests/runtime/V2SessionToolsCloseoutBundle.test.mjs
- dev/tests/runtime/V2SessionUxStabilization.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/V2WorkspaceDefaultToolInitialization.test.mjs
- dev/tests/schemas/tool.manifest.schema.json
- dev/tests/shared/ContractFixtureIsolationValidation.test.mjs
- dev/tests/shared/ContractIndexValidation.test.mjs
- dev/tests/shared/ContractNegativeCaseCoverage.test.mjs
- dev/tests/tools/ToolManifestBoundary.test.mjs
- www/owner/notes.html

## Legacy Layout Decision

No tracked root legacy folders were removed because git tracking showed no active tracked files under obsolete root locations such as docs_build/, tests/, archive/, tmp/, projects/, scripts/, project-instructions/, dev/docs_build/, dev/project-instructions/, dev/workspace/artifacts/, or dev/build/dev/.

Local ignored or untracked legacy-looking files were left untouched and were not staged.

## Compatibility Notes

Browser public routes remain unchanged. References to /toolbox, /assets, /games, /account, /admin, and related public URLs are compatibility routes, not obsolete filesystem references.

## Dependency Order

- Previous dependency: PR_26180_OWNER_012-update-ci-and-scripts
- Current PR: PR_26180_OWNER_013-remove-legacy-layout
- Next dependency: final layout validation PR

## Owner Recommendation

Ready for review after validation, with no product behavior changes and no new feature scope.
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# PR_26180_OWNER_013-remove-legacy-layout Requirement Checklist

Generated: 2026-06-28T23:33:01.682Z

| Requirement | Result | Notes |
|---|---|---|
| Remove or retire obsolete legacy layout paths | PASS | Active stale references were updated to canonical www/api/dev/dev-build paths. |
| Confirm references are updated before removal | PASS | Targeted legacy-path scan passed for active code/config/tests/package/CI surfaces. |
| Hard stop if active runtime/test/CI/package references still point to obsolete paths | PASS | No blocking active references found after cleanup. |
| Preserve documented compatibility references only where intentional | PASS | Public URLs like /toolbox and /assets remain intentionally preserved. |
| Do not change product behavior | PASS | Changes are reference/path cleanup and validation alignment. |
| Do not move new www/api/dev files | PASS | No www/api/dev application move was performed. |
| Update required reports under dev/reports | PASS | Required reports generated. |
| Produce repo-structured ZIP under dev/workspace/zips | PASS | ZIP generated during closeout. |
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
# PR_26180_OWNER_013-remove-legacy-layout Validation Report

Generated: 2026-06-28T23:33:01.682Z

## Commands

| Command | Result |
|---|---|
| git diff --check | PASS |
| npm run validate:canonical-structure | PASS |
| npm run validate:platform | PASS |
| targeted legacy-path scan | PASS |
| node --test dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs dev/tests/runtime/V2ToolLaunch.test.mjs dev/tests/shared/ContractIndexValidation.test.mjs dev/tests/dev-runtime/AdminNotesBoundary.test.mjs dev/tests/tools/ToolManifestBoundary.test.mjs dev/tests/production/TestsValidationCombinedPass.test.mjs dev/tests/shared/ContractFixtureIsolationValidation.test.mjs dev/tests/shared/ContractNegativeCaseCoverage.test.mjs dev/tests/dev-runtime/ApiMenuPathCleanup.test.mjs | PASS |

## Platform Validation

`npm run validate:platform` completed successfully with 8/8 scenarios passing.

## Canonical Structure

`npm run validate:canonical-structure` completed successfully with 0 blocking violations.

## Legacy Path Scan

Targeted scan excluded reports, archives, and generated workspace output, and checked active GitHub workflows, api, dev/scripts, dev/tests, src, www, and package.json for obsolete filesystem references.

Result: PASS.

## Bootstrap Command Check

Covered by targeted TeamAwareBootstrap test. The public command surface remains preserved by prior PR012 and is not changed by this PR.
Loading
Loading