Skip to content

fix(devin): keep Codex core tools and unstick Deepseek compact - #189

Merged
caigee-cmd merged 1 commit into
mainfrom
fix/devin-codex-tools-and-compact
Sep 18, 2026
Merged

caigee-cmd merged 1 commit into
mainfrom
fix/devin-codex-tools-and-compact

Conversation

@caigee-cmd

Copy link
Copy Markdown
Owner

Summary

  • Neutralize Codex/Desktop MCP-looking Devin tools into reversible cx_tool_* aliases and restore original names on tool calls.
  • If Devin still returns MCP configuration permission_denied, fall back to core local tools (exec_command / write_stdin / view_image / request_user_input) and log each fallback stage instead of dropping all tools.
  • Ignore orphan tool_choice when Codex compact/recovery turns send a choice with no remaining tools.
  • Stop session affinity from pinning a later model onto an empty-catalog account, so Deepseek compact after a Devin turn routes to WorkBuddy.

Test plan

  • go test ./internal/providers/devin ./internal/translate ./internal/executor ./internal/accounts ./internal/api
  • Re-run Codex compact on devin/gpt-5-6-sol and deepseek-v4.1-flash
  • Confirm Devin MCP denials keep exec_command available and emit devin mcp fallback logs

Neutralize MCP-looking Devin tools, fall back to core local tools with logs, drop orphan tool_choice on compact, and escape sticky sessions when a later model cannot use the bound empty-catalog account.
@caigee-cmd
caigee-cmd merged commit 624874a into main Sep 18, 2026
6 checks passed
@caigee-cmd
caigee-cmd deleted the fix/devin-codex-tools-and-compact branch September 19, 2026 15:02
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