fix(replays): Drop and log click events with negative node-ids

[cmanallen]

Negative node-ids can't be inserted into our database because of type issues. Let's drop them for now. We'll also log them so we can figure out if they're important (and possibly debug an SDK issue).

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.27%. Comparing base (7f57781) to head (cac856f).
Report is 37 commits behind head on master.

❗ Current head cac856f differs from pull request most recent head 54e1c7f. Consider uploading reports for the commit 54e1c7f to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #66993      +/-   ##
==========================================
- Coverage   84.27%   84.27%   -0.01%     
==========================================
  Files        5306     5306              
  Lines      237276   237238      -38     
  Branches    41045    41034      -11     
==========================================
- Hits       199971   199938      -33     
+ Misses      37087    37082       -5     
  Partials      218      218              

Files	Coverage Δ
src/sentry/replays/usecases/ingest/dom_index.py	91.51% <100.00%> (+0.21%)	⬆️

... and 16 files with indirect coverage changes

[JoshFerge]

should we add a quick test with a negative node id?