Skip to content

Handle plain-text Claude OAuth responses#408

Merged
edwin-zvs merged 1 commit into
mainfrom
smith-claude-oauth-text-fallback
Jun 16, 2026
Merged

Handle plain-text Claude OAuth responses#408
edwin-zvs merged 1 commit into
mainfrom
smith-claude-oauth-text-fallback

Conversation

@edwin-zvs

Copy link
Copy Markdown
Contributor

Summary

  • handle plain-text claude-oauth CLI results as normal final assistant text
  • keep empty results and malformed structured JSON as provider errors
  • add a regression test for plain-text Claude output

Verification

  • cargo test -p construct-adapter-smith
  • cargo build

Notes

This addresses a failure seen in the Greeting To Anthropic Model Smith session after /model claude-oauth:opus, where Claude Code returned a non-JSON text result and Smith surfaced parse claude-oauth structured turn: expected value at line 1 column 1.

Relevant changed binary: construct-adapter-smith.

Debug binaries built in this worktree:

  • /Users/moon/agentd/.claude/worktrees/smith-claude-oauth-text-fallback/target/debug/construct
  • /Users/moon/agentd/.claude/worktrees/smith-claude-oauth-text-fallback/target/debug/construct-mcp
  • /Users/moon/agentd/.claude/worktrees/smith-claude-oauth-text-fallback/target/debug/construct-adapter-antigravity
  • /Users/moon/agentd/.claude/worktrees/smith-claude-oauth-text-fallback/target/debug/construct-adapter-claude
  • /Users/moon/agentd/.claude/worktrees/smith-claude-oauth-text-fallback/target/debug/construct-adapter-codex
  • /Users/moon/agentd/.claude/worktrees/smith-claude-oauth-text-fallback/target/debug/construct-adapter-shell
  • /Users/moon/agentd/.claude/worktrees/smith-claude-oauth-text-fallback/target/debug/construct-adapter-smith

target/debug/constructd was not produced by the current workspace build.

@edwin-zvs
edwin-zvs merged commit ae6080c into main Jun 16, 2026
1 check passed
@edwin-zvs
edwin-zvs deleted the smith-claude-oauth-text-fallback branch June 16, 2026 15:07
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