🐛 fix(cdk envs): pass env variables to CDK commands so it's accepted on Windows as valid syntax

Author: sladg

Diff for: lib/cli/deploy.ts

@@ -16,21 +16,24 @@ export const deployHandler = async ({ stackName, appPath, bootstrap, lambdaMemor
 	// All paths are absolute.
 	const cdkApp = `node ${appPath}`
 	const cdkCiFlags = `--require-approval never --ci`
-	const envConfig = [
-		`STACK_NAME=${stackName}`,
-		lambdaMemory ? `LAMBDA_MEMORY=${lambdaMemory}` : '',
-		lambdaTimeout ? `LAMBDA_TIMEOUT=${lambdaTimeout}` : '',
-		hostedZone ? `HOSTED_ZONE=${hostedZone}` : '',
-		domainNamePrefix ? `DNS_PREFIX=${domainNamePrefix}` : '',
-	].join(' ')
+
+	const variables = {
+		STACK_NAME: stackName,
+		...(lambdaMemory && { LAMBDA_MEMORY: lambdaMemory.toString() }),
+		...(lambdaTimeout && { LAMBDA_TIMEOUT: lambdaTimeout.toString() }),
+		...(hostedZone && { HOSTED_ZONE: hostedZone }),
+		...(domainNamePrefix && { DNS_PREFIX: domainNamePrefix }),
+	}
 
 	if (bootstrap) {
 		await executeAsyncCmd({
-			cmd: `${envConfig} ${cdkExecutable} bootstrap --app "${cdkApp}"`,
+			cmd: `${cdkExecutable} bootstrap --app "${cdkApp}"`,
+			env: variables,
 		})
 	}
 
 	await executeAsyncCmd({
-		cmd: `${envConfig} ${cdkExecutable} deploy --app "${cdkApp}" ${cdkCiFlags}`,
+		cmd: `${cdkExecutable} deploy --app "${cdkApp}" ${cdkCiFlags}`,
+		env: variables,
 	})
 }

Diff for: lib/utils.ts

@@ -79,15 +79,16 @@ export const zipMultipleFoldersOrFiles = async ({ outputName, inputDefinition }:
 interface CommandProps {
 	cmd: string
 	path?: string
+	env?: Record<string, string>
 }
 
-export const executeAsyncCmd = async ({ cmd, path }: CommandProps) => {
+export const executeAsyncCmd = async ({ cmd, path, env }: CommandProps) => {
 	if (path) {
 		process.chdir(path)
 	}
 
 	return new Promise((resolve, reject) => {
-		const sh = exec(cmd, (error, stdout, stderr) => {
+		const sh = exec(cmd, { env: { ...process.env, ...env } }, (error, stdout, stderr) => {
 			if (error) {
 				reject(error)
 			} else {