useImmediateIntrospection is dangerously broken when combined with useValidationCache

[aarne]

useImmediateIntrospection assumes introspection query by default

envelop/packages/plugins/immediate-introspection/src/index.ts

Line 39 in e45066c

let isIntrospectionOnly = true;

and sets it to false when validation fails.

and when the validation is skipped then this will stay true ... so for the next run context creation is skipped for all queries.

[EmrysMyrddin]

Hi! Thank you for point this out!

Yes, I'm not sure why this has been implemented like this. It would probably be better to walk the AST on the after parse phase.

If you want to give it a try, PR are very welcome! We are here to guide you if you have questions.