fix(er): include nonlocals in snapshots

[P403n1x87]

We include nonlocal variables in snapshots to provide for better visibility into exception occurrences.

Checklist

• PR author has checked that all the criteria below are met
• The PR description includes an overview of the change
• The PR description articulates the motivation for the change
• The change includes tests OR the PR description describes a testing strategy
• The PR description notes risks associated with the change, if any
• Newly-added code is easy to change
• The change follows the library release note guidelines
• The change includes or references documentation updates if necessary
• Backport labels are set (if applicable)

Reviewer Checklist

• Reviewer has checked that all the criteria below are met
• Title is accurate
• All changes are related to the pull request's stated goal
• Avoids breaking API changes
• Testing strategy adequately addresses listed risks
• Newly-added code is easy to change
• Release note makes sense to a user of the library
• If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment
• Backport labels are set in a manner that is consistent with the release branch maintenance policy

[github-actions]

CODEOWNERS have been resolved as:

releasenotes/notes/fix-er-include-nonlocals-bbeecfbbbde35496.yaml       @DataDog/apm-python
ddtrace/debugging/_safety.py                                            @DataDog/debugger-python
tests/debugging/exception/test_replay.py                                @DataDog/debugger-python
tests/debugging/test_safety.py                                          @DataDog/debugger-python

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: fix/er-include-nonlocals
Commit report: 0cb0431
Test service: dd-trace-py

✅ 0 Failed, 55 Passed, 1434 Skipped, 2m 2.41s Total duration (31m 12.06s time saved)

[pr-commenter]

Benchmarks

Benchmark execution time: 2025-01-10 15:05:24

Comparing candidate commit 2b42c91 in PR branch fix/er-include-nonlocals with baseline commit afe439e in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 394 metrics, 2 unstable metrics.

[github-actions]

The backport to 2.16 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-2.16 2.16
# Navigate to the new working tree
cd .worktrees/backport-2.16
# Create a new branch
git switch --create backport-11894-to-2.16
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 d7927e6296679a3d7def5b31347262b7cce8b7dd
# Push it to GitHub
git push --set-upstream origin backport-11894-to-2.16
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-2.16

Then, create a pull request where the base branch is 2.16 and the compare/head branch is backport-11894-to-2.16.

[github-actions]

The backport to 2.17 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-2.17 2.17
# Navigate to the new working tree
cd .worktrees/backport-2.17
# Create a new branch
git switch --create backport-11894-to-2.17
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 d7927e6296679a3d7def5b31347262b7cce8b7dd
# Push it to GitHub
git push --set-upstream origin backport-11894-to-2.17
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-2.17

Then, create a pull request where the base branch is 2.17 and the compare/head branch is backport-11894-to-2.17.