fix: remove fs-extra

e2e/autofix-test.js

@@ -1,6 +1,6 @@
 const chai = require('chai')
 const { expect } = chai
-const fs = require('fs-extra')
+const fs = require('fs')
 const os = require('os')
 const path = require('path')
 const shell = require('shelljs')
@@ -566,10 +566,12 @@ function useFixture(dir) {
     const tmpDirContainer = os.tmpdir()
     this.testDirPath = path.join(tmpDirContainer, `solhint-tests-${dir}`)
 
-    fs.ensureDirSync(this.testDirPath)
-    fs.emptyDirSync(this.testDirPath)
+    fs.mkdirSync(this.testDirPath, { recursive: true })
+    for (const entry of fs.readdirSync(this.testDirPath)) {
+      fs.rmSync(path.join(this.testDirPath, entry), { recursive: true, force: true });
+    }
 
-    fs.copySync(fixturePath, this.testDirPath)
+    fs.cpSync(fixturePath, this.testDirPath, { recursive: true })
 
     shell.cd(this.testDirPath)
   })

e2e/formatters-test.js

@@ -1,6 +1,6 @@
 const chai = require('chai')
 const { expect } = chai
-const fs = require('fs-extra')
+const fs = require('fs')
 const shell = require('shelljs')
 const url = require('url')
 const os = require('os')
@@ -705,10 +705,12 @@ function useFixture(dir) {
     const tmpDirContainer = os.tmpdir()
     this.testDirPath = path.join(tmpDirContainer, `solhint-tests-${dir}`)
 
-    fs.ensureDirSync(this.testDirPath)
-    fs.emptyDirSync(this.testDirPath)
+    fs.mkdirSync(this.testDirPath, { recursive: true })
+    for (const entry of fs.readdirSync(this.testDirPath)) {
+      fs.rmSync(path.join(this.testDirPath, entry), { recursive: true, force: true });
+    }
 
-    fs.copySync(fixturePath, this.testDirPath)
+    fs.cpSync(fixturePath, this.testDirPath, { recursive: true })
 
     shell.cd(this.testDirPath)
   })

e2e/test.js

@@ -1,6 +1,6 @@
 const { expect } = require('chai')
 const cp = require('child_process')
-const fs = require('fs-extra')
+const fs = require('fs')
 const getStream = require('get-stream')
 const os = require('os')
 const path = require('path')
@@ -478,9 +478,12 @@ function useFixtureFolder(ctx, dir) {
 
   ctx.testDirPath = testDirPath
 
-  fs.ensureDirSync(testDirPath)
-  fs.emptyDirSync(testDirPath)
-  fs.copySync(fixturePath, testDirPath)
+    fs.mkdirSync(testDirPath, { recursive: true })
+    for (const entry of fs.readdirSync(testDirPath)) {
+      fs.rmSync(path.join(testDirPath, entry), { recursive: true, force: true });
+    }
+
+    fs.cpSync(fixturePath, testDirPath, { recursive: true })
 
   shell.cd(testDirPath)
 }

lib/cache/cache-manager.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra')
+const fs = require('fs')
 const crypto = require('crypto')
 const path = require('path')
 

lib/common/utils.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra')
+const fs = require('fs')
 const path = require('path')
 
 const getLocFromIndex = (text, index) => {

lib/config/config-file.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra')
+const fs = require('fs')
 const path = require('path')
 const _ = require('lodash')
 const { cosmiconfigSync } = require('cosmiconfig')

lib/index.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra')
+const fs = require('fs')
 const path = require('path')
 const parser = require('@solidity-parser/parser')
 const glob = require('glob')
@@ -100,7 +100,7 @@ function processFile(file, config, rootDir = process.cwd(), explicitConfigPath)
   const { report, updatedCache } = processAndCache(file, finalConfig, cacheState)
 
   if (finalConfig.cache && updatedCache) {
-    fs.ensureDirSync(path.dirname(cachePath)) // make sure the directory exists
+    fs.mkdirSync(path.dirname(cachePath), { recursive: true }) // make sure the directory exists
     writeCache(cachePath, updatedCache)
   }
 
@@ -135,7 +135,7 @@ function processPath(pattern, config, rootDir = process.cwd(), explicitConfigPat
   }
 
   if (useCache) {
-    fs.ensureDirSync(path.dirname(cachePath)) // make sure the directory exists
+    fs.mkdirSync(path.dirname(cachePath), { recursive: true }) // make sure the directory exists
     writeCache(cachePath, updatedCache)
   }
 

lib/rules/miscellaneous/import-path-check.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra')
+const fs = require('fs')
 const path = require('path')
 const BaseChecker = require('../base-checker')
 const { severityDescription } = require('../../doc/utils')

package.json

@@ -44,9 +44,9 @@
     "@solidity-parser/parser": "^0.20.0",
     "ajv": "^6.12.6",
     "ajv-errors": "^1.0.1",
-    "better-ajv-errors": "^2.0.2",
     "antlr4": "^4.13.1-patch-1",
     "ast-parents": "^0.0.1",
+    "better-ajv-errors": "^2.0.2",
     "chalk": "^4.1.2",
     "commander": "^10.0.0",
     "cosmiconfig": "^8.0.0",
@@ -60,8 +60,7 @@
     "semver": "^7.5.2",
     "strip-ansi": "^6.0.1",
     "table": "^6.8.1",
-    "text-table": "^0.2.0",
-    "fs-extra": "^11.1.0"
+    "text-table": "^0.2.0"
   },
   "devDependencies": {
     "assert": "^2.0.0",

scripts/generate-rule-docs.js

@@ -1,5 +1,5 @@
 #!env node
-const fs = require('fs-extra')
+const fs = require('fs')
 const { exec, mkdir } = require('shelljs')
 const semver = require('semver')
 const path = require('path')

scripts/generate-rulesets.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra')
+const fs = require('fs')
 
 const { loadRules } = require('../lib/load-rules')
 

solhint.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 const program = require('commander')
 const _ = require('lodash')
-const fs = require('fs-extra')
+const fs = require('fs')
 const process = require('process')
 const readline = require('readline')
 const chalk = require('chalk')

test/common/utils.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra')
+const fs = require('fs')
 const os = require('os')
 const path = require('path')
 

test/rules/miscellaneous/config-hierarchy.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra')
+const fs = require('fs')
 const path = require('path')
 const os = require('os')
 const rimraf = require('rimraf')

test/rules/miscellaneous/import-path-check.js

@@ -17,7 +17,7 @@ describe('import-path-check (mocked fs)', () => {
   }
 
   beforeEach(() => {
-    existsStub = sinon.stub(require('fs-extra'), 'existsSync').callsFake((filePath) => {
+    existsStub = sinon.stub(require('fs'), 'existsSync').callsFake((filePath) => {
       return currentFakeFileSystem.has(normalizePath(filePath))
     })
   })