feat: allow to manage display_order value for building block definiti…#218
feat: allow to manage display_order value for building block definiti…#218j0g3sc wants to merge 2 commits into
Conversation
📊 Test Coverage
❌ Acceptance tests failed (advisory — the job runs against the last merged meshfed-release backend, so a change needing a companion backend PR fails here until that backend merges; the real gate is meshfed-release's Uncovered functions (combined run)Combined with |
…on I/O without affecting the calculated hash for change detection CU-86cabn76y
5321918 to
49e6c12
Compare
|
|
…on I/O without affecting the calculated hash for change detection
CU-86cabn76y
Summary
Added a manageable display_order field to building block definition version inputs and outputs.
Implementation:
Client
DisplayOrder int64 with json:"displayOrder,omitempty" on both MeshBuildingBlockDefinitionInput and MeshBuildingBlockDefinitionOutput.
Schema
display_order Int64Attribute on inputs and outputs, Optional + Computed + Default(0) — the Default is what keeps re-plans clean given the backend returns 0 when unset.
Content hash
stripped to 0 before hashing, so reordering never changes content_hash or forces a new version
Released immutability
displayOrdersDiffer guard in Update — needed because display_order is outside the hash, so this restores the "released versions are immutable" behavior with a clear error.
Mock
mirrors input→output display_order for manual BBDs.