Skip to content

Commit dbaeb81

Browse files
thymikeeclaude
andcommitted
docs(agents): say a production source change runs the unit suite locally
Under "Interpret selection", state that related tests replace only provider integration and that a production source change also runs the unit suite, because some unit gates read sources as text. To stay within the agent-docs budget, three lists become sentences and two sentences that repeat tool output are removed. Co-Authored-By: Claude Opus 5.5 <[email protected]>
1 parent 919d757 commit dbaeb81

1 file changed

Lines changed: 8 additions & 26 deletions

File tree

‎docs/agents/testing.md‎

Lines changed: 8 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -10,21 +10,14 @@
1010
Serialized validation requires macOS or Linux.
1111
On Windows, use plan mode or focused checks.
1212
Both broad commands and heavy `pnpm gate` ids acquire the same host lock.
13-
Wait for the owner to finish, or cancel your waiter with Ctrl-C.
13+
Wait for the owner to finish.
1414
Do not add lock wrappers. Do not delete lock files.
15-
A wait ends after two hours. Nested validation fails immediately.
15+
A wait ends after two hours.
1616
Focused tests do not lock. Avoid subprocess-heavy tests beside a broad gate.
1717

18-
The runner prints progress and a receipt path. The receipt contains:
18+
The runner prints progress and a receipt path. The receipt contains refs, checkout state, command logs, exits, skips, and Vitest results.
1919

20-
- Refs.
21-
- Checkout state.
22-
- Command logs.
23-
- Exits.
24-
- Skips.
25-
- Vitest results.
26-
27-
Execution stops at the first failure. Unexecuted checks remain `not-run`.
20+
Execution stops at the first failure.
2821

2922
- `pnpm check:affected`: compact plan.
3023
- `pnpm check:affected --verbose`: plan with selection reasons.
@@ -47,15 +40,9 @@ Do not reduce selection to get a pass.
4740

4841
Read required-check status on the PR.
4942
The receipt lists skipped checks and owners in `results[].reason`.
50-
GitHub owns:
51-
52-
- Full coverage.
53-
- Native builds.
54-
- Device lanes.
55-
- Provider integration.
56-
- Compatibility that uses history.
57-
58-
Related tests do not replace those obligations.
43+
GitHub owns full coverage, native builds, device lanes, provider integration, and compatibility that uses history.
44+
Locally, related tests replace only provider integration.
45+
A production source change also runs the unit suite (about 3 minutes), because some unit gates read sources as text.
5946
Check the Integration Tests and Coverage jobs at the actual PR head.
6047
For docs-only changes, run the selected documentation checks.
6148
Do not add tests that assert prose.
@@ -73,12 +60,7 @@ Never call a failing gate passed.
7360
| Base comparison unavailable or inconclusive | Record the missing tool/device/evidence and next action. | Blocked, cause unproven. An authorized draft can record this. |
7461
| Failure source fixed, environment repaired, or new upstream commit contains the fix | Run the failing check. Then run the affected gate on the current head. | Replace blocked status only with new evidence. |
7562

76-
Match failures by:
77-
78-
- Check.
79-
- Location.
80-
- Diagnostic code.
81-
- Assertion.
63+
Match failures by check, location, diagnostic code, and assertion.
8264

8365
Until a named owner or issue accepts the blocker, the implementing agent owns diagnosis.
8466
Do not loop on an unchanged failure.

0 commit comments

Comments
 (0)