fix(plan): prevent repeated "Ready to implement?" prompt in plan mode

[shssoichiro]

Context

Fixes #6143

Implementation

Skip the plan followup prompt when the last user message is synthetic (programmatically injected), breaking the cycle where custom text input triggers agent response → prompt → custom text → agent response → prompt indefinitely. Real user messages still trigger the followup as expected.

Screenshots

The Plan agent now properly finishes the plan before requesting to implement.

How to Test

• Enter Plan mode and begin developing a plan interactively
• Provide responses to questions from the agent
• Expect to see a proper workflow, and see a plan presented before being asked to implement

I only tested in a limited capacity, so a maintainer may want to perform more thorough real-world verification.

Get in Touch

ExpedientFalcon on Discord

[github-actions]

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

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Clean, well-scoped fix that prevents the repeated "Ready to implement?" prompt cycle in plan mode. The approach of detecting synthetic messages via lastUserParts and skipping the followup prompt is sound and follows the existing pattern used elsewhere in the codebase (e.g., the ensureTitle function at lines 1950-1956). The optional chaining and nullish coalescing on lastUserParts correctly handles the undefined case, and the texts.length > 0 guard properly handles messages with no text parts.

Files Reviewed (1 file)

• packages/opencode/src/session/prompt.ts - 0 issues

[alex-alecu]

@shssoichiro Thank you for your input! This PR is a quick fix and not the proper solution: kilo should suggest "Ready to implement?" only when it's sure it has all the information necessary to implement the plan. I will follow up with the proper solution shortly.

[alex-alecu]

@shssoichiro fixed here #6240

[shssoichiro]

Thanks! I will close this one then.