Skip to content

chore: backport #21080 — prefix aztec-nr contract logs with [aztec-nr]#21536

Merged
nchamo merged 10 commits into
backport-to-v4-next-stagingfrom
claudebox/backport-21080-aztec-nr-logs
Mar 13, 2026
Merged

chore: backport #21080 — prefix aztec-nr contract logs with [aztec-nr]#21536
nchamo merged 10 commits into
backport-to-v4-next-stagingfrom
claudebox/backport-21080-aztec-nr-logs

Conversation

@AztecBot

@AztecBot AztecBot commented Mar 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

Backport of #21080 to v4-next.

Adds a logging module with comptime wrappers that prepend [aztec-nr] to all log messages at compile time. All existing call sites are updated to use the new prefixed functions.

Conflicts resolved

  • CLAUDE.md: File was deleted in v4-next but modified in source — kept the file since it contains relevant logging guidelines.
  • process_message.nr: v4-next has simpler message handling (no custom message support). Resolved by using the new aztecnr_ prefixed logging functions while keeping v4-next's simpler structure (single else clause for unknown message types instead of the custom message handler logic on next).

Note on GitHub diff

GitHub's PR diff may show 3 unrelated files (spartan/environments/network-defaults.yml, deploy_method.test.ts, gossip_network.test.ts). These are not changed by this PR — they are a GitHub display artifact from the merge commit at v4-next's tip (ddcc584155). The actual diff (git diff origin/v4-next..HEAD) only includes the 9 noir files.

Commit structure

  1. Cherry-pick with conflicts — raw cherry-pick of 9e0dd1e with conflict markers
  2. Conflict resolution — resolved the two conflicts described above
  3. Merge of v4-next — to establish correct merge-base for the PR"

aminsammara and others added 4 commits March 13, 2026 11:45
## Summary
- Lower `SLASH_INACTIVITY_TARGET_PERCENTAGE` from 0.8 to 0.7
- Disable `SLASH_DUPLICATE_PROPOSAL_PENALTY` and
`SLASH_DUPLICATE_ATTESTATION_PENALTY` (set to 0)
- Increase `SLASH_GRACE_PERIOD_L2_SLOTS` from 1200 to 8400
- Reduce `perBlockAllocationMultiplier` from 2 to 1.2

## Test plan
- Verify generated config files are updated after running `yarn
generate` in relevant packages
- Deploy to devnet/testnet and confirm slasher behavior matches new
parameters

---------

Co-authored-by: Aztec Bot <49558828+AztecBot@users.noreply.github.com>
This PR was created because merging `v4` into `v4-next` resulted in
conflicts.

Please resolve the conflicts and merge this PR to keep `v4-next` up to
date.

This PR was auto-generated by the `pull-v4-into-v4-next` workflow.
@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 13, 2026
@AztecBot AztecBot force-pushed the claudebox/backport-21080-aztec-nr-logs branch from ecf1b56 to b56ebca Compare March 13, 2026 17:15
nventuro and others added 3 commits March 13, 2026 17:16
- CLAUDE.md: keep file (deleted in v4-next, modified in source) since logging guidelines are relevant
- process_message.nr: use aztecnr prefixed logging while keeping v4-next's simpler structure (no custom message handling)
@AztecBot AztecBot force-pushed the claudebox/backport-21080-aztec-nr-logs branch from b56ebca to 3c03d98 Compare March 13, 2026 17:16
@nchamo nchamo marked this pull request as ready for review March 13, 2026 17:26
@nchamo nchamo requested a review from nventuro as a code owner March 13, 2026 17:26
@nchamo nchamo enabled auto-merge (squash) March 13, 2026 17:26
@nchamo nchamo self-assigned this Mar 13, 2026
@nchamo nchamo merged commit 2538590 into backport-to-v4-next-staging Mar 13, 2026
13 checks passed
@nchamo nchamo deleted the claudebox/backport-21080-aztec-nr-logs branch March 13, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants