Both isEval() and isWasm() are defined in the .c++ file (not in the header), so the inline keyword on their declarations in the header was indeed redundant and misleading. The other similar accessors in the same class (isEsm(), isMain()) already don't have inline, and the ones that do retain inline (type(), contentType()) are defined inline in the header body. This is a clean, correct consistency fix.

LGTM

github run