fix(device): Allow omitted platform in device tool schemas

cameroncooke · codex · cameroncooke · commit 4e924c2785da · 2026-04-30T21:09:08.000+01:00
Make the device platform schema truly optional so public device tool
schemas continue to accept omitted platform values. Device tools already
map a missing platform to iOS, and requiring the field would break
session-default-backed calls.

Co-Authored-By: OpenAI Codex &lt;noreply@openai.com&gt;
diff --git a/src/mcp/tools/device/build-settings.ts b/src/mcp/tools/device/build-settings.ts
@@ -21,7 +21,8 @@ function normalizeDevicePlatform(platform?: unknown): unknown {
 }
 
 export const devicePlatformSchema = z
-  .preprocess(normalizeDevicePlatform, z.enum(devicePlatformValues).optional())
+  .preprocess(normalizeDevicePlatform, z.enum(devicePlatformValues))
+  .optional()
   .describe('Device platform: iOS, watchOS, tvOS, or visionOS. Defaults to iOS.');
 
 export function mapDevicePlatform(platform?: DevicePlatform): XcodePlatform {

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,8 @@ function normalizeDevicePlatform(platform?: unknown): unknown {`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`export const devicePlatformSchema = z`
`24`		`- .preprocess(normalizeDevicePlatform, z.enum(devicePlatformValues).optional())`
	`24`	`+ .preprocess(normalizeDevicePlatform, z.enum(devicePlatformValues))`
	`25`	`+ .optional()`
`25`	`26`	`.describe('Device platform: iOS, watchOS, tvOS, or visionOS. Defaults to iOS.');`
`26`	`27`
`27`	`28`	`export function mapDevicePlatform(platform?: DevicePlatform): XcodePlatform {`