shoko/kugetsu
1
0
Fork 0
Code Issues 17 Pull Requests Actions Packages Projects Releases 3 Wiki Activity

fix: enforce MAX_CONCURRENT_AGENTS limit properly (fixes #63) #64

Closed
shoko wants to merge 0 commits from fix/issue-63 into main
pull from: fix/issue-63
merge into: shoko:main
Conversation 0 Commits - Files Changed -
shoko commented 2026-04-01 04:13:41 +02:00
Owner
Copy Link

Summary

  • Fix MAX_CONCURRENT_AGENTS slot being released immediately after forking
  • Wait for child process to complete before releasing slot

Bug

In cmd_start(), the slot acquired via acquire_agent_slot() was released immediately after calling opencode run --fork because the parent returns immediately while the child continues running.

Fix

Capture the child PID with $! and add wait $child_pid before release_agent_slot(). This ensures the slot is only released after the forked process completes.

## Summary - Fix MAX_CONCURRENT_AGENTS slot being released immediately after forking - Wait for child process to complete before releasing slot ## Bug In cmd_start(), the slot acquired via acquire_agent_slot() was released immediately after calling `opencode run --fork` because the parent returns immediately while the child continues running. ## Fix Capture the child PID with `$!` and add `wait $child_pid` before `release_agent_slot()`. This ensures the slot is only released after the forked process completes.
shoko added 1 commit 2026-04-01 04:13:41 +02:00
fix: enforce MAX_CONCURRENT_AGENTS limit properly (fixes #63) a952204e15
shoko force-pushed fix/issue-63 from a952204e15 to bd4e8587b4 2026-04-01 05:05:32 +02:00 Compare
shoko closed this pull request 2026-04-01 08:42:28 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
0.2.x-only
Available only in 0.2.x, not backportable
 0.3.x-only
Available only in 0.3.x, not backportable
 backport
Backport compatible - can merge to 0.1.x
 blocked
Issue needs clarification/discussion before it can be worked on
 bug
Bug report
 critical
Critical - blocking parallel work
 feature
New feature
 foundation
Foundation work for 0.2.x
 high
High priority - workflow improvements
 meta
Meta issue tracking multiple issues
 need-review
PR needs review before merge
 parallelization
Issues related to parallelization
 ready
Issue is clear and ready to be implemented/delegated
 ready-to-merge
PR approved and ready to merge
 task-delegation
Issues related to task-delegation
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
shoko
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shoko/kugetsu#64
Delete Branch "fix/issue-63"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?