Dong Hao
Dong Hao
Hi @dsp-ant , thank you for your advice. I have made commit. Would you help to check again?
Technical Details - The issue occurred when internal notifications were received during cleanup - Added _closed flag to track session state accurately - Modified notification handling to be cleanup-aware -...
> @dsp-ant @donghao1393 - if I may suggest a more straight-forward fix for this inline with codebase patterns? - [txbm@1918562](https://github.com/txbm/python-sdk/commit/19185626d57fdd670219784d10859c7d0a302c2e) I have tested this commit on my local and it's...
> Hey @donghao1393 - It seems to me that this can be simplified a lot now that there is the canceled notification types. > > I'm going to be doing...
Hi @jerome3o-anthropic , I have updated accordingly and passed my test. Would you help to take a review for the next?
> It seem the issue is fixed, either Claude Desktop update or using the latest main branch. As already I see cancelled got added as valid type, so should no...
> I have used the main branch. It's not in 1.2.1. For me this seem solving it. > > https://github.com/modelcontextprotocol/python-sdk/blob/f10665db4c2f676da1131617ad67715952258712/src/mcp/types.py#L995 > > > I testing 1 min task withtout issue...
> The "proceed while running" button is clickable. :) Oh, I will check.