@x12i/ai-gateway 10.4.4 → 11.0.0

package/dist-cjs/gateway-memory.cjs

@@ -104,10 +104,6 @@ export function buildWorkingMemory(request, existingWorkingMemory, otherMemories
     if (!workingMemory.job.objective && request.instructions) {
         workingMemory.job.objective = request.instructions;
     }
-    if (!workingMemory.job.context && request.context) {
-        workingMemory.job.context = request.context;
-    }
-    // Input field has been removed - data should come from workingMemory.input
     if (!workingMemory.job.narrative && request.prompt) {
         workingMemory.job.narrative = request.prompt;
     }
@@ -118,9 +114,6 @@ export function buildWorkingMemory(request, existingWorkingMemory, otherMemories
     if (!workingMemory.task.objective && request.instructions) {
         workingMemory.task.objective = request.instructions;
     }
-    if (!workingMemory.task.context && request.context) {
-        workingMemory.task.context = request.context;
-    }
     // Input field has been removed - data should come from workingMemory.input
     if (!workingMemory.task.id && request.identity.taskId) {
         workingMemory.task.id = request.identity.taskId;

package/dist-cjs/gateway-messages.cjs

@@ -13,21 +13,17 @@ function isAIRequest(request) {
         'reasoningEncrypted' in request;
 }
 /**
- * Constructs messages from instructions/prompt/input/context
+ * Constructs messages from instructions and prompt (two-message contract).
  *
  * Uses direct message builder which handles:
- * - Token resolution (3-tier system)
  * - Template parsing (via Rendrix, @x12i/rendrix)
- * - Instruction block resolution and composition
- * - Flex-md format specification
- * - Message assembly
+ * - Message assembly: system (instructions) + user (prompt or input fallback)
  */
 export async function constructMessages(request, config, logger, parsedSnapshot) {
     logger.verbose('Constructing messages from request', {
         jobId: request.identity.jobId,
         agentId: request.agentId,
         hasInstructions: !!request.instructions,
-        hasContext: !!request.context,
         hasPrompt: !!request.prompt,
         hasWorkingMemory: !!request.workingMemory
     });

package/dist-cjs/gateway-messages.d.ts

@@ -7,14 +7,11 @@ import type { Logxer } from '@x12i/logxer';
 import { type MessageBuilderConfig } from './message-builder.js';
 type Request = ChatRequest | AIRequest;
 /**
- * Constructs messages from instructions/prompt/input/context
+ * Constructs messages from instructions and prompt (two-message contract).
  *
  * Uses direct message builder which handles:
- * - Token resolution (3-tier system)
  * - Template parsing (via Rendrix, @x12i/rendrix)
- * - Instruction block resolution and composition
- * - Flex-md format specification
- * - Message assembly
+ * - Message assembly: system (instructions) + user (prompt or input fallback)
  */
 export declare function constructMessages(request: Request, config: MessageBuilderConfig, logger: Logxer, parsedSnapshot?: any): Promise<Array<{
     role: string;

package/dist-cjs/gateway-validation.cjs

@@ -2,6 +2,17 @@
  * Gateway Validation Module
  * Basic validation for clean proxy implementation
  */
+function rejectRemovedContextField(request) {
+    if ('context' in request && request.context !== undefined) {
+        const err = new Error(`The 'context' field has been removed. Context is not sent to the LLM. Merge background data into workingMemory upstream and use the prompt template for the user turn.`);
+        err.code = 'CONTEXT_FIELD_REMOVED';
+        err.details = {
+            field: 'context',
+            alternative: 'workingMemory + prompt template'
+        };
+        throw err;
+    }
+}
 function validateMandatoryRuntimeIdentity(request) {
     const id = request.identity;
     if (id === undefined || id === null || typeof id !== 'object') {
@@ -20,6 +31,7 @@ export function validateChatRequest(request) {
         throw new Error('agentId is required');
     }
     validateMandatoryRuntimeIdentity(request);
+    rejectRemovedContextField(request);
     // Reject input field - it has been removed
     if ('input' in request && request.input !== undefined) {
         const err = new Error(`The 'input' field has been removed. Use workingMemory.input instead for template rendering. Prompt templates should contain {{input}} which will be resolved from workingMemory.input.`);
@@ -44,6 +56,7 @@ export function validateAIRequest(request) {
         throw new Error('agentId is required for AI requests');
     }
     validateMandatoryRuntimeIdentity(request);
+    rejectRemovedContextField(request);
     if (!request.actionType ||
         !GATEWAY_ACTION_TYPES.includes(request.actionType)) {
         throw new Error(`actionType is required and must be one of: ${GATEWAY_ACTION_TYPES.join(', ')}`);

package/dist-cjs/gateway.cjs

@@ -93,8 +93,54 @@ export class AIGateway {
             const startTime = Date.now();
             // Generate simple task type ID
             const taskTypeId = request.taskTypeId || `task-${Date.now()}`;
-            // Simple message construction
-            const messages = this.buildSimpleMessages(request);
+            const parsedSnapshot = {};
+            let messages = [];
+            try {
+                const builtMessages = await buildMessages(request, this.messageBuilderConfig, {
+                    parsedSnapshot
+                });
+                messages = builtMessages.messages;
+            }
+            catch (error) {
+                const err = error instanceof Error ? error : new Error(String(error));
+                const endTime = Date.now();
+                const duration = endTime - startTime;
+                const errWithCode = err;
+                const isResolutionError = err.name === 'InstructionNotFoundError' ||
+                    err.name === 'InstructionBackendError' ||
+                    err.name === 'TemplateResolutionError' ||
+                    errWithCode.code === 'PROMPT_NOT_FOUND' ||
+                    errWithCode.code === 'PROMPT_RESOLUTION_ERROR' ||
+                    errWithCode.code === 'PROMPT_RENDERED_EMPTY' ||
+                    errWithCode.code === 'TEMPLATE_RESOLUTION_ERROR' ||
+                    errWithCode.code === 'TEMPLATE_VARIABLE_MISSING' ||
+                    errWithCode.code === 'USER_CONTENT_REQUIRED' ||
+                    err.message.includes('Failed to resolve') ||
+                    err.message.includes('Failed to render prompt template') ||
+                    err.message.includes('not found') ||
+                    err.message.includes('Instruction not found') ||
+                    err.message.includes('Prompt not found');
+                if (isResolutionError && this.activityManager) {
+                    await this.activityManager.logBadRequest(request, err, {
+                        endTime,
+                        duration,
+                        error: err.message,
+                        errorType: errWithCode.code || 'MessageBuildError',
+                        diagnosticInfo: {
+                            errorCode: errWithCode.code,
+                            errorName: err.name,
+                            failureType: 'validation-failure',
+                            stage: 'message-building',
+                            prompt: request.prompt,
+                            instructions: typeof request.instructions === 'string' ? request.instructions.substring(0, 100) : '(object)'
+                        },
+                        failureType: 'validation-failure'
+                    }, startTime);
+                }
+                throw err;
+            }
+            parsedSnapshot.messages = messages;
+            request._parsedRequest = parsedSnapshot;
             // Merge config (modelConfig > request.config > gateway defaults)
             const aiTools = await this.getAiTools();
             const mergedConfig = await mergeConfig(request, this.config, this.logger, {
@@ -247,6 +293,7 @@ export class AIGateway {
                     errWithCode.code === 'PROMPT_RENDERED_EMPTY' ||
                     errWithCode.code === 'TEMPLATE_RESOLUTION_ERROR' ||
                     errWithCode.code === 'TEMPLATE_VARIABLE_MISSING' ||
+                    errWithCode.code === 'USER_CONTENT_REQUIRED' ||
                     err.message.includes('Failed to resolve') ||
                     err.message.includes('Failed to render prompt template') ||
                     err.message.includes('not found') ||
@@ -284,9 +331,6 @@ export class AIGateway {
             parsedSnapshot.messages = messages;
             // parsed.instructions and parsed.prompt are set by buildMessages to the resolved/rendered content
             // (after key resolution and Rendrix). Do not overwrite with raw request keys.
-            if (parsedSnapshot.context === undefined) {
-                parsedSnapshot.context = request.context;
-            }
             // Attach parsedSnapshot to request for activity tracking
             request._parsedRequest = parsedSnapshot;
             // Merge config (modelConfig > request.config > gateway defaults)
@@ -775,37 +819,6 @@ export class AIGateway {
             }
         });
     }
-    /**
-     * Build simple messages from request (instructions and prompt as literal template text; no registry).
-     */
-    buildSimpleMessages(request) {
-        const messages = [];
-        // Add instructions as system message if present
-        if (request.instructions) {
-            const instructions = typeof request.instructions === 'string'
-                ? request.instructions
-                : 'Default instructions';
-            messages.push({ role: 'system', content: instructions });
-        }
-        // Add context as assistant message if present
-        if (request.context) {
-            const context = typeof request.context === 'string'
-                ? request.context
-                : JSON.stringify(request.context);
-            messages.push({ role: 'assistant', content: context });
-        }
-        // Add prompt/input as user message
-        // Input field has been removed - prompt template should contain {{input}} which resolves from workingMemory.input
-        const userContent = request.prompt || '';
-        if (userContent) {
-            messages.push({ role: 'user', content: userContent });
-        }
-        // Add direct messages if present
-        if (request.messages) {
-            messages.push(...request.messages);
-        }
-        return messages;
-    }
     // Provider management methods
     register(provider) {
         this.router.registerProvider(provider);

package/dist-cjs/gateway.d.ts

@@ -29,10 +29,6 @@ export declare class AIGateway {
      * Invoke AI request (with structured output support)
      */
     invoke<TContent = unknown>(request: AIInvokeRequest): Promise<EnhancedLLMResponse<TContent>>;
-    /**
-     * Build simple messages from request (instructions and prompt as literal template text; no registry).
-     */
-    private buildSimpleMessages;
     register(provider: any): void;
     listProviders(): string[];
     getRouter(): LLMProviderRouter;