Skip to content

Commit 599d6ed

Browse files
author
DavidQ
committed
docs: add BUILD_PR asset pipeline converters bundle
1 parent f394259 commit 599d6ed

9 files changed

+416
-30
lines changed

docs/dev/CODEX_COMMANDS.md

Lines changed: 6 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -2,25 +2,16 @@ MODEL: GPT-5.4
22
REASONING: high
33

44
COMMAND:
5-
Create BUILD_PR_ASSET_PIPELINE_FOUNDATION as the next follow-up to project-level tool integration.
5+
Create BUILD_PR_ASSET_PIPELINE_CONVERTERS
66

77
Scope:
8-
- establish a minimal shared asset-pipeline foundation
9-
- target only ingest/normalize/validate/register stages
10-
- reuse existing tools/shared asset/project helpers first
11-
- active tools only
12-
- minimal adapters only
13-
- no UI/theme work
14-
- no editor-state redesign
15-
- no render-pipeline changes
16-
- no converter suite in this PR
17-
- do not touch templates/ cleanup in this PR
8+
- add converter abstraction layer
9+
- implement minimal converters
10+
- integrate with pipeline foundation
1811

1912
Validation:
2013
- npm run test:launch-smoke -- --tools
21-
- verify touched tools still ingest/load assets
22-
- report exact files changed
23-
- report normalized asset rules and adapters
14+
- verify conversions usable
2415

2516
Output:
26-
<project folder>/tmp/BUILD_PR_ASSET_PIPELINE_FOUNDATION_delta.zip
17+
<project>/tmp/BUILD_PR_ASSET_PIPELINE_CONVERTERS_delta.zip

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
docs: add BUILD_PR asset pipeline foundation bundle
1+
docs: add BUILD_PR asset pipeline converters bundle
Lines changed: 48 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,48 @@
1+
# BUILD_PR_ASSET_PIPELINE_CONVERTERS Report
2+
3+
## Scope Outcome
4+
- Added a minimal shared converter abstraction layer for the asset pipeline.
5+
- Implemented conservative converter defaults for:
6+
- tile -> vector (basic)
7+
- vector -> tile (basic)
8+
- sprite normalization
9+
- Integrated conversion into the shared pipeline foundation between normalize and validate/register stages.
10+
- Kept scope limited to shared pipeline/converter logic (no UI/theme/editor-state/render changes).
11+
12+
## Converter Layer
13+
- New shared module: `tools/shared/assetPipelineConverters.js`
14+
- Introduced:
15+
- `createAssetPipelineConverterRegistry(...)`
16+
- `listAssetPipelineConverters(...)`
17+
- `convertAssetPipelineCandidate(...)`
18+
- `DEFAULT_ASSET_PIPELINE_CONVERTERS`
19+
- Converter behavior is metadata-safe and path/type normalization focused (minimal transforms only).
20+
21+
## Pipeline Integration
22+
- Updated `tools/shared/assetPipelineFoundation.js` to include `CONVERT` stage.
23+
- `registerAssetPipelineCandidate(...)` now performs:
24+
- normalize -> optional convert -> validate -> register
25+
- Added `convertNormalizedAssetPipelineCandidate(...)` and `validateNormalizedAssetPipelineCandidate(...)`.
26+
- Added converter inventory to `summarizeAssetPipelineRules()` output.
27+
28+
## Validation
29+
- Command: `npm run test:launch-smoke -- --tools`
30+
- Result: PASS (`9/9` tools)
31+
- Smoke report: `docs/dev/reports/launch_smoke_report.md`
32+
33+
- Converter usability check:
34+
- Verified converter registry exposes expected converter IDs.
35+
- Verified tile->vector registration applies conversion and registers converted output.
36+
- Verified vector->tile registration applies conversion and registers converted output.
37+
- Verified sprite normalization converter produces normalized sprite output.
38+
- Result: PASS
39+
- Evidence: `docs/dev/reports/BUILD_PR_ASSET_PIPELINE_CONVERTERS_validation.txt`
40+
41+
## Exact Files Changed
42+
- `docs/pr/BUILD_PR_ASSET_PIPELINE_CONVERTERS.md`
43+
- `docs/dev/reports/asset_pipeline_converters_targets.txt`
44+
- `tools/shared/assetPipelineConverters.js`
45+
- `tools/shared/assetPipelineFoundation.js`
46+
- `docs/dev/reports/launch_smoke_report.md`
47+
- `docs/dev/reports/BUILD_PR_ASSET_PIPELINE_CONVERTERS_validation.txt`
48+
- `docs/dev/reports/BUILD_PR_ASSET_PIPELINE_CONVERTERS_report.md`
592 Bytes
Binary file not shown.
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
ASSET PIPELINE CONVERTER TARGETS
2+
3+
- tile → vector (basic)
4+
- vector → tile (basic)
5+
- sprite normalization
6+
7+
Keep:
8+
- minimal logic
9+
- no heavy transforms

docs/dev/reports/launch_smoke_report.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Launch Smoke Report
22

3-
Generated: 2026-04-12T00:24:06.723Z
3+
Generated: 2026-04-12T00:31:13.799Z
44

55
Filters: games=false, samples=false, tools=true, sampleRange=all
66

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
# BUILD_PR_ASSET_PIPELINE_CONVERTERS
2+
3+
## Purpose
4+
Extend asset pipeline with initial converter layer for transforming assets between formats used by tools.
5+
6+
## Goals
7+
- introduce converter abstraction
8+
- enable tile ↔ vector ↔ sprite transformations (basic only)
9+
- keep converters modular and optional
10+
11+
## Scope
12+
- converter interface
13+
- minimal converters (safe/common only)
14+
- integration with pipeline foundation
15+
16+
## Out of Scope
17+
- full production-grade converters
18+
- 3D formats
19+
- UI redesign
20+
- rendering changes
21+
22+
## Strategy
23+
- define converter registry
24+
- plug into pipeline after normalize stage
25+
- keep transformations reversible where possible
26+
27+
## Validation
28+
- npm run test:launch-smoke -- --tools
29+
- converted assets usable in at least 2 tools

0 commit comments

Comments
 (0)