@isi-ui7/bos7-shared 0.4.4 → 0.4.5

package/dist/form-renderer-utils.d.ts

@@ -22,7 +22,28 @@ export declare function runFieldValidation<TData extends Record<string, unknown>
  * mungkin jadi medan yatim seperti di atas.
  */
 export declare function ringkasGalatMedan<TData extends Record<string, unknown>>(fieldErrors: Partial<Record<keyof TData, string>>, fields: FormField<TData>[], maks?: number): string | null;
-export declare function collectAllFields<TData extends Record<string, unknown>>(layout: FormLayout<TData>): FormField<TData>[];
+/**
+ * Seluruh field yang BOLEH divalidasi — section `hidden` DILEWATI, sama seperti
+ * `collectPageFields`.
+ *
+ * 🔴 Dulu ia nol menyaring apa pun, bahkan `hidden: true` literal. Akibatnya
+ * `validateAll` (Save) menuntut field yang aplikasinya sendiri sembunyikan,
+ * sementara `validatePage` (Next) melewatinya dengan benar — jadi operator
+ * lolos di setiap halaman lalu ditolak di langkah terakhir, untuk field yang
+ * nol ada di layar mana pun dan nol bisa diisi gestur apa pun.
+ *
+ * `mode`/`data` opsional supaya pemanggil lama tetap jalan; tanpa keduanya
+ * hanya `hidden: true` literal yang bisa dievaluasi — predikat fungsi
+ * diperlakukan TIDAK tersembunyi, arah aman untuk validasi.
+ *
+ * ⚠️ Konsekuensi yang wajib dijaga di sisi SKEMA: kalau satu key punya dua
+ * salinan field (satu tampil di `create`, satu di `edit`), keduanya WAJIB
+ * membawa `validation`. Sebelum perbaikan ini, salinan tersembunyi ikut
+ * divalidasi sehingga satu salinan bervalidasi terasa cukup — ketergantungan
+ * terbalik yang kini nol berlaku lagi. Skema yang bersandar padanya akan
+ * kehilangan "wajib diisi" tanpa satu galat pun.
+ */
+export declare function collectAllFields<TData extends Record<string, unknown>>(layout: FormLayout<TData>, mode?: FormMode, data?: TData): FormField<TData>[];
 /**
  * Field yang BOLEH divalidasi pada satu halaman.
  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@isi-ui7/bos7-shared",
-  "version": "0.4.4",
+  "version": "0.4.5",
   "description": "Shared auth7 and layout primitives for bos7 applications.",
   "license": "MIT",
   "repository": {
@@ -70,7 +70,7 @@
   "peerDependencies": {
     "@carbon/icons-react": "^11.71.0",
     "@carbon/react": "^1.97.0",
-    "@isi-ui7/corporate-themes": ">=0.2.14",
+    "@isi-ui7/corporate-themes": ">=0.2.15",
     "@isi-ui7/data-table": ">=0.2.5",
     "@isi-ui7/editable-table": ">=0.2.3",
     "@isi-ui7/i18n": ">=0.2.3",
@@ -106,14 +106,14 @@
     "vite": "^5.0.0",
     "vite-plugin-dts": "^3.6.0",
     "vitest": "^1.0.0",
+    "@isi-ui7/corporate-themes": "0.2.15",
     "@isi-ui7/data-table": "0.2.5",
     "@isi-ui7/editable-table": "0.2.3",
-    "@isi-ui7/i18n": "0.2.3",
-    "@isi-ui7/corporate-themes": "0.2.14",
     "@isi-ui7/lookup-input": "0.2.4",
+    "@isi-ui7/realtime": "0.2.3",
     "@isi-ui7/modal-manager": "0.2.2",
     "@isi-ui7/ui-shell": "0.2.8",
-    "@isi-ui7/realtime": "0.2.3"
+    "@isi-ui7/i18n": "0.2.3"
   },
   "dependencies": {
     "@carbon/icons-react": "^11.71.0",

package/src/form-renderer-utils.ts

@@ -155,13 +155,36 @@ export function ringkasGalatMedan<TData extends Record<string, unknown>>(
 
 // ── Layout helpers ────────────────────────────────────────────────────────────
 
+/**
+ * Seluruh field yang BOLEH divalidasi — section `hidden` DILEWATI, sama seperti
+ * `collectPageFields`.
+ *
+ * 🔴 Dulu ia nol menyaring apa pun, bahkan `hidden: true` literal. Akibatnya
+ * `validateAll` (Save) menuntut field yang aplikasinya sendiri sembunyikan,
+ * sementara `validatePage` (Next) melewatinya dengan benar — jadi operator
+ * lolos di setiap halaman lalu ditolak di langkah terakhir, untuk field yang
+ * nol ada di layar mana pun dan nol bisa diisi gestur apa pun.
+ *
+ * `mode`/`data` opsional supaya pemanggil lama tetap jalan; tanpa keduanya
+ * hanya `hidden: true` literal yang bisa dievaluasi — predikat fungsi
+ * diperlakukan TIDAK tersembunyi, arah aman untuk validasi.
+ *
+ * ⚠️ Konsekuensi yang wajib dijaga di sisi SKEMA: kalau satu key punya dua
+ * salinan field (satu tampil di `create`, satu di `edit`), keduanya WAJIB
+ * membawa `validation`. Sebelum perbaikan ini, salinan tersembunyi ikut
+ * divalidasi sehingga satu salinan bervalidasi terasa cukup — ketergantungan
+ * terbalik yang kini nol berlaku lagi. Skema yang bersandar padanya akan
+ * kehilangan "wajib diisi" tanpa satu galat pun.
+ */
 export function collectAllFields<TData extends Record<string, unknown>>(
   layout: FormLayout<TData>,
+  mode?: FormMode,
+  data?: TData,
 ): FormField<TData>[] {
   const sections = layout.type === "single-page"
     ? layout.sections
     : layout.pages.flatMap((p) => p.sections);
-  return sections.flatMap((s) => s.fields);
+  return sections.filter((s) => !isSectionHidden(s, mode, data)).flatMap((s) => s.fields);
 }
 
 /**

package/src/workflow/use-crud-form-hidden.test.tsx

@@ -0,0 +1,62 @@
+import { describe, it, expect } from "vitest";
+import { act, renderHook } from "@testing-library/react";
+import { useCrudForm } from "./use-crud-form";
+import type { CrudForm } from "../form-types";
+
+type D = Record<string, unknown>;
+
+/**
+ * `validateAll` (Save) dulu memakai `collectAllFields` TANPA mode, jadi ia
+ * menuntut field di section TERSEMBUNYI — sementara `validatePage` (Next)
+ * melewatinya dengan benar. Operator lolos di tiap halaman lalu ditolak di
+ * langkah terakhir, untuk field yang nol ada di layarnya.
+ *
+ * Diuji lewat PERILAKU handleSave, bukan lewat collectAllFields langsung:
+ * uji atas helper tetap hijau meski hook nol memakainya.
+ */
+function schema(sembunyi: boolean | ((m: string, d: D) => boolean)): CrudForm<D> {
+  return {
+    title: { create: "c", edit: "e", view: "v" },
+    emptyData: { tampak: "", tersembunyi: "" } as D,
+    createApiPath: "/api/x",
+    layout: {
+      type: "single-page",
+      sections: [
+        { title: "tampak", fields: [
+          { key: "tampak", label: "Tampak", type: "text", span: 4, validation: { required: true } }] },
+        { title: "rahasia", hidden: sembunyi as never, fields: [
+          { key: "tersembunyi", label: "Tersembunyi", type: "text", span: 4, validation: { required: true } }] },
+      ],
+    },
+  } as unknown as CrudForm<D>;
+}
+
+async function galat(s: CrudForm<D>, isi: D) {
+  const { result } = renderHook(() => useCrudForm<D>({ schema: s, mode: "create" }));
+  act(() => { result.current.setForm(isi); });
+  await act(async () => { await result.current.handleSave(); });
+  return { pesan: String(result.current.error ?? ""), medan: result.current.fieldErrors as Record<string,string> };
+}
+
+describe("Save nol menuntut field di section tersembunyi", () => {
+  it("hidden literal: field tersembunyi NOL memblokir Save", async () => {
+    const r = await galat(schema(true), { tampak: "x", tersembunyi: "" });
+    expect(r.medan.tersembunyi, "field tersembunyi nol boleh dituntut").toBeUndefined();
+  });
+
+  it("hidden predikat: sama", async () => {
+    const r = await galat(schema(() => true), { tampak: "x", tersembunyi: "" });
+    expect(r.medan.tersembunyi).toBeUndefined();
+  });
+
+  it("KONTROL POSITIF: section TERLIHAT tetap dituntut", async () => {
+    const r = await galat(schema(true), { tampak: "", tersembunyi: "" });
+    expect(r.medan.tampak, "penjaga masih menggigit utk yg terlihat").toBeTruthy();
+    expect(r.pesan).toContain("Tampak");
+  });
+
+  it("KONTROL POSITIF: section yang TIDAK tersembunyi tetap dituntut", async () => {
+    const r = await galat(schema(false), { tampak: "x", tersembunyi: "" });
+    expect(r.medan.tersembunyi, "hidden=false ⇒ tetap divalidasi").toBeTruthy();
+  });
+});

package/src/workflow/use-crud-form.ts

@@ -109,7 +109,7 @@ export function useCrudForm<TData extends Record<string, unknown>>({
   };
 
   const validateAll = () => {
-    const errs = runFieldValidation(collectAllFields(schema.layout), form, mode, labels);
+    const errs = runFieldValidation(collectAllFields(schema.layout, mode, form), form, mode, labels);
     if (schema.validate) {
       return { ...errs, ...schema.validate(form) };
     }