airkjw/claude-mem
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: reset session when age limit exceeded, not just when completed

Browse Source 
Previous fix only reset sessions with completed_at_epoch set.
But mac sleep/resume without SessionEnd leaves sessions alive with stale
started_at_epoch, causing age limit to block all processing on next use.

Now resets started_at_epoch whenever the session is older than the
4-hour wall-clock limit (MAX_SESSION_MS), matching SDKAgent's threshold.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
JOUNGWOOK KWON
2026-04-17 09:13:19 +09:00
parent 1c71a2bd98
commit 87479d1706
1 changed files with 15 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/services/sqlite/sessions/create.ts
+15 -7
View File
@@ -43,16 +43,24 @@ export function createSDKSession(
// Check for existing session
const existing = db.prepare(`
SELECT id, platform_source, completed_at_epoch FROM sdk_sessions WHERE content_session_id = ?
`).get(contentSessionId) as { id: number; platform_source: string | null; completed_at_epoch: number | null } | undefined;
SELECT id, platform_source, completed_at_epoch, started_at_epoch FROM sdk_sessions WHERE content_session_id = ?
`).get(contentSessionId) as { id: number; platform_source: string | null; completed_at_epoch: number | null; started_at_epoch: number } | undefined;
// 4-hour wall-clock limit (must match SDKAgent.MAX_SESSION_WALL_CLOCK_MS)
const MAX_SESSION_MS = 4 * 60 * 60 * 1000;
if (existing) {
// If the session was previously completed (e.g., mac sleep/resume without proper SessionEnd),
// reset it so the age limit check starts fresh from now.
if (existing.completed_at_epoch) {
logger.info('SESSION', 'Resetting completed session on resume (mac sleep/resume detected)', {
// Reset session if it was completed OR if it's older than the wall-clock age limit.
// This handles mac sleep/resume scenarios where the worker restarts but SessionEnd
// never fired, leaving the session with a stale started_at_epoch that causes the
// age limit check to abort all observations and summaries.
const ageMs = nowEpoch - existing.started_at_epoch;
const needsReset = !!existing.completed_at_epoch || ageMs > MAX_SESSION_MS;
if (needsReset) {
logger.info('SESSION', 'Resetting stale session on resume (mac sleep/worker-restart detected)', {
contentSessionId,
previousCompletedAt: new Date(existing.completed_at_epoch).toISOString()
ageHours: Math.round(ageMs / 36e5 * 10) / 10,
wasCompleted: !!existing.completed_at_epoch
});
db.prepare(`
UPDATE sdk_sessions