|
| 1 | +# PR_26180_OWNER_020-src-legacy-teardown Report |
| 2 | + |
| 3 | +## Executive Summary |
| 4 | + |
| 5 | +PR_26180_OWNER_020 continues the root `src/` retirement work by removing legacy files that no longer have active runtime, API, dev script, validation, or test references. The PR preserves active root `src/shared/contracts/**`, `src/shared/projectDataStore/**`, and active schema files because they are still used by current dev validation/tests and require separate follow-up ownership moves. |
| 6 | + |
| 7 | +No browser, API, database, or product behavior changes are introduced. |
| 8 | + |
| 9 | +## Source Of Truth |
| 10 | + |
| 11 | +- Base branch: `PR_26180_OWNER_019b-move-browser-shared-schemas-to-www` |
| 12 | +- Workstream: Repository Architecture Simplification |
| 13 | +- Audit source: PR017 src destination audit and PR019a contracts/schemas obsolescence audit |
| 14 | +- Project Instructions version updated to `2026.06.28.020` |
| 15 | + |
| 16 | +## Deleted Legacy Files |
| 17 | + |
| 18 | +| File | Decision | Reason | |
| 19 | +|------|----------|--------| |
| 20 | +| `src/dev-runtime/admin/.gitkeep` | Deleted | Obsolete placeholder. Admin compatibility runtime now lives under `www/src/dev-runtime/admin/`. | |
| 21 | +| `src/engine/README.md` | Deleted | Legacy empty-engine-era note, not an active architecture source. | |
| 22 | +| `src/shared/debug/config.js` | Deleted | Old debug helper referenced only by archived v1-v2 material. No active runtime/test/script references. | |
| 23 | +| `src/shared/debug/network.js` | Deleted | Old debug helper referenced only by archived v1-v2 material. No active runtime/test/script references. | |
| 24 | +| `src/shared/debug/noopDevConsoleIntegration.js` | Deleted | Old debug helper referenced only by archived v1-v2 material. No active runtime/test/script references. | |
| 25 | +| `src/shared/schemas/README.md` | Deleted | Legacy schema README superseded by active Project Instructions/governance. | |
| 26 | +| `src/shared/schemas/tools/README.md` | Deleted | Legacy tool schema README superseded by active Project Instructions/governance. | |
| 27 | + |
| 28 | +## Preserved Root src Files |
| 29 | + |
| 30 | +| Path | Decision | Reason | |
| 31 | +|------|----------|--------| |
| 32 | +| `src/shared/contracts/**` | Preserved | Still actively imported by `dev/tests/shared/**` contract validation. | |
| 33 | +| `src/shared/projectDataStore/**` | Preserved | Still actively imported by current project data store validation tests. | |
| 34 | +| `src/shared/schemas/game.manifest.schema.json` | Preserved | Still actively referenced by current validation/tests. | |
| 35 | +| `src/shared/schemas/samples/sample.tool-payload.schema.json` | Preserved | Still actively referenced by current validation/tests. | |
| 36 | + |
| 37 | +## Legacy Pattern Audit |
| 38 | + |
| 39 | +| Pattern | Result | |
| 40 | +|---------|--------| |
| 41 | +| `src/shared/debug/*` active references | PASS: no active references outside archive/report/workspace folders. | |
| 42 | +| `src/engine/README` active references | PASS: no active references outside archive/report/workspace folders. | |
| 43 | +| `src/shared/schemas/README` active references | PASS: no active references outside archive/report/workspace folders. | |
| 44 | +| `src/dev-runtime/admin/.gitkeep` active references | PASS: no active references outside archive/report/workspace folders. | |
| 45 | +| `www/src/tools/**` | Not present in this PR scope. | |
| 46 | +| Protected dev workspace paths | PASS: no protected `dev/workspace/generated`, `zips`, or `tmp` files moved. | |
| 47 | + |
| 48 | +Archive-only v1-v2 references to the removed debug helpers remain under `dev/archive/v1-v2/**`. They are historical reference material, not active runtime or validation dependencies. |
| 49 | + |
| 50 | +## Shared Extraction Guard |
| 51 | + |
| 52 | +`dev/tools/toolbox-dev/checkSharedExtractionGuard.baseline.json` was updated by removing the single baseline entry for deleted file `src/shared/debug/network.js`. No broad baseline regeneration was committed. |
| 53 | + |
| 54 | +## Owner Recommendation |
| 55 | + |
| 56 | +Proceed with this teardown PR. Remaining root `src/` files require follow-up PRs because active validation currently depends on them. |
0 commit comments