Observation
In cmdInstantiate() you do:
WasmEdge_VMLoadWasmFromFile()
WasmEdge_VMValidate()
WasmEdge_VMInstantiate()
This is correct, but:
VMInstantiate() already implies validation
Calling VMValidate() explicitly is optional
Why this matters
Not a bug — but reviewers may ask why both are used.
✅ Recommendation (not mandatory)
Either:
keep it and document why, or
remove VMValidate() for minimalism
📌 Severity: Moderate
📌 Phase impact: Phase 4 polish
Observation
In cmdInstantiate() you do:
This is correct, but:
VMInstantiate() already implies validation
Calling VMValidate() explicitly is optional
Why this matters
Not a bug — but reviewers may ask why both are used.
✅ Recommendation (not mandatory)
Either:
keep it and document why, or
remove VMValidate() for minimalism
📌 Severity: Moderate
📌 Phase impact: Phase 4 polish