incorrect arguments by default with `this`

𝓭𝓮𝓬𝓪𝓯𝓯𝓮𝓲𝓷𝓪𝓽𝓮 is incorrectly transpiling my CoffeeScript input:

source

class Component extends React.PureComponent
  state: {intent: null, yearMonth: null}
  loadData: ({ intent = @state.intent, yearMonth = @state.yearMonth } = {}) =>
    @loadable.request =>
      Api.get '/admin/document_checks/stat_data', { yearMonth, intent }
        .then ({ stats, teams, operators }) => @setState { stats, teams, operators }

REPL

incorrect result (indentation issue is there too):

class Component extends React.PureComponent {
  // ...
  loadData({ intent = intent1, yearMonth = yearMonth1 } = {}) { // 😛 UNDEF
    this.state = state; // 😛 
    this.intent = intent1; // 😛
    this.state = state1; // 😛
    this.yearMonth = yearMonth1; // 😛
    return this.loadable.request(() => {
      return Api.get('/admin/document_checks/stat_data', { yearMonth, intent })
        .then(({ stats, teams, operators }) => this.setState({ stats, teams, operators }));
  });
  }

expected:

class Component extends React.PureComponent {
  // ...
  loadData({ intent = this.state.intent, yearMonth = this.state.yearMonth } = {}) {
    return this.loadable.request(() => {
      return Api.get('/admin/document_checks/stat_data', { yearMonth, intent })
        .then(({ stats, teams, operators }) => this.setState({ stats, teams, operators }));
    });
  }

[eventualbuddha]

Have you looked at this? https://github.com/decaffeinate/decaffeinate/blob/master/docs/contribution-guide.md

[eventualbuddha]

This definitely appears to be a bug. Here’s a minimal reproduction:

({ a = @a }) ->

(function({ a = a1 }) {
  this.a = a1;
});

I think the problem is that it’s inspecting the whole subtree of nodes for default values to assign, when it shouldn’t be doing so with the right hand side of the assignment. I don’t know that’s the case, but that’s what it feels like. I’d be happy to review a PR to fix this.