fix: resolve all code quality issues from #108 review#146

Merged

intendednull merged 21 commits into

mainfrom

fix/issue-108-final

Apr 13, 2026

Commits on Apr 12, 2026

fix(messaging): replace O(N²) sort-on-insert with BTreeMap channel index (#121)

intendednull
and
claude
committed
network: remove ConnectionEvent / connection_events stub (#119)

intendednull
and
claude
committed
state: O(1) message lookup via index (#122) and log buffer evictions (#123)

intendednull
and
claude
committed
crypto: add RatchetCache to eliminate O(counter) replay on decrypt

intendednull
and
claude
committed
fix(client): add actor timeouts (#130) and safe UUID parsing in topic map (#141)

intendednull
and
claude
committed
worker: sign WorkerWireMessage with Ed25519 (#117)

intendednull
and
claude
committed
web: replace browser-API .unwrap() with graceful error handling (#128, #129)

intendednull
and
claude
committed
Merge fix/108/state: O(1) message index (#122) and log evictions (#123)
intendednull
committed
Merge fix/108/crypto: RatchetCache to avoid O(counter) replay (#120)
intendednull
committed
Merge fix/108/messaging: BTreeMap channel index (#121)
intendednull
committed
Merge fix/108/network: remove connection_events placeholder (#119)
intendednull
committed
Merge fix/108/worker: sign WorkerWireMessage with Ed25519 (#117)
intendednull
committed
Merge fix/108/web: replace browser-API unwraps (#128, #129)
intendednull
committed
Merge fix/108/client: actor timeouts (#130) and safe UUID parsing (#141)
intendednull
committed
lint: enable clippy::let_underscore_must_use workspace-wide (#131)

intendednull
and
claude
committed
Merge fix/108/ci: enable clippy::let_underscore_must_use workspace-wide (#131)
intendednull
committed
docs(common): update WorkerWireMessage security note to reflect Ed25519 signing

intendednull
and
claude
committed
fix: remaining .unwrap() in voice.rs and unused label on WASM

intendednull
and
claude
committed
fix: performance audit findings — HashMap index, ratchet guard, warn on timeout

intendednull
and
claude
committed
test: add missing tests flagged by coverage audit

intendednull
and
claude
committed
style: cargo fmt

intendednull
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: resolve all code quality issues from #108 review#146

fix: resolve all code quality issues from #108 review#146
intendednull merged 21 commits into
mainfrom
fix/issue-108-final

Commits on Apr 12, 2026

fix(messaging): replace O(N²) sort-on-insert with BTreeMap channel index (#121)

network: remove ConnectionEvent / connection_events stub (#119)

state: O(1) message lookup via index (#122) and log buffer evictions (#123)

crypto: add RatchetCache to eliminate O(counter) replay on decrypt

fix(client): add actor timeouts (#130) and safe UUID parsing in topic map (#141)

worker: sign WorkerWireMessage with Ed25519 (#117)

web: replace browser-API .unwrap() with graceful error handling (#128, #129)

Merge fix/108/state: O(1) message index (#122) and log evictions (#123)

Merge fix/108/crypto: RatchetCache to avoid O(counter) replay (#120)

Merge fix/108/messaging: BTreeMap channel index (#121)

Merge fix/108/network: remove connection_events placeholder (#119)

Merge fix/108/worker: sign WorkerWireMessage with Ed25519 (#117)

Merge fix/108/web: replace browser-API unwraps (#128, #129)

Merge fix/108/client: actor timeouts (#130) and safe UUID parsing (#141)

lint: enable clippy::let_underscore_must_use workspace-wide (#131)

Merge fix/108/ci: enable clippy::let_underscore_must_use workspace-wide (#131)

docs(common): update WorkerWireMessage security note to reflect Ed25519 signing

fix: remaining .unwrap() in voice.rs and unused label on WASM

fix: performance audit findings — HashMap index, ratchet guard, warn on timeout

test: add missing tests flagged by coverage audit

style: cargo fmt