review feedback: lenght cache, es modules exports should be stored, even if it is undefined. adding tests ftw