superkit-mcp-server 1.2.1 → 1.2.3

package/build/tools/todoTools.js

@@ -35,49 +35,49 @@ export async function createTodo(priority, title, problemStatement, criteriaArgs
     const filename = `${nextId}-pending-${priority}-${sanitizedDesc}.md`;
     const filePath = path.join(todosDir, filename);
     const dateStr = new Date().toISOString().split('T')[0];
-    let content = `---
-status: pending
-priority: ${priority}
-issue_id: "${nextId}"
-tags: [generated, cleanup]
-dependencies: []
----
-
-# ${title}
-
-## Problem Statement
-
-**What's broken/missing:**
-${problemStatement}
-
-**Impact:**
-This issue currently affects the system quality or functionality and needs to be addressed.
-
-## Findings
-- **Status:** Identified during workflow execution.
-- **Priority:** ${priority}
-- **System Impact:** This item is tracked to ensure continuous improvement of the codebase. Addressing it will contribute to overall system stability and feature completeness. The findings section provides context on origin and importance.
-
-## Recommended Action
-Implement the solution according to the acceptance criteria below.
-
-## Acceptance Criteria
+    let content = `---
+status: pending
+priority: ${priority}
+issue_id: "${nextId}"
+tags: [generated, cleanup]
+dependencies: []
+---
+
+# ${title}
+
+## Problem Statement
+
+**What's broken/missing:**
+${problemStatement}
+
+**Impact:**
+This issue currently affects the system quality or functionality and needs to be addressed.
+
+## Findings
+- **Status:** Identified during workflow execution.
+- **Priority:** ${priority}
+- **System Impact:** This item is tracked to ensure continuous improvement of the codebase. Addressing it will contribute to overall system stability and feature completeness. The findings section provides context on origin and importance.
+
+## Recommended Action
+Implement the solution according to the acceptance criteria below.
+
+## Acceptance Criteria
 `;
     for (const criteria of criteriaArgs) {
         content += `- [ ] ${criteria}\n`;
     }
-    content += `
-
-## Work Log
-
-### ${dateStr} - Created
-
-**By:** Agent
-**Actions:**
-- Auto-generated via createTodo MCP tool
-
-## Notes
-Source: Workflow automation
+    content += `
+
+## Work Log
+
+### ${dateStr} - Created
+
+**By:** Agent
+**Actions:**
+- Auto-generated via createTodo MCP tool
+
+## Notes
+Source: Workflow automation
 `;
     await fs.writeFile(filePath, content);
     return `✅ Created todo: ${filePath}\n   ID: ${nextId}\n   Priority: ${priority}\n   Title: ${title}`;

package/build/tools/validators/__tests__/apiSchema.test.js

@@ -9,16 +9,16 @@ describe('schemaValidator', () => {
     });
     describe('validatePrismaSchema', () => {
         it('should detect bad model names and missing fields', async () => {
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                 model user {
-                     name String
-                 }
-                 model Post {
-                     id String @id
-                     userId String
-                     createdAt DateTime
-                 }
-                 enum role { ADMIN, USER }
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                 model user {
+                     name String
+                 }
+                 model Post {
+                     id String @id
+                     userId String
+                     createdAt DateTime
+                 }
+                 enum role { ADMIN, USER }
              `);
             const issues = await validatePrismaSchema('/mock.prisma');
             expect(issues.some(i => i.includes("Model 'user' should be PascalCase"))).toBe(true);
@@ -47,11 +47,11 @@ describe('apiValidator', () => {
     });
     describe('checkApiCode', () => {
         it('should detect missing api practices', async () => {
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                 function handler() {
-                     // no try, no status, no security check
-                     return "hello";
-                 }
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                 function handler() {
+                     // no try, no status, no security check
+                     return "hello";
+                 }
              `);
             const res = await checkApiCode('route.ts');
             expect(res.issues.some(i => i.includes('No error handling'))).toBe(true);
@@ -59,14 +59,14 @@ describe('apiValidator', () => {
             expect(res.passed.length).toBe(0);
         });
         it('should pass good practices', async () => {
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                 import { z } from 'zod';
-                 function handler(req, res) {
-                     try {
-                         const jwtToken = "123";
-                         return res.status(200).send("hello");
-                     } catch(e) {}
-                 }
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                 import { z } from 'zod';
+                 function handler(req, res) {
+                     try {
+                         const jwtToken = "123";
+                         return res.status(200).send("hello");
+                     } catch(e) {}
+                 }
              `);
             const res = await checkApiCode('route.ts');
             expect(res.passed.some(i => i.includes('Error handling'))).toBe(true);

package/build/tools/validators/__tests__/convertRules.test.js

@@ -15,11 +15,11 @@ describe('convertRules', () => {
     it('should correctly parse frontmatter and generate rules', async () => {
         vi.mocked(fs.stat).mockResolvedValue({ isDirectory: () => true });
         vi.mocked(fs.readdir).mockResolvedValue(['async-waterfall.md']);
-        vi.mocked(fs.readFile).mockResolvedValue(`---
-title: Waterfall check
-impact: HIGH
-tags: perf
----
+        vi.mocked(fs.readFile).mockResolvedValue(`---
+title: Waterfall check
+impact: HIGH
+tags: perf
+---
 Content body of the rule here.`);
         vi.mocked(fs.mkdir).mockResolvedValue(undefined);
         vi.mocked(fs.writeFile).mockResolvedValue(undefined);

package/build/tools/validators/__tests__/frontendDesign.test.js

@@ -9,20 +9,20 @@ describe('accessibilityChecker', () => {
     });
     describe('checkAccessibility', () => {
         it('should detect input without label and img without alt', async () => {
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                 <input type="text" name="bad" />
-                 <button>Click me</button>
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                 <input type="text" name="bad" />
+                 <button>Click me</button>
              `);
             const issues = await checkAccessibility('test.html');
             expect(issues.some(i => i.includes('Input without label'))).toBe(true);
             expect(issues.some(i => i.includes('Missing lang'))).toBe(false); // only checks if <html> exists
         });
         it('should pass good inputs', async () => {
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                 <html lang="en">
-                   <input type="text" aria-label="Good" />
-                   <button aria-label="Close">X</button>
-                 </html>
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                 <html lang="en">
+                   <input type="text" aria-label="Good" />
+                   <button aria-label="Close">X</button>
+                 </html>
              `);
             const issues = await checkAccessibility('test.html');
             expect(issues).toEqual([]); // Skip link is only requested if <main> or <body> is present
@@ -39,10 +39,10 @@ describe('uxAudit', () => {
                     name: 'test.tsx',
                     isDirectory: () => false
                 }]);
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                 <button onClick={() => {}}>Submit</button>
-                 <img src="foo.jpg">
-                 <p style="color: #000000; font-family: purple;">Hello</p>
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                 <button onClick={() => {}}>Submit</button>
+                 <img src="foo.jpg">
+                 <p style="color: #000000; font-family: purple;">Hello</p>
              `);
             const res = await runUxAudit('.');
             expect(res.passed).toBe(false);

package/build/tools/validators/__tests__/geoChecker.test.js

@@ -8,21 +8,21 @@ describe('geoChecker', () => {
     });
     describe('checkGeoPage', () => {
         it('should detect structured schema and H1s', async () => {
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                <html lang="en">
-                <head>
-                <script type="application/ld+json">
-                {
-                  "@type": "Article",
-                  "author": "Test Author"
-                }
-                </script>
-                </head>
-                <body>
-                    <h1>Main Title</h1>
-                    <p>Some content with numbers like 50% and $100.</p>
-                </body>
-                </html>
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                <html lang="en">
+                <head>
+                <script type="application/ld+json">
+                {
+                  "@type": "Article",
+                  "author": "Test Author"
+                }
+                </script>
+                </head>
+                <body>
+                    <h1>Main Title</h1>
+                    <p>Some content with numbers like 50% and $100.</p>
+                </body>
+                </html>
             `);
             const result = await checkGeoPage('/mock/page.html');
             expect(result.passed.some(m => m.includes('JSON-LD structured data'))).toBe(true);
@@ -31,10 +31,10 @@ describe('geoChecker', () => {
             expect(result.score).toBeGreaterThan(0);
         });
         it('should flag missing structural tags', async () => {
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                <body>
-                    <p>No headings, no structure, no stats.</p>
-                </body>
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                <body>
+                    <p>No headings, no structure, no stats.</p>
+                </body>
             `);
             const result = await checkGeoPage('/mock/bad.html');
             expect(result.issues.some(m => m.includes('No JSON-LD'))).toBe(true);

package/build/tools/validators/__tests__/mobileAudit.test.js

@@ -11,16 +11,16 @@ describe('mobileAudit', () => {
                 name: 'Screen.tsx',
                 isDirectory: () => false
             }]);
-        vi.mocked(fs.readFile).mockResolvedValue(`
-            import React from 'react';
-            import { View, ScrollView } from 'react-native';
-            export const Screen = () => {
-                return (
-                    <ScrollView>
-                       {data.map(d => <View key={index} />)}
-                    </ScrollView>
-                )
-            }
+        vi.mocked(fs.readFile).mockResolvedValue(`
+            import React from 'react';
+            import { View, ScrollView } from 'react-native';
+            export const Screen = () => {
+                return (
+                    <ScrollView>
+                       {data.map(d => <View key={index} />)}
+                    </ScrollView>
+                )
+            }
         `);
         const res = await runMobileAudit('.');
         expect(res.passed).toBe(false);
@@ -31,8 +31,8 @@ describe('mobileAudit', () => {
                 name: 'regular.tsx',
                 isDirectory: () => false
             }]);
-        vi.mocked(fs.readFile).mockResolvedValue(`
-            export const App = () => <div>Hello Web</div>
+        vi.mocked(fs.readFile).mockResolvedValue(`
+            export const App = () => <div>Hello Web</div>
         `);
         const res = await runMobileAudit('.');
         expect(res.passed).toBe(true);

package/build/tools/validators/__tests__/reactPerformanceChecker.test.js

@@ -11,14 +11,14 @@ describe('reactPerformanceChecker', () => {
                 name: 'Home.tsx',
                 isDirectory: () => false
             }]);
-        vi.mocked(fs.readFile).mockResolvedValue(`
-            import { something } from '@/components/index';
-            
-            async function loadData() {
-                const a = await fetchA();
-                const b = await fetchB();
-                return { a, b };
-            }
+        vi.mocked(fs.readFile).mockResolvedValue(`
+            import { something } from '@/components/index';
+            
+            async function loadData() {
+                const a = await fetchA();
+                const b = await fetchB();
+                return { a, b };
+            }
         `);
         const res = await runReactPerformanceChecker('.');
         console.log("RPC BAD REPORT:", res.report);
@@ -31,15 +31,15 @@ describe('reactPerformanceChecker', () => {
                 name: 'App.tsx',
                 isDirectory: () => false
             }]);
-        vi.mocked(fs.readFile).mockResolvedValue(`
-            import { useQuery } from '@tanstack/react-query';
-            import Image from 'next/image';
-            
-            const App = React.memo((props: Props) => {
-                const q = useQuery('data', fetchParallel);
-                return <Image src="foo" alt="bar" />
-            });
-            export default App;
+        vi.mocked(fs.readFile).mockResolvedValue(`
+            import { useQuery } from '@tanstack/react-query';
+            import Image from 'next/image';
+            
+            const App = React.memo((props: Props) => {
+                const q = useQuery('data', fetchParallel);
+                return <Image src="foo" alt="bar" />
+            });
+            export default App;
         `);
         const res = await runReactPerformanceChecker('.');
         console.log("RPC GOOD REPORT:", res.report);

package/build/tools/validators/__tests__/securityScan.test.js

@@ -13,8 +13,8 @@ describe('securityScan', () => {
                     name: 'config.json',
                     isDirectory: () => false
                 }]);
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                 { "aws_key": "AKIA1234567890123456" }
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                 { "aws_key": "AKIA1234567890123456" }
              `);
             const result = await scanSecrets('/mock');
             expect(result.findings).toBeDefined();
@@ -28,10 +28,10 @@ describe('securityScan', () => {
                     name: 'bad_code.js',
                     isDirectory: () => false
                 }]);
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                 eval('2 + 2');
-                 import { exec } from 'child_process';
-                 child_process.exec('rm -rf /');
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                 eval('2 + 2');
+                 import { exec } from 'child_process';
+                 child_process.exec('rm -rf /');
              `);
             const result = await scanCodePatterns('/mock');
             expect(result.findings.length).toBeGreaterThan(1);

package/build/tools/validators/__tests__/seoChecker.test.js

@@ -11,22 +11,22 @@ describe('seoChecker', () => {
                 name: 'index.tsx', // valid page layout file
                 isDirectory: () => false
             }]);
-        vi.mocked(fs.readFile).mockResolvedValue(`
-            export default function Index() {
-                return (
-                    <html>
-                        <head>
-                            <title>Oops missing description and og</title>
-                        </head>
-                        <body>
-                            <h1>Title</h1>
-                            <h1>Duplicate Title</h1>
-                            <img src="foo" />
-                            <img src="bar" alt="" />
-                        </body>
-                    </html>
-                )
-            }
+        vi.mocked(fs.readFile).mockResolvedValue(`
+            export default function Index() {
+                return (
+                    <html>
+                        <head>
+                            <title>Oops missing description and og</title>
+                        </head>
+                        <body>
+                            <h1>Title</h1>
+                            <h1>Duplicate Title</h1>
+                            <img src="foo" />
+                            <img src="bar" alt="" />
+                        </body>
+                    </html>
+                )
+            }
         `);
         const res = await runSeoChecker('.');
         expect(res.passed).toBe(false);

package/build/tools/validators/__tests__/typeCoverage.test.js

@@ -14,9 +14,9 @@ describe('typeCoverage', () => {
                     isDirectory: () => false
                 }]);
             // Mock readFile to return a snippet with an untyped function and an "any"
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                function test(): any { return 1; }
-                const test2 = (x) => console.log(x);
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                function test(): any { return 1; }
+                const test2 = (x) => console.log(x);
             `);
             const result = await checkTypescriptCoverage('/mock/folder');
             expect(result.type).toBe('typescript');
@@ -29,8 +29,8 @@ describe('typeCoverage', () => {
                     name: 'good.ts',
                     isDirectory: () => false
                 }]);
-            vi.mocked(fs.readFile).mockResolvedValue(`
-                function typedFunc(val: string): number { return val.length; }
+            vi.mocked(fs.readFile).mockResolvedValue(`
+                function typedFunc(val: string): number { return val.length; }
             `);
             const result = await checkTypescriptCoverage('/mock/good');
             expect(result.stats.any_count).toBe(0);
@@ -44,15 +44,15 @@ describe('typeCoverage', () => {
                     name: 'script.py',
                     isDirectory: () => false
                 }]);
-            vi.mocked(fs.readFile).mockResolvedValue(`
-def bad_func(arg):
-    pass
-    
-def good_func(arg: str) -> bool:
-    return True
-    
-def any_func(arg: Any) -> Any:
-    pass
+            vi.mocked(fs.readFile).mockResolvedValue(`
+def bad_func(arg):
+    pass
+    
+def good_func(arg: str) -> bool:
+    return True
+    
+def any_func(arg: Any) -> Any:
+    pass
              `);
             const result = await checkPythonCoverage('/mock/py');
             expect(result.stats.any_count).toBeGreaterThan(0);

package/build/tools/validators/convertRules.js

@@ -135,12 +135,12 @@ async function generateSectionFile(sectionPrefix, rules, outputDir, getOutputOnl
 }
 export async function runConvertRules(projectPath = ".") {
     let report = `============================================================\n`;
-    report += `CONVERSION SCRIPT: React Best Practices -> .agent Format\n`;
+    report += `CONVERSION SCRIPT: React Best Practices -> Skills Format\n`;
     report += `============================================================\n`;
     const baseDir = path.resolve(projectPath);
     // Mimic the python logic for paths if run from anywhere, or adapt to superkit structure
     const rulesDir = path.join(baseDir, "skills", "react-best-practices", "rules");
-    const outputDir = path.join(baseDir, ".agent", "skills", "react-best-practices");
+    const outputDir = path.join(baseDir, "skills", "react-best-practices");
     report += `[*] Reading rules from: ${rulesDir}\n[*] Output to: ${outputDir}\n\n`;
     try {
         const stat = await fs.stat(rulesDir);