Skip to content

fix(api): inline _conversation_to_response after conversations module removal#46

Merged
charlie83Gs merged 1 commit intomainfrom
fix/missing-conversations-import
Mar 27, 2026
Merged

fix(api): inline _conversation_to_response after conversations module removal#46
charlie83Gs merged 1 commit intomainfrom
fix/missing-conversations-import

Conversation

@charlie83Gs
Copy link
Copy Markdown
Contributor

Summary

  • Inlines _conversation_to_response and _message_to_response in research.py after conversations.py was removed in PR feat: replace orchestrator/query with synthesis agents #39
  • Fixes export.py to import from research instead of deleted conversations module
  • Fixes 500 error on POST /api/v1/research/bottom-up/prepare

Test plan

  • All 72 API tests pass locally
  • CI checks pass
  • POST /api/v1/research/bottom-up/prepare returns 200 in dev cluster after deploy

🤖 Generated with Claude Code

… removal

The conversations module was removed in PR #39 but research.py and
export.py still imported _conversation_to_response from it, causing a
500 error on POST /api/v1/research/bottom-up/prepare.

Inline the helper functions in research.py and update export.py to
import from research instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@charlie83Gs charlie83Gs merged commit 207fcdd into main Mar 27, 2026
4 checks passed
@charlie83Gs charlie83Gs deleted the fix/missing-conversations-import branch March 27, 2026 21:43
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