@su-record/vibe 2.6.13 → 2.6.15

package/dist/tools/convention/analyzeComplexity.test.js

@@ -4,10 +4,10 @@ import { analyzeComplexity } from './analyzeComplexity.js';
 describe('analyzeComplexity', () => {
     describe('code analysis', () => {
         it('should analyze simple code', async () => {
-            const code = `
-function add(a, b) {
-  return a + b;
-}
+            const code = `
+function add(a, b) {
+  return a + b;
+}
 `;
             const result = await analyzeComplexity({ code });
             expect(result.content).toBeDefined();
@@ -15,70 +15,70 @@ function add(a, b) {
             expect(result.content[0].text).toContain('Complexity');
         });
         it('should detect high cyclomatic complexity', async () => {
-            const code = `
-function complex(a, b, c, d) {
-  if (a > 0) {
-    if (b > 0) {
-      if (c > 0) {
-        if (d > 0) {
-          for (let i = 0; i < 10; i++) {
-            while (true) {
-              switch (i) {
-                case 1: break;
-                case 2: break;
-                case 3: break;
-              }
-              break;
-            }
-          }
-        }
-      }
-    }
-  }
-}
+            const code = `
+function complex(a, b, c, d) {
+  if (a > 0) {
+    if (b > 0) {
+      if (c > 0) {
+        if (d > 0) {
+          for (let i = 0; i < 10; i++) {
+            while (true) {
+              switch (i) {
+                case 1: break;
+                case 2: break;
+                case 3: break;
+              }
+              break;
+            }
+          }
+        }
+      }
+    }
+  }
+}
 `;
             const result = await analyzeComplexity({ code });
             expect(result.content[0].text).toContain('Complexity');
             // High complexity code should have lower score or issues
         });
         it('should calculate cyclomatic metrics only', async () => {
-            const code = `
-function test() {
-  if (true) return 1;
-  return 0;
-}
+            const code = `
+function test() {
+  if (true) return 1;
+  return 0;
+}
 `;
             const result = await analyzeComplexity({ code, metrics: 'cyclomatic' });
             expect(result.content[0].text).toContain('Complexity');
         });
         it('should calculate cognitive metrics only', async () => {
-            const code = `
-function test() {
-  if (true) {
-    for (let i = 0; i < 10; i++) {
-      console.log(i);
-    }
-  }
-}
+            const code = `
+function test() {
+  if (true) {
+    for (let i = 0; i < 10; i++) {
+      console.log(i);
+    }
+  }
+}
 `;
             const result = await analyzeComplexity({ code, metrics: 'cognitive' });
             expect(result.content[0].text).toContain('Complexity');
         });
         it('should calculate halstead metrics only', async () => {
-            const code = `
-const x = 1 + 2 * 3 - 4 / 5;
+            const code = `
+const x = 1 + 2 * 3 - 4 / 5;
 `;
             const result = await analyzeComplexity({ code, metrics: 'halstead' });
             expect(result.content[0].text).toContain('Complexity');
         });
         it('should calculate all metrics', async () => {
-            const code = `
-function calculate(a, b) {
-  if (a > b) {
-    return a - b;
-  }
-  return b - a;
-}
+            const code = `
+function calculate(a, b) {
+  if (a > b) {
+    return a - b;
+  }
+  return b - a;
+}
 `;
             const result = await analyzeComplexity({ code, metrics: 'all' });
             expect(result.content[0].text).toContain('Complexity');
@@ -102,18 +102,18 @@ function calculate(a, b) {
     });
     describe('Python code detection', () => {
         it('should detect and analyze Python code', async () => {
-            const pythonCode = `
-def calculate(a, b):
-    if a > b:
-        return a - b
-    return b - a
-
-class Calculator:
-    def __init__(self):
-        self.value = 0
-
-    def add(self, x):
-        self.value += x
+            const pythonCode = `
+def calculate(a, b):
+    if a > b:
+        return a - b
+    return b - a
+
+class Calculator:
+    def __init__(self):
+        self.value = 0
+
+    def add(self, x):
+        self.value += x
 `;
             const result = await analyzeComplexity({ code: pythonCode });
             expect(result.content[0].text).toContain('Python');
@@ -121,34 +121,34 @@ class Calculator:
     });
     describe('score calculation', () => {
         it('should give high score for simple code', async () => {
-            const simpleCode = `
-function simple() {
-  return 42;
-}
+            const simpleCode = `
+function simple() {
+  return 42;
+}
 `;
             const result = await analyzeComplexity({ code: simpleCode });
             // Simple code should have good score
             expect(result.content[0].text).toContain('Score');
         });
         it('should give lower score for complex code', async () => {
-            const complexCode = `
-function veryComplex(a, b, c, d, e, f, g, h) {
-  if (a && b || c && d || e && f || g && h) {
-    for (let i = 0; i < 10; i++) {
-      for (let j = 0; j < 10; j++) {
-        if (i > j) {
-          switch (i) {
-            case 1: break;
-            case 2: break;
-            case 3: break;
-            case 4: break;
-            case 5: break;
-          }
-        }
-      }
-    }
-  }
-}
+            const complexCode = `
+function veryComplex(a, b, c, d, e, f, g, h) {
+  if (a && b || c && d || e && f || g && h) {
+    for (let i = 0; i < 10; i++) {
+      for (let j = 0; j < 10; j++) {
+        if (i > j) {
+          switch (i) {
+            case 1: break;
+            case 2: break;
+            case 3: break;
+            case 4: break;
+            case 5: break;
+          }
+        }
+      }
+    }
+  }
+}
 `;
             const result = await analyzeComplexity({ code: complexCode });
             expect(result.content[0].text).toContain('Score');
@@ -157,46 +157,46 @@ function veryComplex(a, b, c, d, e, f, g, h) {
     });
     describe('TypeScript code analysis', () => {
         it('should analyze TypeScript code', async () => {
-            const tsCode = `
-interface User {
-  name: string;
-  age: number;
-}
-
-function greet(user: User): string {
-  if (user.age > 18) {
-    return \`Hello, \${user.name}!\`;
-  }
-  return 'Hello!';
-}
+            const tsCode = `
+interface User {
+  name: string;
+  age: number;
+}
+
+function greet(user: User): string {
+  if (user.age > 18) {
+    return \`Hello, \${user.name}!\`;
+  }
+  return 'Hello!';
+}
 `;
             const result = await analyzeComplexity({ code: tsCode });
             expect(result.content[0].text).toContain('Complexity');
         });
         it('should handle class definitions', async () => {
             // Use explicit TypeScript syntax that won't be mistaken for Python
-            const tsCode = `
-// TypeScript class
-export class Calculator {
-  private value: number = 0;
-
-  public add(x: number): this {
-    this.value += x;
-    return this;
-  }
-
-  public subtract(x: number): this {
-    if (x > this.value) {
-      throw new Error('Cannot be negative');
-    }
-    this.value -= x;
-    return this;
-  }
-
-  public getValue(): number {
-    return this.value;
-  }
-}
+            const tsCode = `
+// TypeScript class
+export class Calculator {
+  private value: number = 0;
+
+  public add(x: number): this {
+    this.value += x;
+    return this;
+  }
+
+  public subtract(x: number): this {
+    if (x > this.value) {
+      throw new Error('Cannot be negative');
+    }
+    this.value -= x;
+    return this;
+  }
+
+  public getValue(): number {
+    return this.value;
+  }
+}
 `;
             const result = await analyzeComplexity({ code: tsCode });
             // Should analyze as TypeScript, not Python

package/dist/tools/convention/getCodingGuide.d.ts

@@ -0,0 +1,7 @@
+import { ToolResult, ToolDefinition } from '../../types/tool.js';
+export declare const getCodingGuideDefinition: ToolDefinition;
+export declare function getCodingGuide(args: {
+    name: string;
+    category?: string;
+}): Promise<ToolResult>;
+//# sourceMappingURL=getCodingGuide.d.ts.map

package/dist/tools/convention/getCodingGuide.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"getCodingGuide.d.ts","sourceRoot":"","sources":["../../../src/tools/convention/getCodingGuide.ts"],"names":[],"mappings":"AAIA,OAAO,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAC;AAsCjE,eAAO,MAAM,wBAAwB,EAAE,cAmBtC,CAAC;AAEF,wBAAsB,cAAc,CAAC,IAAI,EAAE;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,UAAU,CAAC,CAmBnG"}

package/dist/tools/convention/getCodingGuide.js

@@ -0,0 +1,69 @@
+// Convention management tool - completely independent
+import { promises as fs } from 'fs';
+import path from 'path';
+const GUIDES_DIR = path.join(process.cwd(), 'guides');
+const GUIDES_FILE = path.join(GUIDES_DIR, 'coding_guides.json');
+async function ensureGuidesDir() {
+    try {
+        await fs.access(GUIDES_DIR);
+    }
+    catch { /* ignore: optional operation */
+        await fs.mkdir(GUIDES_DIR, { recursive: true });
+    }
+}
+async function loadGuides() {
+    try {
+        await ensureGuidesDir();
+        const data = await fs.readFile(GUIDES_FILE, 'utf-8');
+        return JSON.parse(data);
+    }
+    catch { /* ignore: optional operation */
+        return [];
+    }
+}
+async function findGuide(name) {
+    const guides = await loadGuides();
+    return guides.find(g => g.name === name);
+}
+export const getCodingGuideDefinition = {
+    name: 'get_coding_guide',
+    description: '가이드|규칙|컨벤션|guide|rules|convention|standards|best practices - Get coding guide',
+    inputSchema: {
+        type: 'object',
+        properties: {
+            name: { type: 'string', description: 'Guide name to retrieve' },
+            category: { type: 'string', description: 'Guide category' }
+        },
+        required: ['name']
+    },
+    annotations: {
+        title: 'Get Coding Guide',
+        audience: ['user', 'assistant'],
+        readOnlyHint: true,
+        destructiveHint: false,
+        idempotentHint: true,
+        openWorldHint: false
+    }
+};
+export async function getCodingGuide(args) {
+    const { name: guideName, category: guideCategory } = args;
+    try {
+        const guide = await findGuide(guideName);
+        if (guide) {
+            return {
+                content: [{ type: 'text', text: `Guide: ${guide.name}\nCategory: ${guide.category}\n\n${guide.content}\n\nTags: ${guide.tags.join(', ')} | Updated: ${guide.lastUpdated}` }]
+            };
+        }
+        else {
+            return {
+                content: [{ type: 'text', text: `Guide not found: "${guideName}". Use list_coding_guides to see available guides.` }]
+            };
+        }
+    }
+    catch (error) {
+        return {
+            content: [{ type: 'text', text: `Error retrieving guide: ${error instanceof Error ? error.message : 'Unknown error'}` }]
+        };
+    }
+}
+//# sourceMappingURL=getCodingGuide.js.map

package/dist/tools/convention/getCodingGuide.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"getCodingGuide.js","sourceRoot":"","sources":["../../../src/tools/convention/getCodingGuide.ts"],"names":[],"mappings":"AAAA,sDAAsD;AAEtD,OAAO,EAAE,QAAQ,IAAI,EAAE,EAAE,MAAM,IAAI,CAAC;AACpC,OAAO,IAAI,MAAM,MAAM,CAAC;AAaxB,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,QAAQ,CAAC,CAAC;AACtD,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAAC,UAAU,EAAE,oBAAoB,CAAC,CAAC;AAEhE,KAAK,UAAU,eAAe;IAC5B,IAAI,CAAC;QACH,MAAM,EAAE,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;IAC9B,CAAC;IAAC,MAAM,CAAC,CAAC,gCAAgC;QACxC,MAAM,EAAE,CAAC,KAAK,CAAC,UAAU,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAClD,CAAC;AACH,CAAC;AAED,KAAK,UAAU,UAAU;IACvB,IAAI,CAAC;QACH,MAAM,eAAe,EAAE,CAAC;QACxB,MAAM,IAAI,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;QACrD,OAAO,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC1B,CAAC;IAAC,MAAM,CAAC,CAAC,gCAAgC;QACxC,OAAO,EAAE,CAAC;IACZ,CAAC;AACH,CAAC;AAED,KAAK,UAAU,SAAS,CAAC,IAAY;IACnC,MAAM,MAAM,GAAG,MAAM,UAAU,EAAE,CAAC;IAClC,OAAO,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,IAAI,CAAC,CAAC;AAC3C,CAAC;AAED,MAAM,CAAC,MAAM,wBAAwB,GAAmB;IACtD,IAAI,EAAE,kBAAkB;IACxB,WAAW,EAAE,+EAA+E;IAC5F,WAAW,EAAE;QACX,IAAI,EAAE,QAAQ;QACd,UAAU,EAAE;YACV,IAAI,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,WAAW,EAAE,wBAAwB,EAAE;YAC/D,QAAQ,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,WAAW,EAAE,gBAAgB,EAAE;SAC5D;QACD,QAAQ,EAAE,CAAC,MAAM,CAAC;KACnB;IACD,WAAW,EAAE;QACX,KAAK,EAAE,kBAAkB;QACzB,QAAQ,EAAE,CAAC,MAAM,EAAE,WAAW,CAAC;QAC/B,YAAY,EAAE,IAAI;QAClB,eAAe,EAAE,KAAK;QACtB,cAAc,EAAE,IAAI;QACpB,aAAa,EAAE,KAAK;KACrB;CACF,CAAC;AAEF,MAAM,CAAC,KAAK,UAAU,cAAc,CAAC,IAAyC;IAC5E,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,aAAa,EAAE,GAAG,IAAI,CAAC;IAE1D,IAAI,CAAC;QACH,MAAM,KAAK,GAAG,MAAM,SAAS,CAAC,SAAS,CAAC,CAAC;QACzC,IAAI,KAAK,EAAE,CAAC;YACV,OAAO;gBACL,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,UAAU,KAAK,CAAC,IAAI,eAAe,KAAK,CAAC,QAAQ,OAAO,KAAK,CAAC,OAAO,aAAa,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,eAAe,KAAK,CAAC,WAAW,EAAE,EAAE,CAAC;aAC7K,CAAC;QACJ,CAAC;aAAM,CAAC;YACN,OAAO;gBACL,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,qBAAqB,SAAS,oDAAoD,EAAE,CAAC;aACtH,CAAC;QACJ,CAAC;IACH,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO;YACL,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,2BAA2B,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,eAAe,EAAE,EAAE,CAAC;SACzH,CAAC;IACJ,CAAC;AACH,CAAC"}

package/dist/tools/convention/validateCodeQuality.test.js

@@ -4,10 +4,10 @@ import { validateCodeQuality } from './validateCodeQuality.js';
 describe('validateCodeQuality', () => {
     describe('code validation', () => {
         it('should validate simple clean code', async () => {
-            const code = `
-function add(a: number, b: number): number {
-  return a + b;
-}
+            const code = `
+function add(a: number, b: number): number {
+  return a + b;
+}
 `;
             const result = await validateCodeQuality({ code });
             expect(result.content[0].type).toBe('text');
@@ -15,56 +15,56 @@ function add(a: number, b: number): number {
             expect(result.content[0].text).toContain('Grade');
         });
         it('should detect high nesting depth', async () => {
-            const code = `
-function nested() {
-  if (true) {
-    if (true) {
-      if (true) {
-        if (true) {
-          console.log('deep');
-        }
-      }
-    }
-  }
-}
+            const code = `
+function nested() {
+  if (true) {
+    if (true) {
+      if (true) {
+        if (true) {
+          console.log('deep');
+        }
+      }
+    }
+  }
+}
 `;
             const result = await validateCodeQuality({ code });
             expect(result.content[0].text).toContain('Nesting');
         });
         it('should detect any type usage', async () => {
-            const code = `
-function process(data: any): any {
-  return data;
-}
+            const code = `
+function process(data: any): any {
+  return data;
+}
 `;
             const result = await validateCodeQuality({ code });
             expect(result.content[0].text).toContain('type');
         });
         it('should detect loose equality', async () => {
-            const code = `
-function compare(a, b) {
-  return a == b;
-}
+            const code = `
+function compare(a, b) {
+  return a == b;
+}
 `;
             const result = await validateCodeQuality({ code });
             expect(result.content[0].text).toContain('Issues');
         });
         it('should detect var usage', async () => {
-            const code = `
-function oldStyle() {
-  var x = 1;
-  var y = 2;
-  return x + y;
-}
+            const code = `
+function oldStyle() {
+  var x = 1;
+  var y = 2;
+  return x + y;
+}
 `;
             const result = await validateCodeQuality({ code });
             expect(result.content[0].text).toContain('Issues');
         });
         it('should detect magic numbers', async () => {
-            const code = `
-function calculate(value) {
-  return value * 123 + 456;
-}
+            const code = `
+function calculate(value) {
+  return value * 123 + 456;
+}
 `;
             const result = await validateCodeQuality({ code });
             expect(result.content[0].text).toContain('Issues');
@@ -72,26 +72,26 @@ function calculate(value) {
     });
     describe('async code validation', () => {
         it('should warn about missing error handling in async', async () => {
-            const code = `
-async function fetchData() {
-  const response = await fetch('/api');
-  return response.json();
-}
+            const code = `
+async function fetchData() {
+  const response = await fetch('/api');
+  return response.json();
+}
 `;
             const result = await validateCodeQuality({ code });
             expect(result.content[0].text).toContain('Issues');
         });
         it('should not warn when error handling exists', async () => {
-            const code = `
-async function fetchData() {
-  try {
-    const response = await fetch('/api');
-    return response.json();
-  } catch (error) {
-    console.error(error);
-    throw error;
-  }
-}
+            const code = `
+async function fetchData() {
+  try {
+    const response = await fetch('/api');
+    return response.json();
+  } catch (error) {
+    console.error(error);
+    throw error;
+  }
+}
 `;
             const result = await validateCodeQuality({ code });
             // Should have fewer issues or no error handling issue
@@ -100,27 +100,27 @@ async function fetchData() {
     });
     describe('React component validation', () => {
         it('should validate React component with type', async () => {
-            const code = `
-import React from 'react';
-
-function MyComponent({ name }) {
-  return <div>Hello, {name}!</div>;
-}
+            const code = `
+import React from 'react';
+
+function MyComponent({ name }) {
+  return <div>Hello, {name}!</div>;
+}
 `;
             const result = await validateCodeQuality({ code, type: 'component' });
             expect(result.content[0].text).toContain('Type: component');
         });
         it('should suggest memo for React components', async () => {
-            const code = `
-import React from 'react';
-
-function ExpensiveComponent({ data }) {
-  return (
-    <div>
-      {data.map(item => <span key={item.id}>{item.name}</span>)}
-    </div>
-  );
-}
+            const code = `
+import React from 'react';
+
+function ExpensiveComponent({ data }) {
+  return (
+    <div>
+      {data.map(item => <span key={item.id}>{item.name}</span>)}
+    </div>
+  );
+}
 `;
             const result = await validateCodeQuality({ code, type: 'component' });
             // Should mention performance optimization
@@ -144,27 +144,27 @@ function ExpensiveComponent({ data }) {
     });
     describe('grading system', () => {
         it('should give A grade for excellent code', async () => {
-            const code = `
-const add = (a: number, b: number): number => a + b;
+            const code = `
+const add = (a: number, b: number): number => a + b;
 `;
             const result = await validateCodeQuality({ code });
             expect(result.content[0].text).toContain('Grade: A');
         });
         it('should give lower grade for problematic code', async () => {
-            const code = `
-function badCode(data: any) {
-  var result;
-  if (data == null) {
-    if (true) {
-      if (true) {
-        if (true) {
-          result = data * 123456;
-        }
-      }
-    }
-  }
-  return result;
-}
+            const code = `
+function badCode(data: any) {
+  var result;
+  if (data == null) {
+    if (true) {
+      if (true) {
+        if (true) {
+          result = data * 123456;
+        }
+      }
+    }
+  }
+  return result;
+}
 `;
             const result = await validateCodeQuality({ code });
             // Should not be grade A
@@ -173,10 +173,10 @@ function badCode(data: any) {
     });
     describe('strict mode', () => {
         it('should accept strict flag', async () => {
-            const code = `
-function test() {
-  return 1;
-}
+            const code = `
+function test() {
+  return 1;
+}
 `;
             const result = await validateCodeQuality({ code, strict: true });
             expect(result.content[0].text).toContain('Score');
@@ -184,23 +184,23 @@ function test() {
     });
     describe('metrics option', () => {
         it('should accept complexity metrics', async () => {
-            const code = `
-function test() {
-  return 1;
-}
+            const code = `
+function test() {
+  return 1;
+}
 `;
             const result = await validateCodeQuality({ code, metrics: 'complexity' });
             expect(result.content[0].text).toContain('Complexity');
         });
         it('should accept all metrics', async () => {
-            const code = `
-function test() {
-  if (true) {
-    for (let i = 0; i < 10; i++) {
-      console.log(i);
-    }
-  }
-}
+            const code = `
+function test() {
+  if (true) {
+    for (let i = 0; i < 10; i++) {
+      console.log(i);
+    }
+  }
+}
 `;
             const result = await validateCodeQuality({ code, metrics: 'all' });
             expect(result.content[0].text).toContain('Score');
@@ -208,18 +208,18 @@ function test() {
     });
     describe('deductions', () => {
         it('should show deduction details', async () => {
-            const code = `
-async function process(data: any) {
-  var x = data == null ? 0 : data;
-  return x * 12345;
-}
+            const code = `
+async function process(data: any) {
+  var x = data == null ? 0 : data;
+  return x * 12345;
+}
 `;
             const result = await validateCodeQuality({ code });
             expect(result.content[0].text).toContain('Deductions');
         });
         it('should calculate score correctly', async () => {
-            const code = `
-const clean = (x: number): number => x + 1;
+            const code = `
+const clean = (x: number): number => x + 1;
 `;
             const result = await validateCodeQuality({ code });
             // Clean code should have high score (90+)