Skip to content

Wake task polling on terminal notifications#1728

Open
HarperZ9 wants to merge 1 commit into
modelcontextprotocol:mainfrom
HarperZ9:agent/task-notification-wakes-polling
Open

Wake task polling on terminal notifications#1728
HarperZ9 wants to merge 1 commit into
modelcontextprotocol:mainfrom
HarperZ9:agent/task-notification-wakes-polling

Conversation

@HarperZ9

Copy link
Copy Markdown

Summary

  • wakes the active task polling loop when a matching notifications/tasks/status terminal update arrives
  • caches the latest notified task status so a terminal notification received just before the polling wait is not missed
  • adds regression coverage proving a task with a long poll interval fetches its result immediately after a terminal notification

Closes #1037

Verification

  • Red check before implementation: npm.cmd test -- --runTestsByPath src/__tests__/App.taskPolling.test.tsx --runInBand failed because the result remained Task created: task-notified-789. Polling for status...
  • Green check after implementation: npm.cmd test -- --runTestsByPath src/__tests__/App.taskPolling.test.tsx --runInBand -> 3 passed
  • npx.cmd prettier --check client/src/App.tsx client/src/__tests__/App.taskPolling.test.tsx -> passed
  • git diff --check client/src/App.tsx client/src/__tests__/App.taskPolling.test.tsx -> passed
  • npm.cmd run build-client -> passed

@HarperZ9
HarperZ9 marked this pull request as ready for review July 20, 2026 03:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Task complete notification is not handled

1 participant