Skip to content

feat(agent-engine): CI failure auto-detect & fix loop - #2

Merged
HelloWorldU merged 1 commit into
mainfrom
feat/ci-monitor-loop
May 14, 2026
Merged

feat(agent-engine): CI failure auto-detect & fix loop#2
HelloWorldU merged 1 commit into
mainfrom
feat/ci-monitor-loop

Conversation

@HelloWorldU

Copy link
Copy Markdown
Owner

实现 PR 创建后的 CI 事后闭环:

  • github-api.ts: 新增 getCheckRuns + getCheckRunLogs,轮询 GitHub Checks API
  • agent.ts: 新增 startCiMonitor / stopCiMonitor / fixBasedOnCiFailure
    • PR 创建后自动启动 30s 间隔轮询
    • CI 失败时获取日志 -> runInstructionSilent 修复 -> autoSubmitForReview 重新提交
    • 支持已有 PR 追加 commit(不重复创建 PR)
    • Agent stop / engine delete-agent 时自动清理定时器
  • engine.ts: delete-agent 前调用 stopCiMonitor 防止内存泄漏
  • types/schemas: 新增可选 ciStatus 字段(不改 TaskStatus,最小侵入)
  • docs: 同步 ARCHITECTURE.md + STATUS.md

实现 PR 创建后的 CI 事后闭环:

- github-api.ts: 新增 getCheckRuns + getCheckRunLogs,轮询 GitHub Checks API
- agent.ts: 新增 startCiMonitor / stopCiMonitor / fixBasedOnCiFailure
  - PR 创建后自动启动 30s 间隔轮询
  - CI 失败时获取日志 → runInstructionSilent 修复 → autoSubmitForReview 重新提交
  - 支持已有 PR 追加 commit(不重复创建 PR)
  - Agent stop / engine delete-agent 时自动清理定时器
- engine.ts: delete-agent 前调用 stopCiMonitor 防止内存泄漏
- types/schemas: 新增可选 ciStatus 字段(不改 TaskStatus,最小侵入)
- docs: 同步 ARCHITECTURE.md + STATUS.md
@HelloWorldU
HelloWorldU merged commit 59ed1ae into main May 14, 2026
1 check passed
HelloWorldU pushed a commit that referenced this pull request May 20, 2026
#1 F5 刷新 dashboard 后 agents 不可见(前端没有「engine 已活着、拉回
当前 agents」的命令路径)
#2 orphan agent 删除时工作目录不清(engine 对未知 id 的 delete-agent
直接走早期分支,没兜底清理 E:/workspace/<id>)

每条带现象 / 根因猜测 / 修复方向 / 优先级,落地后从本文件移除。

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
HelloWorldU added a commit that referenced this pull request May 27, 2026
review-flow-fixes.md v4 阶段 4 实施完毕(C-1 / A / B / C-2+F / E 共
5 个 commit 全部落地)。原 backlog #1 F5 刷新 / #2 orphan 删目录
保留;新增 #3 收录 Bug D,记录调查现状 + 下次复现需收集的诊断数据。

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
HelloWorldU added a commit that referenced this pull request May 28, 2026
- 新增「engine→runtime」架构决策项作为首位优先级
- 新增 Bug #1 reviewer.comment 丢失(PR #25 暴露,源码确认)
- 新增 Bug #2 stop() 不级联取消异步链
- 新增 Bug #3 syncBranchWithMain 同步成功却报「同步提交失败」
- 保留 Bug #4 F5 刷新失踪
- 重新收回 Bug #5 CI log 截断方向错(PR #28 重现,被 8KB git config 噪音占满)
- 删除已实施的 syncBranchWithMain 完成项、低优 orphan 工作目录项、旧设计哲学段
HelloWorldU added a commit that referenced this pull request May 28, 2026
…iewer 修订

接管 PR #29 + 应用自审 reviewer 两处修订:

PR #29 原内容:
- agent.ts submitReview: 补 review.comment = comment || '',解决 fix loop 拿不到具体审阅意见的根因(Bug #1)
- agent.ts syncBranchWithMain: 删 merge 成功后多余的 gitCommit/abortMerge,squash merge 场景不再误报「同步提交失败」(Bug #3)
- agent.ts: sendInstruction 后增加 stopped 状态检查,最小可行的 stop 级联兜底(Bug #2 增量修,根治仍归并到架构决策项)
- github-api.ts getCheckRunLogs: 截断方向从 slice(0, 8000) 改为 slice(-8000)(Bug #5)
- 配套文档同步 + AgentDetail/Store 透传 review.comment 字段

reviewer 修订(本次额外):
- github-api.ts: '...[truncated]' 标记从尾部挪到头部,避免误导用户以为尾部还有截断
- backlog.md: footer 同步更新(4 个活跃 bug → 1 个 #4 F5 失踪)+ 应用本地删除架构项「风险」段
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant