fix(schema): Preserve video fps constraints
GitHub Actions / warden: xcodebuildmcp-tool-contract-review
completed
May 18, 2026 in 35m 5s
1 issue
xcodebuildmcp-tool-contract-review: Found 1 issue (1 medium)
Medium
`ctx.nextStepParams` set without manifest `nextSteps` template entries — params are silently discarded - `src/mcp/tools/ui-automation/wait_for_ui.ts:352-357`
The wait_for_ui.yaml manifest has no nextSteps template entries, so the ctx.nextStepParams set here is never consumed: postProcessSession only merges params into template steps, and falls through to return when no template steps exist. Follow the pattern in snapshot_ui.ts and set ctx.nextSteps directly instead.
⏱ 34m 35s · 5.1M in / 168.2k out · $5.93
Annotations
Check warning on line 357 in src/mcp/tools/ui-automation/wait_for_ui.ts
github-actions / warden: xcodebuildmcp-tool-contract-review
`ctx.nextStepParams` set without manifest `nextSteps` template entries — params are silently discarded
The `wait_for_ui.yaml` manifest has no `nextSteps` template entries, so the `ctx.nextStepParams` set here is never consumed: `postProcessSession` only merges params into template steps, and falls through to `return` when no template steps exist. Follow the pattern in `snapshot_ui.ts` and set `ctx.nextSteps` directly instead.
Loading