fix: improper wrapping of fs.promises.readFile

[codebytere]

Description of Change

Fixes an issue where fs.promises.readFile would improperly error when running e.g:

const path = require('path')
const fs = require('fs');
const {
  open,
  readFile
} = fs.promises;

let fileHandle;
const run = async () => {
  const filePathForHandle = path.join('/Users', 'codebytere', 'Desktop', 'dogs-running.txt');
  fileHandle = await open(filePathForHandle, 'w+');
  const buffer = Buffer.from('Dogs running'.repeat(10000), 'utf8');
  fs.writeFileSync(filePathForHandle, buffer);
  await readFile(fileHandle)
  await fileHandle.close();
}

run();

with:

(node:89865) UnhandledPromiseRejectionWarning: TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string or an instance of Buffer or URL. Received an instance of FileHandle
    at readFile (fs.js:326:10)
    at e.readFile (electron/js2c/asar_bundle.js:5:7666)
    at internal/util.js:297:30
    at new Promise (<anonymous>)
    at internal/util.js:296:12
    at run (/private/var/folders/w4/6ckkx75165l174lkz07vgx_m0000gn/T/tmp-29494-eQiOkaJ4iCxZ/main.js:14:9)

becuase our fs wrapper simply used util.promisify on the asar-ified version of fs.readFile instead of calling into Node.js' actual version, which is implemented slightly differently.

Checklist

• PR description included and stakeholders cc'd
• npm test passes
• tests are changed or added
• PR release notes describe the change in a way relevant to app developers, and are capitalized, punctuated, and past tense.

Release Notes

Notes: Fixd an issue where fs.promises.readFile would improperly error when passing a FileHandle to the path argument.

[MarshallOfSound]

💯

Test? 😉

[release-clerk]

Release Notes Persisted

Fixd an issue where fs.promises.readFile would improperly error when passing a FileHandle to the path argument.

[trop]

I have automatically backported this PR to "12-x-y", please check out #29575

[trop]

I have automatically backported this PR to "13-x-y", please check out #29576

[trop]

I have automatically backported this PR to "14-x-y", please check out #29577