Skip to content

Conversation

@Olyray
Copy link
Contributor

@Olyray Olyray commented Oct 23, 2025

Description

In #5561 there was a problem with the Replicate integration for Claude. I fixed the integration issue.

After fixing the specific claude integration issues, I noticed that it still wasn't working because there was a deeper issue with the Replicate integration. The API wasn't accepting the signal parameter. So I also fixed that issue and the integration is working fine.

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

image image

Summary by cubic

Fixed the Replicate integration for Claude by removing the unsupported signal parameter, adding chat streaming, and mapping Claude 4 Sonnet correctly. This enables using Claude via Replicate in the app and improves error handling.

  • Bug Fixes

    • Removed the signal parameter from Replicate API calls to match the API.
    • Corrected prompt handling for chat (system and user/assistant roles).
    • Added clear errors for invalid API keys and missing models.
    • Fixed docs tip to link to OpenRouter models.
  • New Features

    • Streamed chat responses from Replicate with proper prompt formatting.
    • Enabled Claude 4 Sonnet selection under the Replicate provider in the model picker.

@Olyray Olyray requested a review from a team as a code owner October 23, 2025 13:59
@Olyray Olyray requested review from sestinj and removed request for a team October 23, 2025 13:59
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Oct 23, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

@Olyray Olyray force-pushed the claude-replicate-integration branch from 1a2c6b9 to 1ad456d Compare October 23, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant