formats js with flow code incorrectly when enabled to format on save

I have this file:

// @flow

export default class OrganisationModel extends TimestampedModel {
  async findMembers(
    search?: string
  ): Promise<{ users: UserModel[], groups: GroupModel[] }> {
    const usersQuery = UserModel.query()
      .select('users.*')
      .where('is_archived', false)
      .andWhere('organisation_id', this.id)
    if (search) {
      const s: string = search
      usersQuery.andWhere(function() {
        this.where('email', 'ilike', `%${s}%`).orWhere(
          'username',
          'ilike',
          `%${s}%`
        )
      })
    }
    usersQuery.orderBy('username', 'asc')

    const groupsQuery = GroupModel.query()
      .select('groups.*')
      .andWhere('organisation_id', this.id)
    if (search) {
      groupsQuery.andWhere('name', '~*', `^${search}`)
    }
    groupsQuery.orderBy('name')

    const [users, groups] = await Promise.all([usersQuery, groupsQuery])
    return { users, groups }
  }

}

whenever I save the file while I have format on save enabled I get:

// @flow

export default class OrganisationModel extends TimestampedModel {
  async findMembers(search?: string) : Promise < {
    users: UserModel[],
    groups: GroupModel[]
  } > {
    const usersQuery = UserModel
      .query()
      .select('users.*')
      .where('is_archived', false)
      .andWhere('organisation_id', this.id)if (search) {
        const s : string = search
        usersQuery.andWhere(function () {
          this
            .where('email', 'ilike', `%${s}%`)
            .orWhere('username', 'ilike', `%${s}%`)
        })
      }
      usersQuery
      .orderBy('username', 'asc')

      const groupsQuery = GroupModel
      .query()
      .select('groups.*')
      .andWhere('organisation_id', this.id)if (search) {
        groupsQuery.andWhere('name', '~*', `^${search}`)
      }
      groupsQuery
      .orderBy('name')

      const [users,
        groups] = await Promise.all([usersQuery, groupsQuery])return {users, groups}
  }

}

Now this file obviously is not correct because you can’t just write if without putting it on a new line.

My prettier config is:

{
  "singleQuote": true,
  "semi": false
}

When I run formatting manually or via the CLI I get the correct output. It’s only doing it when I have it enabled on save.

[capaj]

finally figured it out! it was this extension: https://marketplace.visualstudio.com/items?itemName=HookyQR.beautify#review-details

I’d be pretty interested to know how vscode decides who should be formatting a file and if it allows multiple formatters to run-who decides the order? It would be awesome to get some kind of a warning when I have two formatters running at once for a single file.

@clarkie yeah something must have changed in 1.23 because before this version I only had issues like that very rarely.

actually I had two extensions for formatting code which were taking precedence over prettier. Will try to open a PR for a readme warning for new users.

[CiGit]

Format Document and formatOnSave take the exact same code path from our perspective. You may have an other extension which makes things on save