@aria-framework/ai 0.20.0 → 0.22.0

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.20.0",
+  "version": "0.22.0",
   "license": "UNLICENSED",
   "private": false,
   "publishConfig": {
@@ -47,7 +47,7 @@
     }
   },
   "scripts": {
-    "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/lmxVerify.js && node test/lmxStore.js && node test/lmxStatus.js && node test/jobCard.js && node test/untrusted.js && node test/packaging.js && node test/views.js && node test/fenced.js"
+    "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/lmxVerify.js && node test/lmxStore.js && node test/lmxStatus.js && node test/jobCard.js && node test/untrusted.js && node test/packaging.js && node test/views.js && node test/fenced.js && node test/polish.js"
   },
   "devDependencies": {
     "undici": "^8.10.0",

package/polish.js

@@ -97,7 +97,27 @@ function system(opts) {
   return lines.join('\n');
 }
 
-function normalise(s) { return String(s).replace(/\s+/g, ' ').trim(); }
+/**
+ * Two texts are "the same" when they differ only in SPACING NOISE — not when they differ in LINE
+ * STRUCTURE.
+ *
+ * This was `replace(/\s+/g, ' ')`, which collapses newlines along with spaces, and that made the
+ * `unchanged` flag structurally wrong for the one mode whose entire job is line structure. `tidy`
+ * is instructed to "break walls of text into paragraphs, turn a sequence of steps into a list" —
+ * so a SUCCESSFUL tidy of a single paragraph changes nothing but newlines, and the panel said
+ * "The model returned the same text — nothing to change." directly beneath the new paragraph
+ * break and the model's own note explaining that it had added one.
+ *
+ * Two contradictory statements and an Accept button, inviting the reader to discard a correct
+ * result. Horizontal whitespace is still noise; a line break is content.
+ */
+function normalise(s) {
+  return String(s)
+    .replace(/\r\n?/g, '\n')     // line-ending flavour is not a change
+    .replace(/[ \t]+/g, ' ')     // how many spaces is not a change
+    .replace(/ *\n/g, '\n')      // a trailing space before a break is not a change
+    .trim();
+}
 
 /**
  * @param {(opts:object, cfg?:object) => Promise<object>} complete  the client's complete()

package/usageStore.js

@@ -114,7 +114,10 @@ function createUsageStore(opts = {}) {
 
       const placeholders = cols.map(() => '?').join(', ');
       const r = await driver.run(
-        `INSERT INTO ${table} (${cols.join(', ')}) VALUES (${placeholders})`, values);
+        // RETURNING, because the id is read back. Empty on SQLite, ` RETURNING "id"` on Postgres —
+        // and without it lastId is undefined there, so every caller gets { id: undefined }.
+        `INSERT INTO ${table} (${cols.join(', ')}) VALUES (${placeholders})${driver.dialect.returning('id')}`,
+        values);
       return { id: r.lastId };
     },