Skip to content

fix(runtime): replay #49 run-budget wiring onto clean main - #54

Merged
helsome merged 1 commit into
mainfrom
fix/17-run-budget-wiring
Sep 11, 2026
Merged

fix(runtime): replay #49 run-budget wiring onto clean main#54
helsome merged 1 commit into
mainfrom
fix/17-run-budget-wiring

Conversation

@helsome

@helsome helsome commented Sep 11, 2026

Copy link
Copy Markdown
Owner

背景

#49 (kbkb628:feat/17-run-budget-wiring) 是 stacked PR,依赖 #47#47 以 squash 方式合入后,#49 因历史不同而变成 mergeable_state=dirty,无法直接 rebase/merge;同时 GitHub App 无权限改写贡献者 fork 分支。

这条 PR 不重写业务实现,而是以当前已全绿的 main 为父提交,将 #49 最终 head (6f03674f...) 中的最终文件 blob 原样重放,从而移除已经由 #47 合入的 step1 历史。

实际 diff

相对当前 main:1 commit / 0 behind,仅保留 #17 step2 的 8 个真实变化:

  • packages/core/src/index.ts
  • packages/shared/src/kernel/agent-kernel-budget.test.ts
  • packages/shared/src/kernel/agent-kernel.ts
  • packages/shared/src/kernel/run-budget.ts
  • packages/shared/src/kernel/run-manager.test.ts
  • packages/shared/src/kernel/run-manager.ts
  • packages/shared/src/kernel/runaway-detector.ts
  • scripts/eval/budget-smoke.ts

已有验证

#49 已由贡献者完成真实 provider E2E:

  • AgentKernel → RunManager → PiRuntimeAdapter → pi runtime → DeepSeek
  • --max-model-calls 1budget_exhausted,partial answer 保留
  • 重复相同工具调用 → loop_detected,tool calls 保留

此前 PR 红灯的 7 个用例来自 gitignored Longbridge account fixtures,与 #17 无关;#10 已合入并验证当前 main 的 Unit tests / Typecheck / Secret scan 全绿。

本 PR 用于让 GitHub CI 在最新 main + #49 最终实现上重新验证。

Refs #17
Supersedes #49

Co-authored-by: kbkb628 278338969+kbkb628@users.noreply.github.com

Replays the completed #49 implementation onto the latest clean main after #47 was squash-merged, removing the stacked-PR history conflict while preserving the contributor's final code and real-provider smoke harness.

Refs #17
Replaces #49

Co-authored-by: kbkb628 <278338969+kbkb628@users.noreply.github.com>
@helsome
helsome merged commit 4e00e4a into main Sep 11, 2026
3 checks passed
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