Fluent Builder Interface about shortcuts-js HOT 3 CLOSED

It's really an interesting idea! The current method for building a Shortcut (as an array of functions) is by no means the way it has to be done forever, so it's interesting to see options for other ways it could be done. This isn't something I'd considered before now, but it's very cool to see - Thanks so much for putting it forward!

To be honest I've been thinking about something similar, but using chained function calls rather than the builder pattern you suggested.

At this point in time, even a basic shortcut will look horribly messy in this format. You'll end up with either this:

comment({ text: 'Hello World' })(number({ number: 42 })(calculate({ operand: 3, operation: '/' }, id => (calcId = id))(showResult({ text: withVariables`Total is ${calcId}!` }))));

Or this (which is only slightly more readable).

comment({ text: 'Hello World' })(
  number({ number: 42 })(
    calculate({ operand: 3, operation: '/' }, id => (calcId = id))(
      showResult({ text: withVariables`Total is ${calcId}!` })
    )
  )
);

Hopefully we can all agree that these are both awful!

However, with the magic of the Pipeline Operator (Currently a Stage 1 proposal), we could end up with something like this:

shortcuts
  |> comment({ text: 'Hello World' })
  |> number({ number: 42 })
  |> calculate({ operand: 3, operation: '/' }, id => (calcId = id))
  |> showResult({ text: withVariables`Total is ${calcId}!` })

Granted, I've not looked into it in enough detail to know exactly how it would be implemented or used, but it's something that's been in the back of my mind that I thought I'd share. At this point it seems very likely that the Pipeline Operator will make it's way into JS (and then hopefully TypeScript), so it's another thing to think about!

from shortcuts-js.

Since Shortcuts can also take a tree path, instead of linear, (e.g. if/else, chooseMenu), I can foresee a chained/builder approach not being an elegant solution.

Secondly, one thing to keep in mind is that import questions target their corresponding action by index, so the array approach lends some credibility. However, I have not checked what index is used for a import question on an action within a branched path.

from shortcuts-js.

Good point. This is something I was thinking about before I implemented if/else and chooseFromMenu, so it made a bit more sense then. The addition of those actions does make it a bit more complicated.

from shortcuts-js.