latitude-mcp-server 2.2.5 → 2.2.7

package/dist/api.d.ts

@@ -65,6 +65,24 @@ export declare function computeDiff(incoming: Array<{
     content: string;
 }>, existing: Document[]): DocumentChange[];
 export declare function runDocument(path: string, parameters?: Record<string, unknown>, versionUuid?: string): Promise<RunResult>;
+export interface ValidationIssue {
+    type: 'error' | 'warning';
+    code: string;
+    message: string;
+    rootCause: string;
+    suggestion: string;
+    location?: {
+        line: number;
+        column: number;
+    };
+    codeFrame?: string;
+}
+/**
+ * Pre-validate PromptL content using the official promptl-ai library.
+ * Returns detailed, actionable error messages with code frames.
+ * EXPORTED for use by tools.ts to pre-validate before pushing.
+ */
+export declare function validatePromptLContent(content: string, path: string): Promise<ValidationIssue[]>;
 /**
  * Deploy changes to LIVE version using the proper workflow:
  * 1. Create a draft version

package/dist/api.js

@@ -27,6 +27,7 @@ exports.pushChanges = pushChanges;
 exports.hashContent = hashContent;
 exports.computeDiff = computeDiff;
 exports.runDocument = runDocument;
+exports.validatePromptLContent = validatePromptLContent;
 exports.deployToLive = deployToLive;
 exports.getPromptNames = getPromptNames;
 const logger_util_js_1 = require("./utils/logger.util.js");
@@ -443,16 +444,10 @@ const ERROR_SUGGESTIONS = {
         suggestion: 'Move the <tool-call> tag inside an <assistant> block.',
     },
 };
-/**
- * Get snippet from content around a position
- */
-function getSnippet(content, startIndex, endIndex) {
-    const snippet = content.substring(startIndex, Math.min(endIndex, startIndex + 100));
-    return snippet.split('\n')[0] || snippet;
-}
 /**
  * Pre-validate PromptL content using the official promptl-ai library.
- * Returns detailed, actionable error messages.
+ * Returns detailed, actionable error messages with code frames.
+ * EXPORTED for use by tools.ts to pre-validate before pushing.
  */
 async function validatePromptLContent(content, path) {
     const issues = [];
@@ -465,9 +460,6 @@ async function validatePromptLContent(content, path) {
         });
         // Convert CompileErrors to our ValidationIssue format
         for (const compileError of result.errors) {
-            // CompileError has start?.line (Position) and startIndex (number)
-            const lineNumber = compileError.start?.line;
-            const snippet = getSnippet(content, compileError.startIndex, compileError.endIndex);
             // Get human-readable suggestion based on error code
             const suggestionInfo = ERROR_SUGGESTIONS[compileError.code] || {
                 rootCause: compileError.message,
@@ -475,36 +467,43 @@ async function validatePromptLContent(content, path) {
             };
             issues.push({
                 type: 'error',
+                code: compileError.code,
                 message: compileError.message,
                 rootCause: suggestionInfo.rootCause,
                 suggestion: suggestionInfo.suggestion,
-                lineNumber,
-                snippet,
+                location: compileError.start ? {
+                    line: compileError.start.line,
+                    column: compileError.start.column,
+                } : undefined,
+                codeFrame: compileError.frame, // The beautiful formatted code frame!
             });
         }
     }
     catch (err) {
         // Handle parse errors (thrown, not accumulated)
         if (err instanceof promptl_ai_1.CompileError) {
-            const lineNumber = err.start?.line;
-            const snippet = getSnippet(content, err.startIndex, err.endIndex);
             const suggestionInfo = ERROR_SUGGESTIONS[err.code] || {
                 rootCause: err.message,
                 suggestion: 'Fix the syntax error at the indicated location.',
             };
             issues.push({
                 type: 'error',
+                code: err.code,
                 message: err.message,
                 rootCause: suggestionInfo.rootCause,
                 suggestion: suggestionInfo.suggestion,
-                lineNumber,
-                snippet,
+                location: err.start ? {
+                    line: err.start.line,
+                    column: err.start.column,
+                } : undefined,
+                codeFrame: err.frame,
             });
         }
         else {
             // Unknown error - still report it
             issues.push({
                 type: 'error',
+                code: 'unknown-error',
                 message: err instanceof Error ? err.message : 'Unknown validation error',
                 rootCause: 'An unexpected error occurred during validation.',
                 suggestion: 'Check the prompt content for syntax errors.',
@@ -533,10 +532,11 @@ async function identifyFailingDocuments(changes) {
             failed.push({
                 path: change.path,
                 error: mainError.message,
+                code: mainError.code,
                 rootCause: mainError.rootCause,
                 suggestion: mainError.suggestion,
-                lineNumber: mainError.lineNumber,
-                snippet: mainError.snippet,
+                location: mainError.location,
+                codeFrame: mainError.codeFrame,
             });
         }
     }
@@ -601,9 +601,9 @@ async function testSingleDocument(change) {
         return {
             path: change.path,
             error: errorMsg,
+            code: 'api-validation-error',
             rootCause,
             suggestion,
-            snippet: change.content?.substring(0, 300),
         };
     }
 }
@@ -731,13 +731,14 @@ async function deployToLive(changes, _versionName) {
             const errorLines = [];
             for (const doc of failedDocs) {
                 errorLines.push(`\n## ❌ ${doc.path}`);
+                errorLines.push(`**Error Code:** \`${doc.code}\``);
                 errorLines.push(`**Error:** ${doc.error}`);
                 errorLines.push(`**Root Cause:** ${doc.rootCause}`);
-                if (doc.lineNumber) {
-                    errorLines.push(`**Location:** Line ${doc.lineNumber}`);
+                if (doc.location) {
+                    errorLines.push(`**Location:** Line ${doc.location.line}, Column ${doc.location.column}`);
                 }
-                if (doc.snippet) {
-                    errorLines.push(`**Snippet:** \`${doc.snippet}\``);
+                if (doc.codeFrame) {
+                    errorLines.push(`**Code Context:**\n\`\`\`\n${doc.codeFrame}\n\`\`\``);
                 }
                 errorLines.push(`**Fix:** ${doc.suggestion}`);
             }

package/dist/tools.js

@@ -106,6 +106,55 @@ function formatAvailablePrompts(names) {
     const formatted = names.map(n => `\`${n}\``).join(', ');
     return `\n\n**Available prompts (${names.length}):** ${formatted}`;
 }
+/**
+ * Validate all prompts BEFORE pushing.
+ * If ANY prompt fails validation, returns all errors and NOTHING is pushed.
+ */
+async function validateAllPrompts(prompts) {
+    const errors = [];
+    for (const prompt of prompts) {
+        const issues = await (0, api_js_1.validatePromptLContent)(prompt.content, prompt.name);
+        const errorIssues = issues.filter(i => i.type === 'error');
+        if (errorIssues.length > 0) {
+            errors.push({ name: prompt.name, issues: errorIssues });
+        }
+    }
+    return {
+        valid: errors.length === 0,
+        errors,
+    };
+}
+/**
+ * Format validation errors into a detailed MCP-friendly error message.
+ */
+function formatValidationErrors(errors) {
+    const lines = [
+        `## ❌ Validation Failed - No Changes Made\n`,
+        `**${errors.length} prompt(s) have errors.** Fix all errors before pushing.\n`,
+    ];
+    for (const { name, issues } of errors) {
+        for (const issue of issues) {
+            lines.push(`### ${name}`);
+            lines.push(`**Error Code:** \`${issue.code}\``);
+            lines.push(`**Error:** ${issue.message}`);
+            lines.push(`**Root Cause:** ${issue.rootCause}`);
+            if (issue.location) {
+                lines.push(`**Location:** Line ${issue.location.line}, Column ${issue.location.column}`);
+            }
+            if (issue.codeFrame) {
+                lines.push(`**Code Context:**\n\`\`\`\n${issue.codeFrame}\n\`\`\``);
+            }
+            lines.push(`**Fix:** ${issue.suggestion}`);
+            lines.push('');
+        }
+    }
+    lines.push(`---`);
+    lines.push(`**Action Required:** Fix the errors above, then retry.`);
+    return {
+        content: [{ type: 'text', text: lines.join('\n') }],
+        isError: true,
+    };
+}
 // ============================================================================
 // Tool Handlers
 // ============================================================================
@@ -194,6 +243,15 @@ async function handlePushPrompts(args) {
         if (prompts.length === 0) {
             return formatError(new Error('No prompts provided. Use either prompts array or filePaths.'));
         }
+        // PRE-VALIDATE ALL PROMPTS BEFORE PUSHING
+        // If ANY prompt fails validation, return errors and push NOTHING
+        logger.info(`Validating ${prompts.length} prompt(s) before push...`);
+        const validation = await validateAllPrompts(prompts);
+        if (!validation.valid) {
+            logger.warn(`Validation failed for ${validation.errors.length} prompt(s)`);
+            return formatValidationErrors(validation.errors);
+        }
+        logger.info(`All ${prompts.length} prompt(s) passed validation`);
         // Get existing prompts for diff computation
         const existingDocs = await (0, api_js_1.listDocuments)('live');
         // Compute diff - this determines what needs to be added, modified, or deleted
@@ -279,6 +337,15 @@ async function handleAppendPrompts(args) {
         if (prompts.length === 0) {
             return formatError(new Error('No prompts provided. Use either prompts array or filePaths.'));
         }
+        // PRE-VALIDATE ALL PROMPTS BEFORE APPENDING
+        // If ANY prompt fails validation, return errors and append NOTHING
+        logger.info(`Validating ${prompts.length} prompt(s) before append...`);
+        const validation = await validateAllPrompts(prompts);
+        if (!validation.valid) {
+            logger.warn(`Validation failed for ${validation.errors.length} prompt(s)`);
+            return formatValidationErrors(validation.errors);
+        }
+        logger.info(`All ${prompts.length} prompt(s) passed validation`);
         // Get existing prompts
         const existingDocs = await (0, api_js_1.listDocuments)('live');
         const existingMap = new Map(existingDocs.map((d) => [d.path, d]));
@@ -454,6 +521,15 @@ async function handleReplacePrompt(args) {
         if (!content) {
             return formatError(new Error('Prompt content is required. Provide either `content` or `filePath`.'));
         }
+        // PRE-VALIDATE PROMPT BEFORE REPLACING
+        // If validation fails, return error and replace NOTHING
+        logger.info(`Validating prompt "${name}" before replace...`);
+        const validation = await validateAllPrompts([{ name, content }]);
+        if (!validation.valid) {
+            logger.warn(`Validation failed for prompt "${name}"`);
+            return formatValidationErrors(validation.errors);
+        }
+        logger.info(`Prompt "${name}" passed validation`);
         // Check if prompt exists
         const existingDocs = await (0, api_js_1.listDocuments)('live');
         const exists = existingDocs.some((d) => d.path === name);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "latitude-mcp-server",
-	"version": "2.2.5",
+	"version": "2.2.7",
 	"description": "Simplified MCP server for Latitude.so prompt management - 8 focused tools for push, pull, run, and manage prompts",
 	"main": "dist/index.js",
 	"types": "dist/index.d.ts",