You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
final independent exact-SHA review passed at d107dc6cdaffdbe1f4e069a1a6b4c3d7a16472c6
CI note
All substantive CI lanes passed except one recurring Preview 3 runtime-group-2 transport failure (fetch_with_referrer_policy, Hyper IncompleteMessage / HttpProtocolError). The targeted group-8 rerun passed, and the same group-2 signature is independently reproduced and owned by GOL-420; no GOL-418 source change is indicated.
Exact-SHA correction review and local verification completed for d107dc6cdaffdbe1f4e069a1a6b4c3d7a16472c6.
The correction establishes one AST-based TypeScript module-format decision for CommonJS preparation, including type-only import/export erasure, .mts/.cts precedence over package type, CommonJS-wrapper lexical bindings and destructuring, erased ambient declarations, import.meta, top-level await, for await, and await using with nested-function exclusions.
Regression coverage now verifies recursive CJS reexport preparation, exact transform ownership/counts, cached child reuse, true ESM avoidance, failure rollback, a type: module package containing .cts, lexical-only ESM markers, and top-level versus nested for await / await using behavior.
Independent delta review: PASS with no remaining findings.
targeted skeleton Clippy: no findings in changed internal/typescript.rs; it reproduces the same 39 documented older-branch module_loading.rs diagnostics
The tracked tree is clean, Cargo.toml_ is restored, generated skeleton artifacts are removed, and Cargo.lock is unchanged. CI is now running against this exact SHA.
Targeted rerun of test-runtime-p3 (2) completed at the reviewed SHA d107dc6cdaffdbe1f4e069a1a6b4c3d7a16472c6 and reproduced the same transport-owned failure a second time.
The failing case is again runtime::fetch::fetch_with_referrer_policy; the host reports Hyper IncompleteMessage, and Preview 3 surfaces ErrorCode::HttpProtocolError. This is the independently reproduced GOL-420 transport lifecycle flake, not a TypeScript transformation failure.
No source changes were made. Every other substantive P2/P3 runtime, node-compat, build, and test lane is green, including the targeted group 8 rerun.
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
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
Validation
d107dc6cdaffdbe1f4e069a1a6b4c3d7a16472c6CI note
All substantive CI lanes passed except one recurring Preview 3 runtime-group-2 transport failure (
fetch_with_referrer_policy, HyperIncompleteMessage/HttpProtocolError). The targeted group-8 rerun passed, and the same group-2 signature is independently reproduced and owned by GOL-420; no GOL-418 source change is indicated.