Skip to content
This repository was archived by the owner on Jul 19, 2026. It is now read-only.

ci: add Bazel RBE lane#12

Merged
haasonsaas merged 1 commit into
mainfrom
codex/bazel-rbe-agent-harness-20260516
May 16, 2026
Merged

ci: add Bazel RBE lane#12
haasonsaas merged 1 commit into
mainfrom
codex/bazel-rbe-agent-harness-20260516

Conversation

@haasonsaas

Copy link
Copy Markdown
Collaborator

Summary

  • add Bazel 9/Bzlmod/rules_python wiring for the core agent harness library and pytest suite
  • add a repo-scoped Bazel RBE workflow gated by BAZEL_RBE_ENABLED and the evalops-agent-harness-rbe runner label
  • add a compiled Python requirements lock and Make targets for local and remote Bazel validation

Test plan

  • actionlint .github/workflows/ci.yml .github/workflows/bazel-rbe.yml
  • python3 -m pytest -q
  • make bazel-check
  • make bazel-test

@cursor

cursor Bot commented May 16, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Medium risk because it introduces a new Bazel/Bzlmod build+test path (including remote execution) and a large generated MODULE.bazel.lock, which can affect CI reliability and dependency resolution but does not change runtime application logic.

Overview
Introduces a Bazel-based build/test lane for the Python agent harness: adds BUILD.bazel targets (py_library + py_test) and MODULE.bazel/MODULE.bazel.lock for Bazel 9 + Bzlmod + rules_python dependency resolution.

Adds remote execution (RBE) support via .bazelrc (remote-gcp-dev config + linux platform definition) and a scripts/run-bazel-rbe.sh helper that tunnels to a GCP buildfarm.

Updates CI/tooling: a new gated GitHub Actions workflow bazel-rbe.yml (runs only when BAZEL_RBE_ENABLED or manual dispatch), plus Makefile targets for locking Python deps (requirements_lock.txt), Bazel formatting/tidying, and local/remote Bazel test runs; also ignores Bazel outputs in .gitignore/.bazelignore.

Reviewed by Cursor Bugbot for commit 3c714e9. Bugbot is set up for automated code reviews on this repo. Configure here.

@socket-security

Copy link
Copy Markdown

@haasonsaas
haasonsaas merged commit 7e428ae into main May 16, 2026
7 checks passed
@haasonsaas
haasonsaas deleted the codex/bazel-rbe-agent-harness-20260516 branch May 16, 2026 05:25

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.

Reviewed by Cursor Bugbot for commit 3c714e9. Configure here.

Comment thread Makefile
bazel-rbe-smoke:
scripts/run-bazel-rbe.sh test //:pytest

bazel-check: requirements-lock bazel-format bazel-mod-tidy bazel-test

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI "check" target modifies files instead of validating

Medium Severity

The bazel-check target depends on requirements-lock, bazel-format, and bazel-mod-tidy, all of which mutate files in-place (regenerating the lock file, reformatting BUILD files, tidying MODULE.bazel) before running bazel-test. When the CI workflow runs make bazel-check, it silently fixes stale or unformatted committed files and then tests against the modified working tree, so it will never catch a PR with a stale requirements_lock.txt or unformatted BUILD files. The committed files remain incorrect but CI passes.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 3c714e9. Configure here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant