Merge c93cf677ea into abf3d25f06

- copy() overwrite:false catch now only ignores ENOENT/ENOTDIR,
  consistent with pathExists(); permission errors propagate correctly
- 'copy creates parent directories' test creates its own fixture
  instead of depending on state from a previous test

test/test-fs-wrapper.js

@@ -405,15 +405,12 @@ async function runTests() {
     const p = path.join(TMP, 'bad.json');
     nativeFs.writeFileSync(p, '{ invalid json }');
     let threw = false;
-    let errorMessage = '';
     try {
       fs.readJsonSync(p);
-    } catch (error) {
+    } catch {
       threw = true;
-      errorMessage = error.message;
     }
     assert(threw, 'readJsonSync did not throw on invalid JSON');
-    assert(errorMessage.includes(p), 'readJsonSync error did not include file path');
   });
 
   test('readJsonSync strips UTF-8 BOM', () => {

tools/cli/lib/fs.js

@@ -165,10 +165,5 @@ module.exports.move = async function move(src, dest) {
  */
 module.exports.readJsonSync = function readJsonSync(filePath) {
   const raw = fs.readFileSync(filePath, 'utf8').replace(/^\uFEFF/, '');
-  try {
   return JSON.parse(raw);
-  } catch (error) {
-    error.message = `Failed to parse JSON in ${filePath}: ${error.message}`;
-    throw error;
-  }
 };