File attachment integration branch

[fractalwrench]

Goal

Adds an API for logging a message with a file attachment. This API is not fully functional yet and only creates a log message in the existing way, without any attachment.

Testing

Added skeleton of integration tests.

[codecov]

Codecov Report

Attention: Patch coverage is 89.77273% with 18 lines in your changes missing coverage. Please review.

Project coverage is 85.38%. Comparing base (3a22455) to head (ad53a57).
Report is 21 commits behind head on main.

Files with missing lines	Patch %	Lines
...mbracesdk/internal/api/delegate/LogsApiDelegate.kt	81.08%	2 Missing and 5 partials ⚠️
...nal/resurrection/PayloadResurrectionServiceImpl.kt	0.00%	0 Missing and 2 partials ⚠️
...elivery/execution/OkHttpRequestExecutionService.kt	95.00%	0 Missing and 2 partials ⚠️
...nal/delivery/storage/CachedLogEnvelopeStoreImpl.kt	0.00%	0 Missing and 2 partials ⚠️
...roid/embracesdk/internal/logs/EmbraceLogService.kt	75.00%	0 Missing and 1 partial ⚠️
...embracesdk/internal/logs/attachments/Attachment.kt	94.11%	1 Missing ⚠️
...dk/internal/session/orchestrator/V1PayloadStore.kt	0.00%	1 Missing ⚠️
...ndroid/embracesdk/internal/delivery/PayloadType.kt	93.33%	0 Missing and 1 partial ⚠️
.../java/io/embrace/android/embracesdk/EmbraceImpl.kt	92.30%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1804      +/-   ##
==========================================
+ Coverage   85.34%   85.38%   +0.04%     
==========================================
  Files         473      474       +1     
  Lines       11079    11158      +79     
  Branches     1660     1680      +20     
==========================================
+ Hits         9455     9527      +72     
+ Misses        888      882       -6     
- Partials      736      749      +13     

Files with missing lines	Coverage Δ
...oid/embracesdk/internal/injection/LogModuleImpl.kt	92.10% <100.00%> (+0.67%)	⬆️
...ace/android/embracesdk/internal/logs/LogService.kt	100.00% <100.00%> (ø)
...k/internal/logs/attachments/AttachmentErrorCode.kt	100.00% <ø> (ø)
...sdk/internal/logs/attachments/AttachmentService.kt	100.00% <100.00%> (ø)
...sdk/internal/opentelemetry/EmbraceAttributeKeys.kt	100.00% <100.00%> (ø)
...dk/internal/session/orchestrator/V2PayloadStore.kt	95.12% <100.00%> (+0.67%)	⬆️
...cesdk/internal/delivery/StoredTelemetryMetadata.kt	79.31% <100.00%> (-0.69%)	⬇️
...racesdk/internal/delivery/SupportedEnvelopeType.kt	100.00% <100.00%> (ø)
...esdk/internal/delivery/intake/IntakeServiceImpl.kt	100.00% <100.00%> (ø)
...sdk/internal/delivery/storage/AttachmentStorage.kt	100.00% <100.00%> (ø)
... and 12 more

... and 2 files with indirect coverage changes

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

Package	Version	Score	Details

Scanned Files

[priettt]

nit: maybe you can use something like generateSequence { it.nextPart() }, and map the result to a FormPart. The while(true) looks weird, though it does the same 😅

[bidetofevil]

LGTM