timdp/depviz

Visualizes intra-monorepo dependencies.

Yes, hello.

depviz's Issues

require.context(../to/file) that isn't a packages results in error

in our packages directory we have a folder that just contains example files and isn't thus a real package

When using depviz 0.1.0 we get following error:

TypeError: Cannot read property 'name' of undefined
    at /Users/wouterdebie/Projects/depviz/depviz.js:89:26
    at async Promise.all (index 0)
    at async addModuleRequireContext (/Users/wouterdebie/Projects/depviz/depviz.js:84:24)
    at async Promise.all (index 0)
    at async addModuleRequireContexts (/Users/wouterdebie/Projects/depviz/depviz.js:147:3)
    at async Promise.all (index 1)
    at async addPkgRequireContexts (/Users/wouterdebie/Projects/depviz/depviz.js:160:3)
    at async Promise.all (index 43)
    at async addRequireContexts (/Users/wouterdebie/Projects/depviz/depviz.js:181:3)
    at async buildDeps (/Users/wouterdebie/Projects/depviz/depviz.js:233:5)

Using version 0.5.0 it passes.

Line of code that causes the error

const dependencies = await Promise.all(
    dependencyModules.map(async mod => {
      const cwd = path.join(pkgsPath, path.dirname(mod))
      const {
        package: { name: dependency }
      } = await schedule(() => readPkgUp({ cwd }))
      return dependency
    })
  )

In version 0.1.0 packages read-pkg-up has been upgraded from version 6.0.0 to version 7.0.1 and it is thus not backwards compatible which results in the error above

Warn if `dot` is not installed

depviz currently prints an ugly spawn error if dot is not available. It could check before doing its thing.