Inline lambda in argument list causes problems for argument chopping and lambda alignment

[cmeeren]

Issue created from fantomas-online

Code

module Lifecycle =


  let init config =
    async {
      cfg <- config
      do!
        MassTransit.init
          cfg.LoggerFactory cfg.AzureServiceBusConnStr cfg.QueueName cfg.LoggerFactory
          (fun reg ->
            reg.Consume User.handleUserInitiatedRegistration
            reg.Consume User.handleUserUpdated
            reg.Consume User.handleGetSessionUserIdRequest
          )
    }

Result

module Lifecycle =


  let init config =
    async {
      cfg <- config
      do! MassTransit.init cfg.LoggerFactory cfg.AzureServiceBusConnStr
            cfg.QueueName cfg.LoggerFactory (fun reg ->
            reg.Consume User.handleUserInitiatedRegistration
            reg.Consume User.handleUserUpdated
            reg.Consume User.handleGetSessionUserIdRequest)
    }

Problem description

Due to the inline lambda, the arguments don't break correctly. I would expect:

module Lifecycle =


  let init config =
    async {
      cfg <- config
      do! MassTransit.init
            cfg.LoggerFactory
            cfg.AzureServiceBusConnStr
            cfg.QueueName
            cfg.LoggerFactory
            (fun reg ->
               reg.Consume User.handleUserInitiatedRegistration
               reg.Consume User.handleUserUpdated
               reg.Consume User.handleGetSessionUserIdRequest)
    }

Or something similar.

Extra information

• The formatted result breaks by code.
• The formatted result gives compiler warnings.
• I or my company would be willing to help fix this. (I'll help in any way I can except implementation; I don't have the capacity to ramp up on a new codebase now)

Options

Fantomas Master at 08/21/2020 08:12:02 - 8665356

    { config with
                IndentSize = 2
                MaxLineLength = 80 }

[cmeeren]

Judging from fantomas-online, this seems to be fixed in 4.3.

[nojaf]

Great, I'll add a regression test someday.