fix(notification): Add session.refresh in reminders handler

mjunaidca · claude · mjunaidca · commit 300e8856240f · 2025-12-16T00:18:56.000+05:00
Same greenlet bug as task-events - need to refresh after commit before accessing notification attributes. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
diff --git a/apps/notification-service/src/notification_service/routers/dapr.py b/apps/notification-service/src/notification_service/routers/dapr.py
@@ -129,6 +129,7 @@ async def handle_reminders(
             )
             session.add(notification)
             await session.commit()
+            await session.refresh(notification)  # Refresh to get user_id without lazy load
             logger.info("[DAPR] Created reminder notification for user %s", notification.user_id)
 
         return {"status": "SUCCESS"}

Original file line number	Diff line number	Diff line change
`@@ -129,6 +129,7 @@ async def handle_reminders(`
`129`	`129`	`)`
`130`	`130`	`session.add(notification)`
`131`	`131`	`await session.commit()`
	`132`	`+ await session.refresh(notification) # Refresh to get user_id without lazy load`
`132`	`133`	`logger.info("[DAPR] Created reminder notification for user %s", notification.user_id)`
`133`	`134`
`134`	`135`	`return {"status": "SUCCESS"}`