Update dependency rubocop to '~> 1.87.0'

Gemfile

@@ -5,7 +5,7 @@ gem_sources.each { |gem_source| source gem_source }
 group :syntax do
   gem 'metadata-json-lint'
   gem 'puppet-lint-trailing_comma-check', require: false
-  gem 'rubocop', '~> 1.84.0'
+  gem 'rubocop', '~> 1.87.0'
   gem 'rubocop-performance', '~> 1.26.0'
   gem 'rubocop-rake', '~> 0.7.1'
   gem 'rubocop-rspec', '~> 3.7.0'

build/simp-utils.spec

@@ -43,7 +43,7 @@ end
 
 Summary: SIMP Utils
 Name: simp-utils
-Version: 6.8.1
+Version: 6.8.2
 Release: %{lua: print(package_release)}%{?dist}
 License: Apache License, Version 2.0
 Group: Applications/System
@@ -113,6 +113,9 @@ chmod -R u=rwx,g=rx,o=rx %{buildroot}/usr/local/*bin
 # Post uninstall stuff
 
 %changelog
+* Sun Jun 07 2026 Steven Pritchard <steve@sicura.us> - 6.8.2-1
+- Additional cleanup for rubocop
+
 * Wed Sep 17 2025 Steven Pritchard <steve@sicura.us> - 6.8.1-1
 - Cleanup for rubocop
 

scripts/bin/set_environment

@@ -86,7 +86,7 @@ require 'yaml'
 module Simp; end
 
 # Simp::YamlNodeClassifier class
-class Simp::YamlNodeClassifier
+class Simp::YamlNodeClassifier # rubocop:disable Style/OneClassPerFile
   DEFAULT_CONFIG_FILE = '/etc/puppetlabs/puppet/environments.yaml'.freeze
   DEFAULT_ENVIRONMENT = 'production'.freeze
 

scripts/sbin/gen-ldap-update

@@ -17,13 +17,13 @@ end
 # Try /etc/openldap/ldap.conf first.
 base_dn = nil
 if File.readable?('/etc/openldap/ldap.conf')
-  File.open('/etc/openldap/ldap.conf', 'r').each do |ln|
+  File.foreach('/etc/openldap/ldap.conf') do |ln|
     if ln =~ %r{^\s*BASE\s+(.*)$}
       base_dn = Regexp.last_match(1).strip
     end
   end
 elsif File.readable?('/etc/ldap.conf')
-  File.open('/etc/ldap.conf').each do |ln|
+  File.foreach('/etc/ldap.conf') do |ln|
     if ln =~ %r{^\s*base\s+(.*)$}
       base_dn = Regexp.last_match(1).strip
     end
@@ -55,7 +55,7 @@ update_dn = [
 new_entries = {}
 
 current_dn = nil
-File.open(default_ldif, 'r').each do |ln|
+File.foreach(default_ldif) do |ln|
   if %r{^\s*$}.match?(ln)
     current_dn = nil
   end

share/transition_scripts/openldap_to_389ds/openldap_to_389ds.rb

@@ -156,14 +156,11 @@ def convert_user(attrs)
 end
 
 # Read in the slapcat file
-fh = File.open(input_ldif, 'r')
 begin
-  ldifs = Net::LDAP::Dataset.read_ldif(fh)
+  ldifs = File.open(input_ldif, 'r') { |fh| Net::LDAP::Dataset.read_ldif(fh) }
 rescue StandardError => e
   warn "ERROR: Malformed LDIF input:\n#{e}\n#{e.backtrace.join("\n")}"
   exit 1
-ensure
-  fh.close
 end
 
 basedn ||= ldifs.select { |_k, v| v[:structuralobjectclass].include?('domain') }.keys.first