rubocop-vicenzo 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a18abf5413cb9807bf31adf6adeb6ac47e3fd0e7ec8cb2d47217db9b1617ed40
-  data.tar.gz: 871b1cb1de89c418517b67919402004ea95044aad2b1a8a0602ceb29953d1093
+  metadata.gz: a5b82e09fef23aac921adb4f653bfbdc1117af279326d24339f68db8abe25ffd
+  data.tar.gz: f5f4d255aacd3851e4f8b7968b663d22db0b0ed9be99bab01a17df43d819c80f
 SHA512:
-  metadata.gz: c629161e84a929d57e0ef7fdec80d17a21742adc08c7f1bafb601356815e99862d987806bfced0e88fb5d1c085af35a4f0e6039dde7cdd664c635a51e9984110
-  data.tar.gz: 05ddcb76888e50820eda315fa63087c12a01ace730090bd5250f363a2a01b46e8277342ff903d371a5b5d6a0d8c9abcf28023a88caca421c7bfca558acbae110
+  metadata.gz: 79d36dbf73503ca38cca4d5ffd77fd84d56cf4b8d320ff389035c359116e36d66d15a3aad4fd3de10d9787fe29f4b4af66b0b669ce98754415fac8ad954106fa
+  data.tar.gz: 84055ca6301311a752670e04fa5fcd9646fc38844993428a5f952be17a7384cfa5842b9a2daaa7f3be3242d70e0641b7ba02fdd718fee111f995bd1cd4a8b3e0

data/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "0.6.0"
+}

data/CHANGELOG.md

@@ -1,4 +1,11 @@
-## [Unreleased]
+# Changelog
+
+## [0.6.0](https://github.com/bvicenzo/rubocop-vicenzo/compare/v0.5.0...v0.6.0) (2026-08-19)
+
+
+### Features
+
+* add RSpec cops against mutated and derived premises ([#26](https://github.com/bvicenzo/rubocop-vicenzo/issues/26)) ([b138dd0](https://github.com/bvicenzo/rubocop-vicenzo/commit/b138dd00a76e1bcd62e95c1f2c51dca5b4232319))
 
 ## [0.5.0] - 2026-06-29
 

data/CONTRIBUTING.md

@@ -0,0 +1,100 @@
+# Contributing
+
+## Development setup
+
+Run `bin/setup` once after cloning. It installs dependencies and points
+`core.hooksPath` at `.githooks/`, enabling a `commit-msg` hook that enforces
+Conventional Commits locally.
+
+Run the test suite and the linter with the default task — the same one CI runs:
+
+```bash
+bundle exec rake
+```
+
+## Commit messages — Conventional Commits
+
+This project follows [Conventional Commits](https://www.conventionalcommits.org/).
+The version bump and the `CHANGELOG.md` are generated automatically from commit
+messages, so the prefix matters:
+
+| Type | When to use | Release effect |
+|------|-------------|----------------|
+| `feat:` | New cop or user-facing capability | minor bump |
+| `fix:` | Bug fix in a cop or in the config | patch bump |
+| `feat!:` or a `BREAKING CHANGE:` footer | Backwards-incompatible change | minor bump while `0.x`, major from `1.0` on |
+| `chore:` `ci:` `docs:` `style:` `test:` `refactor:` | Maintenance | no release |
+
+Title format: `type: short description` (e.g.
+`feat: add Vicenzo/Style/JsonParseSymbolizeNames cop`).
+
+Pull requests are squash-merged, so **the PR title becomes the commit message
+release-please reads**. Get the title right even when the individual commits are
+messy.
+
+## Adding a new cop
+
+Generate the cop, its spec and its `config/default.yml` entry with:
+
+```bash
+bundle exec rake 'new_cop[Vicenzo/Department/CopName]'
+```
+
+The generator writes `VersionAdded: '<<next>>'` into `config/default.yml`. Leave
+that placeholder alone — never replace it with a version by hand. The release
+workflow fills it in with the version being released (see below).
+
+## Releasing (release-please)
+
+Releases are automated with [release-please](https://github.com/googleapis/release-please):
+
+1. Every push to `master` creates or updates a **release PR** titled
+   `chore(master): release X.Y.Z`. It bumps
+   `lib/rubocop/vicenzo/version.rb`, `.release-please-manifest.json`,
+   `Gemfile.lock` and the version in the `README.md` install snippet, and updates
+   `CHANGELOG.md` from the Conventional Commits made since the last release.
+2. New cops are added with `VersionAdded: '<<next>>'`. The release workflow
+   replaces every `<<next>>` in `config/default.yml` with the version being
+   released, directly on the release PR — no manual editing needed. This matters
+   because `rake docs:generate` reads `VersionAdded` to build the "Version Added"
+   column of the documentation site.
+3. **Merging the release PR** creates the git tag `vX.Y.Z` and a GitHub Release.
+   That in turn triggers two things automatically:
+   - `.github/workflows/publish.yml` publishes the gem to RubyGems via
+     [Trusted Publishing](https://guides.rubygems.org/trusted-publishing/) (OIDC,
+     no API key stored in the repository).
+   - `.github/workflows/docs.yml` rebuilds and deploys the documentation site to
+     GitHub Pages, because `lib/rubocop/vicenzo/version.rb` changed.
+4. The release PR always reflects **everything on `master` since the last
+   release**. Cut releases promptly: merge the release PR before landing
+   unrelated new work you don't want included in that release.
+
+Nothing about a release is done by hand — there is no `rake release` to run
+locally and no version to edit yourself.
+
+## If the release PR is failing
+
+release-please only rebuilds the release branch when a **releasable** commit
+(`feat:` / `fix:`) lands on `master`. Non-releasable fixes (`chore:`, `style:`,
+`ci:`, `docs:`, `test:`) do **not** rebuild it, so the release branch can fall
+behind `master` and its CI can run stale code.
+
+When the release PR's CI is red:
+
+1. Push the fix to `master` through a normal PR. **Never** push directly to the
+   `release-please--…` branch — release-please overwrites it.
+2. If those fix commits are non-releasable (so the release PR won't refresh on
+   its own), **close the release PR and delete its branch**. On the next push to
+   `master`, release-please recreates the release PR from the current `master`
+   (now including the fix) and its CI passes.
+3. Merge the freshly recreated release PR.
+
+Never manually rebase or force-push the release-please branch.
+
+## If the gem was not published
+
+The tag and the GitHub Release already exist, so publishing is retryable: fix the
+cause and use **Re-run jobs** on the failed `Publish gem` workflow run. An
+authentication failure points at the trusted publisher registration on RubyGems
+(the repository or the workflow filename must match `publish.yml`), not at the
+workflow itself.

data/README.md

@@ -6,7 +6,20 @@
 
 ## Installation
 
-Install the gem and add to the application's Gemfile by executing:
+Add it to the `development`/`test` group of your `Gemfile`:
+
+<!-- x-release-please-start-version -->
+```ruby
+group :development, :test do
+  gem 'rubocop-vicenzo', '~> 0.6.0', require: false
+end
+```
+<!-- x-release-please-end-version -->
+
+> The version above is kept current automatically on every release; pin to
+> whichever version you prefer.
+
+Or let Bundler add the latest version for you:
 
 ```bash
 bundle add rubocop-vicenzo --group=development --require=false
@@ -55,7 +68,13 @@ rubocop --plugin rubocop-vicenzo
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
 
-To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and the created tag, and push the `.gem` file to [rubygems.org](https://rubygems.org).
+To install this gem onto your local machine, run `bundle exec rake install`.
+
+Releases are automated: the version, the `CHANGELOG.md`, the git tag and the push
+to [rubygems.org](https://rubygems.org) are all derived from
+[Conventional Commits](https://www.conventionalcommits.org/) by release-please.
+There is no version to edit and no release command to run by hand — see
+[CONTRIBUTING.md](CONTRIBUTING.md).
 
 ### Documentation
 
@@ -92,7 +111,7 @@ bundle exec rake 'new_cop[Vicenzo/OptionalNamespace/CopName]'
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/[USERNAME]/rubocop-vicenzo. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://github.com/[USERNAME]/rubocop-vicenzo/blob/master/CODE_OF_CONDUCT.md).
+Bug reports and pull requests are welcome on GitHub at https://github.com/bvicenzo/rubocop-vicenzo. Read [CONTRIBUTING.md](CONTRIBUTING.md) first — it covers the commit conventions the release automation depends on. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://github.com/bvicenzo/rubocop-vicenzo/blob/master/CODE_OF_CONDUCT.md).
 
 ## License
 
@@ -100,4 +119,4 @@ The gem is available as open source under the terms of the [MIT License](https:/
 
 ## Code of Conduct
 
-Everyone interacting in the Rubocop::Vicenzo project's codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/[USERNAME]/rubocop-vicenzo/blob/master/CODE_OF_CONDUCT.md).
+Everyone interacting in the Rubocop::Vicenzo project's codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/bvicenzo/rubocop-vicenzo/blob/master/CODE_OF_CONDUCT.md).

data/config/default.yml

@@ -22,6 +22,17 @@ Vicenzo/RSpec/ConditionalInSpec:
     - '**/spec/factories/**/*'
   VersionAdded: '0.4.0'
 
+Vicenzo/RSpec/DerivedPremises:
+  Description: 'Do not derive a premise from another premise; declare the complete value where it is needed.'
+  Enabled: true
+  Severity: warning
+  Include:
+    - '**/spec/**/*_spec.rb'
+  Exclude:
+    - '**/spec/support/**/*'
+    - '**/spec/factories/**/*'
+  VersionAdded: '0.6.0'
+
 Vicenzo/RSpec/DynamicExampleGeneration:
   Description: 'Do not use iteration to dynamically generate example groups or examples.'
   Enabled: true
@@ -40,6 +51,17 @@ Vicenzo/RSpec/IterationInsideExample:
   Severity: warning
   VersionAdded: '0.4.0'
 
+Vicenzo/RSpec/MutatedPremises:
+  Description: 'Do not mutate the premises (let, let_it_be, subject) an example starts from.'
+  Enabled: true
+  Severity: warning
+  Include:
+    - '**/spec/**/*_spec.rb'
+  Exclude:
+    - '**/spec/support/**/*'
+    - '**/spec/factories/**/*'
+  VersionAdded: '0.6.0'
+
 Vicenzo/RSpec/NestedContextImproperStart:
   Description: 'Check if the nested context does not start as a root one.'
   Enabled: true

data/lib/rubocop/cop/vicenzo/rspec/derived_premises.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+require_relative 'mixin/premise_tracking'
+
+module RuboCop
+  module Cop
+    module Vicenzo
+      module RSpec
+        # Do not derive a premise from another premise.
+        #
+        # `let(:params) { base_params.merge(age: 10) }` does not mutate anything, but it splits a single premise
+        # across two definitions: to know what `params` holds in a context you have to read another `let` — usually
+        # one that exists only to be modified. Declare the complete value in the context that needs it.
+        #
+        # Only premise definitions (`let`, `let_it_be`, `subject`) are inspected, and only when the copy starts from
+        # another premise: deriving from a factory or from a literal is untouched.
+        #
+        # @example
+        #   # bad
+        #
+        #   let(:base_params) { { name: 'Ada' } }
+        #   let(:params) { base_params.merge(age: 10) }
+        #
+        #   # good
+        #
+        #   context 'when the age is informed' do
+        #     let(:params) { { name: 'Ada', age: 10 } }
+        #   end
+        #
+        #   context 'when the age is unknown' do
+        #     let(:params) { { name: 'Ada' } }
+        #   end
+        #
+        # @example
+        #   # good — the source is a factory, not another premise
+        #
+        #   let(:params) { attributes_for(:order).merge(value: 10) }
+        class DerivedPremises < RuboCop::Cop::RSpec::Base
+          include PremiseTracking
+
+          MSG = 'Do not derive the premise `%<name>s` from `%<source>s`. ' \
+                'Declare the complete value in the context that needs it.'
+
+          DERIVING_METHODS = %i[merge deep_merge reverse_merge dup clone except slice].freeze
+
+          def on_block(node)
+            return unless example_group?(node) && outermost_example_group?(node)
+
+            walk_example_group(node, all_premises(node))
+          end
+
+          alias on_numblock on_block
+
+          private
+
+          def on_premise(node, premises)
+            name = premise_name(node)
+
+            mutating_calls(node).each do |call|
+              next unless DERIVING_METHODS.include?(call.method_name)
+
+              source = premise_receiver(call, premises)
+
+              next if source.nil?
+
+              add_offense(call, message: format(MSG, name:, source:))
+            end
+          end
+
+          def on_setup_hook(node, premises); end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/vicenzo/rspec/dynamic_example_generation.rb

@@ -14,7 +14,7 @@ module RuboCop
         # @example
         #   # bad
         #
-        #   [:admin, :driver].each do |role|
+        #   [:admin, :editor].each do |role|
         #     context "when role is #{role}" do
         #       it 'does something' do
         #         ...
@@ -32,8 +32,8 @@ module RuboCop
         #     end
         #   end
         #
-        #   context 'when role is driver' do
-        #     let(:role) { :driver }
+        #   context 'when role is editor' do
+        #     let(:role) { :editor }
         #
         #     it 'does something' do
         #       ...

data/lib/rubocop/cop/vicenzo/rspec/iteration_inside_example.rb

@@ -16,27 +16,27 @@ module RuboCop
         # @example
         #   # bad — expect is called inside the iteration
         #
-        #   it 'returns vehicle costs general values' do
-        #     response_body[:vehicle_costs].first.each do |attribute, value|
-        #       expect(value).to eq(vehicle_cost.send(attribute).to_s)
+        #   it 'returns product prices general values' do
+        #     response_body[:product_prices].first.each do |attribute, value|
+        #       expect(value).to eq(product_price.send(attribute).to_s)
         #     end
         #   end
         #
         #   # good — iteration builds data, expect is called once outside
         #
         #   it 'returns the expected column names' do
-        #     columns = VehicleCost.column_names.map { |column| column.gsub('_centavos', '') }
-        #     expect(response_body[:vehicle_costs].first.keys).to match_array(columns.map(&:to_sym))
+        #     columns = ProductPrice.column_names.map { |column| column.gsub('_cents', '') }
+        #     expect(response_body[:product_prices].first.keys).to match_array(columns.map(&:to_sym))
         #   end
         #
         #   # good — each attribute has an explicit example
         #
         #   it 'returns the correct name' do
-        #     expect(response_body[:vehicle_costs].first[:name]).to eq(vehicle_cost.name)
+        #     expect(response_body[:product_prices].first[:name]).to eq(product_price.name)
         #   end
         #
         #   it 'returns the correct value' do
-        #     expect(response_body[:vehicle_costs].first[:value]).to eq(vehicle_cost.value.to_s)
+        #     expect(response_body[:product_prices].first[:value]).to eq(product_price.value.to_s)
         #   end
         class IterationInsideExample < RuboCop::Cop::RSpec::Base
           MSG = 'Do not call `expect` inside an iteration. ' \

data/lib/rubocop/cop/vicenzo/rspec/mixin/premise_tracking.rb

@@ -0,0 +1,117 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Vicenzo
+      module RSpec
+        # Shared helpers to walk example groups keeping track of the premises
+        # (`let`, `let!`, `let_it_be`, `subject`) visible at each level, and to
+        # resolve which premise a call chain starts from.
+        module PremiseTracking
+          extend RuboCop::AST::NodePattern::Macros
+
+          # Brings `Helpers` and `Subjects` into this module's constant lookup, so the patterns below resolve the
+          # same RSpec DSL names the cops do.
+          include RuboCop::RSpec::Language
+
+          # Hooks that build the state an example starts from. `after` is left
+          # out on purpose: mutating there is teardown, not a premise.
+          SETUP_HOOKS = %i[before around prepend_before append_before].freeze
+
+          # @!method premise_name(node)
+          def_node_matcher :premise_name, <<~PATTERN
+            (any_block
+              (send nil?
+                {
+                  {#Helpers.all :let_it_be :let_it_be!} ({str sym} $_)
+                  | #Subjects.all (sym $_)
+                  | $#Subjects.all
+                }
+                ...
+              )
+              ...
+            )
+          PATTERN
+
+          # @!method root_receiver_name(node)
+          #   Name of the local call a chain starts from, e.g. `:order`
+          #   for `order.shipment.update!`.
+          def_node_matcher :root_receiver_name, <<~PATTERN
+            (send nil? $_)
+          PATTERN
+
+          private
+
+          def walk_example_group(group, premises)
+            each_child_block(group) do |child|
+              if example_group?(child)
+                walk_example_group(child, premises)
+              elsif premise?(child)
+                on_premise(child, premises)
+              elsif setup_hook?(child)
+                on_setup_hook(child, premises)
+              end
+            end
+          end
+
+          # Every premise in the file, not only the ones an ancestor group declares. A premise defined in a child
+          # group is still the premise a definition above reads — the "abstract let" shape — and that is the hardest
+          # one to follow when reading.
+          def all_premises(root)
+            names = Set.new
+
+            names << premise_name(root) if premise?(root)
+            root.each_descendant(:block, :numblock) { |node| names << premise_name(node) if premise?(node) }
+
+            names
+          end
+
+          def each_child_block(group)
+            body = group.body
+
+            return if body.nil?
+
+            children = body.begin_type? ? body.children : [body]
+
+            children.each { |child| yield(child) if child.any_block_type? }
+          end
+
+          def outermost_example_group?(node)
+            node.each_ancestor(:block, :numblock).none? { |ancestor| example_group?(ancestor) }
+          end
+
+          def premise?(node) = let?(node) || let_it_be?(node) || subject?(node)
+
+          # @!method let_it_be?(node)
+          def_node_matcher :let_it_be?, <<~PATTERN
+            (any_block (send nil? {:let_it_be :let_it_be!} ...) ...)
+          PATTERN
+
+          def setup_hook?(node) = hook?(node) && SETUP_HOOKS.include?(node.method_name)
+
+          def premise_receiver(node, premises)
+            receiver = node.receiver
+
+            while receiver&.send_type?
+              name = root_receiver_name(receiver)
+
+              return name if name && premises.include?(name)
+
+              receiver = receiver.receiver
+            end
+
+            nil
+          end
+
+          def mutating_calls(node)
+            body = node.body
+
+            return [] if body.nil?
+
+            body.each_node(:send).to_a
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/vicenzo/rspec/mutated_premises.rb

@@ -0,0 +1,140 @@
+# frozen_string_literal: true
+
+require_relative 'mixin/premise_tracking'
+
+module RuboCop
+  module Cop
+    module Vicenzo
+      module RSpec
+        # Do not mutate the premises of an example.
+        #
+        # A `let`, `let_it_be` or `subject` is the premise a reader assumes true when reading the file. Mutating it
+        # somewhere else breaks that in two ways:
+        #
+        # 1. The file lies. You read `let(:params) { { name: 'Ada' } }`, run it, and the value is different — the
+        #    explanation is a `before` dozens of lines away.
+        # 2. It breeds flakiness. `let` is lazy, so whichever example materialises it first decides the final state.
+        #    A context that touches the premises in another order silently changes the result.
+        #
+        # Persisting changes while building a premise is flagged wherever it happens inside the definition, including
+        # inline forms such as `tap`, because a factory trait or transient is always available instead. Collection
+        # mutations and attribute writers are flagged when they change a premise declared in an ancestor group, so
+        # building a local value inside the definition itself stays allowed.
+        #
+        # Examples are not inspected: there the mutation usually *is* the behaviour under test.
+        #
+        # @example
+        #   # bad — persisting while building the premise
+        #
+        #   let(:order) { create(:order).tap { |record| record.update!(status: :shipped) } }
+        #
+        #   # bad — the same laziness spelled out
+        #
+        #   let(:order) do
+        #     order = create(:order)
+        #     order.update_column(:status, :shipped)
+        #     order
+        #   end
+        #
+        #   # good — the factory owns it
+        #
+        #   let(:order) { create(:order, :shipped) }
+        #
+        # @example
+        #   # bad — the hook rewrites the premise
+        #
+        #   let(:params) { { name: 'Ada' } }
+        #   before { params[:age] = 10 }
+        #
+        #   # bad — patching a record the premise built
+        #
+        #   before { order.shipment.update!(carrier:) }
+        #
+        #   # good — each context declares its complete premise
+        #
+        #   context 'when the age is informed' do
+        #     let(:params) { { name: 'Ada', age: 10 } }
+        #   end
+        #
+        #   # good — creation order lets the production code do the wiring
+        #
+        #   before do
+        #     carrier
+        #     order
+        #   end
+        #
+        # @example
+        #   # good — local builder, the value is born and dies inside the premise
+        #
+        #   let(:csv_content) { CSV.generate { |csv| csv << header } }
+        class MutatedPremises < RuboCop::Cop::RSpec::Base
+          include PremiseTracking
+
+          MSG_MUTATION = 'Do not mutate the premise `%<name>s`. Declare the final state where it is needed — ' \
+                         'factory trait, factory transient, or creation order.'
+          MSG_PERSISTENCE = 'Do not persist changes while building `%<name>s`. Move it to a factory ' \
+                            'trait/transient, or declare the final state in the context that needs it.'
+
+          PERSISTENCE_METHODS = %i[
+            update update! update_attribute update_column update_columns save save! destroy destroy! touch
+            increment! decrement! toggle!
+          ].freeze
+
+          COLLECTION_METHODS = %i[<< push concat unshift store merge! deep_merge! clear delete].freeze
+
+          def on_block(node)
+            return unless example_group?(node) && outermost_example_group?(node)
+
+            walk_example_group(node, all_premises(node))
+          end
+
+          alias on_numblock on_block
+
+          private
+
+          def on_premise(node, premises)
+            name = premise_name(node)
+
+            mutating_calls(node).each do |call|
+              if persisting?(call)
+                add_offense(call, message: format(MSG_PERSISTENCE, name:))
+              else
+                check_premise_mutation(call, premises)
+              end
+            end
+          end
+
+          def on_setup_hook(node, premises)
+            mutating_calls(node).each do |call|
+              next unless persisting?(call) || mutating_collection?(call)
+
+              check_premise_mutation(call, premises)
+            end
+          end
+
+          def check_premise_mutation(call, premises)
+            return unless persisting?(call) || mutating_collection?(call)
+
+            mutated_premise = premise_receiver(call, premises)
+
+            return if mutated_premise.nil?
+
+            add_offense(call, message: format(MSG_MUTATION, name: mutated_premise))
+          end
+
+          # `FileUtils.touch(path)` and friends act on a class, never on a premise.
+          def persisting?(call)
+            PERSISTENCE_METHODS.include?(call.method_name) && !call.receiver&.const_type?
+          end
+
+          def mutating_collection?(call)
+            COLLECTION_METHODS.include?(call.method_name) || assignment?(call)
+          end
+
+          # Covers both `params[:age] = 10` (`:[]=`) and `record.status = :done` (`:status=`).
+          def assignment?(call) = call.assignment_method? && !call.receiver.nil?
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/vicenzo_cops.rb

@@ -1,9 +1,11 @@
 # frozen_string_literal: true
 
 require_relative 'vicenzo/rspec/conditional_in_spec'
+require_relative 'vicenzo/rspec/derived_premises'
 require_relative 'vicenzo/rspec/dynamic_example_generation'
 require_relative 'vicenzo/rspec/inconsistent_sibling_structure'
 require_relative 'vicenzo/rspec/iteration_inside_example'
+require_relative 'vicenzo/rspec/mutated_premises'
 require_relative 'vicenzo/rspec/nested_context_improper_start'
 require_relative 'vicenzo/rspec/nested_let_redefinition'
 require_relative 'vicenzo/rspec/nested_subject_redefinition'

data/lib/rubocop/vicenzo/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module Vicenzo
-    VERSION = '0.5.0'
+    VERSION = '0.6.0'
   end
 end

data/release-please-config.json

@@ -0,0 +1,15 @@
+{
+  "$schema": "https://raw.githubusercontent.com/googleapis/release-please/main/schemas/config.json",
+  "bump-minor-pre-major": true,
+  "include-component-in-tag": false,
+  "packages": {
+    ".": {
+      "release-type": "ruby",
+      "package-name": "rubocop-vicenzo",
+      "version-file": "lib/rubocop/vicenzo/version.rb",
+      "extra-files": [
+        { "type": "generic", "path": "README.md" }
+      ]
+    }
+  }
+}

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-vicenzo
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Bruno Vicenzo
@@ -59,10 +59,12 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".release-please-manifest.json"
 - ".rspec"
 - ".rubocop.yml"
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
+- CONTRIBUTING.md
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -74,10 +76,13 @@ files:
 - lib/rubocop/cop/vicenzo/layout/multiline_method_call_line_breaks.rb
 - lib/rubocop/cop/vicenzo/rails/enum_inclusion_of_validation.rb
 - lib/rubocop/cop/vicenzo/rspec/conditional_in_spec.rb
+- lib/rubocop/cop/vicenzo/rspec/derived_premises.rb
 - lib/rubocop/cop/vicenzo/rspec/dynamic_example_generation.rb
 - lib/rubocop/cop/vicenzo/rspec/inconsistent_sibling_structure.rb
 - lib/rubocop/cop/vicenzo/rspec/iteration_inside_example.rb
 - lib/rubocop/cop/vicenzo/rspec/leaky_definition.rb
+- lib/rubocop/cop/vicenzo/rspec/mixin/premise_tracking.rb
+- lib/rubocop/cop/vicenzo/rspec/mutated_premises.rb
 - lib/rubocop/cop/vicenzo/rspec/nested_context_improper_start.rb
 - lib/rubocop/cop/vicenzo/rspec/nested_let_redefinition.rb
 - lib/rubocop/cop/vicenzo/rspec/nested_subject_redefinition.rb
@@ -88,7 +93,7 @@ files:
 - lib/rubocop/vicenzo/plugin.rb
 - lib/rubocop/vicenzo/version.rb
 - rakelib/docs.rake
-- rakelib/release.rake
+- release-please-config.json
 - sig/rubocop/vicenzo.rbs
 homepage: https://github.com/bvicenzo/rubocop-vicenzo
 licenses:
@@ -114,7 +119,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 4.0.16
 specification_version: 4
 summary: Cops of Bruno Vicenzo
 test_files: []

data/rakelib/release.rake

@@ -1,73 +0,0 @@
-# frozen_string_literal: true
-
-require 'date'
-
-desc 'Prepare release: update config, version file, and changelog'
-task :cut_release, [:version] do |_t, args|
-  version = args[:version]
-
-  # 1. Validation
-  abort 'Error: You must provide a version. Example: rake cut_release[0.3.0]' unless version
-
-  config_file    = 'config/default.yml'
-  version_file   = 'lib/rubocop/vicenzo/version.rb'
-  changelog_file = 'CHANGELOG.md'
-
-  # Check if files exist
-  [config_file, version_file, changelog_file].each do |file|
-    abort "Error: File not found at #{file}" unless File.exist?(file)
-  end
-
-  puts "✂️  Cutting release for version #{version}..."
-
-  # -------------------------------------------------------
-  # 2. Update config/default.yml (<<next>> -> version)
-  # -------------------------------------------------------
-  config_content = File.read(config_file)
-  if config_content.include?("'<<next>>'")
-    updated_config = config_content.gsub("'<<next>>'", "'#{version}'")
-    File.write(config_file, updated_config)
-    puts "   ✅ Updated 'VersionAdded' in #{config_file}"
-  else
-    puts "   ⚠️  No '<<next>>' found in #{config_file} (skipping)"
-  end
-
-  # -------------------------------------------------------
-  # 3. Update lib/rubocop/vicenzo/version.rb
-  # -------------------------------------------------------
-  version_content = File.read(version_file)
-  # Regex looks for: VERSION = '...' or VERSION = "..."
-  if version_content.match?(/VERSION\s*=\s*['"](.+)['"]/)
-    updated_version = version_content.gsub(/VERSION\s*=\s*['"](.+)['"]/, "VERSION = '#{version}'")
-    File.write(version_file, updated_version)
-    puts "   ✅ Updated VERSION constant in #{version_file}"
-  else
-    puts "   ❌ Could not find VERSION constant in #{version_file}"
-  end
-
-  # -------------------------------------------------------
-  # 4. Update CHANGELOG.md
-  # -------------------------------------------------------
-  changelog_content = File.read(changelog_file)
-  unreleased_header = '## [Unreleased]'
-  date = Date.today.to_s # YYYY-MM-DD
-
-  # We replace "## [Unreleased]" with:
-  # ## [Unreleased]
-  #
-  # ## [version] - date
-  #
-  # This pushes the existing unreleased items down under the new version header.
-  new_entry_header = "#{unreleased_header}\n\n## [#{version}] - #{date}"
-
-  if changelog_content.include?(unreleased_header)
-    # We use 'sub' to replace only the first occurrence (the top one)
-    updated_changelog = changelog_content.sub(unreleased_header, new_entry_header)
-    File.write(changelog_file, updated_changelog)
-    puts "   ✅ Updated #{changelog_file} (moved items to [#{version}])"
-  else
-    puts "   ❌ Could not find '#{unreleased_header}' in #{changelog_file}"
-  end
-
-  puts "\n🎉 Release preparation complete! Don't forget to commit."
-end