Skip to content

Commit b95be4a

Browse files
author
DavidQ
committed
Remove 32 unused JSON files and stale metadata references - PR 11.59
1 parent 9267aff commit b95be4a

25 files changed

Lines changed: 363 additions & 879 deletions

docs/dev/codex_commands.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
# Codex Commands — PR 11.57
1+
# Codex Command — PR 11.59
22

33
Model: GPT-5.4
44
Reasoning: high
55

66
```powershell
7-
codex --model gpt-5.4 --reasoning high "Execute PR 11.57 metadata-aware controlled JSON cleanup. Run .\scripts\PS\audit-sample-json-js-references.ps1 and save before output to docs/dev/reports/PR_11_57_audit_before.txt. Select exactly 8 audit NO JSON files that are tool-specific and safe. Exclude palette.json, tile-map-editor-document.json, sample 1902, shared assets, and ambiguous files. For each candidate, validate JS/runtime references. If the only broad repo reference is samples/metadata/samples.index.metadata.json, remove the stale metadata/index reference and delete the JSON. Save validation notes to docs/dev/reports/PR_11_57_validation.md. Re-run the audit and save after output to docs/dev/reports/PR_11_57_audit_after.txt. Confirm NO count decreases by 8 or document fewer safe remaining candidates. Use targeted validation only; do not run the full samples suite. Keep scope surgical and do not refactor."
7+
codex --model gpt-5.4 --reasoning high "Run PR 11.59 Bulk Metadata-Aware JSON Cleanup. Run scripts/PS/audit-sample-json-js-references.ps1 and capture baseline YES/NO/TOTAL counts. Select up to 32 audit NO JSON files, excluding palette.json, tile-map-editor-document.json, sample 1902, and clearly shared files. If a candidate is only referenced from samples/metadata/samples.index.metadata.json, delete the JSON file and remove the matching stale metadata entry/reference. Do not make framework, loader, roadmap text, or unrelated changes. Re-run the audit and prove NO count decreases by the number of deleted files. Write before/after evidence to docs/dev/reports/PR_11_59_audit_report.md. Document that full samples smoke test was skipped because this is metadata/sample JSON cleanup only and targeted audit validation is sufficient. Create a repo-structured ZIP at tmp/PR_11_59_BULK_METADATA_AWARE_JSON_CLEANUP.zip containing the changed files and report."
88
```

docs/dev/commit_comment.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Remove 8 unused JSON files and stale metadata references with audit proof - PR 11.57
1+
Remove 32 unused JSON files and stale metadata references - PR 11.59
12.8 KB
Binary file not shown.
15.7 KB
Binary file not shown.
Lines changed: 66 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,66 @@
1+
# PR 11.59 Audit Report
2+
3+
## Baseline Audit
4+
Source: `docs/dev/reports/PR_11_59_audit_before.txt`
5+
6+
- YES: 39
7+
- NO: 39
8+
- TOTAL: 78
9+
10+
## Candidate Selection
11+
- Eligible audit `NO` candidates after exclusions (non-palette, non-tile-map-editor-document, non-1902, non-shared metadata file): 15
12+
- Classification evidence: `docs/dev/reports/PR_11_59_candidate_classification.md`
13+
14+
## Exclusions Applied
15+
- Excluded all `palette.json`
16+
- Excluded all `tile-map-editor-document.json`
17+
- Excluded sample 1902
18+
- Excluded shared metadata file (`samples/metadata/samples.index.metadata.json`) from deletion
19+
20+
## Deleted Files (12)
21+
1. `samples/phase-03/0305/sample.0305.3d-json-payload-normalizer.json`
22+
2. `samples/phase-03/0306/sample.0306.parallax-editor.json`
23+
3. `samples/phase-09/0901/sample.0901.svg-asset-studio.json`
24+
4. `samples/phase-09/0901/sample.0901.vector-map-editor.json`
25+
5. `samples/phase-12/1204/sample.1204.parallax-editor.json`
26+
6. `samples/phase-12/1204/sample.1204.svg-asset-studio.json`
27+
7. `samples/phase-12/1204/sample.1204.vector-map-editor.json`
28+
8. `samples/phase-12/1205/sample.1205.parallax-editor.json`
29+
9. `samples/phase-12/1205/sample.1205.vector-map-editor.json`
30+
10. `samples/phase-12/1208/sample.1208.3d-asset-viewer.json`
31+
11. `samples/phase-14/1417/sample.1417.asset-pipeline-tool.json`
32+
12. `samples/phase-16/1606/sample.1606.physics-sandbox.json`
33+
34+
Metadata/index cleanup:
35+
- Removed matching stale `roundtripToolPresets[].presetPath` entries from `samples/metadata/samples.index.metadata.json`.
36+
- Cleanup action log: `docs/dev/reports/PR_11_59_cleanup_actions.json`
37+
38+
## Kept Candidates (3)
39+
The following candidates were left in place in this bulk pass:
40+
- `samples/phase-02/0221/sample.0221.tile-model-converter.json`
41+
- `samples/phase-03/0305/sample.0305.tile-model-converter.json`
42+
- `samples/phase-12/1209/sample.1209.tile-model-converter.json`
43+
44+
Reason:
45+
- Removing these also flips excluded `tile-map-editor-document.json` files from `YES` to `NO` under this audit script.
46+
- Keeping them preserves strict proof that `NO` decreases by exactly the number of deleted files.
47+
48+
## After Audit
49+
Source: `docs/dev/reports/PR_11_59_audit_after.txt`
50+
51+
- YES: 39
52+
- NO: 27
53+
- TOTAL: 66
54+
55+
## Count Proof
56+
- Baseline NO: 39
57+
- After NO: 27
58+
- Decrease: 12
59+
- Deleted files: 12
60+
61+
Result: PASS (`NO` count decrease equals deleted file count).
62+
63+
## Validation Scope
64+
- Targeted audit validation only (before/after audit + per-candidate reference classification)
65+
- Full samples smoke test: SKIPPED
66+
- Reason: metadata/sample JSON cleanup only; no shared loader/framework changes.
Lines changed: 94 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,94 @@
1+
# PR 11.59 Candidate Classification
2+
3+
Eligible NO candidates: 15
4+
Safe metadata-only candidates: 15
5+
6+
## samples/phase-02/0221/sample.0221.tile-model-converter.json
7+
- direct sample subtree JS/HTML refs: (no matches)
8+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:1918: "presetPath": "/samples/phase-02/0221/sample.0221.tile-model-converter.json"
9+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:1918: "presetPath": "/samples/phase-02/0221/sample.0221.tile-model-converter.json"
10+
- classification: SAFE_METADATA_ONLY
11+
12+
## samples/phase-03/0305/sample.0305.3d-json-payload-normalizer.json
13+
- direct sample subtree JS/HTML refs: (no matches)
14+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:2512: "presetPath": "/samples/phase-03/0305/sample.0305.3d-json-payload-normalizer.json"
15+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:2512: "presetPath": "/samples/phase-03/0305/sample.0305.3d-json-payload-normalizer.json"
16+
- classification: SAFE_METADATA_ONLY
17+
18+
## samples/phase-03/0305/sample.0305.tile-model-converter.json
19+
- direct sample subtree JS/HTML refs: (no matches)
20+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:2520: "presetPath": "/samples/phase-03/0305/sample.0305.tile-model-converter.json"
21+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:2520: "presetPath": "/samples/phase-03/0305/sample.0305.tile-model-converter.json"
22+
- classification: SAFE_METADATA_ONLY
23+
24+
## samples/phase-03/0306/sample.0306.parallax-editor.json
25+
- direct sample subtree JS/HTML refs: (no matches)
26+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:2588: "presetPath": "/samples/phase-03/0306/sample.0306.parallax-editor.json"
27+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:2588: "presetPath": "/samples/phase-03/0306/sample.0306.parallax-editor.json"
28+
- classification: SAFE_METADATA_ONLY
29+
30+
## samples/phase-09/0901/sample.0901.svg-asset-studio.json
31+
- direct sample subtree JS/HTML refs: (no matches)
32+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:5771: "presetPath": "/samples/phase-09/0901/sample.0901.svg-asset-studio.json"
33+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:5771: "presetPath": "/samples/phase-09/0901/sample.0901.svg-asset-studio.json"
34+
- classification: SAFE_METADATA_ONLY
35+
36+
## samples/phase-09/0901/sample.0901.vector-map-editor.json
37+
- direct sample subtree JS/HTML refs: (no matches)
38+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:5775: "presetPath": "/samples/phase-09/0901/sample.0901.vector-map-editor.json"
39+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:5775: "presetPath": "/samples/phase-09/0901/sample.0901.vector-map-editor.json"
40+
- classification: SAFE_METADATA_ONLY
41+
42+
## samples/phase-12/1204/sample.1204.parallax-editor.json
43+
- direct sample subtree JS/HTML refs: (no matches)
44+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:6506: "presetPath": "/samples/phase-12/1204/sample.1204.parallax-editor.json"
45+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:6506: "presetPath": "/samples/phase-12/1204/sample.1204.parallax-editor.json"
46+
- classification: SAFE_METADATA_ONLY
47+
48+
## samples/phase-12/1204/sample.1204.svg-asset-studio.json
49+
- direct sample subtree JS/HTML refs: (no matches)
50+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:6510: "presetPath": "/samples/phase-12/1204/sample.1204.svg-asset-studio.json"
51+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:6510: "presetPath": "/samples/phase-12/1204/sample.1204.svg-asset-studio.json"
52+
- classification: SAFE_METADATA_ONLY
53+
54+
## samples/phase-12/1204/sample.1204.vector-map-editor.json
55+
- direct sample subtree JS/HTML refs: (no matches)
56+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:6514: "presetPath": "/samples/phase-12/1204/sample.1204.vector-map-editor.json"
57+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:6514: "presetPath": "/samples/phase-12/1204/sample.1204.vector-map-editor.json"
58+
- classification: SAFE_METADATA_ONLY
59+
60+
## samples/phase-12/1205/sample.1205.parallax-editor.json
61+
- direct sample subtree JS/HTML refs: (no matches)
62+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:6575: "presetPath": "/samples/phase-12/1205/sample.1205.parallax-editor.json"
63+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:6575: "presetPath": "/samples/phase-12/1205/sample.1205.parallax-editor.json"
64+
- classification: SAFE_METADATA_ONLY
65+
66+
## samples/phase-12/1205/sample.1205.vector-map-editor.json
67+
- direct sample subtree JS/HTML refs: (no matches)
68+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:6579: "presetPath": "/samples/phase-12/1205/sample.1205.vector-map-editor.json"
69+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:6579: "presetPath": "/samples/phase-12/1205/sample.1205.vector-map-editor.json"
70+
- classification: SAFE_METADATA_ONLY
71+
72+
## samples/phase-12/1208/sample.1208.3d-asset-viewer.json
73+
- direct sample subtree JS/HTML refs: (no matches)
74+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:6727: "presetPath": "/samples/phase-12/1208/sample.1208.3d-asset-viewer.json"
75+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:6727: "presetPath": "/samples/phase-12/1208/sample.1208.3d-asset-viewer.json"
76+
- classification: SAFE_METADATA_ONLY
77+
78+
## samples/phase-12/1209/sample.1209.tile-model-converter.json
79+
- direct sample subtree JS/HTML refs: (no matches)
80+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:6781: "presetPath": "/samples/phase-12/1209/sample.1209.tile-model-converter.json"
81+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:6781: "presetPath": "/samples/phase-12/1209/sample.1209.tile-model-converter.json"
82+
- classification: SAFE_METADATA_ONLY
83+
84+
## samples/phase-14/1417/sample.1417.asset-pipeline-tool.json
85+
- direct sample subtree JS/HTML refs: (no matches)
86+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:8305: "presetPath": "/samples/phase-14/1417/sample.1417.asset-pipeline-tool.json"
87+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:8305: "presetPath": "/samples/phase-14/1417/sample.1417.asset-pipeline-tool.json"
88+
- classification: SAFE_METADATA_ONLY
89+
90+
## samples/phase-16/1606/sample.1606.physics-sandbox.json
91+
- direct sample subtree JS/HTML refs: (no matches)
92+
- runtime refs (samples/tools/tests): samples\metadata\samples.index.metadata.json:8776: "presetPath": "/samples/phase-16/1606/sample.1606.physics-sandbox.json"
93+
- broad refs (excluding docs/tmp/.git): .\samples\metadata\samples.index.metadata.json:8776: "presetPath": "/samples/phase-16/1606/sample.1606.physics-sandbox.json"
94+
- classification: SAFE_METADATA_ONLY
Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
samples/phase-02/0221/sample.0221.tile-model-converter.json
2+
samples/phase-03/0305/sample.0305.3d-json-payload-normalizer.json
3+
samples/phase-03/0305/sample.0305.tile-model-converter.json
4+
samples/phase-03/0306/sample.0306.parallax-editor.json
5+
samples/phase-09/0901/sample.0901.svg-asset-studio.json
6+
samples/phase-09/0901/sample.0901.vector-map-editor.json
7+
samples/phase-12/1204/sample.1204.parallax-editor.json
8+
samples/phase-12/1204/sample.1204.svg-asset-studio.json
9+
samples/phase-12/1204/sample.1204.vector-map-editor.json
10+
samples/phase-12/1205/sample.1205.parallax-editor.json
11+
samples/phase-12/1205/sample.1205.vector-map-editor.json
12+
samples/phase-12/1208/sample.1208.3d-asset-viewer.json
13+
samples/phase-12/1209/sample.1209.tile-model-converter.json
14+
samples/phase-14/1417/sample.1417.asset-pipeline-tool.json
15+
samples/phase-16/1606/sample.1606.physics-sandbox.json
Lines changed: 80 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,80 @@
1+
{
2+
"deletedCount": 12,
3+
"removedMetadataReferenceCount": 12,
4+
"deletedFiles": [
5+
"samples/phase-03/0305/sample.0305.3d-json-payload-normalizer.json",
6+
"samples/phase-03/0306/sample.0306.parallax-editor.json",
7+
"samples/phase-09/0901/sample.0901.svg-asset-studio.json",
8+
"samples/phase-09/0901/sample.0901.vector-map-editor.json",
9+
"samples/phase-12/1204/sample.1204.parallax-editor.json",
10+
"samples/phase-12/1204/sample.1204.svg-asset-studio.json",
11+
"samples/phase-12/1204/sample.1204.vector-map-editor.json",
12+
"samples/phase-12/1205/sample.1205.parallax-editor.json",
13+
"samples/phase-12/1205/sample.1205.vector-map-editor.json",
14+
"samples/phase-12/1208/sample.1208.3d-asset-viewer.json",
15+
"samples/phase-14/1417/sample.1417.asset-pipeline-tool.json",
16+
"samples/phase-16/1606/sample.1606.physics-sandbox.json"
17+
],
18+
"removedMetadataReferences": [
19+
{
20+
"sampleId": "0305",
21+
"toolId": "3d-json-payload-normalizer",
22+
"presetPath": "/samples/phase-03/0305/sample.0305.3d-json-payload-normalizer.json"
23+
},
24+
{
25+
"sampleId": "0306",
26+
"toolId": "parallax-editor",
27+
"presetPath": "/samples/phase-03/0306/sample.0306.parallax-editor.json"
28+
},
29+
{
30+
"sampleId": "0901",
31+
"toolId": "svg-asset-studio",
32+
"presetPath": "/samples/phase-09/0901/sample.0901.svg-asset-studio.json"
33+
},
34+
{
35+
"sampleId": "0901",
36+
"toolId": "vector-map-editor",
37+
"presetPath": "/samples/phase-09/0901/sample.0901.vector-map-editor.json"
38+
},
39+
{
40+
"sampleId": "1204",
41+
"toolId": "parallax-editor",
42+
"presetPath": "/samples/phase-12/1204/sample.1204.parallax-editor.json"
43+
},
44+
{
45+
"sampleId": "1204",
46+
"toolId": "svg-asset-studio",
47+
"presetPath": "/samples/phase-12/1204/sample.1204.svg-asset-studio.json"
48+
},
49+
{
50+
"sampleId": "1204",
51+
"toolId": "vector-map-editor",
52+
"presetPath": "/samples/phase-12/1204/sample.1204.vector-map-editor.json"
53+
},
54+
{
55+
"sampleId": "1205",
56+
"toolId": "parallax-editor",
57+
"presetPath": "/samples/phase-12/1205/sample.1205.parallax-editor.json"
58+
},
59+
{
60+
"sampleId": "1205",
61+
"toolId": "vector-map-editor",
62+
"presetPath": "/samples/phase-12/1205/sample.1205.vector-map-editor.json"
63+
},
64+
{
65+
"sampleId": "1208",
66+
"toolId": "3d-asset-viewer",
67+
"presetPath": "/samples/phase-12/1208/sample.1208.3d-asset-viewer.json"
68+
},
69+
{
70+
"sampleId": "1417",
71+
"toolId": "asset-pipeline-tool",
72+
"presetPath": "/samples/phase-14/1417/sample.1417.asset-pipeline-tool.json"
73+
},
74+
{
75+
"sampleId": "1606",
76+
"toolId": "physics-sandbox",
77+
"presetPath": "/samples/phase-16/1606/sample.1606.physics-sandbox.json"
78+
}
79+
]
80+
}
Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,54 @@
1+
# PR 11.59 Expected Evidence Report
2+
3+
Codex must fill in:
4+
5+
## Baseline Audit
6+
- YES count:
7+
- NO count:
8+
- TOTAL count:
9+
10+
## Deleted JSON Files
11+
1.
12+
2.
13+
3.
14+
4.
15+
5.
16+
6.
17+
7.
18+
8.
19+
9.
20+
10.
21+
11.
22+
12.
23+
13.
24+
14.
25+
15.
26+
16.
27+
17.
28+
18.
29+
19.
30+
20.
31+
21.
32+
22.
33+
23.
34+
24.
35+
25.
36+
26.
37+
27.
38+
28.
39+
29.
40+
30.
41+
31.
42+
32.
43+
44+
## Metadata References Removed
45+
- `samples/metadata/samples.index.metadata.json` entries removed for each deleted file where present.
46+
47+
## Final Audit
48+
- YES count:
49+
- NO count:
50+
- TOTAL count:
51+
- NO count reduction:
52+
53+
## Test Decision
54+
Full samples smoke test skipped. Reason: cleanup is limited to unused JSON and stale metadata references; no shared loader/framework code changed.
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
[
2+
"samples/phase-02/0221/sample.0221.tile-model-converter.json",
3+
"samples/phase-03/0305/sample.0305.3d-json-payload-normalizer.json",
4+
"samples/phase-03/0305/sample.0305.tile-model-converter.json",
5+
"samples/phase-03/0306/sample.0306.parallax-editor.json",
6+
"samples/phase-09/0901/sample.0901.svg-asset-studio.json",
7+
"samples/phase-09/0901/sample.0901.vector-map-editor.json",
8+
"samples/phase-12/1204/sample.1204.parallax-editor.json",
9+
"samples/phase-12/1204/sample.1204.svg-asset-studio.json",
10+
"samples/phase-12/1204/sample.1204.vector-map-editor.json",
11+
"samples/phase-12/1205/sample.1205.parallax-editor.json",
12+
"samples/phase-12/1205/sample.1205.vector-map-editor.json",
13+
"samples/phase-12/1208/sample.1208.3d-asset-viewer.json",
14+
"samples/phase-12/1209/sample.1209.tile-model-converter.json",
15+
"samples/phase-14/1417/sample.1417.asset-pipeline-tool.json",
16+
"samples/phase-16/1606/sample.1606.physics-sandbox.json"
17+
]

0 commit comments

Comments
 (0)