Skip to content

fix(plugin): correct AUTO mode default agent id to auto-mode (#1226)#1227

Merged
JeremyDev87 merged 1 commit into
masterfrom
taskmaestro/1775228415/pane-3
Apr 3, 2026
Merged

fix(plugin): correct AUTO mode default agent id to auto-mode (#1226)#1227
JeremyDev87 merged 1 commit into
masterfrom
taskmaestro/1775228415/pane-3

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

Summary

  • mode_engine.py DEFAULT_AGENTS의 AUTO 모드 agent id를 auto-mode-agent에서 auto-mode로 수정
  • test_mode_engine.py의 assertion도 동일하게 수정

Problem

auto-mode-agent는 canonical id(auto-mode)와 불일치하여 _load_agent_details()가 None을 반환, AUTO 모드에서 agent enrichment이 누락됨.

Closes #1226

Test plan

  • python3 -m pytest hooks/lib/test_mode_engine.py -v — 37 passed
  • yarn workspace codingbuddy typecheck — pass
  • yarn workspace codingbuddy test — 5822 passed

DEFAULT_AGENTS used 'auto-mode-agent' for AUTO mode but the canonical
id is 'auto-mode' (matching keyword.types.ts and auto-mode.json).
This caused _load_agent_details to return None, skipping enrichment.
@JeremyDev87 JeremyDev87 added the fix label Apr 3, 2026
@vercel

vercel Bot commented Apr 3, 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 3, 2026 3:07pm

@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 comments addressed

Code Review:

  • mode_engine.py: auto-mode-agentauto-mode — matches canonical id in keyword.types.ts and agents/auto-mode.json
  • test_mode_engine.py: assertion updated to match
  • Minimal, focused change — no side effects
  • CI: 28/28 checks pass

LGTM

@JeremyDev87 JeremyDev87 self-assigned this Apr 3, 2026
@JeremyDev87 JeremyDev87 merged commit 114be3d into master Apr 3, 2026
28 checks passed
@JeremyDev87 JeremyDev87 deleted the taskmaestro/1775228415/pane-3 branch April 3, 2026 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(plugin): correct AUTO mode default agent id to auto-mode

1 participant