RFC: DX wants

[zomars]

Got a nice summary of things we may want to add/simplify to enhance overall developer experience from this issue.

Alex is responsible for most of the Stack we use today and can say with confidence that his recommendations are always on point. 👌

Please let me know if you would like to implement any of the suggestions described in the mentioned issue.

[PeerRich]

great idea. maybe @KATT wants to help with some? ❤️

[leog]

As far as environment variables, regarding making sure they exist, we currently use @chempogonzalez's dotenv-checker which already helps making sure a good report on their existence is present, but it does not error out when required variables are missing. Would be very neat if we could maybe add that functionality to dotenv-checker as a contribution, to avoid maintaining two different implementations for the same purpose because there's a possible feature missing in the library we use. Thoughts?

Example output:

[zomars]

@leog the thing is, on deployments we don't have an .env file. So @KATT 's solution using zod is actually genius since we can validate the whole process.env object. Also there's this tool by KATT as well but I don't think is still recommendable to use.

[zomars]

Also we already do some kind of validation but we could expand this further and maybe have better error messages and recommendations to print on the logs:

cal.com/apps/web/next.config.js

Lines 19 to 20 in 4cd4308

	if (!process.env.NEXTAUTH_SECRET) throw new Error("Please set NEXTAUTH_SECRET");
	if (!process.env.CALENDSO_ENCRYPTION_KEY) throw new Error("Please set CALENDSO_ENCRYPTION_KEY");

[KATT]

Env parsing

Zod is more flexible than envsafe, I outlined details on how I use zod instead nowadays here.

Zod allows you to .transform() things as well --- like you can have a group of related keys like this which is useful when you have a lot of integrations:

https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAbwF4F84DMoRHA5EiAE1wG4BYAKEoGMIA7AZ3gCMIIAbAUwEM6AVKAFdOcALxwkAOk51BIABQBtAEQwhnZQBo4ygIxadAT04NlAXQCUktbwbpoC+RbEA+OGuFxuDOLUYwLGnomOFYOHjoAMW52BhFxKRk5JWV0GLiDZQAGTLoIcysbRnsoR2dRNzTYkW9fYIDKIP8JNhAAUToANzEJSQhmACtOahh5BEo4OAAtAHkZgFkAfQBJADk+NoBxACUAQT5lmdXFttXdgCEAGTaAEQAuULYuXgFhTQm4AHpPuBm6dkMcCgnAAjoJgMDCHBgOhpnMlmsNjt9odjqcLtcbtCfB4RAAfOC6OAE4wMD6zBaLXYABWWiwA0m0AJoPKRMKDAOgAcyckhAnPkugAbBZ3hRJhSlgBhXaXS7nXZS+mLACq20urMk7M5PKsgig7CcYpQVmg8g+Un6QxGYw+EvhK3WWz2ByOJzOV1uDzCzyi6U4kkInDSgnYo2yykC4rgJsoUcaFD8IWY3k4HU6PgSfUGw1G42jN32F12AGU2qr1ZrtdzefrDaLKLGqBROAAPSCwOrNGTdcQpuLphiSXiEeQEbDpiyfABUw7oo8ki8kFmnnwTlG+cBVDG4XM42i4MFwPjA3CgcS8dDgPeA9FKMhgTRCp-PnEI6Z6PckL7itujksdJEXVRd0MS9HR9DFe1KRpOlGRZHRSxuLRyQdGU5QVJUKw1HQAAsYBgMAGDub4224cAuEkWgQGURt42bJNwkkdgIB5H830nDcfkmHjeLgAA9AB+BMYTgeR2PfLpJAAxFnRRN10U9G5nHzSZGMoli2LPOJJM6aSHVk5FXTRD1MSjSZNz4qzrIE4To3UgNNPE7SOKkgD0PlRVlTVS5zK+bibNsxtr2qRAPk3AABGAGAAWjbMBcziqAsCgOFKQ8zDvPVbFoTobhqGoEwGGAZguDgXDOGBD4JPTfT0tlTysJ8xsgA

Note: the above is untested

[KATT]

Hey gang & thank y'all so much for your support of tRPC 🙏 ❤️

Spent a while outlining more stuff here, hopefully, some of it can help you in your thought process :)