website/docs/server-config.md (1)

54-65: Good docs update; add a permalink and minor copy tweak.

• Prefer a permalink to constants (specific commit) to avoid line drift in main.
• Consider “Starknet chain info” (lowercase “chain”) for consistency.

Example link style:

• https://github.com/0xSpaceShard/starknet-devnet/blob//crates/starknet-devnet-core/src/constants.rs#L121

tests/integration/general_integration_tests.rs (1)

73-92: to_string() usage for numeric CLI args is fine.

Consistent and readable. Consider expanding env-var parity test to include the three new flags in a follow-up.

Also applies to: 105-111

crates/starknet-devnet/src/cli.rs (1)

288-292: Wiring CLI → config is correct.

Values properly populate ClassSizeConfig.

Consider adding simple validation (e.g., nonzero) if zero has no defined semantics.

crates/starknet-devnet-core/src/starknet/add_declare_transaction.rs (1)

35-37: Avoid narrowing casts when enforcing configurable limits

Casting the u64 limits down to usize loses information on 32-bit targets (and for any caller who configures a value > usize::MAX), effectively shrinking the cap and causing false rejections. Compare in the wider u64 domain instead by lifting the measured lengths to u64 (widening cast) before checking.

-    if serialized_class.len() > config.maximum_contract_class_size as usize
-        || sierra_length > config.maximum_sierra_length as usize
-        || casm_length > config.maximum_contract_bytecode_size as usize
+    let serialized_class_len_u64 = serialized_class.len() as u64;
+    let sierra_length_u64 = sierra_length as u64;
+    let casm_length_u64 = casm_length as u64;
+
+    if serialized_class_len_u64 > config.maximum_contract_class_size
+        || sierra_length_u64 > config.maximum_sierra_length
+        || casm_length_u64 > config.maximum_contract_bytecode_size

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro