Comment by Christopher Fuller (JIRA)

One thought I had was that possibly we need `buildExpression` to make this work. So the builder could become:

@_functionBuilder
public struct ScriptBuilder {

    public static func buildExpression(_ step: Step) -> Script {
        [step]
    }

    public static func buildBlock(_ scripts: Script...) -> Script {
        scripts.flatMap { $0 }
    }

    public static func buildIf(_ script: Script?) -> Script {
        script ?? []
    }

    public static func buildEither(first: Script) -> Script {
        first
    }

    public static func buildEither(second: Script) -> Script {
        second
    }
}

But I cannot seem to get `buildExpression` to work … possibly it's not implemented … so am unable to test this theory.

Hmm, I thought adding a buildBlock that takes a single item and returns an array would work but apparently not. I think this is just something that isn't supported yet. In the meantime, you can do two things:

1. Define a new buildBlock that takes a single Step and returns Script and return [step] in the body. Then, in singleStep, invoke it directly by doing ScriptBuilder.buildBlock(ExampleStep()).
2. Simply return [ExampleStep()] in testSingle.

buildExpression has been implemented and is now supported on master: #27716

@swift-ci create

Comment by Christopher Fuller (JIRA)

@theblixguy Thanks for the quick reply! 👍

I tested both #1 and #2, and they both work. Out of those, #2 seems preferred since it is more concise. But overall I prefer the convenience initializer approach to either of those two options, as it allows the call-site to look as expected.

And thanks for the link and info on `buildExpression`.

Comment by Christopher Fuller (JIRA)

Does anyone know if `buildExpression` is required to make this work?

I tried it and it didn’t work. I think there’s some issues with single expression closures that are yet to be resolved,

Comment by Christopher Fuller (JIRA)

Ah, okay. And I can see that single expression functions do in fact work…

@ScriptBuilder
func makeScript() -> Script {
    ExampleStep()
}

output:

buildBlock called [__lldb_expr_1.ExampleStep()]
[__lldb_expr_1.ExampleStep()]

So the issue indeed appears to be limited to the single expression closures specifically.

Comment by Christopher Fuller (JIRA)

updated the ticket title to reflect closures specifically

This is covered by SR-11628