Skip to content

Commit 698456e

Browse files
authored
ci: reduce verbosity of testing suggestions in AI reviews (#958)
1 parent abf4a51 commit 698456e

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

.github/workflows/claude-code-review.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -127,11 +127,11 @@ jobs:
127127
- If component APIs changed, check for updates to JSDoc comments or usage examples.
128128
- If new environment variables added, ensure .env.example is updated.
129129
- If OpenAPI types changed (src/types/core.ts), verify it was generated not hand-edited.
130-
- Testing requirements:
131-
- Unit tests (*.spec.ts) for new utilities in src/lib/
132-
- E2E tests (e2e-tests/) for new user flows
133-
- Suggest test coverage for critical business logic
134-
- Verify test commands: pnpm test:unit, pnpm test:e2e, pnpm lint, pnpm build
130+
- Testing guidance (keep concise—avoid verbose or generic suggestions):
131+
- Only suggest tests directly relevant to the specific changes in this PR
132+
- Briefly note if new utilities in src/lib/ lack a *.spec.ts file
133+
- For UI changes, suggest 1-2 specific manual testing steps the author can perform
134+
- Do NOT suggest comprehensive test suites or generic test patterns unrelated to the PR scope
135135
- If migration or breaking changes are introduced, recommend adding explicit notes.
136136
137137
Review Format:

.github/workflows/codex-comment.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,10 @@ jobs:
126126
Documentation & Testing:
127127
- Cross-check docs (README, CLAUDE.md, AGENTS.md) when user flows/components change.
128128
- Confirm env vars and OpenAPI types are updated where applicable.
129-
- Recommend test coverage (unit/e2e) for critical flows.
129+
- Testing (keep concise—avoid verbose or generic suggestions):
130+
* Only suggest tests directly relevant to the specific changes in this PR
131+
* For UI changes, suggest 1-2 specific manual testing steps
132+
* Do NOT recommend comprehensive test suites or patterns unrelated to the PR scope
130133
131134
Output Instructions:
132135
- Start with a summary of what the PR achieves.

0 commit comments

Comments
 (0)