Skip to content

feat(agent-manager): answer pending questions - #13372

Merged
marius-kilocode merged 2 commits into
mainfrom
implement-issue-13332
Aug 24, 2026
Merged

feat(agent-manager): answer pending questions#13372
marius-kilocode merged 2 commits into
mainfrom
implement-issue-13332

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

Agent Manager can identify sessions waiting for input, but it cannot resolve a pending question. Prompting such a session previously waited for the idle timeout or forced the orchestrator to stop it.

This adds an answer action that resolves a managed session's pending question through the existing question reply route. A blocked prompt now fails immediately and includes the exact session ID, question ID, question text, and option labels needed for a follow-up answer. The bridge verifies session ownership, question ownership, and answer cardinality before replying. If a caller targets the wrong session, the error identifies sessions that actually have pending questions.

Answering remains a separately hardened Agent Manager side effect, and pending permission requests remain UI-resolved rather than answerable through this action.

Closes #13332

Comment thread packages/kilo-vscode/src/agent-manager/orchestration-domain.ts Outdated
Comment thread packages/kilo-vscode/src/agent-manager/orchestration-domain.ts Outdated
@kilo-code-bot

kilo-code-bot Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/kilo-vscode/src/agent-manager/orchestration-domain.ts
  • packages/kilo-vscode/tests/unit/agent-manager-orchestration-domain.test.ts
Previous Review Summary (commit 7baefdd)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 7baefdd)

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 2
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/src/agent-manager/orchestration-domain.ts 366 blocked() treats question/permission list errors as no blocker
packages/kilo-vscode/src/agent-manager/orchestration-domain.ts 369 Failed-prompt error only names the first item of the first request
Files Reviewed (14 files)
  • .changeset/agent-manager-answer-action.md - 0 issues
  • packages/kilo-vscode/src/agent-manager/orchestration-bridge.ts - 0 issues
  • packages/kilo-vscode/src/agent-manager/orchestration-domain.ts - 2 issues
  • packages/kilo-vscode/tests/unit/agent-manager-orchestration-bridge.test.ts - 0 issues
  • packages/kilo-vscode/tests/unit/agent-manager-orchestration-domain.test.ts - 0 issues
  • packages/opencode/src/kilocode/agent-manager/protocol.ts - 0 issues
  • packages/opencode/src/kilocode/permission/agent-manager.ts - 0 issues
  • packages/opencode/src/kilocode/tool/agent-manager.ts - 0 issues
  • packages/opencode/src/kilocode/tool/agent-manager.txt - 0 issues
  • packages/opencode/test/kilocode/agent-manager-tool.test.ts - 0 issues
  • packages/opencode/test/kilocode/permission/agent-manager-prompt.test.ts - 0 issues
  • packages/sdk/js/src/v2/gen/sdk.gen.ts - 0 issues
  • packages/sdk/js/src/v2/gen/types.gen.ts - 0 issues
  • packages/sdk/openapi.json - 0 issues

Fix these issues in Kilo Cloud


Reviewed by grok-4.6 · Input: 51.1K · Output: 6.6K · Cached: 373.8K

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode merged commit 21ae999 into main Aug 24, 2026
47 of 50 checks passed
@marius-kilocode
marius-kilocode deleted the implement-issue-13332 branch August 24, 2026 14:38
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.

[FEATURE]: Let the Agent Manager tool answer a session's pending question

2 participants