Compare commits
4 Commits
fix/issue-
...
fix/issue-
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
2af8e54f42 | ||
| ab99647193 | |||
|
|
3deae2dd81 | ||
| 41c56a859c |
@@ -364,10 +364,14 @@ kugetsu queue-daemon logs # Show recent daemon logs
|
|||||||
|
|
||||||
**Daemon Behavior:**
|
**Daemon Behavior:**
|
||||||
1. Runs at configurable interval (default: 5 minutes)
|
1. Runs at configurable interval (default: 5 minutes)
|
||||||
2. Checks if active agents < MAX_CONCURRENT_AGENTS
|
2. Checks queue for pending items
|
||||||
3. Picks 1-N pending items (configurable batch size)
|
3. For each pending item:
|
||||||
4. Forks PM session for each picked item
|
- Acquires lock to prevent duplicate processing
|
||||||
5. PM decides whether to use `start` or `continue`
|
- Sources kugetsu-session.sh and calls `cmd_continue`
|
||||||
|
- `cmd_continue` handles worktree/session creation and forks dev agent from base session
|
||||||
|
- Updates queue item state to "notified"
|
||||||
|
4. Uses per-issue locking to prevent race conditions
|
||||||
|
5. Implements timeout for tasks that don't complete (marks as "error" after TASK_TIMEOUT_HOURS)
|
||||||
|
|
||||||
**Queue Directory:**
|
**Queue Directory:**
|
||||||
```
|
```
|
||||||
@@ -526,6 +530,13 @@ The script will:
|
|||||||
|
|
||||||
See [docs/kugetsu-setup.md](../../docs/kugetsu-setup.md) for full Tailscale setup documentation.
|
See [docs/kugetsu-setup.md](../../docs/kugetsu-setup.md) for full Tailscale setup documentation.
|
||||||
|
|
||||||
|
## Version History
|
||||||
|
|
||||||
|
### v0.2.3
|
||||||
|
- **Queue daemon context drift fix** (issue #156): Daemon now sources kugetsu-session.sh and calls `cmd_continue` directly instead of forking PM session. This fixes context drift where daemon would lose track of task state.
|
||||||
|
- **Daemon locking**: Added per-issue locking to prevent race conditions when multiple daemon instances run.
|
||||||
|
- **Timeout handling**: Tasks that don't complete within TASK_TIMEOUT_HOURS are marked as "error".
|
||||||
|
|
||||||
## Without kugetsu
|
## Without kugetsu
|
||||||
|
|
||||||
If kugetsu is not available, use opencode directly:
|
If kugetsu is not available, use opencode directly:
|
||||||
|
|||||||
@@ -26,6 +26,9 @@ QUEUE_DAEMON_INTERVAL_MINUTES="${QUEUE_DAEMON_INTERVAL_MINUTES:-5}"
|
|||||||
QUEUE_CLEANUP_AGE_DAYS="${QUEUE_CLEANUP_AGE_DAYS:-7}"
|
QUEUE_CLEANUP_AGE_DAYS="${QUEUE_CLEANUP_AGE_DAYS:-7}"
|
||||||
TASK_TIMEOUT_HOURS="${TASK_TIMEOUT_HOURS:-1}"
|
TASK_TIMEOUT_HOURS="${TASK_TIMEOUT_HOURS:-1}"
|
||||||
|
|
||||||
|
NETWORK_RETRY_ATTEMPTS="${NETWORK_RETRY_ATTEMPTS:-3}"
|
||||||
|
NETWORK_RETRY_DELAY_SECONDS="${NETWORK_RETRY_DELAY_SECONDS:-5}"
|
||||||
|
|
||||||
# Load user config overrides (~/.kugetsu/config)
|
# Load user config overrides (~/.kugetsu/config)
|
||||||
if [ -f "$KUGETSU_DIR/config" ]; then
|
if [ -f "$KUGETSU_DIR/config" ]; then
|
||||||
source "$KUGETSU_DIR/config"
|
source "$KUGETSU_DIR/config"
|
||||||
@@ -87,3 +90,24 @@ set_debug_mode() {
|
|||||||
|
|
||||||
echo "${filtered_args[@]}"
|
echo "${filtered_args[@]}"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
retry_with_backoff() {
|
||||||
|
local max_attempts="${1:-$NETWORK_RETRY_ATTEMPTS}"
|
||||||
|
local delay_seconds="${2:-$NETWORK_RETRY_DELAY_SECONDS}"
|
||||||
|
local command="$3"
|
||||||
|
local remaining_attempts=$max_attempts
|
||||||
|
|
||||||
|
while [ $remaining_attempts -gt 0 ]; do
|
||||||
|
if eval "$command"; then
|
||||||
|
return 0
|
||||||
|
fi
|
||||||
|
remaining_attempts=$((remaining_attempts - 1))
|
||||||
|
if [ $remaining_attempts -gt 0 ]; then
|
||||||
|
log "warn" "retry_with_backoff" "Command failed, $remaining_attempts retries remaining. Waiting ${delay_seconds}s..."
|
||||||
|
sleep "$delay_seconds"
|
||||||
|
delay_seconds=$((delay_seconds * 2))
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
log "error" "retry_with_backoff" "Command failed after $max_attempts attempts"
|
||||||
|
return 1
|
||||||
|
}
|
||||||
|
|||||||
@@ -13,7 +13,8 @@ count_active_dev_sessions() {
|
|||||||
if [ -d "$SESSIONS_DIR" ]; then
|
if [ -d "$SESSIONS_DIR" ]; then
|
||||||
for session_file in "$SESSIONS_DIR"/*.json; do
|
for session_file in "$SESSIONS_DIR"/*.json; do
|
||||||
if [ -f "$session_file" ]; then
|
if [ -f "$session_file" ]; then
|
||||||
local filename=$(basename "$session_file")
|
local filename
|
||||||
|
filename=$(basename "$session_file")
|
||||||
if [ "$filename" != "base.json" ] && [ "$filename" != "pm-agent.json" ]; then
|
if [ "$filename" != "base.json" ] && [ "$filename" != "pm-agent.json" ]; then
|
||||||
count=$((count + 1))
|
count=$((count + 1))
|
||||||
fi
|
fi
|
||||||
@@ -239,18 +240,39 @@ create_session() {
|
|||||||
return 1
|
return 1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
local before_file="$KUGETSU_DIR/sessions/before$$.json"
|
local before_file
|
||||||
local after_file="$KUGETSU_DIR/sessions/after$$.json"
|
before_file="$KUGETSU_DIR/sessions/before$$.json"
|
||||||
|
local after_file
|
||||||
|
after_file="$KUGETSU_DIR/sessions/after$$.json"
|
||||||
|
|
||||||
opencode session list --format=json > "$before_file" 2>/dev/null || printf '{}' > "$before_file"
|
opencode session list --format=json > "$before_file" 2>/dev/null || printf '{}' > "$before_file"
|
||||||
|
|
||||||
opencode run --fork --session "$base_session" "new session" >/dev/null 2>&1
|
local fork_success=false
|
||||||
|
local attempt=0
|
||||||
|
local max_attempts="${NETWORK_RETRY_ATTEMPTS:-3}"
|
||||||
|
|
||||||
|
while [ $attempt -lt $max_attempts ] && [ "$fork_success" = false ]; do
|
||||||
|
attempt=$((attempt + 1))
|
||||||
|
if opencode run --fork --session "$base_session" "new session" >/dev/null 2>&1; then
|
||||||
|
fork_success=true
|
||||||
|
elif [ $attempt -lt $max_attempts ]; then
|
||||||
|
log "warn" "create_session" "Fork attempt $attempt failed, retrying..."
|
||||||
|
sleep "$((attempt * 2))"
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
|
||||||
|
if [ "$fork_success" = false ]; then
|
||||||
|
log "error" "create_session" "Failed to fork session after $max_attempts attempts"
|
||||||
|
rm -f "$before_file" "$after_file"
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
|
||||||
sleep 1
|
sleep 1
|
||||||
|
|
||||||
opencode session list --format=json > "$after_file" 2>/dev/null || printf '{}' > "$after_file"
|
opencode session list --format=json > "$after_file" 2>/dev/null || printf '{}' > "$after_file"
|
||||||
|
|
||||||
local new_session_id=$(python3 << PYEOF
|
local new_session_id
|
||||||
|
new_session_id=$(python3 << PYEOF
|
||||||
import json
|
import json
|
||||||
|
|
||||||
with open("$before_file", 'r') as f:
|
with open("$before_file", 'r') as f:
|
||||||
@@ -389,20 +411,20 @@ ensure_session() {
|
|||||||
session_exists=true
|
session_exists=true
|
||||||
fi
|
fi
|
||||||
|
|
||||||
if $worktree_exists && $session_exists; then
|
if [ "$worktree_exists" = true ] && [ "$session_exists" = true ]; then
|
||||||
log "info" "ensure_session" "Session already exists for $issue_ref"
|
log "info" "ensure_session" "Session already exists for $issue_ref"
|
||||||
echo "continued"
|
echo "continued"
|
||||||
return 0
|
return 0
|
||||||
fi
|
fi
|
||||||
|
|
||||||
if ! $worktree_exists && $session_exists; then
|
if [ "$worktree_exists" = false ] && [ "$session_exists" = true ]; then
|
||||||
log "warn" "ensure_session" "Session exists but worktree is missing. Removing stale session..."
|
log "warn" "ensure_session" "Session exists but worktree is missing. Removing stale session..."
|
||||||
rm -f "$session_path"
|
rm -f "$session_path"
|
||||||
remove_issue_from_index "$issue_ref"
|
remove_issue_from_index "$issue_ref"
|
||||||
session_exists=false
|
session_exists=false
|
||||||
fi
|
fi
|
||||||
|
|
||||||
if ! $worktree_exists; then
|
if [ "$worktree_exists" = false ]; then
|
||||||
local wt_status=$(ensure_worktree "$issue_ref")
|
local wt_status=$(ensure_worktree "$issue_ref")
|
||||||
if [ "$wt_status" != "created" ] && [ "$wt_status" != "existed" ]; then
|
if [ "$wt_status" != "created" ] && [ "$wt_status" != "existed" ]; then
|
||||||
log "error" "ensure_session" "Failed to ensure worktree for $issue_ref"
|
log "error" "ensure_session" "Failed to ensure worktree for $issue_ref"
|
||||||
|
|||||||
@@ -76,7 +76,8 @@ create_worktree() {
|
|||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
local worktree_parent_dir=$(dirname "$worktree_path")
|
local worktree_parent_dir
|
||||||
|
worktree_parent_dir=$(dirname "$worktree_path")
|
||||||
mkdir -p "$worktree_parent_dir"
|
mkdir -p "$worktree_parent_dir"
|
||||||
|
|
||||||
if worktree_exists "$issue_ref" "$parent_dir"; then
|
if worktree_exists "$issue_ref" "$parent_dir"; then
|
||||||
@@ -85,15 +86,36 @@ create_worktree() {
|
|||||||
fi
|
fi
|
||||||
|
|
||||||
echo "Creating worktree at '$worktree_path'..."
|
echo "Creating worktree at '$worktree_path'..."
|
||||||
git clone "$repo_url" "$worktree_path" 2>/dev/null || {
|
|
||||||
echo "Error: Failed to clone repository" >&2
|
local clone_success=false
|
||||||
|
local attempt=0
|
||||||
|
local max_attempts="${NETWORK_RETRY_ATTEMPTS:-3}"
|
||||||
|
|
||||||
|
while [ $attempt -lt $max_attempts ] && [ "$clone_success" = false ]; do
|
||||||
|
attempt=$((attempt + 1))
|
||||||
|
if [ $attempt -gt 1 ]; then
|
||||||
|
echo "Clone attempt $attempt of $max_attempts..."
|
||||||
|
sleep "$((attempt * 2))"
|
||||||
|
fi
|
||||||
|
|
||||||
|
if git clone "$repo_url" "$worktree_path" 2>/dev/null; then
|
||||||
|
clone_success=true
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
|
||||||
|
if [ "$clone_success" = false ]; then
|
||||||
|
echo "Error: Failed to clone repository after $max_attempts attempts" >&2
|
||||||
exit 1
|
exit 1
|
||||||
}
|
fi
|
||||||
|
|
||||||
echo "Creating branch '$branch_name'..."
|
echo "Creating branch '$branch_name'..."
|
||||||
(cd "$worktree_path" && git checkout -b "$branch_name" origin/main 2>/dev/null || git checkout -b "$branch_name" main 2>/dev/null) || {
|
if git -C "$worktree_path" checkout -b "$branch_name" origin/main 2>/dev/null; then
|
||||||
|
:
|
||||||
|
elif git -C "$worktree_path" checkout -b "$branch_name" main 2>/dev/null; then
|
||||||
|
:
|
||||||
|
else
|
||||||
echo "Warning: Could not checkout branch (may need to run from within worktree after session)" >&2
|
echo "Warning: Could not checkout branch (may need to run from within worktree after session)" >&2
|
||||||
}
|
fi
|
||||||
|
|
||||||
echo "Worktree created at: $worktree_path"
|
echo "Worktree created at: $worktree_path"
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user