Rename InternalModuleReadFile and friends.

[jdalton]

With the addition of PR #15767 the internal InternalModuleReadFile and pseudo exposed process.binding("fs").internalModuleReadFile are no longer generic read file helpers and are essentially locked down to just json. InternalModuleReadFile was more generically useful before and could have been applied to loading more than just json.

I think the name InternalModuleReadFile, and the pseudo exposed process.binding("fs").internalModuleReadFile, should either be renamed or a new internal for json-only should be created (maybe that wraps InternalModuleReadFile).

[bnoordhuis]

That pull request's been open for six weeks, why didn't you comment then?

I don't think the name matters, it's used in only one place. Pull requests welcome, I guess, but honestly, it seems very low value.

[jdalton]

That pull request's been open for six weeks, why didn't you comment then?

I wasn't aware of the PR until I saw the commit message next to "lib/module.js" when navigating files on GitHub 😋

I don't think the name matters, it's used in only one place. Pull requests welcome, I guess, but honestly, it seems very low value.

The name and function matter, in part, because it's an appealing pseudo private. Since this is designated a semver major change anyway it might as well be a clean break with either an API name change or a new helper.

I mentioned the potential usefulness of this helper to @Trott in the hallway track of Node Interactive. Do others have an opinion or preference for direction?

Update:
Opened PR #17084.