Skip to content

Commit 72933a3

Browse files
committed
Address PR review feedback
- Add git history protection rule to AGENTS.md - Fix missing colon in gradle test command - Improve test rule wording for production code changes
1 parent 6cc4b6c commit 72933a3

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

.agents/skills/implement/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ If other modules were changed, compile those too:
5656
### 3c: Unit Tests
5757

5858
```bash
59-
./gradlew aztec:testRelease
59+
./gradlew :aztec:testRelease
6060
```
6161

6262
Or run specific tests related to the changes:
@@ -66,7 +66,7 @@ Or run specific tests related to the changes:
6666

6767
**Test rules:**
6868
- NEVER weaken or remove assertions to make tests pass
69-
- NEVER modify production code just to pass a test without permission
69+
- Don't make unrelated or incorrect changes to production code solely to appease a test. If a test reveals a real issue, fix it properly.
7070
- Tests that pass only by not crashing are invalid — every test needs meaningful assertions
7171
- If a test won't pass after reasonable attempts: stop and ask
7272

AGENTS.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,7 @@ All plugin/loader modules depend on `:aztec`. Published independently to Automat
7373
- **NEVER create a PR without explicit permission**
7474
- When asked to "fix" or "update" something, that does NOT imply permission to commit/push
7575
- Always wait for explicit "commit", "push", or "create PR" commands
76+
- Do not create branches, rebase, or modify git history unless explicitly asked
7677

7778
## Build Commands
7879

0 commit comments

Comments
 (0)