Skip to content

Commit f0be6d6

Browse files
ToolboxAidToolboxAid
committed
Remove empty root shells
1 parent b8297c7 commit f0be6d6

12 files changed

Lines changed: 250 additions & 155 deletions

‎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.015
120+
- Project Instructions Version: 2026.06.28.016
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.015
3+
Current Project Instructions Version: 2026.06.28.016
44

55
Last Updated: 2026-06-28
66

@@ -23,6 +23,7 @@ Last Updated: 2026-06-28
2323
- Removed active references to obsolete legacy layout paths after the `www/`, `api/`, and `dev/` migration steps.
2424
- Moved the tracked browser-served favicon under `www/`, added remaining root/src audit governance, and documented root `.env` placement policy.
2525
- Added root leftover audit and `src/` transition plan for the final repository layout cleanup stage.
26+
- Removed safe empty local root directory shells and documented blocked local-only `assets/` cleanup.
2627
- Codex must read this version file and the latest repository copy of `PROJECT_INSTRUCTIONS.md` before performing work.
2728
- Codex must discard previously remembered Project Instructions and treat the repository copy as authoritative.
2829
- 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.015"
4+
project_state_version: "2026.06.28.016"
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.015"
14-
repository_structure_version: "2026.06.28.015"
15-
canonical_layout_version: "2026.06.28.015"
13+
project_instructions_version: "2026.06.28.016"
14+
repository_structure_version: "2026.06.28.016"
15+
canonical_layout_version: "2026.06.28.016"
1616
active_teams:
1717
- "Owner"
1818
- "Alfa"
1919
- "Bravo"
2020
- "Charlie"
2121
- "Delta"
2222
- "Golf"
23-
latest_owner_pr: "PR_26180_OWNER_015-root-empty-folder-and-src-transition-audit"
24-
latest_structure_pr: "PR_26180_OWNER_015-root-empty-folder-and-src-transition-audit"
23+
latest_owner_pr: "PR_26180_OWNER_016-remove-empty-root-shells"
24+
latest_structure_pr: "PR_26180_OWNER_016-remove-empty-root-shells"
2525
valid_top_level_folders:
2626
- "api/"
2727
- "deploy/"

‎dev/build/ProjectInstructions/backlog/BACKLOG_MASTER.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,8 @@ 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: 95%
43-
- Active PR: PR_26180_OWNER_015-root-empty-folder-and-src-transition-audit
42+
- Percent Complete: 98%
43+
- Active PR: PR_26180_OWNER_016-remove-empty-root-shells
4444
- 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

‎dev/build/ProjectInstructions/repository/repository_layout_architecture_plan.md‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,8 @@ No browser, API, server, test, script, or runtime files move in the scaffold or
4040

4141
`PR_26180_OWNER_015-root-empty-folder-and-src-transition-audit` audits named root leftovers and records the `src/` transition plan without moving application source or changing product behavior.
4242

43+
`PR_26180_OWNER_016-remove-empty-root-shells` removes safe empty local root directory shells left behind after the migration and documents the blocked local-only `assets/` shell that contains ignored user data.
44+
4345
## Proposed Future Layout
4446

4547
```text
@@ -181,7 +183,13 @@ Rules:
181183
- Produce the `src/` transition plan for the follow-up final validation stage.
182184
- Preserve product behavior.
183185

184-
12. `PR_26180_OWNER_016-final-layout-validation`
186+
12. `PR_26180_OWNER_016-remove-empty-root-shells`
187+
- Remove safe empty local root directory shells after the named root leftover audit.
188+
- Do not remove `src/`.
189+
- Do not delete ignored local-only data.
190+
- Document any folder that cannot be removed.
191+
192+
13. `PR_26180_OWNER_017-final-layout-validation`
185193
- Validate final repository layout.
186194
- Confirm `www/` owns the browser-served app, `api/` owns the server app, and `dev/` owns the developer workspace.
187195
- Confirm runtime does not depend on `dev/`, browser code does not import `api/`, and legacy references are removed or documented.

‎dev/reports/PR_26180_OWNER_016-remove-empty-root-shells_branch-validation.md‎

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# PR_26180_OWNER_016 Branch Validation
2+
3+
| Check | Result | Evidence |
4+
|---|---:|---|
5+
| Current branch | PASS | `PR_26180_OWNER_016-remove-empty-root-shells` |
6+
| Expected stacked base | PASS | Created from `PR_26180_OWNER_015-root-empty-folder-and-src-transition-audit` |
7+
| Base branch for PR | PASS | `PR_26180_OWNER_015-root-empty-folder-and-src-transition-audit` |
8+
| Project Instructions loaded | PASS | Repository Project Instructions version `2026.06.28.016` |
9+
| Branching policy loaded | PASS | `PROJECT_BRANCHING_POLICY.md` version `2026.06.28.002` |
10+
| Scope | PASS | Empty root-shell cleanup and governance reports only |
11+
| Runtime/API/database implementation changes | PASS | None |

‎dev/reports/PR_26180_OWNER_016-remove-empty-root-shells_manual-validation-notes.md‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
# PR_26180_OWNER_016 Manual Validation Notes
2+
3+
- Removed local empty shells: `games/`, `learn/`, `toolbox/`, and `tmp/`.
4+
- `test-results/` was already absent.
5+
- `assets/` remains because `assets/DemoGame-26168-001.gfsp` is ignored local data.
6+
- No tracked files were deleted.
7+
- `src/` remains untouched.
8+
- `www/favicon.svg` remains tracked and browser-served.
9+
- Canonical Codex ZIP output remains under `dev/workspace/zips/`.

‎dev/reports/PR_26180_OWNER_016-remove-empty-root-shells_report.md‎

Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
1+
# PR_26180_OWNER_016 Remove Empty Root Shells Report
2+
3+
## Executive Summary
4+
5+
PR_26180_OWNER_016 removes safe empty local root directory shells left behind after the `www/`, `api/`, and `dev/` migration. The removed shells were not tracked by Git and contained no files.
6+
7+
`assets/` was not removed because it contains ignored local data: `assets/DemoGame-26168-001.gfsp`. This is intentionally documented instead of deleted.
8+
9+
## Folder Cleanup Result
10+
11+
| Folder | Pre-Cleanup Status | Action | Post-Cleanup Status |
12+
|---|---|---|---|
13+
| `assets/` | 0 tracked files; 1 ignored file; 47 local empty subdirectories | Not removed | Still exists because `assets/DemoGame-26168-001.gfsp` is local ignored data |
14+
| `games/` | 0 tracked files; 0 files; local empty directory shell | Removed locally | Absent |
15+
| `learn/` | 0 tracked files; 0 files; local empty directory shell | Removed locally | Absent |
16+
| `toolbox/` | 0 tracked files; 0 files; local empty directory shell | Removed locally | Absent |
17+
| `tmp/` | 0 tracked files; 0 files; local empty directory shell | Removed locally | Absent |
18+
| `test-results/` | Absent | No action | Absent |
19+
20+
## Required Confirmations
21+
22+
- Root approved tracked application/workspace structure remains `www/`, `api/`, `dev/`, and `src/`.
23+
- `www/` contains browser-served content and `www/favicon.svg`.
24+
- `api/` contains server/API content.
25+
- `dev/` contains the developer workspace.
26+
- `.env` remains root/local-only and ignored.
27+
- `.env.example` remains tracked.
28+
- `src/` was not removed.
29+
- No tracked `src/` files were moved.
30+
- No runtime behavior changed.
31+
- No product code changed.
32+
33+
## Folders Not Removed
34+
35+
`assets/` could not be removed because it contains ignored local-only content. The file was not deleted because the request prohibited deleting non-empty content and required documenting folders that could not be removed.
36+
37+
## Notes
38+
39+
The removed empty shells were local filesystem cleanup only because Git does not track empty directories. The committed PR evidence is the governance report, validation evidence, and updated active workstream metadata.

‎dev/reports/PR_26180_OWNER_016-remove-empty-root-shells_requirement-checklist.md‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# PR_26180_OWNER_016 Requirement Checklist
2+
3+
| Requirement | Result | Notes |
4+
|---|---:|---|
5+
| Base on `PR_26180_OWNER_015-root-empty-folder-and-src-transition-audit` | PASS | Branch created from PR015. |
6+
| Remove safe empty root shells | PASS | Removed `games/`, `learn/`, `toolbox/`, and `tmp/` locally after confirming no files/tracked content. |
7+
| Inspect `assets/` | PASS | Not removed because it contains ignored local `assets/DemoGame-26168-001.gfsp`. |
8+
| Inspect `games/` | PASS | Removed; no tracked files or local files. |
9+
| Inspect `learn/` | PASS | Removed; no tracked files or local files. |
10+
| Inspect `toolbox/` | PASS | Removed; no tracked files or local files. |
11+
| Inspect `tmp/` | PASS | Removed; no tracked files or local files. |
12+
| Inspect `test-results/` | PASS | Already absent. |
13+
| Do not remove `src/` | PASS | `src/` remains. |
14+
| Do not move tracked `src/` files | PASS | No `src/` files moved. |
15+
| Do not modify runtime behavior | PASS | Governance/reporting only. |
16+
| Do not modify product code | PASS | No product code changed. |
17+
| Do not delete tracked files unless obsolete empty-shell related | PASS | No tracked files deleted. |
18+
| Confirm root approved structure after cleanup | PASS | Tracked application/workspace roots remain `www/`, `api/`, `dev/`, and `src/`. |
19+
| Confirm `www/` contains browser-served content | PASS | `www/` has tracked content including `www/favicon.svg`. |
20+
| Confirm `api/` contains server/API content | PASS | `api/` has tracked server/API content. |
21+
| Confirm `dev/` contains developer workspace | PASS | `dev/` has tracked developer workspace content. |
22+
| Confirm `.env` remains root/local-only | PASS | `.env` is ignored; `.env.example` is tracked. |
23+
| Document folders that could not be removed | PASS | `assets/` documented as blocked by ignored local data. |

‎dev/reports/PR_26180_OWNER_016-remove-empty-root-shells_validation-report.md‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# PR_26180_OWNER_016 Validation Report
2+
3+
| Validation | Result | Notes |
4+
|---|---:|---|
5+
| Targeted root directory scan | PASS | `games/`, `learn/`, `toolbox/`, `tmp/`, and `test-results/` absent; `assets/` blocked by ignored local data. |
6+
| Tracked legacy root scan | PASS | No tracked `assets/`, `games/`, `learn/`, `toolbox/`, `tmp/`, or `test-results/` files. |
7+
| Approved structure confirmation | PASS | `www/`, `api/`, `dev/`, and `src/` remain tracked active roots. |
8+
| `.env` confirmation | PASS | `.env` remains ignored/root-local; `.env.example` remains tracked. |
9+
| `git diff --check` | PASS | Whitespace check passed. |
10+
| `npm run validate:canonical-structure` | PASS | Blocking violations: 0. |
11+
12+
## Bootstrap Check
13+
14+
No bootstrap/startup files were changed, so a bootstrap test lane was not required for this PR.

0 commit comments

Comments
 (0)