Values API default values

We know that you’re working hard on getting v2 ready for release, so it seems like a good time to formally propose an enhancement to the specification proposed in #202 to support providing default values.

The best syntax I’ve seen so far was proposed by @dancallaghan in his comment on the PR:

static values = {
  url: String,
  refreshInterval: [Number, 300],
  loadOnConnect: [Boolean, true]
}

I love this because it’s non-intrusive and falls back to the current syntax.

[dhh]

I dig this default values design where types are inferred from the values. So 👍 from here.

[paulozoom]

In that example, [Number, 300] feels redundant because 300 is already a Number. So Stimulus would also need to deal with inconsistency between declared type and default type (or decide not to, and error).

Couldn’t it be instead either the type or the value?

static values = {
  url: String,
  refreshInterval: 300,
  loadOnConnect: true,
}

The type could always be inferred from the value. (Even if typeof Array being Object is a little annoying, the Object case would always be further checked for the Array type.)