refactor: make clarity-cli to use clap

[simone-stacks]

Description

Refactors the clarity-cli command-line interface to use clap for argument parsing, replacing the previous manual argument handling.

Applicable issues

• fixes refactor: make clarity-cli to use clap #6671
• fixes https://github.com/stx-labs/core-epics/issues/66

Additional info (benefits, drawbacks, caveats)

Benefits:

• Auto-generated --help output for all commands and subcommands
• Better error messages for invalid arguments
• Cleaner, more maintainable argument parsing code

Checklist

• Test coverage for new or modified code paths
• Changelog is updated
• Required documentation changes (e.g., docs/rpc/openapi.yaml and rpc-endpoints.md for v2 endpoints, event-dispatcher.md for new events)
• New clarity functions have corresponding PR in clarity-benchmarking repo

[codecov]

Codecov Report

❌ Patch coverage is 14.23358% with 235 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.18%. Comparing base (3da96c1) to head (9e17138).
⚠️ Report is 35 commits behind head on develop.

Files with missing lines	Patch %	Lines
contrib/clarity-cli/src/main.rs	14.23%	235 Missing ⚠️

❌ Your project status has failed because the head coverage (77.18%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #6720      +/-   ##
===========================================
- Coverage    78.18%   77.18%   -1.00%     
===========================================
  Files          580      580              
  Lines       361096   361230     +134     
===========================================
- Hits        282312   278807    -3505     
- Misses       78784    82423    +3639     

Files with missing lines	Coverage Δ
contrib/clarity-cli/src/lib.rs	80.72% <ø> (+4.44%)	⬆️
contrib/stacks-inspect/src/main.rs	0.00% <ø> (ø)
contrib/clarity-cli/src/main.rs	14.49% <14.23%> (+14.49%)	⬆️

... and 102 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3da96c1...9e17138. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[simone-stacks]

stacks-core/contrib/clarity-cli/src/lib.rs

Lines 438 to 493 in 90033c3

	/// Execute program in a transient environment. To be used only by CLI tools
	/// for program evaluation, not by consensus critical code.
	pub fn vm_execute_in_epoch(
	program: &str,
	clarity_version: ClarityVersion,
	epoch: StacksEpochId,
	) -> Result<Option<Value>, VmExecutionError> {
	let contract_id = QualifiedContractIdentifier::transient();
	let mut contract_context = ContractContext::new(contract_id.clone(), clarity_version);
	let mut marf = MemoryBackingStore::new();
	let conn = marf.as_clarity_db();
	let mut global_context = GlobalContext::new(
	false,
	default_chain_id(false),
	conn,
	LimitedCostTracker::new_free(),
	epoch,
	);
	global_context.execute(|g| {
	let parsed =
	ast::build_ast(&contract_id, program, &mut (), clarity_version, epoch)?.expressions;
	eval_all(&parsed, &mut contract_context, g, None)
	})
	}
	/// Execute program in a transient environment in the latest epoch.
	/// To be used only by CLI tools for program evaluation, not by consensus
	/// critical code.
	/// TODO: Unused internally in the cli, but used in other functions - Shall we use vm_execute_in_epoch with StacksEpochId::latest() instead, so we can remove this function?
	pub fn vm_execute(
	program: &str,
	clarity_version: ClarityVersion,
	) -> Result<Option<Value>, VmExecutionError> {
	let contract_id = QualifiedContractIdentifier::transient();
	let mut contract_context = ContractContext::new(contract_id.clone(), clarity_version);
	let mut marf = MemoryBackingStore::new();
	let conn = marf.as_clarity_db();
	let mut global_context = GlobalContext::new(
	false,
	default_chain_id(false),
	conn,
	LimitedCostTracker::new_free(),
	StacksEpochId::latest(),
	);
	global_context.execute(|g| {
	let parsed = ast::build_ast(
	&contract_id,
	program,
	&mut (),
	clarity_version,
	StacksEpochId::latest(),
	)?
	.expressions;
	eval_all(&parsed, &mut contract_context, g, None)
	})
	}

I noticed that vm_execute and vm_execute_in_epoch are basically the same function - with the only difference that vm_execute just hardcodes StacksEpochId::latest().

I think we could either remove vm_execute and update all callers to pass the epoch explicitly, or turn it into a simple wrapper that calls vm_execute_in_epoch with StacksEpochId::latest() parameter.

What do you think? If it makese sense, I'd tackle this in a separate PR.

[brice-stacks]

This is a great improvement. Just a couple minor comments.

[brice-stacks]

LGTM!