coord(M03): claim M2 mesh resource extractor
Work-Package: M03-RND-M2-MESH-RESOURCE-EXTRACTOR-001 Agent: sindo-main-codex Tests: coordination gate passed before claim Fidelity: extraction-only package; traversal and visible behavior unchanged
This commit is contained in:
@@ -0,0 +1,91 @@
|
||||
# M03-RND-M2-MESH-RESOURCE-EXTRACTOR-001 — M2 mesh resource extractor
|
||||
|
||||
<!-- OPENWC_CLAIM:M03-RND-M2-MESH-RESOURCE-EXTRACTOR-001:sindo-main-codex:2026-07-20 -->
|
||||
|
||||
## Ownership
|
||||
|
||||
- Target: M03
|
||||
- Program: RND
|
||||
- Owner/Agent ID: sindo-main-codex
|
||||
- Branch: `work/sindo-main-codex/m03-m2-mesh-resource-extractor`
|
||||
- Lease expires UTC: 2026-07-20
|
||||
- Integrator: M03 milestone integrator
|
||||
|
||||
## Outcome
|
||||
|
||||
Extract first-Mesh selection from loaded M2 Resources and Node subtrees,
|
||||
including temporary PackedScene instance cleanup, from `StreamingWorldLoader`.
|
||||
|
||||
## Non-goals
|
||||
|
||||
- Call ResourceLoader or select/request cache paths.
|
||||
- Refresh/rebuild materials or load raw M2 data.
|
||||
- Own the Mesh cache, missing state, prototypes or MultiMesh materialization.
|
||||
- Change traversal order, cache formats, profiles or visible behavior.
|
||||
|
||||
## Paths
|
||||
|
||||
- Exclusive: `src/render/m2/m2_mesh_resource_extractor.gd`,
|
||||
`src/tools/verify_m2_mesh_resource_extractor.gd`,
|
||||
`docs/modules/m2-mesh-resource-extractor.md`, this claim
|
||||
- Shared/hotspots: `src/scenes/streaming/streaming_world_loader.gd`,
|
||||
M2 pipeline/cache/material/shutdown verifiers, `docs/modules/world-renderer.md`,
|
||||
`docs/modules/README.md`, M2 module specs, `RENDER.md`,
|
||||
`targets/03-renderer-facade.md`
|
||||
- Generated/ignored: `.godot`, native DLL, generated M2 resources, caches and
|
||||
proprietary renderer corpus
|
||||
|
||||
## Contracts and data
|
||||
|
||||
- A direct Mesh Resource returns the same Mesh reference.
|
||||
- A PackedScene is instantiated once, searched depth-first in child order and
|
||||
its temporary root is freed before return.
|
||||
- A Node subtree search returns the first MeshInstance3D Mesh in historical
|
||||
depth-first preorder without changing the tree.
|
||||
- Null, unsupported Resources and subtrees without a Mesh return null.
|
||||
|
||||
## Dependencies
|
||||
|
||||
- Requires: accepted Mesh resource cache package on master `5e2afbc`
|
||||
- Blocks: isolated material refresh/rebuild finalizer extraction
|
||||
- External state: none; verification uses synthetic Mesh/Node/PackedScene fixtures
|
||||
|
||||
## Verification
|
||||
|
||||
- Commands: dedicated direct/PackedScene/subtree/order/lifetime/source/timing
|
||||
verifier; M2 pipeline/cache/classifier/material/shutdown/build/facade/internal-
|
||||
access/manifest, documentation and coordination gates
|
||||
- Fixtures: ArrayMesh, MeshInstance3D subtrees and synthetic PackedScene
|
||||
- Fidelity evidence: exact traversal order and temporary-node destruction
|
||||
- Performance budget: 10,000 synthetic subtree lookups under 1 second
|
||||
|
||||
## Documentation deliverables
|
||||
|
||||
- Inline public API docs
|
||||
- Module specification with inputs/outputs, ownership and source map
|
||||
- Data-flow, lifecycle/state, sequence and dependency diagrams
|
||||
- M2 cache/pipeline, world-renderer, registry and RENDER updates
|
||||
|
||||
## Simplicity and naming
|
||||
|
||||
- Important name introduced: `M2MeshResourceExtractor`
|
||||
- Simplest solution: one stateless RefCounted with two explicit extraction APIs
|
||||
- Rejected complexity: callback-driven finalizer, generic scene utility,
|
||||
cache ownership, signals or visitor framework
|
||||
- Unavoidable complexity: temporary PackedScene instance must be freed while
|
||||
its referenced Mesh remains borrowed
|
||||
- Measured optimization evidence: bounded synthetic depth-first traversal
|
||||
|
||||
## Status
|
||||
|
||||
- State: claimed
|
||||
- Done: three first-Mesh call sites and temporary PackedScene lifecycle identified
|
||||
- Next: implement extractor, migrate loader adapters, verify and document
|
||||
- Blocked by:
|
||||
|
||||
## Handoff
|
||||
|
||||
- Commit:
|
||||
- Results:
|
||||
- Remaining risks:
|
||||
- Documentation updated:
|
||||
Reference in New Issue
Block a user