kaizo 0.8.0 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a33b49134e0a0a871e9b6358cf3fcc9ab88d9ebaca5e8928003de08594c01a11
-  data.tar.gz: 7ed8c9d6780bc64c233b94135850694da8b2bfa1e314dd7369dddf7560f7d42d
+  metadata.gz: cad3868f708e0f017a8591fbc77881d53054d4fcc672c9a6adbfdb2ff7928834
+  data.tar.gz: 37b9f18bfd87518a2027ac6786352600d1c3d1f68153eeadbab8d964c835a90d
 SHA512:
-  metadata.gz: b4c8a8645f32a48c82b52d23966d37d2608c63a0c84e4820e33c15ed6f58bb84c40658eef9de990874e3e0d304c8d6d27abec8ef5f708ff3468f6430663cdd12
-  data.tar.gz: f4189d76f5bb36959f71c84f0497b33805e77ae4e8b04941c78e8517a9012f92447dbbb08c8063b5738746018330fba474abcee324656461af7ec83f2946a96a
+  metadata.gz: 74e87adf1362de156eac0285be0c51b89eec0b2209bf9371a3bf3ac52c41404b49c3f694bfebfea9ec9d3bf32ab572816e580575a39af5a568fc74bd844c91c5
+  data.tar.gz: 6789d9a9dabe6bcc17fb2d90702478d42d3593aed4e934743ed1f6c05946bfc1294d0a229173f1e345a1efc2f897a0f436630597ce33fac79e69430d50bc5cd6

data/CHANGELOG.md

@@ -1,6 +1,89 @@
 # Changelog
 
-## Unreleased
+## 0.9.1
+
+- `Kaizo/KeywordArguments` and `Kaizo/TotalArguments` no longer run on `spec/`
+  and `test/` trees: wide keyword interfaces are the testing idiom there
+  (FactoryBot-style `create`/`build` helpers), and a total bound would
+  re-police the keywords the exemption frees. Override with `Exclude: []` to
+  police tests too. `Kaizo/PositionalArguments` still runs everywhere --
+  positional arguments communicate nothing unless they are solo.
+- Rewrite the README example-first: every cop section leads with its bad/good
+  example, a fourteen-cop table replaces the catalog paragraph, a real
+  `rubocop --only Kaizo` transcript opens the document, and the enablement
+  policy is stated plainly (every cop ships enabled; the plugin's
+  out-of-department changes are disabling `Style/RedundantBegin` and the
+  `Style/HashSyntax` shorthand enforcement shipped since 0.8.0).
+- Verify the README in CI: `spec/readme_examples_spec.rb` executes every
+  example against the shipped config — bad code must be flagged by the cop
+  its marker names, good code must pass the whole department, YAML fences
+  must parse into real cops with shipped option keys, suffix claims run with
+  the documented merge applied, the console transcript is re-derived from the
+  cops, and every cop must have both a bad and a good example.
+- Fix two rdoc errors found in review: `Kaizo/AgentNounClassName`'s
+  `inherit_mode` example was a no-op (`Ledger` is already a default), and
+  `Kaizo/ExplicitBegin`'s docs called re-enabling `Style/RedundantBegin` an
+  opt-out when it leaves both cops fighting; the opt-out is disabling
+  `Kaizo/ExplicitBegin`.
+- Fix a README error found in review: the enablement policy named
+  `Style/RedundantBegin` as the only out-of-department change, omitting the
+  `Style/HashSyntax` shorthand enforcement the Public Config has shipped
+  since 0.8.0. The policy now names both, and a guard spec pins the Public
+  Config's foreign-cop list to exactly the two the README names.
+
+## 0.9.0
+
+- Add `Kaizo/TempfileCreate` cop: requires temporary files to be created with
+  block-form `Tempfile.create`, the only temp-file API whose cleanup is
+  deterministic (the block closes and removes the file when it returns).
+  `Tempfile.new` and `Tempfile.open` are flagged — their cleanup rides on a GC
+  finalizer that may run late or never — as is blockless `Tempfile.create`,
+  which returns a plain `File` that is never removed automatically. A
+  block-pass argument (`Tempfile.create("x", &writer)`) counts as a block. No
+  autocorrection. Enabled by default.
+- Add `Kaizo/SpecSubject` cop: requires the unit under test to be declared with
+  `subject`, not `let`. A `let`/`let!` is flagged when its block confidently
+  builds the class under test — a `.new` on `described_class`, on the constant
+  an enclosing `describe`/`context` names (full or short name), or on a
+  constant matching the spec's file name (`pool_spec.rb` names `Pool`,
+  `api_client_spec.rb` names `APIClient`). Deliberate second instances (an
+  `other` in an equality spec) are exempted with `AllowedMethods` /
+  `AllowedPatterns`, matched against the `let` name. Runs on `**/*_spec.rb`.
+  No autocorrection. Enabled by default.
+- `Kaizo/SpecComment` no longer inspects `spec/helpers` or `spec/support` by
+  default: those directories hold infrastructure, not specs, and prose there is
+  legitimate. The exclusion is plain per-cop `Exclude` configuration, so
+  override it (`Exclude: []`) to police them again, or broaden/narrow the scope
+  with the standard `Include`/`Exclude` as before.
+- `Kaizo/SpecDescriptionProse` no longer flags error class names in
+  descriptions: `it "raises Foo::Error"` passes, because the error is part of
+  the specified behavior — it is what the user ultimately sees raised. Any
+  constant path whose final segment ends in `Error` or `Exception` reads as
+  prose; the rest of the description is still checked.
+- `Kaizo/SpecDescriptionProse` gains `AllowedPatterns`: regexps matched against
+  the whole description that exempt it from every rule — the ad-hoc escape
+  hatch for a description that must quote something code-shaped.
+- **Renamed** `Kaizo/SpecDescriptionProse`'s options: `Conjunctions` is now
+  `ForbiddenWords` and `ContextPrefixes` is now `RequiredContextPrefixes`. The
+  old names said nothing about whether the lists include or exclude, or how
+  the filter works; the new ones do. This is a clean break: the old keys are
+  no longer read — RuboCop warns that the parameter is unsupported and the
+  shipped defaults apply — so rename them when upgrading.
+- The argument-counting cops (`Kaizo/PositionalArguments`,
+  `Kaizo/KeywordArguments`, `Kaizo/TotalArguments`) gain `AllowedMethods` and
+  `AllowedPatterns`: exempt individual methods by name or regexp — e.g. allow
+  `#initialize` to gather collaborators while everything else stays under a
+  strict `Max`. Covers `def`, `def self.`, and the symbol/string
+  `define_method` forms.
+- `Kaizo/PluralCollectionName`'s `ArrayMethods` list is now genuinely
+  configurable (the README already described it as such); the default list is
+  spelled out in the shipped config so `inherit_mode: merge` can extend it.
+- Every cop's rdoc header now carries a `== Configuration` section documenting
+  each option, its semantics, and its default, with a ready-to-paste
+  `.rubocop.yml` snippet — the config plane was previously undocumented at the
+  class level.
+
+## 0.8.0
 
 - Add `Kaizo/PluralCollectionName` cop: flags a method that returns an array
   under a singular name (`def user` handing back `[first, second]`), since a
@@ -31,6 +114,11 @@
   or `tmp` satisfies the rule while missing what it asks for. The offense message
   itself is unchanged.
 
+## 0.7.0
+
+- No user-facing changes. Internal tooling and gem metadata only — nothing in
+  the shipped cops or their configuration differed from 0.6.0.
+
 ## 0.6.0
 
 - **Renamed the gem from `rubocop-design` to `kaizo`**, and moved every cop from