Skip to content

fix: queue concurrent subagent questions - #36916

Closed
lucas-gaitzsch wants to merge 1 commit into
anomalyco:devfrom
lucas-gaitzsch:question-request-queue
Closed

lucas-gaitzsch wants to merge 1 commit into
anomalyco:devfrom
lucas-gaitzsch:question-request-queue

Conversation

@lucas-gaitzsch

Copy link
Copy Markdown

Issue for this PR

Closes #36915

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Collects pending questions across the full root session tree, orders them by request ID, and keeps the active request selected until it resolves.

The TUI now remounts prompt state per request, preserves interrupted drafts, blocks duplicate submissions, and routes replies through the owning session location. Desktop/web uses the same sticky queue selection and owner-location routing.

How did you verify your code works?

  • Added queue tests for sibling and nested subagents, FIFO ordering, sticky selection, and tree isolation.
  • packages/tui: 194 passed, 1 skipped; typecheck passed.
  • packages/app: targeted queue tests and typecheck passed.
  • The full app unit suite has 612 passing tests and one unrelated existing i18n parity failure for three missing Arabic keys.

Screenshots / recordings

Not applicable; this changes request selection and routing without changing the UI.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@github-actions

Copy link
Copy Markdown
Contributor

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

Based on my search, I found one potentially related PR that might be worth reviewing in context:

Related PR:

However, this appears to be addressing a different but complementary aspect (preserving session work) rather than a duplicate of the current PR's focus on queueing and routing concurrent subagent questions.

The current PR (#36916) is the primary PR implementing the queue management for concurrent subagent questions and does not appear to have exact duplicates in the open PR list.

@github-actions

Copy link
Copy Markdown
Contributor

Automated PR Cleanup

Thank you for contributing to opencode.

Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions.

This PR was closed because it matched the following cleanup criteria:

  • The PR was created more than 1 month ago
  • The PR had fewer than 2 positive reactions
  • Positive reactions are counted as thumbs-up, heart, celebration, or rocket reactions on the PR

PRs created within the last month are not affected by this cleanup.

If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate.

Thanks again for taking the time to contribute.

@lucas-gaitzsch

Copy link
Copy Markdown
Author

Please reopen the PR. It ist not fixed yet and and still relevant for me.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Concurrent subagent questions replace the active prompt

1 participant