myaidev-method 0.3.2 → 0.3.4

package/src/cli/commands/addon.js

@@ -17,16 +17,24 @@ import inquirer from 'inquirer';
 import fs from 'fs-extra';
 import path from 'path';
 import os from 'os';
-import { execSync } from 'child_process';
+
 import matter from 'gray-matter';
 import { isAuthenticated, getAuthToken, getAuthData, authFetch } from '../../lib/auth-helper.js';
 import { validateSkill, generateSlug } from '../../lib/skill-validator.js';
 
 const API_BASE = 'https://dev.myai1.ai/api/skills';
 const MARKETPLACE_REPO = 'myaione/myaidev-marketplace';
+const FETCH_TIMEOUT_MS = 10000;
 
 // ── Helpers ─────────────────────────────────────────────────────────────
 
+function fetchWithTimeout(url, options = {}, timeoutMs = FETCH_TIMEOUT_MS) {
+  const controller = new AbortController();
+  const timeout = setTimeout(() => controller.abort(), timeoutMs);
+  return fetch(url, { ...options, signal: controller.signal })
+    .finally(() => clearTimeout(timeout));
+}
+
 function projectSkillsDir() {
   return path.join(process.cwd(), '.claude', 'skills');
 }
@@ -57,6 +65,7 @@ async function requireAuth() {
 export function registerAddonCommand(program) {
   const addon = program
     .command('addon [name]')
+    .enablePositionalOptions()
     .description('Install a skill from the MyAIDev marketplace')
     .option('--project', 'Install to project .claude/skills/')
     .option('--global', 'Install to global ~/.claude/skills/')
@@ -68,6 +77,16 @@ export function registerAddonCommand(program) {
       await installSkill(name, options);
     });
 
+  // ── addon install ───────────────────────────────────────────────────
+  addon
+    .command('install <name>')
+    .description('Install a skill from the marketplace')
+    .option('--project', 'Install to project .claude/skills/')
+    .option('--global', 'Install to global ~/.claude/skills/')
+    .action(async (name, opts) => {
+      await installSkill(name, opts);
+    });
+
   // ── addon list ──────────────────────────────────────────────────────
   addon
     .command('list')
@@ -154,7 +173,7 @@ async function installSkill(name, options) {
 
   try {
     // Search for skill
-    const res = await fetch(`${API_BASE}?search=${encodeURIComponent(name)}&limit=5`);
+    const res = await fetchWithTimeout(`${API_BASE}?search=${encodeURIComponent(name)}&limit=5`);
     if (!res.ok) throw new Error(`API error: ${res.status}`);
     const body = await res.json();
     const skills = body.skills || [];
@@ -196,11 +215,12 @@ async function installSkill(name, options) {
       const { location } = await inquirer.prompt([{
         type: 'list',
         name: 'location',
-        message: 'Install location:',
+        message: 'Where do you want to install this skill?',
         choices: [
-          { name: `Project  ${chalk.gray('.claude/skills/')}`, value: 'project' },
-          { name: `Global   ${chalk.gray('~/.claude/skills/')}`, value: 'global' },
+          { name: `Project ${chalk.gray('— .claude/skills/ (this project only)')}`, value: 'project' },
+          { name: `Global  ${chalk.gray('— ~/.claude/skills/ (available everywhere)')}`, value: 'global' },
         ],
+        default: 'project',
       }]);
       targetDir = location === 'project' ? projectSkillsDir() : globalSkillsDir();
     }
@@ -210,14 +230,23 @@ async function installSkill(name, options) {
 
     // Download
     const dlSpinner = ora(`Downloading ${chalk.cyan(skill.name)}...`).start();
-    const dlRes = await fetch(`${API_BASE}/${skill.id}/download`);
+    const dlRes = await fetchWithTimeout(`${API_BASE}/${skill.id}/download`);
     if (!dlRes.ok) throw new Error(`Download failed: ${dlRes.status}`);
-    const dlBody = await dlRes.json();
 
-    if (!dlBody.content) throw new Error('Skill content is empty');
+    // Handle both JSON and raw markdown responses
+    const contentType = dlRes.headers.get('content-type') || '';
+    let content;
+    if (contentType.includes('application/json')) {
+      const dlBody = await dlRes.json();
+      content = dlBody.content;
+    } else {
+      content = await dlRes.text();
+    }
+
+    if (!content) throw new Error('Skill content is empty');
 
     await fs.ensureDir(installPath);
-    await fs.writeFile(path.join(installPath, 'SKILL.md'), dlBody.content, 'utf8');
+    await fs.writeFile(path.join(installPath, 'SKILL.md'), content, 'utf8');
 
     dlSpinner.succeed(chalk.green(`Installed ${chalk.bold(skill.name)}`));
 
@@ -225,7 +254,13 @@ async function installSkill(name, options) {
     console.log(chalk.gray(`  → ${rel}/SKILL.md`));
     console.log(chalk.cyan('\n  Restart Claude Code to load the new skill.\n'));
   } catch (err) {
-    spinner.fail(chalk.red(`Failed to install: ${err.message}`));
+    if (err.name === 'AbortError') {
+      spinner.fail(chalk.red('Request timed out. Check your internet connection.'));
+    } else if (err.cause?.code === 'ENOTFOUND' || err.message?.includes('ENOTFOUND')) {
+      spinner.fail(chalk.red('Cannot reach marketplace API. Check your internet connection.'));
+    } else {
+      spinner.fail(chalk.red(`Failed to install: ${err.message}`));
+    }
     process.exit(1);
   }
 }
@@ -241,7 +276,7 @@ async function listMarketplace(opts) {
     if (opts.sort) params.set('sort', opts.sort);
     params.set('limit', opts.limit || '20');
 
-    const res = await fetch(`${API_BASE}?${params}`);
+    const res = await fetchWithTimeout(`${API_BASE}?${params}`);
     if (!res.ok) throw new Error(`API error: ${res.status}`);
     const body = await res.json();
     const skills = body.skills || [];
@@ -264,15 +299,22 @@ async function listMarketplace(opts) {
     for (const s of skills) {
       const stars = String(s.stars ?? 0).padStart(3);
       const downloads = String(s.downloads ?? 0).padStart(5);
+      const author = s.authorName || s.author || 'unknown';
       console.log(
-        `  ${chalk.green(pad(s.name, 24))} ${chalk.gray(pad(truncate(s.description, 38), 40))} ${chalk.white(pad(truncate(s.author, 14), 16))} ${chalk.yellow(stars)}  ${chalk.blue(downloads)}`
+        `  ${chalk.green(pad(s.name, 24))} ${chalk.gray(pad(truncate(s.description, 38), 40))} ${chalk.white(pad(truncate(author, 14), 16))} ${chalk.yellow(stars)}  ${chalk.blue(downloads)}`
       );
     }
 
     console.log(chalk.gray('\n  Install: myaidev-method addon <name>'));
     console.log(chalk.gray('  Search:  myaidev-method addon search <query>\n'));
   } catch (err) {
-    spinner.fail(chalk.red(`Failed to fetch marketplace: ${err.message}`));
+    if (err.name === 'AbortError') {
+      spinner.fail(chalk.red('Request timed out. Check your internet connection.'));
+    } else if (err.cause?.code === 'ENOTFOUND' || err.message?.includes('ENOTFOUND')) {
+      spinner.fail(chalk.red('Cannot reach marketplace API. Check your internet connection.'));
+    } else {
+      spinner.fail(chalk.red(`Failed to fetch marketplace: ${err.message}`));
+    }
     process.exit(1);
   }
 }
@@ -284,7 +326,7 @@ async function searchMarketplace(query, opts) {
 
   try {
     const params = new URLSearchParams({ search: query, limit: opts.limit || '10' });
-    const res = await fetch(`${API_BASE}?${params}`);
+    const res = await fetchWithTimeout(`${API_BASE}?${params}`);
     if (!res.ok) throw new Error(`API error: ${res.status}`);
     const body = await res.json();
     const skills = body.skills || [];
@@ -301,19 +343,26 @@ async function searchMarketplace(query, opts) {
     for (const s of skills) {
       const stars = s.stars ?? 0;
       const downloads = s.downloads ?? 0;
+      const author = s.authorName || s.author || 'unknown';
       // Highlight matching parts of the name
       const highlighted = s.name.replace(
         new RegExp(`(${query.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')})`, 'gi'),
         (m) => chalk.yellow.bold(m)
       );
 
-      console.log(`  ${chalk.green('●')} ${highlighted} ${chalk.gray(`by ${s.author}`)}`);
+      console.log(`  ${chalk.green('●')} ${highlighted} ${chalk.gray(`by ${author}`)}`);
       console.log(`    ${chalk.gray(truncate(s.description, 70))}`);
       console.log(`    ${chalk.yellow(`⭐ ${stars}`)}  ${chalk.blue(`⬇ ${downloads}`)}  ${chalk.gray(`Install: myaidev-method addon ${s.slug || s.name.toLowerCase().replace(/\s+/g, '-')}`)}`);
       console.log('');
     }
   } catch (err) {
-    spinner.fail(chalk.red(`Search failed: ${err.message}`));
+    if (err.name === 'AbortError') {
+      spinner.fail(chalk.red('Request timed out. Check your internet connection.'));
+    } else if (err.cause?.code === 'ENOTFOUND' || err.message?.includes('ENOTFOUND')) {
+      spinner.fail(chalk.red('Cannot reach marketplace API. Check your internet connection.'));
+    } else {
+      spinner.fail(chalk.red(`Search failed: ${err.message}`));
+    }
     process.exit(1);
   }
 }
@@ -648,26 +697,10 @@ async function scaffoldSkill() {
   return targetDir;
 }
 
-function ghAvailable() {
-  try {
-    execSync('gh auth status', { stdio: 'pipe' });
-    return true;
-  } catch {
-    return false;
-  }
-}
-
 async function submitSkill(opts) {
-  // Step 1: Auth checks
+  // Step 1: Auth check
   if (!(await requireAuth())) return;
 
-  if (!ghAvailable()) {
-    console.log(chalk.red('\n✗ GitHub CLI (gh) is required for submissions.'));
-    console.log(chalk.gray('  Install: https://cli.github.com'));
-    console.log(chalk.gray('  Then: gh auth login\n'));
-    return;
-  }
-
   // Step 2: Detect or scaffold SKILL.md
   let targetPath = opts.dir ? path.resolve(opts.dir) : process.cwd();
   let skillDir = targetPath;
@@ -699,14 +732,13 @@ async function submitSkill(opts) {
 
     const scaffoldedDir = await scaffoldSkill();
     if (!scaffoldedDir) return;
-    // User needs to edit the scaffold first
     return;
   }
 
-  // Step 3: Validate
+  // Step 3: Local validation
   const spinner = ora('Validating skill...').start();
   const result = await validateSkill(skillDir, {
-    checkDuplicates: true,
+    checkDuplicates: false,
     apiBase: API_BASE,
   });
   spinner.stop();
@@ -729,11 +761,11 @@ async function submitSkill(opts) {
   }
 
   // Step 4: Show summary and confirm
-  const slug = result.info.slug;
   console.log(chalk.cyan('\n📤 Submit to MyAIDev Marketplace\n'));
   console.log(`  ${chalk.gray('Name:')}        ${chalk.white.bold(result.info.name)}`);
-  console.log(`  ${chalk.gray('Slug:')}        ${chalk.white(slug)}`);
+  console.log(`  ${chalk.gray('Slug:')}        ${chalk.white(result.info.slug)}`);
   console.log(`  ${chalk.gray('Description:')} ${chalk.white(truncate(result.info.description, 60))}`);
+  console.log(`  ${chalk.gray('Category:')}    ${result.info.category || 'general'}`);
   console.log(`  ${chalk.gray('Sections:')}    ${result.info.sectionCount}`);
   console.log('');
 
@@ -750,116 +782,75 @@ async function submitSkill(opts) {
     }
   }
 
-  // Step 5: Fork, branch, commit, PR
-  const prSpinner = ora('Creating pull request...').start();
-  const tmpDir = path.join(os.tmpdir(), `myaidev-submit-${Date.now()}`);
+  // Step 5: Submit via API
+  const submitSpinner = ora('Submitting skill for review...').start();
 
   try {
-    // Fork and clone
-    try {
-      execSync(`gh repo fork ${MARKETPLACE_REPO} --clone=false`, { stdio: 'pipe' });
-    } catch {
-      // Fork may already exist, that's fine
-    }
-
-    // Get the user's GitHub username
-    const ghUser = execSync('gh api user --jq .login', { encoding: 'utf8' }).trim();
-
-    // Clone the user's fork
-    execSync(`gh repo clone ${ghUser}/myaidev-marketplace "${tmpDir}" -- --depth 1`, { stdio: 'pipe' });
-
-    // Create branch
-    const branch = `skill/${slug}`;
-    execSync(`git -C "${tmpDir}" checkout -b ${branch}`, { stdio: 'pipe' });
-
-    // Copy skill files
-    const destDir = path.join(tmpDir, 'skills', slug);
-    await fs.ensureDir(destDir);
-    await fs.copy(skillDir, destDir, {
-      filter: (src) => {
-        const ext = path.extname(src).toLowerCase();
-        const base = path.basename(src);
-        // Copy directories and allowed file types
-        if (fs.statSync(src).isDirectory()) return true;
-        return ['.md', '.json', '.yaml', '.yml', '.sh', '.js', '.ts', '.py', '.txt'].includes(ext) || base === '.gitkeep';
-      },
+    const content = await fs.readFile(skillFile, 'utf8');
+
+    const res = await authFetch(`${API_BASE}/submissions`, {
+      method: 'POST',
+      body: JSON.stringify({
+        content,
+        category: result.info.category || undefined,
+      }),
     });
 
-    // Commit
-    execSync(`git -C "${tmpDir}" add skills/${slug}`, { stdio: 'pipe' });
-    execSync(`git -C "${tmpDir}" commit -m "feat: add skill ${result.info.name}"`, { stdio: 'pipe' });
-
-    // Push
-    execSync(`git -C "${tmpDir}" push -u origin ${branch}`, { stdio: 'pipe' });
-
-    // Create PR
-    const prBody = [
-      `## Skill Submission: ${result.info.name}`,
-      '',
-      `**Description**: ${result.info.description || 'N/A'}`,
-      `**Category**: ${result.info.category || 'general'}`,
-      '',
-      '### Checklist',
-      '- [x] SKILL.md has valid YAML frontmatter',
-      '- [x] Passes local validation',
-      '- [ ] Tested locally with Claude Code',
-    ].join('\n');
-
-    const prOutput = execSync(
-      `gh pr create --repo ${MARKETPLACE_REPO} --head ${ghUser}:${branch} --title "feat: add skill ${result.info.name}" --body "${prBody.replace(/"/g, '\\"')}"`,
-      { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] }
-    ).trim();
-
-    prSpinner.succeed(chalk.green('Pull request created!'));
-
-    // Extract PR URL from output
-    const prUrl = prOutput.match(/https:\/\/github\.com\S+/)?.[0] || prOutput;
-    console.log(chalk.gray(`  PR: ${prUrl}`));
-
-    // Step 6: Record submission
-    try {
-      await authFetch(`${API_BASE}/submissions`, {
-        method: 'POST',
-        body: JSON.stringify({
-          skillName: result.info.name,
-          slug,
-          description: result.info.description || '',
-          prUrl,
-        }),
-      });
-    } catch {
-      // Non-fatal: submission tracking is optional
-    }
-
-    console.log(chalk.cyan('\n  Status: awaiting review'));
-    console.log(chalk.gray('  Check: myaidev-method addon status\n'));
+    if (!res.ok) {
+      const body = await res.json().catch(() => ({}));
+      if (body.validation?.errors?.length) {
+        submitSpinner.fail(chalk.red('Server validation failed:'));
+        for (const err of body.validation.errors) {
+          console.log(chalk.red(`  • ${err}`));
+        }
+        return;
+      }
+      throw new Error(body.error || `API error: ${res.status}`);
+    }
+
+    const body = await res.json();
+    const submission = body.submission;
+
+    submitSpinner.succeed(chalk.green('Skill submitted for review!'));
+
+    console.log('');
+    console.log(`  ${chalk.gray('Submission ID:')} ${chalk.white(submission.id)}`);
+    console.log(`  ${chalk.gray('Status:')}        ${chalk.yellow(submission.status)}`);
+    console.log('');
+    console.log(chalk.gray('  AI analysis will run automatically.'));
+    console.log(chalk.gray('  An admin will review and approve your skill.'));
+    console.log(chalk.gray(`  Check status: ${chalk.white('myaidev-method addon status')}\n`));
   } catch (err) {
-    prSpinner.fail(chalk.red(`Submission failed: ${err.message}`));
+    submitSpinner.fail(chalk.red(`Submission failed: ${err.message}`));
     process.exit(1);
-  } finally {
-    // Clean up temp directory
-    await fs.remove(tmpDir).catch(() => {});
   }
 }
 
 // ── Status Command ─────────────────────────────────────────────────────
 
-async function checkStatus(name) {
+async function checkStatus(nameOrId) {
   if (!(await requireAuth())) return;
 
   const spinner = ora('Fetching submission status...').start();
 
   try {
-    const params = name ? `?name=${encodeURIComponent(name)}` : '';
-    const res = await authFetch(`${API_BASE}/submissions${params}`);
-
-    if (!res.ok) {
-      throw new Error(`API error: ${res.status}`);
+    // If it looks like a UUID, fetch by ID
+    const isUuid = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(nameOrId || '');
+
+    let submissions;
+    if (isUuid) {
+      const res = await authFetch(`${API_BASE}/submissions/${nameOrId}`);
+      if (!res.ok) throw new Error(`API error: ${res.status}`);
+      const body = await res.json();
+      submissions = body.submission ? [body.submission] : [];
+    } else {
+      const params = nameOrId ? `?name=${encodeURIComponent(nameOrId)}` : '';
+      const res = await authFetch(`${API_BASE}/submissions${params}`);
+      if (!res.ok) throw new Error(`API error: ${res.status}`);
+      const body = await res.json();
+      submissions = body.submissions || [];
     }
 
-    const body = await res.json();
-    const submissions = body.submissions || [];
-
     spinner.stop();
 
     if (submissions.length === 0) {
@@ -873,16 +864,36 @@ async function checkStatus(name) {
     for (const sub of submissions) {
       const statusColor = {
         pending_review: chalk.yellow,
+        ai_analyzing: chalk.magenta,
+        ready_for_review: chalk.blue,
         approved: chalk.green,
-        merged: chalk.green.bold,
         rejected: chalk.red,
         changes_requested: chalk.yellow,
       }[sub.status] || chalk.gray;
 
-      console.log(`  ${chalk.white.bold(sub.skillName)} ${statusColor(`[${sub.status}]`)}`);
-      if (sub.prUrl) console.log(chalk.gray(`    PR: ${sub.prUrl}`));
-      if (sub.reviewNotes) console.log(chalk.gray(`    Notes: ${sub.reviewNotes}`));
-      console.log(chalk.gray(`    Submitted: ${new Date(sub.createdAt).toLocaleDateString()}`));
+      const statusLabel = {
+        pending_review: 'pending review',
+        ai_analyzing: 'AI analyzing...',
+        ready_for_review: 'ready for review',
+        approved: 'approved',
+        rejected: 'rejected',
+        changes_requested: 'changes requested',
+      }[sub.status] || sub.status;
+
+      console.log(`  ${chalk.white.bold(sub.skillName)} ${statusColor(`[${statusLabel}]`)}`);
+
+      // Show AI score if available
+      if (sub.aiAnalysis?.overallScore != null) {
+        const score = sub.aiAnalysis.overallScore;
+        const scoreColor = score >= 75 ? chalk.green : score >= 50 ? chalk.yellow : chalk.red;
+        console.log(`    ${chalk.gray('AI Score:')} ${scoreColor(`${score}/100`)} ${chalk.gray(`(${sub.aiAnalysis.recommendation})`)}`);
+      } else if (sub.aiAnalysisStatus === 'running') {
+        console.log(`    ${chalk.gray('AI Score:')} ${chalk.magenta('analyzing...')}`);
+      }
+
+      if (sub.reviewNotes) console.log(`    ${chalk.gray('Review:')} ${sub.reviewNotes}`);
+      console.log(`    ${chalk.gray('Submitted:')} ${new Date(sub.createdAt).toLocaleDateString()}`);
+      if (sub.reviewedAt) console.log(`    ${chalk.gray('Reviewed:')} ${new Date(sub.reviewedAt).toLocaleDateString()}`);
       console.log('');
     }
   } catch (err) {

package/src/cli/commands/auth.js

@@ -9,6 +9,14 @@ import { getAuthData, saveAuth, clearAuth, isAuthenticated } from '../../lib/aut
 
 const API_BASE = 'https://dev.myai1.ai';
 
+function ensureHttpsUrl(url) {
+  const value = String(url ?? '').trim();
+  if (!value) return value;
+  if (/^https:\/\//i.test(value)) return value;
+  if (/^http:\/\//i.test(value)) return value.replace(/^http:\/\//i, 'https://');
+  return `https://${value.replace(/^\/+/, '')}`;
+}
+
 export function registerAuthCommands(program) {
   // ── login ───────────────────────────────────────────────────────────
   program
@@ -31,7 +39,7 @@ export function registerAuthCommands(program) {
         if (!initRes.ok) throw new Error(`Server returned ${initRes.status}`);
         const initData = await initRes.json();
         sessionId = initData.sessionId;
-        authUrl = initData.authUrl;
+        authUrl = ensureHttpsUrl(initData.authUrl);
       } catch (err) {
         console.log(chalk.red('  ✗ Failed to connect to MyAIDev marketplace.'));
         console.log(chalk.gray(`  ${err.message}\n`));

package/src/config/workflows.js

@@ -74,12 +74,17 @@ const WORKFLOWS = {
     name: 'Development Workflow',
     description: 'SPARC methodology for systematic software development',
     skills: [
-      'sparc-architect',
-      'sparc-coder',
-      'sparc-tester',
-      'sparc-reviewer',
-      'sparc-documenter',
-      'sparc-workflow'
+      'myaidev-workflow',
+      'myaidev-architect',
+      'myaidev-coder',
+      'myaidev-tester',
+      'myaidev-reviewer',
+      'myaidev-documenter',
+      'myaidev-analyze',
+      'myaidev-debug',
+      'myaidev-refactor',
+      'myaidev-performance',
+      'myaidev-migrate'
     ],
     scripts: [],
     libs: [],

package/src/lib/ascii-banner.js

@@ -77,10 +77,10 @@ ${chalk.bold.hex('#808080')('─────────────────
  */
 export function getInitSuccessMessage(cliType) {
   const commands = cliType === 'claude'
-    ? '  • View README: Press Ctrl+Shift+P → "Toggle Documentation"\n  • Run workflow: /myai-sparc-workflow "Build your feature"'
+    ? '  • View README: Press Ctrl+Shift+P → "Toggle Documentation"\n  • Run workflow: /myaidev-workflow "Build your feature"'
     : cliType === 'gemini'
-    ? '  • View commands: ls .gemini/commands/\n  • Run workflow: /myai-sparc-workflow "Build your feature"'
-    : '  • View commands: ls .opencode/commands/\n  • Run workflow: /myai-sparc-workflow "Build your feature"';
+    ? '  • View commands: ls .gemini/commands/\n  • Run workflow: /myaidev-workflow "Build your feature"'
+    : '  • View commands: ls .opencode/commands/\n  • Run workflow: /myaidev-workflow "Build your feature"';
 
   return `
 ${chalk.green.bold('✨ Installation Complete!')}