refactor: replacing use of capsules 1.0 with pxe_db + nuking capsules 1.0 by benesjan · Pull Request #11885 · AztecProtocol/aztec-packages

benesjan · 2025-02-10T16:05:09Z

Capsules 1.0 where now only used in one place in the codebase and its use could have been replaced with pxe_db (which is to be renamed to capsules in a followup PR). I do that in this PR along with nuking capsules 1.0.

I will rename pxe_db to capsules in a follow-up PR.

benesjan · 2025-02-10T16:05:24Z

refactor: replacing use of capsules 1.0 with pxe_db + nuking capsules 1.0 #11885 👈 (View in Graphite)
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

benesjan · 2025-02-10T16:12:13Z

noir-projects/noir-contracts/contracts/contract_class_registerer_contract/src/main.nr

@@ -33,8 +32,7 @@ pub contract ContractClassRegisterer {
    };

    // docs:start:import_pop_capsule


The docs need to be updated. I plan on doing that in a follow-up PR once I rename pxe_db to capsules.

benesjan · 2025-02-10T16:13:37Z

yarn-project/aztec.js/src/wallet/base_wallet.ts

-  addCapsule(capsule: Fr[]): Promise<void> {
-    return this.pxe.addCapsule(capsule);
+  addCapsule(contract: AztecAddress, storageSlot: Fr, capsule: Fr[]): Promise<void> {
+    return this.pxe.addCapsule(contract, storageSlot, capsule);


Kept the capsules name here as I will rename pxe_db to it in a followup PR.

AztecBot · 2025-02-11T08:18:05Z

Docs Preview

Hey there! 👋 You can check your preview at https://67ab478e3df3ac068c7bda9a--aztec-docs-dev.netlify.app

sklppy88

Have a small question, looks great! Love the nuking of more stuff on the interface ! Also unrelated to your pr but "storageslot" being used here as a term in the pxe db is weird to me when it already has connotations to contract storage.

sklppy88 · 2025-02-11T12:20:05Z

noir-projects/noir-contracts/contracts/contract_class_registerer_contract/src/main.nr

-        let packed_public_bytecode: [Field; MAX_PACKED_PUBLIC_BYTECODE_SIZE_IN_FIELDS] =
-            unsafe { pop_capsule() };
+        let packed_public_bytecode: [Field; MAX_PACKED_PUBLIC_BYTECODE_SIZE_IN_FIELDS] = unsafe {
+            pxe_db::load(


I know before we only had one logical storage slot for pub / priv / unconstrained due to capsule design, but does it make sense still given the new design ?

It worked before with the same storage slot so I didn't bother with changing it. Would change it if it for whatever reason becomes an issue.

…ith_pxe_db

🤖 I have created a release *beep* *boop* --- <details><summary>aztec-package: 0.76.2</summary> ## [0.76.2](aztec-package-v0.76.1...aztec-package-v0.76.2) (2025-02-11) ### Miscellaneous * **logging:** Support explicit FORCE_COLOR parameter ([#11902](#11902)) ([3b3f859](3b3f859)) </details> <details><summary>barretenberg.js: 0.76.2</summary> ## [0.76.2](barretenberg.js-v0.76.1...barretenberg.js-v0.76.2) (2025-02-11) ### Miscellaneous * **barretenberg.js:** Synchronize aztec-packages versions </details> <details><summary>aztec-packages: 0.76.2</summary> ## [0.76.2](aztec-packages-v0.76.1...aztec-packages-v0.76.2) (2025-02-11) ### Features * Batch writes to the proving broker database ([#11900](#11900)) ([608f887](608f887)) ### Bug Fixes * Cleanup also post test_kind.sh ([#11886](#11886)) ([50cdb15](50cdb15)) * Dont skip wasm civc tests ([#11909](#11909)) ([0395e0b](0395e0b)) * Note hash collision ([#11869](#11869)) ([f289b7c](f289b7c)) * Orchestrator test ([#11901](#11901)) ([f1bb51c](f1bb51c)) * Smt_verification: negative bitvecs, changed gates indicies. acir_formal_proofs: noir-style signed division ([#11649](#11649)) ([4146496](4146496)) * Update path of stern logs ([#11906](#11906)) ([05afb5b](05afb5b)) ### Miscellaneous * Arm runner start fix ([#11903](#11903)) ([6c83c40](6c83c40)) * Fixing the sizes of VMs in CIVC ([#11793](#11793)) ([1afddbd](1afddbd)) * **logging:** Support explicit FORCE_COLOR parameter ([#11902](#11902)) ([3b3f859](3b3f859)) * Misc fixes to devnet deploy flow ([#11738](#11738)) ([bc4cca7](bc4cca7)) * Remove warnings from noir protocol circuits ([#11803](#11803)) ([c6cc3d3](c6cc3d3)) * Replace relative paths to noir-protocol-circuits ([74d6e6a](74d6e6a)) * Replacing use of capsules 1.0 with pxe_db + nuking capsules 1.0 ([#11885](#11885)) ([72be678](72be678)) </details> <details><summary>barretenberg: 0.76.2</summary> ## [0.76.2](barretenberg-v0.76.1...barretenberg-v0.76.2) (2025-02-11) ### Bug Fixes * Note hash collision ([#11869](#11869)) ([f289b7c](f289b7c)) * Smt_verification: negative bitvecs, changed gates indicies. acir_formal_proofs: noir-style signed division ([#11649](#11649)) ([4146496](4146496)) ### Miscellaneous * Fixing the sizes of VMs in CIVC ([#11793](#11793)) ([1afddbd](1afddbd)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release *beep* *boop* --- <details><summary>aztec-package: 0.76.2</summary> ## [0.76.2](AztecProtocol/aztec-packages@aztec-package-v0.76.1...aztec-package-v0.76.2) (2025-02-11) ### Miscellaneous * **logging:** Support explicit FORCE_COLOR parameter ([#11902](AztecProtocol/aztec-packages#11902)) ([3b3f859](AztecProtocol/aztec-packages@3b3f859)) </details> <details><summary>barretenberg.js: 0.76.2</summary> ## [0.76.2](AztecProtocol/aztec-packages@barretenberg.js-v0.76.1...barretenberg.js-v0.76.2) (2025-02-11) ### Miscellaneous * **barretenberg.js:** Synchronize aztec-packages versions </details> <details><summary>aztec-packages: 0.76.2</summary> ## [0.76.2](AztecProtocol/aztec-packages@aztec-packages-v0.76.1...aztec-packages-v0.76.2) (2025-02-11) ### Features * Batch writes to the proving broker database ([#11900](AztecProtocol/aztec-packages#11900)) ([608f887](AztecProtocol/aztec-packages@608f887)) ### Bug Fixes * Cleanup also post test_kind.sh ([#11886](AztecProtocol/aztec-packages#11886)) ([50cdb15](AztecProtocol/aztec-packages@50cdb15)) * Dont skip wasm civc tests ([#11909](AztecProtocol/aztec-packages#11909)) ([0395e0b](AztecProtocol/aztec-packages@0395e0b)) * Note hash collision ([#11869](AztecProtocol/aztec-packages#11869)) ([f289b7c](AztecProtocol/aztec-packages@f289b7c)) * Orchestrator test ([#11901](AztecProtocol/aztec-packages#11901)) ([f1bb51c](AztecProtocol/aztec-packages@f1bb51c)) * Smt_verification: negative bitvecs, changed gates indicies. acir_formal_proofs: noir-style signed division ([#11649](AztecProtocol/aztec-packages#11649)) ([4146496](AztecProtocol/aztec-packages@4146496)) * Update path of stern logs ([#11906](AztecProtocol/aztec-packages#11906)) ([05afb5b](AztecProtocol/aztec-packages@05afb5b)) ### Miscellaneous * Arm runner start fix ([#11903](AztecProtocol/aztec-packages#11903)) ([6c83c40](AztecProtocol/aztec-packages@6c83c40)) * Fixing the sizes of VMs in CIVC ([#11793](AztecProtocol/aztec-packages#11793)) ([1afddbd](AztecProtocol/aztec-packages@1afddbd)) * **logging:** Support explicit FORCE_COLOR parameter ([#11902](AztecProtocol/aztec-packages#11902)) ([3b3f859](AztecProtocol/aztec-packages@3b3f859)) * Misc fixes to devnet deploy flow ([#11738](AztecProtocol/aztec-packages#11738)) ([bc4cca7](AztecProtocol/aztec-packages@bc4cca7)) * Remove warnings from noir protocol circuits ([#11803](AztecProtocol/aztec-packages#11803)) ([c6cc3d3](AztecProtocol/aztec-packages@c6cc3d3)) * Replace relative paths to noir-protocol-circuits ([74d6e6a](AztecProtocol/aztec-packages@74d6e6a)) * Replacing use of capsules 1.0 with pxe_db + nuking capsules 1.0 ([#11885](AztecProtocol/aztec-packages#11885)) ([72be678](AztecProtocol/aztec-packages@72be678)) </details> <details><summary>barretenberg: 0.76.2</summary> ## [0.76.2](AztecProtocol/aztec-packages@barretenberg-v0.76.1...barretenberg-v0.76.2) (2025-02-11) ### Bug Fixes * Note hash collision ([#11869](AztecProtocol/aztec-packages#11869)) ([f289b7c](AztecProtocol/aztec-packages@f289b7c)) * Smt_verification: negative bitvecs, changed gates indicies. acir_formal_proofs: noir-style signed division ([#11649](AztecProtocol/aztec-packages#11649)) ([4146496](AztecProtocol/aztec-packages@4146496)) ### Miscellaneous * Fixing the sizes of VMs in CIVC ([#11793](AztecProtocol/aztec-packages#11793)) ([1afddbd](AztecProtocol/aztec-packages@1afddbd)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

benesjan force-pushed the 02-10-refactor_replacing_use_of_capsules_with_pxe_db branch from b54c276 to 7fa607e Compare February 10, 2025 16:10

benesjan commented Feb 10, 2025

View reviewed changes

benesjan changed the title ~~refactor: replacing use of capsules with pxe_db~~ refactor: replacing use of capsules 1.0 with pxe_db + nuking capsules 1.0 Feb 10, 2025

benesjan added the e2e-all label Feb 10, 2025

benesjan marked this pull request as ready for review February 10, 2025 16:21

benesjan added 4 commits February 11, 2025 07:56

refactor: replacing use of capsules with pxe_db

22656d6

docs fix

cc2a554

docs fix

3051ea4

fsfds

7d28d49

benesjan force-pushed the 02-10-refactor_replacing_use_of_capsules_with_pxe_db branch from 7fa607e to 7d28d49 Compare February 11, 2025 07:56

benesjan requested a review from sklppy88 February 11, 2025 08:58

sklppy88 approved these changes Feb 11, 2025

View reviewed changes

benesjan removed the e2e-all label Feb 11, 2025

Merge branch 'master' into 02-10-refactor_replacing_use_of_capsules_w…

c0fdc3f

…ith_pxe_db

benesjan enabled auto-merge (squash) February 11, 2025 12:25

benesjan merged commit 72be678 into master Feb 11, 2025
52 checks passed

benesjan deleted the 02-10-refactor_replacing_use_of_capsules_with_pxe_db branch February 11, 2025 13:05

AztecBot mentioned this pull request Feb 11, 2025

chore(master): Release 0.76.2 #11899

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor: replacing use of capsules 1.0 with pxe_db + nuking capsules 1.0#11885

refactor: replacing use of capsules 1.0 with pxe_db + nuking capsules 1.0#11885
benesjan merged 5 commits intomasterfrom
02-10-refactor_replacing_use_of_capsules_with_pxe_db

benesjan commented Feb 10, 2025 •

edited

Loading

Uh oh!

benesjan commented Feb 10, 2025

Uh oh!

benesjan Feb 10, 2025

Uh oh!

benesjan Feb 10, 2025

Uh oh!

AztecBot commented Feb 11, 2025 •

edited

Loading

Uh oh!

sklppy88 left a comment •

edited

Loading

Uh oh!

sklppy88 Feb 11, 2025

Uh oh!

benesjan Feb 11, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		@@ -33,8 +32,7 @@ pub contract ContractClassRegisterer {
		};

		// docs:start:import_pop_capsule

Conversation

benesjan commented Feb 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

benesjan commented Feb 10, 2025

Uh oh!

benesjan Feb 10, 2025

Choose a reason for hiding this comment

Uh oh!

benesjan Feb 10, 2025

Choose a reason for hiding this comment

Uh oh!

AztecBot commented Feb 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Docs Preview

Uh oh!

sklppy88 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sklppy88 Feb 11, 2025

Choose a reason for hiding this comment

Uh oh!

benesjan Feb 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

benesjan commented Feb 10, 2025 •

edited

Loading

AztecBot commented Feb 11, 2025 •

edited

Loading

sklppy88 left a comment •

edited

Loading

benesjan Feb 11, 2025 •

edited

Loading