Skip to content

fix(plugin): clear stale workflow state when entering a new mode#1337

Merged
JeremyDev87 merged 1 commit into
masterfrom
taskmaestro/1775315752/pane-2
Apr 4, 2026
Merged

fix(plugin): clear stale workflow state when entering a new mode#1337
JeremyDev87 merged 1 commit into
masterfrom
taskmaestro/1775315752/pane-2

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

Summary

  • on_mode_entry() now resets activeAgent, executionStrategy, councilStatus, lastHandoff to None on mode entry
  • Prevents stale workflow state from leaking across PLAN/ACT/EVAL/AUTO transitions
  • Added parametrized tests covering all 4 modes with full field seeding and reset verification

Closes #1334

Test plan

  • python3 -m pytest hooks/tests/test_hud_helpers.py -v — 49 tests passed
  • Parametrized test seeds all workflow fields with non-default values, calls on_mode_entry() for each mode, verifies complete reset

on_mode_entry() now resets activeAgent, executionStrategy, councilStatus,
and lastHandoff to None alongside the existing field resets, preventing
stale workflow state from leaking across mode transitions.
@vercel

vercel Bot commented Apr 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
codingbuddy-landing Ready Ready Preview, Comment Apr 4, 2026 3:40pm

@JeremyDev87 JeremyDev87 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Review complete — all CI checks pass

Changes:

  • on_mode_entry() now resets activeAgent, executionStrategy, councilStatus, lastHandoff on mode entry
  • Parametrized regression test across all 4 modes (PLAN/ACT/EVAL/AUTO)

Clean, minimal fix. LGTM.

@JeremyDev87 JeremyDev87 self-assigned this Apr 4, 2026
@JeremyDev87 JeremyDev87 added fix sub-issue 상위 이슈의 하위 작업 priority:medium Medium priority labels Apr 4, 2026
@JeremyDev87 JeremyDev87 merged commit 5ee7a82 into master Apr 4, 2026
29 checks passed
@JeremyDev87 JeremyDev87 deleted the taskmaestro/1775315752/pane-2 branch April 4, 2026 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix priority:medium Medium priority sub-issue 상위 이슈의 하위 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(plugin): clear stale workflow state when entering a new mode

1 participant