feat(executor): impl 活性检测替代纯硬墙——转录停更早杀,有增长跑满预算
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 46/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Active
- Domain
- backend, documentation, testing
Research direction
Start with the AGENTRUN timeout path and the preflight kill-stale code in .dev/flows/self_improve_flow_v2.py, including the referenced transcript and child-process checks. Add harness scenarios in .dev/flows/test/flow_v2_paths.py for stalled and growing transcripts, then document RECURSIVE_STALL_SECS and its relationship to RECURSIVE_IMPL_TIMEOUT in OPERATIONS.md. Done means stalled work is killed early, active work reaches its budget, and the unset-parameter behavior is unchanged.
Written by the indexing model from the issue text.
Description
Summary
impl/fix 等 AGENTRUN 长节点目前只有一条扁平的 timeout_secs 硬墙(self_improve_flow_v2.py:20 自注「watchdog(journal 增长/后代活性)暂由 AGENTRUN timeout_secs 硬墙替代」——本单就是还这笔账)。两个方向的代价都不对:
- 合法长活被截断:10-03 实证 #78/#68 超时轮转录在满 1h57m 时仍活跃推进(566 轮、尾部在跑 rustc 编译)——工作真实在做,墙到即死,只能靠 engine_error 重派 + L2 会话续跑接续,每轮吃 reaper 周期与台账噪音。
- 卡死烧满墙:若 agent 挂起(等输入/静默死循环),硬墙要等满整个预算(现
RECURSIVE_IMPL_TIMEOUT=14400s)才肯杀。
提案
给 AGENTRUN(执行器或宿主层)加活性检测:转录停止增长超过 stall 阈值且无活跃子进程 → 提前按 timeout 类击杀(沿用现有 SIGTERM watchdog 优雅刷盘链路);有增长则允许跑满预算。
- 检测对象:
--transcript-out的 transcript.jsonl size/mtime 增长 +pgrep -f "recursive.*--transcript-out"子进程活性(preflight 的 kill-stale 已用同款 pattern,.dev/flows/self_improve_flow_v2.pypreflight code 段)。 - stall 阈值可配:env
RECURSIVE_STALL_SECS,缺省关闭 = 现状零变化,显式配置才启用。 - 击杀语义沿用 timeout 类:engine_error + checkpoint/worktree 保留(10-03 起回收豁免)+ keeper 重派 L2 续跑;不引入新的重试类别。
验收
- harness(
.dev/flows/test/flow_v2_paths.py)新场景:模拟转录停更 → 提前击杀(先红后绿);模拟持续增长 → 跑满预算不被误杀。 - 缺省(未配
RECURSIVE_STALL_SECS)行为与现状逐字节一致。 OPERATIONS.md补参数说明(与RECURSIVE_IMPL_TIMEOUT的关系:本参数解决「何时提前杀」,后者解决「预算多少」)。
边界
- 纯 recursive 侧执行器/宿主改动,不改 agentproc 协议。
- 不动 D4 语义(timeout 类不原地重试)。
- Dominant language
- Rust
- Stars
- 4
- Forks
- 0
- Avg merge
- 5h 32m
- Merged PRs (30d)
- 7
Getting set up
- Ships a Dockerfile or Docker Compose file
- No pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from jeffkit/recursive
-
Difficulty 5/5 Over a week Newbie friendliness 20/100
jeffkit/recursive#134 · 4 comments ·
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 30/100
jeffkit/recursive#132 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
jeffkit/recursive#88 · 4 comments ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 58/100
jeffkit/recursive#87 · 11 comments ·
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
jeffkit/recursive#86 · 6 comments ·
Maintainers usually reply within 1 day
All issues in jeffkit/recursive
Similar issues
-
agent:triaged bug bughunt pm:npm priority:p1
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
SocketDev/socket-patch#1127 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
documentation enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
adorsys/status-list-server#619 ·
Maintainers usually reply within 2 days
-
batch-backport only backports the first 30 matching PRsPossibly taken @DvirDukhan claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 5 days
-
Configuration-level resource: `Allocate` rejects the kubelet's re-offer of the same device for a later container of the same Pod ("Unable to claim slot")Possibly taken @fang80913 claimed this 38 days ago. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
project-akri/akri#854 ·