Skip to content

Commit 360882e

Browse files
author
DavidQ
committed
Remove duplicate sample index metadata evidence copy and preserve samples metadata as SSoT - PR 10.25
1 parent 5a6352c commit 360882e

10 files changed

Lines changed: 183 additions & 9904 deletions

File tree

docs/dev/codex_commands.md

Lines changed: 12 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -1,54 +1,15 @@
1-
MODEL: GPT-5.3-codex
2-
REASONING: low
1+
# CODEX COMMANDS
32

4-
PR purpose:
5-
Remove future/advisory import hint fields that are not required for current tool operation.
3+
model: gpt-5.3-codex
4+
reasoning: low
65

7-
User decision:
8-
If it is not required now, remove it. Do not keep future intent fields like importDestination/importName that point to non-existing files/folders.
6+
Apply PR_10_25_SAMPLE_METADATA_SSOT_DUPLICATE_EVIDENCE_FIX.
97

10-
Target fields:
11-
- importDestination
12-
- importName
13-
- destinationFolder
14-
- future-only import plan defaults
15-
- fields only used to prefill advisory/manual import plans
16-
17-
Scope:
18-
- Asset Browser metadata
19-
- Import Hub metadata
20-
- tool launch presets
21-
- sample tool manifests
22-
- active-project manifest tool sections
23-
- docs explaining these fields as expected behavior
24-
25-
Do not modify:
26-
- KOTI gameplay
27-
- runtime game engine files
28-
- start_of_day folders
29-
30-
Rules:
31-
- Remove fields if not required for current load/display/edit/validate/export.
32-
- Do not replace with another future hint.
33-
- Do not introduce fallback data.
34-
- If a destination/name is required for an actual import action, require it at action time with validation.
35-
- Asset Browser 0204 and 1505 must still show clear empty/missing/invalid state.
36-
37-
Validation:
38-
- rg "importDestination|importName|destinationFolder" tools samples docs
39-
- Explain remaining hits.
40-
- node --check changed JS files only.
41-
- Target browser validation for Asset Browser 0204 and 1505.
42-
- Do not run full sample suite.
43-
44-
Create report:
45-
docs/dev/reports/PR_tool_remove_future_import_hints_report.md
46-
47-
Report must include:
48-
- PASS/FAIL
49-
- changed files
50-
- fields removed
51-
- remaining search hits and why they remain
52-
- validation commands/results
53-
- confirmation no start_of_day changes
54-
- confirmation no runtime engine changes
8+
- Preserve samples/metadata/samples.index.metadata.json as the only live SSoT.
9+
- Remove, move, or clearly demote the duplicate report evidence copy.
10+
- Do not keep a repo-relative duplicate that looks like runtime source.
11+
- Confirm runtime/tool code references only the live SSoT.
12+
- Do not rewrite sample metadata content.
13+
- Do not modify start_of_day folders.
14+
- Add validation report.
15+
- Return ZIP artifact at tmp/PR_10_25_SAMPLE_METADATA_SSOT_DUPLICATE_EVIDENCE_FIX_delta.zip.

docs/dev/commit_comment.txt

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1 @@
1-
Remove future-only import hint fields
2-
3-
- Remove non-required importDestination/importName-style defaults
4-
- Stop presenting future paths or filenames as expected tool data
5-
- Keep Asset Browser states explicit without silent fallback
6-
- Require import destinations only at actual import action time
1+
Remove duplicate sample index metadata evidence copy and preserve samples metadata as SSoT - PR 10.25
Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,47 @@
1+
# PR_10_25_SAMPLE_METADATA_SSOT_DUPLICATE_EVIDENCE_FIX Report
2+
3+
## Result
4+
PASS
5+
6+
## Scope
7+
- Preserved `samples/metadata/samples.index.metadata.json` as the only live runtime SSoT.
8+
- Removed the duplicate report-evidence copy that could be mistaken for runtime source.
9+
- Verified runtime/tool code references only the live SSoT location.
10+
11+
## Changes
12+
- Removed duplicate evidence file:
13+
- `docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/samples/metadata/samples.index.metadata.json`
14+
- Added this validation report:
15+
- `docs/dev/reports/PR_10_25_SAMPLE_METADATA_SSOT_DUPLICATE_EVIDENCE_FIX_report.md`
16+
17+
## Duplicate Comparison Evidence
18+
Compared:
19+
- Live SSoT: `samples/metadata/samples.index.metadata.json`
20+
- Duplicate evidence copy: `docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/samples/metadata/samples.index.metadata.json`
21+
22+
Result before removal:
23+
- `live_sha256`: `8d1ef36717fa26a0ac75589cad6fbaaeef40f94c962a31386b3dcb44fbdf3e96`
24+
- `dup_sha256`: `283e72b2cea10716bf47923941a16a26bb47a561c58a3a37224915e00d1913c1`
25+
- `identical`: `false`
26+
27+
This confirms the duplicate was stale/non-identical and unsafe to leave in a repo-relative runtime-looking path.
28+
29+
## Runtime/Tool Reference Validation
30+
Command:
31+
- `rg -n "repo_relative/samples/metadata/samples\.index\.metadata\.json|level_10_6d_palette_contract_evidence\\repo_relative\\samples\\metadata\\samples\.index\.metadata\.json" tools samples tests`
32+
33+
Result:
34+
- No matches.
35+
36+
Command:
37+
- `rg -n "samples\.index\.metadata\.json" tools samples tests`
38+
39+
Result:
40+
- Matches point only to live SSoT path under `samples/metadata/samples.index.metadata.json`.
41+
- No runtime/tool/test path references the removed report duplicate.
42+
43+
## Constraints Check
44+
- No sample metadata content rewrite performed.
45+
- No broad report cleanup performed.
46+
- No roadmap text changes.
47+
- No `start_of_day` folder changes.
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
# REPORT_PR_10_25_SAMPLE_METADATA_SSOT_DUPLICATE_EVIDENCE_FIX
2+
3+
Summary: Removes source-of-truth confusion from duplicated sample index metadata JSON inside report evidence.
4+
5+
Target:
6+
- Live SSoT remains samples/metadata/samples.index.metadata.json
7+
- Report evidence no longer contains a repo-relative duplicate that looks authoritative
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
# REPORT_PR_10_26_REPO_DUPLICATE_FILE_CONTENT_AUDIT
2+
3+
## Summary
4+
Adds a reporting-only audit for exact duplicate file contents across the repo.
5+
6+
## Result
7+
This prepares a safe cleanup pass by identifying duplicate groups before any delete/move work.

0 commit comments

Comments
 (0)