hot-glue 0.7.7 → 0.7.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 19812e0ed05939741ec5f22550345a5d94a07ad32f3925540bd6bb2ec76e0f6f
-  data.tar.gz: d2fba980427bd8c8961a6fb475b063305956d8a99ac428ffc28317291b961209
+  metadata.gz: 0e4491ce711db972625af0df18b41705f69a87451b34237a1a3ab1c565fc4729
+  data.tar.gz: 13799e4f92016ccf6156abf90a85cd5bda7418b1287ce3d7c112112b853ff92f
 SHA512:
-  metadata.gz: 2a6c67fe502cce091097b4e72f1806a23b9126eea7c1498a8ba5f38aec5a581ddb59fd54543e4ba00257f70bfaa987aaf9ae07c9f80de093cccd0a3584dd9133
-  data.tar.gz: 3143a9b97b4338421c2bc4b910afd2f18cddb96178e8e27e6beaf4189d705d54222aa6335c425b7beb2ea2910557979135d84919c410ad63d3c357d3c5e06885
+  metadata.gz: 32a835125c53bd4aa3d14080ac048c17d556db3b6db7716ce4a21eefba18db38fe33f76403e16f5e8a2f9415d6591430412d0a33029d0634ec65118f8363d345
+  data.tar.gz: 55a21a7aba9909332640275d0b6cb24dcfb0784323eb9ca7e8248db70143ec4a6520f4c1537aa39c25ca36c4ac3f844ec42eab20e3345db3298915569c203897

data/.ruby-version

@@ -1 +1 @@
-3.2.4
+3.3.10

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    hot-glue (0.7.6.2)
+    hot-glue (0.7.7)
       ffaker (~> 2.16)
       rails (> 5.1)
 
@@ -117,19 +117,19 @@ GEM
     loofah (2.19.0)
       crass (~> 1.0.2)
       nokogiri (>= 1.5.9)
-    mail (2.9.0)
+    mail (2.9.1)
       logger
       mini_mime (>= 0.1.1)
       net-imap
       net-pop
       net-smtp
-    marcel (1.1.0)
+    marcel (1.2.1)
     matrix (0.4.2)
     method_source (1.0.0)
     mini_mime (1.1.2)
     mini_portile2 (2.8.4)
     minitest (5.16.3)
-    net-imap (0.6.4)
+    net-imap (0.6.6)
       date
       net-protocol
     net-pop (0.1.2)
@@ -148,7 +148,7 @@ GEM
     puma (5.6.5)
       nio4r (~> 2.0)
     racc (1.6.0)
-    rack (2.2.4)
+    rack (2.2.24)
     rack-test (2.0.2)
       rack (>= 1.3)
     rails (7.0.4)
@@ -235,7 +235,7 @@ GEM
     warden (1.2.9)
       rack (>= 2.0.9)
     websocket (1.2.9)
-    websocket-driver (0.8.0)
+    websocket-driver (0.8.2)
       base64
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.5)
@@ -247,6 +247,7 @@ PLATFORMS
   arm64-darwin-21
   arm64-darwin-22
   arm64-darwin-23
+  arm64-darwin-25
   x86_64-linux
 
 DEPENDENCIES

data/README.md

@@ -2489,6 +2489,14 @@ These automatic pickups for partials are detected at build time. This means that
 
 # VERSION HISTORY
 
+#### 2026-08-22 - v0.7.8
+- fix for time field output when no user is present (in gd mode)
+- in new builds, create and edit will now interact as turbo-stream interactions (except for big edit); this is to remove a race condition in the capy spec; note that your old edit.erb and new.erb will get deleted if you rebuild
+
+
+#### 2026-06-01 - v0.7.7
+- Dynamic blocks (denoted with **) now use fully namespaced paths when rendering, this avoid out of scope issues when an action was used accross controllers
+
 #### 2026-04-08 - v0.7.6.2
 - Syntax fix in code, no functional changes
 

data/app/helpers/hot_glue/controller_helper.rb

@@ -57,7 +57,7 @@ module HotGlue
     end
 
     def formatted_time_display(object, method, current_user)
-      tz = ActiveSupport::TimeZone[current_user.timezone]
+      tz = current_user.try(:timezone) ? ActiveSupport::TimeZone[current_user.timezone] : ActiveSupport::TimeZone['UTC']
 
       t = object.public_send(method)
 
@@ -78,9 +78,10 @@ module HotGlue
     end
 
     def formatted_time_field(object, method, current_user)
-      tz = ActiveSupport::TimeZone[current_user.timezone]
+      tz = current_user.try(:timezone) ? ActiveSupport::TimeZone[current_user.timezone] : ActiveSupport::TimeZone['UTC']
 
       t = object.public_send(method)
+      return nil if t.nil?
 
       # Build UTC datetime from the stored time
       utc_datetime = Time.utc(

data/lib/generators/hot_glue/fields/association_field.rb

@@ -45,7 +45,7 @@ class AssociationField < Field
     if which_partial == :update_show_only && update_show_only.include?(name)
 
     else
-      "      #{name}_selector = find(\"[name='#{singular}[#{name}]']\").click \n" +
+      "      #{name}_selector = find(\"[name='#{singular}[#{name}]']\")\n" +
       "      #{name}_selector.first('option', text: #{assoc}1.name).select_option"
     end
   end

data/lib/generators/hot_glue/fields/date_time_field.rb

@@ -10,7 +10,10 @@ class DateTimeField < Field
 
 
   def spec_setup_and_change_act(which_partial = nil)
-    "      " + "new_#{name} = DateTime.current + 1.year \n" +
+    # beginning_of_minute: an html5 datetime-local input rejects (as invalid,
+    # silently blocking form submit) a value whose seconds don't match the
+    # field's precision, so keep the test value minute-aligned.
+    "      " + "new_#{name} = DateTime.current.beginning_of_minute + 1.year \n" +
     '      ' + "find(\"[name='#{testing_name}[#{ name.to_s }]']\").fill_in(with: new_#{name.to_s})"
 
   end
@@ -24,7 +27,9 @@ class DateTimeField < Field
   end
 
   def spec_setup_let_arg
-    "#{name}: DateTime.current + 1.day"
+    # minute-aligned so the rendered datetime-local edit field is valid when
+    # the spec fills it (see spec_setup_and_change_act)
+    "#{name}: DateTime.current.beginning_of_minute + 1.day"
   end
 
   def spec_list_view_assertion

data/lib/generators/hot_glue/fields/time_field.rb

@@ -5,12 +5,12 @@ class TimeField < Field
   end
 
   def form_field_output
-    "<%= time_field_localized(f, :#{name}, formatted_time_field(#{singular}, :#{name}, current_user), label: \"#{ name.to_s.humanize } \#{ current_user.timezone }\") %>"
+    "<%= time_field_localized(f, :#{name}, formatted_time_field(#{singular}, :#{name}, try(:current_user)), label: \"#{ name.to_s.humanize } \#{ try(:current_user).try(:timezone) }\") %>"
   end
 
   def line_field_output
     "\n    <% unless #{singular}.#{name}.nil? %>
-      <%= formatted_time_display(#{singular}, :#{name}, current_user) %>
+      <%= formatted_time_display(#{singular}, :#{name}, try(:current_user)) %>
     <% else %>
       <span class=''>MISSING</span>
     <% end %>\n"

data/lib/generators/hot_glue/scaffold_generator.rb

@@ -1437,12 +1437,16 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
 
   def datetime_fields_list
-    @columns.each_with_object({}) do |col, hash|
+    # build the hash literal as a string explicitly rather than interpolating a
+    # live Ruby Hash into the template: Hash#inspect renders symbol keys as
+    # `:k=>v` on Ruby <= 3.3 but `k: v` on Ruby 3.4+, so relying on it made the
+    # generated controller's syntax depend on the host app's Ruby version.
+    pairs = @columns.filter_map do |col|
       column = @the_object.columns_hash[col.to_s]
-      if column && [:datetime, :time].include?(column.type)
-        hash[col.to_sym] = column.type
-      end
+      next unless column && [:datetime, :time].include?(column.type)
+      "#{col}: :#{column.type}"
     end
+    "{#{pairs.join(', ')}}"
   end
 
 
@@ -1699,6 +1703,28 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
       end
     end
+
+    remove_stale_html_new_edit_views
+  end
+
+  # When new/edit are served as turbo_stream we no longer generate the classic
+  # html new.erb / edit.erb pages. A host app previously built by an older Hot
+  # Glue would still have those stale files on disk; clean them out on
+  # re-generation so the app doesn't keep serving (or fall back to) them.
+  def remove_stale_html_new_edit_views
+    return if @specs_only
+    return unless turbo_stream_new_and_edit?
+
+    stale_views = []
+    stale_views << 'new' unless @no_create
+    stale_views << 'edit' unless @no_edit
+
+    stale_views.each do |view|
+      dest_filename = cc_filename_with_extensions("#{view}", "#{@markup}")
+      dest_filepath = File.join("#{filepath_prefix}app/views#{namespace_with_dash}",
+                                @controller_build_folder, dest_filename)
+      remove_file dest_filepath if File.exist?(dest_filepath)
+    end
   end
 
   def append_model_callbacks
@@ -1767,15 +1793,25 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     @namespace ? "#{@namespace}/" : ""
   end
 
+  # new & edit are served as turbo_stream (frame replace) rather than as
+  # standalone html pages, EXCEPT in --big-edit mode (edit is a full page)
+  # and --display-edit-after-create mode (create redirects to the html edit
+  # page). In those modes we keep the classic html new.erb / edit.erb views.
+  def turbo_stream_new_and_edit?
+    !@big_edit && !@display_edit_after_create
+  end
+
   def all_views
     res = %w(index  _line _list _show)
 
     unless @no_create
-      res += %w(new _new_form _new_button)
+      res += %w(_new_form _new_button)
+      res << 'new' unless turbo_stream_new_and_edit?
     end
 
     unless @no_edit
-      res += %w{edit _edit}
+      res << '_edit'
+      res << 'edit' unless turbo_stream_new_and_edit?
     end
 
     if !(@no_edit && @no_create)
@@ -1801,6 +1837,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
     unless @no_create
       res << 'create'
+      res << 'new' if turbo_stream_new_and_edit?
     end
 
     unless @no_edit

data/lib/generators/hot_glue/templates/capybara_login.rb

@@ -5,4 +5,11 @@ def login_as(user)
     fill_in 'Password', with: 'password'
   end
   click_button 'Log in'
+
+  # headless Chrome/Selenium occasionally drops this click without submitting
+  # the form at all (confirmed via server log: no POST /users/sign_in received).
+  # Retry once before failing for real.
+  click_button 'Log in' unless page.has_content?('Signed in successfully', wait: 3)
+
+  expect(page).to have_content("Signed in successfully")
 end

data/lib/generators/hot_glue/templates/controller.rb.erb

@@ -105,7 +105,10 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
     authorize @<%= singular %><% elsif @pundit && @pundit_policy_override %>
     skip_authorization
     raise Pundit::NotAuthorizedError if ! <%= @pundit_policy_override %>.new?<% end %>
-    @action = 'new' <% if @pundit %>
+    @action = 'new'<% if turbo_stream_new_and_edit? %>
+    respond_to do |format|
+      format.turbo_stream
+    end<% else %> <% end %><% if @pundit %>
   rescue Pundit::NotAuthorizedError
     flash[:alert] = 'You are not authorized to perform this action.'
     load_all_<%= plural %>
@@ -181,7 +184,9 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
     skip_authorization
     raise Pundit::NotAuthorizedError if ! <%= @pundit_policy_override %>.edit?<% end %>
     @action = 'edit'
-    render :edit<% if @pundit %>
+    <% if turbo_stream_new_and_edit? %>respond_to do |format|
+      format.turbo_stream
+    end<% else %>render :edit<% end %><% if @pundit %>
   rescue Pundit::NotAuthorizedError
     flash[:notice] = "Editing <%= singular + ' ' if @include_object_names %>#{@<%= singular %>.<%= display_class %>} not authorized."
     render :index <% end %>

data/lib/generators/hot_glue/templates/erb/_new_button.erb

@@ -1,5 +1,5 @@
 <\%= turbo_frame_tag "<%= @namespace %>__<%= singular %>-new" do %>
   <div>
-    <\%= link_to "<%= @new_button_label %>",  <%= new_path_name %>, disable_with: "Loading...", class: "new-<%= singular %>-button btn btn-primary pull-right <%= 'btn-sm' if @nested_set.any? %> " %>
+    <\%= link_to "<%= @new_button_label %>",  <%= new_path_name %>, disable_with: "Loading...", class: "new-<%= singular %>-button btn btn-primary pull-right <%= 'btn-sm' if @nested_set.any? %> "<% if turbo_stream_new_and_edit? %>, data: { turbo_stream: true }<% end %> %>
   </div>
 <\% end %>

data/lib/generators/hot_glue/templates/erb/_show.erb

@@ -42,6 +42,6 @@
   <% end %>
 
   <% unless @no_edit %>
-  <\%= link_to "Edit <% if @button_icons == 'font-awesome' %><i class='fa fa-1x fa-list-alt'></i><% end %>".html_safe, <%= edit_path_helper %> + (params[:page] ? "?page=#{params[:page]}" : ""), <% if @big_edit %>'data-turbo' => 'false', <% end %>disable_with: "Loading...", class: "edit-<%= singular %>-button btn btn-primary btn-sm" %>
+  <\%= link_to "Edit <% if @button_icons == 'font-awesome' %><i class='fa fa-1x fa-list-alt'></i><% end %>".html_safe, <%= edit_path_helper %> + (params[:page] ? "?page=#{params[:page]}" : ""), <% if @big_edit %>'data-turbo' => 'false', <% end %>disable_with: "Loading...", class: "edit-<%= singular %>-button btn btn-primary btn-sm"<% if turbo_stream_new_and_edit? %>, data: { turbo_stream: true }<% end %> %>
   <% end %>
 </div>

data/lib/generators/hot_glue/templates/erb/edit.turbo_stream.erb

@@ -1,3 +1,3 @@
 <\%= turbo_stream.replace "<%= @namespace %>__#{dom_id(@<%= singular %>)}" do %>
-  <\%= render 'edit', locals: { <%= singular %>: @<%= singular %><%= @nested_set.collect{|arg| ", #{arg[:singular]}: @#{arg[:singular]} " }.join("") %> } %>
+  <\%= render partial: 'edit', locals: { <%= singular %>: @<%= singular %><%= @nested_set.collect{|arg| ", #{arg[:singular]}: @#{arg[:singular]} " }.join("") %> } %>
 <\% end %>

data/lib/generators/hot_glue/templates/erb/new.turbo_stream.erb

@@ -0,0 +1,3 @@
+<\%= turbo_stream.replace "<%= @namespace %>__<%= singular %>-new" do %>
+  <\%= render partial: "new_form", locals: { <%= singular %>: @<%= singular %><%= @nested_set.collect{|arg| ", #{arg[:singular]}: @#{arg[:singular]}" }.join %> } %>
+<\% end %>

data/lib/generators/hot_glue/templates/system_spec.rb.erb

@@ -69,7 +69,7 @@ describe 'interaction for <%= controller_class_name %>' do
       expect(page).to have_content("Editing #{<%= factory_testing_name %>.<%= @display_class %>.squish || "(no name)"}")
 <%= capybara_make_updates(:update) %>
       click_button "Save"
-      within("turbo-frame#<%= @namespace %>__#{dom_id(<%= factory_testing_name %>)} ") do
+      within("turbo-frame#<%= @namespace %>__#{dom_id(<%= factory_testing_name %>)}") do
 <%= "        " + @columns_map.map{ |col, col_object|
   if @attachments.keys.collect(&:to_sym).include?(col)
   elsif @show_only.include?(col)

data/lib/hotglue/version.rb

@@ -1,5 +1,5 @@
 module HotGlue
   class Version
-    CURRENT = '0.7.7'
+    CURRENT = '0.7.8'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hot-glue
 version: !ruby/object:Gem::Version
-  version: 0.7.7
+  version: 0.7.8
 platform: ruby
 authors:
 - Jason Fleetwood-Boldt
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-06-01 00:00:00.000000000 Z
+date: 2026-08-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -115,6 +115,7 @@ files:
 - lib/generators/hot_glue/templates/erb/edit.turbo_stream.erb
 - lib/generators/hot_glue/templates/erb/index.erb
 - lib/generators/hot_glue/templates/erb/new.erb
+- lib/generators/hot_glue/templates/erb/new.turbo_stream.erb
 - lib/generators/hot_glue/templates/erb/update.turbo_stream.erb
 - lib/generators/hot_glue/templates/javascript/date_range_picker_controller.js
 - lib/generators/hot_glue/templates/javascript/dropzone_controller.js
@@ -164,7 +165,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.19
+rubygems_version: 3.5.22
 signing_key:
 specification_version: 4
 summary: A gem to build Turbo Rails scaffolding.