feat(tracing): add Subsegment wrapper to prevent exposing Amazon.XRayRecorder.Core.Internal

[JonkaSusaki]

Issue number: #603

Summary

Changes

This PR creates a wrapper to prevent exposing internal lib Amazon.XRayRecorder.

• Created libraries/src/AWS.Lambda.Powertools.Tracing/Internal/TracingSubsegment.cs to place the wrapper;
• Changed libraries/src/AWS.Lambda.Powertools.Tracing/Tracing.cs to include the wrapper;

User experience

Before the changes, users had to include the reference to Amazon.XRay.Recorder.Core.Internal.Entities to use the Subsegment class. Now, the TracingSubsegment was created and users can now import them using the AWS.Lambda.Powertools.Tracing.Internal reference.

Checklist

Please leave checklist items unchecked if they do not apply to your change.

• Meets tenets criteria
• I have performed a self-review of this change
• Changes have been tested
• Changes are documented
• PR title follows conventional commit semantics

Is this a breaking change?

NO

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[boring-cyborg]

Thanks a lot for your first contribution! Please check out our contributing guidelines and don't hesitate to ask whatever you need.
In the meantime, check out the #dotnet channel on our Powertools for AWS Lambda Discord: Invite link

[JonkaSusaki]

Hey, @hjgraca! I tried implementing it this way.

Please, tell me your thoughts! If there's anything you think might need changes or you believe another approach is better, let me know!

[leandrodamascena]

Hi @JonkaSusaki, thank you so much for working on this PR! We will review this PR as soon as possible and provide feedback.

[codecov]

Codecov Report

❌ Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.33%. Comparing base (dd6d208) to head (e713cc8).
⚠️ Report is 17 commits behind head on develop.

Files with missing lines	Patch %	Lines
...a.Powertools.Tracing/Internal/TracingSubsegment.cs	0.00%	1 Missing ⚠️
...aries/src/AWS.Lambda.Powertools.Tracing/Tracing.cs	66.66%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #950      +/-   ##
===========================================
- Coverage    77.39%   77.33%   -0.07%     
===========================================
  Files          271      272       +1     
  Lines        10874    10876       +2     
  Branches      1282     1282              
===========================================
- Hits          8416     8411       -5     
- Misses        2041     2047       +6     
- Partials       417      418       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[hjgraca]

Hi @JonkaSusaki , thank you so much for the pull request, and sorry for the slow response I have been away for the last 2 weeks.
The pull request looks good, but is missing the tests, If you can add them that would be great, if not I will do the work to add to the pull request.
Again thanks and really appreciate your contribution!

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[JonkaSusaki]

Hey @hjgraca . Thanks for the response!
I'll add the tests for sure. Going to commit soon.