use Proxy for TaggedEnum constructors

[tim-smart]

No description provided.

[changeset-bot]

🦋 Changeset detected

Latest commit: a5bab07

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
effect	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[mikearnaldi]

We may want to change:

const Result = Data.taggedEnum<ResultDefinition>()
const Success = Result.Success
const Failure = Result.Failure

to:

const { Success, Failure } = Data.taggedEnum<ResultDefinition>()

as a convention

[tim-smart]

Sounds good. I added back support for the previous syntax - but I feel like the api is too complex with both.

[tim-smart]

Sounds good. I added back support for the previous syntax - but I feel like the api is too complex with both.

Yeah if we push for destructuring as the recommended usage pattern, I think dropping the function constructors is reasonable.

[mikearnaldi]

Sounds good. I added back support for the previous syntax - but I feel like the api is too complex with both.

Yeah if we push for destructuring as the recommended usage pattern, I think dropping the function constructors is reasonable.

I kind of agree yeah