Yeah... one of these days we should actually start _running_ Closure on our code for warnings so that these annotations matter :-)

Fwiw, I think we've sometimes been explicitly putting `?` anyway just to clarify that it's a case we've thought about the nullability of (if there's no `?` I find it usually means the author of the code didn't think about it, and it might actually be a required value).