simp-rake-helpers 6.2.0 → 7.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 61c7d3acd2f50f1fdcb52ffd0aee26f0274416020e98b4eb8f0cfed3bfeb9abc
-  data.tar.gz: cecd92355967a1344509c4b280967d7039d2bacd1eb11ce61a7975fc9a3cc18a
+  metadata.gz: 17a61d8752d73045b67c31db5865a27333eb179929cbe3547077293196212f9a
+  data.tar.gz: 3a207e85eb8c285d67cf1f013db9da5f9ce4bef508eea997d796cd3b08632e99
 SHA512:
-  metadata.gz: 857533e0ae2bdb2b2f62fb43492f67d1f332ec343c0f022e9fb9ad65c4f0931ad5c67bf5142f0d34720cfbc703fac41d011c7b9afec96ed92f2781976dc6a44c
-  data.tar.gz: b4f8dcabef643a3ba93b26cf2e1e71538597fe766e8aee46102c25c5b412726f1fdbedb96d86a18085cfdc664ceb32a814ba720396231f7fc28cf4cada2e9fee
+  metadata.gz: cd50a6943242009704ebb20410c4ed70b14af882268b648d6dd5ffbaf206c2394e2244b93bee220e6b40d96baeb86048dd8b8b0cd026afdc1b6c369d0a6f0f89
+  data.tar.gz: 891d1280e437f99c3a8fdba93b20d7931bfb6e3eaa86887a1eb9e5567f241505436a569eb057f805d979d188b84d92d2a312256fd8f43d5f01df4e2900d51ff1

data/CHANGELOG.md

@@ -1,3 +1,27 @@
+### 7.0.0 / 2026-10-05
+- Changed
+  - **BREAKING:** `pkg:compare_latest_tag` decides whether a Puppet module
+    needs a new version from a list of the paths Puppet and Bolt load plus
+    `README.md` and `REFERENCE.md`, instead of a list of paths to ignore.
+    Changes to those two files now require a version bump; other Markdown
+    files and unlisted paths (such as `examples/` and tooling config) no
+    longer do
+  - **BREAKING:** Removed the `compare_latest_tag` and
+    `changelog_annotation` Rake tasks for Puppet modules, deprecated in
+    5.12.0; use `pkg:compare_latest_tag` and `pkg:create_tag_changelog`
+
+### 6.3.0 / 2026-10-03
+- Changed
+  - Allow voxpupuli-test 15, which moves to rubocop 1.91, rubocop-rspec 3.10
+    and rspec-puppet 6
+  - The OpenVox 9 spec row tests the 9.0.0 release (`~> 9.0`, openfact
+    `~> 6.0`) and is blocking
+- Fixed
+  - Require `json` < 3. openvox's `Puppet::Util::Json` is incompatible with
+    json 3.0, which breaks every rspec-puppet catalog compilation; rubocop
+    1.85 used to keep json below 3, but the rubocop 1.91 that voxpupuli-test
+    15 brings does not
+
 ### 6.2.0 / 2026-09-23
 - Added
   - Support for OpenVox 9

data/lib/simp/rake/build/pkg.rb

@@ -32,8 +32,8 @@ class Simp::Rake::Build::Pkg < Rake::TaskLib
     # nil = default not set; use local preference
     @fetch_published_rpm_default = if ENV['SIMP_PKG_fetch_published_rpm']
                                      (
-                                           (ENV.fetch('SIMP_PKG_fetch_published_rpm', nil) =~ %r{\A(yes|true)\Z}i) ? true : false
-                                         )
+                                       (ENV.fetch('SIMP_PKG_fetch_published_rpm', nil) =~ %r{\A(yes|true)\Z}i) ? true : false
+                                     )
                                    else
                                      nil
                                    end

data/lib/simp/rake/helpers/version.rb

@@ -4,5 +4,5 @@ module Simp; end
 module Simp::Rake; end
 
 class Simp::Rake::Helpers
-  VERSION = '6.2.0'
+  VERSION = '7.0.0'
 end

data/lib/simp/rake/pkg.rb

@@ -318,9 +318,9 @@ class Simp::Rake::Pkg < Rake::TaskLib
             end
 
             if latest_rpm && (
-                Gem::Version.new(latest_rpm.full_version) >=
-                Gem::Version.new(@spec_info.full_version)
-              )
+              Gem::Version.new(latest_rpm.full_version) >=
+              Gem::Version.new(@spec_info.full_version)
+            )
               latest_rpm.rpm_name
             else
               "#{f}-#{@spec_info.full_version}-#{@spec_info.arch}.rpm"
@@ -550,15 +550,21 @@ class Simp::Rake::Pkg < Rake::TaskLib
               CHANGELOG and metadata.json files.
           (3) The latest version is < latest tag.
 
-          Changes to the following files/directories are not considered
-          significant:
+          For a Puppet module, only changes to the paths Puppet or Bolt
+          load, plus the docs users read, are significant:
+          - CHANGELOG, LICENSE, README.md, REFERENCE.md, metadata.json,
+            hiera.yaml and bolt_plugin.json
+          - SIMP, build, data, facts.d, files, functions, lib, locales,
+            manifests, plans, tasks, templates and types directories
+
+          For any other component, changes to the following
+          files/directories are not considered significant:
           - Any hidden file/directory (entry that begins with a '.')
-          - Gemfile
-          - Gemfile.lock
+          - Gemfile*
           - Rakefile
-          - rakelib directory
-          - spec directory
-          - doc directory
+          - rakelib, spec and doc directories
+          - Markdown (*.md) files
+          - renovate.json
       EOM
       task :compare_latest_tag, [:tags_source, :verbose] do |_t, args|
         tags_source = args[:tags_source].nil? ? 'origin' : args[:tags_source]

data/lib/simp/rake/pupmod/helpers.rb

@@ -34,9 +34,6 @@ end
 
 # Rake tasks for SIMP Puppet modules
 class Simp::Rake::Pupmod::Helpers < Rake::TaskLib
-  # See https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Changelogs
-  CHANGELOG_ENTRY_REGEX = %r{^\*\s+((?:Mon|Tue|Wed|Thu|Fri|Sat|Sun) (?:Jan|Feb|Mar|Apr|May|Jun|Jul|Aug|Sep|Oct|Nov|Dec) \d{1,2} \d{4})\s+(.+<.+>)(?:\s+|\s*-\s*)?(\d+\.\d+\.\d+)}.freeze
-
   def initialize(base_dir = Dir.pwd)
     @base_dir = base_dir
     @temp_fixtures_path = File.join(base_dir, 'spec', 'fixtures', 'simp_rspec')
@@ -111,49 +108,6 @@ class Simp::Rake::Pupmod::Helpers < Rake::TaskLib
       @metadata ||= JSON.parse(File.read(_file))
     end
 
-    # Generate an appropriate annotated tag entry from the modules' CHANGELOG
-    #
-    # @note this currently does not support the valid RPM `%changelog` format
-    #       that places the version number on the next line:
-    #
-    #       * Fri Mar 02 2012 Maintenance
-    #       4.0.0-2
-    #       - Improved test stubs.
-    #
-    def changelog_annotation(quiet = false, file = nil)
-      result         = ''
-      changelog_file = file || File.join(@base_dir, 'CHANGELOG')
-      module_version = metadata['version']
-      changelogs = {}
-
-      _entry = {} # current log entry's metadata (empty unless valid entry)
-      File.read(changelog_file).each_line do |line|
-        if line =~ %r{^\*}
-          if CHANGELOG_ENTRY_REGEX.match(line).nil?
-            warn %(WARNING: invalid changelogs entry: "#{line}") unless quiet
-            _entry = {}
-          else
-            _entry = {
-              :date => ::Regexp.last_match(1),
-              :user => ::Regexp.last_match(2),
-              :release => ::Regexp.last_match(3)
-            }
-            changelogs[_entry[:release]] ||= []
-            changelogs[_entry[:release]] << line
-            next
-          end
-        end
-
-        # Don't add anything to the annotation unless reach the next valid entry
-        changelogs[_entry[:release]] << "  #{line}" if _entry.fetch(:release, false)
-      end
-
-      raise "Did not find any changelogs entries for version #{module_version}" if changelogs[module_version].nil?
-
-      result += "\nRelease of #{module_version}\n\n"
-      result + changelogs[module_version].join
-    end
-
     def custom_fixtures_hook(opts = {
       :short_name => nil,
       :puppetfile => nil,
@@ -264,137 +218,6 @@ class Simp::Rake::Pupmod::Helpers < Rake::TaskLib
       custom_fixtures_path
     end
 
-    desc <<-EOM
-      Generate an appropriate annotated tag entry from a CHANGELOG.
-
-      ARGS:
-        * :quiet => Set to 'true' if you want to suppress warning messages
-
-      NOTES:
-        * The entries are extracted from a match with the version from the
-          module's metadata.json
-        * If no match is found, the task will fail
-        * Changelog entries must follow the format:
-          * Wed Jul 05 2017 UserName <username@simp.com> - 1.2.3-4
-            - The entry must start with *. Any line beginning with * will be
-              interpreted as an entry.
-            - The dates must be RPM compatible, in chronological order
-            - The user email must be contained in < >
-            - The entry must be terminated by the release
-        * Any entry that does not follow the prescribed format will not be
-          annotated properly
-    EOM
-    # TODO: Hook in a query of the auto-generated specfile:
-    #   `rpm -q --specfile dist/tmp/*.spec --changelog`
-    # That will give Travis a way of warning us if the changelog
-    # will prevent the rpm from building.
-    task :changelog_annotation, [:quiet] do |_t, args|
-      warn('DEPRECATED: use pkg:create_tag_changelog')
-      quiet = true if args[:quiet].to_s == 'true'
-      puts changelog_annotation(quiet)
-    end
-
-    desc <<-EOM
-    Compare to latest tag.
-      ARGS:
-        * :tags_source => Set to the remote from which the tags for this
-                      project can be fetched, e.g. 'upstream' for a
-                      forked project. Defaults to 'origin'.
-        * :ignore_owner => Execute comparison even if the project owner
-                      is not 'simp'.
-        * :verbose => Set to 'true' if you want to see detailed messages
-
-      NOTES:
-      Compares mission-impacting (significant) files with the latest
-      tag and identifies the relevant files that have changed.
-
-      Does nothing if the project owner, as specified in the
-      metadata.json file, is not 'simp'.
-
-      When mission-impacting files have changed, fails if
-      (1) Latest version cannot be extracted from the top-most
-          CHANGELOG entry.
-      (2) The latest version in the CHANGELOG (minus the release
-          qualifier) does not match the version in the metadata.json
-          file.
-      (3) A version bump is required but not recorded in both the
-          CHANGELOG and metadata.json files.
-      (4) The latest version is < latest tag.
-
-      Changes to the following files/directories are not considered
-      significant:
-      - Any hidden file/directory (entry that begins with a '.')
-      - Gemfile
-      - Gemfile.lock
-      - Rakefile
-      - rakelib directory
-      - spec directory
-      - doc directory
-    EOM
-    task :compare_latest_tag, [:tags_source, :ignore_owner, :verbose] do |_t, args|
-      warn('DEPRECATED: use pkg:compare_latest_tag')
-      require 'json'
-
-      tags_source = args[:tags_source].nil? ? 'origin' : args[:tags_source]
-      ignore_owner = true if args[:ignore_owner].to_s == 'true'
-      verbose = true if args[:verbose].to_s == 'true'
-
-      module_version = metadata['version']
-      owner = metadata['name'].split('-')[0]
-
-      if (owner == 'simp') || ignore_owner
-        # determine last tag
-        `git fetch -t #{tags_source} 2>/dev/null`
-        tags = `git tag -l`.split("\n")
-        puts "Available tags from #{tags_source} = #{tags}" if verbose
-        tags.delete_if { |tag| tag.include?('-') or (tag =~ %r{^v}) }
-
-        if tags.empty?
-          puts "No tags exist from #{tags_source}"
-        else
-          last_tag = tags.max { |a, b| Gem::Version.new(a) <=> Gem::Version.new(b) }
-
-          # determine mission-impacting files that have changed
-          files_changed = `git diff tags/#{last_tag} --name-only`.strip.split("\n")
-          files_changed.delete_if do |file|
-            file[0] ==  '.' || file =~ %r{^Gemfile} || file == 'Rakefile' || file =~ %r{^spec/} || file =~ %r{^doc/} || file =~ %r{^rakelib/}
-          end
-
-          if files_changed.empty?
-            puts "  No new tag required: No significant files have changed since '#{last_tag}' tag"
-          else
-            unless ignore_owner
-              # determine latest version from CHANGELOG, which will present
-              # for all SIMP Puppet modules
-              line = File.readlines('CHANGELOG')[0]
-              match = line.match(CHANGELOG_ENTRY_REGEX)
-              unless match
-                raise("ERROR: Invalid CHANGELOG entry. Unable to extract version from '#{line}'")
-              end
-
-              changelog_version = match[3]
-              unless module_version == changelog_version
-                raise("ERROR: Version mismatch.  module version=#{module_version}  changelog version=#{changelog_version}")
-              end
-            end
-
-            curr_module_version = Gem::Version.new(module_version)
-            last_tag_version = Gem::Version.new(last_tag)
-
-            if curr_module_version < last_tag_version
-              raise("ERROR: Version regression. '#{module_version}' < last tag '#{last_tag}'")
-            elsif curr_module_version == last_tag_version
-              raise("ERROR: Version update beyond last tag '#{last_tag}' is required for #{files_changed.count} changed files:\n  * #{files_changed.join("\n  * ")}")
-            else
-              puts "NOTICE: New tag of version '#{module_version}' is required for #{files_changed.count} changed files:\n  * #{files_changed.join("\n  * ")}"
-            end
-          end
-        end
-      else
-        puts "  Not evaluating module owned by '#{owner}'"
-      end
-    end
-
     desc 'Run syntax, lint, and spec tests.'
     task :test => [
       :syntax,

data/lib/simp/relchecks.rb

@@ -9,6 +9,13 @@ module Simp; end
 
 # Class that provide release-related checks
 class Simp::RelChecks
+  # Paths in a Puppet module that Puppet or Bolt load, plus the docs users
+  # read; a change to any of them requires a new version
+  MODULE_SIGNIFICANT_FILES = %r{
+    \A(?:CHANGELOG|LICENSE|README\.md|REFERENCE\.md|metadata\.json|hiera\.yaml|bolt_plugin\.json)\z |
+    \A(?:SIMP|build|data|facts\.d|files|functions|lib|locales|manifests|plans|tasks|templates|types)/
+  }x.freeze
+
   # Check a component's RPM changelog using the 'rpm' command.
   #
   # This task will fail if 'rpm' detects any changelog problems,
@@ -63,16 +70,8 @@ class Simp::RelChecks
   # (2) A version bump is required but not recorded in both the
   #     CHANGELOG and metadata.json files.
   # (3) The latest version is < latest tag.
-
-  # Changes to the following files/directories are not considered
-  # significant:
-  # - Any hidden file/directory (entry that begins with a '.')
-  # - Gemfile
-  # - Gemfile.lock
-  # - Rakefile
-  # - rakelib directory
-  # - spec directory
-  # - doc directory
+  #
+  # Which changed files are significant is decided by significant_file?
   #
   # +component_dir+:: The root directory of the component project.
   # +tags_source+::   The remote from which the tags for this project
@@ -93,10 +92,8 @@ class Simp::RelChecks
         last_tag = tags.max { |a, b| Gem::Version.new(a) <=> Gem::Version.new(b) }
 
         # determine mission-impacting files that have changed
-        files_changed = `git diff tags/#{last_tag} --name-only`.strip.split("\n")
-        files_changed.delete_if do |file|
-          file[0] == '.' or file == 'Rakefile' or file =~ %r{^Gemfile|^spec/|^doc/|^rakelib/|.*\.md\Z} or file == 'renovate.json'
-        end
+        files_changed = `git diff -z tags/#{last_tag} --name-only --no-renames`.b.split("\0".b)
+        files_changed.select! { |file| significant_file?(file, info.type) }
 
         if files_changed.empty?
           puts "  No new tag required: No significant files have changed since '#{last_tag}' tag"
@@ -116,6 +113,27 @@ class Simp::RelChecks
     end
   end
 
+  # Whether a change to a file requires a new version of the component
+  #
+  # A Puppet module lists the paths Puppet or Bolt load, plus the docs users
+  # read (MODULE_SIGNIFICANT_FILES); any other path is not significant.
+  #
+  # For any other component, every path is significant except:
+  # - Any hidden file/directory (entry that begins with a '.')
+  # - Gemfile*
+  # - Rakefile
+  # - rakelib, spec and doc directories
+  # - Markdown (*.md) files
+  # - renovate.json
+  #
+  # +file+:: Path relative to the root of the component
+  # +type+:: The component type (:module or :asset), from Simp::ComponentInfo
+  def self.significant_file?(file, type)
+    return file.match?(MODULE_SIGNIFICANT_FILES) if type == :module
+
+    !(file[0] == '.' or file == 'Rakefile' or file =~ %r{^Gemfile|^spec/|^doc/|^rakelib/|.*\.md\Z} or file == 'renovate.json')
+  end
+
   # Generate an appropriate changelog for an annotated tag from a
   # component's CHANGELOG or RPM spec file.
   #

data/spec/lib/simp/rake/pupmod/helpers_spec.rb

@@ -22,47 +22,4 @@ describe Simp::Rake::Pupmod::Helpers do
       expect(@othermod.send(:metadata)['name']).to eq 'other-othermod'
     end
   end
-
-  describe '::CHANGELOG_ENTRY_REGEX' do
-    before :all do
-      @rgx = Simp::Rake::Pupmod::Helpers::CHANGELOG_ENTRY_REGEX
-    end
-
-    it 'matches valid one-line CHANGELOG entries' do
-      expect("* Mon Jan 1 1970 First Last <email@domain.com> - 0.0.1\n").to match @rgx
-      expect("* Mon Jan 1 1970 First Last <email@domain.com> 0.0.1\n").to match @rgx
-      expect("* Wed Oct 22 2014 First Middle Last <first.last-a1999@domain.com> - 2.0.0\n").to match @rgx
-      expect('* Wed Oct 22 2014 Name <first.last-a1999+gmail.label@domain.com> - 2.0.0').to match @rgx
-      expect('* Fri Jul 02 2010 Kamil Dudka <kdudka@redhat.com> 2.1.5-20').to match @rgx
-      expect('* Tue Feb 08 2011 Fedora Release Engineering <rel-eng@lists.fedoraproject.org> - 2.1.5-21').to match @rgx
-    end
-
-    it "doesn't match invalid one-line CHANGELOG entries" do
-      expect("* Foo Jan 1 1970 First Last <email@domain.com> - 0.0.1\n").not_to match @rgx
-      expect("* Mon Xxn 1 1970 First Last <email@domain.com> - 0.0.1\n").not_to match @rgx
-      expect("* Mon Jan 111 1970 First Last <email@domain.com> - 0.0.1\n").not_to match @rgx
-      expect("* Mon Jan 1 1970 <email@domain.com> - 0.0.1\n").not_to match @rgx
-    end
-
-    it 'matches valid two-line CHANGELOG entries' do
-      skip 'Not worth implementing right now'
-    end
-  end
-
-  describe '#changelog_annotation' do
-    it "generates a tag annotation from a valid SIMP module's CHANGELOG" do
-      _log = @simpmod.send(:changelog_annotation)
-      expect(_log.class).to be String
-      expect(_log.size).to be > 0
-      expect(_log.scan(%r{^Release of}).size).to eq 1
-    end
-
-    it 'handles multiple CHANGELOG entries for the same release' do
-      _log = @simpmod.send(:changelog_annotation)
-      expect(_log.scan(%r{^\*.*\d+\.\d+\.\d+$})).to eq [
-        '* Tue Jan 2 1970 Second Author <email1@domain.com> - 0.1.0',
-        '* Mon Jan 1 1970 First Author <email2@domain.com> - 0.1.0',
-      ]
-    end
-  end
 end

data/spec/lib/simp/relchecks_compare_latest_tag_spec.rb

@@ -22,7 +22,7 @@ describe 'Simp::RelChecks.compare_latest_tag' do
     it 'reports no new tag required when no files have changed' do
       expect(Simp::RelChecks).to receive(:`).with('git fetch -t origin 2>/dev/null').and_return("\n")
       expect(Simp::RelChecks).to receive(:`).with('git tag -l').and_return("1.0.0-pre\n1.0.0\n1.1.0\n")
-      expect(Simp::RelChecks).to receive(:`).with('git diff tags/1.1.0 --name-only').and_return("\n")
+      expect(Simp::RelChecks).to receive(:`).with('git diff -z tags/1.1.0 --name-only --no-renames').and_return('')
 
       msg = "  No new tag required: No significant files have changed since '1.1.0' tag\n"
       expect { Simp::RelChecks.compare_latest_tag(component_dir) }
@@ -32,8 +32,9 @@ describe 'Simp::RelChecks.compare_latest_tag' do
     it 'reports no new tag required when no significant files have changed' do
       expect(Simp::RelChecks).to receive(:`).with('git fetch -t origin 2>/dev/null').and_return("\n")
       expect(Simp::RelChecks).to receive(:`).with('git tag -l').and_return("1.0.0\nv1.0.1\n1.1.0\n")
-      expect(Simp::RelChecks).to receive(:`).with('git diff tags/1.1.0 --name-only').and_return(
-        ".travis.yml\nRakefile\nREFERENCE.md\nGemfile.lock\nspec/some_spec.rb\ndoc/index.html\nrakelib/mytasks.rake\nrenovate.json\n",
+      expect(Simp::RelChecks).to receive(:`).with('git diff -z tags/1.1.0 --name-only --no-renames').and_return(
+        ".travis.yml\0Rakefile\0AGENTS.md\0Gemfile.lock\0spec/some_spec.rb\0doc/index.html\0rakelib/mytasks.rake\0renovate.json\0" \
+        ".github/workflows/pr_tests.yml\0examples/init.pp\0",
       )
 
       msg = "  No new tag required: No significant files have changed since '1.1.0' tag\n"
@@ -44,15 +45,15 @@ describe 'Simp::RelChecks.compare_latest_tag' do
     it 'reports a new tag is required for significant changes with bumped version' do
       expect(Simp::RelChecks).to receive(:`).with('git fetch -t origin 2>/dev/null').and_return("\n")
       expect(Simp::RelChecks).to receive(:`).with('git tag -l').and_return("1.0.0\n1.1.0-RC01\n")
-      expect(Simp::RelChecks).to receive(:`).with('git diff tags/1.0.0 --name-only').and_return(
-        "CHANGELOG\nmetadata.json\nmanifest/init.pp\n",
+      expect(Simp::RelChecks).to receive(:`).with('git diff -z tags/1.0.0 --name-only --no-renames').and_return(
+        "CHANGELOG\0metadata.json\0manifests/init.pp\0",
       )
 
       msg = <<-EOM
 NOTICE: New tag of version '1.1.0' is required for 3 changed files:
   * CHANGELOG
   * metadata.json
-  * manifest/init.pp
+  * manifests/init.pp
       EOM
 
       expect { Simp::RelChecks.compare_latest_tag(component_dir) }
@@ -64,8 +65,8 @@ NOTICE: New tag of version '1.1.0' is required for 3 changed files:
     it 'fails when latest version < latest tag' do
       expect(Simp::RelChecks).to receive(:`).with('git fetch -t origin 2>/dev/null').and_return("\n")
       expect(Simp::RelChecks).to receive(:`).with('git tag -l').and_return("1.0.0\n1.2.0\n")
-      expect(Simp::RelChecks).to receive(:`).with('git diff tags/1.2.0 --name-only').and_return(
-        "CHANGELOG\nmetadata.json\nmanifest/init.pp\n",
+      expect(Simp::RelChecks).to receive(:`).with('git diff -z tags/1.2.0 --name-only --no-renames').and_return(
+        "CHANGELOG\0metadata.json\0manifests/init.pp\0",
       )
 
       expect { Simp::RelChecks.compare_latest_tag(component_dir) }
@@ -75,8 +76,30 @@ NOTICE: New tag of version '1.1.0' is required for 3 changed files:
     it 'fails when significant file changes need a version bump' do
       expect(Simp::RelChecks).to receive(:`).with('git fetch -t origin 2>/dev/null').and_return("\n")
       expect(Simp::RelChecks).to receive(:`).with('git tag -l').and_return("1.0.0\n1.1.0\n")
-      expect(Simp::RelChecks).to receive(:`).with('git diff tags/1.1.0 --name-only').and_return(
-        "manifest/init.pp\n",
+      expect(Simp::RelChecks).to receive(:`).with('git diff -z tags/1.1.0 --name-only --no-renames').and_return(
+        "manifests/init.pp\0",
+      )
+
+      expect { Simp::RelChecks.compare_latest_tag(component_dir) }
+        .to raise_error(%r{ERROR: Version update beyond last tag '1.1.0' is required for 1 changed files:})
+    end
+
+    it 'fails when shipped documentation changes need a version bump' do
+      expect(Simp::RelChecks).to receive(:`).with('git fetch -t origin 2>/dev/null').and_return("\n")
+      expect(Simp::RelChecks).to receive(:`).with('git tag -l').and_return("1.0.0\n1.1.0\n")
+      expect(Simp::RelChecks).to receive(:`).with('git diff -z tags/1.1.0 --name-only --no-renames').and_return(
+        "README.md\0REFERENCE.md\0AGENTS.md\0",
+      )
+
+      expect { Simp::RelChecks.compare_latest_tag(component_dir) }
+        .to raise_error(%r{ERROR: Version update beyond last tag '1.1.0' is required for 2 changed files:})
+    end
+
+    it 'fails when a significant path has characters git would quote' do
+      expect(Simp::RelChecks).to receive(:`).with('git fetch -t origin 2>/dev/null').and_return("\n")
+      expect(Simp::RelChecks).to receive(:`).with('git tag -l').and_return("1.0.0\n1.1.0\n")
+      expect(Simp::RelChecks).to receive(:`).with('git diff -z tags/1.1.0 --name-only --no-renames').and_return(
+        "templates/caf\u00e9.epp\0",
       )
 
       expect { Simp::RelChecks.compare_latest_tag(component_dir) }

data/spec/lib/simp/relchecks_significant_file_spec.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require 'simp/relchecks'
+require 'spec_helper'
+
+describe 'Simp::RelChecks.significant_file?' do
+  context 'with a Puppet module' do
+    [
+      'CHANGELOG', 'LICENSE', 'README.md', 'REFERENCE.md', 'metadata.json', 'hiera.yaml', 'bolt_plugin.json',
+      'SIMP/compliance_profiles/checks.yaml', 'build/rpm_metadata/requires', 'data/common.yaml',
+      'facts.d/x.sh', 'files/x.conf', 'functions/x.pp', 'lib/facter/x.rb', 'locales/config.yaml',
+      'manifests/init.pp', 'manifests/README.md', 'plans/x.pp', 'tasks/x.json', 'templates/x.epp', 'types/x.pp'
+    ].each do |file|
+      it "considers #{file} significant" do
+        expect(Simp::RelChecks.significant_file?(file, :module)).to be true
+      end
+    end
+
+    [
+      'AGENTS.md', 'CLAUDE.md', 'Gemfile', 'Gemfile.project', 'Rakefile',
+      'renovate.json', '.github/workflows/pr_tests.yml', '.fixtures.yml', 'spec/classes/init_spec.rb',
+      'examples/init.pp', 'doc/index.html', 'rakelib/x.rake', 'metadata.json.bak', 'manifest/init.pp'
+    ].each do |file|
+      it "does not consider #{file} significant" do
+        expect(Simp::RelChecks.significant_file?(file, :module)).to be false
+      end
+    end
+  end
+
+  context 'with a non-module component' do
+    ['build/simp.spec', 'src/x.sh', 'examples/x.conf', 'LICENSE'].each do |file|
+      it "considers #{file} significant" do
+        expect(Simp::RelChecks.significant_file?(file, :asset)).to be true
+      end
+    end
+
+    [
+      '.github/workflows/pr_tests.yml', 'Gemfile', 'Gemfile.lock', 'Rakefile', 'README.md',
+      'docs/x.md', 'spec/x_spec.rb', 'doc/index.html', 'rakelib/x.rake', 'renovate.json'
+    ].each do |file|
+      it "does not consider #{file} significant" do
+        expect(Simp::RelChecks.significant_file?(file, :asset)).to be false
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: simp-rake-helpers
 version: !ruby/object:Gem::Version
-  version: 6.2.0
+  version: 7.0.0
 platform: ruby
 authors:
 - Chris Tessmer
 - Trevor Vaughan
 bindir: bin
 cert_chain: []
-date: 2026-09-28 00:00:00.000000000 Z
+date: 2026-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simp-beaker-helpers
@@ -84,6 +84,26 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: json
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.3'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.3'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: puppet-modulebuilder
   requirement: !ruby/object:Gem::Requirement
@@ -173,7 +193,7 @@ dependencies:
         version: '14.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '15.0'
+        version: '16.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -183,7 +203,7 @@ dependencies:
         version: '14.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '15.0'
+        version: '16.0'
 - !ruby/object:Gem::Dependency
   name: metadata-json-lint
   requirement: !ruby/object:Gem::Requirement
@@ -623,6 +643,7 @@ files:
 - spec/lib/simp/relchecks_check_rpm_changelog_spec.rb
 - spec/lib/simp/relchecks_compare_latest_tag_spec.rb
 - spec/lib/simp/relchecks_create_tag_changelog_spec.rb
+- spec/lib/simp/relchecks_significant_file_spec.rb
 - spec/lib/simp/rpm_signer_spec.rb
 - spec/lib/simp/rpm_spec.rb
 - spec/spec_helper.rb