Skip to content

feat: prefix aztec-nr contract logs with [aztec-nr] #21080

Merged
benesjan merged 1 commit into
merge-train/fairiesfrom
nico/f-390-prefix-aztecnr-contract-logs-with-a-prefix
Mar 4, 2026
Merged

feat: prefix aztec-nr contract logs with [aztec-nr] #21080
benesjan merged 1 commit into
merge-train/fairiesfrom
nico/f-390-prefix-aztecnr-contract-logs-with-a-prefix

Conversation

@nventuro

@nventuro nventuro commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

We now prefix all logs that aztecnr emits with [aztec-nr] , so that these are identifiable from user messages. It unfortunately requires some comtpime magic but nothing too crazy.

I put these in a new logging mod that for now is not made public. In the future we'll likely move debug_log and friends there. We'll never use those from aztecnr, they're just for users.

Add a `logging` module with comptime wrappers that prepend `[aztec-nr] `
to all log messages at compile time using `CtString` concatenation. This
makes it easy to filter framework-generated logs from application logs.

All existing call sites are updated to use the new prefixed functions,
and `set_as_fee_payer`/`end_setup` logging is downgraded to trace level.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@benesjan benesjan 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.

Cool!

@benesjan benesjan merged commit 49b07b1 into merge-train/fairies Mar 4, 2026
16 of 20 checks passed
@benesjan benesjan deleted the nico/f-390-prefix-aztecnr-contract-logs-with-a-prefix branch March 4, 2026 04:35
@AztecBot

AztecBot commented Mar 4, 2026

Copy link
Copy Markdown
Collaborator

❌ Failed to cherry-pick to v4 due to conflicts. Dispatching ClaudeBox to resolve. View backport run.

@AztecBot

AztecBot commented Mar 4, 2026

Copy link
Copy Markdown
Collaborator

❌ Failed to cherry-pick to v4 due to conflicts. Dispatching ClaudeBox to resolve. View backport run.

@AztecBot

Copy link
Copy Markdown
Collaborator

❌ Failed to cherry-pick to v4-next due to conflicts. (🤖) View backport run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants