fix(core): Fix/otel compat (#6119)

* refactor(otel): use compatibility helpers instead of duplicating entire file

Address PR review feedback to avoid copying entire OTelService.kt for version compatibility.

Changes:
- Move OTelService.kt to common src/ directory (168 lines, shared by all versions)
- Create small version-specific HttpPostCompat.kt files (18-19 lines each)
- Eliminates 137 lines of code duplication
- Main logic now maintained in single location

The httpPost API changed between IDE versions 2024.2-2025.2 and 2025.3+:
- Old API uses contentLength parameter with streaming body
- New API uses body parameter with byte array

Follows existing codebase pattern (see PythonModuleUtil.kt, JavascriptLanguage.kt)
for handling compile-time API incompatibilities with minimal duplication.

* fix: remove unused NonInjectable import

* refactor(otel): use Apache HttpClient for cross-version compatibility

Address PR review feedback - use Apache HttpClient instead of platform httpPost API.

Changes:
- Replace IntelliJ Platform httpPost with Apache HttpClient
- Works consistently across all IDE versions (2024.2 - 2025.3+)
- Matches existing pattern used in SigV4OtlpSpanProcessor
- No version-specific compatibility wrappers needed
- Single OTelService.kt in common src/ directory

This approach:
- Uses same HTTP library as CawsEnvironmentClient and SigV4OtlpSpanProcessor
- Avoids platform API incompatibilities between IDE versions
- Follows established codebase patterns for HTTP operations

* refactor(otel): use HttpRequests for proxy and user agent support

Use IntelliJ's HttpRequests API instead of Apache HttpClient to respect IDE proxy settings and use AWS Toolkit user agent.

* Optimize telemetry export by writing directly to output stream

* perf(otel): write telemetry directly to HTTP stream

Address PR review feedback to avoid intermediate ByteArrayOutputStream.

Instead of:
  1. Write to ByteArrayOutputStream
  2. Convert to byte array
  3. Write byte array to HTTP connection

Now:
  1. Write directly to HTTP connection.outputStream

This reduces memory allocation and improves performance by streaming
data directly to the network without intermediate buffering.

Author: aseemxs

plugins/core/jetbrains-community/src-253+/software/aws/toolkits/jetbrains/services/telemetry/otel/OTelService.kt

@@ -9,9 +9,8 @@ import com.intellij.openapi.components.Service
 import com.intellij.openapi.components.service
 import com.intellij.openapi.diagnostic.thisLogger
 import com.intellij.openapi.util.SystemInfoRt
-import com.intellij.platform.util.http.ContentType
-import com.intellij.platform.util.http.httpPost
 import com.intellij.serviceContainer.NonInjectable
+import com.intellij.util.io.HttpRequests
 import io.opentelemetry.api.common.AttributeKey
 import io.opentelemetry.api.common.Attributes
 import io.opentelemetry.api.trace.propagation.W3CTraceContextPropagator
@@ -34,6 +33,7 @@ import software.amazon.awssdk.http.SdkHttpMethod
 import software.amazon.awssdk.http.SdkHttpRequest
 import software.amazon.awssdk.http.apache.ApacheHttpClient
 import software.amazon.awssdk.http.auth.aws.signer.AwsV4HttpSigner
+import software.aws.toolkits.jetbrains.core.AwsClientManager
 import java.io.ByteArrayOutputStream
 import java.net.ConnectException
 
@@ -51,9 +51,11 @@ private class BasicOtlpSpanProcessor(
             try {
                 val item = TraceRequestMarshaler.create(listOf(data))
 
-                httpPost(traceUrl, contentLength = item.binarySerializedSize.toLong(), contentType = ContentType.XProtobuf) {
-                    item.writeBinaryTo(this)
-                }
+                HttpRequests.post(traceUrl, "application/x-protobuf")
+                    .userAgent(AwsClientManager.getUserAgent())
+                    .connect { request ->
+                        item.writeBinaryTo(request.connection.outputStream)
+                    }
             } catch (e: CancellationException) {
                 throw e
             } catch (e: ConnectException) {