From 6fde280c9a391e5ccc8a4cb3ce4923dfba223f42 Mon Sep 17 00:00:00 2001 From: Michael Bolin Date: Fri, 9 Jan 2026 15:36:43 -0800 Subject: [PATCH] fix: include AGENTS.md as repo root marker for integration tests --- .github/workflows/bazel.yml | 16 ++++++++++++++++ BUILD.bazel | 2 ++ codex-rs/core/BUILD.bazel | 9 +++++++++ 3 files changed, 27 insertions(+) diff --git a/.github/workflows/bazel.yml b/.github/workflows/bazel.yml index 4f97d2de02..7cb3d2c647 100644 --- a/.github/workflows/bazel.yml +++ b/.github/workflows/bazel.yml @@ -108,3 +108,19 @@ jobs: --build_metadata=ROLE=CI \ --build_metadata=VISIBILITY=PUBLIC \ "--remote_header=x-buildbuddy-api-key=$BUILDBUDDY_API_KEY" + + cloud-build: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v6 + - name: Set up Bazel + uses: bazelbuild/setup-bazelisk@v3 + - name: bazel test //... --config=remote + run: | + bazel $BAZEL_STARTUP_ARGS --bazelrc=.github/workflows/ci.bazelrc test //... \ + --build_metadata=REPO_URL=https://github.com/openai/codex.git \ + --build_metadata=COMMIT_SHA=$(git rev-parse HEAD) \ + --build_metadata=ROLE=CI \ + --build_metadata=VISIBILITY=PUBLIC \ + "--remote_header=x-buildbuddy-api-key=$BUILDBUDDY_API_KEY" \ + --config=remote --platforms=//:rbe --keep_going diff --git a/BUILD.bazel b/BUILD.bazel index 5365fb8922..372a3aee7c 100644 --- a/BUILD.bazel +++ b/BUILD.bazel @@ -27,3 +27,5 @@ platform( "OSFamily": "Linux", }, ) + +exports_files(["AGENTS.md"]) diff --git a/codex-rs/core/BUILD.bazel b/codex-rs/core/BUILD.bazel index c505c32066..67f87999b7 100644 --- a/codex-rs/core/BUILD.bazel +++ b/codex-rs/core/BUILD.bazel @@ -20,6 +20,15 @@ codex_rust_crate( "//codex-rs/apply-patch:apply_patch_tool_instructions.md", "prompt.md", ], + # This is a bit of a hack, but empirically, some of our integration tests + # are relying on the presence of this file as a repo root marker. When + # running tests locally, this "just works," but in remote execution, + # the working directory is different and so the file is not found unless it + # is explicitly added as test data. + # + # TODO(aibrahim): Update the tests so that `just bazel-remote-test` succeeds + # without this workaround. + test_data_extra = ["//:AGENTS.md"], integration_deps_extra = ["//codex-rs/core/tests/common:common"], test_tags = ["no-sandbox"], extra_binaries = [