proagents 1.6.2 → 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 +111 -3
- 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.**
|
|
@@ -1990,7 +2014,47 @@ For `pa:alias-add`:
|
|
|
1990
2014
|
**How to execute Mobile Test Suite commands:**
|
|
1991
2015
|
|
|
1992
2016
|
For `pa:test-mobile`:
|
|
1993
|
-
1.
|
|
2017
|
+
1. **FIRST: Check required tools are installed (FAIL if missing, do NOT skip):**
|
|
2018
|
+
```
|
|
2019
|
+
Checking required tools...
|
|
2020
|
+
✓ Node.js (v18.0.0)
|
|
2021
|
+
✓ Jest / Vitest
|
|
2022
|
+
✓ React Native Testing Library
|
|
2023
|
+
✗ MISSING: Maestro - Required for E2E tests
|
|
2024
|
+
✗ MISSING: Detox - Required for E2E tests
|
|
2025
|
+
|
|
2026
|
+
══════════════════════════════════════════════════════════
|
|
2027
|
+
ERROR: Missing required tools. Cannot run mobile test suite.
|
|
2028
|
+
══════════════════════════════════════════════════════════
|
|
2029
|
+
|
|
2030
|
+
Missing tools detected. Auto-installing...
|
|
2031
|
+
|
|
2032
|
+
Installing Maestro...
|
|
2033
|
+
→ curl -Ls "https://get.maestro.mobile.dev" | bash
|
|
2034
|
+
✓ Maestro installed successfully
|
|
2035
|
+
|
|
2036
|
+
Installing Detox CLI...
|
|
2037
|
+
→ npm install -g detox-cli
|
|
2038
|
+
✓ Detox CLI installed successfully
|
|
2039
|
+
|
|
2040
|
+
All tools installed. Running test suite...
|
|
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
|
+
|
|
2057
|
+
2. If all tools present, run comprehensive mobile test suite:
|
|
1994
2058
|
```
|
|
1995
2059
|
Mobile Test Suite
|
|
1996
2060
|
═════════════════
|
|
@@ -2043,10 +2107,54 @@ For `pa:test-mobile`:
|
|
|
2043
2107
|
2. E2E: User can reset password
|
|
2044
2108
|
3. Visual: SignupScreen design mismatch
|
|
2045
2109
|
|
|
2046
|
-
|
|
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
|
|
2047
2155
|
```
|
|
2048
2156
|
|
|
2049
|
-
|
|
2157
|
+
5. Generate detailed test report at `./.proagents/test-reports/mobile-{timestamp}.md`
|
|
2050
2158
|
|
|
2051
2159
|
For `pa:test-visual`:
|
|
2052
2160
|
1. Take screenshots of all screens/components
|