Feature: Auto-suspend idle sessions #10

New issue

Closed

opened 2026-03-10 03:46:35 +00:00 by Mark · 2 comments

Mark commented 2026-03-10 03:46:35 +00:00

Owner

Copy link

Issue

Running sessions consume resources indefinitely. Auto-suspending idle sessions would save resources.

Location

• services/orchestrator/orchestrator.py
• Session monitoring logic

Current Implementation

Orchestrator only monitors but doesn't auto-stop sessions.

Proposed Solution

1. Track session activity (terminal input, file operations)
2. Configurable idle timeout (default: 30 min)
3. Warning before suspension
4. Graceful state preservation via snapshots

Implementation Details

# In orchestrator
def check_idle_sessions(self):
    for session in running_sessions:
        if time_since_last_activity > IDLE_TIMEOUT:
            create_snapshot(session)
            stop_session(session)

Acceptance Criteria

• Add activity tracking (last_active timestamp)
• Implement idle detection logic
• Add configurable timeout setting
• Create snapshot before auto-suspend
• Add UI indicator for idle time remaining
• Allow users to disable auto-suspend per session

Priority: Medium
Type: Feature
References: ROADMAP.md Resource Management

## Issue Running sessions consume resources indefinitely. Auto-suspending idle sessions would save resources. ## Location - `services/orchestrator/orchestrator.py` - Session monitoring logic ## Current Implementation Orchestrator only monitors but doesn't auto-stop sessions. ## Proposed Solution 1. Track session activity (terminal input, file operations) 2. Configurable idle timeout (default: 30 min) 3. Warning before suspension 4. Graceful state preservation via snapshots ## Implementation Details ```python # In orchestrator def check_idle_sessions(self): for session in running_sessions: if time_since_last_activity > IDLE_TIMEOUT: create_snapshot(session) stop_session(session) ``` ## Acceptance Criteria - [ ] Add activity tracking (last_active timestamp) - [ ] Implement idle detection logic - [ ] Add configurable timeout setting - [ ] Create snapshot before auto-suspend - [ ] Add UI indicator for idle time remaining - [ ] Allow users to disable auto-suspend per session **Priority:** Medium **Type:** Feature **References:** ROADMAP.md Resource Management

Mark added the

feature-request

label 2026-05-11 20:11:40 +00:00

Mark added

triage-approved

and removed

feature-request

labels 2026-05-19 02:51:06 +00:00

Mark added

blocked:verify-failed

and removed

triage-approved

labels 2026-05-21 08:40:00 +00:00

Mark commented 2026-05-21 08:40:00 +00:00

Author

Owner

Copy link

GSD VERIFY phase failed after 3 attempts.
═══ Gate: verify-passed (devforge#10) ═══
Mode: Tier 1 (functional only)

GATE OK: All gates up to 'implement' pass
── Tier 1: Functional ────────────────────────────────
Running tests...
❌ Tests failed (exit 1)
 FAIL  services/web/src/utils/terminalUtils.test.ts > terminalUtils > decodeTerminalMessage > should return empty Uint8Array for Blob input in sync decoder
 FAIL  services/web/src/utils/terminalUtils.test.ts > terminalUtils > debug mode > should return false when debug is not enabled
 FAIL  services/web/src/utils/terminalUtils.test.ts > terminalUtils > debug mode > should return true when debug is enabled
 FAIL  services/web/src/utils/terminalUtils.test.ts > terminalUtils > debug mode > should disable debug mode
ReferenceError: localStorage is not defined
 ❯ services/web/src/utils/terminalUtils.test.ts:12:5
 10|  beforeEach(() => {
 11|  vi.restoreAllMocks();
 12|  localStorage.clear();
 |  ^
 13|  });
 14| 

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[4/9]⎯

 Test Files  4 failed | 1 passed (5)
 Tests  6 failed | 4 passed (10)
 Start at  01:39:58
 Duration  645ms (transform 512ms, setup 0ms, import 142ms, tests 11ms, environment 1ms)
Running linter...
✅ Lint clean
⚠️ No type checker discovered
Previous test count: unknown

❌ GATE FAIL: verify-passed
- Tests failed with exit code 1
GATE FAIL: verify — blocked at 2026-05-21T08:40:00Z
Class: test-failure
Evidence: Verification failed: 1 failures
Git tag: devforge#10-gate-verify-fail-17424555cd2c (use gate-retry after fixing)

GSD VERIFY phase failed after 3 attempts. ═══ Gate: verify-passed (devforge#10) ═══ Mode: Tier 1 (functional only) GATE OK: All gates up to 'implement' pass ── Tier 1: Functional ──────────────────────────────── Running tests... ❌ Tests failed (exit 1)  FAIL  services/web/src/utils/terminalUtils.test.ts > terminalUtils > decodeTerminalMessage > should return empty Uint8Array for Blob input in sync decoder  FAIL  services/web/src/utils/terminalUtils.test.ts > terminalUtils > debug mode > should return false when debug is not enabled  FAIL  services/web/src/utils/terminalUtils.test.ts > terminalUtils > debug mode > should return true when debug is enabled  FAIL  services/web/src/utils/terminalUtils.test.ts > terminalUtils > debug mode > should disable debug mode ReferenceError: localStorage is not defined  ❯ services/web/src/utils/terminalUtils.test.ts:12:5  10|  beforeEach(() => {  11|  vi.restoreAllMocks();  12|  localStorage.clear();  |  ^  13|  });  14|  ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[4/9]⎯  Test Files  4 failed | 1 passed (5)  Tests  6 failed | 4 passed (10)  Start at  01:39:58  Duration  645ms (transform 512ms, setup 0ms, import 142ms, tests 11ms, environment 1ms) Running linter... ✅ Lint clean ⚠️ No type checker discovered Previous test count: unknown ❌ GATE FAIL: verify-passed - Tests failed with exit code 1 GATE FAIL: verify — blocked at 2026-05-21T08:40:00Z Class: test-failure Evidence: Verification failed: 1 failures Git tag: devforge#10-gate-verify-fail-17424555cd2c (use gate-retry after fixing)

Mark removed the

blocked:verify-failed

label 2026-05-28 07:23:12 +00:00

Mark added the

feature-request

triage-approved

labels 2026-05-30 15:25:29 +00:00

Mark added

blocked:needs-decision

and removed

feature-request

triage-approved

labels 2026-06-01 08:03:28 +00:00

Mark commented 2026-06-01 08:03:28 +00:00

Author

Owner

Copy link

GSD RESEARCH phase could not determine a clear fix.

]11;?\Now I have the complete picture. Let me provide my evaluation:

STATUS: UNCLEAR

EVIDENCE:
The feature is partially implemented but non-functional:

Already exists:

• last_activity_at field on Session model (models.py:71-72)
• idle_timeout_hours field on Session model (models.py:74)
• Idle detection logic in HealthMonitor.cleanup_old_sessions() (health.py:182-203)
• Stops containers when idle_hours > session.idle_timeout_hours

**Critical gap - activity tracking nev

GSD RESEARCH phase could not determine a clear fix. ]11;?\Now I have the complete picture. Let me provide my evaluation: STATUS: UNCLEAR EVIDENCE: The feature is **partially implemented but non-functional**: **Already exists:** - `last_activity_at` field on Session model (`models.py:71-72`) - `idle_timeout_hours` field on Session model (`models.py:74`) - Idle detection logic in `HealthMonitor.cleanup_old_sessions()` (`health.py:182-203`) - Stops containers when `idle_hours > session.idle_timeout_hours` **Critical gap - activity tracking nev

Mark added

triage-approved

and removed

blocked:needs-decision

labels 2026-07-27 16:42:03 +00:00

Mark added the

in-progress

label 2026-08-05 03:54:41 +00:00

Mark removed the

triage-approved

in-progress

labels 2026-08-09 19:23:01 +00:00

Mark added the

blocked:needs-attention

label 2026-08-10 19:29:43 +00:00

Mark 2026-08-12 18:30:30 +00:00

• closed this issue
• removed the
  blocked:needs-attention
  label

Mark referenced this issue from a commit 2026-08-12 21:29:48 +00:00

feat(api/orchestrator): auto-suspend idle sessions (#10)

Mark referenced this issue from a commit 2026-08-12 21:29:48 +00:00

docs(issue-10): update Verification section to current test paths and field names

Mark referenced this issue from a commit 2026-08-12 21:29:48 +00:00

fix(#10): unblock verify gate — exempt /settings/ from CSRF, repair ssh invalid_name test

Mark referenced this issue from a commit 2026-08-12 21:29:48 +00:00

feat(web): idle auto-suspend UI indicator + per-session toggle (#10)

Mark referenced this issue from a commit 2026-08-12 21:29:48 +00:00

fix(orchestrator): apply clock-skew margin before auto-suspend; wire idle env config in compose (#10)

Mark referenced this issue from a commit 2026-08-12 21:29:48 +00:00

fix(#10): touch session activity on file download; add regression test

Mark referenced this issue from a commit 2026-08-12 21:29:49 +00:00

fix(#10): surface idle fields in session detail/list responses

Mark referenced this issue from a commit 2026-08-12 21:29:49 +00:00

Apply changes from kilo-gsd.py run for Mark/devforge#10

Mark referenced this issue from a commit 2026-08-12 21:29:49 +00:00

fix(#10): convert idle_timeout_hours to minutes in migration; repair health.py column reference

Mark referenced this issue 2026-08-14 03:47:22 +00:00

SSH key management UI missing from web frontend (backend-only fix shipped for #19) #47

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

main

No results found.

Labels

Clear labels   

blocked:gate-plan

  

blocked:gate-research

  

blocked:lint-failed

  

blocked:needs-attention

Issue requires human attention to resolve

  

blocked:needs-decision

  

blocked:no-changes

  

blocked:no-procedure

  

blocked:verify-failed

  

bug

Bug report

  

chore

Config / cleanup / ops housekeeping

  

feature-request

  

in-progress

  

needs-research

Ready for research pipeline

  

ops-task

Operations task — install, configure, deploy, API calls

  

quickfix

  

security/medium

Medium severity security issue

  

triage-approved

No labels

blocked:gate-plan

blocked:gate-research

blocked:lint-failed

blocked:needs-attention

blocked:needs-decision

blocked:no-changes

blocked:no-procedure

blocked:verify-failed

bug

chore

feature-request

in-progress

needs-research

ops-task

quickfix

security/medium

triage-approved

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

Mark/devforge#10

No description provided.