Bump to v8 145 #4073
Conversation
Centril
left a comment
There was a problem hiding this comment.
Approved modulo the 2 requested changes below.
|
/update-llm-benchmark |
LLM Benchmark Results (ci-quickfix)
Compared against master branch baseline Generated at: 2026-01-21T19:26:17.891Z Failure Analysis (click to expand)Benchmark Failure AnalysisGenerated from: Summary
Analysis of SpacetimeDB Benchmark FailuresIn this analysis, I've categorized the SpacetimeDB benchmark test failures by language and mode, detailing the differences between the generated code and expected outputs, and providing actionable insights for documentation changes. Rust / rustdoc_json FailuresCommon Errors: Schema Parity and Publishing IssuesFailures: t_002_scheduled_table, t_017_scheduled_columns
Specific Reducer and Table DefinitionsFailures: t_003_struct_in_table
CRUD and User Insertion ErrorsFailures: t_004_insert, t_006_delete, t_007_crud
C# / docs FailuresIndex Lookup IssuesFailures: t_008_index_lookup
This analysis provides insights into the recurring issues across the benchmark failures and identifies actionable steps for documentation improvements to prevent similar failures in the future. |
Co-authored-by: Mazdak Farrokhzad <twingoow@gmail.com> Signed-off-by: Noa <coolreader18@gmail.com>
Re-enable the Nix flake on aarch64-darwin. PR clockworklabs#3422 added the flake but bailed out on Darwin pending a fix for "could not find native static library `rusty_v8`". With v8 now on 145.0.0 (PR clockworklabs#4073) the build itself works on aarch64-darwin, so this removes the `builtins.abort` guard and fills in the real sha256 for the v145.0.0 rusty_v8 archive on aarch64-darwin. The underlying bug — v8's build.rs writing `librusty_v8.a` outside the locations cargo and crane treat as authoritative — already has a known workaround in PR clockworklabs#3921, but that fix only lives in `.github/workflows/ci.yml` and so does not protect the Nix build. This ports the equivalent guard into the flake as a `preBuild` on `commonArgs`: if the v8 build directory exists but `librusty_v8.a` is missing, clean and rebuild just the v8 crate. With current nixpkgs/crane the file does in fact survive the `buildDepsOnly` → `buildPackage` handoff on aarch64-darwin, so the guard no-ops on the happy path; it is defence-in-depth for the next time crane or nixpkgs shifts. x86_64-darwin and aarch64-linux still use placeholder hashes; users on those platforms will continue to hit the existing fail-then-paste-hash loop documented in `librusty_v8.nix`. # API and ABI breaking changes None. Build-system only, no runtime change. # Expected complexity level and risk 1. # Testing - [x] `nix flake check --no-build` passes on aarch64-darwin. - [x] `nix build .#default` produces working `spacetime`, `spacetimedb-cli`, and `spacetimedb-standalone` binaries; all three report `spacetimedb tool version 2.3.0`. - [x] `nix build .#checks.aarch64-darwin.workspace-fmt` passes. - [x] `nix develop --command rustc --version` succeeds (the command originally reported failing before this change). - [x] Confirmation from a reviewer with x86_64-linux that the change has not regressed the previously working platform. Co-authored-by: Phoebe Goldman <phoebe@clockworklabs.io>
Description of Changes
This release has a couple of patches I've been waiting on for a bit:
StackFrame::get_script_source[_mapping_url]denoland/rusty_v8#1886, which allows us to lazily parse sourcemaps.sourcemapto get Support charset indecode_data_urlgetsentry/rust-sourcemap#137, so that we no longer have to have our hack to work around that.InterruptCallbacksignature denoland/rusty_v8#1892, which allows us to re-enablerun_timeout_and_cb_every(though I haven't done that in this PR)Expected complexity level and risk
2: v8 is a very big and important dependency, but also very stable.
Testing