@aria-framework/ai 0.14.2 → 0.14.3

package/index.js

@@ -103,7 +103,12 @@ function createAiClient(deps = {}) {
     if (!cfg.enabled) {
       throw new AiError('disabled', 'AI assistance is switched off. An administrator can enable it in Settings.');
     }
-    if (!cfg.model) {
+    // A MODEL NAME IS REQUIRED OF EVERY PROVIDER THAT HAS ONE TO CONFIGURE — which is all of them
+    // except a supervised stack. There the model is a FACT the supervisor reports about an engine,
+    // not a setting: the operator picks an engine and whatever it is running answers. Insisting on
+    // a model name would make an lmx endpoint unusable by demanding the one field the design says
+    // not to store, and would say so with a message naming nothing an operator could go and fill in.
+    if (!cfg.model && cfg.provider !== 'lmx') {
       throw new AiError('unconfigured', 'No model name is configured.');
     }
     const adapter = PROVIDERS[cfg.provider];

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@aria-framework/ai",
   "description": "Aria App Framework — AI module. A dependency-injected model seam (createAiClient) over several providers (LM Studio / OpenAI-compatible / Anthropic), with a fact-preservation guard, generic Polish and Generate writing engines, and a browser polish widget. Prompts and config stay in the consuming app.",
-  "version": "0.14.2",
+  "version": "0.14.3",
   "license": "UNLICENSED",
   "private": false,
   "publishConfig": {
@@ -43,6 +43,7 @@
     "test": "node test/smoke.js && node test/usageStore.js && node test/providerStore.js && node test/speedStore.js && node test/health.js && node test/listModels.js && node test/benchmark.js && node test/lmxDiscovery.js && node test/lmx.js && node test/packaging.js && node test/views.js"
   },
   "devDependencies": {
-    "undici": "^8.10.0"
+    "undici": "^8.10.0",
+    "ejs": "^3.1.10"
   }
 }

package/providerStore.js

@@ -183,6 +183,16 @@ function createProviderStore(opts = {}) {
         maxTokens: Number(row.max_tokens) || 1024,
         contextTokens: Number(row.context_tokens) || 8192,
         dailyTokenCap: Number(row.daily_token_cap) || 0,
+        // WHICH ENGINE ON WHICH SUPERVISED STACK. Carried through because a `kind: lmx` config
+        // without it is unusable: the adapter has no address of its own to fall back on, and would
+        // report "no supervisor is configured" for a row that names one perfectly well.
+        //
+        // A THIRD LIST THAT HAS TO AGREE. FIELDS says what may be written, schemaFor() creates the
+        // columns, and this decides what a caller actually receives — a field can be stored
+        // correctly and still never arrive. The guard in test/providerStore.js now compares all
+        // three, having originally compared only the first two and missed exactly this.
+        lmxInstance: row.lmx_instance || null,
+        lmxEngine: row.lmx_engine || null,
         // Carried on the resolved config so a speed test can judge a result without a second read.
         // 0 means no expectation was recorded, which is different from "expected to be slow".
         minTokensPerSec: Number(row.min_tokens_per_sec) || 0