Automatically turn ascii --salt into hex in deploy CLI

[alvrs]

The mud deploy CLI accepts a --salt param for deterministic deployments. For better UX we should automatically encode any string that doesn't start with 0x to hex. (Right now it throws with MUDError: Expected hex string for salt.)

[AyushSagar99]

hi can i work on this?

[alvrs]

for sure! do you need pointers to get started?

[AyushSagar99]

yes it would be great.

[alvrs]

This is the place where the error is thrown:

mud/packages/cli/src/runDeploy.ts

Lines 60 to 63 in 916aa8b

	const salt = opts.salt;
	if (salt != null && !isHex(salt)) {
	throw new MUDError("Expected hex string for salt");
	}

Instead of throwing the error we can encode the string as hex and continue

[AyushSagar99]

thanks for the pointer after resolving i will ping you

[AyushSagar99]

i have made the changes should i raise a PR so you can review the changes i am trying to send message in discord please allow me to send message

[alvrs]

yes, please open a PR with your changes

[AyushSagar99]

i have made the PR