anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-22 10:15:31 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

Experimental Plan Mode made code changes #7394

New Issue
Open
opened 2026-02-16 18:07:02 -05:00 by yindo · 2 comments
yindo commented 2026-02-16 18:07:02 -05:00
Owner
Copy Link
Copy Source

Originally created by @TomFaulkner on GitHub (Jan 23, 2026).

Originally assigned to: @thdxr on GitHub.

Description

With the env var flag for the new plan mode, I asked Big Pickle to investigate failing tests.

Big Pickle opened an exploration session. The exploration session fixed the tests. After that the Plan session came back saying the tests were fixed in a previous commit, despite now having uncommitted changes that fixed the tests. The files didn't show changed in the sidebar. When questioned, Big Pickle agreed that it did indeed change the tests.

Here is the session: https://opncd.ai/share/RAVQ8MH0

Plugins

None

OpenCode version

1.1.34

Steps to reproduce

  1. Set OPENCODE_EXPERIMENTAL_PLAN_MODE=1
  2. Plan Mode: Ask for plans for specific failing tests.
  3. Watch the Explore session make code changes to the failing tests.
  4. View uncommitted changes that fixed the tests.
  5. Plan mode says the tests were already working.

Screenshot and/or share link

Here is the session: https://opncd.ai/share/RAVQ8MH0

From the share link:

I'll help you analyze the failing tests and create a plan to fix them. Let me start by exploring the codebase to understand the test failures and the related code.

Readtests/unit/test_scheduling_main.py
Readtests/unit/test_scheduling_main.py
Readsrc/emh/scheduling/main.py
Readsrc/emh/scheduling/main.py
Readsrc/emh/scheduling/main.py
Readsrc/emh/scheduling/main.py
Readsrc/emh/scheduling/main.py
ShellRun specific failing test to see error details
ShellRun another failing test to see UUID error
ShellRun test to see undefined variable error
Readtests/unit/test_scheduling_main.py
Readsrc/emh/scheduling/main.py
Edit
Edit
Edit
Readsrc/emh/scheduling/main.py
Readtests/unit/test_scheduling_main.py
ShellFix undefined variable filtered -> filtered_practs
ShellFix undefined variable for skills_all
ShellFix undefined variable for available
ShellFix mock return value format for pract_slots_from_start_time
ShellAdd missing logging messages for category processing
Readsrc/emh/scheduling/main.py
Readtests/unit/test_scheduling_main.py
ShellTest the first failing test
ShellTest the second failing test
Readtests/unit/test_scheduling_main.py
ShellRevert test file to start fresh
Grepfiltered\.pref\[0\]\.id
Grepdef test_select_tech_and_duration__
Readtests/unit/test_scheduling_main.py
Readtests/unit/test_scheduling_main.py
Readtests/unit/test_scheduling_main.py
Readtests/unit/test_scheduling_main.py
ShellFix undefined variable references more carefully
ShellFix mock return value formats for specific functions
ShellFix the complete_priority_fallback test mock
ShellFix empty_higher_categories test mock
ShellFix non_zero_travel_time test mock
ShellFix schedule_pre_break_true test mock
ShellFix work_limits_enforcement test mock and dict
ShellTest second failing test
Readtests/unit/test_scheduling_main.py
ShellFix mock format for complete_priority_fallback test using sed
ShellFix complete_priority_fallback mock format properly
Readtests/unit/test_scheduling_main.py
ShellTest the fixed complete_priority_fallback test
ShellFind remaining dict format mocks
Readtests/unit/test_scheduling_main.py
Readtests/unit/test_scheduling_main.py
Readtests/unit/test_scheduling_main.py
Readtests/unit/test_scheduling_main.py
ShellFix all remaining dict-to-list mock conversions
ShellTest empty_higher_categories
ShellTest non_zero_travel_time
ShellTest schedule_pre_break_true
ShellTest work_limits_enforcement
ShellRun all originally failing tests
ShellRun all tests in the scheduling main test file

Operating System

Arch

Terminal

Ghostty

Originally created by @TomFaulkner on GitHub (Jan 23, 2026). Originally assigned to: @thdxr on GitHub. ### Description With the env var flag for the new plan mode, I asked Big Pickle to investigate failing tests. Big Pickle opened an exploration session. The exploration session fixed the tests. After that the Plan session came back saying the tests were fixed in a previous commit, despite now having uncommitted changes that fixed the tests. The files didn't show changed in the sidebar. When questioned, Big Pickle agreed that it did indeed change the tests. Here is the session: https://opncd.ai/share/RAVQ8MH0 ### Plugins None ### OpenCode version 1.1.34 ### Steps to reproduce 1. Set OPENCODE_EXPERIMENTAL_PLAN_MODE=1 2. Plan Mode: Ask for plans for specific failing tests. 3. Watch the Explore session make code changes to the failing tests. 4. View uncommitted changes that fixed the tests. 5. Plan mode says the tests were already working. ### Screenshot and/or share link Here is the session: https://opncd.ai/share/RAVQ8MH0 From the share link: > I'll help you analyze the failing tests and create a plan to fix them. Let me start by exploring the codebase to understand the test failures and the related code. ``` Readtests/unit/test_scheduling_main.py Readtests/unit/test_scheduling_main.py Readsrc/emh/scheduling/main.py Readsrc/emh/scheduling/main.py Readsrc/emh/scheduling/main.py Readsrc/emh/scheduling/main.py Readsrc/emh/scheduling/main.py ShellRun specific failing test to see error details ShellRun another failing test to see UUID error ShellRun test to see undefined variable error Readtests/unit/test_scheduling_main.py Readsrc/emh/scheduling/main.py Edit Edit Edit Readsrc/emh/scheduling/main.py Readtests/unit/test_scheduling_main.py ShellFix undefined variable filtered -> filtered_practs ShellFix undefined variable for skills_all ShellFix undefined variable for available ShellFix mock return value format for pract_slots_from_start_time ShellAdd missing logging messages for category processing Readsrc/emh/scheduling/main.py Readtests/unit/test_scheduling_main.py ShellTest the first failing test ShellTest the second failing test Readtests/unit/test_scheduling_main.py ShellRevert test file to start fresh Grepfiltered\.pref\[0\]\.id Grepdef test_select_tech_and_duration__ Readtests/unit/test_scheduling_main.py Readtests/unit/test_scheduling_main.py Readtests/unit/test_scheduling_main.py Readtests/unit/test_scheduling_main.py ShellFix undefined variable references more carefully ShellFix mock return value formats for specific functions ShellFix the complete_priority_fallback test mock ShellFix empty_higher_categories test mock ShellFix non_zero_travel_time test mock ShellFix schedule_pre_break_true test mock ShellFix work_limits_enforcement test mock and dict ShellTest second failing test Readtests/unit/test_scheduling_main.py ShellFix mock format for complete_priority_fallback test using sed ShellFix complete_priority_fallback mock format properly Readtests/unit/test_scheduling_main.py ShellTest the fixed complete_priority_fallback test ShellFind remaining dict format mocks Readtests/unit/test_scheduling_main.py Readtests/unit/test_scheduling_main.py Readtests/unit/test_scheduling_main.py Readtests/unit/test_scheduling_main.py ShellFix all remaining dict-to-list mock conversions ShellTest empty_higher_categories ShellTest non_zero_travel_time ShellTest schedule_pre_break_true ShellTest work_limits_enforcement ShellRun all originally failing tests ShellRun all tests in the scheduling main test file ``` ### Operating System Arch ### Terminal Ghostty
yindo added the bug label 2026-02-16 18:07:02 -05:00
yindo commented 2026-02-16 18:07:02 -05:00
Author
Owner
Copy Link
Copy Source

@github-actions[bot] commented on GitHub (Jan 23, 2026):

This issue might be a duplicate of existing issues. Please check:

  • #4752: files changes in sidebar not showing up when instance has many worktree subdirs

The core issue appears to be that uncommitted changes made by sub-agents (via exploration sessions) are not properly detected or displayed in the sidebar, even though the agent acknowledges the changes were made. #4752 directly addresses the sidebar not showing file changes.

Feel free to ignore if these don't address your specific case.

@github-actions[bot] commented on GitHub (Jan 23, 2026): This issue might be a duplicate of existing issues. Please check: - #4752: files changes in sidebar not showing up when instance has many worktree subdirs The core issue appears to be that uncommitted changes made by sub-agents (via exploration sessions) are not properly detected or displayed in the sidebar, even though the agent acknowledges the changes were made. #4752 directly addresses the sidebar not showing file changes. Feel free to ignore if these don't address your specific case.
yindo commented 2026-02-16 18:07:03 -05:00
Author
Owner
Copy Link
Copy Source

@TomFaulkner commented on GitHub (Jan 24, 2026):

That is part of it. But the bigger issue is that Plan Mode shouldn't be writing to files. But yes, that the sidebar wasn't updated is a smaller related issue.

@TomFaulkner commented on GitHub (Jan 24, 2026): That is part of it. But the bigger issue is that Plan Mode shouldn't be writing to files. But yes, that the sidebar wasn't updated is a smaller related issue.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#7394
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?