Skip to content

feat(acp): emit session plan from todo updates - #40746

Closed
Petr28 wants to merge 1 commit into
anomalyco:devfrom
Petr28:acp-plan
Closed

Petr28 wants to merge 1 commit into
anomalyco:devfrom
Petr28:acp-plan

Conversation

@Petr28

@Petr28 Petr28 commented Aug 5, 2026

Copy link
Copy Markdown

ACP clients never show the session plan: the adapter doesn't emit sessionUpdate: "plan". todo.updated already carries the data on the stream the adapter consumes, so this maps it through (Todo.Info → PlanEntry is 1:1). Statuses outside the ACP set (e.g. cancelled) collapse to completed.

Fixes #40745

Verified: bun test packages/opencode/test/acp/event.test.ts. Also confirmed a standalone ACP agent emitting the same plan updates renders a live plan panel in Zed.

@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label Aug 5, 2026
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

I found a potentially related PR:

Related PR:

All other results were either the current PR (40746) itself or unrelated changes.

@Petr28

Petr28 commented Aug 5, 2026

Copy link
Copy Markdown
Author

Closing as a duplicate of #31834 — it implements the same todo.updated → plan mapping and is further along (fingerprint dedup, richer tests). I'll help move #31834 forward instead of fragmenting the effort.

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

Labels

needs:compliance This means the issue will auto-close after 2 hours.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ACP clients don't show the session plan — plan sessionUpdate is never emitted

1 participant