fix: implement session-counting for MAX_CONCURRENT_AGENTS limit (fixes #63) #65

Merged
shoko merged 6 commits from fix/issue-63-session-counting into main 2026-04-01 07:40:00 +02:00

6 Commits

Author SHA1 Message Date
shokollm
e38cf6bc8b docs: update benchmark with cloud architecture and memory analysis 2026-04-01 05:18:30 +00:00
shokollm
3cc2082a21 docs: update benchmark with session-counting and PM inclusion 2026-04-01 05:03:44 +00:00
shokollm
7ac4578369 fix: include PM in session count (all sessions count toward limit) 2026-04-01 04:57:52 +00:00
shokollm
7342a9a394 fix: implement session-counting for MAX_CONCURRENT_AGENTS limit (fixes #63)
Replaced broken slot-based mechanism with session-counting:

- Added count_active_dev_sessions() function that counts actual
  session files in ~/.kugetsu/sessions/, excluding base.json and pm-agent.json
- Modified cmd_start() to check session count before creating new session:
  - If count >= MAX_CONCURRENT_AGENTS, reject with error
  - Otherwise allow new session creation
- Removed wait  since --fork returns immediately
- cmd_continue() no longer counts toward limit (existing sessions
  can always continue via --continue)

This properly enforces MAX_CONCURRENT_AGENTS while preserving --fork
functionality. The slot mechanism didn't work because opencode run
--fork returns immediately after forking, not after child completes.
2026-04-01 04:14:15 +00:00
shokollm
bd4e8587b4 fix: enforce MAX_CONCURRENT_AGENTS limit properly (fixes #63)
- Fixed cmd_start() and cmd_continue() to wait for forked child
- Capture child PID with $! and wait before release_agent_slot
- This ensures slot is only released after child process completes
2026-04-01 03:05:13 +00:00
shokollm
bc60e644bf docs: add agent concurrency benchmark results 2026-04-01 01:46:27 +00:00