fix: pin Bazel 8.4.0 to unblock CI for all dependabot PRs#82
Conversation
Bazelisk reads USE_BAZEL_VERSION, not BAZEL_VERSION. Without a .bazelversion file and with the wrong env var name, Bazelisk was downloading Bazel 9.0.0, which is incompatible with rules_go@0.57.0 (Facts.clear() not supported) and aspect_bazel_lib (@local_config_platform resolution failure). This caused 5 CI jobs to fail on every PR. - Create .bazelversion file with 8.4.0 (primary fix) - Rename BAZEL_VERSION to USE_BAZEL_VERSION in ci.yml and performance.yml - Update stale version pins: rules_cc 0.2.4→0.2.14, bazel_skylib 1.8.1→1.8.2 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The performance-benchmark and regression-detection jobs use github-script to post comments on PRs, which requires pull-requests: write permission. Without it, the step fails with 403 "Resource not accessible by integration". Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
🚀 Performance Benchmark ResultsTinyGo Performance Benchmarks
TinyGo AOT Performance Benchmarks
AOT vs Regular Comparison
✅ TinyGo AOT component benchmarks completed 📊 Performance Comparison (ubuntu-latest)Results from ubuntu-latest |
📈 Performance Regression Analysis
✅ No Significant Change: Performance within acceptable range |
🚀 Performance Benchmark ResultsTinyGo Performance Benchmarks
TinyGo AOT Performance Benchmarks
AOT vs Regular Comparison
✅ TinyGo AOT component benchmarks completed 📊 Performance Comparison (macos-latest)Results from macos-latest |
![temper-pulseengine[bot]](https://avatars.githubusercontent.com/in/2863826?s=60&v=4){kind=small}
There was a problem hiding this comment.
AI Code Review for PR #82
This Bazel build file defines a module named bazel-file-ops-component that provides a set of universal file operations for Bazel build systems using WebAssembly components. The module includes dependencies on various Bazel rules and toolchains, including:
- Platform Support: Ensures compatibility across different platforms.
- C++ Toolchain Auto-Configuration: Automatically configures the C++ toolchain for cross-platform support.
- WebAssembly and Component Support: Provides tools for building WebAssembly components using TinyGo and Rust.
- Development Dependencies: Includes buildifier for linting and other development utilities.
- Documentation Site Dependencies: Simplifies setting up a documentation site with Node.js.
The module also includes Git repository overrides to use specific versions of the rules_wasm_component and rules_rust modules, ensuring compatibility and security. Additionally, it registers toolchains for C++, WASI SDK, Wasmtime runtime, and Rust toolchains to facilitate building and testing WebAssembly components.
This setup is designed to be flexible and reusable across different Bazel rule sets while maintaining security through WebAssembly sandboxing.
Reviewed by qwen2.5-coder:3b (local Ollama). Advisory only — may miss issues or report false positives.
Commands
| Command | Description |
|---|---|
/ask <question> |
Discuss this review — ask questions or disagree with findings |
/review-pr |
Re-run the review from scratch |
/review-pr <focus> |
Re-run with specific instructions (e.g. /review-pr focus on error handling) |
1 participant
Summary
USE_BAZEL_VERSION, notBAZEL_VERSION. Without a.bazelversionfile and with the wrong env var, Bazelisk downloaded Bazel 9.0.0 — which is incompatible withrules_go@0.57.0andaspect_bazel_lib.bazelversionwith8.4.0as the primary version pinBAZEL_VERSION→USE_BAZEL_VERSIONinci.ymlandperformance.ymlrules_cc0.2.4→0.2.14,bazel_skylib1.8.1→1.8.2Test plan
@dependabot rebasethe 7 blocked PRs to retrigger CI🤖 Generated with Claude Code