Skip to content

fix(devin): catalog-driven model UIDs and tool-result images - #227

Merged
caigee-cmd merged 1 commit into
mainfrom
fix/devin-catalog-drift
Sep 24, 2026
Merged

caigee-cmd merged 1 commit into
mainfrom
fix/devin-catalog-drift

Conversation

@caigee-cmd

Copy link
Copy Markdown
Owner

Summary

Track CLIProxyAPI's Devin registry update (v7.3.16 / commit 65459ca) and its tool-result image fix (v7.3.7):

  • mapping.go: drop the hardcoded swe-1-7 / swe-1-6 / glm-5-2 suffix tables. The upstream registry drifted — swe-1-7 now only accepts medium (with a separate swe-1-7-lightning id), glm-5-2 levels are [none, max], and swe-1-6 has no thinking. Suffixed chat_model_uids are built purely from catalog thinking.levels, clamped per project rules.
  • Default effort: never pick none implicitly — a [none, max] catalog defaults to max; none is only sent when explicitly requested.
  • payload.go: tool role messages go through splitContent, so images embedded in tool results reach the upstream prompt (prompt.Images) instead of being flattened away.
  • Fixture: refreshed to current upstream — max_completion_tokens 64000, swe-1-7-lightning, glm-5-2 [none,max], image modalities on glm-5-3-flash / swe-1-6.

Test plan

  • go test ./internal/providers/devin/... — new assertions cover glm-5-2 (none/max), swe-1-7, swe-1-7-lightning, suffixed passthrough, and tool-result image passthrough
  • go test ./... — all green
  • git diff --check clean
  • Live-account acceptance against a real Devin account (experimental provider; not run here)

…images

Drop hardcoded swe-1-7/swe-1-6/glm-5-2 suffix tables that drifted from the
upstream registry; suffixed UIDs are built from catalog thinking levels.
Never pick "none" as an implicit default effort. Refresh the models
fixture (64k completions, swe-1-7-lightning, glm-5-2 [none,max], image
modalities). Tool results now carry images through to the Devin prompt.
@caigee-cmd
caigee-cmd merged commit 8dfed05 into main Sep 24, 2026
6 checks passed
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