Skip to content

Conversation

thebrianchen
Copy link
Contributor

@thebrianchen thebrianchen commented Aug 26, 2025

Add viem's http options into alchemyTransport for easier debugging

Pull Request Checklist


PR-Codex overview

This PR focuses on enhancing the alchemyTransport functionality by adding HTTP debugging options and refining the configuration interface. It updates documentation and tests to reflect these changes, improving usability for developers.

Detailed summary

  • Added HTTP debugging options: onFetchRequest and onFetchResponse.
  • Introduced timeout parameter in alchemyTransport.
  • Updated AlchemyTransportConfig to extend HttpTransportConfig.
  • Revised documentation to clarify configuration options.
  • Enhanced test coverage for the new debugging features.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

github-actions bot commented Aug 26, 2025

🌿 Documentation Preview

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview Aug 27, 2025, 5:49 PM

@github-actions github-actions bot temporarily deployed to docs-preview August 26, 2025 15:39 Inactive
Copy link

vercel bot commented Aug 26, 2025

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

Project Deployment Preview Comments Updated (UTC)
aa-sdk-ui-demo Ready Ready Preview Comment Aug 27, 2025 5:53pm

@github-actions github-actions bot temporarily deployed to docs-preview August 26, 2025 17:28 Inactive
fetchOptions?: NoUndefined<HttpTransportConfig["fetchOptions"]>;
/** HTTP transport options for debugging and configuration */
httpOptions?: Pick<
HttpTransportConfig,
Copy link
Contributor

Choose a reason for hiding this comment

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

aren't the retry & fetch options part of this HttpTransportConfig too in viem? do you think it's weird that we are picking some of these to be here, but others to be top level? curious to hear your thoughts.

@github-actions github-actions bot temporarily deployed to docs-preview August 27, 2025 17:48 Inactive
@thebrianchen thebrianchen merged commit 8a0dbec into moldy/v5-base Aug 27, 2025
6 of 7 checks passed
@thebrianchen thebrianchen deleted the bc/http-options branch August 27, 2025 18:58
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.

2 participants