@flowselections/floriday-voorraad 1.0.139 → 1.0.141

package/dist-lib/lib/__tests__/release-140.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=release-140.test.d.ts.map

package/dist-lib/lib/__tests__/release-140.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"release-140.test.d.ts","sourceRoot":"","sources":["../../../src/lib/__tests__/release-140.test.ts"],"names":[],"mappings":""}

package/dist-lib/lib/__tests__/release-140.test.js

@@ -0,0 +1,42 @@
+import { describe, expect, it } from "vitest";
+import { readFileSync } from "node:fs";
+import { opslaanMelding, valuesAfterSave, OPSLAAN_MISLUKT_TEKST } from "../stock-save.js";
+import { dbFoutTekst } from "../db-error-message.js";
+// @ts-expect-error plain ESM script
+import { controleerSchema } from "../../../scripts/check-schema-version.mjs";
+const saved = { physical_stock: 10, minimum_stock: 0, reserved_stock: 2, incoming_stock: 0, notes: null };
+const entered = { ...saved, physical_stock: 100 };
+describe("1.0.140 opslaan bij dichte poort", () => {
+    it("bevestigde opslag met dichte poort: 'Opgeslagen; er is niets naar Floriday gestuurd.'", () => {
+        const m = opslaanMelding({ opgeslagen: true, floriday_klaargezet_toegestaan: false }, true);
+        expect(m).toEqual({ ok: true, tekst: "Opgeslagen; er is niets naar Floriday gestuurd." });
+    });
+    it("teruggedraaide opslag (fout, geen antwoord) = mislukt en oude waarden terug", () => {
+        const m = opslaanMelding(null, true);
+        expect(m).toEqual({ ok: false, tekst: OPSLAAN_MISLUKT_TEKST });
+        expect(valuesAfterSave(m.ok, entered, saved).physical_stock).toBe(10);
+    });
+    it("antwoord zonder bevestiging telt niet als opgeslagen", () => {
+        expect(opslaanMelding({ opgeslagen: false }, true).ok).toBe(false);
+    });
+});
+describe("1.0.140 databasefouten in gewone taal", () => {
+    it("ontbrekende controlewaarde wordt 'nog niet beschikbaar'", () => {
+        const t = dbFoutTekst({ code: "23514", message: 'new row for relation "stock_mutation_log" violates check constraint' }, "Afboeken");
+        expect(t).toBe("Afboeken is nog niet beschikbaar in deze omgeving, neem contact op met de beheerder.");
+    });
+    it("oude poortfout toont geen technische tekst", () => {
+        const t = dbFoutTekst({ message: "Uitgaand Floriday-verkeer (stock_publish) is uitgeschakeld" }, "Opslaan");
+        expect(t).not.toContain("stock_publish");
+    });
+});
+describe("1.0.140 schemacontrole", () => {
+    it("ouder schema blokkeert", () => {
+        expect(controleerSchema("-- @flowselections/floriday-voorraad 1.0.136 x", "1.0.140").length).toBeGreaterThan(0);
+    });
+    it("het meegeleverde schema hoort bij de versie in package.json", () => {
+        const versie = JSON.parse(readFileSync("package.json", "utf8")).version;
+        const sql = readFileSync("sql/floriday-voorraad-schema.sql", "utf8");
+        expect(controleerSchema(sql, versie)).toEqual([]);
+    });
+});

package/dist-lib/lib/db-error-message.d.ts

@@ -0,0 +1,7 @@
+/**
+ * Databasefouten nooit letterlijk tonen (1.0.140). De technische tekst gaat
+ * alleen naar de console; de gebruiker krijgt gewone taal.
+ */
+export type DbFoutActie = "Opslaan" | "Afboeken" | "Geschiedenis laden" | "Deze actie";
+export declare function dbFoutTekst(e: unknown, actie?: DbFoutActie): string;
+//# sourceMappingURL=db-error-message.d.ts.map

package/dist-lib/lib/db-error-message.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"db-error-message.d.ts","sourceRoot":"","sources":["../../src/lib/db-error-message.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,MAAM,MAAM,WAAW,GAAG,SAAS,GAAG,UAAU,GAAG,oBAAoB,GAAG,YAAY,CAAC;AAmBvF,wBAAgB,WAAW,CAAC,CAAC,EAAE,OAAO,EAAE,KAAK,GAAE,WAA0B,GAAG,MAAM,CA0BjF"}

package/dist-lib/lib/db-error-message.js

@@ -0,0 +1,36 @@
+/** Eigen Nederlandse meldingen uit onze functies (ERRCODE 22023/P0002/42501) mogen wel door. */
+const EIGEN_MELDINGEN = [
+    /^Kies een reden/i,
+    /^Vul een (aantal|voorraadaantal)/i,
+    /^Je kunt niet meer afboeken/i,
+    /^Voorraadaantallen mogen niet negatief/i,
+    /^Geen voorraadregel/i,
+    /^Je hebt geen schrijfrechten/i,
+    /^Onvoldoende voorraad/i,
+];
+export function dbFoutTekst(e, actie = "Deze actie") {
+    const err = (e ?? {});
+    const raw = String(err.message ?? (e instanceof Error ? e.message : e ?? ""));
+    const code = String(err.code ?? "");
+    const lower = raw.toLowerCase();
+    if (typeof console !== "undefined")
+        console.warn(`[voorraad] ${actie} mislukt:`, raw, code);
+    const eigen = EIGEN_MELDINGEN.find((re) => re.test(raw.trim()));
+    if (eigen)
+        return raw.trim();
+    if (lower.includes("uitgaand floriday-verkeer") || lower.includes("stock_publish")) {
+        return `${actie} is nog niet beschikbaar in deze omgeving (de databaseversie is ouder dan de app), neem contact op met de beheerder.`;
+    }
+    if (code === "42883" || code === "PGRST202" || code === "23514" ||
+        lower.includes("could not find the function") || lower.includes("does not exist") ||
+        lower.includes("violates check constraint")) {
+        return `${actie} is nog niet beschikbaar in deze omgeving, neem contact op met de beheerder.`;
+    }
+    if (code === "42501" || lower.includes("permission denied") || lower.includes("niet ingelogd")) {
+        return "Je hebt geen rechten voor deze actie. Log opnieuw in of vraag de beheerder.";
+    }
+    if (lower.includes("failed to fetch") || lower.includes("network")) {
+        return "Geen verbinding. Controleer je internet en probeer het opnieuw.";
+    }
+    return `${actie} is niet gelukt. Probeer het opnieuw of neem contact op met de beheerder.`;
+}

package/dist-lib/lib/stock-save.d.ts

@@ -29,4 +29,15 @@ export type AfboekReden = (typeof AFBOEK_REDENEN)[number]["value"];
 export declare function afboekFout(aantal: number, reden: string | null, liggend: number): string | null;
 /** Melding na een geslaagde lokale opslag van een Floriday-artikel. */
 export declare function floridayNaOpslaanTekst(isFloridayArtikel: boolean, poortOpen: boolean | null): string | null;
+/**
+ * 1.0.140: alleen een bevestiging van de database ({ opgeslagen: true }) telt
+ * als opgeslagen. Alles anders (fout, leeg antwoord, teruggedraaid) = mislukt.
+ */
+export declare function opslagBevestigd(result: unknown): boolean;
+export declare const OPSLAAN_MISLUKT_TEKST = "Opslaan mislukt, er is niets gewijzigd";
+/** Melding na opslaan; null-resultaat of fout = mislukt (waarden terugzetten). */
+export declare function opslaanMelding(result: unknown, isFloridayArtikel: boolean): {
+    ok: boolean;
+    tekst: string;
+};
 //# sourceMappingURL=stock-save.d.ts.map

package/dist-lib/lib/stock-save.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"stock-save.d.ts","sourceRoot":"","sources":["../../src/lib/stock-save.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AACH,MAAM,WAAW,WAAW;IAC1B,cAAc,EAAE,MAAM,CAAC;IACvB,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,CAAC;IACvB,cAAc,EAAE,MAAM,CAAC;IACvB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;CACtB;AAED,wBAAgB,eAAe,CAAC,CAAC,SAAS,WAAW,EAAE,EAAE,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,GAAG,CAAC,CAG3F;AAED,eAAO,MAAM,cAAc;;;;;;;;;;;;EAKjB,CAAC;AAEX,MAAM,MAAM,WAAW,GAAG,CAAC,OAAO,cAAc,CAAC,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,CAAC;AAEnE,wBAAgB,UAAU,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAK/F;AAED,uEAAuE;AACvE,wBAAgB,sBAAsB,CAAC,iBAAiB,EAAE,OAAO,EAAE,SAAS,EAAE,OAAO,GAAG,IAAI,GAAG,MAAM,GAAG,IAAI,CAI3G"}
+{"version":3,"file":"stock-save.d.ts","sourceRoot":"","sources":["../../src/lib/stock-save.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AACH,MAAM,WAAW,WAAW;IAC1B,cAAc,EAAE,MAAM,CAAC;IACvB,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,CAAC;IACvB,cAAc,EAAE,MAAM,CAAC;IACvB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;CACtB;AAED,wBAAgB,eAAe,CAAC,CAAC,SAAS,WAAW,EAAE,EAAE,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,GAAG,CAAC,CAG3F;AAED,eAAO,MAAM,cAAc;;;;;;;;;;;;EAKjB,CAAC;AAEX,MAAM,MAAM,WAAW,GAAG,CAAC,OAAO,cAAc,CAAC,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,CAAC;AAEnE,wBAAgB,UAAU,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAK/F;AAED,uEAAuE;AACvE,wBAAgB,sBAAsB,CAAC,iBAAiB,EAAE,OAAO,EAAE,SAAS,EAAE,OAAO,GAAG,IAAI,GAAG,MAAM,GAAG,IAAI,CAI3G;AAED;;;GAGG;AACH,wBAAgB,eAAe,CAAC,MAAM,EAAE,OAAO,GAAG,OAAO,CAExD;AAED,eAAO,MAAM,qBAAqB,2CAA2C,CAAC;AAE9E,kFAAkF;AAClF,wBAAgB,cAAc,CAAC,MAAM,EAAE,OAAO,EAAE,iBAAiB,EAAE,OAAO,GAAG;IAAE,EAAE,EAAE,OAAO,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,CAK1G"}

package/dist-lib/lib/stock-save.js

@@ -26,3 +26,20 @@ export function floridayNaOpslaanTekst(isFloridayArtikel, poortOpen) {
         return null;
     return "Lokaal opgeslagen. Er gaat niets naar Floriday: versturen staat uit.";
 }
+/**
+ * 1.0.140: alleen een bevestiging van de database ({ opgeslagen: true }) telt
+ * als opgeslagen. Alles anders (fout, leeg antwoord, teruggedraaid) = mislukt.
+ */
+export function opslagBevestigd(result) {
+    return !!result && typeof result === "object" && result.opgeslagen === true;
+}
+export const OPSLAAN_MISLUKT_TEKST = "Opslaan mislukt, er is niets gewijzigd";
+/** Melding na opslaan; null-resultaat of fout = mislukt (waarden terugzetten). */
+export function opslaanMelding(result, isFloridayArtikel) {
+    if (!opslagBevestigd(result))
+        return { ok: false, tekst: OPSLAAN_MISLUKT_TEKST };
+    const klaar = result.floriday_klaargezet_toegestaan === true;
+    if (isFloridayArtikel && !klaar)
+        return { ok: true, tekst: "Opgeslagen; er is niets naar Floriday gestuurd." };
+    return { ok: true, tekst: "Voorraad opgeslagen" };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flowselections/floriday-voorraad",
-  "version": "1.0.139",
+  "version": "1.0.141",
   "sideEffects": false,
   "type": "module",
   "main": "./dist-lib/index.js",
@@ -32,6 +32,8 @@
     "preview": "vite preview",
     "lint": "eslint .",
     "test": "vitest run",
+    "check:schema": "node scripts/check-schema-version.mjs",
+    "prepublishOnly": "node scripts/check-schema-version.mjs",
     "test:watch": "vitest"
   },
   "peerDependencies": {

package/sql/AGENTS.md

@@ -5,3 +5,5 @@
 - An order line counts as cancelled for fust when `order_items.quarantined_at` is set or its order is cancelled (`voorraad_order_geannuleerd`: status geannuleerd/cancelled case-insensitive, `cancelled_at` or `deleted_at`): `sync_order_item_fusten` wants 0 and triggers on order_items (quarantine on/off) and orders (those three fields) book the difference against the line's own bookings, because quarantined or cancelled orders otherwise kept fust booked.
 - The correct fust count per order line comes only from `voorraad_fust_juist(order_item_id)` (same rule as `sync_order_item_fusten`; removed, quarantined, externally fulfilled or cancelled = 0); fust corrections compare against it, because every hand-built copy of the rule drifted.
 - Local stock saves never depend on the Floriday write gate: every Voorraad queue insert first checks `voorraad_floriday_klaarzetten_toegestaan()` and runs in its own exception block, because the Engine gate trigger raises and rolled back the local save.
+- A note per stock change travels via tx setting `stock.change_note` (read by the deferred log trigger at commit, so never clear it inside the call); `trade_item_stock.notes` is the fixed article note and changes only when the caller asks (`stock.keep_notes`), because appending each save stacked notes.
+- `sql/floriday-voorraad-schema.sql` must carry the package version and required objects; `scripts/check-schema-version.mjs` (prepublishOnly + test) blocks publishing otherwise, because 1.0.139 shipped with a 1.0.136 schema.