Skip to content

[FEATURE]: Land two-stage TUI streaming freeze fix for #36043 #48430

Closed as not planned

Description

@dcerisano
  • I have verified this feature I'm about to request hasn't been suggested before.

Description

Tracking issue for the fix plan — the freeze itself is reported in #36043,
this is not a re-report.

Diagnosis (from #41472): streaming pays O(n²) at two client-side stages.
Each PR applies and merges independently, in either order; together they
fully close the freeze:

Related: #41472 (root-cause analysis; tui-delta-coalesce supersedes it).

Done = both merged, at which point I'll close this issue.

Activity

  1. dcerisano commented on Oct 15, 2026

    @dcerisano
    Author

    V2 status update: the markdown-live-relex half is ported and open as #55857 (markdown-incremental-projection → dev). The V1 tui-delta-coalesce counterpart (#48431) remains V1-only and is not part of this PR.

  2. opencodejr commented on Oct 28, 2026

    @opencodejr
    Contributor

    Thanks for the thorough diagnosis and for tracking this so carefully.

    This issue tracks fixes for OpenCode v1: #36043 was reported on 1.17.15, and #48431 and #48432 were both written for v1. Both PRs were closed unmerged because v1 no longer takes PRs, and v1 only gets fixes for critical bugs (data loss, security, crashes or breakage with no workaround). A temporary stall while a model streams doesn't meet that bar, so this is unlikely to land in v1, and the "both merged" condition here can't be met.

    The v2 side is already moving:

    If you can still reproduce the TUI freeze on a current 2.x release (opencode upgrade --major, see https://anoma.ly/notes/opencode-2-0), for example from per-delta store updates in the v2 TUI, please open a new v2 issue with the version, terminal, model and steps. It would go through the normal flow. Also reply here if you think the v1 freeze counts as critical.

  3. opencodejr commented on Oct 29, 2026

    @opencodejr
    Contributor

    Closing this as not planned. It tracks landing #48431 and #48432 in OpenCode v1. Both were closed unmerged because v1 no longer takes PRs and only gets critical fixes. We waited a day in case you could show the freeze is critical or also happens on v2, and there's been no reply.

    The v2 side is tracked in #56202, and #57016 (incremental markdown parsing via @opencode/patchmark) has been merged. If you can reproduce the TUI streaming freeze on a current 2.x release, please open a new issue with the version, terminal, model and steps. Thanks again for the detailed diagnosis.

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

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions