Skip to content

fix(pxe): repoint broken debugging docs link in PXE error messages#24145

Merged
vezenovm merged 3 commits into
merge-train/fairies-v5from
mv/fix-debugging-error-docs-link
Jun 17, 2026
Merged

fix(pxe): repoint broken debugging docs link in PXE error messages#24145
vezenovm merged 3 commits into
merge-train/fairies-v5from
mv/fix-debugging-error-docs-link

Conversation

@vezenovm

@vezenovm vezenovm commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Two PXE error messages linked to a docs page that no longer exists (developers/resources/debugging/aztecnr-errors), and suggested methods that aren't public API.

  • utility_execution_oracle.ts ("No public key registered for address")
  • contract_store.ts ("Unknown contract")

Both now point at the live page https://docs.aztec.network/developers/docs/aztec-nr/debugging#contract-errors and recommend the real methods (wallet.registerSender(...) / wallet.registerContract(...)) instead of the nonexistent pxe.addAccount(...) / server.addContracts(...).

Discovered in http://ci.aztec-labs.com/7f36172a9dadb066

@vezenovm vezenovm requested review from Thunkar, mverzilli and nchamo June 17, 2026 13:53
throw new Error(
`No public key registered for address ${account}.
Register it by calling pxe.addAccount(...).\nSee docs for context: https://docs.aztec.network/developers/resources/debugging/aztecnr-errors#simulation-error-no-public-key-registered-for-address-0x0-register-it-by-calling-pxeregisterrecipient-or-pxeregisteraccount`,
Register it by calling wallet.registerSender(...).\nSee docs for context: https://docs.aztec.network/developers/docs/aztec-nr/debugging#contract-errors`,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this, but could we use links like https://docs.aztec.network/errors/6? See netlify.toml

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@vezenovm vezenovm requested a review from nchamo June 17, 2026 15:40

@nchamo nchamo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank for this!

@vezenovm vezenovm enabled auto-merge (squash) June 17, 2026 15:52
@vezenovm vezenovm merged commit d6b8227 into merge-train/fairies-v5 Jun 17, 2026
12 checks passed
@vezenovm vezenovm deleted the mv/fix-debugging-error-docs-link branch June 17, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants