feat(packages/sui-lint): fix RepositoryLinter node_modules ignore for pnpm

packages/sui-lint/src/RepositoryLinter/Match.js

@@ -18,8 +18,8 @@ class Match {
 
   static create(path) {
     const ext = extname(path)
-    if (!ext && CustomFileReader.create().isDirectory(path)) {
-      return new Match(path, undefined, undefined, true)
+    if (CustomFileReader.create().isDirectory(path)) {
+      return new Match(path, undefined, '', true)
     }
 
     let parsed

packages/sui-lint/src/RepositoryLinter/Runner.js

@@ -11,7 +11,7 @@ module.exports.Runner = class Runner {
   }
 
   assertion(key) {
-    const files = this.fg.sync(key, {ignore: ['node_modules'], onlyFiles: false}) ?? []
+    const files = this.fg.sync(key, {ignore: ['**/node_modules/**'], onlyFiles: false}) ?? []
     return files.map(Match.create)
   }
 }

packages/sui-lint/test/server/MatchSpec.js

@@ -29,6 +29,20 @@ describe('Match', function () {
 
     // Then
     expect(match.isDir).to.be.eqls(true)
+    expect(match.raw).to.be.eqls('')
+  })
+
+  it('Should detect directories with file-like extensions', function () {
+    // Given
+    this.isDirStub.returns(true)
+
+    // When
+    const match = Match.create('cypress/screenshots/PTA/CategoryPicker.js')
+
+    // Then
+    expect(match.isDir).to.be.eqls(true)
+    expect(match.raw).to.be.eqls('')
+    expect(match.raw.match(/pattern/)).to.be.eqls(null)
   })
 
   it('Should detect files w/out extensions', function () {

packages/sui-lint/test/server/RunnerSpec.js

@@ -23,4 +23,20 @@ describe('Runner', function () {
     expect(this.matchCreateStub.firstCall.firstArg).to.be.eql('path/file.json')
     expect(this.syncStub.firstCall.firstArg).to.be.eql('**/*.json')
   })
+
+  it('Should ignore node_modules at any depth', function () {
+    this.syncStub.returns([])
+
+    Runner.create({sync: this.syncStub}).assertion('**/*.js')
+
+    expect(this.syncStub.firstCall.args[1].ignore).to.be.eql(['**/node_modules/**'])
+  })
+
+  it('Should include directories in results', function () {
+    this.syncStub.returns(['app/src'])
+
+    Runner.create({sync: this.syncStub}).assertion('app/src')
+
+    expect(this.syncStub.firstCall.args[1].onlyFiles).to.be.eql(false)
+  })
 })