fix: 修复 CI 高失败率：fmt/audit/tests/docker - #92
Merged
Merged
Conversation
Co-authored-by: Henry Zhang <hello@zhanghe.dev>
|
Co-authored-by: Henry Zhang <hello@zhanghe.dev>
Co-authored-by: Henry Zhang <hello@zhanghe.dev>
CI run 35316789035 的 9 个 bundler_integration_tests 失败源于按行改写
无法处理多行 import/export、export { } from 和写盘前缺失导出校验。
Docker run 35302823643 在 cargo fetch 时缺少 workspace crates/。
Co-authored-by: Henry Zhang <hello@zhanghe.dev>
cursor
Bot
changed the title
http_fetch_tests 写入 X-Amber-Trace，却仍 get/期望 x-amberjs-trace。 response.type 在 keepalive 路径为 Fetch 的 basic，一并接受。 Co-authored-by: Henry Zhang <hello@zhanghe.dev>
祖先探测仍把沙箱拒绝当作无 package.json，避免夹具启动失败； require 目标包必须把 broker deny 暴露给 JS，不能伪装成找不到模块。 Co-authored-by: Henry Zhang <hello@zhanghe.dev>
x86_64-apple-darwin 已在 macos-latest 交叉编译；GetCurrentProcess 在 process.rs。 Co-authored-by: Henry Zhang <hello@zhanghe.dev>
JS 实现先于 Rust setup 安装后会短路完整实现。补上 getAll/entries/sort、 application/x-www-form-urlencoded 编解码，以及 URL.search 联动。 Co-authored-by: Henry Zhang <hello@zhanghe.dev>
Co-authored-by: Henry Zhang <hello@zhanghe.dev>
WebAssembly.compile 返回的 Promise 在 execute_code 的微任务循环里不会 settle， 导致 wasm mmap 零拷贝测试失败。 Co-authored-by: Henry Zhang <hello@zhanghe.dev>
streams_fast.js 覆盖 Rust Streams 后，队列用 shift 丢掉 _readIndex， pipeThrough 只返回 readable，pipeTo 忽略 preventClose/signal。 Co-authored-by: Henry Zhang <hello@zhanghe.dev>
输入已是 hello amberjs，toUpperCase 得到 HELLO AMBERJS， 期望仍停留在重命名前的 HELLO AMBER。 Co-authored-by: Henry Zhang <hello@zhanghe.dev>
cargo check --features benchmarks 仍用旧的 Context::new(&mut scope)。 macOS Library tests 在 test_concurrent_multi_thread_checkout 对同一 CoW snapshot 并发 Isolate::new 时 SIGSEGV。 Co-authored-by: Henry Zhang <hello@zhanghe.dev>
Co-authored-by: Henry Zhang <hello@zhanghe.dev>
CI cargo test --verbose 并行下该用例实测 39ms，原 25ms debug 预算过紧。先 thread-local prewarm + warmup execute，再测。 Co-authored-by: Henry Zhang <hello@zhanghe.dev>
Co-authored-by: Henry Zhang <hello@zhanghe.dev>
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.
变更范围
修复 Amberjs GitHub Actions 高失败率，覆盖 Format / cargo-audit / clippy /
cargo test/ macOS lib tests / Docker。-D warnings。X-Amber-Trace/ package.json deny / release-assets / URLSearchParams / WASM mmap / Streams_readIndex+pipeTo/ workerHELLO AMBERJS。crates/+extensions/。PR 上只 build+smoke；GHCR push 仅 main/tag/workflow_dispatch。javascript_core.rs对齐 v8 152Context::new。prewarm_one串行化并发Isolate::new，消除test_concurrent_multi_thread_checkoutSIGSEGV。test_prewarmed_execution_latency_benchmark先 thread-local prewarm + warmup 再测；debug 上限 150ms。运行过的测试/检查
本机：
cargo fmt --all -- --checkcargo clippy --all-targets -- -D warningscargo audit（无 RUSTSEC-2026-0285）cargo test与cargo test --lib --tests -- --test-threads=1cargo check --features benchmarks/observabilityCI（
ffc5cc8b）6/6 绿：未验证项
相关链接