runem / lit-analyzer

Monorepository for tools that analyze lit-html templates
MIT License
319 stars 36 forks source link

bug(lit-analyzer) breaks on cyclical symblinks #355

Open maxpatiiuk opened 5 months ago

maxpatiiuk commented 5 months ago

Example: if /lit-components/tests/lazy-consumer/node_modules/lit-components/ is a symlink to /lit-components/, lit-analyzer will crash with this error:

~/site/esri/arcgis-web-components/packages/lit-components lit-research* ⇡ ❯ yarn lit-analyzer .
[Error: ENAMETOOLONG: name too long, scandir '/Users/mak13180/site/esri/arcgis-web-components/packages/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/@esbuild'] {
  errno: -63,
  code: 'ENAMETOOLONG',
  syscall: 'scandir',
  path: '/Users/mak13180/site/esri/arcgis-web-components/packages/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/lit-components/tests/lazy-consumer/node_modules/@esbuild'
}

Not a big deal as I shouldn't be running it on node_modules anyway, but this may cause issues for other users.