genai-lite 0.2.0 → 0.3.0

package/dist/config/presets.json

@@ -6,7 +6,23 @@
         "providerId": "anthropic",
         "modelId": "claude-sonnet-4-20250514",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7,
+            "reasoning": {
+                "enabled": false
+            }
+        }
+    },
+    {
+        "id": "anthropic-claude-sonnet-4-20250514-thinking",
+        "displayName": "Anthropic - Claude Sonnet 4 (Thinking)",
+        "description": "Claude Sonnet 4 with reasoning enabled for step-by-step thinking.",
+        "providerId": "anthropic",
+        "modelId": "claude-sonnet-4-20250514",
+        "settings": {
+            "temperature": 0.7,
+            "reasoning": {
+                "enabled": true
+            }
         }
     },
     {
@@ -16,7 +32,23 @@
         "providerId": "anthropic",
         "modelId": "claude-opus-4-20250514",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7,
+            "reasoning": {
+                "enabled": false
+            }
+        }
+    },
+    {
+        "id": "anthropic-claude-opus-4-20250514-thinking",
+        "displayName": "Anthropic - Claude Opus 4 (Thinking)",
+        "description": "Claude Opus 4 with reasoning enabled for complex problem solving.",
+        "providerId": "anthropic",
+        "modelId": "claude-opus-4-20250514",
+        "settings": {
+            "temperature": 0.7,
+            "reasoning": {
+                "enabled": true
+            }
         }
     },
     {
@@ -26,7 +58,23 @@
         "providerId": "anthropic",
         "modelId": "claude-3-7-sonnet-20250219",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7,
+            "reasoning": {
+                "enabled": false
+            }
+        }
+    },
+    {
+        "id": "anthropic-claude-3-7-sonnet-20250219-thinking",
+        "displayName": "Anthropic - Claude 3.7 Sonnet (Thinking)",
+        "description": "Claude 3.7 Sonnet with full reasoning output for detailed analysis.",
+        "providerId": "anthropic",
+        "modelId": "claude-3-7-sonnet-20250219",
+        "settings": {
+            "temperature": 0.7,
+            "reasoning": {
+                "enabled": true
+            }
         }
     },
     {
@@ -36,7 +84,7 @@
         "providerId": "anthropic",
         "modelId": "claude-3-5-sonnet-20241022",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7
         }
     },
     {
@@ -46,7 +94,7 @@
         "providerId": "anthropic",
         "modelId": "claude-3-5-haiku-20241022",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7
         }
     },
     {
@@ -56,7 +104,7 @@
         "providerId": "gemini",
         "modelId": "gemini-2.5-pro",
         "settings": {
-            "temperature": 0.3,
+            "temperature": 0.7,
             "geminiSafetySettings": [
                 { "category": "HARM_CATEGORY_HATE_SPEECH", "threshold": "BLOCK_NONE" },
                 {
@@ -78,7 +126,7 @@
         "providerId": "gemini",
         "modelId": "gemini-2.5-flash",
         "settings": {
-            "temperature": 0.3,
+            "temperature": 0.7,
             "geminiSafetySettings": [
                 { "category": "HARM_CATEGORY_HATE_SPEECH", "threshold": "BLOCK_NONE" },
                 {
@@ -90,7 +138,35 @@
                     "threshold": "BLOCK_NONE"
                 },
                 { "category": "HARM_CATEGORY_HARASSMENT", "threshold": "BLOCK_NONE" }
-            ]
+            ],
+            "reasoning": {
+                "enabled": false
+            }
+        }
+    },
+    {
+        "id": "google-gemini-2.5-flash-thinking",
+        "displayName": "Google - Gemini 2.5 Flash (Thinking)",
+        "description": "Gemini 2.5 Flash with dynamic reasoning for adaptive problem solving.",
+        "providerId": "gemini",
+        "modelId": "gemini-2.5-flash",
+        "settings": {
+            "temperature": 0.7,
+            "geminiSafetySettings": [
+                { "category": "HARM_CATEGORY_HATE_SPEECH", "threshold": "BLOCK_NONE" },
+                {
+                    "category": "HARM_CATEGORY_SEXUALLY_EXPLICIT",
+                    "threshold": "BLOCK_NONE"
+                },
+                {
+                    "category": "HARM_CATEGORY_DANGEROUS_CONTENT",
+                    "threshold": "BLOCK_NONE"
+                },
+                { "category": "HARM_CATEGORY_HARASSMENT", "threshold": "BLOCK_NONE" }
+            ],
+            "reasoning": {
+                "enabled": true
+            }
         }
     },
     {
@@ -100,7 +176,7 @@
         "providerId": "gemini",
         "modelId": "gemini-2.5-flash-lite-preview-06-17",
         "settings": {
-            "temperature": 0.3,
+            "temperature": 0.7,
             "geminiSafetySettings": [
                 { "category": "HARM_CATEGORY_HATE_SPEECH", "threshold": "BLOCK_NONE" },
                 {
@@ -112,7 +188,35 @@
                     "threshold": "BLOCK_NONE"
                 },
                 { "category": "HARM_CATEGORY_HARASSMENT", "threshold": "BLOCK_NONE" }
-            ]
+            ],
+            "reasoning": {
+                "enabled": false
+            }
+        }
+    },
+    {
+        "id": "google-gemini-2.5-flash-lite-preview-thinking",
+        "displayName": "Google - Gemini 2.5 Flash-Lite Preview (Thinking)",
+        "description": "Gemini 2.5 Flash-Lite with dynamic reasoning for efficient thinking.",
+        "providerId": "gemini",
+        "modelId": "gemini-2.5-flash-lite-preview-06-17",
+        "settings": {
+            "temperature": 0.7,
+            "geminiSafetySettings": [
+                { "category": "HARM_CATEGORY_HATE_SPEECH", "threshold": "BLOCK_NONE" },
+                {
+                    "category": "HARM_CATEGORY_SEXUALLY_EXPLICIT",
+                    "threshold": "BLOCK_NONE"
+                },
+                {
+                    "category": "HARM_CATEGORY_DANGEROUS_CONTENT",
+                    "threshold": "BLOCK_NONE"
+                },
+                { "category": "HARM_CATEGORY_HARASSMENT", "threshold": "BLOCK_NONE" }
+            ],
+            "reasoning": {
+                "enabled": true
+            }
         }
     },
     {
@@ -122,7 +226,7 @@
         "providerId": "gemini",
         "modelId": "gemini-2.0-flash",
         "settings": {
-            "temperature": 0.3,
+            "temperature": 0.7,
             "geminiSafetySettings": [
                 { "category": "HARM_CATEGORY_HATE_SPEECH", "threshold": "BLOCK_NONE" },
                 {
@@ -144,7 +248,7 @@
         "providerId": "gemini",
         "modelId": "gemini-2.0-flash-lite",
         "settings": {
-            "temperature": 0.3,
+            "temperature": 0.7,
             "geminiSafetySettings": [
                 { "category": "HARM_CATEGORY_HATE_SPEECH", "threshold": "BLOCK_NONE" },
                 {
@@ -176,7 +280,7 @@
         "providerId": "openai",
         "modelId": "gpt-4.1",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7
         }
     },
     {
@@ -186,7 +290,7 @@
         "providerId": "openai",
         "modelId": "gpt-4.1-mini",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7
         }
     },
     {
@@ -196,7 +300,7 @@
         "providerId": "openai",
         "modelId": "gpt-4.1-nano",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7
         }
     },
     {
@@ -206,7 +310,7 @@
         "providerId": "mistral",
         "modelId": "codestral-2501",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7
         }
     },
     {
@@ -216,7 +320,7 @@
         "providerId": "mistral",
         "modelId": "devstral-small-2505",
         "settings": {
-            "temperature": 0.3
+            "temperature": 0.7
         }
     }
 ]

package/dist/index.d.ts

@@ -1,11 +1,11 @@
 export type { ApiKeyProvider } from "./types";
 export { LLMService } from "./llm/LLMService";
-export type { LLMServiceOptions, PresetMode } from "./llm/LLMService";
+export type { LLMServiceOptions, PresetMode, CreateMessagesResult } from "./llm/LLMService";
 export type { ModelPreset } from "./types/presets";
 export * from "./llm/types";
 export * from "./llm/clients/types";
 export { fromEnvironment } from "./providers/fromEnvironment";
 export { renderTemplate } from "./prompting/template";
 export { countTokens, getSmartPreview, extractRandomVariables } from "./prompting/content";
-export { buildMessagesFromTemplate } from "./prompting/builder";
-export { parseStructuredContent } from "./prompting/parser";
+export { parseStructuredContent, parseRoleTags, extractInitialTaggedContent, parseTemplateWithMetadata } from "./prompting/parser";
+export type { TemplateMetadata } from "./prompting/parser";

package/dist/index.js

@@ -14,7 +14,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.parseStructuredContent = exports.buildMessagesFromTemplate = exports.extractRandomVariables = exports.getSmartPreview = exports.countTokens = exports.renderTemplate = exports.fromEnvironment = exports.LLMService = void 0;
+exports.parseTemplateWithMetadata = exports.extractInitialTaggedContent = exports.parseRoleTags = exports.parseStructuredContent = exports.extractRandomVariables = exports.getSmartPreview = exports.countTokens = exports.renderTemplate = exports.fromEnvironment = exports.LLMService = void 0;
 // --- LLM Service ---
 var LLMService_1 = require("./llm/LLMService");
 Object.defineProperty(exports, "LLMService", { enumerable: true, get: function () { return LLMService_1.LLMService; } });
@@ -32,7 +32,8 @@ var content_1 = require("./prompting/content");
 Object.defineProperty(exports, "countTokens", { enumerable: true, get: function () { return content_1.countTokens; } });
 Object.defineProperty(exports, "getSmartPreview", { enumerable: true, get: function () { return content_1.getSmartPreview; } });
 Object.defineProperty(exports, "extractRandomVariables", { enumerable: true, get: function () { return content_1.extractRandomVariables; } });
-var builder_1 = require("./prompting/builder");
-Object.defineProperty(exports, "buildMessagesFromTemplate", { enumerable: true, get: function () { return builder_1.buildMessagesFromTemplate; } });
 var parser_1 = require("./prompting/parser");
 Object.defineProperty(exports, "parseStructuredContent", { enumerable: true, get: function () { return parser_1.parseStructuredContent; } });
+Object.defineProperty(exports, "parseRoleTags", { enumerable: true, get: function () { return parser_1.parseRoleTags; } });
+Object.defineProperty(exports, "extractInitialTaggedContent", { enumerable: true, get: function () { return parser_1.extractInitialTaggedContent; } });
+Object.defineProperty(exports, "parseTemplateWithMetadata", { enumerable: true, get: function () { return parser_1.parseTemplateWithMetadata; } });

package/dist/llm/LLMService.createMessages.test.d.ts

@@ -0,0 +1,4 @@
+/**
+ * Tests for LLMService.createMessages method
+ */
+export {};

package/dist/llm/LLMService.createMessages.test.js

@@ -0,0 +1,364 @@
+"use strict";
+/**
+ * Tests for LLMService.createMessages method
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+const LLMService_1 = require("./LLMService");
+// Create a mock API key provider
+const mockApiKeyProvider = async () => 'test-api-key';
+describe('LLMService.createMessages', () => {
+    let service;
+    beforeEach(() => {
+        service = new LLMService_1.LLMService(mockApiKeyProvider);
+    });
+    describe('Basic template parsing', () => {
+        it('should parse simple template without model context', async () => {
+            const result = await service.createMessages({
+                template: 'Hello, how can I help you?'
+            });
+            expect(result.messages).toEqual([
+                { role: 'user', content: 'Hello, how can I help you?' }
+            ]);
+            expect(result.modelContext).toBeNull();
+        });
+        it('should parse multi-turn template without model context', async () => {
+            const result = await service.createMessages({
+                template: `
+          <SYSTEM>You are a helpful assistant.</SYSTEM>
+          <USER>Hello</USER>
+          <ASSISTANT>Hi! How can I help you today?</ASSISTANT>
+          <USER>Can you explain {{topic}}?</USER>
+        `,
+                variables: { topic: 'promises in JavaScript' }
+            });
+            expect(result.messages).toEqual([
+                { role: 'system', content: 'You are a helpful assistant.' },
+                { role: 'user', content: 'Hello' },
+                { role: 'assistant', content: 'Hi! How can I help you today?' },
+                { role: 'user', content: 'Can you explain promises in JavaScript?' }
+            ]);
+            expect(result.modelContext).toBeNull();
+        });
+    });
+    describe('Model-aware templates', () => {
+        it('should inject model context for valid preset', async () => {
+            const result = await service.createMessages({
+                template: `
+          <SYSTEM>You are a {{ thinking_enabled ? "thoughtful" : "standard" }} assistant.</SYSTEM>
+          <USER>Help me understand {{concept}}</USER>
+        `,
+                variables: { concept: 'recursion' },
+                presetId: 'anthropic-claude-3-7-sonnet-20250219-thinking'
+            });
+            expect(result.modelContext).not.toBeNull();
+            expect(result.modelContext?.thinking_enabled).toBe(true);
+            expect(result.modelContext?.thinking_available).toBe(true);
+            expect(result.modelContext?.model_id).toBe('claude-3-7-sonnet-20250219');
+            expect(result.modelContext?.provider_id).toBe('anthropic');
+            expect(result.messages).toEqual([
+                { role: 'system', content: 'You are a thoughtful assistant.' },
+                { role: 'user', content: 'Help me understand recursion' }
+            ]);
+        });
+        it('should inject model context for valid provider/model combo', async () => {
+            const result = await service.createMessages({
+                template: 'Model: {{model_id}}, Provider: {{provider_id}}',
+                providerId: 'openai',
+                modelId: 'gpt-4.1'
+            });
+            expect(result.modelContext).not.toBeNull();
+            expect(result.modelContext?.model_id).toBe('gpt-4.1');
+            expect(result.modelContext?.provider_id).toBe('openai');
+            expect(result.messages).toEqual([
+                { role: 'user', content: 'Model: gpt-4.1, Provider: openai' }
+            ]);
+        });
+        it('should handle model without reasoning support', async () => {
+            const result = await service.createMessages({
+                template: 'Thinking available: {{thinking_available}}, enabled: {{thinking_enabled}}',
+                presetId: 'openai-gpt-4.1-default'
+            });
+            expect(result.modelContext?.thinking_available).toBe(false);
+            expect(result.modelContext?.thinking_enabled).toBe(false);
+        });
+    });
+    describe('Complex scenarios', () => {
+        it('should handle variables that inject role tags', async () => {
+            const result = await service.createMessages({
+                template: `
+          <SYSTEM>Base system prompt</SYSTEM>
+          {{extraMessages}}
+          <USER>Final question</USER>
+        `,
+                variables: {
+                    extraMessages: '<USER>First question</USER>\n<ASSISTANT>First answer</ASSISTANT>'
+                }
+            });
+            expect(result.messages).toEqual([
+                { role: 'system', content: 'Base system prompt' },
+                { role: 'user', content: 'First question' },
+                { role: 'assistant', content: 'First answer' },
+                { role: 'user', content: 'Final question' }
+            ]);
+        });
+        it('should handle conditional role injection based on model context', async () => {
+            const result = await service.createMessages({
+                template: `
+          {{ thinking_enabled ? '<SYSTEM>Think step-by-step before answering.</SYSTEM>' : '' }}
+          <USER>Solve: {{problem}}</USER>
+        `,
+                variables: { problem: 'What is 15% of 240?' },
+                presetId: 'anthropic-claude-3-7-sonnet-20250219-thinking'
+            });
+            expect(result.messages).toEqual([
+                { role: 'system', content: 'Think step-by-step before answering.' },
+                { role: 'user', content: 'Solve: What is 15% of 240?' }
+            ]);
+        });
+        it('should handle nested conditionals with model context', async () => {
+            const result = await service.createMessages({
+                template: `
+          <SYSTEM>
+            You are using {{ model_id || "no model" }}.
+            {{ thinking_available ? 'You have reasoning capabilities.' : 'Standard model.' }}
+          </SYSTEM>
+          <USER>Hello</USER>
+        `,
+                providerId: 'anthropic',
+                modelId: 'claude-3-5-haiku-20241022'
+            });
+            expect(result.messages[0].role).toBe('system');
+            // Check that we have model context
+            expect(result.modelContext).not.toBeNull();
+            if (result.modelContext) {
+                expect(result.modelContext.model_id).toBe('claude-3-5-haiku-20241022');
+                expect(result.modelContext.thinking_available).toBe(false);
+            }
+        });
+    });
+    describe('Error handling', () => {
+        it('should proceed without model context on invalid preset', async () => {
+            const result = await service.createMessages({
+                template: 'Has model context: {{model_id ? "yes" : "no"}}',
+                presetId: 'invalid-preset-id'
+            });
+            expect(result.modelContext).toBeNull();
+            expect(result.messages).toEqual([
+                { role: 'user', content: 'Has model context: no' }
+            ]);
+        });
+        it('should handle invalid template syntax gracefully', async () => {
+            const result = await service.createMessages({
+                template: 'Unclosed conditional: {{ if true',
+                variables: {}
+            });
+            // The template engine doesn't throw errors for invalid syntax, it renders as-is
+            expect(result.messages).toEqual([
+                { role: 'user', content: 'Unclosed conditional: {{ if true' }
+            ]);
+        });
+        it('should handle empty template', async () => {
+            const result = await service.createMessages({
+                template: ''
+            });
+            expect(result.messages).toEqual([]);
+            expect(result.modelContext).toBeNull();
+        });
+        it('should handle whitespace-only template', async () => {
+            const result = await service.createMessages({
+                template: '   \n\t   '
+            });
+            expect(result.messages).toEqual([]);
+            expect(result.modelContext).toBeNull();
+        });
+    });
+    describe('Integration with reasoning settings', () => {
+        it('should handle reasoning settings in model context', async () => {
+            const result = await service.createMessages({
+                template: `
+          Thinking enabled: {{thinking_enabled}}
+          Thinking available: {{thinking_available}}
+          Model: {{model_id}}
+        `,
+                presetId: 'anthropic-claude-3-7-sonnet-20250219-thinking'
+            });
+            expect(result.modelContext?.thinking_enabled).toBe(true);
+            expect(result.modelContext?.thinking_available).toBe(true);
+            expect(result.messages[0].content).toContain('Thinking enabled: true');
+            expect(result.messages[0].content).toContain('Thinking available: true');
+        });
+        it('should handle models with always-on reasoning', async () => {
+            const result = await service.createMessages({
+                template: 'Provider: {{provider_id}}, Model: {{model_id}}',
+                providerId: 'gemini',
+                modelId: 'gemini-2.5-pro'
+            });
+            expect(result.modelContext).not.toBeNull();
+            expect(result.modelContext?.provider_id).toBe('gemini');
+            expect(result.modelContext?.model_id).toBe('gemini-2.5-pro');
+        });
+    });
+    describe('Variable precedence', () => {
+        it('should allow user variables to override model context', async () => {
+            const result = await service.createMessages({
+                template: 'Model: {{model_id}}',
+                variables: { model_id: 'user-override' },
+                presetId: 'openai-gpt-4.1-default'
+            });
+            expect(result.messages).toEqual([
+                { role: 'user', content: 'Model: user-override' }
+            ]);
+        });
+        it('should merge variables correctly', async () => {
+            const result = await service.createMessages({
+                template: 'Model: {{model_id}}, Task: {{task}}, Thinking: {{thinking_enabled}}',
+                variables: { task: 'code review' },
+                presetId: 'anthropic-claude-3-7-sonnet-20250219-thinking'
+            });
+            expect(result.messages[0].content).toBe('Model: claude-3-7-sonnet-20250219, Task: code review, Thinking: true');
+        });
+    });
+    describe('Template metadata parsing', () => {
+        it('should extract settings from META block', async () => {
+            const result = await service.createMessages({
+                template: `<META>
+{
+  "settings": {
+    "temperature": 0.9,
+    "thinkingExtraction": { "enabled": true, "tag": "reasoning" }
+  }
+}
+</META>
+<SYSTEM>You are a creative writer.</SYSTEM>
+<USER>Write a story about {{topic}}</USER>`,
+                variables: { topic: 'a robot discovering music' }
+            });
+            expect(result.messages).toEqual([
+                { role: 'system', content: 'You are a creative writer.' },
+                { role: 'user', content: 'Write a story about a robot discovering music' }
+            ]);
+            expect(result.settings).toEqual({
+                temperature: 0.9,
+                thinkingExtraction: { enabled: true, tag: 'reasoning' }
+            });
+        });
+        it('should return empty settings when no META block exists', async () => {
+            const result = await service.createMessages({
+                template: '<USER>Simple message</USER>'
+            });
+            expect(result.messages).toEqual([
+                { role: 'user', content: 'Simple message' }
+            ]);
+            expect(result.settings).toEqual({});
+        });
+        it('should handle invalid settings in META block with warnings', async () => {
+            const consoleWarnSpy = jest.spyOn(console, 'warn').mockImplementation();
+            const result = await service.createMessages({
+                template: `<META>
+{
+  "settings": {
+    "temperature": 3.0,
+    "unknownSetting": "value",
+    "maxTokens": -50
+  }
+}
+</META>
+<USER>Test</USER>`
+            });
+            expect(result.messages).toEqual([
+                { role: 'user', content: 'Test' }
+            ]);
+            expect(result.settings).toEqual({}); // All settings were invalid
+            expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining('Invalid temperature value'));
+            expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining('Unknown setting "unknownSetting"'));
+            expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining('Invalid maxTokens value'));
+            consoleWarnSpy.mockRestore();
+        });
+        it('should work with model context and META settings', async () => {
+            const result = await service.createMessages({
+                template: `<META>
+{
+  "settings": {
+    "temperature": 0.7,
+    "maxTokens": 2000
+  }
+}
+</META>
+<SYSTEM>You are a {{ thinking_enabled ? "thoughtful" : "quick" }} assistant.</SYSTEM>
+<USER>Help me understand {{concept}}</USER>`,
+                variables: { concept: 'recursion' },
+                presetId: 'anthropic-claude-3-7-sonnet-20250219-thinking'
+            });
+            expect(result.messages).toEqual([
+                { role: 'system', content: 'You are a thoughtful assistant.' },
+                { role: 'user', content: 'Help me understand recursion' }
+            ]);
+            expect(result.settings).toEqual({
+                temperature: 0.7,
+                maxTokens: 2000
+            });
+            expect(result.modelContext).not.toBeNull();
+            expect(result.modelContext?.thinking_enabled).toBe(true);
+        });
+        it('should validate complex nested settings', async () => {
+            const result = await service.createMessages({
+                template: `<META>
+{
+  "settings": {
+    "reasoning": {
+      "enabled": true,
+      "effort": "high",
+      "maxTokens": 5000
+    },
+    "stopSequences": ["\\n\\n", "END"],
+    "frequencyPenalty": 0.5
+  }
+}
+</META>
+<USER>Complex request</USER>`
+            });
+            expect(result.settings).toEqual({
+                reasoning: {
+                    enabled: true,
+                    effort: 'high',
+                    maxTokens: 5000
+                },
+                stopSequences: ['\n\n', 'END'],
+                frequencyPenalty: 0.5
+            });
+        });
+        it('should handle invalid nested settings gracefully', async () => {
+            const consoleWarnSpy = jest.spyOn(console, 'warn').mockImplementation();
+            const result = await service.createMessages({
+                template: `<META>
+{
+  "settings": {
+    "reasoning": {
+      "enabled": "yes",
+      "effort": "maximum",
+      "maxTokens": -1000
+    }
+  }
+}
+</META>
+<USER>Test</USER>`
+            });
+            expect(result.settings).toEqual({}); // All fields were invalid, so empty object
+            expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining('Invalid reasoning.enabled'));
+            expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining('Invalid reasoning.effort'));
+            expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining('Invalid reasoning.maxTokens'));
+            consoleWarnSpy.mockRestore();
+        });
+        it('should maintain backward compatibility for callers not using settings', async () => {
+            // Old code that destructures without settings should still work
+            const { messages, modelContext } = await service.createMessages({
+                template: `<META>{"settings": {"temperature": 0.8}}</META><USER>Test</USER>`
+            });
+            expect(messages).toEqual([
+                { role: 'user', content: 'Test' }
+            ]);
+            expect(modelContext).toBeNull();
+            // settings field exists but old code doesn't need to know about it
+        });
+    });
+});