Conversation
Expose actual failures so they can be investigated and fixed. https://claude.ai/code/session_017rGQUxVUsNpmQVSEpEA2BJ
http-request-options: The component model codegen was skipping constructor alias exports for all resources except Fields and Response. This caused RequestOptions::new to be missing from the wasi instance, producing a validation error. Fix by including constructors in the dynamic resource function alias loop (filtered by used_wasi_functions). http_client_send: The test was writing trailers before Client::send and missing scheme/authority/path, causing async timeouts. Reorder to match the working http-client-send-simple pattern. https://claude.ai/code/session_017rGQUxVUsNpmQVSEpEA2BJ
…pattern Differentiate from http-client-send-simple by testing a more realistic pattern: send to a specific upstream path, verify the status code, then forward the response. https://claude.ai/code/session_017rGQUxVUsNpmQVSEpEA2BJ
Tests the full reverse-proxy pattern: send outgoing request, read the upstream response body via Response::consume_body + Stream::read, then build a new response with a prefixed body. https://claude.ai/code/session_017rGQUxVUsNpmQVSEpEA2BJ
External HTTP response bodies should be validated as UTF-8. https://claude.ai/code/session_017rGQUxVUsNpmQVSEpEA2BJ
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
http-request-options: Component model codegen was only aliasing Fields/Response constructors, skipping other resource constructors likeRequestOptions::new. Fixed by including constructors in the dynamic resource function alias loop filtered byused_wasi_functions.http_client_send: Renamed tohttp-client-send-proxywith a differentiated reverse-proxy pattern (specific upstream path, status verification, response forwarding). The original test had trailers written beforeClient::sendand missing URI fields, causing async timeouts.http-client-send-body-read: New test exercisingResponse::consume_body+Stream::readto read upstream body and build a prefixed response.Test plan
cargo test --test e2e http_request_options— all opt levels passcargo test --test e2e http_client_send_proxy— all opt levels passcargo test --test e2e http_client_send_body_read— all opt levels passcargo test --test e2e http_client_send_simple— all opt levels pass (unchanged)mise run on-task-done— 1269 passed, 0 failedhttps://claude.ai/code/session_017rGQUxVUsNpmQVSEpEA2BJ