@almadar/std 16.235.0 → 16.237.0

package/behaviors/lolo/ui/core/atoms/std-file-store.lolo

@@ -1,7 +1,7 @@
 app std-file-store "2.0.0"
 "std-file-store — the generic file/page tree primitive. A file-tree + config-driven data table over a hierarchical collection; opens a row in `viewing_single` with config-driven detail actions. Rebindable: pass `fields`/`itemActions`/`detailActions`/`tree` to retarget it at any hierarchical entity (files, wiki pages, note trees). Folds in the former std-page-tree via `reorderable` + a page field/action set."
 type FieldSpec = { name : string!, label : string, variant : "h3" | "h4" | "body" | "caption" | "badge" | "small" | "progress", icon : string, format : "date" | "currency" | "number" | "boolean" | "percent", key : string, header : string }
-type ItemAction = { label : string!, event : event!, variant : "primary" | "secondary" | "ghost" | "danger", icon : string }
+type ItemAction = { label : string!, event : event!, variant : "primary" | "secondary" | "ghost" | "danger", icon : string, roles : [string], when : SExpr }
 type TreeNode = { name : string!, path : string!, type : "file" | "dir"!, children : [TreeNode] }
 orbital StoredFileOrbital {
     type StoredFileLoaded = Event { data : [StoredFile] } "Fired when the collection finishes loading"
@@ -63,7 +63,8 @@ orbital StoredFileOrbital {
             entity: ?data
             fields: @config.fields
             gap: sm
-            itemActions: @config.itemActions
+            itemClickEvent: @config.itemClickEvent
+            itemActions: (array/filter @config.itemActions (fn action (or (= (array/len (object/get @action roles [])) 0) (array/includes (object/get @action roles []) @config.viewerRole))))
             look: @config.tableLook
             reorderEvent: REORDER
             reorderable: @config.reorderable
@@ -154,7 +155,7 @@ orbital StoredFileOrbital {
             gap: sm
             type: stack
           }, { type: divider }, {
-            entity: @config.detailActions
+            entity: (array/filter @config.detailActions (fn action (or (= (array/len (object/get @action roles [])) 0) (array/includes (object/get @action roles []) @config.viewerRole))))
             fields: []
             gap: sm
             renderItem: (fn a {
@@ -385,7 +386,7 @@ orbital StoredFileOrbital {
       fields : [FieldSpec] = [{
         label: Name
         name: name
-        variant: caption
+        variant: h4
       }, {
         label: Folder
         name: folder
@@ -399,6 +400,7 @@ orbital StoredFileOrbital {
         name: sizeBytes
         variant: caption
       }, {
+        format: date
         label: Uploaded
         name: uploadedAt
         variant: caption
@@ -406,6 +408,11 @@ orbital StoredFileOrbital {
         @label "Columns to show"
         @description "Each column maps to one field on the linked entity"
         @tier "presentation"
+      itemClickEvent : event = OPEN_FILE
+        @label "Which event does clicking a row fire?"
+        @description "Row-click channel; empty = rows are not clickable."
+        @synonyms "row click event, item click, open event"
+        @tier "presentation"
       itemActions : [ItemAction] = [{
         event: OPEN_FILE
         icon: arrow-right
@@ -431,6 +438,10 @@ orbital StoredFileOrbital {
         @label "Detail actions"
         @description "Buttons shown on the opened row's detail"
         @tier "presentation"
+      viewerRole : string = ""
+        @label "Viewer's role"
+        @description "The signed-in viewer's role that each action's `roles` is checked against. Bind it explicitly at the call site (e.g. `viewerRole: @user.role`) in an app whose identity carries a role; left empty, only actions with no `roles` show."
+        @tier "policy"
       tree : [TreeNode] = [{
         children: [{
           name: "Active.pdf"

package/behaviors/lolo/ui/core/atoms/std-form-advanced.lolo

@@ -1,7 +1,6 @@
 app std-form-advanced "1.2.0"
-"std-form-advanced — full-typed form-section primitive. Hosts the `editing` state with a configurable `fields` array (each entry mirrors @almadar/ui's `form-section` field schema: `{ name, label, type?, values?, relation?, required? }`). Field-level constraints live in `config.fieldValidation` as `[{ field, format?: 'email'|'url'|'phone', regex?, minLength?, maxLength?, message? }]` — form-section surfaces inline errors when a submit violates a declared constraint. Agents drive shape by overriding `config.fields` + `config.fieldValidation` per-call-site."
+"std-form-advanced — full-typed form-section primitive. Hosts the `editing` state with a configurable `fields` array (each entry mirrors @almadar/ui's `form-section` field schema: `{ name, label, type?, values?, relation?, required? }`). Agents drive shape by overriding `config.fields` per-call-site."
 type FieldSpec = { name : string!, label : string, variant : string, icon : string, format : string, key : string, header : string }
-type ValidationSpec = { field : string!, rule : string, message : string }
 orbital FormEntryOrbital {
     type FormEntryLoaded = Event { data : [FormEntry] } "Fired when FormEntry finishes loading"
 
@@ -251,10 +250,6 @@ orbital FormEntryOrbital {
         @label "Form fields"
         @description "Inputs to render; each entry follows the form-section field schema"
         @tier "presentation"
-      fieldValidation : [ValidationSpec] = []
-        @label "Field validation"
-        @description "Per-field rules (format, regex, length) shown inline on submit"
-        @tier "domain"
       submitLabel : string
         @label "Submit button"
         @description "Label on the submit button; unset keeps the translated Save"

package/behaviors/lolo/ui/core/atoms/std-gantt-schedule.lolo

@@ -2,6 +2,7 @@ app std-gantt-schedule "1.0.0"
 "std-gantt-schedule — GENERIC view-only Gantt/timeline atom: fetches a task collection on INIT and renders it as bars on a day-scale axis via the gantt pattern, with group headers, dependency arrows, and a today marker. Fully rebindable: a host overrides the linked entity and points the field-mapping configs (labelField/startField/endField/durationField/statusField/groupField) at its own columns; clicking a bar emits the config-named onBarClick event with the row id. No drag-edit, no zoom — schedule viewing only."
 type GanttLinkSpec = { from : string!, to : string! }
 orbital GanttScheduleOrbital {
+  type BarColor = "default" | "primary" | "secondary" | "success" | "warning" | "danger" | "error" | "info" | "neutral" | "destructive"
     type TaskLoaded = Event { data : [Task] } "Fired when Task finishes loading; payload.data holds the tasks"
 
     type TaskLoadFailed = Event { error : string, code : string } "Fired when Task fails to load"
@@ -61,6 +62,7 @@ orbital GanttScheduleOrbital {
             showToday: @config.showToday
             startField: @config.startField
             statusField: @config.statusField
+            statusColorMap: @config.statusColorMap
             tasks: ?data
             titleField: @config.labelField
             type: gantt
@@ -167,9 +169,14 @@ orbital GanttScheduleOrbital {
         @tier "presentation"
       statusField : string = "status"
         @label "Status field"
-        @description "Row column holding the status value that drives bar colour."
+        @description "Row column holding each bar's status value."
         @synonyms "status column, state field, stage field"
         @tier "presentation"
+      statusColorMap : Map string BarColor = {}
+        @label "Bar colour per status"
+        @description "Bar colour for each exact status value, e.g. {blocked: danger, complete: success}. Unmapped statuses are primary bars."
+        @synonyms "status colours, bar colours, colour map"
+        @tier "presentation"
       groupField : string = ""
         @label "Group field"
         @description "Row column rows are grouped under header rows by (e.g. team, project). Empty (default) = flat list, no group headers."

package/behaviors/lolo/ui/core/atoms/std-geosearch.lolo

@@ -22,6 +22,7 @@ orbital GeoSearchResultOrbital {
 
     state idle {
       INIT -> idle
+        (set @entity.radiusKm @config.defaultRadiusKm)
         (render-ui main {
           children: [{
             event: @config.event

package/behaviors/lolo/ui/core/atoms/std-graphs.lolo

@@ -1,5 +1,5 @@
 app std-graphs "2.0.0"
-"std-graphs — embeddable chart atom. ITEMS_LOADED groups @payload.data by @config.categoryField (or @config.timeField when timeAxis=true) and aggregates per bucket. Multi-series rendering via @config.series (preformatted ChartSeries shape). Histograms via chartType=histogram pass pre-binned @payload.data through unchanged. Stacked / 100% stacked bars via @config.stack. Post-process sort + limit. Clicking a data point emits UI:@config.drillEvent with { label, value, seriesLabel? }. Drop into a dashboard alongside std-stats / std-browse."
+"std-graphs — embeddable chart atom. ITEMS_LOADED keeps the rows whose @config.filterField equals @config.filterValue (blank = all), groups them by @config.categoryField, or (timeAxis=true) by the @config.period bucket of @config.timeField — rows without a time value have no place on a time axis and are left out — and aggregates per bucket. Multi-series rendering via @config.series (preformatted ChartSeries shape). Histograms via chartType=histogram pass pre-binned @payload.data through unchanged. Stacked / 100% stacked bars via @config.stack. Post-process sort + limit. Clicking a data point emits UI:@config.drillEvent with { label, value, seriesLabel? }. Drop into a dashboard alongside std-stats / std-browse."
 type ObjectSpec = { id : string, label : string, value : string }
 orbital GraphItemOrbital {
     type ChartPoint = { label : string!, value : number! }
@@ -27,17 +27,22 @@ orbital GraphItemOrbital {
           stack: @config.stack
           subtitle: @config.subtitle
           timeAxis: @config.timeAxis
+          period: @config.period
           title: @config.title
           type: @config.viewPattern
         })
       ITEMS_LOADED -> idle
-        (set @entity.chartData (if ["=", @config.chartType, histogram] (array/map ?data (fn p {
+        (let (
+            (rows (if ["=", @config.filterField, ""] ?data (array/filter ?data (fn row (= (object/get @row @config.filterField) @config.filterValue)))))
+          )
+          (set @entity.chartData (if ["=", @config.chartType, histogram] (array/map @rows (fn p {
           label: (object/get @p label)
           value: (object/get @p value)
-        })) (array/map (object/entries (array/groupBy ?data (if @config.timeAxis @config.timeField @config.categoryField))) (fn entry {
+        })) (array/map (object/entries (if @config.timeAxis (array/groupBy (array/map (array/filter @rows (fn row (object/get @row @config.timeField))) (fn row (object/set @row "periodStart" (time/startOf (time/parse (object/get @row @config.timeField)) @config.period)))) "periodStart") (array/groupBy @rows @config.categoryField))) (fn entry {
           label: (object/get @entry "0")
           value: (if ["=", @config.aggregation, sum] (array/sum (object/get @entry "1") @config.valueField) (if ["=", @config.aggregation, avg] (array/avg (object/get @entry "1") @config.valueField) (if ["=", @config.aggregation, min] (array/min (object/get @entry "1") @config.valueField) (if ["=", @config.aggregation, max] (array/max (object/get @entry "1") @config.valueField) (array/len (object/get @entry "1"))))))
-        }))))
+        })))))
+        (set @entity.chartData (if @config.timeAxis (array/sort @entity.chartData label asc) @entity.chartData))
         (set @entity.chartData (if ["!=", @config.sortField, none] (array/sort @entity.chartData @config.sortField @config.sortDir) @entity.chartData))
         (set @entity.chartData (if [">", @config.limit, 0] (array/take @entity.chartData @config.limit) @entity.chartData))
         (render-ui main {
@@ -51,6 +56,7 @@ orbital GraphItemOrbital {
           stack: @config.stack
           subtitle: @config.subtitle
           timeAxis: @config.timeAxis
+          period: @config.period
           title: @config.title
           type: @config.viewPattern
         })
@@ -92,6 +98,16 @@ orbital GraphItemOrbital {
         @description "Group by month, quarter, week, day, or year — the calendar bucket the chart groups data into. Set ALONGSIDE timeAxis=true whenever the user says 'group by <time-unit>' or 'by <time-unit>' or 'rebucket to <time-unit>' or 'instead of <other-time-unit>'."
         @synonyms "regroup / rebucket / change bucket / group differently. user phrases: 'group by month' / 'by month' / 'monthly' / 'monthly buckets' / 'switch to monthly' / 'instead of quarter' -> month; 'group by quarter' / 'by quarter' / 'quarterly' -> quarter; 'group by week' / 'by week' / 'weekly' / 'weekly buckets' -> week; 'group by day' / 'by day' / 'daily' / 'daily buckets' -> day; 'group by year' / 'by year' / 'yearly' / 'annually' -> year. Whenever the user voices 'group by <time-unit>' BOTH period AND timeAxis=true should be set."
         @tier "presentation"
+      filterField : string = ""
+        @label "Only count rows where field"
+        @description "Row field a row must match to be charted (blank = every row), compared with filterValue — e.g. status + paid charts paid invoices only"
+        @synonyms "only include, where, filter by, restrict to. user phrases: 'only paid invoices' -> filterField=status + filterValue=paid; 'just the open deals' -> filterField=status + filterValue=open"
+        @tier "presentation"
+      filterValue : string = ""
+        @label "Equals"
+        @description "The value filterField must equal for a row to be charted"
+        @synonyms "the kept value. user phrases: 'only paid' -> paid; 'just active ones' -> active"
+        @tier "presentation"
       valueField : string = ""
         @label "Value field"
         @description "Numeric field aggregated per bucket (blank = row count)"

package/behaviors/lolo/ui/core/atoms/std-image-upload-multi.lolo

@@ -68,7 +68,7 @@ orbital UploadedImageOrbital {
             maxFiles: @config.maxImages
             maxSize: @config.maxBytesPerImage
             type: upload-drop-zone
-          }, { entity: ?data, type: media-gallery }]
+          }, { captionField: name, entity: ?data, srcField: url, type: media-gallery }]
           direction: vertical
           gap: md
           type: stack
@@ -199,7 +199,7 @@ orbital UploadedImageOrbital {
             maxFiles: @config.maxImages
             maxSize: @config.maxBytesPerImage
             type: upload-drop-zone
-          }, { entity: ?data, type: media-gallery }]
+          }, { captionField: name, entity: ?data, srcField: url, type: media-gallery }]
           direction: vertical
           gap: md
           type: stack

package/behaviors/lolo/ui/core/atoms/std-import.lolo

@@ -107,11 +107,6 @@ orbital ImportOrbital {
         @description "Text shown on the toolbar button that opens the import source popover"
         @synonyms "the text on the import trigger button. user phrases: 'rename import button to X' / 'set the button label to X' -> X (e.g. 'Import Data', 'Add Sources')"
         @tier "presentation"
-      event : string = IMPORT_SOURCE_PICKED
-        @label "Source-picked event"
-        @description "Event emitted when the user picks a source. Each entry in sources[] carries its own literal event field (no per-item templating in .lolo) — overriding this alone does not retarget the click; also set sources[].event to match."
-        @synonyms "bus event name. internal wiring; users rarely target directly"
-        @tier "internal"
       sources : [SourceOption] = [{
         event: IMPORT_SOURCE_PICKED
         id: paste

package/behaviors/lolo/ui/core/atoms/std-list.lolo

@@ -1,6 +1,7 @@
 app std-list "1.0.0"
 "std-list as a Function"
 type FilterSpec = { field : string!, label : string, options : [string] }
+type ItemAction = { label : string!, event : event, variant : string, roles : [string], when : SExpr }
 ;;
 ;; Daisy-chain composition showcase: no `on EVENT { ... }` trait-ref overrides.
 ;; Modal + Confirmation trait refs are configured via `config: { ... }` at the
@@ -11,6 +12,10 @@ type FilterSpec = { field : string!, label : string, options : [string] }
 ;; old version had — form-section's mode set only supports "create" | "edit",
 ;; and a dedicated detail-view pattern is tracked separately (std-detail).
 orbital ListItemOrbital {
+  uses RoleGate from "std/behaviors/std-role-gate"
+
+  uses Button from "std/behaviors/ui-button"
+
   uses Confirmation from "std/behaviors/std-confirmation"
 
   uses Modal from "std/behaviors/std-modal"
@@ -61,13 +66,7 @@ orbital ListItemOrbital {
               direction: horizontal
               gap: md
               type: stack
-            }, {
-              action: CREATE
-              icon: plus
-              label: Create
-              type: button
-              variant: primary
-            }]
+            }, <trait.ListItemCreateGate />]
             direction: horizontal
             gap: md
             justify: between
@@ -118,15 +117,8 @@ orbital ListItemOrbital {
               name: status
               variant: badge
             }]
-            itemActions: [{
-              event: EDIT
-              label: Edit
-              variant: ghost
-            }, {
-              event: DELETE
-              label: Delete
-              variant: danger
-            }]
+            itemClickEvent: @config.itemClickEvent
+            itemActions: (array/filter @config.itemActions (fn action (or (= (array/len (object/get @action roles [])) 0) (array/includes (object/get @action roles []) @config.viewerRole))))
             pageSize: @config.displayPageSize
             type: data-grid
           }]
@@ -208,13 +200,7 @@ orbital ListItemOrbital {
               direction: horizontal
               gap: md
               type: stack
-            }, {
-              action: CREATE
-              icon: plus
-              label: Create
-              type: button
-              variant: primary
-            }]
+            }, <trait.ListItemCreateGate />]
             direction: horizontal
             gap: md
             justify: between
@@ -265,15 +251,8 @@ orbital ListItemOrbital {
               name: status
               variant: badge
             }]
-            itemActions: [{
-              event: EDIT
-              label: Edit
-              variant: ghost
-            }, {
-              event: DELETE
-              label: Delete
-              variant: danger
-            }]
+            itemClickEvent: @config.itemClickEvent
+            itemActions: (array/filter @config.itemActions (fn action (or (= (array/len (object/get @action roles [])) 0) (array/includes (object/get @action roles []) @config.viewerRole))))
             pageSize: @config.displayPageSize
             type: data-grid
           }]
@@ -392,6 +371,48 @@ orbital ListItemOrbital {
         @description "Rows visible per page before the data-grid's own pagination control appears — 20 matches the CRUD-list density sweet spot (Miller's law, 10-25 rows)."
         @synonyms "visible rows, items per page, display limit, show per page"
         @tier "presentation"
+      itemClickEvent : event = EDIT
+        @label "Which event does clicking a row fire?"
+        @description "Row-click channel; empty = rows are not clickable."
+        @synonyms "row click event, item click, open event"
+        @tier "presentation"
+      itemActions : [ItemAction] = [{
+        event: EDIT
+        label: Edit
+        variant: ghost
+      }, {
+        event: DELETE
+        label: Delete
+        variant: danger
+      }]
+        @label "Row actions"
+        @description "Actions offered on each row; give an item `roles` to show it only to those viewer roles."
+        @tier "presentation"
+      viewerRole : string = ""
+        @label "Viewer's role"
+        @description "The signed-in viewer's role that each action's `roles` is checked against. Bind it explicitly at the call site (e.g. `viewerRole: @user.role`) in an app whose identity carries a role; left empty, only actions with no `roles` show."
+        @tier "policy"
+      createRoles : [string] = []
+        @label "Roles that can create"
+        @description "Roles (matched against viewerRole) that see the Create button. Empty = everyone."
+        @tier "policy"
+    }
+  }
+
+  trait ListItemCreateButton = Button.traits.ButtonRender {
+    config {
+      action: CREATE
+      icon: plus
+      label: Create
+      variant: primary
+    }
+  }
+
+  trait ListItemCreateGate = RoleGate.traits.RoleGate {
+    config {
+      roles: @config.createRoles
+      viewerRole: @config.viewerRole
+      content: <trait.ListItemCreateButton />
     }
   }
 

package/behaviors/lolo/ui/core/atoms/std-modal-editor.lolo

@@ -293,7 +293,7 @@ orbital ModalEditorOrbital {
         (set @entity.cmdline "")
         (emit SET_MODE { editorId: ?editorId, mode: "NORMAL", caret: "block" })
 
-      KEY -> COMMAND when (and ["!=", ?key, "Enter"] ["!=", ?key, "Escape"] ["!=", ?key, "Backspace"])
+      KEY -> COMMAND when (and (array/includes (object/get @config.keymap "COMMAND") ?key) ["!=", ?key, "Enter"] ["!=", ?key, "Escape"] ["!=", ?key, "Backspace"])
         (set @entity.cmdline (str/concat @entity.cmdline ?key))
 
       EDITOR_FOCUS -> COMMAND
@@ -312,7 +312,7 @@ orbital ModalEditorOrbital {
         meta : boolean
         repeat : boolean
       }
-        @description "One physical keydown, delivered by the host's keyboard router. The host decides preventDefault from config.keymap; this trait only reads key + editorId."
+        @description "One physical keydown, delivered by the host's keyboard router. The host decides preventDefault from config.keymap; the command line accepts only keys config.keymap captures in COMMAND."
       EDITOR_FOCUS {
         editorId : string
       }

package/behaviors/lolo/ui/core/atoms/std-multi-party-flow.lolo

@@ -259,6 +259,10 @@ orbital MultiPartyFlowOrbital {
           }, {
             entity: @entity.audit
             fields: (title description timestamp status)
+            titleField: "title"
+            descriptionField: "description"
+            dateField: "timestamp"
+            statusField: "status"
             look: @config.timelineLook
             type: timeline
           }, { type: divider }, {
@@ -508,6 +512,10 @@ orbital MultiPartyFlowOrbital {
           }, {
             entity: @entity.audit
             fields: (title description timestamp status)
+            titleField: "title"
+            descriptionField: "description"
+            dateField: "timestamp"
+            statusField: "status"
             look: @config.timelineLook
             type: timeline
           }, { type: divider }, {
@@ -671,6 +679,10 @@ orbital MultiPartyFlowOrbital {
           }, {
             entity: @entity.audit
             fields: (title description timestamp status)
+            titleField: "title"
+            descriptionField: "description"
+            dateField: "timestamp"
+            statusField: "status"
             look: @config.timelineLook
             type: timeline
           }]
@@ -827,6 +839,10 @@ orbital MultiPartyFlowOrbital {
           }, {
             entity: @entity.audit
             fields: (title description timestamp status)
+            titleField: "title"
+            descriptionField: "description"
+            dateField: "timestamp"
+            statusField: "status"
             look: @config.timelineLook
             type: timeline
           }, {
@@ -905,6 +921,10 @@ orbital MultiPartyFlowOrbital {
           }, {
             entity: @entity.audit
             fields: (title description timestamp status)
+            titleField: "title"
+            descriptionField: "description"
+            dateField: "timestamp"
+            statusField: "status"
             look: @config.timelineLook
             type: timeline
           }, {
@@ -989,6 +1009,10 @@ orbital MultiPartyFlowOrbital {
           }, {
             entity: @entity.audit
             fields: (title description timestamp status)
+            titleField: "title"
+            descriptionField: "description"
+            dateField: "timestamp"
+            statusField: "status"
             look: @config.timelineLook
             type: timeline
           }, {

package/behaviors/lolo/ui/core/atoms/std-rating-review.lolo

@@ -312,6 +312,7 @@ orbital RatingReviewOrbital {
                 mode: edit
                 submitEvent: SUBMIT_REVIEW
                 submitLabel: @config.submitLabel
+                cancelLabel: @config.cancelLabel
                 type: form-section
               }]
               direction: vertical
@@ -1114,6 +1115,7 @@ orbital RatingReviewOrbital {
                 mode: edit
                 submitEvent: SUBMIT_REVIEW
                 submitLabel: @config.submitLabel
+                cancelLabel: @config.cancelLabel
                 type: form-section
               }]
               direction: vertical
@@ -1426,10 +1428,11 @@ orbital RatingReviewOrbital {
             }, {
               cancelEvent: CANCEL_REVIEW
               entity: @entity
-              fields: (draftComment)
+              fields: (if @config.allowComment ["draftComment"] [])
               mode: edit
               submitEvent: SUBMIT_REVIEW
               submitLabel: @config.submitLabel
+              cancelLabel: @config.cancelLabel
               type: form-section
             }]
             direction: vertical
@@ -1455,10 +1458,11 @@ orbital RatingReviewOrbital {
             }, {
               cancelEvent: CANCEL_REVIEW
               entity: @entity
-              fields: (draftComment)
+              fields: (if @config.allowComment ["draftComment"] [])
               mode: edit
               submitEvent: SUBMIT_REVIEW
               submitLabel: @config.submitLabel
+              cancelLabel: @config.cancelLabel
               type: form-section
             }]
             direction: vertical
@@ -1504,10 +1508,11 @@ orbital RatingReviewOrbital {
             }, {
               cancelEvent: CANCEL_REVIEW
               entity: @entity
-              fields: (draftComment)
+              fields: (if @config.allowComment ["draftComment"] [])
               mode: edit
               submitEvent: SUBMIT_REVIEW
               submitLabel: @config.submitLabel
+              cancelLabel: @config.cancelLabel
               type: form-section
             }]
             direction: vertical
@@ -1569,10 +1574,11 @@ orbital RatingReviewOrbital {
             }, {
               cancelEvent: CANCEL_REVIEW
               entity: @entity
-              fields: (draftComment)
+              fields: (if @config.allowComment ["draftComment"] [])
               mode: edit
               submitEvent: SUBMIT_REVIEW
               submitLabel: @config.submitLabel
+              cancelLabel: @config.cancelLabel
               type: form-section
             }]
             direction: vertical
@@ -1600,10 +1606,11 @@ orbital RatingReviewOrbital {
             }, {
               cancelEvent: CANCEL_REVIEW
               entity: @entity
-              fields: (draftComment)
+              fields: (if @config.allowComment ["draftComment"] [])
               mode: edit
               submitEvent: SUBMIT_REVIEW
               submitLabel: @config.submitLabel
+              cancelLabel: @config.cancelLabel
               type: form-section
             }]
             direction: vertical
@@ -1696,10 +1703,6 @@ orbital RatingReviewOrbital {
         @label "Cancel button"
         @description "Label on the cancel button"
         @tier "presentation"
-      submitEvent : string = SUBMIT_REVIEW
-        @label "Submit event"
-        @description "Event emitted when the user submits the form"
-        @tier "internal"
       maxStars : number = 5
         @label "Max stars"
         @description "Top of the rating scale (typically 5)"
@@ -1708,10 +1711,6 @@ orbital RatingReviewOrbital {
         @label "Allow comment"
         @description "Show a free-text comment field below the stars"
         @tier "presentation"
-      allowPhotos : boolean = false
-        @label "Allow photos"
-        @description "Let the user attach photos to the review"
-        @tier "presentation"
       cardLook : "elevated" | "flat-bordered" | "borderless-divider" | "ticket" | "invoice" | "chip" | "tile-image-first" = elevated
         @label "Card look"
         @description "Visual style of the wrapper card. Layer 2 cardLook knob — render-ui references @config.cardLook on the compose card."