perf(middleware): await background task cancellation on timeout

[ishaanxgupta]

Summary
Added an await asyncio.gather(*tasks_to_cancel, return_exceptions=True) call inside the except asyncio.TimeoutError: block of wait_for_background_tasks

Previously, when the wait_for_background_tasks method timed out, it iterated over unfinished tasks and called task.cancel(). However, calling cancel() only requests cancellation; it doesn't wait for the task's cleanup logic (e.g., except asyncio.CancelledError:) to actually complete. This created orphaned tasks running asynchronously in the background.
Failing to await cancelled tasks leads to "Task was destroyed but it is pending!" errors at shutdown. More importantly, it leaks memory and limits scalability because background cleanup continues executing unbounded.

[MaheshtheDev]

@claude review this PR

[claude]

Claude Code is working…

I'll analyze this and get back to you.

View job run

[ishaanxgupta]

hey @MaheshtheDev I think the claude review is failing, please have a look

[MaheshtheDev]

@ishaanxgupta please update the package version number and rest all looking good

[ishaanxgupta]

@ishaanxgupta please update the package version number and rest all looking good

Done

[entelligence-ai-pr-reviews]

---

Confidence Score: 5/5 - Safe to Merge

Safe to merge — this patch correctly addresses a classic concurrent-modification hazard in wait_for_background_tasks by collecting incomplete tasks into a separate list before cancellation, and then properly awaiting their teardown via asyncio.gather(return_exceptions=True), preventing both silent coroutine leaks and potential RuntimeErrors from mutating a live collection during iteration. The fix is minimal, targeted, and follows established asyncio best practices for cooperative cancellation. No review comments were generated, heuristic analysis found zero issues across any severity tier, and the version bump in pyproject.toml and uv.lock is consistent with a patch release.

Key Findings:

• The separation of task collection from cancellation in wait_for_background_tasks correctly eliminates mutation of the live task collection during iteration, which was the root cause of the bug.
• asyncio.gather(..., return_exceptions=True) ensures that CancelledError instances raised by individual tasks are captured rather than propagated, allowing all tasks to be awaited cleanly without one failure short-circuiting the rest.
• Version bump from v1.0.2 to v1.0.3 in both pyproject.toml and uv.lock is consistent and appropriate for a bug-fix patch release with no API surface changes.
• No new tests are included, but the fix is a well-understood asyncio pattern with deterministic behavior, and the absence of tests is a minor concern rather than a blocking one for a patch of this scope.

Files requiring special attention

• supermemory_openai_sdk/middleware.py
• pyproject.toml
• uv.lock