hephaestus 0.8.21 → 0.8.22.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ff85095b28dd317366e31d4a27335a18a69dd8d56f50d8b0528024ab1967f948
-  data.tar.gz: 38e0995783957661d1475f99c9ba22a9fdc1d512ca32917e3cfd48b912e1fc84
+  metadata.gz: d5b4ca95eb5c3d37230345f2eb437caa07cda0c73b1a71a586731039c119eae3
+  data.tar.gz: ffe0445e8a72761a85e190ab72910e181b1eaec7bc4b4127365a9da312460c86
 SHA512:
-  metadata.gz: 68ae814bf3df3e3e94a6238c55885e9a98a8a45e82925fdeb73cdb2f277028f4cb8e00b8edfde953f8616a1dec7afa1f0ce4ebf6d9358a799d3723031779fd8d
-  data.tar.gz: 3d526424e3c31748a9ffa26634cdf06b758fe16cb04b6649c35657ee2bf955e76965e39b8944e6b9fc2bac2dd783ea836b9fcc17cb6ba9bbf798ce4bc0a411d1
+  metadata.gz: 50f7c10df070ad29e581cd0bfc06abe1a0392c0642452395b4f01e155766ad30a24406ad0078837bb49f1f30646a7858a0d9217322ed870b6af1d2e8b4b48ef9
+  data.tar.gz: 1620a666d30dd1cf8eeb3e7f6e8a2b923c5781f2a7e8341a8283a4d7b041dfe567d8ea62688447d8f6170555ad590f39dc415e2ba7e76f0aad6d66e75d5ff79f

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+# [v0.8.22.1] - 04-07-2025
+## What's Changed
+* Revert bad lint by @gjtorikian in https://github.com/yettoapp/hephaestus/pull/133
+
+
+**Full Changelog**: https://github.com/yettoapp/hephaestus/compare/v0.8.22...v0.8.22.1
+# [v0.8.22] - 02-07-2025
+## What's Changed
+* change plugs to use solstice dependabot updates by @gjtorikian in https://github.com/yettoapp/hephaestus/pull/130
+
+
+**Full Changelog**: https://github.com/yettoapp/hephaestus/compare/v0.8.21...v0.8.22
 # [v0.8.21] - 02-07-2025
 ## What's Changed
 * Dashes instead of dots in hostnames by @balevine in https://github.com/yettoapp/hephaestus/pull/128

data/app/controllers/hephaestus/settings_controller.rb

@@ -21,7 +21,7 @@ module Hephaestus
     end
 
     def set_step
-      @step = params.fetch(:step, 1).to_i || 1
+      @step = params.fetch(:step, 1).to_i
     end
   end
 end

data/app/services/hephaestus/yetto_service.rb

@@ -21,7 +21,7 @@ module Hephaestus
 
     class << self
       def yetto_client
-        @yetto_client ||= Httpsensible::Client.new(user_agent: "#{Rails.application.class.module_parent.name}/#{Hephaestus::Engine::GIT_SHA}")
+        Thread.current[:yetto_client] ||= Httpsensible::Client.new(user_agent: "#{Rails.application.class.module_parent.name}/#{Hephaestus::Engine::GIT_SHA}")
       end
 
       def encoded_jwt

data/config/initializers/filter_job_params.rb

@@ -9,7 +9,7 @@ Rails.application.config.after_initialize do
     class LogSubscriber < ActiveSupport::LogSubscriber
       alias_method :original_format, :format
 
-      private define_method(:format) do |arg|
+      private def format(arg)
         if arg.is_a?(Hash)
           parameter_filter = ActiveSupport::ParameterFilter.new(Rails.application.config.filter_parameters)
           parameter_filter.filter(arg.transform_values { |value| original_format(value) })

data/lib/hephaestus/engine.rb

@@ -75,7 +75,7 @@ module Hephaestus
 
     def op_read(label)
       if within_op_rate_limit?
-        %x(#{include_sudo?}op read "#{label}").chomp.tap do
+        %x(#{sudo_ish}op read "#{label}").chomp.tap do
           Rails.logger.error("Failed to fetch `#{label}` from 1Password") && return unless $CHILD_STATUS.success?
         end
       end
@@ -85,7 +85,7 @@ module Hephaestus
       # skip this unless we're in a Docker container
       return true unless ENV.fetch("DOCKER", "0") == "1"
 
-      results = %x(#{include_sudo?}op service-account ratelimit --format=json).chomp.tap do |_result|
+      results = %x(#{sudo_ish}op service-account ratelimit --format=json).chomp.tap do |_result|
         unless $CHILD_STATUS.success?
           Rails.logger.error("Failed to retrieve rate limits")
           return false
@@ -107,7 +107,7 @@ module Hephaestus
     end
 
     def cli_installed?
-      %x(#{include_sudo?}op --version 2> /dev/null)
+      %x(#{sudo_ish}op --version 2> /dev/null)
       $CHILD_STATUS.success?
     end
 
@@ -115,7 +115,7 @@ module Hephaestus
       ENV.fetch("PRECOMPILING", nil).present?
     end
 
-    def include_sudo?
+    def sudo_ish
       ENV.fetch("DOCKER", false) == "1" ? "sudo -E " : ""
     end
 

data/lib/hephaestus/version.rb

@@ -2,7 +2,7 @@
 # frozen_string_literal: true
 
 module Hephaestus
-  VERSION = "0.8.21"
+  VERSION = "0.8.22.1"
   RAILS_VERSION = ">= 8.0"
   RUBY_VERSION = File
     .read("#{File.dirname(__FILE__)}/../../.ruby-version")

data/templates/app/controllers/settings_controller.rb

@@ -8,7 +8,7 @@ class SettingsController < ApplicationController
 
   sig { void }
   def edit
-    @step = params.fetch(:step, 1).to_i || 1
+    @step = params.fetch(:step, 1).to_i
 
     plug_installation_id = params[:plug_installation_id]
     response = ::Hephaestus::YettoService.get_plug_installation(plug_installation_id)

data/templates/bin/bundle

@@ -13,15 +13,15 @@ require "rubygems"
 m = Module.new do
   module_function
 
-  define_method(:invoked_as_script?) do
+  def invoked_as_script?
     File.expand_path($PROGRAM_NAME) == File.expand_path(__FILE__)
   end
 
-  define_method(:env_var_version) do
+  def env_var_version
     ENV["BUNDLER_VERSION"]
   end
 
-  define_method(:cli_arg_version) do
+  def cli_arg_version
     return unless invoked_as_script? # don't want to hijack other binstubs
     return unless "update".start_with?(ARGV.first || " ") # must be running `bundle update`
 
@@ -39,14 +39,14 @@ m = Module.new do
     bundler_version
   end
 
-  define_method(:gemfile) do
+  def gemfile
     gemfile = ENV["BUNDLE_GEMFILE"]
     return gemfile if gemfile && !gemfile.empty? # rubocop:disable Rails/Present
 
     File.expand_path("../Gemfile", __dir__)
   end
 
-  define_method(:lockfile) do
+  def lockfile
     lockfile =
       case File.basename(gemfile)
       when "gems.rb" then gemfile.sub(/\.rb$/, ".locked")
@@ -55,7 +55,7 @@ m = Module.new do
     File.expand_path(lockfile)
   end
 
-  define_method(:lockfile_version) do
+  def lockfile_version
     return unless File.file?(lockfile)
 
     lockfile_contents = File.read(lockfile)
@@ -64,14 +64,16 @@ m = Module.new do
     Regexp.last_match(1)
   end
 
-  define_method(:bundler_requirement) do
+  # rubocop:disable ThreadSafety/ClassInstanceVariable
+  def bundler_requirement
     @bundler_requirement ||=
       env_var_version ||
       cli_arg_version ||
       bundler_requirement_for(lockfile_version)
   end
+  # rubocop:enable ThreadSafety/ClassInstanceVariable
 
-  define_method(:bundler_requirement_for) do |version|
+  def bundler_requirement_for(version)
     return "#{Gem::Requirement.default}.a" unless version
 
     bundler_gem_version = Gem::Version.new(version)
@@ -79,13 +81,13 @@ m = Module.new do
     bundler_gem_version.approximate_recommendation
   end
 
-  define_method(:load_bundler!) do
+  def load_bundler!
     ENV["BUNDLE_GEMFILE"] ||= gemfile
 
     activate_bundler
   end
 
-  define_method(:activate_bundler) do
+  def activate_bundler
     gem_error = activation_error_handling do
       gem("bundler", bundler_requirement)
     end
@@ -100,7 +102,7 @@ m = Module.new do
     exit(42)
   end
 
-  define_method(:activation_error_handling) do
+  def activation_error_handling
     yield
     nil
   rescue StandardError, LoadError => e

data/templates/hephaestus_github/dependabot.yml

@@ -16,9 +16,8 @@ updates:
   - package-ecosystem: bundler
     directory: "/"
     schedule:
-      interval: monthly
-      day: monday
-      time: "09:00"
+      interval: cron
+      cronjob: "0 0 21 3,6,9,12 *"
       timezone: "Etc/UTC"
     open-pull-requests-limit: 10
     groups:

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hephaestus
 version: !ruby/object:Gem::Version
-  version: 0.8.21
+  version: 0.8.22.1
 platform: ruby
 authors:
 - Garen Torikian
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-07-02 00:00:00.000000000 Z
+date: 2025-07-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bootsnap