Skip to content

acp: enrich streaming updates for ide clients#41426

Closed
mbelinky wants to merge 3 commits intomariano/acp-v2-pr2-session-uxfrom
mariano/acp-v2-pr3-streaming
Closed

acp: enrich streaming updates for ide clients#41426
mbelinky wants to merge 3 commits intomariano/acp-v2-pr2-session-uxfrom
mariano/acp-v2-pr3-streaming

Conversation

@mbelinky
Copy link
Copy Markdown
Contributor

@mbelinky mbelinky commented Mar 9, 2026

Summary

This enriches ACP bridge streaming so IDE clients get more useful follow-along information during tool activity.

Stack

What Changed

  • map richer tool updates into ACP events
  • preserve partial tool output in-stream
  • attach best-effort file locations when gateway payloads expose them
  • document the narrower but improved editor-streaming contract

Why It Matters

This improves the editor experience without changing the bridge architecture: clients can follow tool execution and touched files more naturally.

Testing

  • pnpm exec vitest run src/acp/translator.session-rate-limit.test.ts --maxWorkers=1

@openclaw-barnacle openclaw-barnacle bot added docs Improvements or additions to documentation size: XL maintainer Maintainer-authored PR labels Mar 9, 2026
@mbelinky mbelinky force-pushed the mariano/acp-v2-pr2-session-ux branch from 3300fbd to fcabdf7 Compare March 9, 2026 21:14
@mbelinky mbelinky deleted the branch mariano/acp-v2-pr2-session-ux March 9, 2026 21:17
@mbelinky mbelinky closed this Mar 9, 2026
@mbelinky mbelinky self-assigned this Mar 9, 2026
@mbelinky
Copy link
Copy Markdown
Contributor Author

mbelinky commented Mar 9, 2026

Superseded by #41442, which rebased the same ACP streaming work onto after the stacked base for this PR disappeared when #41425 merged.\n\nLanded in #41442: https://github.com/openclaw/openclaw/pull/41442\nMerge commit: 8e3f3bc3cf4744e38442d177573f706b78fbc0c5\n\nWhat landed: richer ACP / events with best-effort text content and file-location hints so IDE clients can follow bridge tool activity more naturally.\n\nThis older PR is being left closed as superseded by the clean rebased replacement. If anything in the streaming behavior still looks wrong after the landed fix, reopen or file a follow-up with repro details.

@mbelinky
Copy link
Copy Markdown
Contributor Author

mbelinky commented Mar 9, 2026

Superseded by #41442, which rebased the same ACP streaming work onto main after the stacked base for this PR disappeared when #41425 merged.

Landed in #41442: #41442
Merge commit: 8e3f3bc

What landed: richer ACP tool_call / tool_call_update events with best-effort text content and file-location hints so IDE clients can follow bridge tool activity more naturally.

This older PR is being left closed as superseded by the clean rebased replacement. If anything in the streaming behavior still looks wrong after the landed fix, reopen or file a follow-up with repro details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation maintainer Maintainer-authored PR size: XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant