proagents 1.6.3 → 1.6.4
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.proagents/AI_INSTRUCTIONS.md +85 -26
- package/package.json +1 -1
|
@@ -2,6 +2,30 @@
|
|
|
2
2
|
|
|
3
3
|
This project uses ProAgents - an AI-agnostic development workflow framework.
|
|
4
4
|
|
|
5
|
+
## CRITICAL PRINCIPLE: ALWAYS DO, NEVER JUST TELL
|
|
6
|
+
|
|
7
|
+
**ProAgents automates development. The AI must EXECUTE actions, not give instructions for users to follow.**
|
|
8
|
+
|
|
9
|
+
| WRONG (Passive) | RIGHT (Active) |
|
|
10
|
+
|-----------------|----------------|
|
|
11
|
+
| "To run tests: `npm test`" | Actually run `npm test` and show results |
|
|
12
|
+
| "Create tests in `__tests__/`" | Actually create the test files |
|
|
13
|
+
| "Install dependencies with `npm install`" | Actually run `npm install` |
|
|
14
|
+
| "You can fix this by..." | Actually fix it |
|
|
15
|
+
| "Next steps: 1. Create... 2. Run..." | Do all steps automatically |
|
|
16
|
+
|
|
17
|
+
**Rules:**
|
|
18
|
+
1. **Execute, don't instruct** - Run commands, don't tell user to run them
|
|
19
|
+
2. **Create, don't suggest** - Write files, don't tell user to create them
|
|
20
|
+
3. **Fix, don't advise** - Fix issues directly, don't explain how to fix
|
|
21
|
+
4. **Complete the task** - Don't stop at giving "next steps"
|
|
22
|
+
5. **Only ask when truly needed** - Ambiguous requirements, destructive actions, or user preferences
|
|
23
|
+
|
|
24
|
+
**Exception:** Only give instructions when:
|
|
25
|
+
- Action requires credentials/secrets you don't have access to
|
|
26
|
+
- Action is destructive and needs explicit user confirmation
|
|
27
|
+
- Action is outside the project scope (external services, deployments)
|
|
28
|
+
|
|
5
29
|
## CRITICAL: Multi-AI Environment
|
|
6
30
|
|
|
7
31
|
**Multiple AI tools may work on this project simultaneously (Claude, Cursor, Gemini, Copilot, etc.). They do NOT share context.**
|
|
@@ -2003,30 +2027,7 @@ For `pa:test-mobile`:
|
|
|
2003
2027
|
ERROR: Missing required tools. Cannot run mobile test suite.
|
|
2004
2028
|
══════════════════════════════════════════════════════════
|
|
2005
2029
|
|
|
2006
|
-
|
|
2007
|
-
brew install maestro
|
|
2008
|
-
npm install -g detox-cli
|
|
2009
|
-
|
|
2010
|
-
Or let me install them for you:
|
|
2011
|
-
→ Type "yes" to auto-install missing tools
|
|
2012
|
-
→ Type "no" to cancel
|
|
2013
|
-
|
|
2014
|
-
Or run with limited tests:
|
|
2015
|
-
pa:test-mobile --unit-only (skip E2E tests)
|
|
2016
|
-
pa:test-mobile --skip-visual (skip visual tests)
|
|
2017
|
-
```
|
|
2018
|
-
|
|
2019
|
-
**IMPORTANT:** Never silently skip tests due to missing tools. Always:
|
|
2020
|
-
- Show clear error about what's missing
|
|
2021
|
-
- Provide installation instructions for each missing tool
|
|
2022
|
-
- Offer to auto-install missing tools (ask user permission first)
|
|
2023
|
-
- Fail the test suite if user declines (exit code 1)
|
|
2024
|
-
- Offer alternative flags (--unit-only, --skip-e2e, --skip-visual)
|
|
2025
|
-
|
|
2026
|
-
**Auto-install behavior:**
|
|
2027
|
-
If user agrees to auto-install:
|
|
2028
|
-
```
|
|
2029
|
-
Installing missing tools...
|
|
2030
|
+
Missing tools detected. Auto-installing...
|
|
2030
2031
|
|
|
2031
2032
|
Installing Maestro...
|
|
2032
2033
|
→ curl -Ls "https://get.maestro.mobile.dev" | bash
|
|
@@ -2039,6 +2040,20 @@ For `pa:test-mobile`:
|
|
|
2039
2040
|
All tools installed. Running test suite...
|
|
2040
2041
|
```
|
|
2041
2042
|
|
|
2043
|
+
**IMPORTANT - FULL AUTOMATION:**
|
|
2044
|
+
- Missing tools? **Install automatically** (no confirmation)
|
|
2045
|
+
- Tests fail? **Fix automatically** (no confirmation)
|
|
2046
|
+
- Fix doesn't work? **Try alternative fix** (no confirmation)
|
|
2047
|
+
- Only stop when: all tests pass OR no fix possible
|
|
2048
|
+
|
|
2049
|
+
**Do NOT:**
|
|
2050
|
+
- Ask "Should I install missing tools?"
|
|
2051
|
+
- Ask "Should I fix this?"
|
|
2052
|
+
- Say "Run `pa:test-auto-fix` to fix"
|
|
2053
|
+
- Give "Next steps" for user to follow
|
|
2054
|
+
|
|
2055
|
+
**Just DO it.**
|
|
2056
|
+
|
|
2042
2057
|
2. If all tools present, run comprehensive mobile test suite:
|
|
2043
2058
|
```
|
|
2044
2059
|
Mobile Test Suite
|
|
@@ -2092,10 +2107,54 @@ For `pa:test-mobile`:
|
|
|
2092
2107
|
2. E2E: User can reset password
|
|
2093
2108
|
3. Visual: SignupScreen design mismatch
|
|
2094
2109
|
|
|
2095
|
-
|
|
2110
|
+
══════════════════════════════════════════
|
|
2111
|
+
AUTO-FIX: Attempting to fix 3 failed tests...
|
|
2112
|
+
══════════════════════════════════════════
|
|
2113
|
+
```
|
|
2114
|
+
|
|
2115
|
+
3. **Automatically fix all failures** (no confirmation needed):
|
|
2116
|
+
```
|
|
2117
|
+
══════════════════════════════════════════
|
|
2118
|
+
AUTO-FIX: Fixing 3 failed tests...
|
|
2119
|
+
══════════════════════════════════════════
|
|
2120
|
+
|
|
2121
|
+
Fix 1/3: Integration - Password reset flow
|
|
2122
|
+
──────────────────────────────────────────
|
|
2123
|
+
Root cause: ResetEmailService.send() not called
|
|
2124
|
+
Fixing: src/services/AuthService.ts:142
|
|
2125
|
+
→ Added: await this.resetEmailService.send(email)
|
|
2126
|
+
Re-running test... ✓ PASSED
|
|
2127
|
+
|
|
2128
|
+
Fix 2/3: E2E - User can reset password
|
|
2129
|
+
──────────────────────────────────────────
|
|
2130
|
+
Root cause: Button text mismatch
|
|
2131
|
+
Fixing: src/screens/ResetPassword.tsx:28
|
|
2132
|
+
→ Changed: "Reset" → "Send Reset Email"
|
|
2133
|
+
Re-running test... ✓ PASSED
|
|
2134
|
+
|
|
2135
|
+
Fix 3/3: Visual - SignupScreen design mismatch
|
|
2136
|
+
──────────────────────────────────────────
|
|
2137
|
+
Root cause: Wrong color and font size
|
|
2138
|
+
Fixing: src/screens/SignupScreen.styles.ts:15-16
|
|
2139
|
+
→ Changed: color #0066CC → #007AFF
|
|
2140
|
+
→ Changed: fontSize 14px → 16px
|
|
2141
|
+
Re-running test... ✓ PASSED
|
|
2142
|
+
|
|
2143
|
+
══════════════════════════════════════════
|
|
2144
|
+
ALL TESTS PASSING: 38/38 ✓
|
|
2145
|
+
══════════════════════════════════════════
|
|
2146
|
+
```
|
|
2147
|
+
|
|
2148
|
+
4. If a fix cannot be applied automatically:
|
|
2149
|
+
```
|
|
2150
|
+
✗ Cannot auto-fix: Database schema mismatch
|
|
2151
|
+
Reason: Requires migration file and database access
|
|
2152
|
+
|
|
2153
|
+
Manual action needed:
|
|
2154
|
+
→ Run: npx prisma migrate dev --name fix_user_table
|
|
2096
2155
|
```
|
|
2097
2156
|
|
|
2098
|
-
|
|
2157
|
+
5. Generate detailed test report at `./.proagents/test-reports/mobile-{timestamp}.md`
|
|
2099
2158
|
|
|
2100
2159
|
For `pa:test-visual`:
|
|
2101
2160
|
1. Take screenshots of all screens/components
|