nollup: TypeError: Cannot read property 'length' of undefined

When I try to run nollup with the reccommended setup for react-refresh I get this error on one of my modules:

ParseError: Users/ashish/thoughtspot/node_modules/use-http/dist/index.js
TypeError: Cannot read property 'length' of undefined
    at /Users/ashish/thoughtspot/blink-v2/node_modules/nollup/lib/impl/utils.js:72:65
    at Array.forEach (<anonymous>)
    at combineSourceMapChain (/Users/ashish/thoughtspot/blink-v2/node_modules/nollup/lib/impl/utils.js:67:22)
    at Object.transform (/Users/ashish/thoughtspot/blink-v2/node_modules/nollup/lib/impl/PluginLifecycle.js:217:11)
    at async parse (/Users/ashish/thoughtspot/blink-v2/node_modules/nollup/lib/index.js:51:31)
    at async parse (/Users/ashish/thoughtspot/blink-v2/node_modules/nollup/lib/index.js:116:17)

The command fails and does not continue after this, just keeps printing the above error. If I comment the above import out, it fails at some other file.

About this issue

Original URL
State: closed
Created 4 years ago
Comments: 17 (14 by maintainers)

Commits related to this issue

#74: sourcesContent.length undefined fix. — committed to houd1ni/nollup by houd1ni 4 years ago
Merge pull request #109 from houd1ni/master #74: sourcesContent.length undefined fix. — committed to PepsRyuu/nollup by PepsRyuu 4 years ago

Most upvoted comments

After some investigation, I saw that the dependency is using live bindings. It is probably related to https://github.com/PepsRyuu/nollup/issues/116

straningera on Aug 6, 2020

Can’t reproduce it still. Tried creating a hello world Vue app with JSX using the same babel config, still no luck. Going to have to be able to at least reproduce it before merging the fix to avoid regression in the future.

PepsRyuu on Jul 17, 2020

Hi, @PepsRyuu ! I’m trying to extract the damaged plugin now. Will tell any progress.

houd1ni on Jul 16, 2020