From 3ea9e983334ce187b295359d19b17706b7dd1144 Mon Sep 17 00:00:00 2001
From: "Adam Perry @ OpenAI" <anp@openai.com>
Date: Fri, 5 Jun 2026 18:53:12 -0700
Subject: [PATCH] Remove `just bench-smoke` from `just test`. (#26716)

## Why

`just test` should run the test suite without also compiling and
executing benchmark smoke tests. Keeping benchmark validation explicit
avoids adding unrelated work to every project-specific test invocation.

## What changed

- Remove the `just bench-smoke` step from the Unix and Windows `test`
recipes.
- Document `just bench` and `just bench-smoke` as the explicit benchmark
commands in `AGENTS.md`.

## Validation

- `just test -p codex-arg0`
- `just --dry-run test`
- `just --dry-run bench-smoke`
---
 AGENTS.md | 6 ++++++
 justfile  | 2 --
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/AGENTS.md b/AGENTS.md
index 4666566f9..46d2626e3 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -192,6 +192,12 @@ If you don’t have the tool:
 
 - `cargo install --locked cargo-insta`
 
+### Benchmarks
+
+cargo benchmarks can be run with `just bench`, use the divan crate to write new ones.
+
+Use `just bench-smoke` to dry-run the benchmark for a single iteration to ensure it works.
+
 ### Test assertions
 
 - Tests should use pretty_assertions::assert_eq for clearer diffs. Import this at the top of the test module if it isn't already.
diff --git a/justfile b/justfile
index fe7e7349b..3b993b767 100644
--- a/justfile
+++ b/justfile
@@ -76,12 +76,10 @@ install:
 [unix]
 test *args:
     RUST_MIN_STACK={{ rust_min_stack }} cargo nextest run --no-fail-fast "$@"
-    just bench-smoke
 
 [windows]
 test *args:
     $env:RUST_MIN_STACK = "{{ rust_min_stack }}"; cargo nextest run --no-fail-fast @($args | Select-Object -Skip 1)
-    just bench-smoke
 
 # Run from the repository root so scripts that resolve paths from `cwd` see
 # the same layout they use in GitHub Actions.