specify commit hash not work

[sko00o]

Describe the bug

comfy-cli: v1.3.1

I want to install specify version of ComfyUI using following command:

COMFYUI_COMMIT=8a5281006f178f9aa0e8cbd4fc29e2f3c119d2e2
comfy install --version nightly --commit ${COMFYUI_COMMIT}

But I still got the latest commit.

To Reproduce
Steps to reproduce the behavior

cd /tmp
COMFYUI_COMMIT=8a5281006f178f9aa0e8cbd4fc29e2f3c119d2e2
comfy --skip-prompt --here install --cpu --version nightly --commit ${COMFYUI_COMMIT}

# view current commit hash
cd ./ComfyUI && git rev-parse HEAD

Terminal output:

# ... omit install output ...
41886af138939719b86f30bacbafc76a142d89de

Expected behavior

I need --commit option works.

Nice to have

• Terminal output
• Screenshots

Additional context
Add any other context about the problem here.

[sko00o]

I think this bug comes from the callback function validate_commit_and_version didn't return value.

comfy-cli/comfy_cli/cmdline.py

Lines 140 to 146 in b2ba7d0

	def validate_commit_and_version(commit: Optional[str], ctx: typer.Context):
	"""
	Validate that the commit is not specified unless the version is 'nightly'.
	"""
	version = ctx.params.get("version")
	if commit and version != "nightly":
	raise typer.BadParameter("You can only specify the commit if the version is 'nightly'.")

I fixed it and send a PR (#213).

[robinjhuang]

Thank you @sko00o! Merged and will make a new version.