Skip to content

fix(core): make follow-up execution aware of the saved plan file#8479

Merged
kirillk merged 1 commit into
Kilo-Org:mainfrom
shssoichiro:issue-8478
Apr 7, 2026
Merged

fix(core): make follow-up execution aware of the saved plan file#8479
kirillk merged 1 commit into
Kilo-Org:mainfrom
shssoichiro:issue-8478

Conversation

@shssoichiro

Copy link
Copy Markdown
Contributor

Context

Users launching a follow-up implementation session now get an explicit plan-file path and instruction to read it first, so execution stays aligned with the canonical plan even when copied plan text drifts.

Fixes #8478

How to Test

  • Make a plan in Plan mode
  • Implement via "Start new session" prompt
  • Verify the implementation session receives an instruction to load the plan file, and loads it

Get in Touch

ExpedientFalcon on Discord

Comment thread packages/opencode/src/kilocode/plan-followup.ts Outdated
@kilo-code-bot

kilo-code-bot Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

None.

Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File Line Issue
packages/opencode/src/kilocode/plan-followup.ts 184 resolvePlan() still reads Session.plan(session) under the caller's instance, so if the saved plan has to be reloaded from disk after switching worktrees it can open the wrong file or return an empty plan.
Files Reviewed (2 files)
  • packages/opencode/src/kilocode/plan-followup.ts - 1 issue
  • packages/opencode/test/kilocode/plan-followup.test.ts - 0 issues

Reviewed by gpt-5.4-20260305 · 306,008 tokens

Users launching a follow-up implementation session now get an explicit plan-file path and instruction to read it first, so execution stays aligned with the canonical plan even when copied plan text drifts.
@kirillk kirillk merged commit d0ca6ee into Kilo-Org:main Apr 7, 2026
13 checks passed
slamj1 pushed a commit to slamj1/kilocode that referenced this pull request May 16, 2026
jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 18, 2026
jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 18, 2026
fix(core): make follow-up execution aware of the saved plan file
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.

Plan mode "Start new session" handoff no longer specifies plan file name

2 participants