Skip to content

refactor: update lib.rs - #5

Closed
HelloWorldU wants to merge 1 commit into
mainfrom
agent/-mp2r
Closed

refactor: update lib.rs#5
HelloWorldU wants to merge 1 commit into
mainfrom
agent/-mp2r

Conversation

@HelloWorldU

Copy link
Copy Markdown
Owner

变更内容

  • 变更 lib.rs

类型

  • feat: 新功能
  • fix: Bug 修复
  • refactor: 代码重构
  • docs: 文档更新
  • test: 测试补充
  • chore: 构建/工具链

检查项

  • 本地 pre-commit 通过
  • 测试已补充或无需补充
  • 文档已同步或无需同步

HelloWorldU added a commit that referenced this pull request May 27, 2026
… fix prompt 污染

SKILL.md 加「Print 模式 ≠ Afk 模式」段:明确 afk 只在 --afk flag
显式传时生效,agent-engine 只传 --print 不传 --afk;用户在 UI 聊天
面板读输出,没 AskUserQuestion 不等于 afk 模式;本 skill GATE 优先级
高于系统指令的 afk 自动决策规则。

铁律段加「用户暂时不在 ≠ 批准」一句,堵掉「让流程往前推 / 自己决策更
高效」这类合理化跳过 GATE 的内部理由。

backlog 加两条:
- #4 kimi 在 print 模式下误用 afk 行为(已加 SKILL 缓解,待复测验证)
- #5 引擎自动注入 fix prompt 污染 UI 任务指令区(已修主场景,待真机
  观察是否有未覆盖路径)

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