Change the usage of tokens for lazy evaluation

[eladb]

Instead of new Token(() => value), since the use case for using lazy evaluation is only for synthesis, maybe what we can do is actually provide a method in Construct, which will also give us the ability to associate the token with the construct:

new QueueResource(this, 'Resource', {
  visibilityTimeout: this.lazyEval(() => 122)
});

See #712 (comment)

[rix0rrr]

Something like

Value.lazy(() => xxx)

might even be better. But I'm having trouble thinking of a good class name to substitute for Value there.

[eladb]

The reason I was thinking to tie this into the Construct is that the mental model is the this is a value that's going to be evaluated during synthesis (under the hood it might use something like Value.lazy but when I read CDK code, it actually makes sense to be more explicit about when is this value going to be resolved.

How about this.synthesisEval(() => 122) or this.synthEval(() => 'hello').

Thoughts?

[rix0rrr]

I'm not sure the more complex name is worth it in this case.

[eladb]

Resolved