|
| 1 | +# PR_26180_OWNER_014 Remaining Legacy Layout Cleanup Report |
| 2 | + |
| 3 | +## Executive Summary |
| 4 | + |
| 5 | +PR_26180_OWNER_014 audits the remaining tracked repository layout after the `www/`, `api/`, and `dev/` migration stack. The only tracked browser-served root file found in scope was `favicon.svg`, which is now under `www/favicon.svg` while public references remain `/favicon.svg`. |
| 6 | + |
| 7 | +The PR also documents environment file placement policy: local `.env` files remain user/environment-owned at the repository root and are never deployed; production uses deployment-managed environment variables/secrets; `www/` must not contain secrets; `api/` reads environment values from `process.env` and the deployment environment. |
| 8 | + |
| 9 | +## Scope Decision |
| 10 | + |
| 11 | +- Browser-served favicon moved from repository root to `www/`. |
| 12 | +- Public favicon URL `/favicon.svg` preserved. |
| 13 | +- `src/` retained as an active transition namespace because tracked browser, API, dev tooling, and tests still import from it. |
| 14 | +- No `.env` files moved into `www/` or `api/`. |
| 15 | +- No runtime/API/UI/database behavior changed. |
| 16 | + |
| 17 | +## Root Audit |
| 18 | + |
| 19 | +Tracked root entries are limited to standard repository metadata/config plus `api/`, `dev/`, `src/`, and `www/`. Ignored local folders/files such as local `.env*`, generated workspace outputs, and dependency folders are not part of this PR and were not staged. |
| 20 | + |
| 21 | +Additional standard root files retained as repository configuration/history: |
| 22 | + |
| 23 | +- `.codex/` |
| 24 | +- `.gitattributes` |
| 25 | +- `.githooks/` |
| 26 | +- `.gitignore` |
| 27 | +- `AGENTS.md` |
| 28 | +- `LICENSE` |
| 29 | + |
| 30 | +## src/ Audit |
| 31 | + |
| 32 | +`src/` is not empty and remains referenced by current `www/`, `api/`, `dev/tests/`, and `dev/tools/` surfaces. Current Project Instructions document `src/advanced`, `src/api`, `src/engine`, `src/shared`, `src/tools`, and `src/dev-runtime/admin` as legacy transition buckets pending explicit follow-up migration. This PR does not move or delete `src/`. |
| 33 | + |
| 34 | +## File Evidence |
| 35 | + |
| 36 | +- `www/favicon.svg`: browser-served favicon location after the move. |
| 37 | +- `dev/tests/dev-runtime/StaticWebRootCompatibility.test.mjs`: verifies `/favicon.svg` resolves to `www/favicon.svg`. |
| 38 | +- `dev/scripts/PS/deploy/WebsiteRepoDeploymentCommon.ps1`: deployment include list now references `www/favicon.svg`. |
| 39 | +- `dev/build/ProjectInstructions/repository/canonical_repository_structure.md`: documents root `.env`, `www/`, and `api/` environment ownership. |
| 40 | +- `dev/build/ProjectInstructions/addendums/environment_configuration_standards.md`: adds the runtime placement policy for environment values. |
| 41 | + |
| 42 | +## Remaining Technical Debt |
| 43 | + |
| 44 | +The PowerShell deployment helper still contains broader legacy include defaults such as root `index.html`, `games`, `samples`, `src`, and `tools`. This PR updates only the tracked favicon path because changing the broader deploy staging semantics would be a separate deployment-layout PR. |
| 45 | + |
| 46 | +## Owner Recommendation |
| 47 | + |
| 48 | +Keep this PR focused as the remaining legacy layout cleanup pass. Follow with the final layout validation PR after stacked dependencies land. |
0 commit comments