@nauticana/sail 0.9.3 → 0.9.5

package/README.md

@@ -634,19 +634,20 @@ The generic edit form renders each column from keel's metadata. By default:
 - **Read-only** applies only to primary keys (and FK columns in child rows).
 
 To override per column without changing the schema, seed keel's
-`column_display_attribute` table (`table_name, column_name, read_only,
+`column_display_attribute` table (`table_name, column_name, display_mode,
 display_width, display_rows`). sail honours three optional `TableColumn`
 fields it produces:
 
 | Field          | Effect |
 |----------------|--------|
-| `read_only`    | Field renders display-only (use for `*_at` audit timestamps, system-managed status). |
+| `DisplayMode`  | `R` read-only (shown display-only; **hidden when empty**, so a new record's audit fields disappear and on edit only populated ones show). `H` hidden everywhere — the new-record builder omits it so the DB default/sequence/trigger fills it. `D` editable, prefilled with the column default on a new record. `I` insert-only — editable while creating, locked once it exists. `U` update-stamp — display-only like `R` in the UI, but keel auto-sets it on every UPDATE (`now()` for timestamps, `user_id` for integers), so `updated_at`/`updated_by` need no DB trigger. Empty/NULL = editable. |
 | `display_rows` | Textarea height. `1` forces a single-line input — keep a column `TEXT` (so bulk-loaded values don't overflow) yet render it on one line. |
 | `display_width`| Field max-width in px. |
 
-No rows seeded = unchanged behaviour. Requires keel with the
-`column_display_attribute` feature; rebuild httpsrv after seeding (metadata
-is read once at startup).
+`R`/`H`/`I` are also enforced server-side by keel (excluded from generic
+INSERT/UPDATE), so the modes are real, not just cosmetic. No rows seeded =
+unchanged behaviour. Requires keel with the `column_display_attribute`
+feature; rebuild httpsrv after seeding (metadata is read once at startup).
 
 ## Account deletion / logout everywhere / push tokens
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nauticana/sail",
-  "version": "0.9.3",
+  "version": "0.9.5",
   "description": "Metadata-driven Angular component library for CRUD admin frontends backed by the keel Go backend. Tables, forms, navigation, OTP/2FA/social auth, billing, payout onboarding.",
   "main": "src/index.ts",
   "files": [

package/src/component/abstract/base_form.ts

@@ -6,6 +6,11 @@ export abstract class BaseForm extends BaseTable {
     title = '';
 
     override isReadOnly(fieldName: string): boolean {
+        // keel column_display_attribute: 'R' is always display-only; 'I'
+        // (insert-only) is editable while creating, locked once it exists.
+        const mode = this.getColumn(fieldName)?.DisplayMode;
+        if (mode === 'R' || mode === 'U') return true; // 'U' = keel-set audit stamp
+        if (mode === 'I') return !this.isNew;
         return !this.isNew && this.isKey(fieldName);
     }
 }

package/src/component/abstract/base_table.ts

@@ -169,7 +169,20 @@ export abstract class BaseTable {
                     if (field.SequenceName) {
                         return; // auto-generated by database sequence
                     }
-                    if (field.HasDefault && field.DataType === 'timestamp'
+                    // column_display_attribute modes (explicit, win over the
+                    // timestamp heuristic). H/R/U are server-managed — omit so the
+                    // DB default/keel stamp fills them. D = seed the column default.
+                    if (field.DisplayMode === 'H' || field.DisplayMode === 'R'
+                        || field.DisplayMode === 'U') {
+                        return;
+                    }
+                    if (field.DisplayMode === 'D') {
+                        record[field.PascalName] = field.DataType === 'boolean'
+                            ? field.DefaultValue === 'true'
+                            : (field.DefaultValue ?? '');
+                        return;
+                    }
+                    if (!field.DisplayMode && field.HasDefault && field.DataType === 'timestamp'
                         && !editableTimestamps.has(field.PascalName)) {
                         return; // audit column — server-managed, don't include
                     }

package/src/component/form/form_field.ts

@@ -54,10 +54,10 @@ export class DynamicField extends BaseTable {
         return this.col.InputType;
     }
 
-    /** Effective read-only: explicit [readonly] input OR the column's own
-     *  Readonly flag from keel's column_display_attribute. */
+    /** Read-only state for this field. The parent form computes it (keys,
+     *  FK columns, and column_display_attribute modes R/I) and feeds it in. */
     get isReadonly(): boolean {
-        return this.readonlyField() || !!this.col?.Readonly;
+        return this.readonlyField();
     }
 
     get isTextArea(): boolean {

package/src/component/form/form_record.ts

@@ -36,9 +36,17 @@ export class RecordForm {
             this.guiConfig.tableOverrides?.[this.tableName()]?.editableTimestamps ?? []
         );
         if (tableDef && tableDef.Columns && tableDef.Columns.length > 0) {
+            const record = this.record();
             return tableDef.Columns
                 .filter(col => !(col.HasDefault && col.DataType === 'timestamp'
                                  && !editableTimestamps.has(col.PascalName)))
+                // column_display_attribute: 'H' is never shown; 'R'/'U' (read-only
+                // and keel-set audit stamp) show display-only but hide when empty —
+                // so a new record's audit fields disappear and on edit only
+                // populated ones show.
+                .filter(col => col.DisplayMode !== 'H')
+                .filter(col => !((col.DisplayMode === 'R' || col.DisplayMode === 'U')
+                                 && this.isEmpty(record[col.PascalName])))
                 .sort((a, b) => a.Order - b.Order)
                 .map(col => col.PascalName)
                 .filter(key => !hidden.has(key) && recordKeys.has(key));
@@ -50,6 +58,10 @@ export class RecordForm {
         return Array.isArray(value);
     }
 
+    private isEmpty(value: any): boolean {
+        return value === null || value === undefined || value === '';
+    }
+
     onRecordUpdate(updates: {[key: string]: any}) {
         Object.assign(this.record(), updates);
     }

package/src/component/table/table_edit.ts

@@ -63,15 +63,18 @@ export class TableEdit extends BaseForm implements OnInit {
         if (rec) Object.assign(this.editableRecord, rec);
     }
 
+    /**
+     * Tabs are the table's child relations (rest_api_child metadata), not just
+     * whatever array fields the backend happened to embed. Seed an empty array
+     * for any child the response omitted so its grid renders (and accepts new rows).
+     */
     private buildTabs() {
-        this.tabCount = 0;
-        this.tabFields = [];
-        for (const field of Object.keys(this.editableRecord)) {
-            if (this.isArray(this.editableRecord[field])) {
-                this.tabCount++;
-                this.tabFields.push(field);
-            }
+        const tableDef = this.cacheService.getTableDefinition(this.tableName());
+        this.tabFields = (tableDef?.Children ?? []).map((child) => child.PascalName);
+        for (const field of this.tabFields) {
+            if (!this.isArray(this.editableRecord[field])) this.editableRecord[field] = [];
         }
+        this.tabCount = this.tabFields.length;
     }
 
     /** Re-fetch the row from the backend after a server-side mutation (TableAction success, etc.). */

package/src/model/appdata.ts

@@ -54,9 +54,10 @@ export interface TableColumn {
   HasDefault:   boolean;
   DefaultValue: string;
   Validators?:  AbstractControlOptions['validators'];
-  // Optional per-column UI overrides from keel's column_display_attribute.
-  // Absent/0 means "no override" — fall back to the Size/InputType heuristics.
-  Readonly?:     boolean;
+  // Optional per-column UI/CRUD overrides from keel's column_display_attribute.
+  // DisplayMode: 'R' read-only, 'H' hidden, 'D' default-seeded, 'I' insert-only;
+  // absent/'' = editable default. Absent width/rows = fall back to heuristics.
+  DisplayMode?:  string;
   DisplayWidth?: number;
   DisplayRows?:  number;
 }