Skip to content

fix: handle list responses from dedup LLM provider#122

Merged
jeff-atriumn merged 1 commit intomainfrom
fix/dedup-list-response
Mar 13, 2026
Merged

fix: handle list responses from dedup LLM provider#122
jeff-atriumn merged 1 commit intomainfrom
fix/dedup-list-response

Conversation

@jeff-atriumn
Copy link
Member

Summary

  • Same list-response bug as validate.py — _call_provider can return a list instead of a dict from some providers
  • Extract first dict element from list responses, fall back gracefully for non-dict responses
  • Discovered during pipeline benchmark testing

Test plan

  • 22 existing dedup tests pass
  • Verified against real benchmark data (3 chunked runs through full pipeline)

🤖 Generated with Claude Code

Same issue as validate.py — _call_provider can return a list instead
of a dict. Extract first element from list, fall back gracefully.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jeff-atriumn jeff-atriumn merged commit 7dd7f63 into main Mar 13, 2026
5 checks passed
@jeff-atriumn jeff-atriumn deleted the fix/dedup-list-response branch March 13, 2026 13:21
jeff-atriumn added a commit that referenced this pull request Mar 13, 2026
Same issue as validate.py — _call_provider can return a list instead
of a dict. Extract first element from list, fall back gracefully.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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