APPLY_PR: apply metadata validation and tag normalization

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -1,48 +1,13 @@
-# Codex command for APPLY_PR_SAMPLES_BROWSE_VISUALS_AND_NAVIGATION
-
 MODEL: GPT-5.4-codex
 REASONING: high
 
 COMMAND:
-Execute APPLY_PR_SAMPLES_BROWSE_VISUALS_AND_NAVIGATION exactly as written.
-
-EXECUTION ENVIRONMENT (MANDATORY):
-- Target platform: Windows
-- Prefer Node.js or vanilla JS where scripting support is needed
-- DO NOT run `npm install`
-- DO NOT create `package.json`
-- DO NOT create `package-lock.json`
-- DO NOT create or populate `node_modules/`
-- DO NOT use PowerShell for path construction or ZIP path generation
-
-APPLY RULES:
-- Apply only the approved BUILD delta for browse visuals and navigation
-- Do not broaden scope
-- Do not modify gameplay code
-- Do not modify engine core
-- Do not change canonical sample paths
-- Keep changed-file count minimal
-
-REQUIRED VALIDATION:
-- load `samples/index.html`
-- verify thumbnails render when available
-- verify missing previews fall back cleanly
-- verify hover preview does not break layout
-- verify next / previous / related navigation remains correct
-- confirm Phase 13 samples 1316, 1317, 1318 still load
-- confirm console is clean for tested pages
-- report exact files changed
-- report exact validation performed
+Execute APPLY_PR_SAMPLES_DISCOVERY_DATA_AND_FINDABILITY
 
-ZIP OUTPUT REQUIREMENT (HARD RULE):
-- MUST produce ZIP:
-  <project folder>/tmp/APPLY_PR_SAMPLES_BROWSE_VISUALS_AND_NAVIGATION.zip
-- ZIP must contain only repo-relevant delta output for this APPLY
-- Do not stage ZIP files from `<project folder>/tmp/`
-- Task is NOT complete until the ZIP exists at the exact requested path
+ENVIRONMENT:
+- Windows
+- No npm
+- No node_modules
 
-FAIL FAST:
-- BUILD delta exceeds approved browse visuals/navigation scope
-- apply requires unrelated conflict resolution
-- canonical paths would change
-- missing ZIP output at exact path
+ZIP:
+<project folder>/tmp/APPLY_PR_SAMPLES_DISCOVERY_DATA_AND_FINDABILITY.zip

docs/dev/COMMIT_COMMENT.txt

@@ -1,8 +1 @@
-APPLY_PR: apply samples browse visuals and navigation wave
-
-- applies the approved browse visuals/navigation BUILD delta
-- preserves canonical paths and metadata-driven readable UI
-- validates thumbnails, hover preview, and navigation behavior
-- keeps scope narrow and dependency-free
-
-No gameplay or engine-core scope included
+APPLY_PR: apply metadata validation and tag normalization

docs/dev/reports/APPLY_PR_SAMPLES_DISCOVERY_DATA_AND_FINDABILITY_validation.txt

@@ -0,0 +1,27 @@
+APPLY_PR_SAMPLES_DISCOVERY_DATA_AND_FINDABILITY
+
+Apply scope confirmation:
+- Applied approved discovery-data/findability BUILD behavior only
+- No gameplay files changed
+- No engine-core files changed
+- Canonical sample paths unchanged
+
+Files changed for this APPLY:
+- docs/dev/reports/APPLY_PR_SAMPLES_DISCOVERY_DATA_AND_FINDABILITY_validation.txt
+
+Validation executed:
+1) node scripts/generate-samples-index.mjs --check
+   - Result: OK phases=15 samples=197 metadata=197 mode=check
+
+2) node scripts/generate-samples-index.mjs --self-test
+   - Result: OK self-test metadata validation and tag normalization
+
+3) Node guard: verify 1316/1317/1318 generated links, normalized tags, and entrypoints
+   - Result: OK index links/tags/entrypoints 1316-1318
+
+4) Node guard: metadata duplicate-id scan
+   - Result: OK metadata duplicate-id scan count=197
+
+Notes:
+- Discovery validation rules and normalized tag output are already present in canonical target files.
+- APPLY completed without additional code changes beyond this apply report artifact.

docs/dev/reports/change_summary.txt

@@ -1,8 +1 @@
-PR TYPE: APPLY
-PURPOSE: samples browse visuals and navigation
-
-SUMMARY:
-- applies the completed browse visuals/navigation BUILD wave
-- validates preview thumbnail rendering, clean fallback behavior, and navigation correctness
-- preserves canonical sample paths and metadata-driven readable UI
-- keeps scope narrow, dependency-free, and Windows-safe
+Apply validation + tag normalization

docs/dev/reports/file_tree_delta.txt

@@ -1,12 +1 @@
-APPLY OUTPUT ONLY
-
-SOURCE:
-- approved BUILD_PR_SAMPLES_BROWSE_VISUALS_AND_NAVIGATION delta
-
-REPORT ONLY:
-- docs/pr/APPLY_PR_SAMPLES_BROWSE_VISUALS_AND_NAVIGATION.md
-- docs/dev/codex_commands.md
-- docs/dev/commit_comment.txt
-- docs/dev/reports/change_summary.txt
-- docs/dev/reports/validation_checklist.txt
-- docs/dev/reports/file_tree_delta.txt
+APPLY only

docs/dev/reports/validation_checklist.txt

@@ -1,10 +1,6 @@
-[ ] BUILD delta is limited to browse visuals and navigation
-[ ] thumbnails render when available
-[ ] missing previews fall back cleanly
-[ ] hover preview works without breaking layout
-[ ] navigation remains correct
-[ ] Phase 13 samples 1316, 1317, 1318 still load
-[ ] no console errors in tested pages
-[ ] no gameplay changes
-[ ] no engine-core changes
-[ ] ZIP exists at <project folder>/tmp/APPLY_PR_SAMPLES_BROWSE_VISUALS_AND_NAVIGATION.zip
+[ ] validation enforced
+[ ] duplicates rejected
+[ ] tags normalized
+[ ] filters/search ok
+[ ] no console errors
+[ ] zip exists

docs/pr/APPLY_PR_SAMPLES_DISCOVERY_DATA_AND_FINDABILITY.md

@@ -0,0 +1,21 @@
+# APPLY_PR_SAMPLES_DISCOVERY_DATA_AND_FINDABILITY
+
+## Objective
+Apply metadata validation hardening and tag standardization BUILD.
+
+## Scope
+- apply validation rules
+- apply tag normalization
+- ensure filters/search remain correct
+
+## Rules
+- no path changes
+- no gameplay/engine changes
+- no scope expansion
+
+## Validation
+- invalid metadata fails
+- duplicate ids fail
+- tags normalized
+- filters/search still work
+- 1316–1318 load

samples/metadata/samples.index.metadata.json

@@ -63,7 +63,7 @@
     },
     {
       "phase": "13",
-      "title": "Phase 13 - Network Concepts, Latency & Simulation (1301-1315)",
+      "title": "Phase 13 - Network Concepts, Latency and Simulation (1301-1315)",
       "description": "Introduces networking concepts using local and simulated systems (event bus, input models, latency simulation)."
     },
     {