Skip to content

Commit 5601a7b

Browse files
author
DavidQ
committed
Add Phase 20 tool sample dropdown foundation across active tools
- ship 3 local samples per active tool - add sample dropdown loading to Tile Map Editor, Parallax Editor, Vector Map Editor, and Vector Asset Studio - preserve fullscreen behavior as complete - keep roadmap updates status-only
1 parent 45dec38 commit 5601a7b

14 files changed

Lines changed: 609 additions & 177 deletions

docs/dev/codex_commands.md

Lines changed: 37 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,33 +1,43 @@
1-
MODEL: GPT-5.3-codex
2-
REASONING: medium
1+
MODEL: GPT-5.4
2+
REASONING: high
33

44
COMMAND:
5-
Create BUILD_PR_TOOL_HOST_SEAMLESS_IFRAME_PARITY.
5+
Implement BUILD_PR_LEVEL_20_1_TOOL_SAMPLE_DROPDOWN_FOUNDATION as a one-pass, testable PR in the HTML-JavaScript-Gaming repo.
66

7-
Refactor Tool Host so opening a tool in host feels visually equivalent to opening the tool directly, except for compact host controls/help inside the collapsible imported section.
7+
Constraints:
8+
- fullscreen is complete; do not modify fullscreen behavior
9+
- docs-first repo workflow
10+
- one PR purpose only: sample packs + dropdown loading
11+
- active tools only:
12+
- tools/Tile Map Editor
13+
- tools/Parallax Editor
14+
- tools/Vector Map Editor
15+
- tools/Vector Asset Studio
16+
- 3 shipped samples per active tool
17+
- sample data must live in each tool's local samples structure
18+
- each tool must expose a Sample dropdown and be able to load all 3 samples
19+
- preserve manual editing flows
20+
- no repo-wide scanning unless required
21+
- no destructive cleanup
22+
- do not modify start_of_day folders
23+
- keep legacy preserved paths intact unless strictly required
24+
- update roadmap status only in docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md
25+
- no roadmap rewrites; status-only changes unless explicit additive line is required
826

9-
Requirements:
10-
- iframe-hosted tool should use the available workspace as if opened directly
11-
- remove obvious host chrome/wrappers that make hosted mode visibly different
12-
- keep Tool Host HTML minimal
13-
- host-specific UI should live only in the collapsible imported section
14-
- collapsing that section should make the hosted tool appear nearly identical to direct launch
15-
- optional small fullscreen button is allowed, but do not depend on browser fullscreen as the main solution
16-
- preserve Open In Host behavior
17-
- preserve direct launch behavior
18-
- no inline styles
19-
- no embedded style blocks
20-
- no JS style-string injection
21-
- no color/token changes
22-
- no unnecessary per-tool hacks
23-
24-
Outputs:
25-
- repo-structured ZIP at <project folder>/tmp/BUILD_PR_TOOL_HOST_SEAMLESS_IFRAME_PARITY.zip
26-
- docs/pr/BUILD_PR_TOOL_HOST_SEAMLESS_IFRAME_PARITY.md
27-
- docs/dev/codex_commands.md
28-
- docs/dev/commit_comment.txt
27+
Required outputs:
28+
- implementation changes for dropdown sample loading in all 4 active tools
29+
- tool-local sample files/assets/manifests as needed
30+
- docs/pr/BUILD_PR_LEVEL_20_1_TOOL_SAMPLE_DROPDOWN_FOUNDATION.md aligned to delivered work
31+
- docs/dev/reports/change_summary.txt
32+
- docs/dev/reports/file_tree.txt
2933
- docs/dev/reports/validation_checklist.txt
34+
- docs/dev/commit_comment.txt
35+
- package final artifact to:
36+
<project folder>/tmp/BUILD_PR_LEVEL_20_1_TOOL_SAMPLE_DROPDOWN_FOUNDATION.zip
3037

31-
Roadmap:
32-
- update status only if execution-backed
33-
- status-only roadmap edits only
38+
Validation target:
39+
- open each tool
40+
- verify dropdown visible
41+
- verify exactly 3 samples per tool
42+
- verify each sample loads and visibly changes content
43+
- verify no fullscreen regression

docs/dev/commit_comment.txt

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,7 @@
1-
Refactor Tool Host iframe presentation for near direct-launch parity by removing host chrome wrappers,
2-
keeping host controls inside the collapsible imported section, and eliminating JS/inline style injection.
1+
BUILD_PR_LEVEL_20_1_TOOL_SAMPLE_DROPDOWN_FOUNDATION
2+
3+
- Add Sample dropdown + explicit Load Sample action to Vector Map Editor.
4+
- Add Vector Map Editor local sample manifest with exactly 3 shipped sample JSON files.
5+
- Preserve manual editing flow and avoid fullscreen behavior changes.
6+
- Confirm all 4 active tool manifests expose exactly 3 local samples.
7+
- Update roadmap with status-only additive marker for tool sample packs + dropdown loading.
Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,10 @@
1-
Scope summary
2-
- Single-purpose style PR
3-
- Header + Intro overlay only
4-
- Transparent overlay background
5-
- Foreground #ed9700
6-
- Collapsed header background #7a00df
7-
- Header border radius removed
8-
- page-shell excluded
1+
BUILD_PR_LEVEL_20_1_TOOL_SAMPLE_DROPDOWN_FOUNDATION
92

10-
Validation checklist
11-
- Visual overlay confirmed on targeted shared entry pages
12-
- Collapsed state color confirmed
13-
- No page-shell edits
14-
- No layout regressions
15-
- No console errors
3+
Summary:
4+
- Completed sample-pack + dropdown foundation for active tools with single PR purpose.
5+
- Added missing Vector Map Editor sample dropdown/load support.
6+
- Added 3 shipped local samples for Vector Map Editor.
7+
- Verified all 4 active tools now have exactly 3 local manifest samples.
8+
- Updated roadmap with execution-backed status line only.
9+
10+
No fullscreen behavior changes were made.

docs/dev/reports/file_tree.txt

Lines changed: 12 additions & 124 deletions
Original file line numberDiff line numberDiff line change
@@ -1,124 +1,12 @@
1-
Repo-Structured Delta File Tree (git status --porcelain)
2-
3-
M README.md
4-
M docs/README.md
5-
M docs/dev/reports/BUILD_PR_LEVEL_18_19_TRACK_G_REPO_HYGIENE_COMPLETION_VALIDATION.md
6-
M docs/dev/reports/BUILD_PR_LEVEL_20_2_TRACK_C_DOCUMENTATION_COMPLETENESS_RESIDUE_CLOSEOUT_DUPLICATE_DOC_SURFACES.md
7-
M docs/dev/reports/BUILD_PR_LEVEL_20_2_TRACK_C_DOCUMENTATION_COMPLETENESS_RESIDUE_CLOSEOUT_ONBOARDING_FLOW_CHECK.md
8-
M docs/dev/reports/change_summary.txt
9-
M docs/dev/reports/validation_checklist.txt
10-
M docs/operations/dev/PROJECT_INSTRUCTIONS.md
11-
M docs/operations/dev/README.md
12-
M docs/operations/dev/ROADMAP_GUARDRAILS.md
13-
M docs/operations/dev/paths.md
14-
M docs/operations/dev/run_commands.ps1
15-
M docs/operations/dev/run_commands.txt
16-
M docs/pr/APPLY_PR_LEVEL_12_6_REMOTE_DEPLOYMENT_CANDIDATE.md
17-
M docs/pr/APPLY_PR_LEVEL_12_7_REAL_NETWORK_COMPLETION_GATE.md
18-
M docs/pr/APPLY_PR_LEVEL_12_8_NETWORK_DIRECTORY_NORMALIZATION.md
19-
M docs/pr/BUILD_PR_ENGINE_IMPORT_BASELINE_AND_CONTRACT.md
20-
M docs/pr/BUILD_PR_FINAL_NON_3D_ACTIVE_LANES_AND_STATUS_CLOSEOUT.md
21-
M docs/pr/BUILD_PR_LEVEL_02_2D_CAPABILITY_COMBINED_FOUNDATION.md
22-
M docs/pr/BUILD_PR_LEVEL_02_ENGINE_CORE_BASELINE_AND_BOUNDARY_PASS.md
23-
M docs/pr/BUILD_PR_LEVEL_03_SHARED_FOUNDATION_COMBINED_PASS.md
24-
M docs/pr/BUILD_PR_LEVEL_04_STATE_REPLAY_TIMELINE_COMBINED_PASS.md
25-
M docs/pr/BUILD_PR_LEVEL_06_SAMPLES_PROGRAM_COMBINED_PASS.md
26-
M docs/pr/BUILD_PR_LEVEL_07_NETWORK_DEBUG_AND_SERVER_DASHBOARD_COMBINED_CLOSEOUT.md
27-
M docs/pr/BUILD_PR_LEVEL_09_TOOLS_NORMALIZATION_AND_REQUIRED_TOOLS_COMBINED_PASS.md
28-
M docs/pr/BUILD_PR_LEVEL_10_23_RESTORE_MASTER_ROADMAP_STATUS_ONLY.md
29-
M docs/pr/BUILD_PR_LEVEL_10_26_LEVEL_10_CLOSEOUT_SAMPLE_TOOL_NORMALIZATION_AND_SHARED_STRATEGY.md
30-
M docs/pr/BUILD_PR_LEVEL_12_10_REAL_NETWORK_SAMPLE_AND_DASHBOARD.md
31-
M docs/pr/BUILD_PR_LEVEL_12_13_SAMPLE_1319_ROOT_TESTS_AND_PS_RUNBOOK.md
32-
M docs/pr/BUILD_PR_LEVEL_12_15_SAMPLE_1319_RUNTIME_RETURN_TO_SERVER.md
33-
M docs/pr/BUILD_PR_LEVEL_12_2D_CAPABILITY_TRACK_RESIDUE_ONLY.md
34-
M docs/pr/BUILD_PR_LEVEL_12_7_REAL_NETWORK_COMPLETION_GATE.md
35-
M docs/pr/BUILD_PR_LEVEL_12_8_NETWORK_DIRECTORY_NORMALIZATION.md
36-
M docs/pr/BUILD_PR_LEVEL_12_9_NETWORK_USAGE_SAMPLE_STANDARDIZATION.md
37-
M docs/pr/BUILD_PR_LEVEL_14_TESTS_VALIDATION_RESIDUE_ONLY.md
38-
M docs/pr/BUILD_PR_LEVEL_16_1_PHASE16_NETWORK_GATE_COMPLETION.md
39-
M docs/pr/BUILD_PR_LEVEL_17_17_MASTER_ROADMAP_STATUS_SYNC.md
40-
M docs/pr/BUILD_PR_LEVEL_17_21_TRACK_H_3D_DEBUG_SUPPORT_CLOSEOUT.md
41-
M docs/pr/BUILD_PR_LEVEL_17_22_PHASE16_3D_DEBUG_AND_ROADMAP_REBASELINE.md
42-
M docs/pr/BUILD_PR_LEVEL_17_38_REAL_GAMEPLAY_SAMPLE.md
43-
M docs/pr/BUILD_PR_LEVEL_18_10_REMOVE_SAMPLE_SPECIFIC_LOGIC_FROM_ENGINE_PATHS.md
44-
M docs/pr/BUILD_PR_LEVEL_18_13_TRACK_E_CSS_UI_NORMALIZATION_FOUNDATION.md
45-
M docs/pr/BUILD_PR_LEVEL_18_18_TRACK_F_DOCS_SYSTEM_COMPLETION.md
46-
M docs/pr/BUILD_PR_LEVEL_18_1_NEXT_PHASE_BOOTSTRAP.md
47-
M docs/pr/BUILD_PR_LEVEL_18_2_PHASE18_FOUNDATION.md
48-
M docs/pr/BUILD_PR_LEVEL_18_3_GAME_TO_SAMPLE_RECLASSIFICATION_RECOMMENDATIONS.md
49-
M docs/pr/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION.md
50-
M docs/pr/BUILD_PR_LEVEL_19_1_NEXT_PHASE_BOOTSTRAP.md
51-
M docs/pr/BUILD_PR_LEVEL_19_4_PERFORMANCE_SCALING_ROADMAP_PROMOTION.md
52-
M docs/pr/BUILD_PR_REPO_CLEANUP_AND_ROADMAP_UPDATE.md
53-
M docs/pr/BUILD_PR_ROADMAP_REPAIR_ADDITIVE_MOVE_ONLY.md
54-
M docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1.md
55-
M docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_3_ARCHIVED_NOTES_POLICY.md
56-
M docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST.md
57-
M docs/pr/BUILD_PR_TEMPLATES_POLICY_CLASSIFICATION.md
58-
M docs/pr/LEVEL_12_4_PLAYABLE_MULTIPLAYER_VALIDATION_PREP.md
59-
M docs/pr/PLAN_PR_LEVEL_12_10_REAL_NETWORK_SAMPLE_AND_DASHBOARD.md
60-
M docs/pr/PLAN_PR_LEVEL_12_4_PLAYABLE_MULTIPLAYER_VALIDATION.md
61-
M docs/pr/PLAN_PR_LEVEL_17_38_REAL_GAMEPLAY_SAMPLE.md
62-
M docs/pr/PLAN_PR_LEVEL_18_1_NEXT_PHASE_BOOTSTRAP.md
63-
M docs/pr/PLAN_PR_LEVEL_19_1_NEXT_PHASE_BOOTSTRAP.md
64-
M docs/pr/PLAN_PR_LEVEL_19_2_PHASE19_FOUNDATION.md
65-
M docs/reference/features/docs-system/README.md
66-
M docs/reference/features/docs-system/move-history-preserved.md
67-
M docs/reference/root/README.md
68-
M docs/reports/BUILD_PR_ENGINE_IMPORT_BASELINE_AND_CONTRACT_report.md
69-
M docs/reports/BUILD_PR_LEVEL_03_EXACT_CLUSTER_NUMBER_STRING_ID_CLOSEOUT_report.md
70-
M docs/reports/BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST_report.md
71-
M docs/reports/BUILD_PR_LEVEL_15_16_LEGACY_REDUCTION_AND_DOCUMENTATION_SYSTEM_COMBINED_CLOSEOUT_report.md
72-
M docs/reports/BUILD_PR_LEVEL_16_1_PHASE16_NETWORK_GATE_COMPLETION_summary.md
73-
M docs/reports/BUILD_PR_LEVEL_16_PHASE_DESCRIPTIONS_REPO_WIDE_NORMALIZATION_report.md
74-
M docs/reports/BUILD_PR_LEVEL_18_10_REMOVE_SAMPLE_SPECIFIC_LOGIC_FROM_ENGINE_PATHS_report.md
75-
M docs/reports/BUILD_PR_LEVEL_18_11_TRACK_A_FINAL_CONFIRMATION_report.md
76-
M docs/reports/BUILD_PR_LEVEL_18_13_TRACK_E_CSS_UI_NORMALIZATION_COMPLETION_VALIDATION.md
77-
M docs/reports/BUILD_PR_LEVEL_18_13_TRACK_E_CSS_UI_NORMALIZATION_FOUNDATION_roadmap_status_reasoning.md
78-
M docs/reports/BUILD_PR_LEVEL_18_13_TRACK_E_CSS_UI_NORMALIZATION_FOUNDATION_touched_files.md
79-
M docs/reports/BUILD_PR_LEVEL_18_1_ENGINE_USAGE_ENFORCEMENT_COMPLETION_report.md
80-
M docs/reports/BUILD_PR_LEVEL_18_2_BOUNDARY_HARDENING_ENFORCEMENT_report.md
81-
M docs/reports/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION_summary.md
82-
M docs/reports/BUILD_PR_LEVEL_18_5_CONTRACT_STABILIZATION_PROGRESS_report.md
83-
M docs/reports/BUILD_PR_LEVEL_18_6_SELECTOR_PROVIDER_STABILITY_ENFORCEMENT_report.md
84-
M docs/reports/BUILD_PR_LEVEL_18_9_TRACK_C_FINALIZATION_report.md
85-
M docs/reports/BUILD_PR_LEVEL_19_13_RUNTIME_LIFECYCLE_VALIDATION_summary.md
86-
M docs/reports/BUILD_PR_LEVEL_19_15_DEBUG_OBSERVABILITY_VALIDATION_summary.md
87-
M docs/reports/BUILD_PR_LEVEL_19_17_TOOLCHAIN_PIPELINE_VALIDATION_summary.md
88-
M docs/reports/BUILD_PR_LEVEL_19_19_TOOLCHAIN_EDITOR_RUNTIME_CONSISTENCY_VALIDATION_summary.md
89-
M docs/reports/BUILD_PR_LEVEL_19_20_TOOLCHAIN_ROADMAP_GUARD_ENFORCEMENT_coverage.md
90-
M docs/reports/BUILD_PR_LEVEL_19_20_TOOLCHAIN_ROADMAP_GUARD_ENFORCEMENT_results.md
91-
M docs/reports/BUILD_PR_LEVEL_19_22_TOOLCHAIN_VALIDATION_SUMMARY_GATE_summary.md
92-
M docs/reports/BUILD_PR_LEVEL_19_24_ENGINE_TOOL_LEAK_REMEDIATION_GATE_report.md
93-
M docs/reports/BUILD_PR_LEVEL_19_25_ENGINE_TOOL_BOUNDARY_FINAL_CONFIRMATION_report.md
94-
M docs/reports/BUILD_PR_REMAINING_ROADMAP_VALIDATE_OR_CLOSEOUT_COMBINED_report.md
95-
M docs/reports/BUILD_PR_REPO_CLEANUP_AND_ROADMAP_UPDATE_report.md
96-
M docs/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1_report.md
97-
M docs/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_3_ARCHIVED_NOTES_POLICY_report.md
98-
M docs/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION_report.md
99-
M docs/reports/BUILD_PR_TEMPLATES_POLICY_CLASSIFICATION_report.md
100-
M docs/reports/BUILD_PR_TRACK_G_NETWORK_MULTIPLAYER_DEBUG_CLOSEOUT_report.md
101-
M docs/reports/PLAN_PR_REPO_CLEANUP_AND_ROADMAP_UPDATE_report.md
102-
M docs/reports/archived_notes_validation_guard.md
103-
M docs/reports/classes_old_keep_normalization_report.md
104-
M docs/reports/classes_old_keep_policy_inventory.md
105-
M docs/reports/classes_old_keep_removal_change_log.md
106-
M docs/reports/classes_old_keep_validation_guard.md
107-
M docs/reports/cleanup_execution_guard.md
108-
M docs/reports/cleanup_live_reference_inventory.txt
109-
M docs/reports/cleanup_target_enforcement_map.md
110-
M docs/reports/cleanup_target_normalization_report.md
111-
M docs/reports/docs_classification_matrix.md
112-
M docs/reports/docs_inventory_tree.txt
113-
M docs/reports/docs_move_map_proposed.md
114-
M docs/reports/overlay/level-17/BUILD_PR_LEVEL_17_59_DEBUG_OVERLAY_PROMOTE_BASELINE_report.md
115-
M docs/reports/overlay/level-19/BUILD_PR_LEVEL_19_6_OVERLAY_MULTI_LAYER_COMPOSITION_report.md
116-
M docs/reports/roadmap_status_update_instructions.txt
117-
M docs/reports/templates_live_usage_inventory.md
118-
M docs/reports/templates_validation_guard.md
119-
D docs/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md
120-
D docs/roadmaps/README.md
121-
D docs/roadmaps/phases.txt
122-
?? docs/dev/reports/file_tree.txt
123-
?? docs/dev/roadmaps/
124-
?? docs/pr/BUILD_PR_FINAL_REPO_CLEANUP_AND_ROADMAP_RENAME_V2.md
1+
docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md
2+
docs/pr/BUILD_PR_LEVEL_20_1_TOOL_SAMPLE_DROPDOWN_FOUNDATION.md
3+
docs/dev/reports/change_summary.txt
4+
docs/dev/reports/file_tree.txt
5+
docs/dev/reports/validation_checklist.txt
6+
docs/dev/commit_comment.txt
7+
tools/Vector Map Editor/index.html
8+
tools/Vector Map Editor/editor/VectorMapEditorApp.js
9+
tools/Vector Map Editor/samples/sample-manifest.json
10+
tools/Vector Map Editor/samples/overworld_route_map.editor.json
11+
tools/Vector Map Editor/samples/dungeon_layout_map.editor.json
12+
tools/Vector Map Editor/samples/arena_zone_map.editor.json
Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,16 @@
1-
# Validation Checklist
1+
BUILD_PR_LEVEL_20_1_TOOL_SAMPLE_DROPDOWN_FOUNDATION Validation Checklist
22

3-
- [x] Hosted tool workspace feels near-identical to direct launch
4-
- [x] Tool Host does not present obvious extra page chrome around the iframe
5-
- [x] Host-specific controls/help exist only in the collapsible imported section
6-
- [x] Collapsing host controls leaves the hosted tool visually close to direct-open mode
7-
- [x] Open In Host still loads tools correctly (selector and runtime mount flow preserved)
8-
- [x] Direct tool launch still works (manifest launchPath behavior unchanged)
9-
- [x] Any fullscreen button is optional and non-essential (none added)
10-
- [x] No inline styles or embedded style blocks added
11-
- [x] No JS style-string injection in Tool Host runtime/main
3+
Execution-backed checks completed:
4+
[x] Verified sample dropdown (`#sampleSelect`) exists in all active tool index pages.
5+
[x] Verified load sample action (`#loadSampleButton`) exists in all active tool index pages.
6+
[x] Verified each active tool local sample manifest has exactly 3 samples.
7+
[x] Verified every sample manifest path resolves to an existing local file.
8+
[x] Verified fullscreen path references remain present and were not altered as feature work.
9+
10+
Manual browser checks required to fully close acceptance:
11+
[ ] Open Tilemap Studio and load all 3 samples; confirm visible content changes.
12+
[ ] Open Parallax Scene Studio and load all 3 samples; confirm visible content changes.
13+
[ ] Open Vector Map Editor and load all 3 samples; confirm visible content changes.
14+
[ ] Open Vector Asset Studio and load all 3 samples; confirm visible content changes.
15+
[ ] Confirm manual editing flows remain functional after sample loading in each tool.
16+
[ ] Confirm no fullscreen regression in each tool.

docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -421,6 +421,7 @@
421421
- [x] ParallaxEditor normalized
422422
- [x] VectorMapEditor normalized
423423
- [x] VectorAssetStudio normalized
424+
- [x] Tool sample packs + dropdown loading (3 samples per active tool)
424425

425426
### New Required Tools (By Dependency)
426427
- [x] 3DMapEditor

0 commit comments

Comments
 (0)