Rename internalModuleReadFile to internalModuleReadJSON.
This PR addresses #17076 (closed) by simply renaming internalModuleReadFile
to internalModuleReadJSON
.
With the addition of PR #15767 the internal
InternalModuleReadFile
and pseudo exposedprocess.binding("fs").internalModuleReadFile
are no longer generic read file helpers and are essentially locked down to justjson
.InternalModuleReadFile
was more generically useful before and could have been applied to loading more than justjson
.I think the name
InternalModuleReadFile
, and the pseudo exposedprocess.binding("fs").internalModuleReadFile
, should either be renamed or a new internal forjson
-only should be created (maybe that wrapsInternalModuleReadFile
).
I'm actually cool with keeping internalModuleReadFile
just for the potential to use it for other things besides json
, but this was the easy way to address the root of #17076 (closed) without digging into c++
.