feat(otlp-transformer): consolidate scope/resource creation in transformer

[pichlermarc]

Which problem is this PR solving?

Resources and scopes creation are currently duplicated across all signals even though they're the same. This PR pulls them out into shared functions for all signals.

Replaces #4429

[codecov]

Codecov Report

Merging #4600 (6094b06) into main (e01f493) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4600      +/-   ##
==========================================
- Coverage   92.83%   92.83%   -0.01%     
==========================================
  Files         328      329       +1     
  Lines        9519     9526       +7     
  Branches     2050     2050              
==========================================
+ Hits         8837     8843       +6     
- Misses        682      683       +1     

Files	Coverage Δ
...l/packages/otlp-transformer/src/common/internal.ts	100.00% <100.00%> (ø)
...mental/packages/otlp-transformer/src/logs/index.ts	100.00% <100.00%> (ø)
.../packages/otlp-transformer/src/metrics/internal.ts	100.00% <100.00%> (ø)
...packages/otlp-transformer/src/resource/internal.ts	100.00% <100.00%> (ø)
...ental/packages/otlp-transformer/src/trace/index.ts	100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

[pichlermarc]

FYI @legendecas this is the stale #4429 with the changes I suggested to make the workflows pass.

[legendecas]

I am fine with droppedAttributesCount since it exists in the proto. However, as the key attributes is optional, I think droppedAttributesCount could be optional as well.

[pichlermarc]

I am fine with droppedAttributesCount since it exists in the proto. However, as the key attributes is optional, I think droppedAttributesCount could be optional as well.

@legendecas, ah sorry I took me a while to understand what you meant on the original PR. 😅

As it's optional in the proto, and we don't set it to anything other than 0, we might as well leave it out, you're right. I pushed a commit to change this (56cf720). This way we also don't need to adapt the tests.