superkit-mcp-server 1.2.4 → 1.2.6

package/build/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
-import { CallToolRequestSchema, ListToolsRequestSchema, ListPromptsRequestSchema, GetPromptRequestSchema } from "@modelcontextprotocol/sdk/types.js";
+import { CallToolRequestSchema, ListToolsRequestSchema, ListPromptsRequestSchema, GetPromptRequestSchema, } from "@modelcontextprotocol/sdk/types.js";
 import * as path from "path";
 import * as fs from "fs/promises";
 import * as toml from "@iarna/toml";
@@ -11,11 +11,11 @@ import { manageSession } from "./tools/sessionManager.js";
 import { runChecklist } from "./tools/checklist.js";
 import { runVerifyAll } from "./tools/verifyAll.js";
 import { logSkill, logWorkflow, rotateLogs } from "./tools/loggerTools.js";
-import { getNextTodoId, createTodo, startTodo, doneTodo, completeTodo } from "./tools/todoTools.js";
-import { compoundSearch, updateSolutionRef, validateCompound, auditStateDrift, suggestSkills, compoundHealth, compoundDashboard, compoundMetrics } from "./tools/compoundTools.js";
-import { bootstrapFolderDocs, checkDocsFreshness, discoverUndocumentedFolders, validateFolderDocs } from "./tools/docsTools.js";
-import { generateChangelog, validateChangelog, archiveCompleted, prePushHousekeeping } from "./tools/gitTools.js";
-import { validateSpecConsistency, completePlan, validateArchitecture, syncSpec, updateSpecPhase } from "./tools/archTools.js";
+import { getNextTodoId, createTodo, startTodo, doneTodo, completeTodo, } from "./tools/todoTools.js";
+import { compoundSearch, updateSolutionRef, validateCompound, auditStateDrift, suggestSkills, compoundHealth, compoundDashboard, compoundMetrics, } from "./tools/compoundTools.js";
+import { bootstrapFolderDocs, checkDocsFreshness, discoverUndocumentedFolders, validateFolderDocs, } from "./tools/docsTools.js";
+import { generateChangelog, validateChangelog, archiveCompleted, prePushHousekeeping, } from "./tools/gitTools.js";
+import { validateSpecConsistency, completePlan, validateArchitecture, syncSpec, updateSpecPhase, } from "./tools/archTools.js";
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 const superKitRoot = path.resolve(__dirname, "../");
@@ -32,7 +32,7 @@ const server = new Server({
 async function listDirectorySafe(dirPath) {
     try {
         const entries = await fs.readdir(dirPath, { withFileTypes: true });
-        return entries.map((entry) => (entry.isDirectory() ? `${entry.name}/` : entry.name));
+        return entries.map((entry) => entry.isDirectory() ? `${entry.name}/` : entry.name);
     }
     catch (err) {
         return [];
@@ -56,10 +56,10 @@ const TOOLS = [
             type: "object",
             properties: {
                 action: { type: "string", enum: ["start", "stop", "status"] },
-                port: { type: "number", default: 3000 }
+                port: { type: "number", default: 3000 },
             },
             required: ["action"],
-        }
+        },
     },
     {
         name: "call_tool_session_manager",
@@ -68,10 +68,10 @@ const TOOLS = [
             type: "object",
             properties: {
                 command: { type: "string", enum: ["status", "info"] },
-                path: { type: "string", default: "." }
+                path: { type: "string", default: "." },
             },
             required: ["command"],
-        }
+        },
     },
     {
         name: "call_tool_checklist",
@@ -81,10 +81,10 @@ const TOOLS = [
             properties: {
                 projectPath: { type: "string", default: "." },
                 url: { type: "string" },
-                skipPerformance: { type: "boolean", default: false }
+                skipPerformance: { type: "boolean", default: false },
             },
-            required: ["projectPath"]
-        }
+            required: ["projectPath"],
+        },
     },
     {
         name: "call_tool_verify_all",
@@ -95,10 +95,10 @@ const TOOLS = [
                 projectPath: { type: "string", default: "." },
                 url: { type: "string" },
                 skipE2E: { type: "boolean", default: false },
-                stopOnFail: { type: "boolean", default: false }
+                stopOnFail: { type: "boolean", default: false },
             },
-            required: ["projectPath", "url"]
-        }
+            required: ["projectPath", "url"],
+        },
     },
     {
         name: "call_tool_logger_manager",
@@ -106,13 +106,16 @@ const TOOLS = [
         inputSchema: {
             type: "object",
             properties: {
-                action: { type: "string", enum: ["logSkill", "logWorkflow", "rotateLogs"] },
+                action: {
+                    type: "string",
+                    enum: ["logSkill", "logWorkflow", "rotateLogs"],
+                },
                 name: { type: "string" },
                 outcome: { type: "string" },
-                projectPath: { type: "string", default: "." }
+                projectPath: { type: "string", default: "." },
             },
-            required: ["action", "projectPath"]
-        }
+            required: ["action", "projectPath"],
+        },
     },
     {
         name: "call_tool_todo_manager",
@@ -120,16 +123,20 @@ const TOOLS = [
         inputSchema: {
             type: "object",
             properties: {
-                action: { type: "string", enum: ["nextId", "create", "start", "done", "complete"] },
+                action: {
+                    type: "string",
+                    enum: ["nextId", "create", "start", "done", "complete"],
+                },
                 title: { type: "string" },
                 description: { type: "string" },
-                priority: { type: "number" },
+                priority: { type: "string" },
+                criteria: { type: "array", items: { type: "string" } },
                 todoId: { type: "string" },
                 force: { type: "boolean", default: false },
-                projectPath: { type: "string", default: "." }
+                projectPath: { type: "string", default: "." },
             },
-            required: ["action", "projectPath"]
-        }
+            required: ["action", "projectPath"],
+        },
     },
     {
         name: "call_tool_compound_manager",
@@ -137,15 +144,27 @@ const TOOLS = [
         inputSchema: {
             type: "object",
             properties: {
-                action: { type: "string", enum: ["search", "updateRef", "validate", "auditDrift", "suggestSkills", "health", "dashboard", "metrics"] },
+                action: {
+                    type: "string",
+                    enum: [
+                        "search",
+                        "updateRef",
+                        "validate",
+                        "auditDrift",
+                        "suggestSkills",
+                        "health",
+                        "dashboard",
+                        "metrics",
+                    ],
+                },
                 terms: { type: "array", items: { type: "string" } },
                 files: { type: "array", items: { type: "string" } },
                 fix: { type: "boolean", default: false },
                 force: { type: "boolean", default: false },
-                projectPath: { type: "string", default: "." }
+                projectPath: { type: "string", default: "." },
             },
-            required: ["action", "projectPath"]
-        }
+            required: ["action", "projectPath"],
+        },
     },
     {
         name: "call_tool_docs_manager",
@@ -153,15 +172,18 @@ const TOOLS = [
         inputSchema: {
             type: "object",
             properties: {
-                action: { type: "string", enum: ["bootstrap", "freshness", "discover", "validate"] },
+                action: {
+                    type: "string",
+                    enum: ["bootstrap", "freshness", "discover", "validate"],
+                },
                 folder: { type: "string" },
                 skipDocs: { type: "boolean", default: false },
                 strict: { type: "boolean", default: false },
                 targetFolders: { type: "array", items: { type: "string" } },
-                projectPath: { type: "string", default: "." }
+                projectPath: { type: "string", default: "." },
             },
-            required: ["action", "projectPath"]
-        }
+            required: ["action", "projectPath"],
+        },
     },
     {
         name: "call_tool_git_manager",
@@ -169,12 +191,15 @@ const TOOLS = [
         inputSchema: {
             type: "object",
             properties: {
-                action: { type: "string", enum: ["changelog", "validateChangelog", "archive", "housekeeping"] },
+                action: {
+                    type: "string",
+                    enum: ["changelog", "validateChangelog", "archive", "housekeeping"],
+                },
                 applyFix: { type: "boolean", default: false },
-                projectPath: { type: "string", default: "." }
+                projectPath: { type: "string", default: "." },
             },
-            required: ["action", "projectPath"]
-        }
+            required: ["action", "projectPath"],
+        },
     },
     {
         name: "call_tool_arch_manager",
@@ -182,17 +207,26 @@ const TOOLS = [
         inputSchema: {
             type: "object",
             properties: {
-                action: { type: "string", enum: ["validateSpecs", "completePlan", "validateArch", "syncSpec", "updatePhase"] },
+                action: {
+                    type: "string",
+                    enum: [
+                        "validateSpecs",
+                        "completePlan",
+                        "validateArch",
+                        "syncSpec",
+                        "updatePhase",
+                    ],
+                },
                 planFile: { type: "string" },
                 force: { type: "boolean", default: false },
                 specDir: { type: "string" },
                 specName: { type: "string" },
                 phaseNum: { type: "string" },
                 status: { type: "string" },
-                projectPath: { type: "string", default: "." }
+                projectPath: { type: "string", default: "." },
             },
-            required: ["action", "projectPath"]
-        }
+            required: ["action", "projectPath"],
+        },
     },
     {
         name: "list_superkit_assets",
@@ -268,6 +302,13 @@ server.setRequestHandler(ListPromptsRequestSchema, async () => {
                     prompts.push({
                         name: file.replace(".toml", ""),
                         description: description,
+                        arguments: [
+                            {
+                                name: "args",
+                                description: "Arguments to pass to the command",
+                                required: false,
+                            },
+                        ],
                     });
                 }
                 catch (e) {
@@ -282,8 +323,18 @@ server.setRequestHandler(ListPromptsRequestSchema, async () => {
         return { prompts: [] };
     }
 });
+export function apply_prompt_args(promptText, userArgs) {
+    // Substitute {{#if args}} ... {{else}} ... {{/if}} Handlebars blocks
+    promptText = promptText.replace(/\{\{#if args\}\}([\s\S]*?)\{\{else\}\}([\s\S]*?)\{\{\/if\}\}/g, (_, ifBlock, elseBlock) => userArgs.trim() ? ifBlock : elseBlock);
+    // Substitute {{#if args}} ... {{/if}} blocks (no else branch)
+    promptText = promptText.replace(/\{\{#if args\}\}([\s\S]*?)\{\{\/if\}\}/g, (_, ifBlock) => (userArgs.trim() ? ifBlock : ""));
+    // Substitute all {{args}} occurrences with the actual user-provided args
+    promptText = promptText.replace(/\{\{args\}\}/g, userArgs);
+    return promptText;
+}
 server.setRequestHandler(GetPromptRequestSchema, async (request) => {
     const promptName = request.params.name;
+    const userArgs = request.params.arguments?.args ?? "";
     const commandFile = `${promptName}.toml`;
     const basePath = path.join(superKitRoot, "commands");
     const safePath = getSafePath(basePath, commandFile);
@@ -294,6 +345,7 @@ server.setRequestHandler(GetPromptRequestSchema, async (request) => {
         const content = await fs.readFile(safePath, "utf-8");
         const parsed = toml.parse(content);
         let promptText = parsed?.prompt || `Execute the ${promptName} command.`;
+        promptText = apply_prompt_args(promptText, userArgs);
         // Resolve @{path} includes from super-kit package root
         const includePattern = /@\{([^}]+)\}/g;
         let match;
@@ -369,7 +421,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             if (args.action === "nextId")
                 res = String(await getNextTodoId(args.projectPath));
             else if (args.action === "create")
-                res = await createTodo(args.title, args.description, args.priority, args.projectPath);
+                res = await createTodo(String(args.priority ?? "p3"), args.title, args.description, args.criteria || [], args.projectPath);
             else if (args.action === "start")
                 res = await startTodo(args.todoId, args.force, args.projectPath);
             else if (args.action === "done")
@@ -497,7 +549,14 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 const fallbackSafePath = getSafePath(path.join(superKitRoot, "skills"), path.join(args.category, args.skillName));
                 if (fallbackSafePath) {
                     const items = await listDirectorySafe(fallbackSafePath);
-                    return { content: [{ type: "text", text: `SKILL.md not found. Directory contains: ${items.join(', ')}` }] };
+                    return {
+                        content: [
+                            {
+                                type: "text",
+                                text: `SKILL.md not found. Directory contains: ${items.join(", ")}`,
+                            },
+                        ],
+                    };
                 }
                 throw e;
             }
@@ -518,7 +577,9 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     }
     catch (error) {
         return {
-            content: [{ type: "text", text: `Error executing tool: ${error.message}` }],
+            content: [
+                { type: "text", text: `Error executing tool: ${error.message}` },
+            ],
             isError: true,
         };
     }

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);