fix: do not broadcast sk_poly_sum and es_poly_sum

[ctrlc03]

Summary by CodeRabbit

• Refactor
  • Simplified internal event structures by removing redundant data fields from the decryption key sharing process. These changes optimize the system's event handling and reduce payload size during key agreement operations.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
crisp	Ready	Preview, Comment	Apr 2, 2026 9:16am
enclave-docs	Ready	Preview, Comment	Apr 2, 2026 9:16am

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: eb7aff72-16e7-45f1-a622-faae9ec78680

📥 Commits

Reviewing files that changed from the base of the PR and between 35c4657 and ea092f4.

📒 Files selected for processing (4)

• crates/events/src/enclave_event/decryption_key_shared.rs
• crates/events/src/enclave_event/decryption_share_proofs.rs
• crates/keyshare/src/threshold_keyshare.rs
• crates/zk-prover/src/actors/proof_request.rs

💤 Files with no reviewable changes (3)

• crates/events/src/enclave_event/decryption_key_shared.rs
• crates/keyshare/src/threshold_keyshare.rs
• crates/zk-prover/src/actors/proof_request.rs

---

📝 Walkthrough

Walkthrough

This PR removes the sk_poly_sum and es_poly_sum polynomial fields from the decryption key sharing event flow, eliminating their storage and propagation across DecryptionKeyShared, DecryptionShareProofsPending, and related proof handling components.

Changes

Cohort / File(s)	Summary
Decryption Key Event Structs crates/events/src/enclave_event/decryption_key_shared.rs, crates/events/src/enclave_event/decryption_share_proofs.rs	Removed sk_poly_sum: ArcBytes and es_poly_sum: Vec<ArcBytes> public fields from DecryptionKeyShared and DecryptionShareProofsPending structs; updated doc comments to reflect removal of aggregated polynomial data.
Proof Derivation Logic crates/keyshare/src/threshold_keyshare.rs	Removed derivation and conversion of sk_poly_sum_bytes and es_poly_sum_bytes in handle_calculate_decryption_key_response handler; eliminated their population into DecryptionShareProofsPending event payload.
Proof Request State crates/zk-prover/src/actors/proof_request.rs	Removed sk_poly_sum and es_poly_sum field storage from PendingDecryptionProofs struct; stopped copying and threading these fields through DecryptionShareProofsPending message handling and DecryptionKeyShared event publication.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• chore: refactor proof gen to zk-prover [skip-line-limit] #1372: Introduced the sk_poly_sum and es_poly_sum polynomial fields for C4 proof aggregation in decryption key sharing; this PR removes them from the data flow.

Suggested reviewers

• cedoor
• 0xjei

Poem

🐰 Polynomials have had their day,
Simplifying the cryptic way,
Less data flows through ev'ry gate,
The proofs still work, now light and straight!
✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing broadcast of sk_poly_sum and es_poly_sum fields across multiple modules.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/do-not-broadcast-sk-esm

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cedoor]

utACK

[0xjei]

utACK