flightdeck 0.5.2 → 0.5.3

data/app/controllers/flightdeck/jobs/retries_controller.rb

@@ -15,7 +15,6 @@ module Flightdeck
           execution.retry
         end
 
-        def verb = "retry"
         def past_tense = "retried"
         def blocked_reason = "failed"
     end

data/app/controllers/flightdeck/processes_controller.rb

@@ -19,7 +19,7 @@ module Flightdeck
       unless node.prunable?
         toast "#{node.kind} #{node.name} is still sending heartbeats — only unresponsive " \
               "processes can be pruned.", level: :error
-        return respond_with_toast("flightdeck/processes/prune", fallback: processes_path)
+        return respond_with_toast(processes_frame, fallback: processes_path)
       end
 
       claimed = node.claimed_count
@@ -27,7 +27,7 @@ module Flightdeck
       load_registry
 
       toast "Pruned #{node.kind} #{node.name}#{released(claimed)}."
-      respond_with_toast "flightdeck/processes/prune", fallback: processes_path
+      respond_with_toast processes_frame, fallback: processes_path
     end
 
     private
@@ -35,6 +35,11 @@ module Flightdeck
         @registry = ProcessRegistry.new
       end
 
+      def processes_frame
+        { id: "fd-processes", url: processes_path,
+          partial: "flightdeck/processes/fleet", locals: { registry: @registry } }
+      end
+
       def released(claimed)
         return "" if claimed.zero?
 
@@ -43,7 +48,7 @@ module Flightdeck
 
       def missing
         toast "That process is no longer registered.", level: :error
-        respond_with_toast "flightdeck/processes/prune", fallback: processes_path
+        respond_with_toast processes_frame, fallback: processes_path
       end
   end
 end

data/app/controllers/flightdeck/queues_controller.rb

@@ -16,14 +16,14 @@ module Flightdeck
       SolidQueue::Queue.new(@queue_name).pause
       reload_stats
       toast "Paused #{@queue_name}. Workers will stop picking up its jobs."
-      respond_with_toast "flightdeck/queues/update", fallback: queues_path
+      respond_with_toast queues_frame, fallback: queues_path
     end
 
     def resume
       SolidQueue::Queue.new(@queue_name).resume
       reload_stats
       toast "Resumed #{@queue_name}."
-      respond_with_toast "flightdeck/queues/update", fallback: queues_path
+      respond_with_toast queues_frame, fallback: queues_path
     end
 
     private
@@ -36,6 +36,11 @@ module Flightdeck
         Flightdeck::Cache.bypass { load_stats }
       end
 
+      def queues_frame
+        { id: "fd-queues", url: queues_path,
+          partial: "flightdeck/queues/cards", locals: { stats: @stats, sparklines: @sparklines } }
+      end
+
       # Only queues Flightdeck is actually showing can be paused. That keeps an
       # arbitrary parameter from creating pause rows for queues that do not
       # exist.
@@ -45,7 +50,7 @@ module Flightdeck
         return if @stats.find(@queue_name)
 
         toast "Unknown queue #{@queue_name.presence || "(blank)"}.", level: :error
-        respond_with_toast "flightdeck/queues/update", fallback: queues_path
+        respond_with_toast queues_frame, fallback: queues_path
       end
   end
 end

data/app/controllers/flightdeck/recurring_tasks_controller.rb

@@ -26,7 +26,7 @@ module Flightdeck
               "class refused to enqueue.", level: :error
       end
 
-      respond_with_toast "flightdeck/recurring_tasks/run", fallback: recurring_tasks_path
+      respond_with_toast recurring_frame, fallback: recurring_tasks_path
     end
 
     private
@@ -34,13 +34,18 @@ module Flightdeck
         @catalog = RecurringCatalog.new
       end
 
+      def recurring_frame
+        { id: "fd-recurring", url: recurring_tasks_path,
+          partial: "flightdeck/recurring_tasks/table", locals: { catalog: @catalog } }
+      end
+
       def job_reference
         @enqueued_job_id ? " as job ##{@enqueued_job_id}" : ""
       end
 
       def missing
         toast "That recurring task no longer exists.", level: :error
-        respond_with_toast "flightdeck/recurring_tasks/run", fallback: recurring_tasks_path
+        respond_with_toast recurring_frame, fallback: recurring_tasks_path
       end
   end
 end

data/app/helpers/flightdeck/application_helper.rb

@@ -2,6 +2,23 @@
 
 module Flightdeck
   module ApplicationHelper
+    # The polling <turbo-frame> every panel lives in: the refresh Stimulus
+    # controller, its interval, and the URL the frame re-requests. One owner for
+    # the markup, so the index views and the turbo-stream replacements can never
+    # drift apart.
+    #
+    # `target:` defaults to "_top" because most frames are lists whose links
+    # leave the frame; the jobs list opts out (`target: nil`) so its pager can
+    # navigate in place.
+    def fd_refresh_frame(id, url:, interval: fd_poll_ms, target: "_top", **options, &block)
+      tag.turbo_frame(
+        id: id,
+        target: target,
+        data: { controller: "refresh", refresh_interval_value: interval, refresh_url_value: url },
+        **options, &block
+      )
+    end
+
     # Path to a built asset, resolved through the committed manifest. Returns
     # nil when assets have not been built yet (fresh checkout / dev), so the
     # layout can simply omit the tag instead of blowing up.

data/app/helpers/flightdeck/jobs_helper.rb

@@ -54,16 +54,7 @@ module Flightdeck
     end
 
     def fd_duration(seconds)
-      return "—" if seconds.nil?
-
-      seconds = seconds.to_f.abs
-      case seconds
-      when 0...1 then "#{(seconds * 1000).round}ms"
-      when 1...60 then "#{seconds.round(1)}s"
-      when 60...3600 then "#{(seconds / 60).round}m"
-      when 3600...86_400 then "#{(seconds / 3600).round(1)}h"
-      else "#{(seconds / 86_400).round(1)}d"
-      end
+      seconds.nil? ? "—" : Flightdeck::Duration.humanize(seconds)
     end
 
     def fd_ago(time)
@@ -102,10 +93,6 @@ module Flightdeck
       Flightdeck::ArgumentsPreview.format(preview, length: length)
     end
 
-    def fd_jobs_url(overrides = {})
-      jobs_path(list_filters.merge(state: params[:state].presence).compact.merge(overrides))
-    end
-
     # The frame refreshes itself by re-requesting the URL it is showing, so
     # filters, state tab and cursor all survive a poll.
     def fd_current_list_url
@@ -123,12 +110,6 @@ module Flightdeck
       text.to_s
     end
 
-    # URL the refreshed frame should poll after an action, rebuilt from the
-    # filters the action carried rather than from the POST's own path.
-    def fd_refreshed_list_url
-      jobs_path(list_filters.merge(state: @refreshed_state == :all ? nil : @refreshed_state).compact)
-    end
-
     def fd_toast_level_class(level)
       level.to_sym == :error ? "fd-toast error" : "fd-toast"
     end

data/app/models/flightdeck/duration.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Flightdeck
+  # The one way a number of seconds is written anywhere in Flightdeck —
+  # list columns, overview tiles and chart axes all speak through this, so
+  # "how long" can never be spelled three subtly different ways.
+  module Duration
+    module_function
+
+    def humanize(seconds)
+      value = seconds.to_f.abs
+      case value
+      when 0...1 then "#{(value * 1000).round}ms"
+      when 1...60 then "#{trim(value.round(1))}s"
+      when 60...3600 then "#{(value / 60).round}m"
+      when 3600...86_400 then "#{trim((value / 3600).round(1))}h"
+      else "#{trim((value / 86_400).round(1))}d"
+      end
+    end
+
+    def trim(number) = number.to_s.sub(/\.0\z/, "")
+  end
+end

data/app/models/flightdeck/job_detail.rb

@@ -27,7 +27,7 @@ module Flightdeck
 
     def initialize(job:, annotation: nil)
       @job = job
-      @state = resolve_state(annotation)
+      @state = JobRow.state_for(job, annotation)
       @execution = load_execution(annotation)
       @process = load_process
     end
@@ -140,13 +140,6 @@ module Flightdeck
     end
 
     private
-      def resolve_state(annotation)
-        return :finished if job.finished_at.present?
-        return annotation[:state] if annotation
-
-        :unknown
-      end
-
       def load_execution(annotation)
         return nil unless annotation
 

data/app/models/flightdeck/job_row.rb

@@ -21,6 +21,16 @@ module Flightdeck
         name && name.constantize
       end
 
+      # How a job's state is derived, in one place: `finished_at` wins, then
+      # whichever execution table `annotate` found the job in, then :unknown
+      # for the transition window where a job is between tables.
+      def state_for(job, annotation)
+        return :finished if job.finished_at.present?
+        return annotation[:state] if annotation
+
+        :unknown
+      end
+
       # Resolves the state of an already-loaded page of jobs with one
       # `WHERE job_id IN (page_ids)` lookup per executions table. Returns
       # { job_id => { state:, execution_id:, process_id: } }.

data/app/models/flightdeck/jobs_query.rb

@@ -90,7 +90,9 @@ module Flightdeck
     end
 
     private
-      JOB_LIST_COLUMNS = -> { SolidQueue::Job.column_names - [ "arguments" ] }
+      # The columns list queries must never drag along: `arguments` on jobs and
+      # `error` on failed_executions can each run to kilobytes per row.
+      PAYLOAD_COLUMNS = %w[arguments error].freeze
 
       # Only the filters change what a count means — the cursor and page size do
       # not, so they are deliberately absent from the key.
@@ -124,11 +126,10 @@ module Flightdeck
         relation.select(driving_columns).to_a
       end
 
-      # Never `SELECT *`: it would drag `arguments` (jobs) or `error`
-      # (failed_executions) through every list query.
+      # Never `SELECT *`: it would drag a payload column through every list
+      # query.
       def driving_columns
-        names = model.column_names - [ "arguments", "error" ]
-        names.map { |name| model.arel_table[name] }
+        (model.column_names - PAYLOAD_COLUMNS).map { |name| model.arel_table[name] }
       end
 
       def rows_from_executions(executions)
@@ -159,25 +160,16 @@ module Flightdeck
         annotations = state == :finished ? {} : JobRow.annotate(job_ids)
 
         jobs.map do |job|
-          annotation = annotations[job.id]
-
           JobRow.new(
             job: job,
-            state: state_for(job, annotation),
+            state: JobRow.state_for(job, annotations[job.id]),
             args_preview: previews[job.id]
           )
         end
       end
 
-      def state_for(job, annotation)
-        return :finished if job.finished_at.present?
-        return annotation[:state] if annotation
-
-        :unknown
-      end
-
       def jobs_by_id(job_ids)
-        SolidQueue::Job.where(id: job_ids).select(JOB_LIST_COLUMNS.call).index_by(&:id)
+        SolidQueue::Job.where(id: job_ids).select(SolidQueue::Job.column_names - PAYLOAD_COLUMNS).index_by(&:id)
       end
 
       def processes_by_id(process_ids)

data/app/models/flightdeck/metrics/line_chart.rb

@@ -49,14 +49,10 @@ module Flightdeck
       private
         def raw_max = points.filter_map(&:seconds).max.to_f
 
+        # The zero gridline reads "0s" rather than "0ms": it is an axis origin,
+        # not a measurement.
         def format_value(value)
-          case value
-          when 0 then "0s"
-          when 0...1 then "#{(value * 1000).round}ms"
-          when 1...60 then "#{value.round(1).to_s.sub(/\.0\z/, "")}s"
-          when 60...3600 then "#{(value / 60).round}m"
-          else "#{(value / 3600).round(1)}h"
-          end
+          value.zero? ? "0s" : Flightdeck::Duration.humanize(value)
         end
 
         def point_title(point)

data/app/models/flightdeck/overview.rb

@@ -8,6 +8,8 @@ module Flightdeck
     RECENT_FAILURES = 5
     TOP_QUEUES = 6
 
+    # `value` is always display-ready text — a delimited count, a duration, or
+    # "—" — so the view prints it without asking what kind of number it was.
     # trend is :up / :down / :flat and says which way the number moved, while
     # `good` says whether that movement is a good thing — more throughput is
     # good, more failures are not.
@@ -143,7 +145,7 @@ module Flightdeck
 
         Tile.new(
           label: "Processed · 24h",
-          value: processed_24h,
+          value: number(processed_24h),
           detail: delta.nil? ? "no prior window to compare" : "#{delta.abs}% vs prior 24h",
           trend: trend_for(delta),
           good: delta.nil? || delta >= 0
@@ -155,7 +157,7 @@ module Flightdeck
 
         Tile.new(
           label: "Failed · 24h",
-          value: failed_24h,
+          value: number(failed_24h),
           detail: rate.nil? ? "nothing finished yet" : "#{rate}% failure rate",
           trend: failed_24h.positive? ? :up : :flat,
           good: failed_24h.zero?
@@ -163,24 +165,24 @@ module Flightdeck
       end
 
       def ready_tile
-        Tile.new(label: "Ready now", value: ready_now, detail: "waiting to be claimed", trend: :flat, good: true)
+        Tile.new(label: "Ready now", value: number(ready_now), detail: "waiting to be claimed", trend: :flat, good: true)
       end
 
       def scheduled_tile
         detail =
           if next_scheduled_at.nil? then "nothing scheduled"
           elsif next_scheduled_at <= now then "due now"
-          else "next due in #{humanize_duration(next_scheduled_at - now)}"
+          else "next due in #{Duration.humanize(next_scheduled_at - now)}"
           end
 
-        Tile.new(label: "Scheduled", value: scheduled, detail: detail, trend: :flat, good: true)
+        Tile.new(label: "Scheduled", value: number(scheduled), detail: detail, trend: :flat, good: true)
       end
 
       def in_progress_tile
         Tile.new(
           label: "In progress",
-          value: in_progress,
-          unit: slots ? "/ #{ActiveSupport::NumberHelper.number_to_delimited(slots)} slots" : nil,
+          value: number(in_progress),
+          unit: slots ? "/ #{number(slots)} slots" : nil,
           detail: utilization ? "#{utilization}% utilization" : "no worker capacity reported",
           trend: :flat,
           good: true
@@ -190,28 +192,19 @@ module Flightdeck
       def oldest_ready_tile
         Tile.new(
           label: "Oldest ready",
-          value: oldest_ready_age ? humanize_duration(oldest_ready_age) : "—",
+          value: oldest_ready_age ? Duration.humanize(oldest_ready_age) : "—",
           detail: oldest_ready_age ? "longest a job has waited" : "nothing waiting",
           trend: :flat,
           good: true
         )
       end
 
+      def number(value) = ActiveSupport::NumberHelper.number_to_delimited(value)
+
       def trend_for(delta)
         return :flat if delta.nil? || delta.abs < 0.05
 
         delta.positive? ? :up : :down
       end
-
-      def humanize_duration(seconds)
-        seconds = seconds.to_f.abs
-        case seconds
-        when 0...1 then "#{(seconds * 1000).round}ms"
-        when 1...60 then "#{seconds.round(1).to_s.sub(/\.0\z/, "")}s"
-        when 60...3600 then "#{(seconds / 60).round}m"
-        when 3600...86_400 then "#{(seconds / 3600).round(1)}h"
-        else "#{(seconds / 86_400).round(1)}d"
-        end
-      end
   end
 end

data/app/models/flightdeck/process_registry.rb

@@ -63,8 +63,6 @@ module Flightdeck
       end
     end
 
-    def self.all = new
-
     def nodes
       @nodes ||= build_nodes
     end
@@ -82,10 +80,6 @@ module Flightdeck
       end
     end
 
-    def flattened
-      tree.flat_map { |root| [ root, *root.children ] }
-    end
-
     def dead = nodes.select(&:dead?)
     def any_dead? = dead.any?
     def dead_claimed_count = dead.sum(&:claimed_count)

data/app/models/flightdeck/queue_stats.rb

@@ -31,8 +31,6 @@ module Flightdeck
       def idle? = depth.zero? && completed_in_window.zero?
     end
 
-    def self.all = new.rows
-
     def rows
       @rows ||= names.sort.map do |name|
         Row.new(

data/app/models/flightdeck/recurring_catalog.rb

@@ -41,8 +41,6 @@ module Flightdeck
       def ever_run? = last_run_at.present?
     end
 
-    def self.all = new.rows
-
     def rows
       @rows ||= tasks.map do |task|
         run = last_runs[task.key]

data/app/views/flightdeck/jobs/_list.html.erb

@@ -9,10 +9,15 @@
               formaction="<%= bulk_discard_jobs_path(list_filters.merge(state: :failed)) %>"
               data-turbo-confirm="Permanently discard the selected jobs? This cannot be undone.">Discard selected</button>
       <span class="spacer"></span>
-      <button type="submit" name="scope" value="all" class="fd-btn sm"
-              data-turbo-confirm="Retry all failed jobs matching the current filter?">
-        or apply to all <%= fd_count(query.count, capped: query.count_capped?) %> matching
-      </button>
+      <%# Each "all matching" button names its own action: a single neutral
+          "apply to all" button inherited the form's action and silently
+          retried when the user meant to discard. %>
+      <span class="fd-bulkbar-all">or all <%= fd_count(query.count, capped: query.count_capped?) %> matching:</span>
+      <button type="submit" name="scope" value="all" class="fd-btn sm primary"
+              data-turbo-confirm="Retry all failed jobs matching the current filter?">Retry all</button>
+      <button type="submit" name="scope" value="all" class="fd-btn sm danger"
+              formaction="<%= bulk_discard_jobs_path(list_filters.merge(state: :failed)) %>"
+              data-turbo-confirm="Permanently discard all failed jobs matching the current filter? This cannot be undone.">Discard all</button>
     </div>
 
     <div class="fd-card">

data/app/views/flightdeck/jobs/index.html.erb

@@ -5,9 +5,6 @@
 
 <%# The frame re-requests the URL it is showing, so state, filters and cursor
     all survive a poll. %>
-<turbo-frame id="fd-jobs"
-             data-controller="refresh"
-             data-refresh-interval-value="<%= (Flightdeck.config.poll_interval.to_f * 1000).round %>"
-             data-refresh-url-value="<%= fd_current_list_url %>">
+<%= fd_refresh_frame "fd-jobs", url: fd_current_list_url, target: nil do %>
   <%= render "flightdeck/jobs/list", query: @query, rows: @rows, groups: @groups %>
-</turbo-frame>
+<% end %>

data/app/views/flightdeck/overview/_tiles.html.erb

@@ -3,7 +3,7 @@
     <div class="fd-card fd-tile">
       <div class="lbl"><%= tile.label %></div>
       <div class="val">
-        <%= tile.value.is_a?(Numeric) ? number_with_delimiter(tile.value) : tile.value %><%
+        <%= tile.value %><%
           %><% if tile.unit %><small><%= tile.unit %></small><% end %>
       </div>
       <div class="delta <%= fd_delta_class(tile) %>">

data/app/views/flightdeck/overview/index.html.erb

@@ -1,54 +1,29 @@
 <% content_for :page_title, "Overview" %>
 
-<turbo-frame id="fd-tiles"
-             target="_top"
-             data-controller="refresh"
-             data-refresh-interval-value="<%= fd_poll_ms %>"
-             data-refresh-url-value="<%= root_path(range: @window.key) %>">
+<%= fd_refresh_frame "fd-tiles", url: root_path(range: @window.key) do %>
   <%= render "flightdeck/overview/tiles", overview: @overview %>
-</turbo-frame>
+<% end %>
 
 <div class="fd-grid-2">
-  <turbo-frame id="fd-throughput"
-               target="_top"
-               data-controller="refresh"
-               data-refresh-interval-value="<%= fd_chart_poll_ms %>"
-               data-refresh-url-value="<%= root_path(range: @window.key) %>">
+  <%= fd_refresh_frame "fd-throughput", url: root_path(range: @window.key), interval: fd_chart_poll_ms do %>
     <%= render "flightdeck/overview/throughput", series: @series, window: @window %>
-  </turbo-frame>
+  <% end %>
 
-  <turbo-frame id="fd-completion"
-               target="_top"
-               data-controller="refresh"
-               data-refresh-interval-value="<%= fd_chart_poll_ms %>"
-               data-refresh-url-value="<%= root_path(range: @window.key) %>">
+  <%= fd_refresh_frame "fd-completion", url: root_path(range: @window.key), interval: fd_chart_poll_ms do %>
     <%= render "flightdeck/overview/completion", series: @series, window: @window %>
-  </turbo-frame>
+  <% end %>
 </div>
 
 <div class="fd-grid-2b">
-  <turbo-frame id="fd-overview-queues"
-               target="_top"
-               data-controller="refresh"
-               data-refresh-interval-value="<%= fd_poll_ms %>"
-               data-refresh-url-value="<%= root_path(range: @window.key) %>">
+  <%= fd_refresh_frame "fd-overview-queues", url: root_path(range: @window.key) do %>
     <%= render "flightdeck/overview/queues", overview: @overview %>
-  </turbo-frame>
+  <% end %>
 
-  <turbo-frame id="fd-overview-failures"
-               target="_top"
-               data-controller="refresh"
-               data-refresh-interval-value="<%= fd_poll_ms %>"
-               data-refresh-url-value="<%= root_path(range: @window.key) %>">
+  <%= fd_refresh_frame "fd-overview-failures", url: root_path(range: @window.key) do %>
     <%= render "flightdeck/overview/failures", overview: @overview %>
-  </turbo-frame>
+  <% end %>
 </div>
 
-<turbo-frame id="fd-fleet"
-             target="_top"
-             class="fd-block"
-             data-controller="refresh"
-             data-refresh-interval-value="<%= fd_poll_ms %>"
-             data-refresh-url-value="<%= root_path(range: @window.key) %>">
+<%= fd_refresh_frame "fd-fleet", url: root_path(range: @window.key), class: "fd-block" do %>
   <%= render "flightdeck/overview/fleet", registry: @overview.registry %>
-</turbo-frame>
+<% end %>

data/app/views/flightdeck/processes/index.html.erb

@@ -1,9 +1,5 @@
 <% content_for :page_title, "Processes" %>
 
-<turbo-frame id="fd-processes"
-             target="_top"
-             data-controller="refresh"
-             data-refresh-interval-value="<%= (Flightdeck.config.poll_interval.to_f * 1000).round %>"
-             data-refresh-url-value="<%= processes_path %>">
+<%= fd_refresh_frame "fd-processes", url: processes_path do %>
   <%= render "flightdeck/processes/fleet", registry: @registry %>
-</turbo-frame>
+<% end %>

data/app/views/flightdeck/queues/index.html.erb

@@ -1,9 +1,5 @@
 <% content_for :page_title, "Queues" %>
 
-<turbo-frame id="fd-queues"
-             target="_top"
-             data-controller="refresh"
-             data-refresh-interval-value="<%= (Flightdeck.config.poll_interval.to_f * 1000).round %>"
-             data-refresh-url-value="<%= queues_path %>">
+<%= fd_refresh_frame "fd-queues", url: queues_path do %>
   <%= render "flightdeck/queues/cards", stats: @stats, sparklines: @sparklines %>
-</turbo-frame>
+<% end %>

data/app/views/flightdeck/recurring_tasks/index.html.erb

@@ -1,9 +1,5 @@
 <% content_for :page_title, "Recurring tasks" %>
 
-<turbo-frame id="fd-recurring"
-             target="_top"
-             data-controller="refresh"
-             data-refresh-interval-value="<%= (Flightdeck.config.poll_interval.to_f * 1000).round %>"
-             data-refresh-url-value="<%= recurring_tasks_path %>">
+<%= fd_refresh_frame "fd-recurring", url: recurring_tasks_path do %>
   <%= render "flightdeck/recurring_tasks/table", catalog: @catalog %>
-</turbo-frame>
+<% end %>

data/app/views/flightdeck/shared/refresh.turbo_stream.erb

@@ -0,0 +1,14 @@
+<turbo-stream action="append" target="fd-toasts">
+  <template><%= render partial: "flightdeck/shared/toast", formats: [ :html ], locals: { toast: @toast } %></template>
+</turbo-stream>
+
+<%# Re-render the frame the action was launched from, so the same round trip
+    that carries the toast also carries settled rows and counts. A page that
+    does not have the frame (the job detail page, say) simply ignores this. %>
+<turbo-stream action="replace" target="<%= @frame[:id] %>">
+  <template>
+    <%= fd_refresh_frame @frame[:id], url: @frame[:url], target: @frame.fetch(:target, "_top") do %>
+      <%= render partial: @frame[:partial], formats: [ :html ], locals: @frame[:locals] %>
+    <% end %>
+  </template>
+</turbo-stream>

data/lib/flightdeck/assets.rb

@@ -58,12 +58,6 @@ module Flightdeck
         CONTENT_TYPES.fetch(File.extname(file), "application/octet-stream")
       end
 
-      def reload!
-        remove_instance_variable(:@manifest) if defined?(@manifest)
-        remove_instance_variable(:@by_digested_name) if defined?(@by_digested_name)
-        manifest
-      end
-
       private
         def load_manifest
           path = root.join("manifest.json")

data/lib/flightdeck/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Flightdeck
-  VERSION = "0.5.2"
+  VERSION = "0.5.3"
 end