chore: Apply some Clippy lints

[jbencin]

Description

Apply some Clippy lints:

Clippy lint	Details
bool_to_int_with_if	Find and replace if condition { 1 } else { 0 }
manual_assert	Replace if condition { panic!() } with assert!()
manual_string_new	Replace "".to_string() with String::new()
needless_continue	Remove continue statements which have no effect
needless_pass_by_value	Pass by reference when appropriate, for example pass &str instead of String

I only partially applied needless_pass_by_value since I didn't want this PR to result in a massive diff. I focused on large objects or functions that were called a lot. I replaced a lot of Strings with &str, which got rid of several unnecessary heap allocations

Breaking change?

No

[codecov[bot]]

Codecov Report

Attention: Patch coverage is 76.55172% with 68 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
components/clarinet-cli/src/devnet/package.rs	0.00%	16 Missing :warning:
components/stacks-network/src/orchestrator.rs	0.00%	14 Missing :warning:
components/clarinet-deployments/src/onchain/mod.rs	0.00%	7 Missing :warning:
components/clarity-repl/src/repl/session.rs	44.44%	5 Missing :warning:
components/clarity-repl/src/repl/interpreter.rs	73.33%	4 Missing :warning:
...inet-deployments/src/onchain/bitcoin_deployment.rs	0.00%	3 Missing :warning:
components/clarity-repl/src/repl/debug/cli.rs	0.00%	3 Missing :warning:
components/stacks-rpc-client/src/rpc_client.rs	0.00%	3 Missing :warning:
...ents/clarity-lsp/src/common/requests/completion.rs	0.00%	2 Missing :warning:
...larity-lsp/src/common/requests/document_symbols.rs	0.00%	2 Missing :warning:
... and 8 more

:loudspeaker: Thoughts on this report? Let us know!

[hugoclrd]

Closing for now. For too many conflict and inactivity. We may want to re-assess this PR in a near future