Improve version metadata binding detection (#125)

* Improve version metadata binding detection

* Check for JSRPC bindings everywhere

* Add changeset

.changeset/tasty-ghosts-tickle.md

@@ -0,0 +1,5 @@
+---
+"@microlabs/otel-cf-workers": patch
+---
+
+Correctly detect version metadata when tag is an empty string. Also, check for RPC bindings in all cases as `isVersionMetadata` was incorrectly picking up rpc bindings too when searching for version bindings.

src/instrumentation/env.ts

@@ -11,28 +11,32 @@ const isJSRPC = (item?: unknown): item is Service => {
 }
 
 const isKVNamespace = (item?: unknown): item is KVNamespace => {
-	return !!(item as KVNamespace)?.getWithMetadata
+	return !isJSRPC(item) && !!(item as KVNamespace)?.getWithMetadata
 }
 
 const isQueue = (item?: unknown): item is Queue<unknown> => {
-	return !!(item as Queue<unknown>)?.sendBatch
+	return !isJSRPC(item) && !!(item as Queue<unknown>)?.sendBatch
 }
 
 const isDurableObject = (item?: unknown): item is DurableObjectNamespace => {
-	return !!(item as DurableObjectNamespace)?.idFromName
+	return !isJSRPC(item) && !!(item as DurableObjectNamespace)?.idFromName
 }
 
 const isServiceBinding = (item?: unknown): item is Fetcher => {
 	const binding = item as Fetcher
-	return !!binding.connect || !!binding.fetch || binding.queue || binding.scheduled
+	return (!isJSRPC(item) && !!binding.connect) || !!binding.fetch || binding.queue || binding.scheduled
 }
 
 export const isVersionMetadata = (item?: unknown): item is WorkerVersionMetadata => {
-	return !!(item as WorkerVersionMetadata).id && !!(item as WorkerVersionMetadata).tag
+	return (
+		!isJSRPC(item) &&
+		typeof (item as WorkerVersionMetadata).id === 'string' &&
+		typeof (item as WorkerVersionMetadata).tag === 'string'
+	)
 }
 
 const isAnalyticsEngineDataset = (item?: unknown): item is AnalyticsEngineDataset => {
-	return !!(item as AnalyticsEngineDataset)?.writeDataPoint
+	return !isJSRPC(item) && !!(item as AnalyticsEngineDataset)?.writeDataPoint
 }
 
 const instrumentEnv = (env: Record<string, unknown>): Record<string, unknown> => {