> On current master, if I extract erikvold-menuitems-jplib-5108038.zip under packages and run cfx docs, I get this error.

On your 821774 branch when I put this error back in, the error isn't thrown in that case.  It should only be thrown if a internal module has no documentation, which should cause tests to break, so this seems fine to have to me.

> Throwing an exception is almost certainly the wrong thing to do here, I think.

I'd rather handle that case separately and have a test written for it.