When using -r, the row.file property doesn't contain a full file path, while row.deps does. So it was possible that we'd look up a full file path but it was never added to the relevant Map. (this should probably be addressed in browserify too!)
row.index and row.indexDepsdo reliably match, so we can use those instead. common-shake accepts integers as module names so everything is fine.
Based on @laduke's test case.
When using
-r, therow.fileproperty doesn't contain a full file path, whilerow.depsdoes. So it was possible that we'd look up a full file path but it was never added to the relevant Map. (this should probably be addressed in browserify too!)row.indexandrow.indexDepsdo reliably match, so we can use those instead. common-shake accepts integers as module names so everything is fine.Fixes https://github.com/browserify/common-shakeify/issues/21 Fixes https://github.com/browserify/common-shakeify/issues/29