From e54d45d119c46e19be163d686c2733ade3469122 Mon Sep 17 00:00:00 2001 From: David Brailovsky Date: Fri, 28 Aug 2026 16:39:28 +0200 Subject: [PATCH] docs(coding-agent): remove issue-specific regression test placement rule (#8775) --- AGENTS.md | 2 +- packages/coding-agent/test/suite/README.md | 6 ------ 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 765f376b8..12387d5dd 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -35,7 +35,7 @@ - `packages/tui` (`node:test`): `node --test test/specific.test.ts` - If you create or modify a test file, run it and iterate on test or implementation until it passes. - For `packages/coding-agent/test/suite/`, use `test/suite/harness.ts` + the faux provider. No real provider APIs, keys, or paid tokens. -- Put issue-specific regressions under `packages/coding-agent/test/suite/regressions/` named `-.test.ts`. +- When regressions tests for fixing a github issue, add a comment with the github issue number next to the test. - For ad-hoc scripts, `write` them to a temp file (e.g. `/tmp`), run, edit if needed, remove when done. Don't embed multi-line scripts in `bash` commands. - Never commit unless the user asks. diff --git a/packages/coding-agent/test/suite/README.md b/packages/coding-agent/test/suite/README.md index cdd25a97f..b6f758dc1 100644 --- a/packages/coding-agent/test/suite/README.md +++ b/packages/coding-agent/test/suite/README.md @@ -8,9 +8,3 @@ Rules: - Do not use real provider APIs, real API keys, network calls, or paid tokens - Keep these tests CI-safe and deterministic - Do not use or extend the legacy `test/test-harness.ts` path unless a missing capability forces it - -Organization: -- Put broad lifecycle and characterization tests directly under `test/suite/` -- Put issue-specific regression tests under `test/suite/regressions/` -- Name regression tests as `-.test.ts` -- Example: `test/suite/regressions/2023-queued-slash-command-followup.test.ts`