@su-record/vibe 2.9.18 → 2.9.20

package/dist/infra/lib/ReviewRace.js

@@ -24,90 +24,90 @@ import { isCodexAvailable, isGeminiAvailable } from './llm-availability.js';
 // Constants
 // ============================================================================
 const REVIEW_PROMPTS = {
-    security: `You are a security expert reviewing code changes.
-Identify security vulnerabilities including:
-- SQL injection, XSS, CSRF
-- Authentication/authorization flaws
-- Sensitive data exposure
-- Insecure dependencies
-- OWASP Top 10 issues
-
-For each issue found, respond in this exact JSON format:
-{
-  "issues": [
-    {
-      "id": "SEC-001",
-      "title": "Brief title",
-      "description": "Detailed description",
-      "location": "file.ts:42",
-      "severity": "critical|high|medium|low",
-      "suggestion": "How to fix"
-    }
-  ]
-}
-
+    security: `You are a security expert reviewing code changes.
+Identify security vulnerabilities including:
+- SQL injection, XSS, CSRF
+- Authentication/authorization flaws
+- Sensitive data exposure
+- Insecure dependencies
+- OWASP Top 10 issues
+
+For each issue found, respond in this exact JSON format:
+{
+  "issues": [
+    {
+      "id": "SEC-001",
+      "title": "Brief title",
+      "description": "Detailed description",
+      "location": "file.ts:42",
+      "severity": "critical|high|medium|low",
+      "suggestion": "How to fix"
+    }
+  ]
+}
+
 If no issues found, respond: {"issues": []}`,
-    performance: `You are a performance expert reviewing code changes.
-Identify performance issues including:
-- N+1 queries
-- Memory leaks
-- Unnecessary re-renders
-- Blocking operations
-- Large bundle size impacts
-- Missing caching opportunities
-
-Respond in JSON format:
-{
-  "issues": [
-    {
-      "id": "PERF-001",
-      "title": "Brief title",
-      "description": "Detailed description",
-      "location": "file.ts:42",
-      "severity": "critical|high|medium|low",
-      "suggestion": "How to fix"
-    }
-  ]
+    performance: `You are a performance expert reviewing code changes.
+Identify performance issues including:
+- N+1 queries
+- Memory leaks
+- Unnecessary re-renders
+- Blocking operations
+- Large bundle size impacts
+- Missing caching opportunities
+
+Respond in JSON format:
+{
+  "issues": [
+    {
+      "id": "PERF-001",
+      "title": "Brief title",
+      "description": "Detailed description",
+      "location": "file.ts:42",
+      "severity": "critical|high|medium|low",
+      "suggestion": "How to fix"
+    }
+  ]
 }`,
-    architecture: `You are an architecture expert reviewing code changes.
-Identify architecture issues including:
-- Circular dependencies
-- Layer violations (e.g., UI calling DB directly)
-- Missing abstractions
-- Tight coupling
-- Single Responsibility violations
-- DRY violations
-
+    architecture: `You are an architecture expert reviewing code changes.
+Identify architecture issues including:
+- Circular dependencies
+- Layer violations (e.g., UI calling DB directly)
+- Missing abstractions
+- Tight coupling
+- Single Responsibility violations
+- DRY violations
+
 Respond in JSON format with issues array.`,
-    complexity: `You are a code quality expert reviewing complexity.
-Identify complexity issues including:
-- Functions > 30 lines
-- Nesting depth > 3 levels
-- Cyclomatic complexity > 10
-- Too many parameters (> 5)
-- Long files (> 300 lines)
-
+    complexity: `You are a code quality expert reviewing complexity.
+Identify complexity issues including:
+- Functions > 30 lines
+- Nesting depth > 3 levels
+- Cyclomatic complexity > 10
+- Too many parameters (> 5)
+- Long files (> 300 lines)
+
 Respond in JSON format with issues array.`,
-    'data-integrity': `You are a data integrity expert reviewing code changes.
-Identify data integrity issues including:
-- Missing validations
-- Race conditions
-- Inconsistent state handling
-- Missing error handling
-- Data loss scenarios
-
+    'data-integrity': `You are a data integrity expert reviewing code changes.
+Identify data integrity issues including:
+- Missing validations
+- Race conditions
+- Inconsistent state handling
+- Missing error handling
+- Data loss scenarios
+
 Respond in JSON format with issues array.`,
-    'test-coverage': `You are a testing expert reviewing code changes.
-Identify missing test coverage including:
-- Untested edge cases
-- Missing error case tests
-- Missing integration tests
-- Untested business logic
-
+    'test-coverage': `You are a testing expert reviewing code changes.
+Identify missing test coverage including:
+- Untested edge cases
+- Missing error case tests
+- Missing integration tests
+- Untested business logic
+
 Respond in JSON format with issues array.`,
-    general: `You are a senior code reviewer.
-Review this code for any issues including security, performance, maintainability, and best practices.
-
+    general: `You are a senior code reviewer.
+Review this code for any issues including security, performance, maintainability, and best practices.
+
 Respond in JSON format with issues array.`,
 };
 // ============================================================================
@@ -203,15 +203,15 @@ function selectBestDescription(issues) {
 async function reviewWithGPT(reviewType, code, context) {
     const startTime = Date.now();
     try {
-        const prompt = `${REVIEW_PROMPTS[reviewType]}
-
-Context: ${context || 'Code review'}
-
-Code to review:
-\`\`\`
-${code}
-\`\`\`
-
+        const prompt = `${REVIEW_PROMPTS[reviewType]}
+
+Context: ${context || 'Code review'}
+
+Code to review:
+\`\`\`
+${code}
+\`\`\`
+
 Respond with JSON only.`;
         const response = await gptApi.ask(prompt, {
             model: isCodexAvailable() ? 'gpt-5.3-codex-spark' : 'gpt-5.3-codex',
@@ -239,15 +239,15 @@ Respond with JSON only.`;
 async function reviewWithGemini(reviewType, code, context) {
     const startTime = Date.now();
     try {
-        const prompt = `${REVIEW_PROMPTS[reviewType]}
-
-Context: ${context || 'Code review'}
-
-Code to review:
-\`\`\`
-${code}
-\`\`\`
-
+        const prompt = `${REVIEW_PROMPTS[reviewType]}
+
+Context: ${context || 'Code review'}
+
+Code to review:
+\`\`\`
+${code}
+\`\`\`
+
 Respond with JSON only.`;
         const response = await geminiApi.ask(prompt, {
             model: 'gemini-flash',

package/dist/infra/lib/SkillFrontmatter.js

@@ -198,33 +198,33 @@ export function substituteTemplateVars(template, vars) {
 /**
  * Example skill template
  */
-export const SKILL_TEMPLATE_EXAMPLE = `---
-name: example-skill
-description: "An example skill demonstrating the frontmatter system"
-model: sonnet
-agent: implementer
-argument-hint: "task description"
-user-invocable: true
-triggers: [example, demo, test]
-priority: 50
----
-
-# Example Skill
-
-This skill demonstrates the frontmatter metadata system.
-
-## Arguments
-
-$ARGUMENTS
-
-## Process
-
-1. Parse the arguments
-2. Execute the task
-3. Return the result
-
-## Output
-
-Provide a clear summary of what was done.
+export const SKILL_TEMPLATE_EXAMPLE = `---
+name: example-skill
+description: "An example skill demonstrating the frontmatter system"
+model: sonnet
+agent: implementer
+argument-hint: "task description"
+user-invocable: true
+triggers: [example, demo, test]
+priority: 50
+---
+
+# Example Skill
+
+This skill demonstrates the frontmatter metadata system.
+
+## Arguments
+
+$ARGUMENTS
+
+## Process
+
+1. Parse the arguments
+2. Execute the task
+3. Return the result
+
+## Output
+
+Provide a clear summary of what was done.
 `;
 //# sourceMappingURL=SkillFrontmatter.js.map

package/dist/infra/lib/SkillQualityGate.js

@@ -137,15 +137,15 @@ export function suggestPrincipleFormat(value) {
         return value;
     }
     // 변환 템플릿 제안
-    return `Consider reformulating as:
-
-"When [describe the situation/trigger],
-[describe the action to take]
-because [explain the reasoning/tradeoff]"
-
-Example:
-"When the auth token expires during a request,
-retry the request after refreshing the token (not before)
+    return `Consider reformulating as:
+
+"When [describe the situation/trigger],
+[describe the action to take]
+because [explain the reasoning/tradeoff]"
+
+Example:
+"When the auth token expires during a request,
+retry the request after refreshing the token (not before)
 because refreshing preemptively causes race conditions with concurrent requests"`;
 }
 /**

package/dist/infra/lib/SkillRepository.js

@@ -285,177 +285,177 @@ export function getDefaultSkills(corePathUrl) {
     return [
         {
             id: 'multi-llm-orchestration',
-            content: `---
-name: multi-llm-orchestration
-description: "Multi-LLM research using GPT and Gemini for comprehensive analysis"
-triggers: [gpt, gemini, multi-llm, research, parallel research]
-priority: 80
----
-# Multi-LLM Orchestration
-
-Use multiple LLMs for comprehensive research and validation.
-
-## Usage
-
-\`\`\`bash
-# Via Bash hook (automatic in /vibe.spec)
-node hooks/scripts/llm-orchestrate.js gpt "your prompt"
-node hooks/scripts/llm-orchestrate.js gemini "your prompt"
-\`\`\`
-
-## Setup
-
-\`\`\`bash
-vibe gpt key <key>     # Configure GPT API key
-vibe gemini key <key>  # Configure Gemini API key
-vibe status       # Check current configuration
-\`\`\`
-
-## Best Practices
-
-1. Use GPT for best practices and code review
-2. Use Gemini for documentation and security analysis
-3. Combine results for comprehensive coverage
+            content: `---
+name: multi-llm-orchestration
+description: "Multi-LLM research using GPT and Gemini for comprehensive analysis"
+triggers: [gpt, gemini, multi-llm, research, parallel research]
+priority: 80
+---
+# Multi-LLM Orchestration
+
+Use multiple LLMs for comprehensive research and validation.
+
+## Usage
+
+\`\`\`bash
+# Via Bash hook (automatic in /vibe.spec)
+node hooks/scripts/llm-orchestrate.js gpt "your prompt"
+node hooks/scripts/llm-orchestrate.js gemini "your prompt"
+\`\`\`
+
+## Setup
+
+\`\`\`bash
+vibe gpt key <key>     # Configure GPT API key
+vibe gemini key <key>  # Configure Gemini API key
+vibe status       # Check current configuration
+\`\`\`
+
+## Best Practices
+
+1. Use GPT for best practices and code review
+2. Use Gemini for documentation and security analysis
+3. Combine results for comprehensive coverage
 `,
         },
         {
             id: 'error-recovery',
-            content: `---
-name: error-recovery
-description: "Error recovery patterns and retry strategies"
-triggers: [error, fail, retry, recover, fix]
-priority: 70
----
-# Error Recovery Patterns
-
-## Common Error Types
-
-### Build Errors
-- Check TypeScript compilation errors
-- Verify dependency versions
-- Run \`npm ci\` to clean install
-
-### Test Failures
-- Run failed tests in isolation
-- Check test fixtures/mocks
-- Verify async timing issues
-
-### Runtime Errors
-- Check stack trace carefully
-- Verify environment variables
-- Check external service connectivity
-
-## Retry Strategy
-
-1. First retry: Same action
-2. Second retry: Clean state (cache clear)
-3. Third retry: Alternative approach
-4. Max retries exceeded: Escalate to user
-
-## Auto-Recovery Commands
-
-\`\`\`bash
-# Clean build
-rm -rf node_modules dist && npm ci && npm run build
-
-# Reset git state (careful!)
-git stash && git checkout .
-
-# Clear caches
-npm cache clean --force
-\`\`\`
+            content: `---
+name: error-recovery
+description: "Error recovery patterns and retry strategies"
+triggers: [error, fail, retry, recover, fix]
+priority: 70
+---
+# Error Recovery Patterns
+
+## Common Error Types
+
+### Build Errors
+- Check TypeScript compilation errors
+- Verify dependency versions
+- Run \`npm ci\` to clean install
+
+### Test Failures
+- Run failed tests in isolation
+- Check test fixtures/mocks
+- Verify async timing issues
+
+### Runtime Errors
+- Check stack trace carefully
+- Verify environment variables
+- Check external service connectivity
+
+## Retry Strategy
+
+1. First retry: Same action
+2. Second retry: Clean state (cache clear)
+3. Third retry: Alternative approach
+4. Max retries exceeded: Escalate to user
+
+## Auto-Recovery Commands
+
+\`\`\`bash
+# Clean build
+rm -rf node_modules dist && npm ci && npm run build
+
+# Reset git state (careful!)
+git stash && git checkout .
+
+# Clear caches
+npm cache clean --force
+\`\`\`
 `,
         },
         {
             id: 'code-quality-check',
-            content: `---
-name: code-quality-check
-description: "Code quality validation using core tools"
-triggers: [quality, lint, complexity, review]
-priority: 60
----
-# Code Quality Check
-
-## Using Core Tools
-
-\`\`\`bash
-# Analyze complexity
-node -e "import('${corePathUrl}/node_modules/@su-record/vibe/dist/tools/index.js').then(t =>
-  t.analyzeComplexity({targetPath: 'src/', projectPath: process.cwd()})
-  .then(r => console.log(r.content[0].text))
-)"
-
-# Validate quality
-node -e "import('${corePathUrl}/node_modules/@su-record/vibe/dist/tools/index.js').then(t =>
-  t.validateCodeQuality({targetPath: 'src/', projectPath: process.cwd()})
-  .then(r => console.log(r.content[0].text))
-)"
-\`\`\`
-
-## Quality Metrics
-
-| Metric | Good | Warning | Critical |
-|--------|------|---------|----------|
-| Cyclomatic Complexity | ≤10 | 11-15 | >15 |
-| Function Length | ≤30 | 31-50 | >50 |
-| Nesting Depth | ≤3 | 4 | >4 |
-
-## Common Issues
-
-1. **High Complexity**: Extract helper functions
-2. **Long Functions**: Split by responsibility
-3. **Deep Nesting**: Use early returns
-4. **Tight Coupling**: Apply dependency injection
+            content: `---
+name: code-quality-check
+description: "Code quality validation using core tools"
+triggers: [quality, lint, complexity, review]
+priority: 60
+---
+# Code Quality Check
+
+## Using Core Tools
+
+\`\`\`bash
+# Analyze complexity
+node -e "import('${corePathUrl}/node_modules/@su-record/vibe/dist/tools/index.js').then(t =>
+  t.analyzeComplexity({targetPath: 'src/', projectPath: process.cwd()})
+  .then(r => console.log(r.content[0].text))
+)"
+
+# Validate quality
+node -e "import('${corePathUrl}/node_modules/@su-record/vibe/dist/tools/index.js').then(t =>
+  t.validateCodeQuality({targetPath: 'src/', projectPath: process.cwd()})
+  .then(r => console.log(r.content[0].text))
+)"
+\`\`\`
+
+## Quality Metrics
+
+| Metric | Good | Warning | Critical |
+|--------|------|---------|----------|
+| Cyclomatic Complexity | ≤10 | 11-15 | >15 |
+| Function Length | ≤30 | 31-50 | >50 |
+| Nesting Depth | ≤3 | 4 | >4 |
+
+## Common Issues
+
+1. **High Complexity**: Extract helper functions
+2. **Long Functions**: Split by responsibility
+3. **Deep Nesting**: Use early returns
+4. **Tight Coupling**: Apply dependency injection
 `,
         },
         {
             id: 'session-management',
-            content: `---
-name: session-management
-description: "Context and session management across conversations"
-triggers: [session, context, memory, save, restore, continue]
-priority: 75
----
-# Session Management
-
-## Starting a Session
-
-\`\`\`bash
-# Auto-restore previous context
-node -e "import('${corePathUrl}/node_modules/@su-record/vibe/dist/tools/index.js').then(t =>
-  t.startSession({projectPath: process.cwd()})
-  .then(r => console.log(r.content[0].text))
-)"
-\`\`\`
-
-## Saving Context
-
-\`\`\`bash
-# Save important decisions
-node -e "import('${corePathUrl}/node_modules/@su-record/vibe/dist/tools/index.js').then(t =>
-  t.saveMemory({
-    key: 'decision-auth-method',
-    value: 'Using JWT with refresh tokens for auth',
-    category: 'architecture',
-    projectPath: process.cwd()
-  }).then(r => console.log(r.content[0].text))
-)"
-\`\`\`
-
-## Context Management
-
-At 70%+ context usage:
-1. Use \`saveMemory\` for important decisions
-2. Start new session with \`/new\`
-3. Previous context auto-restores
-
-## Slash Command
-
-\`\`\`
-/vibe.utils --continue
-\`\`\`
-
-Restores previous session context automatically.
+            content: `---
+name: session-management
+description: "Context and session management across conversations"
+triggers: [session, context, memory, save, restore, continue]
+priority: 75
+---
+# Session Management
+
+## Starting a Session
+
+\`\`\`bash
+# Auto-restore previous context
+node -e "import('${corePathUrl}/node_modules/@su-record/vibe/dist/tools/index.js').then(t =>
+  t.startSession({projectPath: process.cwd()})
+  .then(r => console.log(r.content[0].text))
+)"
+\`\`\`
+
+## Saving Context
+
+\`\`\`bash
+# Save important decisions
+node -e "import('${corePathUrl}/node_modules/@su-record/vibe/dist/tools/index.js').then(t =>
+  t.saveMemory({
+    key: 'decision-auth-method',
+    value: 'Using JWT with refresh tokens for auth',
+    category: 'architecture',
+    projectPath: process.cwd()
+  }).then(r => console.log(r.content[0].text))
+)"
+\`\`\`
+
+## Context Management
+
+At 70%+ context usage:
+1. Use \`saveMemory\` for important decisions
+2. Start new session with \`/new\`
+3. Previous context auto-restores
+
+## Slash Command
+
+\`\`\`
+/vibe.utils --continue
+\`\`\`
+
+Restores previous session context automatically.
 `,
         },
     ];