prjct-cli 0.19.0 → 0.20.0

package/templates/commands/ship.md

@@ -1,31 +1,52 @@
 ---
-allowed-tools: [Read, Write, Bash]
+allowed-tools: [Read, Write, Bash, Glob, Grep]
 description: 'Ship feature with automated workflow'
 ---
 
 # /p:ship - Ship Feature
 
-Ship completed work with quality checks, version bump, and git commit.
+Ship completed work with pre-flight checks, code review, and quality gates.
 
 ## Usage
 ```
-/p:ship [feature] [--blocking]
+/p:ship [feature] [--blocking] [--skip-review]
 ```
 - `feature`: Name of the feature being shipped (required)
-- `--blocking`: Abort if lint/tests fail
+- `--blocking`: Abort if any check fails
+- `--skip-review`: Skip code review (for trivial changes)
 
 ## Flow
 
-### Step 1: Validate
+### Step 1: Validate Project
 READ: `.prjct/prjct.config.json` → extract `projectId`
 IF missing: "No prjct project. Run /p:init first." → STOP
 
 SET: `{globalPath}` = `~/.prjct-cli/projects/{projectId}`
 
+### Step 2: Pre-flight Checks
+
+#### 2.1 Check for changes
 BASH: `git status --porcelain`
 IF empty: "No changes to ship." → STOP
 
-### Step 2: Quality Checks (optional)
+#### 2.2 Analyze change scope
+BASH: `git diff --stat HEAD`
+EXTRACT: {filesChanged}, {insertions}, {deletions}
+SET: {totalLines} = {insertions} + {deletions}
+
+#### 2.3 Detect trivial changes
+IF {totalLines} < 10 AND {filesChanged} <= 2:
+  SET: {changeType} = "trivial"
+  OUTPUT: "Trivial changes detected ({totalLines} lines in {filesChanged} files)"
+ELSE IF {totalLines} < 50:
+  SET: {changeType} = "small"
+ELSE IF {totalLines} < 200:
+  SET: {changeType} = "medium"
+ELSE:
+  SET: {changeType} = "large"
+  OUTPUT: "Large change detected ({totalLines} lines). Full review recommended."
+
+### Step 3: Quality Checks
 BASH: `npm run lint 2>&1 || echo "LINT_SKIP"`
 BASH: `npm test 2>&1 || echo "TEST_SKIP"`
 
@@ -33,7 +54,61 @@ IF `--blocking` AND (lint failed OR tests failed):
   OUTPUT: "Quality checks failed. Ship blocked."
   STOP
 
-### Step 3: Version Bump
+### Step 4: Code Review with Confidence Scoring
+
+IF `--skip-review` OR {changeType} == "trivial":
+  OUTPUT: "Skipping code review."
+  → Go to Step 5
+ELSE:
+  OUTPUT: "Running code review..."
+
+#### 4.1 Get changed files
+BASH: `git diff --name-only HEAD`
+SET: {changedFiles} = result
+
+#### 4.2 Review each file for issues
+FOR each file in {changedFiles}:
+  BASH: `git diff HEAD -- {file}`
+
+  Analyze diff for:
+  - Missing error handling
+  - Security issues (hardcoded secrets, SQL injection, XSS)
+  - Logic errors
+  - Missing null checks
+  - Resource leaks
+
+  FOR each issue found:
+    ASSIGN confidence score (0-100):
+    - 90-100: Definite bug/security issue
+    - 70-89: Likely problem, should fix
+    - 50-69: Maybe a problem
+    - 0-49: Nitpick/style preference
+
+#### 4.3 Filter and report
+SET: {issues} = issues with confidence >= 70
+
+IF {issues}.length > 0:
+  OUTPUT: """
+  ## Code Review Results
+
+  Found {issues.length} issues (confidence >= 70%):
+
+  {FOR each issue:}
+  - [{confidence}%] {description}
+    File: {file}:{line}
+  {END FOR}
+  """
+
+  IF `--blocking`:
+    OUTPUT: "Fix issues before shipping."
+    STOP
+  ELSE:
+    PROMPT: "Continue with ship? (y/n)"
+    IF no: STOP
+ELSE:
+  OUTPUT: "Code review passed. No high-confidence issues found."
+
+### Step 5: Version Bump
 READ: `package.json` (or Cargo.toml, pyproject.toml)
 EXTRACT: current version
 
@@ -45,11 +120,11 @@ Determine bump type:
 
 UPDATE version file with new version
 
-### Step 4: Update CHANGELOG
+### Step 6: Update CHANGELOG
 BASH: `git log --oneline -20 --pretty=format:"- %s"`
 INSERT new entry in CHANGELOG.md
 
-### Step 5: Git Commit & Push
+### Step 7: Git Commit & Push
 BASH: `git add .`
 BASH: `git commit -m "feat: Ship {feature} v{newVersion}
 
@@ -58,7 +133,7 @@ Designed for [Claude](https://www.anthropic.com/claude)
 "`
 BASH: `git push`
 
-### Step 6: Update Storage
+### Step 8: Update Storage
 GET timestamp: `bun -e "console.log(new Date().toISOString())" 2>/dev/null || node -e "console.log(new Date().toISOString())"`
 GET uuid: `bun -e "console.log(crypto.randomUUID())" 2>/dev/null || node -e "console.log(require('crypto').randomUUID())"`
 
@@ -85,12 +160,14 @@ APPEND to `{globalPath}/sync/pending.json`
 Log to memory:
 APPEND to `{globalPath}/memory/events.jsonl`
 
-### Step 7: Output
+### Step 9: Output
 ```
 🚀 Shipped: {feature}
 
 Version: {oldVersion} → {newVersion}
-Lint: {lintStatus} | Tests: {testStatus}
+Changes: {changeType} ({totalLines} lines in {filesChanged} files)
+Quality: Lint {lintStatus} | Tests {testStatus}
+Review: {reviewStatus}
 
 Next: /p:feature | /p:recap
 ```
@@ -103,8 +180,58 @@ Next: /p:feature | /p:recap
 | No changes | "No changes to ship" | STOP |
 | Lint/test fails (blocking) | "Quality checks failed" | STOP |
 | Lint/test fails (non-blocking) | Show warning | CONTINUE |
+| Code review finds issues (blocking) | "Fix issues before shipping" | STOP |
+| Code review finds issues (non-blocking) | Prompt to continue | ASK |
 | Push fails | "Push failed. Try: git pull --rebase" | CONTINUE |
 
+## Examples
+
+### Example 1: Trivial Change (auto-skip review)
+```
+/p:ship "fix typo"
+
+Pre-flight: Trivial changes (3 lines in 1 file)
+Quality: Lint ✅ | Tests ✅
+Review: Skipped (trivial)
+
+🚀 Shipped: fix typo
+Version: 1.2.0 → 1.2.1
+```
+
+### Example 2: Medium Change (with review)
+```
+/p:ship "add auth"
+
+Pre-flight: Medium changes (87 lines in 5 files)
+Quality: Lint ✅ | Tests ✅
+Review: Running...
+
+## Code Review Results
+Found 2 issues (confidence >= 70%):
+- [85%] Missing error handling in OAuth callback
+  File: src/auth/oauth.ts:67
+- [72%] Token not validated before use
+  File: src/auth/validate.ts:23
+
+Continue with ship? (y/n)
+```
+
+### Example 3: Blocking Mode
+```
+/p:ship "deploy script" --blocking
+
+Pre-flight: Small changes (28 lines in 2 files)
+Quality: Lint ✅ | Tests ✅
+Review: Running...
+
+## Code Review Results
+Found 1 issue (confidence >= 70%):
+- [95%] Hardcoded credentials detected
+  File: scripts/deploy.sh:12
+
+Fix issues before shipping.
+```
+
 ## References
 - Architecture details: `~/.prjct-cli/docs/architecture.md`
 - Validation patterns: `~/.prjct-cli/docs/validation.md`

package/templates/commands/sync.md

@@ -167,6 +167,42 @@ GLOB for config files and analyze:
 
 EXTRACT: `{languages}`, `{frameworks}`, `{techStack}`
 
+### Detect Frontend/UI Stack (for UX/UI Agent)
+
+**CRITICAL**: If ANY frontend technology is detected, generate the UX/UI agent.
+
+#### Web Frontend Detection
+```bash
+# Check package.json for web frameworks
+grep -E '"(react|react-dom|next|vue|nuxt|svelte|@sveltejs/kit|@angular/core)"' package.json 2>/dev/null
+```
+
+SET: `{hasWebFrontend}` = true if any match
+
+#### Mobile Frontend Detection
+```bash
+# React Native / Expo
+grep -E '"(react-native|expo)"' package.json 2>/dev/null
+
+# Flutter
+test -f pubspec.yaml && echo "flutter"
+
+# SwiftUI (iOS)
+find . -name "*.swift" -exec grep -l "import SwiftUI" {} \; 2>/dev/null | head -1
+
+# Jetpack Compose (Android)
+find . -name "*.kt" -exec grep -l "androidx.compose" {} \; 2>/dev/null | head -1
+```
+
+SET: `{hasMobileFrontend}` = true if any match
+
+#### Combined Frontend Flag
+```
+{hasFrontendUI} = {hasWebFrontend} OR {hasMobileFrontend}
+```
+
+EXTRACT: `{frontendType}` = "web" | "mobile" | "both" | null
+
 ---
 
 ## Step 4: Regenerate ALL Context Files
@@ -421,13 +457,32 @@ Analyze `{techStack}` from Step 3 and generate ONLY relevant domain agents:
 | PostgreSQL, MySQL, MongoDB, Prisma | `database.md` | `templates/subagents/domain/database.md` |
 | Docker, Kubernetes, GitHub Actions | `devops.md` | `templates/subagents/domain/devops.md` |
 | Jest, Pytest, Vitest, testing | `testing.md` | `templates/subagents/domain/testing.md` |
+| **{hasFrontendUI} = true** | `uxui.md` | `templates/agentic/agents/uxui.md` |
 
 For EACH detected stack:
 1. READ template from `templates/subagents/domain/{name}.md`
 2. ADAPT description with detected frameworks (e.g., "React specialist" not just "frontend")
 3. WRITE to `{globalPath}/agents/{name}.md`
 
-### 7.5 Report Generated Agents
+### 7.5 Generate UX/UI Agent (CRITICAL for Frontend Projects)
+
+**Priority: UX > UI** - User experience is more important than visuals.
+
+IF `{hasFrontendUI}` == true:
+
+1. READ template: `templates/agentic/agents/uxui.md`
+2. WRITE to: `{globalPath}/agents/uxui.md`
+3. ADD to `{domainAgents}`: "uxui"
+
+OUTPUT: "🎨 Generated UX/UI agent for {frontendType} ({frameworks detected})"
+
+The UX/UI agent ensures:
+- **UX First**: Clarity, feedback, reduced friction, error handling, accessibility
+- **Modern UI**: Distinctive typography, bold colors, purposeful animation
+- **Anti-patterns avoided**: No "AI slop" (Inter font, purple gradients, generic layouts)
+- **Checklists**: UX and UI quality gates before shipping
+
+### 7.6 Report Generated Agents
 
 Track which agents were generated for output:
 - `{workflowAgents}`: Always 3 (prjct-workflow, prjct-planner, prjct-shipper)
@@ -516,7 +571,10 @@ IF cloudSync AND no syncError:
 
 🤖 Claude Code Sub-Agents ({workflowAgents.length + domainAgents.length})
 ├── Workflow: prjct-workflow, prjct-planner, prjct-shipper
-└── Domain: {domainAgents.join(', ') || 'none'}
+├── Domain: {domainAgents.join(', ') || 'none'}
+{IF hasFrontendUI}
+└── 🎨 UX/UI: uxui.md (Priority: UX > UI)
+{ENDIF}
 
 {IF cloudSync}
 ☁️ Cloud Sync
@@ -581,5 +639,6 @@ Next: /p:now to start a new task
 ├── backend.md               # (if Node/Go/Python API detected)
 ├── database.md              # (if DB detected)
 ├── devops.md                # (if Docker/K8s detected)
-└── testing.md               # (if test framework detected)
+├── testing.md               # (if test framework detected)
+└── uxui.md                  # (if ANY frontend UI detected - web or mobile)
 ```

package/templates/commands/test.md

@@ -1,6 +1,6 @@
 ---
 allowed-tools: [Bash, Read, Write, Edit]
-description: 'Run tests with auto-fix and AI generation'
+description: 'Run tests with auto-fix'
 timestamp-rule: 'GetTimestamp() for ALL timestamps'
 ---
 
@@ -9,10 +9,8 @@ timestamp-rule: 'GetTimestamp() for ALL timestamps'
 ## Usage
 
 ```
-/p:test [all|unit|e2e|failed|fix|ai]
+/p:test [all|unit|e2e|failed|fix]
         [--blocking]           # Exit with error if tests fail
-        [--testsprite]         # Force TestSprite even if native runner exists
-        [--scope=codebase|diff] # TestSprite: test all or only changed files
 ```
 
 ## Context Variables
@@ -35,90 +33,12 @@ IF file not found:
 SET defaults:
 - {testMode} = "all"
 - {blocking} = false
-- {useTestSprite} = false
-- {testScope} = "diff"
 
 PARSE args:
-- IF arg is "ai": {useTestSprite} = true
 - IF arg is "all|unit|e2e|failed|fix": {testMode} = arg
 - IF arg contains "--blocking": {blocking} = true
-- IF arg contains "--testsprite": {useTestSprite} = true
-- IF arg contains "--scope=codebase": {testScope} = "codebase"
 
-## Step 3: Check TestSprite API Key (If Needed)
-
-IF {useTestSprite}:
-  READ: `{configPath}`
-
-  ### Check if user dismissed TestSprite
-  IF config.testspriteSkip == "never":
-    OUTPUT: "ℹ️ TestSprite disabled. Using native tests."
-    {useTestSprite} = false
-    → Go to Step 4
-
-  IF config.testspriteSkip == "later" AND config.testspriteSkipUntil > now:
-    OUTPUT: "ℹ️ TestSprite reminder snoozed. Using native tests."
-    {useTestSprite} = false
-    → Go to Step 4
-
-  ### Check for existing API key
-  IF config.testspriteApiKey exists AND is not empty:
-    SET: {apiKey} = config.testspriteApiKey
-    → Continue to Step 4
-
-  ### No API key - show options (NON-BLOCKING)
-  OUTPUT: "🤖 TestSprite AI Testing (Optional)"
-  OUTPUT: ""
-  OUTPUT: "TestSprite can generate and run AI-powered tests."
-  OUTPUT: "Free API key at: https://testsprite.com/dashboard/api-keys"
-  OUTPUT: ""
-
-  ASK with options:
-    1. "Enter API key" → Prompt for key
-    2. "Skip for now" → Use native tests this time
-    3. "Remind me in a week" → Snooze reminder
-    4. "Never ask again" → Disable permanently
-
-  HANDLE response:
-
-  IF option 1 (Enter API key):
-    ASK: "Paste your TestSprite API key:"
-    SET: {apiKey} = user input
-
-    UPDATE config.json:
-    - testspriteApiKey: {apiKey}
-
-    OUTPUT: "✅ API key saved"
-    → Continue to Step 4
-
-  IF option 2 (Skip for now):
-    OUTPUT: "⏭️ Skipping TestSprite. Using native tests."
-    {useTestSprite} = false
-    → Go to Step 4
-
-  IF option 3 (Remind me in a week):
-    SET: {skipUntil} = now + 7 days
-
-    UPDATE config.json:
-    - testspriteSkip: "later"
-    - testspriteSkipUntil: {skipUntil}
-
-    OUTPUT: "⏰ Will ask again in a week. Using native tests."
-    {useTestSprite} = false
-    → Go to Step 4
-
-  IF option 4 (Never ask again):
-    UPDATE config.json:
-    - testspriteSkip: "never"
-
-    OUTPUT: "🔕 Won't ask again. Use --testsprite flag to enable manually."
-    {useTestSprite} = false
-    → Go to Step 4
-
-## Step 4: Detect Testing Strategy
-
-IF {useTestSprite}:
-  → Go to Step 6 (TestSprite AI Testing)
+## Step 3: Detect Testing Strategy
 
 ### Check for Native Test Runner
 
@@ -126,27 +46,28 @@ READ: `package.json`
 IF has "scripts.test":
   {runner} = "npm"
   {runnerCmd} = "npm test"
-  → Go to Step 5 (Native Testing)
+  → Go to Step 4 (Native Testing)
 
 IF file exists: `pytest.ini` OR `pyproject.toml` with pytest:
   {runner} = "pytest"
   {runnerCmd} = "pytest"
-  → Go to Step 5
+  → Go to Step 4
 
 IF file exists: `Cargo.toml`:
   {runner} = "cargo"
   {runnerCmd} = "cargo test"
-  → Go to Step 5
+  → Go to Step 4
 
 IF no runner found:
   OUTPUT: "No test runner detected."
   OUTPUT: ""
   OUTPUT: "Options:"
-  OUTPUT: "• /p:test ai - Generate tests with AI (TestSprite)"
   OUTPUT: "• Add 'test' script to package.json"
+  OUTPUT: "• Add pytest.ini for Python projects"
+  OUTPUT: "• Add Cargo.toml for Rust projects"
   STOP
 
-## Step 5: Native Test Runner
+## Step 4: Native Test Runner
 
 OUTPUT: "🧪 Running tests with {runner}..."
 
@@ -177,62 +98,9 @@ IF {testMode} == "fix" AND {testStatus} == "failed":
   BASH: `{runnerCmd} 2>&1`
   CAPTURE and re-parse results
 
-→ Go to Step 8 (Results)
-
-## Step 6: TestSprite AI Testing
-
-OUTPUT: "🤖 Running AI-powered tests with TestSprite..."
-
-### 6.1 Bootstrap Tests
-
-CALL MCP TOOL: `testsprite_bootstrap_tests`
-PARAMETERS:
-- projectPath: current working directory (absolute path)
-- type: auto-detect based on project (frontend/backend)
-- testScope: {testScope}
-
-CAPTURE: bootstrap result
-
-### 6.2 Generate and Execute Tests
-
-CALL MCP TOOL: `testsprite_generate_code_and_execute`
-PARAMETERS:
-- projectName: from package.json "name" or directory name
-- projectPath: current working directory (absolute path)
-- testIds: [] (run all)
-
-CAPTURE: execution result
+→ Go to Step 5 (Results)
 
-### 6.3 Parse TestSprite Results
-
-READ: `testsprite_tests/test_results.json`
-
-EXTRACT:
-- {passed}: count of passed tests
-- {failed}: count of failed tests
-- {coverage}: coverage percentage if available
-
-IF {failed} == 0:
-  {testStatus} = "passed"
-ELSE:
-  {testStatus} = "failed"
-
-### Handle Fix Mode with TestSprite
-IF {testMode} == "fix" AND {testStatus} == "failed":
-  OUTPUT: "🔧 Running auto-healing..."
-
-  CALL MCP TOOL: `testsprite_rerun_tests`
-  PARAMETERS:
-    - projectPath: current working directory
-
-  READ: `testsprite_tests/test_results.json` (updated)
-  Re-parse results
-
-→ Go to Step 8 (Results)
-
-## Step 7: (Reserved for future expansion)
-
-## Step 8: Results & Response
+## Step 5: Results & Response
 
 ### Log to Memory
 
@@ -240,7 +108,7 @@ SET: {now} = GetTimestamp()
 
 APPEND to `{memoryPath}`:
 ```json
-{"ts":"{now}","type":"test_run","tool":"{runner|testsprite}","passed":{passed},"failed":{failed},"mode":"{testMode}"}
+{"ts":"{now}","type":"test_run","tool":"{runner}","passed":{passed},"failed":{failed},"mode":"{testMode}"}
 ```
 
 ### Check Blocking Mode
@@ -268,10 +136,7 @@ IF {testStatus} == "passed":
 IF {testStatus} == "failed":
   OUTPUT: "❌ {failed} tests failing"
   OUTPUT: ""
-  IF {useTestSprite}:
-    OUTPUT: "📋 Report: testsprite_tests/TestSprite_MCP_Test_Report.html"
-  ELSE:
-    OUTPUT: "{testOutput}"  # Show relevant failure output
+  OUTPUT: "{testOutput}"  # Show relevant failure output
   OUTPUT: ""
   OUTPUT: "💡 Auto-fix: /p:test fix"
   OUTPUT: ""
@@ -282,11 +147,8 @@ IF {testStatus} == "failed":
 | Error | Response | Action |
 |-------|----------|--------|
 | No project | "No prjct project" | STOP |
-| No test runner | "No test runner detected" | Suggest /p:test ai |
-| TestSprite API key missing | Show options | Fallback to native (non-blocking) |
-| TestSprite dismissed | Skip silently | Use native tests |
+| No test runner | "No test runner detected" | Show setup options |
 | Tests timeout | "Tests timed out" | Suggest increasing timeout |
-| MCP tool not available | "TestSprite not configured" | Fallback to native tests |
 
 ## Examples
 
@@ -303,13 +165,13 @@ IF {testStatus} == "failed":
 🎯 Next: /p:ship
 ```
 
-### Example 2: AI Tests with Failures
+### Example 2: Tests with Failures
 ```
-🤖 Running AI-powered tests with TestSprite...
+🧪 Running tests with npm...
 
 ❌ 3 tests failing
 
-📋 Report: testsprite_tests/TestSprite_MCP_Test_Report.html
+[test output here]
 
 💡 Auto-fix: /p:test fix
 
@@ -324,32 +186,3 @@ IF {testStatus} == "failed":
 
 Fix failing tests or run without --blocking flag.
 ```
-
-### Example 4: First Time TestSprite (No API Key)
-```
-🤖 TestSprite AI Testing (Optional)
-
-TestSprite can generate and run AI-powered tests.
-Free API key at: https://testsprite.com/dashboard/api-keys
-
-Options:
-1. Enter API key
-2. Skip for now
-3. Remind me in a week
-4. Never ask again
-
-> 2
-
-⏭️ Skipping TestSprite. Using native tests.
-
-🧪 Running tests with npm...
-...
-```
-
-### Example 5: TestSprite Snoozed
-```
-ℹ️ TestSprite reminder snoozed. Using native tests.
-
-🧪 Running tests with npm...
-...
-```

package/templates/mcp-config.json

@@ -4,14 +4,6 @@
       "command": "npx",
       "args": ["-y", "@upstash/context7-mcp@latest"],
       "description": "Library documentation lookup - use for framework/library docs"
-    },
-    "testsprite": {
-      "command": "npx",
-      "args": ["-y", "@testsprite/testsprite-mcp@latest"],
-      "env": {
-        "API_KEY": "${TESTSPRITE_API_KEY}"
-      },
-      "description": "AI-powered test generation and execution"
     }
   },
   "usage": {
@@ -30,34 +22,6 @@
         "resolve-library-id('nextjs') → get-library-docs('/vercel/next.js', 'app router')",
         "resolve-library-id('tailwindcss') → get-library-docs('/tailwindlabs/tailwindcss', 'configuration')"
       ]
-    },
-    "testsprite": {
-      "when": [
-        "User runs /p:test ai or /p:test --testsprite",
-        "Project needs AI-generated tests for frontend or backend",
-        "Auto-fixing flaky or broken tests",
-        "Generating comprehensive test coverage"
-      ],
-      "tools": [
-        "testsprite_bootstrap_tests: Initialize testing environment",
-        "testsprite_generate_code_summary: Analyze project architecture",
-        "testsprite_generate_standardized_prd: Generate structured requirements",
-        "testsprite_generate_frontend_test_plan: Create frontend test plan",
-        "testsprite_generate_backend_test_plan: Create backend test plan",
-        "testsprite_generate_code_and_execute: Generate and run tests",
-        "testsprite_rerun_tests: Re-run tests with auto-healing"
-      ],
-      "examples": [
-        "testsprite_bootstrap_tests(projectPath, 'frontend', 'diff')",
-        "testsprite_generate_code_and_execute(projectName, projectPath)"
-      ],
-      "apiKeySetup": "Get free API key at https://testsprite.com/dashboard/api-keys",
-      "configFields": {
-        "testspriteApiKey": "API key (optional)",
-        "testspriteSkip": "'never' | 'later' | null",
-        "testspriteSkipUntil": "ISO timestamp for snooze"
-      },
-      "nonBlocking": true
     }
   }
 }