From 4c165ad7da0fceebeec4e9c6dd2fa308cf45d115 Mon Sep 17 00:00:00 2001 From: Ryan Yin Date: Fri, 9 Oct 2026 15:57:52 +0800 Subject: [PATCH] docs(agents): prefer E2E verification of real workflows --- agents/AGENTS.md | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/agents/AGENTS.md b/agents/AGENTS.md index 0a2d5a27..845e66df 100644 --- a/agents/AGENTS.md +++ b/agents/AGENTS.md @@ -105,8 +105,10 @@ For an impactful change, follow these steps in order, scaled to its risk: overwrite, discard, or restore unrelated user changes or removals without authorization. - Keep diffs minimal and backward compatible; ask before a breaking change. - Documentation should be self-contained for its reader and omit irrelevant history. -- Verify in proportion to risk. Agents MUST NOT claim a check passed without running it, or make it - pass by weakening what it verifies (e.g. mocking the code under test). +- Verify in proportion to risk. Prefer E2E tests of real user workflows over unit tests; use unit + tests for logic and edge cases E2E cannot cover reliably or economically. Agents MUST NOT claim a + check passed without running it, or make it pass by weakening what it verifies (e.g. mocking the + code under test). - Cleanup: when a PR the agent opened is merged, finish up as part of that task: delete the local branch and worktrees it created and fast-forward the default branch if the checkout is free. Touch nothing it did not create; skip and report instead of forcing.