fix: prefer fs/promises over promisify - #7399

This is likely to collide w/ my "remove tables" PR, I'll push it up tomorrow and would like to land that first.

Arborist had more than needed individual comments but it seems we aren't consistently using fs or fs/promises to require the sync methods. This is a nitpick for sure but let's just do the same thing everywhere in the off chance we require more methods later.

Also should it be node:fs/promises?

The sync methods are not available from fs/promises so if we use them from the fs returned from fs/promises it should error somewhere.

After reviewing again I was wrong, I remembered the sync part but it's also requring from fs