[codex] Recent unresolved review feedback

[haasonsaas]

Recent unresolved review feedback

• Owner: evalops
• Merged since: 2026-05-13
• Minimum severity: high
• Findings: 3

• p1 evalops/deploy#2966 scripts/reconcile-bazel-buildfarm-ops-agent.sh:294
  • PR: https://github.com/evalops/deploy/pull/2966
  • Feedback: https://github.com/evalops/deploy/pull/2966#discussion_r3251574885
  • Remove runner slots above desired RUNNER_COUNT
• p1 evalops/maestro-internal#1941 src/agent/transport.ts:1638
  • PR: https://github.com/evalops/maestro-internal/pull/1941
  • Feedback: https://github.com/evalops/maestro-internal/pull/1941#discussion_r3251301860
  • Skip cached reads while writes are still in flight
• p1 chore: sync public mirror from internal maestro#400 src/agent/transport.ts:912
  • PR: chore: sync public mirror from internal maestro#400
  • Feedback: chore: sync public mirror from internal maestro#400 (comment)
  • Persist provider tool results into turn state

[haasonsaas]

Recent unresolved review feedback

• Owner: evalops
• Merged since: 2026-05-13
• Minimum severity: high
• Findings: 10

• high evalops/nimbus#40 scripts/run-bazel-rbe.sh:28
  • PR: https://github.com/evalops/nimbus/pull/40
  • Feedback: https://github.com/evalops/nimbus/pull/40#discussion_r3252405608

  • Env var BAZEL_RBE_USE_EXISTING_TUNNEL set but never checked

• high evalops/console#42 scripts/run-bazel-rbe.sh:27
  • PR: https://github.com/evalops/console/pull/42
  • Feedback: https://github.com/evalops/console/pull/42#discussion_r3252408138

  • Env var BAZEL_RBE_USE_EXISTING_TUNNEL set but never read

• high [codex] add Bazel RBE smoke kestrel#99 Makefile:26
  • PR: [codex] add Bazel RBE smoke kestrel#99
  • Feedback: [codex] add Bazel RBE smoke kestrel#99 (comment)

  • Script ignores BAZEL_TARGETS, always runs all tests

• p1 evalops/maestro-internal#1976 pr_review
  • PR: https://github.com/evalops/maestro-internal/pull/1976
  • Feedback: https://github.com/evalops/maestro-internal/pull/1976#pullrequestreview-4303199202
  • Continue refreshing interrupted A2A task states
• high ci: add Bazel RBE lane mocktopus#9 .github/workflows/bazel-rbe.yml:66
  • PR: ci: add Bazel RBE lane mocktopus#9
  • Feedback: ci: add Bazel RBE lane mocktopus#9 (comment)

  • CI workflow missing uv needed by bazel-check

• p1 fix: omit non-terminal runtime promotion writes maestro#421 src/server/agent-runtime-ledger.ts:293
  • PR: fix: omit non-terminal runtime promotion writes maestro#421
  • Feedback: fix: omit non-terminal runtime promotion writes maestro#421 (comment)
  • Preserve wait_run emission for pending governance entries
• p1 evalops/platform#2076 internal/agentruntime/agentruntime/remote_runner_work_continuity.go:138
  • PR: https://github.com/evalops/platform/pull/2076
  • Feedback: https://github.com/evalops/platform/pull/2076#discussion_r3252239210
  • Ignore top-level drain protocol when continuity object is absent
• high ci: add Bazel RBE lane agent-pm#19 BUILD.bazel:88
  • PR: ci: add Bazel RBE lane agent-pm#19
  • Feedback: ci: add Bazel RBE lane agent-pm#19 (comment)

  • Missing pyproject.toml in Bazel test data attribute

• p1 chore: sync public mirror from internal maestro#417 MODULE.bazel:2
  • PR: chore: sync public mirror from internal maestro#417
  • Feedback: chore: sync public mirror from internal maestro#417 (comment)
  • Restore public Bazel module name
• p1 chore: sync public mirror from internal maestro#415 src/cli/commands/a2a.ts:264
  • PR: chore: sync public mirror from internal maestro#415
  • Feedback: chore: sync public mirror from internal maestro#415 (comment)
  • Make non-wait sends non-blocking

[haasonsaas]

Closed after the full recent-feedback sweep was addressed and the organization-level merge guard was enforced.

Evidence:

• All open EvalOps PR review threads audited after the sweep returned [] unresolved.
• Conversation resolution is now required on all 60 active EvalOps default branches audited in this run.
• Merged fixes include evalops/deploy#3049 and #3054, test: include pyproject in Bazel pytest data agent-pm#21 and docs: add Actions quota hygiene convention #22, test: guard Bazel workflow uv setup mocktopus#12, evalops/nimbus#42, evalops/maestro-internal#1980/#1981/#1982/#1983/#1984, and public mirrors feat: add first-party operational skill packages maestro#425/#426/#431/#432.
• Representative guardrail classes from [codex] Guardrail backlog: Test coverage gap (test-coverage-gap) #85, [codex] Guardrail backlog: Workflow shell footgun (workflow-shell-footgun) #69, [codex] Guardrail backlog: Runtime smoke coverage gap (runtime-smoke-coverage) #68, [codex] Guardrail backlog: Parser and CLI contract drift (parser-cli-contract) #50, and [codex] Guardrail backlog: Other feedback (other-feedback) #48 were closed with repo-specific evidence.