Skip to content

fix: retry transient pipe open failures after use(filepath)#711

Draft
Leela8256 wants to merge 1 commit intorocketride-org:developfrom
Leela8256:fix/RR-460-python-send-after-use-filepath
Draft

fix: retry transient pipe open failures after use(filepath)#711
Leela8256 wants to merge 1 commit intorocketride-org:developfrom
Leela8256:fix/RR-460-python-send-after-use-filepath

Conversation

@Leela8256
Copy link
Copy Markdown

Handle transient startup races in the Python SDK by retrying DataPipe.open() on short-lived connection-refused errors after use(filepath=...), and add a regression test for the documented use(filepath) -> send(...) flow.

Summary

  • Add bounded retry logic in DataPipe.open() for transient connection-refused errors during task data-port warm-up
  • Preserve existing behavior by failing immediately for non-transient pipe-open errors
  • Add Python SDK regression test covering use(filepath=...) followed by send(...) (Issue client.send() doesn't trigger the engine and throws an error #460 docs flow)

Affected Files

  • packages/client-python/src/rocketride/mixins/data.py
  • packages/client-python/tests/RocketRideClient_test.py

Type

fix

Testing

  • Tests added or updated
  • Tested locally
  • ./builder test passes

Checklist

  • Commit messages follow conventional commits
  • No secrets or credentials included
  • Wiki updated (if applicable)
  • Breaking changes documented (if applicable)

Linked Issue

Fixes #460

co-author: cursor

Handle short data-port startup races in Python SDK send() by retrying transient connection-refused errors in DataPipe.open, and add a regression test that validates the documented use(filepath)->send flow.

Made-with: Cursor
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 27, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 43e2b707-1a03-40f6-9c38-7c606c3ba87b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the module:client-python Python SDK and MCP client label Apr 27, 2026
@github-actions
Copy link
Copy Markdown

No description provided.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:client-python Python SDK and MCP client

Projects

None yet

Development

Successfully merging this pull request may close these issues.

client.send() doesn't trigger the engine and throws an error

1 participant