fix(di): prevent exception when removing unresolved probe

[P403n1x87]

We prevent an exception from occurring when trying to remove a probe that did not resolve to a valid source code location.

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-unresolved-pending-probe-removal-80fbf85a9c068122.yaml  @DataDog/apm-python
ddtrace/debugging/_probe/registry.py                                    @DataDog/debugger-python
tests/debugging/probe/test_registry.py                                  @DataDog/debugger-python

[github-actions]

Bootstrap import analysis

Comparison of import times between this PR and base.

Summary

The average import time from this PR is: 232 ± 4 ms.

The average import time from base is: 234 ± 4 ms.

The import time difference between this PR and base is: -1.6 ± 0.2 ms.

Import time breakdown

The following import paths have shrunk:

ddtrace.auto 1.959 ms (0.84%)

ddtrace.bootstrap.sitecustomize 1.292 ms (0.56%)

ddtrace.bootstrap.preload 1.292 ms (0.56%)

ddtrace.internal.products 1.292 ms (0.56%)

ddtrace.internal.remoteconfig.client 0.626 ms (0.27%)

ddtrace 0.667 ms (0.29%)

[pr-commenter]

Benchmarks

Benchmark execution time: 2025-05-08 10:01:43

Comparing candidate commit 16674f1 in PR branch fix/unresolved-pending-probe-removal with baseline commit 0ee168f in branch main.

Found 0 performance improvements and 6 performance regressions! Performance is the same for 523 metrics, 7 unstable metrics.

scenario:iast_aspects-modulo_aspect_for_bytes

• 🟥 execution_time [+59.132ns; +77.922ns] or [+7.447%; +9.813%]

scenario:iast_aspects-modulo_aspect_for_bytes_bytearray

• 🟥 execution_time [+93.669ns; +111.927ns] or [+9.195%; +10.988%]

scenario:iast_aspects-replace_aspect

• 🟥 execution_time [+607.782ns; +714.813ns] or [+12.918%; +15.193%]

scenario:iast_aspects-rstrip_aspect

• 🟥 execution_time [+1.060µs; +1.367µs] or [+8.185%; +10.556%]

scenario:iast_aspects-strip_aspect

• 🟥 execution_time [+1.336µs; +1.474µs] or [+10.137%; +11.189%]

scenario:iast_aspects-swapcase_aspect

• 🟥 execution_time [+217.933ns; +256.519ns] or [+8.960%; +10.546%]