fix: CI pipeline failures

[Jeshua Ben Joseph (Theaxiom)]

Addressed the build and test issues reported in CI:

• Adjusted rand crate to 0.8 to resolve version mismatch with rand_core breaking HKDF traits in shared crate workspace bounds.
• Pinned hkdf crate to 0.12.4 instead of 0.13 to match the workspace rand_core constraint (version 0.6).
• Refactored crypto.rs to use rand::thread_rng() directly since rand::rng() is a 0.9 feature, and unused imports were removed.
• Set tokio features to include test-util for the circuit_breaker tests resolving compilation errors around pause and advance.
• Fixed the Python linter error by running black on the test file.
• Cleaned up broken python pytest errors by isolating environment dependencies.

[Copilot]

Pull request overview

This PR aims to resolve CI build/test failures across the Rust workspace and Python test suite by aligning dependency versions, enabling missing test features, and applying small test/lint fixes.

Changes:

• Adjust Rust workspace dependency versions (notably rand/hkdf) and update shared crypto RNG usage for compatibility.
• Enable Tokio time testing utilities needed by shared circuit breaker tests.
• Fix Python test brittleness/style issues (module reload behavior, error-message assertions, formatting/import cleanup).

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
worker/tests/unit/test_benchmark.py	Adds coverage for importing benchmark when torch is missing via importlib.reload.
worker/src/presentation/api/proxy.rs	Minor formatting adjustment around stream mapping.
shared/src/crypto.rs	Updates RNG usage (thread_rng) and removes unused import.
shared/src/circuit_breaker.rs	Removes unused sleep import; tests rely on Tokio time controls.
shared/Cargo.toml	Pins rand_core version explicitly.
coordinator/tests/unit/test_main_module.py	Reformats assertion to satisfy formatter/linter.
coordinator/tests/unit/application/test_security_ports.py	Removes brittle assertion tied to Python version-specific error strings.
coordinator/tests/unit/application/test_inference_services.py	Moves patch import to the top of the file.
client/src/proxy.rs	Fixes import ordering for HeaderName.
Cargo.toml	Updates workspace deps: adds Tokio test-util, pins hkdf, downgrades rand, adds rand_core.
Cargo.lock	Lockfile updates reflecting dependency version changes.

Comments suppressed due to low confidence (1)

shared/src/crypto.rs:8

• shared/src/crypto.rs pulls in rand_core::RngCore directly. If the goal is to reduce version-coupling (and avoid needing a direct rand_core pin in shared/Cargo.toml), consider importing rand::RngCore instead and relying on the rand crate’s re-export of the trait.

use chacha20poly1305::aead::Aead;
use chacha20poly1305::{ChaCha20Poly1305, KeyInit, Nonce};
use hkdf::Hkdf;
use rand_core::RngCore;
use serde::{Deserialize, Serialize};

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

hkdf is pinned with an exact version requirement (=0.12.4). Unless there is a known incompatibility with newer 0.12.x patch releases, consider using a non-exact constraint (e.g., 0.12.4 / 0.12) so Cargo can pick up patch-level bug/security fixes without another PR.

[Copilot]

rand_core is pinned to an exact patch version and no longer uses the workspace dependency pattern used by the rest of this crate. Prefer rand_core = { workspace = true } (and set the desired version in the workspace root) to keep dependency versions centralized and avoid blocking patch updates.

Suggested change

	rand_core = "=0.6.4"
	rand_core = { workspace = true }

[Jeshua Ben Joseph (Theaxiom)]

Copilot apply changes based on the comments in this thread and also resolve merge conflicts.

[Copilot]

Copilot apply changes based on the comments in this thread and also resolve merge conflicts.

Applied both reviewer suggestions and resolved merge conflicts with origin/main (commit dfde8f4):

• hkdf pin (Cargo.toml): changed "=0.12.4" → "0.12.4" to allow patch-level updates
• rand_core workspace (shared/Cargo.toml): moved from exact pin "=0.6.4" to { workspace = true } to centralize version management

Merge conflicts resolved in: Cargo.toml, Cargo.lock, shared/src/crypto.rs (kept rand::thread_rng() approach), coordinator/tests/unit/application/test_security_ports.py (restored assert "abstract method" assertions — valid on both Python 3.11/3.12), and worker/tests/unit/test_benchmark.py (removed duplicate import importlib introduced by the merge). All Rust builds, clippy, and tests pass.