fix: default semantic attributes to undefined

Author: rowanmanning

packages/app-info/README.md

@@ -116,6 +116,8 @@ This object contains aliases for the main `appInfo` properties that correspond t
   * `appInfo.semanticConventions.service.version` aliases `appInfo.releaseVersion`
   * `appInfo.semanticConventions.service.instance.id` aliases `appInfo.instanceId`
 
+> [!WARNING]
+> While all other properties default to `null` if they can't be calculated, the semantic conventions properties default to `undefined`. This is to ensure better compatibility with OpenTelemetry SDKs.
 
 ## Migrating
 

packages/app-info/lib/index.js

@@ -163,15 +163,15 @@ exports.instanceId = process.env.HEROKU_DYNO_ID || randomUUID();
  */
 exports.semanticConventions = {
 	cloud: {
-		provider: exports.cloudProvider,
-		region: exports.region
+		provider: exports.cloudProvider || undefined,
+		region: exports.region || undefined
 	},
 	deployment: {
 		environment: exports.environment
 	},
 	service: {
-		name: exports.systemCode,
-		version: exports.releaseVersion,
+		name: exports.systemCode || undefined,
+		version: exports.releaseVersion || undefined,
 		instance: {
 			id: exports.instanceId
 		}

packages/app-info/types/index.d.ts

@@ -13,15 +13,15 @@ declare module '@dotcom-reliability-kit/app-info' {
 
 	export type SemanticConventions = {
 		cloud: {
-			provider: string | null,
-			region: string | null
+			provider?: string,
+			region?: string
 		},
 		deployment: {
 			environment: string
 		},
 		service: {
-			name: string | null
-			version: string | null,
+			name?: string
+			version?: string,
 			instance: {
 				id: string
 			}