Fix schedule_shot async task invocation with incorrect parameters

[Copilot]

The execute_scheduled_shot() nested async function was being invoked with keyword arguments it doesn't accept, causing all scheduled shot tests to fail with TypeError: got an unexpected keyword argument 'schedule_id'.

Changes

• coffee-relay/main.py: Removed erroneous parameter passing in asyncio.create_task() call
  • The nested function captures variables from outer scope by design
  • Was incorrectly being invoked with explicit schedule_id, shot_delay, preheat, etc. arguments
  • Changed from create_task(execute_scheduled_shot(schedule_id=..., shot_delay=..., ...)) to create_task(execute_scheduled_shot())

Example

# Before (incorrect - function doesn't accept parameters)
task = asyncio.create_task(
    execute_scheduled_shot(
        schedule_id=schedule_id,
        shot_delay=shot_delay,
        preheat=preheat,
        profile_id=profile_id,
        # ...
    )
)

# After (correct - function uses outer scope variables)
task = asyncio.create_task(execute_scheduled_shot())

This fixes 6 failing test cases in TestRunShotEndpoints and TestScheduledShotsPersistence.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.