Skip to content

feat(platform): include remote file error details#21413

Draft
coyaSONG wants to merge 1 commit into
PipedreamHQ:masterfrom
coyaSONG:agent/file-stream-error-details
Draft

feat(platform): include remote file error details#21413
coyaSONG wants to merge 1 commit into
PipedreamHQ:masterfrom
coyaSONG:agent/file-stream-error-details

Conversation

@coyaSONG

Copy link
Copy Markdown
Contributor

Summary

When getFileStream() or getFileStreamAndMetadata() receives a non-successful remote response, preserve the HTTP status and add a whitespace-normalized, size-bounded response body to the thrown error.

This is a diagnostic improvement related to #21411. It does not claim to fix the intermittent File Stash / S3 400 itself. It makes service error codes such as SignatureDoesNotMatch, InvalidToken, or NoSuchKey visible so the underlying failure can be distinguished.

The platform package is bumped from 3.4.0 to 3.4.1, and generated output is rebuilt.

Validation

  • corepack pnpm --filter @pipedream/platform run build
  • corepack pnpm exec eslint platform/lib/file-stream.ts platform/__tests__/file-stream.js
  • focused file-stream suite: 12 passed
  • deterministic platform suites: 51 passed, 8 skipped
  • git diff --check

The default Jest launch is currently blocked before test execution by the repository lock resolving Babel 8 with Jest 29 (loadPartialConfig now requires a callback). Running the CommonJS suites with transforms disabled passed. The only full-suite failure under that workaround is the unchanged live-network Axios test timing out after five seconds.

Checklist

Versioning

  • @pipedream/platform version bumped from 3.4.0 to 3.4.1

New app

  • Not applicable; this PR does not add or update an app integration

CodeRabbit review

  • Pending automated review

@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
pipedream-docs-redirect-do-not-edit Ignored Ignored Jul 16, 2026 9:12pm

Request Review

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

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: 2cbf2bf1-5d34-4806-9820-b418f69a2247

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.

@adolfo-pd adolfo-pd added the User submitted Submitted by a user label Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

User submitted Submitted by a user

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants