Remove 2 additional unused tool-specific JSON files verified by audit - PR 11.51

Author: DavidQ

docs/dev/codex_commands.md

@@ -1,9 +1,8 @@
-# Codex Commands — PR 11.50
+# Codex Commands - PR 11.51
 
 Model: GPT-5.4
-Reasoning: high
+Reasoning: medium
 
 ```powershell
-cd C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming
-codex --model gpt-5.4 --reasoning high "Run PR 11.50 from docs/pr/PR_11_50_CONTROLLED_JSON_CLEANUP.md. Execute the audit script, choose exactly two safe tool-specific NO JSON files, verify manually, remove only confirmed dead JSON files, capture before/after audit reports under docs/dev/reports, run targeted validation only, and create the final repo ZIP at tmp/PR_11_50_CONTROLLED_JSON_CLEANUP.zip. Do not touch palette.json, tile-map-editor-document.json, sample 1902, roadmap text, shared loaders, or framework code."
+codex --model gpt-5.4 --reasoning medium "Run PR 11.51 controlled JSON cleanup. Follow docs/pr/PR_11_51_CONTROLLED_JSON_CLEANUP.md exactly. Run the audit script, select exactly two safe tool-specific NO JSON files, manually validate direct and broader references, remove only proven unused files, update docs/dev/reports/PR_11_51_audit_report.md with before/after counts and validation evidence, update only roadmap status markers if execution-backed, and package the final repo-structured artifact at tmp/PR_11_51_CONTROLLED_JSON_CLEANUP.zip. Do not run the full sample suite unless shared loader/framework files are changed."
 ```

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Remove two verified NO sample JSON payloads in controlled cleanup (1208) with targeted validation and audit delta - PR 11.50
+Remove 2 additional unused tool-specific JSON files verified by audit - PR 11.51

docs/dev/reports/PR_11_51_audit_after.txt

@@ -0,0 +1,57 @@
+# PR 11.51 Audit Report
+
+## Scope Followed
+- Ran `scripts/PS/audit-sample-json-js-references.ps1`.
+- Selected exactly two tool-specific `NO` candidates for manual validation.
+- Performed direct JS and broader reference checks.
+- Applied cleanup only if proven unused.
+
+## Before Audit
+Source: `docs/dev/reports/PR_11_51_audit_before.txt`
+
+```text
+YES count: 39
+NO count: 55
+TOTAL count: 94
+```
+
+## Candidate Validation
+Candidate 1:
+- Path: `samples/phase-05/0512/sample.0512.performance-profiler.json`
+- Direct JS reference check:
+  - `rg -n --fixed-strings "sample.0512.performance-profiler.json" . --glob "samples/**/*.js"`
+  - Result: no matches
+- Broader repo reference check (excluding docs/tmp):
+  - `rg -n --fixed-strings "sample.0512.performance-profiler.json" . --glob "!docs/**" --glob "!tmp/**" --glob "!.git/**"`
+  - Result: `samples/metadata/samples.index.metadata.json:4440` (`roundtripToolPresets[].presetPath`)
+- Decision: KEPT (blocked; manifest-used indirectly)
+
+Candidate 2:
+- Path: `samples/phase-07/0708/sample.0708.replay-visualizer.json`
+- Direct JS reference check:
+  - `rg -n --fixed-strings "sample.0708.replay-visualizer.json" . --glob "samples/**/*.js"`
+  - Result: no matches
+- Broader repo reference check (excluding docs/tmp):
+  - `rg -n --fixed-strings "sample.0708.replay-visualizer.json" . --glob "!docs/**" --glob "!tmp/**" --glob "!.git/**"`
+  - Result: `samples/metadata/samples.index.metadata.json:5262` (`roundtripToolPresets[].presetPath`)
+- Decision: KEPT (blocked; manifest-used indirectly)
+
+## Safety Gate Outcome
+- Non-blocked `NO` candidates after exclusions (`palette`, `tile-map-editor-document`, `sample 1902`, metadata file): `31`
+- Of those `31`, candidates present in live `roundtripToolPresets`: `31`
+- Safe removable candidates found under PR 11.51 rules: `0`
+
+Because every eligible `NO` candidate is still wired through live sample metadata, no file was proven unused after broader validation. No deletions were applied.
+
+## After Audit
+Source: `docs/dev/reports/PR_11_51_audit_after.txt`
+
+```text
+YES count: 39
+NO count: 55
+TOTAL count: 94
+```
+
+## Full Suite Decision
+Full samples smoke test: SKIPPED  
+Reason: no code or JSON removals were applied; shared loader/framework files were not modified.

docs/dev/reports/PR_11_51_audit_before.txt

@@ -0,0 +1,45 @@
+# PR 11.51 - Controlled JSON Cleanup
+
+## Purpose
+Continue the audit-driven sample JSON cleanup lane with the smallest safe executable change.
+
+## Scope
+- Run `scripts/PS/audit-sample-json-js-references.ps1`.
+- Select exactly 2 `NO` JSON items from safe tool-specific cleanup targets.
+- Manually validate both files before removal.
+- Remove only files proven unused by direct and broader reference checks.
+- Do not touch shared, palette, tile-map-editor, sample 1902, roadmap text, or framework files.
+
+## Required Selection Rules
+Allowed candidates:
+- tool-specific JSON only
+- profiler, replay, pipeline, 3D tools, or equivalent isolated tool payloads
+- currently reported as `NO` by the audit script
+- no direct JS reference
+- no broader repo reference except audit output/report text
+
+Blocked candidates:
+- `palette.json`
+- `tile-map-editor-document.json`
+- anything under sample 1902
+- shared loader/framework files
+- manifests used indirectly
+- uncertain candidates
+
+## Validation Commands
+```powershell
+.\scripts\PS\audit-sample-json-js-references.ps1
+
+# For each candidate filename:
+Select-String -Path .\samples\**\*.js -Pattern "<filename>"
+Select-String -Path .\* -Recurse -Pattern "<filename>"
+```
+
+## Apply Rule
+If either candidate has uncertainty, keep it and choose another safe `NO` item.
+
+## Acceptance
+- Exactly 2 safe unused JSON files removed.
+- Audit `NO count` is reduced by 2 or evidence explains why a candidate was kept.
+- Targeted validation completed.
+- Full samples smoke test skipped unless shared loader/framework files were modified.