fix(mcp-common): make outcome optional in observability event schema

zCloudflareMiniEvent required outcome on every event, but outcome only
describes an invocation and is absent on console.log lines emitted
inside one. query_worker_observability validates the whole event array
in a single .parse(), so any response containing one of these
non-invocation events was rejected outright and no logs were returned.

Fixes #418

Author: arimu1

.changeset/fix-workers-observability-outcome-optional.md

@@ -0,0 +1,7 @@
+---
+'@repo/mcp-common': patch
+---
+
+fix(mcp-common): make `outcome` optional in the Workers Observability response schema
+
+`zCloudflareMiniEvent` (`packages/mcp-common/src/types/workers-logs.types.ts`) required every event to carry `outcome`, but `outcome` only describes a whole invocation and is absent on the `console.log` lines emitted inside one. Because `query_worker_observability` validates the entire event array in a single `.parse()`, any response containing one of these non-invocation events was rejected outright and the tool returned no logs at all. `outcome` is now optional so responses with a mix of invocation and non-invocation events parse correctly.

packages/mcp-common/src/types/workers-logs.types.spec.ts

@@ -0,0 +1,48 @@
+import { describe, expect, it } from 'vitest'
+
+import { zCloudflareMiniEvent, zReturnedQueryRunEvents } from './workers-logs.types'
+
+describe('zCloudflareMiniEvent', () => {
+	it('parses a console.log event with no outcome', () => {
+		const event = {
+			event: {},
+			scriptName: 'my-worker',
+			eventType: 'cron',
+			requestId: '1RI7X6A7OCMC159U',
+		}
+		expect(() => zCloudflareMiniEvent.parse(event)).not.toThrow()
+	})
+
+	it('still parses an invocation-summary event with an outcome', () => {
+		const event = {
+			event: {},
+			scriptName: 'my-worker',
+			eventType: 'cron',
+			requestId: '1RI7X6A7OCMC159U',
+			outcome: 'ok',
+		}
+		expect(zCloudflareMiniEvent.parse(event).outcome).toBe('ok')
+	})
+})
+
+describe('zReturnedQueryRunEvents', () => {
+	it('does not discard the batch when one event has no outcome', () => {
+		const telemetryEvent = (workers: Record<string, unknown>) => ({
+			dataset: 'cloudflare-workers',
+			timestamp: 1784222146000,
+			source: 'log line',
+			$workers: {
+				event: {},
+				scriptName: 'my-worker',
+				eventType: 'cron',
+				requestId: '1RI7X6A7OCMC159U',
+				...workers,
+			},
+			$metadata: { id: 'evt-1' },
+		})
+		const result = zReturnedQueryRunEvents.parse({
+			events: [telemetryEvent({}), telemetryEvent({ outcome: 'ok' })],
+		})
+		expect(result.events).toHaveLength(2)
+	})
+})

packages/mcp-common/src/types/workers-logs.types.ts

@@ -248,7 +248,7 @@ const zCloudflareMiniEventDetails = z.object({
 export const zCloudflareMiniEvent = z.object({
 	event: zCloudflareMiniEventDetails,
 	scriptName: z.string(),
-	outcome: z.string(),
+	outcome: z.string().optional(),
 	eventType: z.enum([
 		'fetch',
 		'scheduled',