Default exported class is not picked up correctly

First of all thanks for the great tool! It is really providing a lot of value when cleaning up old code.

I found a bug when exporting classes in the default export:

// a.js
export default class User extends Person {
  walk() {
    console.log("I'm walking");
  }
}

// b.js
import User from './a';

new User().walk();

shrimpit! Output:

 Shrimpit!

 > Files tree

{ '.': null,
  'a.js': { exports: [ 'walk', '/' ], imports: [] },
  'b.js': { exports: [], imports: [ '/' ] } }

 > Unused exports

[ 'walk' ]

Expectation: It should not traverse the members of the class declaration, it should simply check whether the “Default” export of a.js is used anywhere.

[peterjuras]

Thanks this issue is solved. I encountered a new one, but as you suggested I’ll open up a new issue.

[yamafaktory]

@peterjuras I’ve released a new version with a fix https://github.com/yamafaktory/shrimpit/commit/396d16ad913e1ab84666ba37904e8a22a8f6d09c.

Please give it another try and let me know! If this fine, please close the issue. If you encounter another bug, please open a new issue (will be easier to track everything in different threads), thanks a lot 👍.