Skip to content

Commit fef340d

Browse files
ToolboxAidToolboxAid
committed
Remove legacy layout references
1 parent b802afc commit fef340d

61 files changed

Lines changed: 1807 additions & 561 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.012
120+
- Project Instructions Version: 2026.06.28.013
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.012
3+
Current Project Instructions Version: 2026.06.28.013
44

55
Last Updated: 2026-06-28
66

@@ -20,6 +20,7 @@ Last Updated: 2026-06-28
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.
2222
- Updated CI, package scripts, and developer utilities for the `www/`, `api/`, and `dev/` layout.
23+
- Removed active references to obsolete legacy layout paths after the `www/`, `api/`, and `dev/` migration steps.
2324
- Codex must read this version file and the latest repository copy of `PROJECT_INSTRUCTIONS.md` before performing work.
2425
- Codex must discard previously remembered Project Instructions and treat the repository copy as authoritative.
2526
- 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.012"
4+
project_state_version: "2026.06.28.013"
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.012"
14-
repository_structure_version: "2026.06.28.012"
15-
canonical_layout_version: "2026.06.28.012"
13+
project_instructions_version: "2026.06.28.013"
14+
repository_structure_version: "2026.06.28.013"
15+
canonical_layout_version: "2026.06.28.013"
1616
active_teams:
1717
- "Owner"
1818
- "Alfa"
1919
- "Bravo"
2020
- "Charlie"
2121
- "Delta"
2222
- "Golf"
23-
latest_owner_pr: "PR_26180_OWNER_012-update-ci-and-scripts"
24-
latest_structure_pr: "PR_26180_OWNER_012-update-ci-and-scripts"
23+
latest_owner_pr: "PR_26180_OWNER_013-remove-legacy-layout"
24+
latest_structure_pr: "PR_26180_OWNER_013-remove-legacy-layout"
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: 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
42+
- Percent Complete: 80%
43+
- Active PR: PR_26180_OWNER_013-remove-legacy-layout
44+
- Next Milestone: Final layout validation and completion report
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
@@ -34,6 +34,8 @@ No browser, API, server, test, script, or runtime files move in the scaffold or
3434

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

37+
`PR_26180_OWNER_013-remove-legacy-layout` removes active stale references to obsolete legacy paths after the tracked `www/`, `api/`, and `dev/` migration steps.
38+
3739
## Proposed Future Layout
3840

3941
```text

‎dev/reports/PR_26180_OWNER_013-remove-legacy-layout_branch-validation.md‎

Lines changed: 74 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,74 @@
1+
# PR_26180_OWNER_013-remove-legacy-layout Branch Validation
2+
3+
Generated: 2026-06-28T23:33:01.682Z
4+
5+
| Check | Result | Evidence |
6+
|---|---|---|
7+
| Current branch | PASS | PR_26180_OWNER_013-remove-legacy-layout |
8+
| Expected branch | PASS | PR_26180_OWNER_013-remove-legacy-layout |
9+
| Stacked base | PASS | PR_26180_OWNER_012-update-ci-and-scripts is an ancestor of HEAD |
10+
| Worktree reviewed | PASS | Only governance, dev test/reference cleanup, reports, and one path-copy Owner page update are in scope |
11+
| Runtime behavior change | PASS | No product behavior change intended |
12+
| New www/api/dev files moved | PASS | No move of new www/api/dev application files in this PR |
13+
| Legacy folders deleted | PASS | No tracked legacy folders remained to delete; untracked/ignored local files left untouched |
14+
15+
HEAD at start of report: b802afcd33b892d33610a577b160cb096fe2e721
16+
17+
## Current Status Snapshot
18+
19+
```text
20+
M dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS.md
21+
M dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS_VERSION.md
22+
M dev/build/ProjectInstructions/PROJECT_STATE.md
23+
M dev/build/ProjectInstructions/backlog/BACKLOG_MASTER.md
24+
M dev/build/ProjectInstructions/repository/repository_layout_architecture_plan.md
25+
M dev/tests/dev-runtime/AdminNotesBoundary.test.mjs
26+
M dev/tests/dev-runtime/ApiMenuPathCleanup.test.mjs
27+
M dev/tests/final/ToolchainEngineIntegrationValidation.test.mjs
28+
M dev/tests/fixtures/games/AsteroidsValidation.test.mjs.patch
29+
M dev/tests/fixtures/tools/VectorNativeTemplate.test.mjs.patch
30+
M dev/tests/fixtures/workspace-v2/uat.manifest.json
31+
M dev/tests/helpers/playwrightV8CoverageReporter.mjs
32+
M dev/tests/helpers/testCoverageCatalog.mjs
33+
M dev/tests/playwright_installation.txt
34+
M dev/tests/production/TestsValidationCombinedPass.test.mjs
35+
M dev/tests/runtime/V2BlockFakeSessionSave.test.mjs
36+
M dev/tests/runtime/V2ClearStaleMergePreview.test.mjs
37+
M dev/tests/runtime/V2ConfirmPreviewEnableState.test.mjs
38+
M dev/tests/runtime/V2CrossToolFlow.test.mjs
39+
M dev/tests/runtime/V2CrossToolMergeBlock.test.mjs
40+
M dev/tests/runtime/V2DeterministicStateTransitions.test.mjs
41+
M dev/tests/runtime/V2DiffViewerMessaging.test.mjs
42+
M dev/tests/runtime/V2DiffViewerSummaryCounts.test.mjs
43+
M dev/tests/runtime/V2EnableStateFeedback.test.mjs
44+
M dev/tests/runtime/V2MergeConflictSummary.test.mjs
45+
M dev/tests/runtime/V2MergeOutputPersistence.test.mjs
46+
M dev/tests/runtime/V2MergePreviewOverlayFix.test.mjs
47+
M dev/tests/runtime/V2MergeResultSummary.test.mjs
48+
M dev/tests/runtime/V2MergeStateSingleSourceOfTruth.test.mjs
49+
M dev/tests/runtime/V2MergeStateStatusReset.test.mjs
50+
M dev/tests/runtime/V2MergedRecentSessionRegistration.test.mjs
51+
M dev/tests/runtime/V2SaveLibraryFromRecentSession.test.mjs
52+
M dev/tests/runtime/V2SavedSessionRowActions.test.mjs
53+
M dev/tests/runtime/V2SelectionSyncRowActions.test.mjs
54+
M dev/tests/runtime/V2SessionLibraryActionCleanup.test.mjs
55+
M dev/tests/runtime/V2SessionLibraryActionLabel.test.mjs
56+
M dev/tests/runtime/V2SessionLibraryActions.test.mjs
57+
M dev/tests/runtime/V2SessionLibraryCardOverwrite.test.mjs
58+
M dev/tests/runtime/V2SessionLibrarySaveGuard.test.mjs
59+
M dev/tests/runtime/V2SessionStateModelConsolidation.test.mjs
60+
M dev/tests/runtime/V2SessionToolsCloseoutBundle.test.mjs
61+
M dev/tests/runtime/V2SessionUxStabilization.test.mjs
62+
M dev/tests/runtime/V2ToolLaunch.test.mjs
63+
M dev/tests/runtime/V2UndoButtonRender.test.mjs
64+
M dev/tests/runtime/V2UndoEnableStateActualAvailability.test.mjs
65+
M dev/tests/runtime/V2UndoEnableStateRefresh.test.mjs
66+
M dev/tests/runtime/V2UndoLastMerge.test.mjs
67+
M dev/tests/runtime/V2WorkspaceDefaultToolInitialization.test.mjs
68+
M dev/tests/schemas/tool.manifest.schema.json
69+
M dev/tests/shared/ContractFixtureIsolationValidation.test.mjs
70+
M dev/tests/shared/ContractIndexValidation.test.mjs
71+
M dev/tests/shared/ContractNegativeCaseCoverage.test.mjs
72+
M dev/tests/tools/ToolManifestBoundary.test.mjs
73+
M www/owner/notes.html
74+
```

‎dev/reports/PR_26180_OWNER_013-remove-legacy-layout_manual-validation-notes.md‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# PR_26180_OWNER_013-remove-legacy-layout Manual Validation Notes
2+
3+
Generated: 2026-06-28T23:33:01.682Z
4+
5+
- Reviewed changed files for scope: governance/status docs, dev test/reference cleanup, validation evidence, and one Owner notes archived path reference.
6+
- Confirmed no UI redesign, API behavior change, database change, or product feature work is included.
7+
- Confirmed public URL references remain intentionally preserved for route compatibility.
8+
- Confirmed tracked obsolete root legacy folders were not present for deletion in this PR.
9+
- Confirmed untracked/ignored local legacy-looking files were not staged or committed.
10+
- Outcome ZIP is expected at dev/workspace/zips/PR_26180_OWNER_013-remove-legacy-layout_delta.zip.

‎dev/reports/PR_26180_OWNER_013-remove-legacy-layout_report.md‎

Lines changed: 104 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,104 @@
1+
# PR_26180_OWNER_013-remove-legacy-layout Report
2+
3+
Generated: 2026-06-28T23:33:01.682Z
4+
Team: Owner
5+
Mode: BUILD_PR
6+
Branch: PR_26180_OWNER_013-remove-legacy-layout
7+
Base dependency: PR_26180_OWNER_012-update-ci-and-scripts
8+
Stack order: after PR_26180_OWNER_012, before final layout validation
9+
10+
## Purpose
11+
12+
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.
13+
14+
## Scope Summary
15+
16+
- Updated Project Instructions, project state, backlog, and repository layout plan to version 2026.06.28.013 and this PR's migration status.
17+
- Retired active references to obsolete filesystem paths such as root tests/, docs_build/, root toolbox filesystem references, and dev/build/dev contract spec paths.
18+
- Updated dev test helpers and validation evidence to use canonical dev/tests, www/, and dev/build/ProjectInstructions paths.
19+
- Kept public URL compatibility references such as /toolbox, /assets, and /games intentionally intact.
20+
- Updated www/owner/notes.html only to point its Admin Notes status/help text at the archived dev location already used by the service.
21+
22+
## Files Changed
23+
24+
- dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS.md
25+
- dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS_VERSION.md
26+
- dev/build/ProjectInstructions/PROJECT_STATE.md
27+
- dev/build/ProjectInstructions/backlog/BACKLOG_MASTER.md
28+
- dev/build/ProjectInstructions/repository/repository_layout_architecture_plan.md
29+
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_branch-validation.md
30+
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_manual-validation-notes.md
31+
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_report.md
32+
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_requirement-checklist.md
33+
- dev/reports/PR_26180_OWNER_013-remove-legacy-layout_validation-report.md
34+
- dev/reports/codex_changed_files.txt
35+
- dev/reports/codex_review.diff
36+
- dev/tests/dev-runtime/AdminNotesBoundary.test.mjs
37+
- dev/tests/dev-runtime/ApiMenuPathCleanup.test.mjs
38+
- dev/tests/final/ToolchainEngineIntegrationValidation.test.mjs
39+
- dev/tests/fixtures/games/AsteroidsValidation.test.mjs.patch
40+
- dev/tests/fixtures/tools/VectorNativeTemplate.test.mjs.patch
41+
- dev/tests/fixtures/workspace-v2/uat.manifest.json
42+
- dev/tests/helpers/playwrightV8CoverageReporter.mjs
43+
- dev/tests/helpers/testCoverageCatalog.mjs
44+
- dev/tests/playwright_installation.txt
45+
- dev/tests/production/TestsValidationCombinedPass.test.mjs
46+
- dev/tests/runtime/V2BlockFakeSessionSave.test.mjs
47+
- dev/tests/runtime/V2ClearStaleMergePreview.test.mjs
48+
- dev/tests/runtime/V2ConfirmPreviewEnableState.test.mjs
49+
- dev/tests/runtime/V2CrossToolFlow.test.mjs
50+
- dev/tests/runtime/V2CrossToolMergeBlock.test.mjs
51+
- dev/tests/runtime/V2DeterministicStateTransitions.test.mjs
52+
- dev/tests/runtime/V2DiffViewerMessaging.test.mjs
53+
- dev/tests/runtime/V2DiffViewerSummaryCounts.test.mjs
54+
- dev/tests/runtime/V2EnableStateFeedback.test.mjs
55+
- dev/tests/runtime/V2MergeConflictSummary.test.mjs
56+
- dev/tests/runtime/V2MergeOutputPersistence.test.mjs
57+
- dev/tests/runtime/V2MergePreviewOverlayFix.test.mjs
58+
- dev/tests/runtime/V2MergeResultSummary.test.mjs
59+
- dev/tests/runtime/V2MergeStateSingleSourceOfTruth.test.mjs
60+
- dev/tests/runtime/V2MergeStateStatusReset.test.mjs
61+
- dev/tests/runtime/V2MergedRecentSessionRegistration.test.mjs
62+
- dev/tests/runtime/V2SaveLibraryFromRecentSession.test.mjs
63+
- dev/tests/runtime/V2SavedSessionRowActions.test.mjs
64+
- dev/tests/runtime/V2SelectionSyncRowActions.test.mjs
65+
- dev/tests/runtime/V2SessionLibraryActionCleanup.test.mjs
66+
- dev/tests/runtime/V2SessionLibraryActionLabel.test.mjs
67+
- dev/tests/runtime/V2SessionLibraryActions.test.mjs
68+
- dev/tests/runtime/V2SessionLibraryCardOverwrite.test.mjs
69+
- dev/tests/runtime/V2SessionLibrarySaveGuard.test.mjs
70+
- dev/tests/runtime/V2SessionStateModelConsolidation.test.mjs
71+
- dev/tests/runtime/V2SessionToolsCloseoutBundle.test.mjs
72+
- dev/tests/runtime/V2SessionUxStabilization.test.mjs
73+
- dev/tests/runtime/V2ToolLaunch.test.mjs
74+
- dev/tests/runtime/V2UndoButtonRender.test.mjs
75+
- dev/tests/runtime/V2UndoEnableStateActualAvailability.test.mjs
76+
- dev/tests/runtime/V2UndoEnableStateRefresh.test.mjs
77+
- dev/tests/runtime/V2UndoLastMerge.test.mjs
78+
- dev/tests/runtime/V2WorkspaceDefaultToolInitialization.test.mjs
79+
- dev/tests/schemas/tool.manifest.schema.json
80+
- dev/tests/shared/ContractFixtureIsolationValidation.test.mjs
81+
- dev/tests/shared/ContractIndexValidation.test.mjs
82+
- dev/tests/shared/ContractNegativeCaseCoverage.test.mjs
83+
- dev/tests/tools/ToolManifestBoundary.test.mjs
84+
- www/owner/notes.html
85+
86+
## Legacy Layout Decision
87+
88+
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/.
89+
90+
Local ignored or untracked legacy-looking files were left untouched and were not staged.
91+
92+
## Compatibility Notes
93+
94+
Browser public routes remain unchanged. References to /toolbox, /assets, /games, /account, /admin, and related public URLs are compatibility routes, not obsolete filesystem references.
95+
96+
## Dependency Order
97+
98+
- Previous dependency: PR_26180_OWNER_012-update-ci-and-scripts
99+
- Current PR: PR_26180_OWNER_013-remove-legacy-layout
100+
- Next dependency: final layout validation PR
101+
102+
## Owner Recommendation
103+
104+
Ready for review after validation, with no product behavior changes and no new feature scope.

‎dev/reports/PR_26180_OWNER_013-remove-legacy-layout_requirement-checklist.md‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# PR_26180_OWNER_013-remove-legacy-layout Requirement Checklist
2+
3+
Generated: 2026-06-28T23:33:01.682Z
4+
5+
| Requirement | Result | Notes |
6+
|---|---|---|
7+
| Remove or retire obsolete legacy layout paths | PASS | Active stale references were updated to canonical www/api/dev/dev-build paths. |
8+
| Confirm references are updated before removal | PASS | Targeted legacy-path scan passed for active code/config/tests/package/CI surfaces. |
9+
| Hard stop if active runtime/test/CI/package references still point to obsolete paths | PASS | No blocking active references found after cleanup. |
10+
| Preserve documented compatibility references only where intentional | PASS | Public URLs like /toolbox and /assets remain intentionally preserved. |
11+
| Do not change product behavior | PASS | Changes are reference/path cleanup and validation alignment. |
12+
| Do not move new www/api/dev files | PASS | No www/api/dev application move was performed. |
13+
| Update required reports under dev/reports | PASS | Required reports generated. |
14+
| Produce repo-structured ZIP under dev/workspace/zips | PASS | ZIP generated during closeout. |

‎dev/reports/PR_26180_OWNER_013-remove-legacy-layout_validation-report.md‎

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
# PR_26180_OWNER_013-remove-legacy-layout Validation Report
2+
3+
Generated: 2026-06-28T23:33:01.682Z
4+
5+
## Commands
6+
7+
| Command | Result |
8+
|---|---|
9+
| git diff --check | PASS |
10+
| npm run validate:canonical-structure | PASS |
11+
| npm run validate:platform | PASS |
12+
| targeted legacy-path scan | PASS |
13+
| 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 |
14+
15+
## Platform Validation
16+
17+
`npm run validate:platform` completed successfully with 8/8 scenarios passing.
18+
19+
## Canonical Structure
20+
21+
`npm run validate:canonical-structure` completed successfully with 0 blocking violations.
22+
23+
## Legacy Path Scan
24+
25+
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.
26+
27+
Result: PASS.
28+
29+
## Bootstrap Command Check
30+
31+
Covered by targeted TeamAwareBootstrap test. The public command surface remains preserved by prior PR012 and is not changed by this PR.

0 commit comments

Comments
 (0)