@aztec/pxe 0.0.1-commit.3e3d0c9cd → 0.0.1-commit.3f5453c7b

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (155) hide show
  1. package/dest/bin/check_oracle_version.js +4 -4
  2. package/dest/block_synchronizer/block_synchronizer.d.ts +1 -1
  3. package/dest/block_synchronizer/block_synchronizer.d.ts.map +1 -1
  4. package/dest/block_synchronizer/block_synchronizer.js +6 -0
  5. package/dest/contract_function_simulator/contract_function_simulator.d.ts +3 -4
  6. package/dest/contract_function_simulator/contract_function_simulator.d.ts.map +1 -1
  7. package/dest/contract_function_simulator/contract_function_simulator.js +13 -4
  8. package/dest/contract_function_simulator/ephemeral_array_service.d.ts +28 -0
  9. package/dest/contract_function_simulator/ephemeral_array_service.d.ts.map +1 -0
  10. package/dest/contract_function_simulator/ephemeral_array_service.js +78 -0
  11. package/dest/contract_function_simulator/noir-structs/event_validation_request.d.ts +3 -4
  12. package/dest/contract_function_simulator/noir-structs/event_validation_request.d.ts.map +1 -1
  13. package/dest/contract_function_simulator/noir-structs/event_validation_request.js +3 -6
  14. package/dest/contract_function_simulator/noir-structs/log_retrieval_request.d.ts +1 -1
  15. package/dest/contract_function_simulator/noir-structs/log_retrieval_request.js +1 -1
  16. package/dest/contract_function_simulator/noir-structs/log_retrieval_response.d.ts +1 -1
  17. package/dest/contract_function_simulator/noir-structs/log_retrieval_response.js +1 -1
  18. package/dest/contract_function_simulator/noir-structs/note_validation_request.d.ts +3 -4
  19. package/dest/contract_function_simulator/noir-structs/note_validation_request.d.ts.map +1 -1
  20. package/dest/contract_function_simulator/noir-structs/note_validation_request.js +3 -6
  21. package/dest/contract_function_simulator/oracle/interfaces.d.ts +31 -19
  22. package/dest/contract_function_simulator/oracle/interfaces.d.ts.map +1 -1
  23. package/dest/contract_function_simulator/oracle/legacy_oracle_mappings.d.ts +1 -1
  24. package/dest/contract_function_simulator/oracle/legacy_oracle_mappings.d.ts.map +1 -1
  25. package/dest/contract_function_simulator/oracle/legacy_oracle_mappings.js +28 -24
  26. package/dest/contract_function_simulator/oracle/oracle.d.ts +50 -19
  27. package/dest/contract_function_simulator/oracle/oracle.d.ts.map +1 -1
  28. package/dest/contract_function_simulator/oracle/oracle.js +177 -41
  29. package/dest/contract_function_simulator/oracle/private_execution.js +1 -1
  30. package/dest/contract_function_simulator/oracle/private_execution_oracle.d.ts +7 -11
  31. package/dest/contract_function_simulator/oracle/private_execution_oracle.d.ts.map +1 -1
  32. package/dest/contract_function_simulator/oracle/private_execution_oracle.js +16 -9
  33. package/dest/contract_function_simulator/oracle/utility_execution_oracle.d.ts +61 -38
  34. package/dest/contract_function_simulator/oracle/utility_execution_oracle.d.ts.map +1 -1
  35. package/dest/contract_function_simulator/oracle/utility_execution_oracle.js +210 -92
  36. package/dest/contract_function_simulator/pick_notes.d.ts +1 -1
  37. package/dest/contract_function_simulator/pick_notes.d.ts.map +1 -1
  38. package/dest/contract_function_simulator/pick_notes.js +9 -2
  39. package/dest/contract_function_simulator/proxied_contract_data_source.d.ts +1 -1
  40. package/dest/contract_function_simulator/proxied_contract_data_source.d.ts.map +1 -1
  41. package/dest/contract_function_simulator/proxied_contract_data_source.js +3 -0
  42. package/dest/contract_logging.d.ts +9 -4
  43. package/dest/contract_logging.d.ts.map +1 -1
  44. package/dest/contract_logging.js +21 -6
  45. package/dest/contract_sync/contract_sync_service.d.ts +6 -5
  46. package/dest/contract_sync/contract_sync_service.d.ts.map +1 -1
  47. package/dest/contract_sync/contract_sync_service.js +44 -37
  48. package/dest/contract_sync/helpers.d.ts +2 -3
  49. package/dest/contract_sync/helpers.d.ts.map +1 -1
  50. package/dest/contract_sync/helpers.js +7 -2
  51. package/dest/debug/pxe_debug_utils.d.ts +3 -3
  52. package/dest/debug/pxe_debug_utils.d.ts.map +1 -1
  53. package/dest/entrypoints/client/bundle/index.d.ts +1 -2
  54. package/dest/entrypoints/client/bundle/index.d.ts.map +1 -1
  55. package/dest/entrypoints/client/bundle/index.js +0 -1
  56. package/dest/entrypoints/client/bundle/utils.d.ts +2 -2
  57. package/dest/entrypoints/client/bundle/utils.d.ts.map +1 -1
  58. package/dest/entrypoints/client/bundle/utils.js +2 -2
  59. package/dest/entrypoints/client/lazy/index.d.ts +1 -2
  60. package/dest/entrypoints/client/lazy/index.d.ts.map +1 -1
  61. package/dest/entrypoints/client/lazy/index.js +0 -1
  62. package/dest/entrypoints/client/lazy/utils.d.ts +2 -2
  63. package/dest/entrypoints/client/lazy/utils.d.ts.map +1 -1
  64. package/dest/entrypoints/client/lazy/utils.js +2 -2
  65. package/dest/entrypoints/pxe_creation_options.d.ts +3 -1
  66. package/dest/entrypoints/pxe_creation_options.d.ts.map +1 -1
  67. package/dest/entrypoints/pxe_creation_options.js +3 -1
  68. package/dest/entrypoints/server/index.d.ts +2 -3
  69. package/dest/entrypoints/server/index.d.ts.map +1 -1
  70. package/dest/entrypoints/server/index.js +1 -2
  71. package/dest/entrypoints/server/utils.d.ts +2 -2
  72. package/dest/entrypoints/server/utils.d.ts.map +1 -1
  73. package/dest/entrypoints/server/utils.js +2 -2
  74. package/dest/events/event_service.d.ts +3 -2
  75. package/dest/events/event_service.d.ts.map +1 -1
  76. package/dest/events/event_service.js +16 -4
  77. package/dest/logs/log_service.d.ts +5 -8
  78. package/dest/logs/log_service.d.ts.map +1 -1
  79. package/dest/logs/log_service.js +24 -37
  80. package/dest/messages/message_context_service.d.ts +3 -3
  81. package/dest/messages/message_context_service.d.ts.map +1 -1
  82. package/dest/messages/message_context_service.js +3 -3
  83. package/dest/notes/note_service.d.ts +4 -5
  84. package/dest/notes/note_service.d.ts.map +1 -1
  85. package/dest/notes/note_service.js +14 -5
  86. package/dest/notes_filter.d.ts +2 -3
  87. package/dest/notes_filter.d.ts.map +1 -1
  88. package/dest/oracle_version.d.ts +4 -3
  89. package/dest/oracle_version.d.ts.map +1 -1
  90. package/dest/oracle_version.js +20 -10
  91. package/dest/pxe.d.ts +4 -5
  92. package/dest/pxe.d.ts.map +1 -1
  93. package/dest/pxe.js +12 -6
  94. package/dest/storage/capsule_store/capsule_service.d.ts +21 -0
  95. package/dest/storage/capsule_store/capsule_service.d.ts.map +1 -0
  96. package/dest/storage/capsule_store/capsule_service.js +50 -0
  97. package/dest/storage/capsule_store/capsule_store.d.ts +9 -9
  98. package/dest/storage/capsule_store/capsule_store.d.ts.map +1 -1
  99. package/dest/storage/capsule_store/capsule_store.js +33 -28
  100. package/dest/storage/capsule_store/index.d.ts +2 -1
  101. package/dest/storage/capsule_store/index.d.ts.map +1 -1
  102. package/dest/storage/capsule_store/index.js +1 -0
  103. package/dest/storage/metadata.d.ts +1 -1
  104. package/dest/storage/metadata.js +1 -1
  105. package/dest/storage/note_store/note_store.d.ts +1 -1
  106. package/dest/storage/note_store/note_store.d.ts.map +1 -1
  107. package/dest/storage/note_store/note_store.js +2 -2
  108. package/package.json +16 -16
  109. package/src/bin/check_oracle_version.ts +4 -4
  110. package/src/block_synchronizer/block_synchronizer.ts +6 -0
  111. package/src/contract_function_simulator/contract_function_simulator.ts +17 -7
  112. package/src/contract_function_simulator/ephemeral_array_service.ts +110 -0
  113. package/src/contract_function_simulator/noir-structs/event_validation_request.ts +1 -4
  114. package/src/contract_function_simulator/noir-structs/log_retrieval_request.ts +1 -1
  115. package/src/contract_function_simulator/noir-structs/log_retrieval_response.ts +1 -1
  116. package/src/contract_function_simulator/noir-structs/note_validation_request.ts +1 -4
  117. package/src/contract_function_simulator/oracle/interfaces.ts +46 -17
  118. package/src/contract_function_simulator/oracle/legacy_oracle_mappings.ts +20 -58
  119. package/src/contract_function_simulator/oracle/oracle.ts +242 -36
  120. package/src/contract_function_simulator/oracle/private_execution.ts +1 -1
  121. package/src/contract_function_simulator/oracle/private_execution_oracle.ts +19 -14
  122. package/src/contract_function_simulator/oracle/utility_execution_oracle.ts +351 -118
  123. package/src/contract_function_simulator/pick_notes.ts +9 -2
  124. package/src/contract_function_simulator/proxied_contract_data_source.ts +8 -1
  125. package/src/contract_logging.ts +18 -5
  126. package/src/contract_sync/contract_sync_service.ts +77 -59
  127. package/src/contract_sync/helpers.ts +4 -4
  128. package/src/debug/pxe_debug_utils.ts +3 -3
  129. package/src/entrypoints/client/bundle/index.ts +0 -1
  130. package/src/entrypoints/client/bundle/utils.ts +2 -3
  131. package/src/entrypoints/client/lazy/index.ts +0 -1
  132. package/src/entrypoints/client/lazy/utils.ts +2 -3
  133. package/src/entrypoints/pxe_creation_options.ts +7 -0
  134. package/src/entrypoints/server/index.ts +1 -2
  135. package/src/entrypoints/server/utils.ts +2 -3
  136. package/src/events/event_service.ts +17 -4
  137. package/src/logs/log_service.ts +52 -78
  138. package/src/messages/message_context_service.ts +3 -4
  139. package/src/notes/note_service.ts +18 -8
  140. package/src/notes_filter.ts +1 -3
  141. package/src/oracle_version.ts +20 -10
  142. package/src/pxe.ts +17 -12
  143. package/src/storage/capsule_store/capsule_service.ts +90 -0
  144. package/src/storage/capsule_store/capsule_store.ts +34 -26
  145. package/src/storage/capsule_store/index.ts +1 -0
  146. package/src/storage/metadata.ts +1 -1
  147. package/src/storage/note_store/note_store.ts +2 -5
  148. package/dest/access_scopes.d.ts +0 -9
  149. package/dest/access_scopes.d.ts.map +0 -1
  150. package/dest/access_scopes.js +0 -6
  151. package/dest/contract_function_simulator/noir-structs/message_tx_context.d.ts +0 -16
  152. package/dest/contract_function_simulator/noir-structs/message_tx_context.d.ts.map +0 -1
  153. package/dest/contract_function_simulator/noir-structs/message_tx_context.js +0 -57
  154. package/src/access_scopes.ts +0 -9
  155. package/src/contract_function_simulator/noir-structs/message_tx_context.ts +0 -55
@@ -1,22 +1,13 @@
1
- import type { Fr } from '@aztec/foundation/curves/bn254';
2
1
  import { type Logger, type LoggerBindings, createLogger } from '@aztec/foundation/log';
3
2
  import type { KeyStore } from '@aztec/key-store';
4
3
  import { AztecAddress } from '@aztec/stdlib/aztec-address';
5
4
  import type { AztecNode } from '@aztec/stdlib/interfaces/server';
6
- import {
7
- ExtendedDirectionalAppTaggingSecret,
8
- PendingTaggedLog,
9
- SiloedTag,
10
- Tag,
11
- TxScopedL2Log,
12
- } from '@aztec/stdlib/logs';
5
+ import { ExtendedDirectionalAppTaggingSecret, PendingTaggedLog, SiloedTag, Tag } from '@aztec/stdlib/logs';
13
6
  import type { BlockHeader } from '@aztec/stdlib/tx';
14
7
 
15
- import type { AccessScopes } from '../access_scopes.js';
16
8
  import type { LogRetrievalRequest } from '../contract_function_simulator/noir-structs/log_retrieval_request.js';
17
9
  import { LogRetrievalResponse } from '../contract_function_simulator/noir-structs/log_retrieval_response.js';
18
10
  import { AddressStore } from '../storage/address_store/address_store.js';
19
- import { CapsuleStore } from '../storage/capsule_store/capsule_store.js';
20
11
  import type { RecipientTaggingStore } from '../storage/tagging_store/recipient_tagging_store.js';
21
12
  import type { SenderAddressBookStore } from '../storage/tagging_store/sender_address_book_store.js';
22
13
  import {
@@ -32,7 +23,6 @@ export class LogService {
32
23
  private readonly aztecNode: AztecNode,
33
24
  private readonly anchorBlockHeader: BlockHeader,
34
25
  private readonly keyStore: KeyStore,
35
- private readonly capsuleStore: CapsuleStore,
36
26
  private readonly recipientTaggingStore: RecipientTaggingStore,
37
27
  private readonly senderAddressBookStore: SenderAddressBookStore,
38
28
  private readonly addressStore: AddressStore,
@@ -42,7 +32,16 @@ export class LogService {
42
32
  this.log = createLogger('pxe:log_service', bindings);
43
33
  }
44
34
 
45
- public async bulkRetrieveLogs(logRetrievalRequests: LogRetrievalRequest[]): Promise<(LogRetrievalResponse | null)[]> {
35
+ public async fetchLogsByTag(
36
+ contractAddress: AztecAddress,
37
+ logRetrievalRequests: LogRetrievalRequest[],
38
+ ): Promise<(LogRetrievalResponse | null)[]> {
39
+ for (const request of logRetrievalRequests) {
40
+ if (!contractAddress.equals(request.contractAddress)) {
41
+ throw new Error(`Got a log retrieval request from ${request.contractAddress}, expected ${contractAddress}`);
42
+ }
43
+ }
44
+
46
45
  return await Promise.all(
47
46
  logRetrievalRequests.map(async request => {
48
47
  const [publicLog, privateLog] = await Promise.all([
@@ -51,8 +50,8 @@ export class LogService {
51
50
  ]);
52
51
 
53
52
  if (publicLog !== null && privateLog !== null) {
54
- throw new Error(
55
- `Found both a public and private log when searching for tag ${request.tag} from contract ${request.contractAddress}`,
53
+ this.log.warn(
54
+ `Found both a public and private log for tag ${request.tag} from contract ${request.contractAddress}. This may indicate a contract bug. Returning the public log.`,
56
55
  );
57
56
  }
58
57
 
@@ -74,9 +73,8 @@ export class LogService {
74
73
  if (logsForTag.length === 0) {
75
74
  return null;
76
75
  } else if (logsForTag.length > 1) {
77
- // TODO(#11627): handle this case
78
- throw new Error(
79
- `Got ${logsForTag.length} logs for tag ${tag} and contract ${contractAddress.toString()}. getPublicLogByTag currently only supports a single log per tag`,
76
+ this.log.warn(
77
+ `Expected at most 1 public log for tag ${tag} and contract ${contractAddress.toString()}, got ${logsForTag.length}. This may indicate a contract bug. Returning the first log.`,
80
78
  );
81
79
  }
82
80
 
@@ -98,9 +96,8 @@ export class LogService {
98
96
  if (logsForTag.length === 0) {
99
97
  return null;
100
98
  } else if (logsForTag.length > 1) {
101
- // TODO(#11627): handle this case
102
- throw new Error(
103
- `Got ${logsForTag.length} logs for tag ${siloedTag}. getPrivateLogByTag currently only supports a single log per tag`,
99
+ this.log.warn(
100
+ `Expected at most 1 private log for tag ${siloedTag}, got ${logsForTag.length}. This may indicate a contract bug. Returning the first log.`,
104
101
  );
105
102
  }
106
103
 
@@ -114,46 +111,36 @@ export class LogService {
114
111
  );
115
112
  }
116
113
 
117
- public async fetchTaggedLogs(contractAddress: AztecAddress, pendingTaggedLogArrayBaseSlot: Fr, scopes: AccessScopes) {
114
+ public async fetchTaggedLogs(contractAddress: AztecAddress, recipient: AztecAddress): Promise<PendingTaggedLog[]> {
118
115
  this.log.verbose(`Fetching tagged logs for ${contractAddress.toString()}`);
119
116
 
120
117
  // We only load logs from block up to and including the anchor block number
121
118
  const anchorBlockNumber = this.anchorBlockHeader.getBlockNumber();
122
119
  const anchorBlockHash = await this.anchorBlockHeader.hash();
123
120
 
124
- // Determine recipients: use scopes if provided, otherwise get all accounts
125
- const recipients = scopes !== 'ALL_SCOPES' && scopes.length > 0 ? scopes : await this.keyStore.getAccounts();
126
-
127
- // For each recipient, fetch secrets, load logs, and store them.
128
- // We run these per-recipient tasks in parallel so that logs are loaded for all recipients concurrently.
129
- await Promise.all(
130
- recipients.map(async recipient => {
131
- // Get all secrets for this recipient (one per sender)
132
- const secrets = await this.#getSecretsForSenders(contractAddress, recipient);
133
-
134
- // Load logs for all sender-recipient pairs in parallel
135
- const logArrays = await Promise.all(
136
- secrets.map(secret =>
137
- loadPrivateLogsForSenderRecipientPair(
138
- secret,
139
- this.aztecNode,
140
- this.recipientTaggingStore,
141
- anchorBlockNumber,
142
- anchorBlockHash,
143
- this.jobId,
144
- ),
145
- ),
146
- );
147
-
148
- // Flatten all logs from all secrets
149
- const allLogs = logArrays.flat();
150
-
151
- // Store the logs for this recipient
152
- if (allLogs.length > 0) {
153
- await this.#storePendingTaggedLogs(contractAddress, pendingTaggedLogArrayBaseSlot, recipient, allLogs);
154
- }
155
- }),
121
+ // Get all secrets for this recipient (one per sender)
122
+ const secrets = await this.#getSecretsForSenders(contractAddress, recipient);
123
+
124
+ // Load logs for all sender-recipient pairs in parallel
125
+ const logArrays = await Promise.all(
126
+ secrets.map(secret =>
127
+ loadPrivateLogsForSenderRecipientPair(
128
+ secret,
129
+ this.aztecNode,
130
+ this.recipientTaggingStore,
131
+ anchorBlockNumber,
132
+ anchorBlockHash,
133
+ this.jobId,
134
+ ),
135
+ ),
156
136
  );
137
+
138
+ return logArrays
139
+ .flat()
140
+ .map(
141
+ scopedLog =>
142
+ new PendingTaggedLog(scopedLog.logData, scopedLog.txHash, scopedLog.noteHashes, scopedLog.firstNullifier),
143
+ );
157
144
  }
158
145
 
159
146
  async #getSecretsForSenders(
@@ -176,38 +163,25 @@ export class LogService {
176
163
  );
177
164
 
178
165
  return Promise.all(
179
- deduplicatedSenders.map(sender => {
180
- return ExtendedDirectionalAppTaggingSecret.compute(
166
+ deduplicatedSenders.map(async sender => {
167
+ const secret = await ExtendedDirectionalAppTaggingSecret.compute(
181
168
  recipientCompleteAddress,
182
169
  recipientIvsk,
183
170
  sender,
184
171
  contractAddress,
185
172
  recipient,
186
173
  );
187
- }),
188
- );
189
- }
190
-
191
- #storePendingTaggedLogs(
192
- contractAddress: AztecAddress,
193
- capsuleArrayBaseSlot: Fr,
194
- recipient: AztecAddress,
195
- privateLogs: TxScopedL2Log[],
196
- ) {
197
- // Build all pending tagged logs from the scoped logs
198
- const pendingTaggedLogs = privateLogs.map(scopedLog => {
199
- const pendingTaggedLog = new PendingTaggedLog(
200
- scopedLog.logData,
201
- scopedLog.txHash,
202
- scopedLog.noteHashes,
203
- scopedLog.firstNullifier,
204
- recipient,
205
- );
206
174
 
207
- return pendingTaggedLog.toFields();
208
- });
175
+ if (!secret) {
176
+ // Note that all senders originate from either the SenderAddressBookStore or the KeyStore.
177
+ // TODO(F-512): make sure we actually prevent registering invalid senders.
178
+ throw new Error(
179
+ `Failed to compute a tagging secret for sender ${sender} - this implies this is an invalid address, which should not happen as they have been previously registered in PXE.`,
180
+ );
181
+ }
209
182
 
210
- // TODO: This looks like it could belong more at the oracle interface level
211
- return this.capsuleStore.appendToCapsuleArray(contractAddress, capsuleArrayBaseSlot, pendingTaggedLogs, this.jobId);
183
+ return secret;
184
+ }),
185
+ );
212
186
  }
213
187
  }
@@ -1,9 +1,8 @@
1
1
  import { Fr } from '@aztec/foundation/curves/bn254';
2
2
  import type { AztecNode } from '@aztec/stdlib/interfaces/server';
3
+ import { MessageContext } from '@aztec/stdlib/logs';
3
4
  import { TxHash } from '@aztec/stdlib/tx';
4
5
 
5
- import { MessageTxContext } from '../contract_function_simulator/noir-structs/message_tx_context.js';
6
-
7
6
  /** Resolves transaction hashes into the context needed to process messages. */
8
7
  export class MessageContextService {
9
8
  constructor(private readonly aztecNode: AztecNode) {}
@@ -15,7 +14,7 @@ export class MessageContextService {
15
14
  * process messages that originated from that transaction. Returns `null` for tx hashes that are zero, not yet
16
15
  * available, or in blocks beyond the anchor block.
17
16
  */
18
- resolveMessageContexts(txHashes: Fr[], anchorBlockNumber: number): Promise<(MessageTxContext | null)[]> {
17
+ getMessageContextsByTxHash(txHashes: Fr[], anchorBlockNumber: number): Promise<(MessageContext | null)[]> {
19
18
  // TODO: optimize, we might be hitting the node to get the same txHash repeatedly
20
19
  return Promise.all(
21
20
  txHashes.map(async txHashField => {
@@ -38,7 +37,7 @@ export class MessageContextService {
38
37
  throw new Error(`Tx effect for ${txHash} has no nullifiers`);
39
38
  }
40
39
 
41
- return new MessageTxContext(data.txHash, data.noteHashes, data.nullifiers[0]);
40
+ return new MessageContext(data.txHash, data.noteHashes, data.nullifiers[0]);
42
41
  }),
43
42
  );
44
43
  }
@@ -7,7 +7,6 @@ import { Note, NoteDao, NoteStatus } from '@aztec/stdlib/note';
7
7
  import { MerkleTreeId } from '@aztec/stdlib/trees';
8
8
  import type { BlockHeader, TxHash } from '@aztec/stdlib/tx';
9
9
 
10
- import type { AccessScopes } from '../access_scopes.js';
11
10
  import type { NoteStore } from '../storage/note_store/note_store.js';
12
11
 
13
12
  export class NoteService {
@@ -32,7 +31,7 @@ export class NoteService {
32
31
  owner: AztecAddress | undefined,
33
32
  storageSlot: Fr,
34
33
  status: NoteStatus,
35
- scopes: AccessScopes,
34
+ scopes: AztecAddress[],
36
35
  ) {
37
36
  const noteDaos = await this.noteStore.getNotes(
38
37
  {
@@ -71,7 +70,7 @@ export class NoteService {
71
70
  *
72
71
  * @param contractAddress - The contract whose notes should be checked and nullified.
73
72
  */
74
- public async syncNoteNullifiers(contractAddress: AztecAddress, scopes: AccessScopes): Promise<void> {
73
+ public async syncNoteNullifiers(contractAddress: AztecAddress, scopes: AztecAddress[]): Promise<void> {
75
74
  const anchorBlockHash = await this.anchorBlockHeader.hash();
76
75
 
77
76
  const contractNotes = await this.noteStore.getNotes({ contractAddress, scopes }, this.jobId);
@@ -121,7 +120,7 @@ export class NoteService {
121
120
  noteHash: Fr,
122
121
  nullifier: Fr,
123
122
  txHash: TxHash,
124
- recipient: AztecAddress,
123
+ scope: AztecAddress,
125
124
  ): Promise<void> {
126
125
  // We are going to store the new note in the NoteStore, which will let us later return it via `getNotes`.
127
126
  // There's two things we need to check before we do this however:
@@ -155,16 +154,28 @@ export class NoteService {
155
154
  this.aztecNode.findLeavesIndexes(anchorBlockHash, MerkleTreeId.NULLIFIER_TREE, [siloedNullifier]),
156
155
  ]);
157
156
  if (!txEffect) {
158
- throw new Error(`Could not find tx effect for tx hash ${txHash}`);
157
+ // We error out instead of just logging a warning and skipping the note because this would indicate a bug. This
158
+ // is because the node has already served info about this tx either when obtaining the log (TxScopedL2Log contain
159
+ // tx info) or when getting metadata for the offchain message (before the message got passed to `process_log`).
160
+ throw new Error(`Could not find tx effect for tx hash ${txHash} when processing a note.`);
159
161
  }
160
162
 
161
163
  if (txEffect.l2BlockNumber > anchorBlockNumber) {
162
- throw new Error(`Could not find tx effect for tx hash ${txHash} as of block number ${anchorBlockNumber}`);
164
+ // If the message was delivered onchain, this would indicate a bug: log sync should never load logs from blocks
165
+ // newer than the anchor block. If the note came via an offchain message, it would likely also be a bug, since we
166
+ // sync a new anchor block before calling `process_message`. For this not to be a bug, the message would need to
167
+ // come from a newer block than the anchor served by the node, implying the node isn't properly synced.
168
+ // We therefore error out here rather than assuming the offchain message was constructed by a malicious
169
+ // sender with the intention of bricking recipient's PXE (if we assumed that we would just ignore the message).
170
+ throw new Error(
171
+ `Obtained a newer tx effect for ${txHash} for a note validation request than the anchor block ${anchorBlockNumber}. This is a bug as we should not ever be processing a note from a newer block than the anchor block.`,
172
+ );
163
173
  }
164
174
 
165
175
  // Find the index of the note hash in the noteHashes array to determine note ordering within the tx
166
176
  const noteIndexInTx = txEffect.data.noteHashes.findIndex(nh => nh.equals(uniqueNoteHash));
167
177
  if (noteIndexInTx === -1) {
178
+ // Similar to the comment above - we error out as this would indicate a bug in nonce discovery.
168
179
  throw new Error(`Note hash ${noteHash} (uniqued as ${uniqueNoteHash}) is not present in tx ${txHash}`);
169
180
  }
170
181
 
@@ -184,8 +195,7 @@ export class NoteService {
184
195
  noteIndexInTx,
185
196
  );
186
197
 
187
- // The note was found by `recipient`, so we use that as the scope when storing the note.
188
- await this.noteStore.addNotes([noteDao], recipient, this.jobId);
198
+ await this.noteStore.addNotes([noteDao], scope, this.jobId);
189
199
 
190
200
  if (nullifierIndex !== undefined) {
191
201
  // We found nullifier index which implies that the note has already been nullified.
@@ -2,8 +2,6 @@ import type { Fr } from '@aztec/foundation/curves/bn254';
2
2
  import type { AztecAddress } from '@aztec/stdlib/aztec-address';
3
3
  import type { NoteStatus } from '@aztec/stdlib/note';
4
4
 
5
- import type { AccessScopes } from './access_scopes.js';
6
-
7
5
  /**
8
6
  * A filter used to fetch notes.
9
7
  * @remarks This filter is applied as an intersection of all its params.
@@ -22,5 +20,5 @@ export type NotesFilter = {
22
20
  status?: NoteStatus;
23
21
  /** The siloed nullifier for the note. */
24
22
  siloedNullifier?: Fr;
25
- scopes: AccessScopes;
23
+ scopes: AztecAddress[];
26
24
  };
@@ -1,12 +1,22 @@
1
- /// The ORACLE_VERSION constant is used to check that the oracle interface is in sync between PXE and Aztec.nr. We need
2
- /// to version the oracle interface to ensure that developers get a reasonable error message if they use incompatible
3
- /// versions of Aztec.nr and PXE. The Noir counterpart is in `noir-projects/aztec-nr/aztec/src/oracle/version.nr`.
1
+ /// The oracle version constants are used to check that the oracle interface is in sync between PXE and Aztec.nr.
2
+ /// We version the oracle interface as `major.minor` where:
3
+ /// - `major` = backward-breaking changes (must match exactly between PXE and Aztec.nr)
4
+ /// - `minor` = oracle additions (non-breaking; PXE minor >= contract minor)
4
5
  ///
5
- /// @dev Whenever a contract function or Noir test is run, the `aztec_utl_assertCompatibleOracleVersion` oracle is called
6
- /// and if the oracle version is incompatible an error is thrown.
7
- export const ORACLE_VERSION = 16;
6
+ /// The Noir counterparts are in `noir-projects/aztec-nr/aztec/src/oracle/version.nr`.
7
+ ///
8
+ /// @dev Whenever a contract function or Noir test is run, the `aztec_utl_assertCompatibleOracleVersion` oracle is called.
9
+ /// If the major version is incompatible, an error is thrown immediately. The minor version is recorded by the PXE and
10
+ /// used to provide helpful error messages if a contract calls an oracle that doesn't exist. We don't throw immediately
11
+ /// if AZTEC_NR_MINOR > PXE_MINOR because if a contract is updated to use a newer Aztec.nr dependency without actually
12
+ /// using any of the new oracles then there is no reason to throw.
13
+ export const ORACLE_VERSION_MAJOR = 22;
14
+ export const ORACLE_VERSION_MINOR = 1;
8
15
 
9
- /// This hash is computed as by hashing the Oracle interface and it is used to detect when the Oracle interface changes,
10
- /// which in turn implies that you need to update the ORACLE_VERSION constant in this file and in
11
- /// `noir-projects/aztec-nr/aztec/src/oracle/version.nr`.
12
- export const ORACLE_INTERFACE_HASH = '73ccb2a24bc9fe7514108be9ff98d7ca8734bc316fb7c1ec4329d1d32f412a55';
16
+ /// This hash is computed from the Oracle interface and is used to detect when that interface changes. When it does,
17
+ /// you need to either:
18
+ /// - increment `ORACLE_VERSION_MAJOR` and reset `ORACLE_VERSION_MINOR` to zero if the change is breaking, or
19
+ /// - increment only `ORACLE_VERSION_MINOR` if the change is additive (a new oracle was added).
20
+ ///
21
+ /// These constants must be kept in sync between this file and `noir-projects/aztec-nr/aztec/src/oracle/version.nr`.
22
+ export const ORACLE_INTERFACE_HASH = 'efafa0db2cc1f94e26d794d0079c8f71115261df0c3d0fa8cb5b64f17a12db92';
package/src/pxe.ts CHANGED
@@ -52,7 +52,6 @@ import {
52
52
 
53
53
  import { inspect } from 'util';
54
54
 
55
- import type { AccessScopes } from './access_scopes.js';
56
55
  import { BlockSynchronizer } from './block_synchronizer/index.js';
57
56
  import type { PXEConfig } from './config/index.js';
58
57
  import { BenchmarkedNodeFactory } from './contract_function_simulator/benchmarked_node.js';
@@ -96,7 +95,7 @@ export type ProfileTxOpts = {
96
95
  /** If true, proof generation is skipped during profiling. Defaults to true. */
97
96
  skipProofGeneration?: boolean;
98
97
  /** Addresses whose private state and keys are accessible during private execution. */
99
- scopes: AccessScopes;
98
+ scopes: AztecAddress[];
100
99
  };
101
100
 
102
101
  /** Options for PXE.simulateTx. */
@@ -112,7 +111,7 @@ export type SimulateTxOpts = {
112
111
  /** State overrides for the simulation, such as contract instances and artifacts. Requires skipKernels: true */
113
112
  overrides?: SimulationOverrides;
114
113
  /** Addresses whose private state and keys are accessible during private execution */
115
- scopes: AccessScopes;
114
+ scopes: AztecAddress[];
116
115
  };
117
116
 
118
117
  /** Options for PXE.executeUtility. */
@@ -120,7 +119,7 @@ export type ExecuteUtilityOpts = {
120
119
  /** The authentication witnesses required for the function call. */
121
120
  authwits?: AuthWitness[];
122
121
  /** The accounts whose notes we can access in this call */
123
- scopes: AccessScopes;
122
+ scopes: AztecAddress[];
124
123
  };
125
124
 
126
125
  /** Args for PXE.create. */
@@ -368,7 +367,7 @@ export class PXE {
368
367
  async #executePrivate(
369
368
  contractFunctionSimulator: ContractFunctionSimulator,
370
369
  txRequest: TxExecutionRequest,
371
- scopes: AccessScopes,
370
+ scopes: AztecAddress[],
372
371
  jobId: string,
373
372
  ): Promise<PrivateExecutionResult> {
374
373
  const { origin: contractAddress, functionSelector } = txRequest;
@@ -417,7 +416,7 @@ export class PXE {
417
416
  contractFunctionSimulator: ContractFunctionSimulator,
418
417
  call: FunctionCall,
419
418
  authWitnesses: AuthWitness[] | undefined,
420
- scopes: AccessScopes,
419
+ scopes: AztecAddress[],
421
420
  jobId: string,
422
421
  ) {
423
422
  try {
@@ -502,7 +501,9 @@ export class PXE {
502
501
  * @returns The synced block header
503
502
  */
504
503
  public getSyncedBlockHeader(): Promise<BlockHeader> {
505
- return this.anchorBlockStore.getBlockHeader();
504
+ return this.#putInJobQueue(() => {
505
+ return this.anchorBlockStore.getBlockHeader();
506
+ });
506
507
  }
507
508
 
508
509
  /**
@@ -569,6 +570,9 @@ export class PXE {
569
570
 
570
571
  if (wasAdded) {
571
572
  this.log.info(`Added sender:\n ${sender.toString()}`);
573
+ // Wipe the entire sync cache: the new sender's tagged logs could contain notes/events for any contract, so
574
+ // all contracts must re-sync to discover them.
575
+ this.contractSyncService.wipe();
572
576
  } else {
573
577
  this.log.info(`Sender:\n "${sender.toString()}"\n already registered.`);
574
578
  }
@@ -939,9 +943,10 @@ export class PXE {
939
943
  }
940
944
  const contractFunctionSimulator = this.#getSimulatorForTx(overrides);
941
945
 
942
- // Set overridden contracts on the sync service so it knows to skip syncing them
943
946
  if (hasOverriddenContracts) {
944
- this.contractSyncService.setOverriddenContracts(jobId, overriddenContracts);
947
+ // Overridden contracts don't have a sync function, so calling sync on them would fail.
948
+ // We exclude them so the sync service skips them entirely.
949
+ this.contractSyncService.setExcludedFromSync(jobId, overriddenContracts);
945
950
  }
946
951
 
947
952
  // Execution of private functions only; no proving, and no kernel logic.
@@ -1036,7 +1041,7 @@ export class PXE {
1036
1041
  inspect(txRequest),
1037
1042
  `simulatePublic=${simulatePublic}`,
1038
1043
  `skipTxValidation=${skipTxValidation}`,
1039
- `scopes=${scopes === 'ALL_SCOPES' ? scopes : scopes.map(s => s.toString()).join(', ')}`,
1044
+ `scopes=${scopes.map(s => s.toString()).join(', ')}`,
1040
1045
  );
1041
1046
  }
1042
1047
  });
@@ -1048,7 +1053,7 @@ export class PXE {
1048
1053
  */
1049
1054
  public executeUtility(
1050
1055
  call: FunctionCall,
1051
- { authwits, scopes }: ExecuteUtilityOpts = { scopes: 'ALL_SCOPES' },
1056
+ { authwits, scopes }: ExecuteUtilityOpts = { scopes: [] },
1052
1057
  ): Promise<UtilityExecutionResult> {
1053
1058
  // We disable concurrent executions since those might execute oracles which read and write to the PXE stores (e.g.
1054
1059
  // to the capsules), and we need to prevent concurrent runs from interfering with one another (e.g. attempting to
@@ -1106,7 +1111,7 @@ export class PXE {
1106
1111
  throw this.#contextualizeError(
1107
1112
  err,
1108
1113
  `executeUtility ${to}:${name}(${stringifiedArgs})`,
1109
- `scopes=${scopes === 'ALL_SCOPES' ? scopes : scopes.map(s => s.toString()).join(', ')}`,
1114
+ `scopes=${scopes.map(s => s.toString()).join(', ')}`,
1110
1115
  );
1111
1116
  }
1112
1117
  });
@@ -0,0 +1,90 @@
1
+ import type { Fr } from '@aztec/foundation/curves/bn254';
2
+ import { AztecAddress } from '@aztec/stdlib/aztec-address';
3
+ import type { Capsule } from '@aztec/stdlib/tx';
4
+
5
+ import type { CapsuleStore } from './capsule_store.js';
6
+
7
+ /**
8
+ * Wraps a CapsuleStore with scope-based access control. Each operation asserts that the requested scope is in the
9
+ * allowed scopes list before delegating to the underlying store.
10
+ */
11
+ export class CapsuleService {
12
+ constructor(
13
+ private readonly capsuleStore: CapsuleStore,
14
+ private readonly allowedScopes: AztecAddress[],
15
+ ) {}
16
+
17
+ setCapsule(contractAddress: AztecAddress, slot: Fr, capsule: Fr[], jobId: string, scope: AztecAddress) {
18
+ assertAllowedScope(scope, this.allowedScopes);
19
+ this.capsuleStore.setCapsule(contractAddress, slot, capsule, jobId, scope);
20
+ }
21
+
22
+ async getCapsule(
23
+ contractAddress: AztecAddress,
24
+ slot: Fr,
25
+ jobId: string,
26
+ scope: AztecAddress,
27
+ transientCapsules?: Capsule[],
28
+ ): Promise<Fr[] | null> {
29
+ assertAllowedScope(scope, this.allowedScopes);
30
+
31
+ // TODO(#12425): On the following line, the pertinent capsule gets overshadowed by the transient one. Tackle this.
32
+ const maybeTransientCapsule = transientCapsules?.find(
33
+ c =>
34
+ c.contractAddress.equals(contractAddress) &&
35
+ c.storageSlot.equals(slot) &&
36
+ (c.scope ?? AztecAddress.ZERO).equals(scope),
37
+ )?.data;
38
+
39
+ return maybeTransientCapsule ?? (await this.capsuleStore.getCapsule(contractAddress, slot, jobId, scope));
40
+ }
41
+
42
+ deleteCapsule(contractAddress: AztecAddress, slot: Fr, jobId: string, scope: AztecAddress) {
43
+ assertAllowedScope(scope, this.allowedScopes);
44
+ this.capsuleStore.deleteCapsule(contractAddress, slot, jobId, scope);
45
+ }
46
+
47
+ copyCapsule(
48
+ contractAddress: AztecAddress,
49
+ srcSlot: Fr,
50
+ dstSlot: Fr,
51
+ numEntries: number,
52
+ jobId: string,
53
+ scope: AztecAddress,
54
+ ): Promise<void> {
55
+ assertAllowedScope(scope, this.allowedScopes);
56
+ return this.capsuleStore.copyCapsule(contractAddress, srcSlot, dstSlot, numEntries, jobId, scope);
57
+ }
58
+
59
+ appendToCapsuleArray(
60
+ contractAddress: AztecAddress,
61
+ baseSlot: Fr,
62
+ content: Fr[][],
63
+ jobId: string,
64
+ scope: AztecAddress,
65
+ ): Promise<void> {
66
+ assertAllowedScope(scope, this.allowedScopes);
67
+ return this.capsuleStore.appendToCapsuleArray(contractAddress, baseSlot, content, jobId, scope);
68
+ }
69
+
70
+ readCapsuleArray(contractAddress: AztecAddress, baseSlot: Fr, jobId: string, scope: AztecAddress): Promise<Fr[][]> {
71
+ assertAllowedScope(scope, this.allowedScopes);
72
+ return this.capsuleStore.readCapsuleArray(contractAddress, baseSlot, jobId, scope);
73
+ }
74
+
75
+ setCapsuleArray(contractAddress: AztecAddress, baseSlot: Fr, content: Fr[][], jobId: string, scope: AztecAddress) {
76
+ assertAllowedScope(scope, this.allowedScopes);
77
+ return this.capsuleStore.setCapsuleArray(contractAddress, baseSlot, content, jobId, scope);
78
+ }
79
+ }
80
+
81
+ function assertAllowedScope(scope: AztecAddress, allowedScopes: AztecAddress[]) {
82
+ if (scope.equals(AztecAddress.ZERO)) {
83
+ return;
84
+ }
85
+ if (!allowedScopes.some((allowed: AztecAddress) => allowed.equals(scope))) {
86
+ throw new Error(
87
+ `Scope ${scope.toString()} is not in the allowed scopes list: [${allowedScopes.map((s: AztecAddress) => s.toString()).join(', ')}]. See https://docs.aztec.network/errors/10`,
88
+ );
89
+ }
90
+ }