Compare commits
12 Commits
ce4116bcb1
...
fix/issue-
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
54aa6419eb | ||
| 98a31070a7 | |||
| 26346235c9 | |||
| 2212fabf22 | |||
|
|
0fa778353b | ||
| 151efadca3 | |||
|
|
379d53cedc | ||
|
|
043542344a | ||
| e763ceb0ad | |||
|
|
61f06f825f | ||
| b76a9b883a | |||
|
|
ac850869fd |
67
.github/ISSUES/fix-queue-daemon-excess-agents.md
vendored
Normal file
67
.github/ISSUES/fix-queue-daemon-excess-agents.md
vendored
Normal file
@@ -0,0 +1,67 @@
|
|||||||
|
# Fix: Queue daemon spawning excess agents due to race condition
|
||||||
|
|
||||||
|
## Problem
|
||||||
|
|
||||||
|
When enqueueing multiple tasks (e.g., 6 tasks), the queue daemon was spawning many more subagents than expected, eventually exhausting container memory.
|
||||||
|
|
||||||
|
**Root Cause:** The combination of:
|
||||||
|
1. `process_queue()` calling `opencode run` directly instead of `kugetsu start`, bypassing all concurrency logic
|
||||||
|
2. `count_active_dev_sessions()` counting `pm-agent.json` toward `MAX_CONCURRENT_AGENTS`, reducing effective dev agent slots
|
||||||
|
3. No atomic locking around session count check + session file creation (TOCTOU race condition)
|
||||||
|
4. Background spawning of multiple concurrent processes in `process_queue()`
|
||||||
|
|
||||||
|
**Expected behavior:** With `MAX_CONCURRENT_AGENTS=3` and 6 tasks:
|
||||||
|
- Tasks should be processed sequentially via `kugetsu start`
|
||||||
|
- Only 3 dev agents should run at a time
|
||||||
|
- Tasks should queue and wait for slots to free up
|
||||||
|
|
||||||
|
## Solution
|
||||||
|
|
||||||
|
### 1. `count_active_dev_sessions()` - Exclude pm-agent
|
||||||
|
Only count actual dev agent session files (exclude `pm-agent.json`).
|
||||||
|
|
||||||
|
### 2. `process_queue()` - Call `kugetsu start` directly + retry logic
|
||||||
|
- Call `kugetsu start` directly (foreground, sequential) instead of spawning `opencode run` background process
|
||||||
|
- Dynamic batch size = available slots (removes need for `QUEUE_DAEMON_BATCH_SIZE`)
|
||||||
|
- Retry logic (max 3 attempts) on failure
|
||||||
|
- On failure: cleanup worktree/session and revert to `pending` state
|
||||||
|
- Save `fork_pid` to queue item for timeout handling
|
||||||
|
|
||||||
|
### 3. `cmd_start()` - Add flock
|
||||||
|
- Add flock around critical section (count check + fork)
|
||||||
|
- Track `fork_pid` for queue item timeout handling
|
||||||
|
|
||||||
|
### 4. Notification System
|
||||||
|
New notification types:
|
||||||
|
| Event | Type |
|
||||||
|
|-------|------|
|
||||||
|
| Task enqueued | `task_queued` |
|
||||||
|
| Task dequeued | `task_dequeued` |
|
||||||
|
| Task started | `task_started` |
|
||||||
|
| Task completed | `task_completed` |
|
||||||
|
| Task error | `task_error` |
|
||||||
|
|
||||||
|
### 5. Config
|
||||||
|
- Remove `QUEUE_DAEMON_BATCH_SIZE` (no longer needed - batch size is now dynamic)
|
||||||
|
|
||||||
|
## Notification Flow
|
||||||
|
|
||||||
|
| Event | Location | Type |
|
||||||
|
|-------|----------|------|
|
||||||
|
| Task enqueued | `enqueue_task()` | `task_queued` |
|
||||||
|
| Task dequeued | `process_queue()` after state change to `notified` | `task_dequeued` |
|
||||||
|
| Task started | `cmd_start()` after session file created | `task_started` |
|
||||||
|
| Task completed | `update_queue_item_state()` | `task_completed` |
|
||||||
|
| Task error | `update_queue_item_state()` | `task_error` |
|
||||||
|
|
||||||
|
## Out of Scope
|
||||||
|
|
||||||
|
- Re-check loop in cmd_start (checking if session DB is reliable) - deferred to separate research issue
|
||||||
|
- Buffer mechanism for excess forking (safety failsafe only)
|
||||||
|
|
||||||
|
## Status
|
||||||
|
|
||||||
|
- [x] Issue created
|
||||||
|
- [x] Implementation
|
||||||
|
- [x] PR created (#147)
|
||||||
|
- [ ] Merged
|
||||||
@@ -49,6 +49,8 @@ A default config file is created during `kugetsu init` with commented examples:
|
|||||||
| `MAX_CONCURRENT_AGENTS` | 3 | Maximum number of concurrent dev agents |
|
| `MAX_CONCURRENT_AGENTS` | 3 | Maximum number of concurrent dev agents |
|
||||||
| `KUGETSU_TEMP_DIR` | `~/.local/share/opencode/tool-output` | Temp directory for subagent tool output (useful in headless environments where /tmp is restricted) |
|
| `KUGETSU_TEMP_DIR` | `~/.local/share/opencode/tool-output` | Temp directory for subagent tool output (useful in headless environments where /tmp is restricted) |
|
||||||
| `KUGETSU_VERBOSITY` | `default` | PM agent verbosity level: `verbose`, `default`, or `quiet` |
|
| `KUGETSU_VERBOSITY` | `default` | PM agent verbosity level: `verbose`, `default`, or `quiet` |
|
||||||
|
| `QUEUE_DAEMON_INTERVAL_MINUTES` | 5 | How often daemon polls queue (in minutes) |
|
||||||
|
| `QUEUE_CLEANUP_AGE_DAYS` | 7 | Auto-cleanup completed/error items older than N days |
|
||||||
|
|
||||||
### Environment Variables for Agents
|
### Environment Variables for Agents
|
||||||
|
|
||||||
@@ -111,6 +113,10 @@ Each issue session gets its own git worktree to prevent conflicts:
|
|||||||
├── worktrees/
|
├── worktrees/
|
||||||
│ ├── github.com-shoko-kugetsu-14/ # Isolated workdir for issue #14
|
│ ├── github.com-shoko-kugetsu-14/ # Isolated workdir for issue #14
|
||||||
│ └── github.com-shoko-kugetsu-15/ # Isolated workdir for issue #15
|
│ └── github.com-shoko-kugetsu-15/ # Isolated workdir for issue #15
|
||||||
|
├── queue/
|
||||||
|
│ ├── items/ # Queue item JSON files
|
||||||
|
│ ├── daemon.pid # Daemon process ID
|
||||||
|
│ └── daemon.log # Daemon log output
|
||||||
└── index.json # Maps session IDs and issue refs to session files
|
└── index.json # Maps session IDs and issue refs to session files
|
||||||
```
|
```
|
||||||
|
|
||||||
@@ -258,16 +264,17 @@ kugetsu destroy --base -y
|
|||||||
|
|
||||||
### kugetsu delegate `<message>`
|
### kugetsu delegate `<message>`
|
||||||
|
|
||||||
Send a message to the PM agent for task coordination (fire-and-forget):
|
Send a message to the PM agent for task coordination via queue:
|
||||||
```bash
|
```bash
|
||||||
kugetsu delegate "work on issue #14"
|
kugetsu delegate "work on issue #14"
|
||||||
kugetsu delegate "review PR #92"
|
kugetsu delegate "review PR #92"
|
||||||
```
|
```
|
||||||
|
|
||||||
- Non-blocking: returns immediately, runs in background
|
- **Always enqueues** (fire-and-forget): returns immediately
|
||||||
- PM agent processes the message asynchronously
|
- Queue daemon polls queue and invokes PM when slots available
|
||||||
- Uses `KUGETSU_VERBOSITY` env var to control PM agent output verbosity
|
- Tasks are processed FIFO (first-in-first-out)
|
||||||
- Log output stored in `~/.kugetsu/logs/delegate-<timestamp>.log`
|
- Use `kugetsu queue list` to see pending tasks
|
||||||
|
- Use `kugetsu queue-daemon logs` to debug queue processing
|
||||||
|
|
||||||
### kugetsu logs [n]
|
### kugetsu logs [n]
|
||||||
|
|
||||||
@@ -328,35 +335,79 @@ kugetsu server default github # Set default server
|
|||||||
kugetsu server get github # Get server URL
|
kugetsu server get github # Get server URL
|
||||||
```
|
```
|
||||||
|
|
||||||
### kugetsu queue <list|enqueue|dequeue|clear>
|
### kugetsu queue <list|stats|clear>
|
||||||
|
|
||||||
Manage task queue for autonomous PM operation:
|
Manage task queue for autonomous PM operation:
|
||||||
```bash
|
```bash
|
||||||
kugetsu queue list # Show queued tasks
|
kugetsu queue list # Show queued tasks with status
|
||||||
kugetsu queue enqueue "task" # Add task to queue
|
kugetsu queue stats # Show queue statistics (total, pending, notified, completed, error)
|
||||||
kugetsu queue dequeue # Remove next task from queue
|
kugetsu queue clear # Clean up old completed/error items
|
||||||
kugetsu queue clear # Clear all queued tasks
|
kugetsu queue enqueue <issue-ref> <message> # Manually enqueue a task
|
||||||
```
|
```
|
||||||
|
|
||||||
- Queue stored in `~/.kugetsu/queue.json`
|
**Queue Item States:**
|
||||||
|
- `pending` - Waiting in queue, daemon can pick up
|
||||||
|
- `notified` - PM agent has picked up the task
|
||||||
|
- `completed` - Dev agent finished, PR created
|
||||||
|
- `error` - Timeout or failure
|
||||||
|
|
||||||
|
### kugetsu queue-daemon <start|stop|restart|status|logs>
|
||||||
|
|
||||||
|
Manage the queue daemon background process:
|
||||||
|
```bash
|
||||||
|
kugetsu queue-daemon start # Start daemon in background
|
||||||
|
kugetsu queue-daemon stop # Stop daemon
|
||||||
|
kugetsu queue-daemon restart # Restart daemon
|
||||||
|
kugetsu queue-daemon status # Check if daemon is running
|
||||||
|
kugetsu queue-daemon logs # Show recent daemon logs
|
||||||
|
```
|
||||||
|
|
||||||
|
**Daemon Behavior:**
|
||||||
|
1. Runs at configurable interval (default: 5 minutes)
|
||||||
|
2. Checks if active agents < MAX_CONCURRENT_AGENTS
|
||||||
|
3. Picks 1-N pending items (configurable batch size)
|
||||||
|
4. Forks PM session for each picked item
|
||||||
|
5. PM decides whether to use `start` or `continue`
|
||||||
|
|
||||||
|
**Queue Directory:**
|
||||||
|
```
|
||||||
|
~/.kugetsu/queue/
|
||||||
|
├── items/ # Queue item JSON files
|
||||||
|
│ ├── q_1234567890.json # One file per queued task
|
||||||
|
│ └── q_1234567891.json
|
||||||
|
├── daemon.pid # Daemon process ID
|
||||||
|
├── daemon.lock # Daemon lock file
|
||||||
|
└── daemon.log # Daemon log output
|
||||||
|
```
|
||||||
|
|
||||||
## Workflow Example
|
## Workflow Example
|
||||||
|
|
||||||
|
### First-time Setup
|
||||||
```bash
|
```bash
|
||||||
# First-time setup (requires TTY)
|
# Initialize kugetsu (requires TTY)
|
||||||
kugetsu init
|
kugetsu init
|
||||||
# Creates: base session + pm-agent session
|
|
||||||
|
|
||||||
# Start work on issue
|
# Start the queue daemon (for autonomous operation)
|
||||||
kugetsu start github.com/shoko/kugetsu#14 "implement feature X"
|
kugetsu queue-daemon start
|
||||||
# Creates: worktree at ~/.kugetsu/worktrees/github.com-shoko-kugetsu-14/
|
```
|
||||||
|
|
||||||
# Continue later
|
### Normal Workflow
|
||||||
|
```bash
|
||||||
|
# Enqueue tasks via delegate - agents will process them automatically
|
||||||
|
kugetsu delegate "work on issue #14"
|
||||||
|
kugetsu delegate "review PR #92"
|
||||||
|
|
||||||
|
# Check queue status
|
||||||
|
kugetsu queue list # See pending tasks
|
||||||
|
kugetsu queue stats # See statistics
|
||||||
|
|
||||||
|
# Debug queue daemon
|
||||||
|
kugetsu queue-daemon status # Is daemon running?
|
||||||
|
kugetsu queue-daemon logs # See daemon logs
|
||||||
|
|
||||||
|
# Continue work on existing issue
|
||||||
kugetsu continue github.com/shoko/kugetsu#14 "add tests"
|
kugetsu continue github.com/shoko/kugetsu#14 "add tests"
|
||||||
|
|
||||||
# Continue again
|
|
||||||
kugetsu continue github.com/shoko/kugetsu#14 "fix failing test"
|
|
||||||
|
|
||||||
# List all sessions
|
# List all sessions
|
||||||
kugetsu list
|
kugetsu list
|
||||||
|
|
||||||
@@ -367,6 +418,21 @@ kugetsu prune --force
|
|||||||
kugetsu destroy github.com/shoko/kugetsu#14
|
kugetsu destroy github.com/shoko/kugetsu#14
|
||||||
```
|
```
|
||||||
|
|
||||||
|
### Queue Daemon Management
|
||||||
|
```bash
|
||||||
|
# Check if daemon is running
|
||||||
|
kugetsu queue-daemon status
|
||||||
|
|
||||||
|
# View daemon logs for debugging
|
||||||
|
kugetsu queue-daemon logs
|
||||||
|
|
||||||
|
# Restart daemon if needed
|
||||||
|
kugetsu queue-daemon restart
|
||||||
|
|
||||||
|
# Stop daemon
|
||||||
|
kugetsu queue-daemon stop
|
||||||
|
```
|
||||||
|
|
||||||
## Headless Operation
|
## Headless Operation
|
||||||
|
|
||||||
This design solves the headless CLI limitation discovered in Issue #14:
|
This design solves the headless CLI limitation discovered in Issue #14:
|
||||||
|
|||||||
@@ -13,8 +13,19 @@ VERBOSITY_DIR="$KUGETSU_DIR/verbosity"
|
|||||||
|
|
||||||
MAX_CONCURRENT_AGENTS="${MAX_CONCURRENT_AGENTS:-3}"
|
MAX_CONCURRENT_AGENTS="${MAX_CONCURRENT_AGENTS:-3}"
|
||||||
KUGETSU_VERBOSITY="${KUGETSU_VERBOSITY:-default}"
|
KUGETSU_VERBOSITY="${KUGETSU_VERBOSITY:-default}"
|
||||||
|
CONTEXT_DIR="${CONTEXT_DIR:-$KUGETSU_DIR/context}"
|
||||||
|
ENABLE_CONTEXT_DUMP="${ENABLE_CONTEXT_DUMP:-true}"
|
||||||
WORKTREE_CHECK_PR_STATUS="${WORKTREE_CHECK_PR_STATUS:-true}"
|
WORKTREE_CHECK_PR_STATUS="${WORKTREE_CHECK_PR_STATUS:-true}"
|
||||||
|
|
||||||
|
QUEUE_DIR="${QUEUE_DIR:-$KUGETSU_DIR/queue}"
|
||||||
|
QUEUE_ITEMS_DIR="${QUEUE_ITEMS_DIR:-$QUEUE_DIR/items}"
|
||||||
|
QUEUE_DAEMON_PID_FILE="${QUEUE_DAEMON_PID_FILE:-$QUEUE_DIR/daemon.pid}"
|
||||||
|
QUEUE_DAEMON_LOCK_FILE="${QUEUE_DAEMON_LOCK_FILE:-$QUEUE_DIR/daemon.lock}"
|
||||||
|
QUEUE_DAEMON_LOG_FILE="${QUEUE_DAEMON_LOG_FILE:-$QUEUE_DIR/daemon.log}"
|
||||||
|
QUEUE_DAEMON_INTERVAL_MINUTES="${QUEUE_DAEMON_INTERVAL_MINUTES:-5}"
|
||||||
|
QUEUE_CLEANUP_AGE_DAYS="${QUEUE_CLEANUP_AGE_DAYS:-7}"
|
||||||
|
TASK_TIMEOUT_HOURS="${TASK_TIMEOUT_HOURS:-1}"
|
||||||
|
|
||||||
# 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"
|
||||||
@@ -51,7 +62,7 @@ count_active_dev_sessions() {
|
|||||||
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=$(basename "$session_file")
|
||||||
if [ "$filename" != "base.json" ]; then
|
if [ "$filename" != "base.json" ] && [ "$filename" != "pm-agent.json" ]; then
|
||||||
count=$((count + 1))
|
count=$((count + 1))
|
||||||
fi
|
fi
|
||||||
fi
|
fi
|
||||||
@@ -79,6 +90,10 @@ Usage:
|
|||||||
kugetsu destroy --pm-agent [-y] Delete pm-agent session (not recommended)
|
kugetsu destroy --pm-agent [-y] Delete pm-agent session (not recommended)
|
||||||
kugetsu destroy --base [-y] Delete base session
|
kugetsu destroy --base [-y] Delete base session
|
||||||
kugetsu set-pr <issue-ref> <pr-url> Set PR URL for session (for PR tracking)
|
kugetsu set-pr <issue-ref> <pr-url> Set PR URL for session (for PR tracking)
|
||||||
|
kugetsu context <issue-ref> Show context for issue
|
||||||
|
kugetsu queue [list|stats|clear] Show queue status or statistics
|
||||||
|
kugetsu queue enqueue <issue-ref> <message> Enqueue a task (normally via delegate)
|
||||||
|
kugetsu queue-daemon [start|stop|restart|status|logs] Manage queue daemon
|
||||||
kugetsu help Show this help
|
kugetsu help Show this help
|
||||||
|
|
||||||
Issue Ref Format:
|
Issue Ref Format:
|
||||||
@@ -317,6 +332,391 @@ filename_to_issue_ref() {
|
|||||||
echo "$name" | sed 's/-\([0-9]*\)$/#\1' | sed 's/-/\//g'
|
echo "$name" | sed 's/-\([0-9]*\)$/#\1' | sed 's/-/\//g'
|
||||||
}
|
}
|
||||||
|
|
||||||
|
issue_ref_to_context_file() {
|
||||||
|
local issue_ref="$1"
|
||||||
|
local context_filename=$(issue_ref_to_filename "$issue_ref")
|
||||||
|
echo "$CONTEXT_DIR/${context_filename}.json"
|
||||||
|
}
|
||||||
|
|
||||||
|
kugetsu_context_load() {
|
||||||
|
local issue_ref="$1"
|
||||||
|
|
||||||
|
if [ "$ENABLE_CONTEXT_DUMP" != "true" ]; then
|
||||||
|
echo ""
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
local context_file=$(issue_ref_to_context_file "$issue_ref")
|
||||||
|
|
||||||
|
if [ ! -f "$context_file" ]; then
|
||||||
|
echo ""
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
python3 << PYEOF
|
||||||
|
import json
|
||||||
|
import sys
|
||||||
|
|
||||||
|
context_file = "$context_file"
|
||||||
|
|
||||||
|
try:
|
||||||
|
with open(context_file, 'r') as f:
|
||||||
|
ctx = json.load(f)
|
||||||
|
|
||||||
|
lines = []
|
||||||
|
lines.append("## PREVIOUS CONTEXT")
|
||||||
|
lines.append(f"Issue: {ctx.get('issue_ref', 'unknown')}")
|
||||||
|
lines.append(f"Last updated: {ctx.get('updated_at', 'unknown')}")
|
||||||
|
lines.append(f"Current branch: {ctx.get('current_branch', 'unknown')}")
|
||||||
|
lines.append("")
|
||||||
|
lines.append("### Previous work summary:")
|
||||||
|
lines.append(ctx.get('last_message', '(no previous message)'))
|
||||||
|
lines.append("")
|
||||||
|
|
||||||
|
history = ctx.get('conversation_history', [])
|
||||||
|
if history:
|
||||||
|
lines.append("### Conversation history:")
|
||||||
|
for msg in history[-5:]:
|
||||||
|
role = msg.get('role', 'unknown')
|
||||||
|
content = msg.get('content', '')
|
||||||
|
ts = msg.get('timestamp', '')
|
||||||
|
lines.append(f"- [{ts}] {role}: {content[:200]}...")
|
||||||
|
|
||||||
|
print('\n'.join(lines))
|
||||||
|
|
||||||
|
except Exception as e:
|
||||||
|
print(f"Warning: Failed to load context: {e}", file=sys.stderr)
|
||||||
|
print("", file=sys.stderr)
|
||||||
|
PYEOF
|
||||||
|
}
|
||||||
|
|
||||||
|
kugetsu_context_dump() {
|
||||||
|
local issue_ref="$1"
|
||||||
|
local message="$2"
|
||||||
|
local branch_name="${3:-}"
|
||||||
|
|
||||||
|
if [ "$ENABLE_CONTEXT_DUMP" != "true" ]; then
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
local context_file=$(issue_ref_to_context_file "$issue_ref")
|
||||||
|
mkdir -p "$CONTEXT_DIR"
|
||||||
|
|
||||||
|
python3 << PYEOF
|
||||||
|
import json
|
||||||
|
import os
|
||||||
|
from datetime import datetime
|
||||||
|
|
||||||
|
context_file = "$context_file"
|
||||||
|
issue_ref = "$issue_ref"
|
||||||
|
message = """$message"""
|
||||||
|
branch_name = "$branch_name"
|
||||||
|
|
||||||
|
context = {
|
||||||
|
"issue_ref": issue_ref,
|
||||||
|
"current_branch": branch_name,
|
||||||
|
"updated_at": datetime.now().isoformat() + "Z",
|
||||||
|
"last_message": message[:500] if message else "",
|
||||||
|
"conversation_history": []
|
||||||
|
}
|
||||||
|
|
||||||
|
if os.path.exists(context_file):
|
||||||
|
try:
|
||||||
|
with open(context_file, 'r') as f:
|
||||||
|
existing = json.load(f)
|
||||||
|
|
||||||
|
history = existing.get('conversation_history', [])
|
||||||
|
history.append({
|
||||||
|
"role": "user",
|
||||||
|
"content": message[:1000] if message else "",
|
||||||
|
"timestamp": datetime.now().isoformat() + "Z"
|
||||||
|
})
|
||||||
|
history = history[-20:]
|
||||||
|
|
||||||
|
context["conversation_history"] = history
|
||||||
|
context["created_at"] = existing.get("created_at", context["updated_at"])
|
||||||
|
except:
|
||||||
|
context["created_at"] = datetime.now().isoformat() + "Z"
|
||||||
|
else:
|
||||||
|
context["created_at"] = datetime.now().isoformat() + "Z"
|
||||||
|
|
||||||
|
with open(context_file, 'w') as f:
|
||||||
|
json.dump(context, f, indent=2)
|
||||||
|
PYEOF
|
||||||
|
}
|
||||||
|
|
||||||
|
kugetsu_context_update_message() {
|
||||||
|
local issue_ref="$1"
|
||||||
|
local message="$2"
|
||||||
|
|
||||||
|
if [ "$ENABLE_CONTEXT_DUMP" != "true" ]; then
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
local context_file=$(issue_ref_to_context_file "$issue_ref")
|
||||||
|
|
||||||
|
if [ ! -f "$context_file" ]; then
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
python3 << PYEOF
|
||||||
|
import json
|
||||||
|
from datetime import datetime
|
||||||
|
|
||||||
|
context_file = "$context_file"
|
||||||
|
message = """$message"""
|
||||||
|
|
||||||
|
try:
|
||||||
|
with open(context_file, 'r') as f:
|
||||||
|
ctx = json.load(f)
|
||||||
|
|
||||||
|
history = ctx.get('conversation_history', [])
|
||||||
|
history.append({
|
||||||
|
"role": "assistant",
|
||||||
|
"content": message[:1000] if message else "",
|
||||||
|
"timestamp": datetime.now().isoformat() + "Z"
|
||||||
|
})
|
||||||
|
history = history[-20:]
|
||||||
|
|
||||||
|
ctx["conversation_history"] = history
|
||||||
|
ctx["last_message"] = message[:500] if message else ""
|
||||||
|
ctx["updated_at"] = datetime.now().isoformat() + "Z"
|
||||||
|
|
||||||
|
with open(context_file, 'w') as f:
|
||||||
|
json.dump(ctx, f, indent=2)
|
||||||
|
except Exception as e:
|
||||||
|
pass
|
||||||
|
PYEOF
|
||||||
|
}
|
||||||
|
|
||||||
|
ensure_queue_dirs() {
|
||||||
|
mkdir -p "$QUEUE_ITEMS_DIR"
|
||||||
|
}
|
||||||
|
|
||||||
|
generate_queue_id() {
|
||||||
|
echo "q_$(date +%s)_$$_$RANDOM"
|
||||||
|
}
|
||||||
|
|
||||||
|
enqueue_task() {
|
||||||
|
local issue_ref="$1"
|
||||||
|
local message="$2"
|
||||||
|
|
||||||
|
if [ -z "$issue_ref" ] || [ -z "$message" ]; then
|
||||||
|
echo "Error: enqueue_task requires <issue-ref> and <message>" >&2
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
|
||||||
|
validate_issue_ref "$issue_ref"
|
||||||
|
ensure_queue_dirs
|
||||||
|
|
||||||
|
local queue_id=$(generate_queue_id)
|
||||||
|
local pending_since=$(date -Iseconds)
|
||||||
|
|
||||||
|
python3 << PYEOF
|
||||||
|
import json
|
||||||
|
|
||||||
|
queue_item = {
|
||||||
|
"id": "$queue_id",
|
||||||
|
"issue_ref": "$issue_ref",
|
||||||
|
"message": """$message""",
|
||||||
|
"state": "pending",
|
||||||
|
"pending_since": "$pending_since",
|
||||||
|
"notified_at": None,
|
||||||
|
"completed_at": None,
|
||||||
|
"error": None
|
||||||
|
}
|
||||||
|
|
||||||
|
with open("$QUEUE_ITEMS_DIR/${queue_id}.json", "w") as f:
|
||||||
|
json.dump(queue_item, f, indent=2)
|
||||||
|
|
||||||
|
print(f"Enqueued: $queue_id")
|
||||||
|
PYEOF
|
||||||
|
|
||||||
|
kugetsu_add_notification "task_queued" "Task queued: $issue_ref" "$issue_ref"
|
||||||
|
}
|
||||||
|
|
||||||
|
get_pending_tasks() {
|
||||||
|
local limit="${1:-10}"
|
||||||
|
|
||||||
|
if [ ! -d "$QUEUE_ITEMS_DIR" ]; then
|
||||||
|
echo "[]"
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
find "$QUEUE_ITEMS_DIR" -name "*.json" -type f 2>/dev/null | while read -r file; do
|
||||||
|
local state=$(python3 -c "import json; print(json.load(open('$file')).get('state', ''))" 2>/dev/null || echo "")
|
||||||
|
if [ "$state" = "pending" ]; then
|
||||||
|
cat "$file"
|
||||||
|
fi
|
||||||
|
done | head -"$limit"
|
||||||
|
}
|
||||||
|
|
||||||
|
get_queue_stats() {
|
||||||
|
local total=0
|
||||||
|
local pending=0
|
||||||
|
local notified=0
|
||||||
|
local completed=0
|
||||||
|
local error=0
|
||||||
|
|
||||||
|
if [ -d "$QUEUE_ITEMS_DIR" ]; then
|
||||||
|
for file in "$QUEUE_ITEMS_DIR"/*.json; do
|
||||||
|
[ -f "$file" ] || continue
|
||||||
|
total=$((total + 1))
|
||||||
|
local state=$(python3 -c "import json; print(json.load(open('$file')).get('state', ''))" 2>/dev/null || echo "")
|
||||||
|
case "$state" in
|
||||||
|
pending) pending=$((pending + 1)) ;;
|
||||||
|
notified) notified=$((notified + 1)) ;;
|
||||||
|
completed) completed=$((completed + 1)) ;;
|
||||||
|
error) error=$((error + 1)) ;;
|
||||||
|
esac
|
||||||
|
done
|
||||||
|
fi
|
||||||
|
|
||||||
|
echo "{\"total\": $total, \"pending\": $pending, \"notified\": $notified, \"completed\": $completed, \"error\": $error}"
|
||||||
|
}
|
||||||
|
|
||||||
|
update_queue_item_state() {
|
||||||
|
local queue_id="$1"
|
||||||
|
local new_state="$2"
|
||||||
|
local session_id="${3:-}"
|
||||||
|
local pid="${4:-}"
|
||||||
|
|
||||||
|
local item_file="$QUEUE_ITEMS_DIR/${queue_id}.json"
|
||||||
|
if [ ! -f "$item_file" ]; then
|
||||||
|
echo "Error: Queue item not found: $queue_id" >&2
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
|
||||||
|
python3 << PYEOF
|
||||||
|
import json
|
||||||
|
import os
|
||||||
|
from datetime import datetime
|
||||||
|
|
||||||
|
item_file = "$item_file"
|
||||||
|
new_state = "$new_state"
|
||||||
|
session_id = "$session_id"
|
||||||
|
pid = "$pid"
|
||||||
|
|
||||||
|
with open(item_file, 'r') as f:
|
||||||
|
item = json.load(f)
|
||||||
|
|
||||||
|
issue_ref = item.get('issue_ref', '')
|
||||||
|
|
||||||
|
item['state'] = new_state
|
||||||
|
|
||||||
|
if new_state == "notified":
|
||||||
|
item['notified_at'] = datetime.now().isoformat() + "Z"
|
||||||
|
if session_id:
|
||||||
|
item['opencode_session_id'] = session_id
|
||||||
|
if pid:
|
||||||
|
item['pid'] = int(pid) if pid.isdigit() else None
|
||||||
|
elif new_state == "completed":
|
||||||
|
item['completed_at'] = datetime.now().isoformat() + "Z"
|
||||||
|
os.system(f"kugetsu_add_notification 'task_completed' 'Task completed: {issue_ref}' '{issue_ref}'")
|
||||||
|
elif new_state == "error":
|
||||||
|
item['error'] = datetime.now().isoformat() + "Z"
|
||||||
|
os.system(f"kugetsu_add_notification 'task_error' 'Task error: {issue_ref}' '{issue_ref}'")
|
||||||
|
|
||||||
|
with open(item_file, 'w') as f:
|
||||||
|
json.dump(item, f, indent=2)
|
||||||
|
|
||||||
|
print(f"Updated $queue_id to state: $new_state")
|
||||||
|
PYEOF
|
||||||
|
}
|
||||||
|
|
||||||
|
check_task_timeouts() {
|
||||||
|
if [ ! -d "$QUEUE_ITEMS_DIR" ]; then
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
local timeout_hours="${TASK_TIMEOUT_HOURS:-1}"
|
||||||
|
|
||||||
|
for item in "$QUEUE_ITEMS_DIR"/*.json; do
|
||||||
|
[ -f "$item" ] || continue
|
||||||
|
|
||||||
|
local state=$(python3 -c "import json; print(json.load(open('$item')).get('state', ''))" 2>/dev/null)
|
||||||
|
if [ "$state" != "notified" ]; then
|
||||||
|
continue
|
||||||
|
fi
|
||||||
|
|
||||||
|
local notified_at=$(python3 -c "import json; print(json.load(open('$item')).get('notified_at', ''))" 2>/dev/null)
|
||||||
|
if [ -z "$notified_at" ]; then
|
||||||
|
continue
|
||||||
|
fi
|
||||||
|
|
||||||
|
local queue_id=$(basename "$item" .json)
|
||||||
|
local pid=$(python3 -c "import json; print(json.load(open('$item')).get('pid', ''))" 2>/dev/null)
|
||||||
|
local session_id=$(python3 -c "import json; print(json.load(open('$item')).get('opencode_session_id', ''))" 2>/dev/null)
|
||||||
|
|
||||||
|
local notified_epoch=$(date -d "$notified_at" +%s 2>/dev/null || echo "0")
|
||||||
|
local now_epoch=$(date +%s)
|
||||||
|
local hours_elapsed=$(( (now_epoch - notified_epoch) / 3600 ))
|
||||||
|
|
||||||
|
if [ "$hours_elapsed" -ge "$timeout_hours" ]; then
|
||||||
|
echo "Task $queue_id timed out after ${hours_elapsed}h (limit: ${timeout_hours}h)"
|
||||||
|
|
||||||
|
if [ -n "$pid" ] && kill -0 "$pid" 2>/dev/null; then
|
||||||
|
echo "Killing process $pid"
|
||||||
|
kill "$pid" 2>/dev/null || true
|
||||||
|
fi
|
||||||
|
|
||||||
|
if [ -n "$session_id" ]; then
|
||||||
|
local worktree_path=""
|
||||||
|
for session_file in "$SESSIONS_DIR"/*.json; do
|
||||||
|
[ -f "$session_file" ] || continue
|
||||||
|
local sess_id=$(python3 -c "import json; print(json.load(open('$session_file')).get('opencode_session_id', ''))" 2>/dev/null)
|
||||||
|
if [ "$sess_id" = "$session_id" ]; then
|
||||||
|
worktree_path=$(python3 -c "import json; print(json.load(open('$session_file')).get('worktree_path', ''))" 2>/dev/null)
|
||||||
|
break
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
|
||||||
|
if [ -n "$worktree_path" ]; then
|
||||||
|
pkill -f "opencode.*$worktree_path" 2>/dev/null || true
|
||||||
|
fi
|
||||||
|
fi
|
||||||
|
|
||||||
|
update_queue_item_state "$queue_id" "error"
|
||||||
|
|
||||||
|
local issue_ref=$(python3 -c "import json; print(json.load(open('$item')).get('issue_ref', ''))" 2>/dev/null)
|
||||||
|
if [ -n "$issue_ref" ]; then
|
||||||
|
local session_file=$(get_session_for_issue "$issue_ref")
|
||||||
|
if [ -n "$session_file" ] && [ "$session_file" != "null" ]; then
|
||||||
|
python3 << PYEOF
|
||||||
|
import json
|
||||||
|
session_path = "$SESSIONS_DIR/$session_file"
|
||||||
|
try:
|
||||||
|
with open(session_path, 'r') as f:
|
||||||
|
session = json.load(f)
|
||||||
|
session['state'] = 'timeout'
|
||||||
|
with open(session_path, 'w') as f:
|
||||||
|
json.dump(session, f, indent=2)
|
||||||
|
print(f"Marked session for $issue_ref as timeout")
|
||||||
|
except Exception as e:
|
||||||
|
print(f"Error marking session: {e}")
|
||||||
|
PYEOF
|
||||||
|
fi
|
||||||
|
fi
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
}
|
||||||
|
|
||||||
|
cleanup_old_queue_items() {
|
||||||
|
local days="${QUEUE_CLEANUP_AGE_DAYS:-7}"
|
||||||
|
|
||||||
|
if [ ! -d "$QUEUE_ITEMS_DIR" ]; then
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
find "$QUEUE_ITEMS_DIR" -name "*.json" -type f -mtime "+$days" 2>/dev/null | while read -r file; do
|
||||||
|
local state=$(python3 -c "import json; print(json.load(open('$file')).get('state', ''))" 2>/dev/null || echo "")
|
||||||
|
if [ "$state" = "completed" ] || [ "$state" = "error" ]; then
|
||||||
|
rm -f "$file"
|
||||||
|
echo "Cleaned up: $(basename "$file")"
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
}
|
||||||
|
|
||||||
update_session_pr_url() {
|
update_session_pr_url() {
|
||||||
local issue_ref="$1"
|
local issue_ref="$1"
|
||||||
local pr_url="$2"
|
local pr_url="$2"
|
||||||
@@ -824,9 +1224,218 @@ find_sessions_by_issue_number() {
|
|||||||
echo "$results"
|
echo "$results"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
cmd_queue() {
|
||||||
|
local action="${1:-list}"
|
||||||
|
shift
|
||||||
|
|
||||||
|
case "$action" in
|
||||||
|
list)
|
||||||
|
ensure_queue_dirs
|
||||||
|
local stats=$(get_queue_stats)
|
||||||
|
echo "Queue Statistics:"
|
||||||
|
echo "$stats" | python3 -c "import json, sys; d=json.load(sys.stdin); print(f\" Total: {d['total']}\n Pending: {d['pending']}\n Notified: {d['notified']}\n Completed: {d['completed']}\n Error: {d['error']}\")"
|
||||||
|
echo ""
|
||||||
|
echo "Pending tasks:"
|
||||||
|
local count=0
|
||||||
|
for item in "$QUEUE_ITEMS_DIR"/*.json; do
|
||||||
|
[ -f "$item" ] || continue
|
||||||
|
local state=$(python3 -c "import json; print(json.load(open('$item')).get('state', '')" 2>/dev/null)
|
||||||
|
if [ "$state" = "pending" ]; then
|
||||||
|
count=$((count + 1))
|
||||||
|
python3 -c "import json; d=json.load(open('$item')); print(f\" [{d['id']}] {d['issue_ref']}: {d['message'][:50]}...\n pending since: {d['pending_since']}\")" 2>/dev/null
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
if [ $count -eq 0 ]; then
|
||||||
|
echo " (none)"
|
||||||
|
fi
|
||||||
|
;;
|
||||||
|
stats)
|
||||||
|
local stats=$(get_queue_stats)
|
||||||
|
echo "$stats" | python3 -c "import json, sys; d=json.load(sys.stdin); print(json.dumps(d, indent=2))"
|
||||||
|
;;
|
||||||
|
clear)
|
||||||
|
echo "Cleaning up old queue items..."
|
||||||
|
cleanup_old_queue_items
|
||||||
|
;;
|
||||||
|
enqueue)
|
||||||
|
local issue_ref="${1:-}"
|
||||||
|
local message="${2:-}"
|
||||||
|
if [ -z "$issue_ref" ] || [ -z "$message" ]; then
|
||||||
|
echo "Usage: kugetsu queue enqueue <issue-ref> <message>" >&2
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
enqueue_task "$issue_ref" "$message"
|
||||||
|
;;
|
||||||
|
*)
|
||||||
|
echo "Usage: kugetsu queue [list|stats|clear|enqueue <issue-ref> <message>]" >&2
|
||||||
|
exit 1
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
}
|
||||||
|
|
||||||
|
cmd_queue_daemon() {
|
||||||
|
local action="${1:-status}"
|
||||||
|
shift
|
||||||
|
|
||||||
|
case "$action" in
|
||||||
|
start)
|
||||||
|
if [ -f "$QUEUE_DAEMON_PID_FILE" ]; then
|
||||||
|
local old_pid=$(cat "$QUEUE_DAEMON_PID_FILE" 2>/dev/null)
|
||||||
|
if [ -n "$old_pid" ] && kill -0 "$old_pid" 2>/dev/null; then
|
||||||
|
echo "Daemon is already running with PID $old_pid"
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
rm -f "$QUEUE_DAEMON_PID_FILE"
|
||||||
|
fi
|
||||||
|
|
||||||
|
mkdir -p "$(dirname "$QUEUE_DAEMON_LOG_FILE")"
|
||||||
|
nohup bash "$0" queue-daemon run >> "$QUEUE_DAEMON_LOG_FILE" 2>&1 &
|
||||||
|
local daemon_pid=$!
|
||||||
|
echo "$daemon_pid" > "$QUEUE_DAEMON_PID_FILE"
|
||||||
|
echo "Queue daemon started with PID $daemon_pid"
|
||||||
|
echo "Log file: $QUEUE_DAEMON_LOG_FILE"
|
||||||
|
;;
|
||||||
|
stop)
|
||||||
|
if [ ! -f "$QUEUE_DAEMON_PID_FILE" ]; then
|
||||||
|
echo "Daemon PID file not found. Is the daemon running?"
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
local pid=$(cat "$QUEUE_DAEMON_PID_FILE")
|
||||||
|
if [ -n "$pid" ] && kill -0 "$pid" 2>/dev/null; then
|
||||||
|
kill "$pid"
|
||||||
|
rm -f "$QUEUE_DAEMON_PID_FILE"
|
||||||
|
echo "Daemon stopped (PID $pid)"
|
||||||
|
else
|
||||||
|
echo "Daemon not running (stale PID file)"
|
||||||
|
rm -f "$QUEUE_DAEMON_PID_FILE"
|
||||||
|
fi
|
||||||
|
;;
|
||||||
|
restart)
|
||||||
|
cmd_queue_daemon stop
|
||||||
|
sleep 1
|
||||||
|
cmd_queue_daemon start
|
||||||
|
;;
|
||||||
|
status)
|
||||||
|
if [ -f "$QUEUE_DAEMON_PID_FILE" ]; then
|
||||||
|
local pid=$(cat "$QUEUE_DAEMON_PID_FILE")
|
||||||
|
if [ -n "$pid" ] && kill -0 "$pid" 2>/dev/null; then
|
||||||
|
echo "Queue daemon is running (PID $pid)"
|
||||||
|
else
|
||||||
|
echo "Daemon not running (stale PID file)"
|
||||||
|
rm -f "$QUEUE_DAEMON_PID_FILE"
|
||||||
|
fi
|
||||||
|
else
|
||||||
|
echo "Queue daemon is not running"
|
||||||
|
fi
|
||||||
|
;;
|
||||||
|
logs)
|
||||||
|
local lines="${1:-50}"
|
||||||
|
if [ -f "$QUEUE_DAEMON_LOG_FILE" ]; then
|
||||||
|
tail -"$lines" "$QUEUE_DAEMON_LOG_FILE"
|
||||||
|
else
|
||||||
|
echo "No daemon log file found"
|
||||||
|
fi
|
||||||
|
;;
|
||||||
|
run)
|
||||||
|
queue_daemon_loop
|
||||||
|
;;
|
||||||
|
*)
|
||||||
|
echo "Usage: kugetsu queue-daemon [start|stop|restart|status|logs]" >&2
|
||||||
|
exit 1
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
}
|
||||||
|
|
||||||
|
queue_daemon_loop() {
|
||||||
|
local pid=$$
|
||||||
|
echo "$pid" > "$QUEUE_DAEMON_PID_FILE"
|
||||||
|
echo "Queue daemon started (PID $pid) at $(date)"
|
||||||
|
|
||||||
|
while true; do
|
||||||
|
sleep $((QUEUE_DAEMON_INTERVAL_MINUTES * 60))
|
||||||
|
|
||||||
|
if [ ! -f "$QUEUE_DAEMON_PID_FILE" ] || [ "$(cat "$QUEUE_DAEMON_PID_FILE")" != "$pid" ]; then
|
||||||
|
echo "PID file changed, stopping daemon"
|
||||||
|
exit 0
|
||||||
|
fi
|
||||||
|
|
||||||
|
check_task_timeouts
|
||||||
|
process_queue
|
||||||
|
done
|
||||||
|
}
|
||||||
|
|
||||||
|
process_queue() {
|
||||||
|
local active_count=$(count_active_dev_sessions)
|
||||||
|
|
||||||
|
if [ "$active_count" -ge "$MAX_CONCURRENT_AGENTS" ]; then
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
local available_slots=$((MAX_CONCURRENT_AGENTS - active_count))
|
||||||
|
|
||||||
|
if [ "$available_slots" -le 0 ]; then
|
||||||
|
return
|
||||||
|
fi
|
||||||
|
|
||||||
|
local count=0
|
||||||
|
for item in $(ls -t "$QUEUE_ITEMS_DIR"/*.json 2>/dev/null | head -20); do
|
||||||
|
[ $count -ge "$available_slots" ] && break
|
||||||
|
[ -f "$item" ] || continue
|
||||||
|
|
||||||
|
local state=$(python3 -c "import json; print(json.load(open('$item')).get('state', ''))" 2>/dev/null)
|
||||||
|
if [ "$state" != "pending" ]; then
|
||||||
|
continue
|
||||||
|
fi
|
||||||
|
|
||||||
|
local queue_id=$(basename "$item" .json)
|
||||||
|
local issue_ref=$(python3 -c "import json; print(json.load(open('$item')).get('issue_ref', ''))" 2>/dev/null)
|
||||||
|
local message=$(python3 -c "import json; print(json.load(open('$item')).get('message', ''))" 2>/dev/null)
|
||||||
|
|
||||||
|
if [ -z "$issue_ref" ] || [ -z "$message" ]; then
|
||||||
|
continue
|
||||||
|
fi
|
||||||
|
|
||||||
|
update_queue_item_state "$queue_id" "notified"
|
||||||
|
kugetsu_add_notification "task_dequeued" "Task dequeued: $issue_ref" "$issue_ref"
|
||||||
|
|
||||||
|
local log_file="$LOGS_DIR/delegate-${queue_id}.log"
|
||||||
|
mkdir -p "$LOGS_DIR"
|
||||||
|
|
||||||
|
local max_retries=3
|
||||||
|
local attempt=1
|
||||||
|
local success=false
|
||||||
|
local fork_pid=""
|
||||||
|
|
||||||
|
while [ $attempt -le $max_retries ]; do
|
||||||
|
if kugetsu start "$issue_ref" "$message" >> "$log_file" 2>&1; then
|
||||||
|
success=true
|
||||||
|
break
|
||||||
|
fi
|
||||||
|
|
||||||
|
echo "Attempt $attempt failed for $queue_id, cleaning up..." >> "$log_file"
|
||||||
|
|
||||||
|
local session_file="$(issue_ref_to_filename "$issue_ref").json"
|
||||||
|
local worktree_path=$(issue_ref_to_worktree_path "$issue_ref" "$PWD")
|
||||||
|
|
||||||
|
[ -f "$SESSIONS_DIR/$session_file" ] && rm -f "$SESSIONS_DIR/$session_file"
|
||||||
|
worktree_exists "$issue_ref" "$PWD" && remove_worktree_for_issue "$issue_ref" "$PWD"
|
||||||
|
remove_issue_from_index "$issue_ref" 2>/dev/null || true
|
||||||
|
|
||||||
|
attempt=$((attempt + 1))
|
||||||
|
done
|
||||||
|
|
||||||
|
if [ "$success" = true ]; then
|
||||||
|
echo "Started task $queue_id: $issue_ref"
|
||||||
|
count=$((count + 1))
|
||||||
|
else
|
||||||
|
echo "Failed to start task $queue_id after $max_retries attempts"
|
||||||
|
update_queue_item_state "$queue_id" "pending"
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
}
|
||||||
|
|
||||||
cmd_delegate() {
|
cmd_delegate() {
|
||||||
local message="${1:-}"
|
local message="${1:-}"
|
||||||
local verbosity="${KUGETSU_VERBOSITY:-default}"
|
|
||||||
|
|
||||||
if [ -z "$message" ]; then
|
if [ -z "$message" ]; then
|
||||||
echo "Error: message is required" >&2
|
echo "Error: message is required" >&2
|
||||||
@@ -834,94 +1443,22 @@ cmd_delegate() {
|
|||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
local pm_session=$(get_pm_agent_session_id)
|
|
||||||
if [ -z "$pm_session" ] || [ "$pm_session" = "null" ] || [ "$pm_session" = "None" ]; then
|
|
||||||
echo "Error: PM agent session not found. Run 'kugetsu init' first." >&2
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
|
|
||||||
mkdir -p "$LOGS_DIR"
|
|
||||||
local log_file="$LOGS_DIR/delegate-$(date +%s).log"
|
|
||||||
|
|
||||||
local parsed=$(parse_issue_ref_from_message "$message")
|
local parsed=$(parse_issue_ref_from_message "$message")
|
||||||
local gitserver=$(echo "$parsed" | cut -d'|' -f1)
|
local gitserver=$(echo "$parsed" | cut -d'|' -f1)
|
||||||
local owner=$(echo "$parsed" | cut -d'|' -f2)
|
local owner=$(echo "$parsed" | cut -d'|' -f2)
|
||||||
local repo=$(echo "$parsed" | cut -d'|' -f3)
|
local repo=$(echo "$parsed" | cut -d'|' -f3)
|
||||||
local issue_number=$(echo "$parsed" | cut -d'|' -f4)
|
local issue_number=$(echo "$parsed" | cut -d'|' -f4)
|
||||||
|
|
||||||
local missing_info=$(get_missing_info "$parsed")
|
if [ -z "$issue_number" ] || [ -z "$gitserver" ] || [ -z "$owner" ] || [ -z "$repo" ]; then
|
||||||
local context_injection=""
|
echo "Error: Could not parse issue reference from message" >&2
|
||||||
if [ -n "$missing_info" ]; then
|
echo "Message should contain an issue reference like 'github.com/user/repo#123'" >&2
|
||||||
context_injection=$(build_missing_info_context "$missing_info")
|
exit 1
|
||||||
echo "NOTE: Delegation missing information: ${missing_info}"
|
|
||||||
fi
|
fi
|
||||||
|
|
||||||
local candidates=""
|
local issue_ref="${gitserver}/${owner}/${repo}#${issue_number}"
|
||||||
local candidate_count=0
|
|
||||||
|
|
||||||
if [ -n "$issue_number" ]; then
|
enqueue_task "$issue_ref" "$message"
|
||||||
local worktrees=$(find_worktrees_by_issue_number "$issue_number")
|
echo "Task enqueued. The queue daemon will process it when a slot is available."
|
||||||
local sessions=$(find_sessions_by_issue_number "$issue_number")
|
|
||||||
|
|
||||||
while IFS=: read -r path type; do
|
|
||||||
if [ -n "$path" ]; then
|
|
||||||
candidate_count=$((candidate_count + 1))
|
|
||||||
candidates="${candidates}${candidate_count}) ${path} (${type})
|
|
||||||
"
|
|
||||||
fi
|
|
||||||
done <<< "$worktrees"
|
|
||||||
|
|
||||||
while IFS=: read -r path type; do
|
|
||||||
if [ -n "$path" ]; then
|
|
||||||
candidate_count=$((candidate_count + 1))
|
|
||||||
candidates="${candidates}${candidate_count}) ${path} (${type})
|
|
||||||
"
|
|
||||||
fi
|
|
||||||
done <<< "$sessions"
|
|
||||||
fi
|
|
||||||
|
|
||||||
local use_worktree=""
|
|
||||||
if [ $candidate_count -gt 0 ]; then
|
|
||||||
echo "Found $candidate_count existing worktree(s)/session(s) for issue #${issue_number}:"
|
|
||||||
echo "$candidates"
|
|
||||||
echo "r) Delegate anyway (without routing)"
|
|
||||||
echo "Which one to use? [1-${candidate_count}/r]: "
|
|
||||||
read -r choice
|
|
||||||
|
|
||||||
if [ "$choice" = "r" ] || [ -z "$choice" ]; then
|
|
||||||
use_worktree=""
|
|
||||||
elif [ "$choice" -ge 1 ] && [ "$choice" -le "$candidate_count" ]; then
|
|
||||||
local selected=$(echo "$candidates" | sed -n "${choice}p")
|
|
||||||
use_worktree=$(echo "$selected" | sed 's/) .*//')
|
|
||||||
fi
|
|
||||||
fi
|
|
||||||
|
|
||||||
local final_message="${message}${context_injection}"
|
|
||||||
|
|
||||||
if [ -n "$use_worktree" ]; then
|
|
||||||
if [ -d "$use_worktree" ]; then
|
|
||||||
echo "Using worktree: $use_worktree"
|
|
||||||
final_message="${final_message}
|
|
||||||
|
|
||||||
NOTE: Worktree selected: ${use_worktree}"
|
|
||||||
fi
|
|
||||||
fi
|
|
||||||
|
|
||||||
local temp_dir="${KUGETSU_TEMP_DIR:-$HOME/.local/share/opencode/tool-output}"
|
|
||||||
|
|
||||||
mkdir -p "$ENV_DIR"
|
|
||||||
local env_sh="set -a; export KUGETSU_TEMP_DIR='$temp_dir'; export KUGETSU_VERBOSITY='$verbosity'; "
|
|
||||||
if [ -f "$ENV_DIR/pm-agent.env" ]; then
|
|
||||||
env_sh="${env_sh}source '$ENV_DIR/pm-agent.env'; "
|
|
||||||
elif [ -f "$ENV_DIR/default.env" ]; then
|
|
||||||
env_sh="${env_sh}source '$ENV_DIR/default.env'; "
|
|
||||||
fi
|
|
||||||
env_sh="${env_sh}set +a; "
|
|
||||||
|
|
||||||
nohup sh -c "${env_sh}opencode run '${final_message}' --continue --session '$pm_session' >> '$log_file' 2>&1" > /dev/null 2>&1 &
|
|
||||||
disown
|
|
||||||
echo "Delegated to PM agent (logged to $(basename "$log_file"))"
|
|
||||||
echo "Verbosity: $verbosity"
|
|
||||||
}
|
}
|
||||||
|
|
||||||
cmd_logs() {
|
cmd_logs() {
|
||||||
@@ -1539,10 +2076,25 @@ cmd_start() {
|
|||||||
create_worktree "$issue_ref" "$parent_dir"
|
create_worktree "$issue_ref" "$parent_dir"
|
||||||
|
|
||||||
local session_file="$(issue_ref_to_filename "$issue_ref").json"
|
local session_file="$(issue_ref_to_filename "$issue_ref").json"
|
||||||
|
local fork_log="$SESSIONS_DIR/$session_file.fork.log"
|
||||||
|
local opencode_db="${OPENCODE_DB:-$HOME/.local/share/opencode/opencode.db}"
|
||||||
|
local lock_file="$KUGETSU_DIR/.session_lock"
|
||||||
|
local lock_fd=200
|
||||||
|
|
||||||
echo "Forking session for '$issue_ref'..."
|
> "$fork_log"
|
||||||
|
|
||||||
|
local fork_context=$(kugetsu_get_fork_context "$issue_ref")
|
||||||
|
local previous_context=$(kugetsu_context_load "$issue_ref")
|
||||||
|
local branch_name=$(issue_ref_to_branch_name "$issue_ref")
|
||||||
|
local full_message="${fork_context}
|
||||||
|
${previous_context}
|
||||||
|
|
||||||
|
## YOUR TASK
|
||||||
|
$message"
|
||||||
|
|
||||||
|
(
|
||||||
|
flock -x $lock_fd
|
||||||
|
|
||||||
# Session-counting: count actual dev sessions, reject if at limit
|
|
||||||
local active_count=$(count_active_dev_sessions)
|
local active_count=$(count_active_dev_sessions)
|
||||||
if [ "$active_count" -ge "$MAX_CONCURRENT_AGENTS" ]; then
|
if [ "$active_count" -ge "$MAX_CONCURRENT_AGENTS" ]; then
|
||||||
echo "Error: Max concurrent agents ($MAX_CONCURRENT_AGENTS) reached" >&2
|
echo "Error: Max concurrent agents ($MAX_CONCURRENT_AGENTS) reached" >&2
|
||||||
@@ -1551,16 +2103,7 @@ cmd_start() {
|
|||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
local fork_log="$SESSIONS_DIR/$session_file.fork.log"
|
echo "Forking session for '$issue_ref'..."
|
||||||
local opencode_db="${OPENCODE_DB:-$HOME/.local/share/opencode/opencode.db}"
|
|
||||||
|
|
||||||
> "$fork_log"
|
|
||||||
|
|
||||||
local fork_context=$(kugetsu_get_fork_context "$issue_ref")
|
|
||||||
local full_message="${fork_context}
|
|
||||||
|
|
||||||
## YOUR TASK
|
|
||||||
$message"
|
|
||||||
|
|
||||||
fix_session_permissions
|
fix_session_permissions
|
||||||
|
|
||||||
@@ -1659,8 +2202,13 @@ PYEOF
|
|||||||
|
|
||||||
add_issue_to_index "$issue_ref" "$session_file"
|
add_issue_to_index "$issue_ref" "$session_file"
|
||||||
|
|
||||||
|
kugetsu_context_dump "$issue_ref" "$message" "$branch_name"
|
||||||
|
|
||||||
|
kugetsu_add_notification "task_started" "Task started: $issue_ref" "$issue_ref"
|
||||||
|
|
||||||
echo "Session started for '$issue_ref': $new_session_id"
|
echo "Session started for '$issue_ref': $new_session_id"
|
||||||
echo "Worktree: $worktree_path"
|
echo "Worktree: $worktree_path"
|
||||||
|
) 200>"$lock_file"
|
||||||
}
|
}
|
||||||
|
|
||||||
cmd_continue() {
|
cmd_continue() {
|
||||||
@@ -1706,22 +2254,31 @@ cmd_continue() {
|
|||||||
local worktree_path=$(python3 -c "import json; print(json.load(open('$session_path')).get('worktree_path', ''))" 2>/dev/null || echo "")
|
local worktree_path=$(python3 -c "import json; print(json.load(open('$session_path')).get('worktree_path', ''))" 2>/dev/null || echo "")
|
||||||
|
|
||||||
echo "Continuing session for '$session_name'..."
|
echo "Continuing session for '$session_name'..."
|
||||||
|
|
||||||
|
local previous_context=$(kugetsu_context_load "$session_name")
|
||||||
|
local full_message="${previous_context}
|
||||||
|
|
||||||
|
## CONTINUE TASK
|
||||||
|
$message"
|
||||||
|
|
||||||
# Note: --continue always allowed (existing sessions don't count toward limit)
|
# Note: --continue always allowed (existing sessions don't count toward limit)
|
||||||
# Wrap in subshell with cd to ensure worktree directory is set correctly in session DB
|
# Wrap in subshell with cd to ensure worktree directory is set correctly in session DB
|
||||||
if [ -n "$worktree_path" ] && [ -d "$worktree_path" ]; then
|
if [ -n "$worktree_path" ] && [ -d "$worktree_path" ]; then
|
||||||
echo "Using worktree: $worktree_path"
|
echo "Using worktree: $worktree_path"
|
||||||
if [ "$DEBUG_MODE" = true ]; then
|
if [ "$DEBUG_MODE" = true ]; then
|
||||||
(cd "$worktree_path" && opencode run "$message" --continue --session "$opencode_session_id" --dir "$worktree_path" 2>&1) | tee "$session_path.debug.log" &
|
(cd "$worktree_path" && opencode run "$full_message" --continue --session "$opencode_session_id" --dir "$worktree_path" 2>&1) | tee "$session_path.debug.log" &
|
||||||
else
|
else
|
||||||
(cd "$worktree_path" && opencode run "$message" --continue --session "$opencode_session_id" --dir "$worktree_path" 2>&1) &
|
(cd "$worktree_path" && opencode run "$full_message" --continue --session "$opencode_session_id" --dir "$worktree_path" 2>&1) &
|
||||||
fi
|
fi
|
||||||
else
|
else
|
||||||
if [ "$DEBUG_MODE" = true ]; then
|
if [ "$DEBUG_MODE" = true ]; then
|
||||||
opencode run "$message" --continue --session "$opencode_session_id" 2>&1 | tee "$session_path.debug.log" &
|
opencode run "$full_message" --continue --session "$opencode_session_id" 2>&1 | tee "$session_path.debug.log" &
|
||||||
else
|
else
|
||||||
opencode run "$message" --continue --session "$opencode_session_id" 2>&1 &
|
opencode run "$full_message" --continue --session "$opencode_session_id" 2>&1 &
|
||||||
fi
|
fi
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
kugetsu_context_update_message "$session_name" "$message"
|
||||||
}
|
}
|
||||||
|
|
||||||
cmd_list() {
|
cmd_list() {
|
||||||
@@ -2022,12 +2579,38 @@ main() {
|
|||||||
local pr_url="${2:-}"
|
local pr_url="${2:-}"
|
||||||
if [ -z "$issue_ref" ] || [ -z "$pr_url" ]; then
|
if [ -z "$issue_ref" ] || [ -z "$pr_url" ]; then
|
||||||
echo "Usage: kugetsu set-pr <issue-ref> <pr-url>" >&2
|
echo "Usage: kugetsu set-pr <issue-ref> <pr-url>" >&2
|
||||||
echo "Example: kugetsu set-pr github.com/shoko/kugetsu#14 https://git.fbrns.co/shoko/kugetsu/pulls/123" >&2
|
echo "Example: kugetsu set-pr github.com/shoko/kugetsu#14 https://github.com/shoko/kugetsu/pulls/123" >&2
|
||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
validate_issue_ref "$issue_ref"
|
validate_issue_ref "$issue_ref"
|
||||||
update_session_pr_url "$issue_ref" "$pr_url"
|
update_session_pr_url "$issue_ref" "$pr_url"
|
||||||
;;
|
;;
|
||||||
|
context)
|
||||||
|
local issue_ref="${1:-}"
|
||||||
|
if [ -z "$issue_ref" ]; then
|
||||||
|
echo "Usage: kugetsu context <issue-ref>" >&2
|
||||||
|
echo "Show context for an issue" >&2
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
validate_issue_ref "$issue_ref"
|
||||||
|
local context_file=$(issue_ref_to_context_file "$issue_ref")
|
||||||
|
if [ -f "$context_file" ]; then
|
||||||
|
cat "$context_file"
|
||||||
|
else
|
||||||
|
echo "No context found for '$issue_ref'" >&2
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
;;
|
||||||
|
queue)
|
||||||
|
local action="${1:-list}"
|
||||||
|
shift
|
||||||
|
cmd_queue "$action" "$@"
|
||||||
|
;;
|
||||||
|
queue-daemon)
|
||||||
|
local action="${1:-status}"
|
||||||
|
shift
|
||||||
|
cmd_queue_daemon "$action" "$@"
|
||||||
|
;;
|
||||||
*)
|
*)
|
||||||
echo "Error: unknown command '$command'" >&2
|
echo "Error: unknown command '$command'" >&2
|
||||||
usage
|
usage
|
||||||
|
|||||||
Reference in New Issue
Block a user