Skip to content

fix(todo-continuation): send runtime agent display name#3357

Open
Gujiassh wants to merge 1 commit intocode-yeongyu:devfrom
Gujiassh:fix/todo-continuation-agent-display
Open

fix(todo-continuation): send runtime agent display name#3357
Gujiassh wants to merge 1 commit intocode-yeongyu:devfrom
Gujiassh:fix/todo-continuation-agent-display

Conversation

@Gujiassh
Copy link
Copy Markdown
Contributor

@Gujiassh Gujiassh commented Apr 11, 2026

Summary

  • ensure todo-continuation-enforcer prefers the runtime display name when only a config key is available so injected agent names match what the server advertises
  • derive the runtime display name via the shared agent display-name helpers when resolveRegisteredAgentName cannot find a prettier alias
  • add a regression test covering the fallback path

Testing

  • bun test src/hooks/todo-continuation-enforcer/continuation-injection.test.ts

Summary by cubic

Send the runtime agent display name during todo continuation injection so the injected agent matches the server-advertised name. Falls back from config key to the runtime display name when no registered alias is found.

  • Bug Fixes
    • Prefer runtime display name via getAgentRuntimeName when resolveRegisteredAgentName can’t find an alias; keep alias when it differs from input.
    • Added a regression test covering the fallback path in continuation-injection.test.ts.

Written for commit 326e49f. Summary will update on new commits.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Auto-approved: Fixes agent name mismatch by preferring runtime display names to align with server expectations. Includes regression tests and the logic correctly handles fallback cases.

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.

1 participant