@flowselections/floriday-voorraad 1.0.137 → 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/__tests__/stock-save.test.d.ts

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

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

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

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

@@ -0,0 +1,35 @@
+import { describe, expect, it } from "vitest";
+import { afboekFout, floridayNaOpslaanTekst, valuesAfterSave } from "../stock-save.js";
+const saved = { physical_stock: 40, minimum_stock: 5, reserved_stock: 12, incoming_stock: 0, notes: null };
+describe("lokaal voorraad opslaan (1.0.138)", () => {
+    it("mislukte opslag zet de oude waarden terug (geen 'Beschikbaar: 100' zonder opslag)", () => {
+        const entered = { ...saved, physical_stock: 112 };
+        expect(valuesAfterSave(false, entered, saved).physical_stock).toBe(40);
+    });
+    it("gereserveerd komt na opslaan uit de opgeslagen rij, niet uit invoer", () => {
+        const entered = { ...saved, physical_stock: 100, reserved_stock: 0 };
+        const out = valuesAfterSave(true, entered, saved);
+        expect(out.physical_stock).toBe(100);
+        expect(out.reserved_stock).toBe(12);
+    });
+    it("Floriday-artikel bij dichte poort: lokaal opgeslagen, melding dat er niets naar Floriday gaat", () => {
+        expect(floridayNaOpslaanTekst(true, false)).toMatch(/niets naar Floriday/);
+        expect(floridayNaOpslaanTekst(true, null)).toMatch(/niets naar Floriday/);
+        expect(floridayNaOpslaanTekst(true, true)).toBeNull();
+        expect(floridayNaOpslaanTekst(false, false)).toBeNull();
+    });
+});
+describe("afboeken met verplichte reden", () => {
+    it("zonder reden geweigerd", () => {
+        expect(afboekFout(5, null, 40)).toBe("Kies een reden.");
+        expect(afboekFout(5, "zomaar", 40)).toBe("Kies een reden.");
+    });
+    it("met reden uit de vaste lijst toegestaan", () => {
+        for (const r of ["derving", "kapot", "eigen_gebruik", "overig"])
+            expect(afboekFout(5, r, 40)).toBeNull();
+    });
+    it("niet meer dan liggend, en groter dan 0", () => {
+        expect(afboekFout(41, "kapot", 40)).toMatch(/niet meer afboeken/);
+        expect(afboekFout(0, "kapot", 40)).toMatch(/groter dan 0/);
+    });
+});

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

@@ -0,0 +1,43 @@
+/**
+ * Regels rond het lokaal opslaan van voorraad (1.0.138).
+ * - Na een mislukte opslag staan weer de laatst opgeslagen waarden in beeld.
+ * - Gereserveerd komt altijd uit de opgeslagen rij (berekend uit orders), nooit uit invoer.
+ * - Afboeken vraagt altijd een reden uit een vaste lijst.
+ */
+export interface StockValues {
+    physical_stock: number;
+    minimum_stock: number;
+    reserved_stock: number;
+    incoming_stock: number;
+    notes: string | null;
+}
+export declare function valuesAfterSave<T extends StockValues>(ok: boolean, entered: T, saved: T): T;
+export declare const AFBOEK_REDENEN: readonly [{
+    readonly value: "derving";
+    readonly label: "Derving";
+}, {
+    readonly value: "kapot";
+    readonly label: "Kapot";
+}, {
+    readonly value: "eigen_gebruik";
+    readonly label: "Eigen gebruik";
+}, {
+    readonly value: "overig";
+    readonly label: "Overig";
+}];
+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

@@ -0,0 +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;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

@@ -0,0 +1,45 @@
+export function valuesAfterSave(ok, entered, saved) {
+    if (!ok)
+        return { ...saved };
+    return { ...entered, reserved_stock: saved.reserved_stock };
+}
+export const AFBOEK_REDENEN = [
+    { value: "derving", label: "Derving" },
+    { value: "kapot", label: "Kapot" },
+    { value: "eigen_gebruik", label: "Eigen gebruik" },
+    { value: "overig", label: "Overig" },
+];
+export function afboekFout(aantal, reden, liggend) {
+    if (!reden || !AFBOEK_REDENEN.some((r) => r.value === reden))
+        return "Kies een reden.";
+    if (!Number.isFinite(aantal) || aantal <= 0)
+        return "Vul een aantal groter dan 0 in.";
+    if (aantal > liggend)
+        return `Je kunt niet meer afboeken (${aantal}) dan er ligt (${liggend}).`;
+    return null;
+}
+/** Melding na een geslaagde lokale opslag van een Floriday-artikel. */
+export function floridayNaOpslaanTekst(isFloridayArtikel, poortOpen) {
+    if (!isFloridayArtikel)
+        return null;
+    if (poortOpen === true)
+        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.137",
+  "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

@@ -4,3 +4,6 @@
 - Fust rows are resolved only via `voorraad_active_fust_id(key, id)` (follows `merged_into`), and an order line books fust once: the ERP line adopts an earlier Floriday-ingest booking and ingest skips lines the ERP already booked, because `fust_key LIMIT 1` hit merged rows and both paths booked the same order.
 - 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.