glib-web 6.7.0 → 6.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c367b3b3e866dd9b73204e7d0cfe4b84e6148283b683c83068a52140c91ae987
-  data.tar.gz: a375dce696ed47b95d76e1f45091b004bb2c3aeafba70ebe0a31f0db37401d5a
+  metadata.gz: 0f8366db6a89ce988b77fb0392bc24078686aabf55873f2d8fcd38a5f39950bf
+  data.tar.gz: b27ab6afcea23ef3ea93f9b1c8289cb0a905ff563fb768d537cb940afae50a2a
 SHA512:
-  metadata.gz: 1dda92d5d21302e9b4064305b9732bcf39fed88a6e904873ba2d4ccb1cfb7dcfec00e88b3792a91574c26b37212d02fd0ce3a1d4e0a7efdd5556effa366d0ddf
-  data.tar.gz: b5abf3dccb58e18a81da0885fd6c7781da4906f9e329d767359c53a14352b20314506f42e6c2eb5742e483ef48a7cbaeebed43d4793d6de828cde8037080b474
+  metadata.gz: 13c54527c80a6b95961dddec9e419c50c36c6d628a2c47577216e144583b44879caa7308c1e5b924a63d29aad3f935e83de86dc72095c333cf5660b8e8f53958
+  data.tar.gz: 4b4037b3e40cd01c00086219414a846e10a978510ebd3084db7b614b5480b83577c326bea105a60f0871ecf7962ad3392cccb2d826fed58034eba32758fd6668

data/app/channels/glib/channel/appearance_channel.rb

@@ -7,15 +7,15 @@ module Glib
 
       def status(data)
         action = {
-          action: 'labels/set',
+          action: Glib::JsonUi::Name.for('labels/set'),
           targetId: 'label-1',
           text: data['text'],
           onSet: {
-            action: 'timeouts/set',
+            action: Glib::JsonUi::Name.for('timeouts/set'),
             interval: 3000,
             timerId: 'label-1-set',
             onTimeout: {
-              action: 'labels/set',
+              action: Glib::JsonUi::Name.for('labels/set'),
               targetId: 'label-1',
               text: "Room ##{data['room'].upcase}"
             }

data/app/channels/glib/channel/is_typing_channel.rb

@@ -23,7 +23,7 @@ module Glib
         ActionCable.server.broadcast \
           "#{self.class.channel_name}_#{params['conversation_id']}",
           action: {
-            action: 'component/set',
+            action: Glib::JsonUi::Name.for('component/set'),
             text: text,
             status: data['status'],
             user_id: data['user_id']

data/app/channels/glib/channel/online_channel.rb

@@ -23,7 +23,7 @@ module Glib
         ActionCable.server.broadcast \
           "#{self.class.channel_name}_#{params['conversation_id']}",
           action: {
-            action: 'component/set',
+            action: Glib::JsonUi::Name.for('component/set'),
             text: text,
             status: data['status'],
             value: data['reset_value'],

data/app/controllers/concerns/glib/auth/response.rb

@@ -21,9 +21,9 @@ module Glib::Auth
             view: 'button',
             text: 'Re-login',
             onClick: {
-              action: 'dialogs/close',
+              action: Glib::JsonUi::Name.for('dialogs/close'),
               onClose: {
-                action: allow_other_host ? 'windows/openWeb' : 'windows/open',
+                action: Glib::JsonUi::Name.for(allow_other_host ? 'windows/openWeb' : 'windows/open'),
                 url: sign_in_url,
                 targetWindow: allow_other_host ? '_self' : nil
               }

data/app/controllers/concerns/glib/json/libs.rb

@@ -172,7 +172,7 @@ module Glib::Json::Libs
           else
             {
               onResponse: {
-                action: 'windows/open', url: user_default_url
+                action: Glib::JsonUi::Name.for('windows/open'), url: user_default_url
               }
             }
           end
@@ -215,7 +215,7 @@ module Glib::Json::Libs
 
   def __glib_error_dialog(title, message, style_classes, bottom_views = [])
     {
-      action: 'dialogs/show',
+      action: Glib::JsonUi::Name.for('dialogs/show'),
       # Don't show another dialog, just use the current one since it's a new dialog anyway, which means
       # that it wouldn't contain existing user data.
       updateExisting: glib_json_dialog_mode?,
@@ -223,7 +223,7 @@ module Glib::Json::Libs
       body: {
         childViews: [
           {
-            view: 'panels/vertical',
+            view: Glib::JsonUi::Name.for('panels/vertical'),
             styleClasses: ['glib-error-dialog'].concat(style_classes),
             childViews: [
               {
@@ -286,12 +286,12 @@ module Glib::Json::Libs
   def json_ui_redirect_to(url, status: :ok)
     # Use windows_reload instead of windows_open so clicking back will skip the bridging page.
     # on_response = { action: 'windows/open', url: url }
-    on_response = { action: 'windows/reload', url: url }
+    on_response = { action: Glib::JsonUi::Name.for('windows/reload'), url: url }
 
     if glib_json_dialog_mode?
       # Only do this in dialog mode because this seems to add to the rendering time
       # (i.e. longer flicker time), presumably due to the use of things like `nextTick()`.
-      on_response = { action: 'dialogs/close', onClose: on_response }
+      on_response = { action: Glib::JsonUi::Name.for('dialogs/close'), onClose: on_response }
     end
 
     # Use onResponse because it is harder to guarantee that `onLoad` executes in Vuejs because it relies
@@ -389,14 +389,14 @@ module Glib::Json::Libs
                       view: 'button',
                       text: 'Refresh',
                       onClick: {
-                        action: 'auth/saveCsrfToken',
+                        action: Glib::JsonUi::Name.for('auth/saveCsrfToken'),
                         token: form_authenticity_token,
                         onSave: {
-                          action: 'dialogs/close',
+                          action: Glib::JsonUi::Name.for('dialogs/close'),
                           onClose: {
-                            action: 'windows/reload',
+                            action: Glib::JsonUi::Name.for('windows/reload'),
                             onReload: {
-                              action: 'snackbars/alert',
+                              action: Glib::JsonUi::Name.for('snackbars/alert'),
                               message: 'Refreshed. You can try again now.'
                             }
                           }

data/app/controllers/concerns/glib/json/traversal.rb

@@ -82,7 +82,7 @@ module Glib::Json::Traversal
     def traverse_multiple(views, block)
       if views.is_a? Array
         views.each do |view|
-          case view['view']
+          case Glib::JsonUi::Name.normalize(view['view'])
           when 'panels/form'
             @forms << view
             traverse_single view, block

data/app/controllers/glib/home_controller.rb

@@ -54,7 +54,7 @@ module Glib
       chat_params = params[:chat]
 
       payload = {
-        action: 'lists/append',
+        action: Glib::JsonUi::Name.for('lists/append'),
         targetId: 'list-1',
         row: {
           template: 'thumbnail', title: chat_params[:user], subtitle: chat_params[:body]

data/app/helpers/glib/json_ui/abstract_builder.rb

@@ -15,7 +15,7 @@ module Glib
 
         def add_singleton_element(type, name, *args)
           name_components = name.to_s.split('_')
-          json.set! type, "#{name_components.join('/')}" if type
+          json.set! type, Name.join(name_components) if type
           args = [yield] if block_given?
 
           # TODO: consider caching these instances

data/app/helpers/glib/json_ui/page_helper.rb

@@ -264,7 +264,7 @@ module Glib
           panel_options = options.slice!(:url, :method, :local, :onSubmit, :paramNameForFormData, :model, :onChange, :onChangeAndLoad, :autoValidate, :disableDirtyCheck)
 
           json.fullPageForm do
-            json.view 'panels/fullPageForm'
+            json.view Name.for('panels/fullPageForm')
             form.props(options)
           end
 

data/app/helpers/glib/json_ui/view_builder/panels.rb

@@ -294,7 +294,7 @@ class Glib::JsonUi::ViewBuilder
         json.childViews do
           if @method != :get
             json.child! do
-              json.view 'fields/hidden'
+              json.view Glib::JsonUi::Name.for('fields/hidden')
               json.name 'authenticity_token'
               json.value page.context.form_authenticity_token
             end
@@ -606,6 +606,116 @@ class Glib::JsonUi::ViewBuilder
 
     end
 
+    # The generic bulk-edit table: bulkEdit2's skeleton plus everything
+    # bulkEditSg hard-coded for the Surfguard imports, expressed as config so
+    # any consumer can compose the same behavior (bulkEditSg's replacement --
+    # see the component's header comment for the full mapping).
+    class BulkEdit3 < View
+      hash :import, required: [:submitUrl, :paramName]
+
+      # Override
+      def created
+        # Same form-peer rule as bulkEdit2 (see its comment).
+        if page.current_form
+          raise 'panels_bulkEdit3 is not allowed inside a form; place it as a sibling and target it with forms_submit targetId'
+        end
+      end
+
+      # Maximum number of data rows an uploaded file may contain; files with
+      # more rows are rejected whole. No limit when unset.
+      int :maxRows
+
+      array :viewHeaders
+
+      string :paramNameForRowId
+
+      action :onCellChange
+      action :onRowsLoad
+      action :onRowsSelect
+      action :onRowsSucceed
+      # Fires when parked rows drain out of the table on save (they never
+      # POST), so the page can keep its Loaded/Uploaded/Skipped arithmetic
+      # honest.
+      action :onRowsSkip
+      action :onSubmitStart
+      action :onSubmitEnd
+
+      # The file shape. Without it the table takes plain CSVs (header row
+      # first, header names = field ids -- the bulkEdit2 shape):
+      #   detectHeader: { allOf: [...], anyOf: [...], scanRows: N } -- find
+      #     the header row BY CONTENT (report preambles survive);
+      #   headerMap: { csvHeader => field } -- header-keyed, order-independent
+      #     mapping (unknown columns ignored);
+      #   requiredHeaders: [...] -- named rejection when missing;
+      #   rejectIf: [{ when: <JsonLogic over {headers, rowCount}>, error: }]
+      #     -- named wrong-file rejections, first match wins;
+      #   transforms: { field => ['unguard', 'date', 'lowercase', 'trim',
+      #     'coalesce:<other_field>' (fills an empty self), 'prefer:<other_field>'
+      #     (the other field wins when present)] };
+      #   derived: { field => <JsonLogic over the mapped values> };
+      #   uniqueBy: { field:, error: } -- duplicate-value rejection;
+      #   errors: { headerNotFound:, missingHeaders:, tooManyRows:,
+      #     noDataRows:, duplicates:, unreadable:, readFailure: } -- copy
+      #     templates ({rowCount} {maxRows} {headers} {values});
+      #   dropzone: { title:, subtitle: }; accept: '.csv'.
+      hash :parse
+
+      # Ordered row-eligibility rules: the first JsonLogic `when` that matches
+      # parks the row (unticked, disabled checkbox, note under the note
+      # column, warning-icon tooltip). note/noteDetail/checkboxTitle are
+      # strings or JsonLogic. Editing a cell re-evaluates the gates and can
+      # revive a parked row.
+      array :rowGates
+
+      # Newest-row-wins: marks superseded rows (older rows per groupBy) with a
+      # `superseded` var that rowGates can test and order against.
+      #   dedup: { groupBy: <JsonLogic>, newestBy: [date fields in priority] }
+      hash :dedup
+
+      # Referenced rows submit last (one level): rows whose `reference` value
+      # (e.g. a guardian email) matches another selected row's `target` value
+      # (e.g. a member email) go after theirs.
+      #   submitOrder: { reference: <JsonLogic>, target: <JsonLogic> }
+      hash :submitOrder
+
+      # File-level notes rendered after load: [{ when: <JsonLogic over
+      # {headers, rowCount, rows}>, text: string-or-JsonLogic, style:
+      # 'info'|'warning' }].
+      array :fileNotes
+
+      # Labels for the visibility chips (rendered when the file left rows
+      # out). Defaults: 'To import' / 'Left out'.
+      hash :viewChips
+
+      # Rendered when the included view is empty and rows were left out.
+      string :emptyNote
+
+      # The column whose cell carries a parked row's note (default: the first
+      # viewHeader's id).
+      string :noteColumn
+
+      # Constants merged into every rule context (row gates, derived fields,
+      # file notes) at the lowest precedence -- e.g. an award-spelling map.
+      hash :variables
+
+      def viewCells(values)
+        json.set! 'viewCells' do
+          values.each do |value|
+            value.call(page.view_builder)
+          end
+        end
+      end
+
+      def dataRows(value)
+        json.dataRows value
+      end
+
+      def viewRows(value)
+        json.viewRows value
+      end
+
+    end
+
     # deprecated
     class BulkEdit < View
       hash :nextPage

data/app/views/json_ui/garage/test_page/panels_bulkEdit3.json.jbuilder

@@ -0,0 +1,279 @@
+json.title 'Test Page (Panels Bulk Edit 3)'
+
+page = json_ui_page json
+
+render 'json_ui/garage/test_page/header', json: json, page: page
+
+# A report-shaped file: preamble rows before the header, apostrophe-guarded
+# IDs, mixed date formats, a Status column and rows with no email -- enough to
+# exercise header detection, headerMap, transforms, gates and the chips.
+report_header_map = {
+  'Member ID' => :sg_id,
+  'First Name' => :given_name,
+  'Last Name' => :family_name,
+  'Email Address 1' => :email,
+  'Status' => :status,
+  'Member Since' => :member_since
+}
+
+# Ordered eligibility rules: first match parks the row. Interpolated copy via
+# printf, static copy as strings.
+report_row_gates = [
+  {
+    when: { '!=' => [{ 'var' => 'status' }, 'Active'] },
+    note: { 'printf' => ['Status: {0}', { 'var' => 'status' }] },
+    checkboxTitle: { 'printf' => ['Status is {0}: only Active members are imported', { 'var' => 'status' }] }
+  },
+  {
+    when: { '!' => { 'or' => [{ 'isPresent' => [{ 'var' => 'email' }] }] } },
+    note: 'No email in this file',
+    noteDetail: 'This row has no email address, so there is no way to reach the member. Record one in the source system and re-export.'
+  }
+]
+
+plain_headers = [
+  { text: 'Month', id: 'month', importable: true, tooltip: { text: 'Month', placement: 'top' } },
+  { text: 'Electricity usage', id: 'electricity_usage', importable: true },
+  { text: 'Gas usage', id: 'gas_usage', importable: true }
+]
+
+plain_cells = [
+  ->(view) do
+    view.fields_text styleClass: 'compact', width: 'matchParent'
+  end,
+  ->(view) do
+    view.fields_number styleClass: 'compact', width: 'matchParent'
+  end,
+  ->(view) do
+    view.fields_number styleClass: 'compact', width: 'matchParent'
+  end
+]
+
+report_headers = [
+  { text: 'Name', id: 'display_name', readOnly: true },
+  { text: 'Source ID', id: 'sg_id', readOnly: true },
+  { text: 'Email', id: 'email', importable: true },
+  { text: 'Status', id: 'status', readOnly: true },
+  { text: 'Member Since', id: 'member_since', readOnly: true }
+]
+
+report_cells = [
+  ->(view) do
+    view.label
+  end,
+  ->(view) do
+    view.label
+  end,
+  ->(view) do
+    view.fields_email styleClass: 'compact', width: 'matchParent'
+  end,
+  ->(view) do
+    view.label
+  end,
+  ->(view) do
+    view.label
+  end
+]
+
+import_settings = { submitUrl: json_ui_garage_url(path: 'forms/bulk_edit_post2'), paramName: 'user' }
+
+page.body(
+  childViews: ->(body) do
+    body.panels_responsive(
+      padding: glib_json_padding_body,
+      childViews: ->(res) do
+        res.h2 text: 'Overview'
+        res.p text: 'panels_bulkEdit3 is the generic bulk-edit table: bulkEdit2’s skeleton plus config-driven file parsing, row gates, read-only columns, view chips and submission extras.'
+        res.spacer height: 12
+        res.hr width: 'matchParent'
+
+        res.h2 text: 'Basic example (plain CSV)'
+        res.spacer height: 8
+        res.p text: 'Without parse config the table takes plain CSVs: header row first, header names as field ids (the bulkEdit2 shape). maxRows 3 rejects oversized files whole.'
+        res.spacer height: 8
+        res.panels_horizontal(
+          align: 'middle',
+          childViews: ->(horizontal) do
+            horizontal.label text: 'Loaded: '
+            horizontal.label id: 'bulk_edit3_loaded_count', text: '0'
+            horizontal.spacer width: 12
+            horizontal.label text: 'Selected: '
+            horizontal.label id: 'bulk_edit3_selected_count', text: '0'
+          end
+        )
+        res.spacer height: 8
+        res.panels_bulkEdit3(
+          id: 'bulk_edit3_plain',
+          width: 'matchParent',
+          paramNameForRowId: 'row_id',
+          maxRows: 3,
+          import: import_settings,
+          viewHeaders: plain_headers,
+          viewCells: plain_cells,
+          onRowsLoad: ->(action) do
+            action.runMultiple(
+              fieldName: 'data',
+              childActions: ->(multiple) do
+                multiple.logics_set(
+                  targetId: 'bulk_edit3_loaded_count',
+                  paramName: 'variables',
+                  fieldName: 'data',
+                  conditionalData: {
+                    text: { 'count': [{ 'var': 'data.loadedRows' }] }
+                  }
+                )
+              end
+            )
+          end,
+          onRowsSelect: ->(action) do
+            action.logics_set(
+              targetId: 'bulk_edit3_selected_count',
+              paramName: 'variables',
+              fieldName: 'data',
+              conditionalData: {
+                text: { 'count': [{ 'var': 'data.selectedRows' }] }
+              }
+            )
+          end
+        )
+        res.spacer height: 12
+        res.hr width: 'matchParent'
+
+        res.h2 text: 'Report-shaped file (parse config + row gates)'
+        res.spacer height: 8
+        res.markdown text: 'The same component with `parse` config: the header row is found by content (preamble rows survive), CSV headers map to fields, IDs are apostrophe-unwrapped, dates normalize, and the first matching row gate parks the row (unticked, disabled checkbox, note under the first column). Parked rows drain into Skipped on save.'
+        res.spacer height: 8
+        res.panels_horizontal(
+          align: 'middle',
+          childViews: ->(horizontal) do
+            horizontal.label text: 'Loaded: '
+            horizontal.label id: 'bulk_edit3_report_loaded_count', text: '0'
+            horizontal.spacer width: 12
+            horizontal.label text: 'Selected: '
+            horizontal.label id: 'bulk_edit3_report_selected_count', text: '0'
+            horizontal.panels_horizontal id: 'bulk_edit3_skipped_counter', displayed: false, childViews: ->(counter) do
+              counter.label text: ' - '
+              counter.chip id: 'bulk_edit3_skipped_count', text: 'Skipped: 0'
+            end
+          end
+        )
+        res.spacer height: 8
+        res.panels_bulkEdit3(
+          id: 'bulk_edit3_report',
+          width: 'matchParent',
+          paramNameForRowId: 'row_id',
+          maxRows: 100,
+          parse: {
+            headerMap: report_header_map,
+            requiredHeaders: ['Member ID', 'First Name', 'Status'],
+            detectHeader: {
+              allOf: ['Member ID'],
+              anyOf: ['Status', 'Email Address 1'],
+              scanRows: 12
+            },
+            transforms: {
+              sg_id: ['unguard'],
+              member_since: ['date'],
+              email: ['lowercase']
+            },
+            derived: {
+              display_name: { 'joinNonEmpty': [' ', { 'var': 'given_name' }, { 'var': 'family_name' }] }
+            },
+            uniqueBy: {
+              field: 'sg_id',
+              error: 'This file contains duplicate Member IDs ({values}). Each member row must have a unique Member ID.'
+            },
+            errors: {
+              headerNotFound: "This doesn't look like the expected export. Couldn't find the column headers."
+            }
+          },
+          rowGates: report_row_gates,
+          fileNotes: [
+            {
+              when: { '<': [0, { 'count': [{ 'filter': [{ 'var': 'rows' }, { '!=': [{ 'var': 'status' }, 'Active'] }] }] }] },
+              text: { 'printf': ['Heads up: {0} member(s) in this file are not Active.', { 'count': [{ 'filter': [{ 'var': 'rows' }, { '!=': [{ 'var': 'status' }, 'Active'] }] }] }] },
+              style: 'info'
+            }
+          ],
+          import: {
+            submitUrl: json_ui_garage_url(path: 'forms/bulk_edit_post2'),
+            paramName: 'member_import',
+            rowFields: ['sg_id', 'status']
+          },
+          viewHeaders: report_headers,
+          viewCells: report_cells,
+          onRowsLoad: ->(action) do
+            action.runMultiple(
+              fieldName: 'data',
+              childActions: ->(multiple) do
+                multiple.logics_set(
+                  targetId: 'bulk_edit3_report_loaded_count',
+                  paramName: 'variables',
+                  fieldName: 'data',
+                  conditionalData: {
+                    text: { 'count': [{ 'var': 'data.loadedRows' }] }
+                  }
+                )
+              end
+            )
+          end,
+          onRowsSelect: ->(action) do
+            action.runMultiple(
+              fieldName: 'data',
+              childActions: ->(multiple) do
+                multiple.logics_set(
+                  targetId: 'bulk_edit3_report_selected_count',
+                  paramName: 'variables',
+                  fieldName: 'data',
+                  conditionalData: {
+                    text: { 'count': [{ 'var': 'data.selectedRows' }] }
+                  }
+                )
+              end
+            )
+          end,
+          onRowsSkip: ->(action) do
+            action.runMultiple(
+              fieldName: 'data',
+              childActions: ->(multiple) do
+                multiple.components_set targetId: 'bulk_edit3_skipped_counter', data: { displayed: true }
+                multiple.logics_set(
+                  targetId: 'bulk_edit3_skipped_count',
+                  paramName: 'variables',
+                  fieldName: 'data',
+                  conditionalData: {
+                    text: {
+                      'printf': ['Skipped: {0}', { 'count': [{ 'var': 'data.skippedRows' }] }]
+                    }
+                  }
+                )
+              end
+            )
+          end
+        )
+        res.spacer height: 12
+        res.hr width: 'matchParent'
+        res.h2 text: 'Actions/events'
+        res.spacer height: 8
+        res.panels_flow(
+          innerPadding: { bottom: 0 },
+          childViews: ->(flow) do
+            flow.button(
+              text: 'Submit (forms_submit)',
+              onClick: ->(action) do
+                action.forms_submit targetId: 'bulk_edit3_plain'
+              end
+            )
+            flow.spacer width: 8
+            flow.button(
+              text: 'Start over (invoke)',
+              onClick: ->(action) do
+                action.components_invoke targetId: 'bulk_edit3_report', name: 'startOver'
+              end
+            )
+          end
+        )
+      end
+    )
+  end
+)

data/lib/glib/json_crawler/action_crawlers/action_http.rb

@@ -4,7 +4,7 @@ module Glib
       def initialize(method, http, args, controller)
         @http = http
         json = @http.send(method, args['url'], controller, args.fetch('formData', {}))
-        @http.router.http_actions.add([args['action'], args['url'], args.fetch('formData', {})])
+        @http.router.http_actions.add([Glib::JsonUi::Name.normalize(args['action']), args['url'], args.fetch('formData', {})])
         perform(json['onResponse'])
       end
     end

data/lib/glib/json_crawler/action_crawlers/forms_submit.rb

@@ -15,8 +15,8 @@ module Glib
 
         raise "Submit action needs to be inside a form: #{http.router.last_log}" unless form
 
-        case form['view']
-        when 'panels/bulkEdit2'
+        case Glib::JsonUi::Name.normalize(form['view'])
+        when 'panels/bulkEdit2', 'panels/bulkEdit3'
           method = 'post'
           action = "forms/#{method}"
           url = form['import']['submitUrl']
@@ -43,7 +43,7 @@ module Glib
         child_views = view['childViews'] || []
 
         # If this form is page level, we need to check the child views of the entire page.
-        if view['view'] == 'panels/fullPageForm'
+        if Glib::JsonUi::Name.eq?(view['view'], 'panels/fullPageForm')
           child_views += @http.router.page_spec['header']&.[]('childViews') || []
           child_views += @http.router.page_spec['body']&.[]('childViews') || []
           child_views += @http.router.page_spec['footer']&.[]('childViews') || []
@@ -52,7 +52,7 @@ module Glib
         @http.router.crawl_multiple child_views, ->(child) do
           next if child['template'].present?
 
-          name = child['view'] || ''
+          name = Glib::JsonUi::Name.normalize(child['view']) || ''
           if name.start_with?('fields/')
             fields << child
 

data/lib/glib/json_crawler/router.rb

@@ -66,7 +66,7 @@ module Glib
 
       def step(http, args)
         # TODO: Refactor
-        case args['view']
+        case Glib::JsonUi::Name.normalize(args['view'])
         when 'fields/submit'
           @depth += 1
           # forms = @visitor.forms
@@ -98,7 +98,10 @@ module Glib
       end
 
       def process_action(http, spec)
-        action = spec&.fetch('action')
+        # Normalize to the canonical slash form so the crawler works against apps
+        # configured with either separator, and so recorded/logged action names
+        # stay stable (permission snapshots compare them).
+        action = Glib::JsonUi::Name.normalize(spec&.fetch('action'))
         params = spec
 
         assert_target_ids_exist(params)
@@ -304,7 +307,7 @@ module Glib
           http_actions.each do |http_action|
             next if http_action[1].blank?
             distance = ::DamerauLevenshtein.distance(http_action[1], params['url'])
-            result = (distance <= 1) && (http_action[0] == params['action'])
+            result = (distance <= 1) && (http_action[0] == Glib::JsonUi::Name.normalize(params['action']))
 
             break if result
           end

data/lib/glib/json_ui/config.rb

@@ -0,0 +1,40 @@
+module Glib
+  module JsonUi
+    # Global JSON UI settings.
+    #
+    # `name_separator` controls the character that joins the namespace and the
+    # component/action name in the generated JSON, e.g. `panels/responsive`
+    # (default) vs `panels_responsive`.
+    #
+    # The default stays `/` for backwards compatibility; `_` becomes the default
+    # in the next major version. Apps opt in early via an initializer:
+    #
+    #   # config/initializers/glib.rb
+    #   Glib::JsonUi::Config.name_separator = '_'
+    #
+    # Frontends built on glib-web-npm accept either form, so flipping this is a
+    # server-side-only change.
+    class Config
+      SEPARATORS = ['/', '_'].freeze
+
+      # The shipped default. Stays '/' until the next major version, when it
+      # becomes '_'. Apps opt in early via `Config.name_separator = '_'`.
+      DEFAULT_NAME_SEPARATOR = '/'
+
+      @@name_separator = DEFAULT_NAME_SEPARATOR
+
+      def self.name_separator
+        @@name_separator
+      end
+
+      def self.name_separator=(value)
+        value = value.to_s
+        unless SEPARATORS.include?(value)
+          raise ArgumentError, "Invalid name separator: #{value.inspect}. Allowed: #{SEPARATORS.map(&:inspect).join(', ')}"
+        end
+
+        @@name_separator = value
+      end
+    end
+  end
+end

data/lib/glib/json_ui/name.rb

@@ -0,0 +1,53 @@
+module Glib
+  module JsonUi
+    # Helpers for the namespaced names that appear in the generated JSON as
+    # `view`, `action` and `template` values.
+    #
+    # Names are always written in canonical form in this codebase (`panels/form`)
+    # regardless of what `Config.name_separator` is set to:
+    #
+    #   - Producing: run the canonical literal through `Name.for` (or build it
+    #     from parts with `Name.join`) so it is emitted using the configured
+    #     separator.
+    #   - Consuming: compare with `Name.eq?` / `Name.start_with?`, which accept
+    #     either separator. Reads are deliberately separator-agnostic so tools
+    #     like the crawler work against any app, and against fixtures recorded
+    #     before the switch.
+    module Name
+      class << self
+        # 'windows/reload' -> 'windows/reload' or 'windows_reload'
+        def for(canonical)
+          return canonical unless canonical.is_a?(String)
+
+          canonical.tr('/', Config.name_separator)
+        end
+
+        # ['windows', 'reload'] -> 'windows/reload' or 'windows_reload'
+        def join(components)
+          components.join(Config.name_separator)
+        end
+
+        # Rewrites a name coming from JSON into canonical (slash) form. Only the
+        # first separator is rewritten: the second part may itself be camelCase
+        # but never contains a separator (e.g. `storageItems_set`).
+        def normalize(value)
+          return value unless value.is_a?(String)
+
+          value.sub('_', '/')
+        end
+
+        # Either side may use either separator.
+        def eq?(value, canonical)
+          normalize(value) == normalize(canonical)
+        end
+
+        def start_with?(value, *canonicals)
+          normalized = normalize(value)
+          return false unless normalized.is_a?(String)
+
+          canonicals.any? { |canonical| normalized.start_with?(normalize(canonical)) }
+        end
+      end
+    end
+  end
+end

data/lib/glib/test_helpers.rb

@@ -229,12 +229,14 @@ module Glib
       end
 
       def __view_is_form?(view)
-        view.is_a?(Hash) && (
-          view['view']&.start_with?('panels/form') ||
-          view['view']&.start_with?('panels/fullPageForm') ||
-          view['view'] == 'panels/bulkEdit2' ||
-          view['view'] == 'panels/splitForm'
-        )
+        return false unless view.is_a?(Hash)
+
+        name = Glib::JsonUi::Name.normalize(view['view'])
+        name&.start_with?('panels/form') ||
+          name&.start_with?('panels/fullPageForm') ||
+          name == 'panels/bulkEdit2' ||
+          name == 'panels/bulkEdit3' ||
+          name == 'panels/splitForm'
       end
 
       def __build_form_params(form, data)
@@ -247,7 +249,7 @@ module Glib
         data_prefixes = data.keys.map { |k| "#{k}[" }
 
         __extract_form_fields(form).each do |field|
-          next unless field[:view] == 'fields/hidden'
+          next unless Glib::JsonUi::Name.eq?(field[:view], 'fields/hidden')
           next if data_prefixes.any? { |prefix| field[:name]&.start_with?(prefix) }
 
           params[field[:name]] = field[:value]
@@ -268,7 +270,7 @@ module Glib
       def __extract_form_fields(form)
         fields = []
         __traverse_form_views(form['childViews'] || []) do |view|
-          if view.is_a?(Hash) && view['view']&.start_with?('fields/')
+          if view.is_a?(Hash) && Glib::JsonUi::Name.start_with?(view['view'], 'fields/')
             fields << { name: view['name'], value: view['value'], view: view['view'] }
           end
         end

data/lib/glib-web.rb

@@ -5,6 +5,11 @@ if defined?(::Rails)
   require 'glib/snapshot'
 end
 require 'glib/value'
+# Loaded eagerly (not autoloaded from app/) so an app initializer can set
+# `Glib::JsonUi::Config.name_separator` without tripping Rails' ban on
+# autoloading during initialization -- and so the setting survives a reload.
+require 'glib/json_ui/config'
+require 'glib/json_ui/name'
 require 'glib/json_crawler'
 
 require 'glib/dynamic_text'

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: glib-web
 version: !ruby/object:Gem::Version
-  version: 6.7.0
+  version: 6.8.0
 platform: ruby
 authors:
 - ''
+autorequire:
 bindir: bin
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2026-08-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activestorage
@@ -177,6 +178,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+description:
 email: ''
 executables: []
 extensions: []
@@ -476,6 +478,7 @@ files:
 - app/views/json_ui/garage/test_page/pagination.json.jbuilder
 - app/views/json_ui/garage/test_page/panels.json.jbuilder
 - app/views/json_ui/garage/test_page/panels_bulkEdit2.json.jbuilder
+- app/views/json_ui/garage/test_page/panels_bulkEdit3.json.jbuilder
 - app/views/json_ui/garage/test_page/popovers.json.jbuilder
 - app/views/json_ui/garage/test_page/progressCircle.json.jbuilder
 - app/views/json_ui/garage/test_page/responsive.json.jbuilder
@@ -558,6 +561,8 @@ files:
 - lib/glib/json_crawler/coverage.rb
 - lib/glib/json_crawler/http.rb
 - lib/glib/json_crawler/router.rb
+- lib/glib/json_ui/config.rb
+- lib/glib/json_ui/name.rb
 - lib/glib/last_resort_unit_test.rb
 - lib/glib/mailer_tester.rb
 - lib/glib/non_http_integration_test.rb
@@ -575,8 +580,10 @@ files:
 - lib/glib/time_returning_mailer.rb
 - lib/glib/value.rb
 - lib/tasks/db.rake
+homepage:
 licenses: []
 metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -591,7 +598,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 3.4.6
+signing_key:
 specification_version: 4
 summary: ''
 test_files: []