@codihaus/claude-skills 1.6.28 → 1.7.0

package/skills/dev-test/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: dev-test
-description: Automated testing after implementation using Playwright
-version: 1.0.0
+description: Automated UI testing using Playwright browser tools. Use when: (1) After completing feature implementation, (2) After fixing bugs, (3) Before code review, (4) Verifying acceptance criteria from specs
+version: 1.1.0
 ---
 
 # /dev-test - Implementation Testing
@@ -113,134 +113,13 @@ Test report showing whether implementation works correctly.
 
 ## Test Patterns
 
-Available as reference in original SKILL.md:
-- Form submission
-- Navigation flow
-- API response validation
-- Error state testing
+See `references/test-patterns.md` for code examples:
+- Form submission, navigation flow, API response validation, error state testing
+- Test data conventions
+- Common issues and solutions (page not loading, element not found, intermittent failures)
 
 Use Playwright MCP tools to interact with browser.
 
-## Test Patterns
-
-### Form Submission
-
-```typescript
-// 1. Fill form
-await browser_fill_form({
-  fields: [
-    { name: 'Email', type: 'textbox', ref: 'input-email', value: 'test@test.com' },
-    { name: 'Password', type: 'textbox', ref: 'input-password', value: 'password123' }
-  ]
-});
-
-// 2. Submit and wait
-await browser_click({ element: 'Submit button', ref: 'btn-submit' });
-await browser_wait_for({ text: 'Success' });
-
-// 3. Check for errors
-const errors = await browser_console_messages({ level: 'error' });
-```
-
-### Navigation Flow
-
-```typescript
-// 1. Start at page A
-await browser_navigate({ url: '/dashboard' });
-
-// 2. Click to page B
-await browser_click({ element: 'Settings link', ref: 'nav-settings' });
-await browser_wait_for({ text: 'Settings' });
-
-// 3. Verify correct page
-const snapshot = await browser_snapshot({});
-// Check snapshot contains expected elements
-```
-
-### API Response Validation
-
-```typescript
-// 1. Trigger API call (via form submit or button)
-await browser_click({ element: 'Load data', ref: 'btn-load' });
-await browser_wait_for({ time: 2 });
-
-// 2. Check network requests
-const requests = await browser_network_requests({});
-
-// 3. Find the API call
-const apiCall = requests.find(r => r.url.includes('/api/data'));
-
-// 4. Validate
-if (apiCall.status >= 400) {
-  // Report error
-}
-```
-
-### Error State Testing
-
-```typescript
-// Test error handling by submitting invalid data
-await browser_type({
-  element: 'email',
-  ref: 'input-email',
-  text: 'invalid-email'  // No @ symbol
-});
-
-await browser_click({ element: 'Submit', ref: 'btn-submit' });
-
-// Verify error message appears
-const snapshot = await browser_snapshot({});
-// Check for validation error in snapshot
-```
-
-## Test Data
-
-Use predictable test data:
-
-```typescript
-const TEST_DATA = {
-  validEmail: 'test@example.com',
-  validPassword: 'Test123!@#',
-  invalidEmail: 'not-an-email',
-  shortPassword: '123',
-};
-```
-
-For existing data, check spec or use API to fetch valid IDs.
-
-## Common Issues & Solutions
-
-### Issue: Page Not Loading
-
-```
-Symptom: Navigation fails or times out
-Check:
-1. Is dev server running? (npm run dev)
-2. Correct port? (3000, 3001, etc.)
-3. Any build errors?
-```
-
-### Issue: Element Not Found
-
-```
-Symptom: Click/type fails with "element not found"
-Check:
-1. Take snapshot to see current state
-2. Is element conditionally rendered?
-3. Has ref changed since snapshot?
-4. Is page still loading?
-```
-
-### Issue: Intermittent Failures
-
-```
-Symptom: Test passes sometimes, fails sometimes
-Check:
-1. Add wait_for before interactions
-2. Check for race conditions
-3. Increase timeout for slow APIs
-```
-
 ## Integration with /dev-coding
 
 When `/dev-coding` completes:

package/skills/dev-test/references/test-patterns.md

@@ -0,0 +1,119 @@
+# Test Patterns & Common Issues
+
+## Form Submission
+
+```typescript
+// 1. Fill form
+await browser_fill_form({
+  fields: [
+    { name: 'Email', type: 'textbox', ref: 'input-email', value: 'test@test.com' },
+    { name: 'Password', type: 'textbox', ref: 'input-password', value: 'password123' }
+  ]
+});
+
+// 2. Submit and wait
+await browser_click({ element: 'Submit button', ref: 'btn-submit' });
+await browser_wait_for({ text: 'Success' });
+
+// 3. Check for errors
+const errors = await browser_console_messages({ level: 'error' });
+```
+
+## Navigation Flow
+
+```typescript
+// 1. Start at page A
+await browser_navigate({ url: '/dashboard' });
+
+// 2. Click to page B
+await browser_click({ element: 'Settings link', ref: 'nav-settings' });
+await browser_wait_for({ text: 'Settings' });
+
+// 3. Verify correct page
+const snapshot = await browser_snapshot({});
+// Check snapshot contains expected elements
+```
+
+## API Response Validation
+
+```typescript
+// 1. Trigger API call (via form submit or button)
+await browser_click({ element: 'Load data', ref: 'btn-load' });
+await browser_wait_for({ time: 2 });
+
+// 2. Check network requests
+const requests = await browser_network_requests({});
+
+// 3. Find the API call
+const apiCall = requests.find(r => r.url.includes('/api/data'));
+
+// 4. Validate
+if (apiCall.status >= 400) {
+  // Report error
+}
+```
+
+## Error State Testing
+
+```typescript
+// Test error handling by submitting invalid data
+await browser_type({
+  element: 'email',
+  ref: 'input-email',
+  text: 'invalid-email'  // No @ symbol
+});
+
+await browser_click({ element: 'Submit', ref: 'btn-submit' });
+
+// Verify error message appears
+const snapshot = await browser_snapshot({});
+// Check for validation error in snapshot
+```
+
+## Test Data
+
+Use predictable test data:
+
+```typescript
+const TEST_DATA = {
+  validEmail: 'test@example.com',
+  validPassword: 'Test123!@#',
+  invalidEmail: 'not-an-email',
+  shortPassword: '123',
+};
+```
+
+For existing data, check spec or use API to fetch valid IDs.
+
+## Common Issues & Solutions
+
+### Page Not Loading
+
+```
+Symptom: Navigation fails or times out
+Check:
+1. Is dev server running? (npm run dev)
+2. Correct port? (3000, 3001, etc.)
+3. Any build errors?
+```
+
+### Element Not Found
+
+```
+Symptom: Click/type fails with "element not found"
+Check:
+1. Take snapshot to see current state
+2. Is element conditionally rendered?
+3. Has ref changed since snapshot?
+4. Is page still loading?
+```
+
+### Intermittent Failures
+
+```
+Symptom: Test passes sometimes, fails sometimes
+Check:
+1. Add wait_for before interactions
+2. Check for race conditions
+3. Increase timeout for slow APIs
+```

package/src/commands/init.js

@@ -9,6 +9,7 @@ import ora from 'ora';
 import inquirer from 'inquirer';
 import path from 'path';
 import fs from 'fs-extra';
+import { execSync } from 'child_process';
 
 import {
   checkGlobalDeps,
@@ -24,7 +25,9 @@ import {
   getInstalledSkills,
   copySkillsToProject,
   copyKnowledgeToProject,
-  copyScriptsToProject
+  copyScriptsToProject,
+  checkForUpdates,
+  cleanupOldSkills
 } from '../utils/skills.js';
 import {
   setupSettings,
@@ -124,8 +127,88 @@ export async function init(options) {
     }
 
     if (action === 'update') {
-      // TODO: Implement update logic
-      console.log(chalk.cyan('Updating skills...'));
+      console.log('');
+
+      // Check for CLI package updates first
+      const packageSpinner = ora('Checking npm for latest CLI version...').start();
+      try {
+        const currentVersion = await getCurrentPackageVersion();
+        const latestVersion = await getLatestNpmVersion();
+
+        if (currentVersion !== latestVersion) {
+          packageSpinner.info(`CLI update available: ${currentVersion} → ${latestVersion}`);
+
+          // Auto-update the CLI package
+          const updateSpinner = ora('Updating CLI package...').start();
+          try {
+            execSync('npm install -g @codihaus/claude-skills@latest', {
+              encoding: 'utf8',
+              stdio: 'pipe'
+            });
+            updateSpinner.succeed(`CLI package updated to ${latestVersion}`);
+            console.log(chalk.green(`\n✅ Please re-run init to update skills with the new CLI version.\n`));
+            process.exit(0);
+          } catch (updateErr) {
+            updateSpinner.fail('Failed to update CLI package');
+            console.log(chalk.yellow(`\nTo update manually, run:\n  npm install -g @codihaus/claude-skills@latest\n`));
+          }
+        } else {
+          packageSpinner.succeed(`CLI package is up to date (${currentVersion})`);
+        }
+      } catch (e) {
+        packageSpinner.warn('Could not check for CLI updates');
+      }
+
+      console.log('');
+
+      // Check for skill updates
+      const skillSpinner = ora('Checking for skill updates...').start();
+      const updates = await checkForUpdates(projectPath);
+      skillSpinner.stop();
+
+      if (updates.length === 0) {
+        console.log(chalk.green('✅ All skills are up to date!\n'));
+        process.exit(0);
+      }
+
+      console.log(chalk.cyan(`Found ${updates.length} skill update(s) available:\n`));
+
+      for (const update of updates) {
+        console.log(`  ${update.name}`);
+        console.log(chalk.gray(`    ${update.current} → ${update.available}`));
+        console.log('');
+      }
+
+      // Apply skill updates
+      const updateSkillsSpinner = ora('Updating skills...').start();
+
+      try {
+        const skillNames = updates.map(u => u.name);
+
+        // Clean up old files first
+        updateSkillsSpinner.text = 'Cleaning up old skill files...';
+        await cleanupOldSkills(projectPath, skillNames);
+
+        // Copy new versions
+        updateSkillsSpinner.text = 'Installing updated skills...';
+        const { copied, errors } = await copySkillsToProject(projectPath, skillNames);
+
+        if (errors.length > 0) {
+          updateSkillsSpinner.warn(`Updated ${copied.length} skills with ${errors.length} errors`);
+          for (const err of errors) {
+            console.log(chalk.red(`  ✗ ${err.name}: ${err.error}`));
+          }
+        } else {
+          updateSkillsSpinner.succeed(`Updated ${copied.length} skills`);
+        }
+      } catch (e) {
+        updateSkillsSpinner.fail('Update failed');
+        console.error(chalk.red(e.message));
+        process.exit(1);
+      }
+
+      console.log(chalk.green('\n✅ Update complete!\n'));
+      process.exit(0);
     }
   }
 
@@ -409,3 +492,30 @@ export async function init(options) {
     console.log('');
   }
 }
+
+/**
+ * Get current package version
+ */
+async function getCurrentPackageVersion() {
+  try {
+    const packageJson = await fs.readJson(path.join(import.meta.dirname, '../../package.json'));
+    return packageJson.version;
+  } catch (e) {
+    return 'unknown';
+  }
+}
+
+/**
+ * Get latest version from npm
+ */
+async function getLatestNpmVersion() {
+  try {
+    const result = execSync('npm view @codihaus/claude-skills version', {
+      encoding: 'utf8',
+      stdio: ['pipe', 'pipe', 'ignore']
+    });
+    return result.trim();
+  } catch (e) {
+    throw new Error('Could not fetch latest version from npm');
+  }
+}

package/src/commands/update.js

@@ -25,7 +25,21 @@ export async function update(options) {
 
     if (currentVersion !== latestVersion) {
       packageSpinner.info(`Package update available: ${currentVersion} → ${latestVersion}`);
-      console.log(chalk.yellow(`\nTo update the CLI package, run:\n  npm install -g @codihaus/claude-skills@latest\n`));
+
+      // Auto-update the CLI package
+      const updateSpinner = ora('Updating CLI package...').start();
+      try {
+        execSync('npm install -g @codihaus/claude-skills@latest', {
+          encoding: 'utf8',
+          stdio: 'pipe'
+        });
+        updateSpinner.succeed(`CLI package updated to ${latestVersion}`);
+        console.log(chalk.green(`\n✅ Please re-run this command to update skills with the new CLI version.\n`));
+        process.exit(0);
+      } catch (updateErr) {
+        updateSpinner.fail('Failed to update CLI package');
+        console.log(chalk.yellow(`\nTo update manually, run:\n  npm install -g @codihaus/claude-skills@latest\n`));
+      }
     } else {
       packageSpinner.succeed(`CLI package is up to date (${currentVersion})`);
     }