evilution 0.26.0 → 0.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3b561b61697995a22463ea15828357375b1ce9868ad6ee87f3f331336d26b890
-  data.tar.gz: 4a0cb24bfe2e9e9478ab5d752f05ea4ea69582e324e736241f1fe6b1e5e581cd
+  metadata.gz: dda90ba09660e134411bad5df79ba56075ed9661f8e94aa3b366478cc3191755
+  data.tar.gz: c1c5606e0a2c082dd1d7ec24738f0558b7cc94bfc3632cd4c2fdd200d5ad5e11
 SHA512:
-  metadata.gz: 416aaea92d50fc3a969c350fd36d47e5b8b625ba4b1c64b66f208731cfaf1250714b8bc8e494bbc1586922357886dd96026f31e9f96b7e2cd849e43f402de3e3
-  data.tar.gz: e81fc9c5edfa9e25b4a7077f2776c2256bf5211bc5d88b53cc604ef735d64366ee3b3a294a480f49d3107c7634ff98459f6f46e0a8ede3248cec2b72007f940e
+  metadata.gz: 30d4d7dcdcb9d88615fa5d37205bd02a307f32b6150a7ee7fa55b8397b677d40dd26e5777043fb6dfeeddf2559c8e79ae23276574ede65627ccf95f9effcd31e
+  data.tar.gz: a657c55edcffd46c6883c4146cd97471db4c51940def54027a49f01ff89a29e7fd6fd4c28e758d9d7207623d664c320ccc22a9a01430abd5d26574afb1142932

data/.beads/interactions.jsonl

@@ -235,3 +235,26 @@
 {"id":"int-2abceed3","kind":"field_change","created_at":"2026-04-24T05:18:13.096916772Z","actor":"Denis Kiselev","issue_id":"EV-kjac","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Closed"}}
 {"id":"int-03bc2f7f","kind":"field_change","created_at":"2026-04-24T05:40:10.83959826Z","actor":"Denis Kiselev","issue_id":"EV-hklf","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Closed"}}
 {"id":"int-2781044c","kind":"field_change","created_at":"2026-04-24T05:40:11.975030418Z","actor":"Denis Kiselev","issue_id":"EV-cpku","extra":{"field":"status","new_value":"closed","old_value":"open","reason":"Folded into EV-hklf — error rename 'no method found' → 'no subject matched' shipped in PR #872."}}
+{"id":"int-3fd6fda2","kind":"field_change","created_at":"2026-04-24T17:15:49.060178577Z","actor":"Denis Kiselev","issue_id":"EV-3ew5","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"PR #884 merged to master"}}
+{"id":"int-95f8d2f1","kind":"field_change","created_at":"2026-04-25T14:49:14.844074734Z","actor":"Denis Kiselev","issue_id":"EV-owgh","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Merged via PR #890"}}
+{"id":"int-d786895b","kind":"field_change","created_at":"2026-04-25T15:46:53.977607499Z","actor":"Denis Kiselev","issue_id":"EV-a6de","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Merged via PR #891"}}
+{"id":"int-a297634a","kind":"field_change","created_at":"2026-04-25T16:56:08.483381206Z","actor":"Denis Kiselev","issue_id":"EV-ilu3","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Merged via PR #892"}}
+{"id":"int-0647d42f","kind":"field_change","created_at":"2026-04-25T17:06:39.083006592Z","actor":"Denis Kiselev","issue_id":"EV-aa4x","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Merged via PR #893"}}
+{"id":"int-86ea8eca","kind":"field_change","created_at":"2026-04-25T17:28:36.516431743Z","actor":"Denis Kiselev","issue_id":"EV-6x6g","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Merged via PR #894"}}
+{"id":"int-811fef5f","kind":"field_change","created_at":"2026-04-25T17:39:31.325092593Z","actor":"Denis Kiselev","issue_id":"EV-6c51","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Merged via PR #895"}}
+{"id":"int-9d182026","kind":"field_change","created_at":"2026-04-25T17:54:26.23542935Z","actor":"Denis Kiselev","issue_id":"EV-67yh","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Merged via PR #897"}}
+{"id":"int-7ede49f5","kind":"field_change","created_at":"2026-04-25T18:16:18.358350457Z","actor":"Denis Kiselev","issue_id":"EV-zvhp","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Merged via PR #898"}}
+{"id":"int-c28f7ee0","kind":"field_change","created_at":"2026-04-26T02:28:38.321564801Z","actor":"Denis Kiselev","issue_id":"EV-vev8","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Merged via PR #899"}}
+{"id":"int-111f631a","kind":"field_change","created_at":"2026-04-26T07:57:40.59622849Z","actor":"Denis Kiselev","issue_id":"EV-p5vh","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Released in v0.27.0 (PR #901 merged, release PR #902 shipped)"}}
+{"id":"int-5a159da9","kind":"field_change","created_at":"2026-04-29T11:51:40.86913186Z","actor":"Denis Kiselev","issue_id":"EV-vm61","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Closed"}}
+{"id":"int-1787900f","kind":"field_change","created_at":"2026-04-30T02:40:52.288627178Z","actor":"Denis Kiselev","issue_id":"EV-v1i6","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"merged via PR #911"}}
+{"id":"int-08958dc0","kind":"field_change","created_at":"2026-04-30T03:15:14.369272687Z","actor":"Denis Kiselev","issue_id":"EV-x3s0","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"merged via PR #912"}}
+{"id":"int-e013b002","kind":"field_change","created_at":"2026-04-30T05:46:44.70653253Z","actor":"Denis Kiselev","issue_id":"EV-eww3","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"merged via PR #913"}}
+{"id":"int-7b15a1b0","kind":"field_change","created_at":"2026-04-30T09:30:21.057039785Z","actor":"Denis Kiselev","issue_id":"EV-7rov","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"merged via PR #914"}}
+{"id":"int-71c9f98d","kind":"field_change","created_at":"2026-04-30T12:30:36.528587334Z","actor":"Denis Kiselev","issue_id":"EV-318q","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"merged"}}
+{"id":"int-42eadbdc","kind":"field_change","created_at":"2026-04-30T16:09:00.539400752Z","actor":"Denis Kiselev","issue_id":"EV-voay","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"merged"}}
+{"id":"int-cb4c0ebf","kind":"field_change","created_at":"2026-05-01T02:38:05.091612839Z","actor":"Denis Kiselev","issue_id":"EV-t918","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"merged via PR #918"}}
+{"id":"int-ffa8f0b2","kind":"field_change","created_at":"2026-05-01T17:47:23.967998678Z","actor":"Denis Kiselev","issue_id":"EV-m3ta","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"merged"}}
+{"id":"int-b9cb7738","kind":"field_change","created_at":"2026-05-01T18:06:39.17748775Z","actor":"Denis Kiselev","issue_id":"EV-gffv","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"merged"}}
+{"id":"int-983c2fe6","kind":"field_change","created_at":"2026-05-02T02:47:27.113692488Z","actor":"Denis Kiselev","issue_id":"EV-3t8l","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Audit confirmed feature already fully implemented in current master: --fail-fast CLI flag, .evilution.yml key, FailFast validator, ResultNotifier trips at threshold, sequential+parallel strategies short-circuit, summary.truncated? indicator, reporter notices (CLI/HTML/JSON), full spec coverage across notifier/sequential/parallel/runner/parser. CI signal available via summary.truncated? in reports."}}
+{"id":"int-1790a8b7","kind":"field_change","created_at":"2026-05-02T17:53:56.73309749Z","actor":"Denis Kiselev","issue_id":"EV-2gpj","extra":{"field":"status","new_value":"closed","old_value":"in_progress","reason":"Audit confirmed refactor already done: ProcessCleanup.safe_kill / safe_wait helpers extracted to lib/evilution/process_cleanup.rb (lines 8-18), used by baseline.rb (lines 85,93,94) and parallel/work_queue/worker.rb. No Style/RescueModifier disables remain anywhere in lib/. bundle exec rubocop lib/evilution/baseline.rb clean. Existing specs pass."}}

data/.rubocop_todo.yml

@@ -0,0 +1,6 @@
+# This file lists per-file metric exclusions that should eventually be paid down
+# (refactor the file rather than expanding the exclude list). Inherited from .rubocop.yml.
+
+Metrics/AbcSize:
+  Exclude:
+    - "lib/evilution/runner.rb"

data/CHANGELOG.md

@@ -1,5 +1,59 @@
 # Changelog
 
+## [0.28.0] - 2026-05-03
+
+### Added
+
+- **Operator profiles: `default` (current 72-operator set) and `strict` (adds aggressive truthiness mutators)** — pre-merge audits can opt into a more sensitive operator mix. The `strict` profile registers `PredicateToNil`, which replaces every `x.predicate?` call with `nil` to surface tests that only assert truthiness rather than exact return values. Wired through CLI (`--profile=strict`, `--strict` shortcut), `.evilution.yml` (`profile: strict`), and a new `Evilution::Mutator::Registry.for_profile(:default | :strict)` factory. `default` is unchanged, so existing CI runs are not affected (#920, PR #926)
+- **Multi-file batch invocation documented** — `evilution path/a.rb path/b.rb path/c.rb` runs every file in a single Runner invocation so the framework (Rails, Sorbet, etc.) and the `preload` chain load **once** in the parent process. With `--isolation=fork` (default for Rails projects under `auto`), every per-mutation fork branches off the warmed parent — materially faster than `for f in ...; do bundle exec evilution run "$f"; done`. README now has a "5a. Multi-file batch scan" workflow section and an end-to-end runner spec covers two positional file paths; session save/load preserves per-file paths in `survived[].file` (#922, PR #927)
+
+### Fixed
+
+- **`Compare::Normalizer` mis-classified Mutant payload lines whose mutated source started with `--` or `++` as unified-diff headers** — pre-existing bug in `extract_from_mutant_diff` that would, for example, drop a removed line `--flag` (emitted as `---flag` in the diff). The new `DiffExtractor::Mutant` requires a trailing space after `---`/`+++` to match a header, preserving real payload. Equivalent Evilution/Mutant mutations on such lines now hash identically and `compare` no longer reports false additions/removals (#917, PR #934)
+
+### Changed
+
+- **Internal `Evilution::Compare::Fingerprint` SOLID refactor** — module-function form replaced with a class taking injectable `(extractor:, normalizer:)` collaborators and a `#call(diff:, file_path:, line:)` interface. Diff parsing extracted into `Evilution::Compare::DiffExtractor::{Evilution,Mutant}` strategy classes (one per format, common duck-typed interface), enabling open/closed extension for future tools without touching the orchestrator. `Compare::Normalizer` constructs both fingerprints once and reuses them across records (#917, PR #934)
+- **`Evilution::Mutation` migrated to value-object composition** — sources, slice, parse status now Data.define-backed value objects (#822, PR #907)
+- **`Evilution::Result::MutationResult` encapsulates memory and error state in dedicated Data.define value objects** — `MemoryStats` and `ErrorInfo` instead of flat positional fields (#823, PR #907)
+- **`Reporter::Suggestion` registry/templates and the RSpec/Minitest template builders unified** — single `build` entrypoint per format (#824, PR #908; #849, PR #905; #850, PR #904)
+- **`Reporter::HTML` namespace inlined into `report.rb`** — separate `namespace.rb` removed, autoload pattern adopted for sub-templates (#826, PR #909)
+- **`Compare::LineNormalizer` extracted into its own class** — whitespace collapse separated from fingerprint orchestration (#829, PR #832)
+- **`Evilution::Config` attribute assignment migrated to a transformation map** — single source of truth for type coercion across simple attributes (#830)
+- **`Runner` `require` chain consolidated** — sub-component loading now centralized; circular-require pitfalls in `MutationExecutor` resolved with `Module#autoload` for child strategy/neutralizer files (#831)
+- **Process cleanup helpers extracted into `Evilution::ProcessCleanup`** — `safe_kill(sig, pid)` and `safe_wait(pid)` shared by `Baseline`, `Isolation::Fork`, and `WorkQueue::Worker`, replacing scattered inline `rescue` modifiers swallowing `Errno::ESRCH`/`ECHILD` (#838)
+- **`ProgressStreamer` and `Loop` error handling tightened** — generic `StandardError` rescues replaced with specific `Errno::EPIPE`/`Errno::EBADF`/etc.; once-only warning suppression added so a flood of failures cannot drown stderr (#827, #840)
+- **Crash detector predicate methods renamed** — `has_*?` → `*?` per Ruby/RSpec conventions (`have_X` matcher calls `has_X?`; the renamed methods are still picked up by `be_X` matchers used in specs) (#839)
+- **Rubocop hygiene sweep across 6 sites** — `Style/RescueModifier`, `Lint/UnusedMethodArgument`, `Lint/SuppressedException` (3 instances), `Security/Eval`, and `Security/MarshalLoad` (3 instances) inline disable comments removed in favor of either narrowed code, explanatory rescue-body comments, or main-`.rubocop.yml` per-file Excludes documented with the underlying trust boundary (#832, #833, #834, #835, #836, #837)
+
+### Documentation
+
+- **README "Operator Profiles" subsection** — explains the `default` vs `strict` profiles, how to opt in (CLI, config, shortcut), and what `strict` adds today (#920, PR #926)
+- **README "5a. Multi-file batch scan" workflow** — documents Rails-loads-once amortisation and qualifies the speed claim by isolation mode (`fork` vs `in_process`) (#922, PR #927)
+- **`.evilution.yml` template gained a `profile:` block** — generated by `evilution init` (#920, PR #926)
+
+## [0.27.0] - 2026-04-26
+
+### Added
+
+- **`prism` declared as a runtime dependency (`>= 1.5, < 2`)** — Rails 7.1 stacks pin `prism 0.19` (which lacks `IfNode#subsequent`), causing a `NoMethodError` on the first `if` evilution mutated. Bundler now refuses incompatible prism versions at install time instead of crashing at runtime (#876, PR #891)
+- **`--[no-]incremental` CLI flag** — `--no-incremental` overrides `incremental: true` from the config file for one invocation (cold-cache debugging, CI escape hatch). Last flag wins when both forms are given (#878, PR #897)
+- **`--quiet-children` and `--quiet-children-dir DIR` flags** — redirect each forked worker's stdout/stderr to per-pid files under `tmp/evilution_children/<pid>.{out,err}` (configurable). Keeps parent output clean when app initializers (Datadog, Bullet, etc.) emit warnings on every fork. Trade-off: live worker errors only appear in the side files (`tail -f tmp/evilution_children/*.err`) (#880, PR #899)
+- **Preload autodetect chain extended** — `Runner::IsolationResolver` now probes `spec/rails_helper.rb` → `spec/spec_helper.rb` → `test/test_helper.rb` (was: rails_helper + test_helper only). Rails projects that consolidate everything into `spec/spec_helper.rb` no longer need an explicit `preload:` setting. When the chain finds nothing under a Rails project, raises a `ConfigError` listing every path tried and pointing at `--preload` / `--no-preload` (#879, PR #898)
+- **`evilution version` prints the bundled `mcp` gem version** — second line shows `mcp gem X.Y.Z (server compatibility)`. Run inside the same bundle the MCP server uses to confirm what's loaded after a `bundle update` (#883, PR #894)
+- **Public feedback channel exposed across CLI and MCP surfaces** — when a run hits friction (`errored`, `unparseable`, or `unresolved` buckets > 0; baseline failure; MCP error response), evilution surfaces a single GitHub Discussions URL so agents can suggest filing feedback. CLI text reports gain a one-line footer; the CLI error-exit path emits the same line on stderr (both suppressed by `--quiet`, `--format=json`, `--format=html`). MCP `evilution-mutate` responses embed `feedback_url` + `feedback_hint` on friction (and always on error payloads); the `minimal` verbosity contract is preserved (no extra keys). New MCP `evilution-info action=feedback` returns `{ discussion_url, version, guidance_for_agent }` on demand for any feedback intent including missing-capability requests on clean runs. Tool descriptions and the README MCP "Feedback channel" subsection prominently document the **explicit user-consent gate** (agents must never post on the user's behalf without explicit approval) and **privacy expectations** (never include secrets, env vars, project name, file paths, source code, or class/method names from user code — the channel is public). By construction, no run-derived data is embedded in any feedback field — agent + user compose any actual payload themselves (#900, PR #901)
+
+### Fixed
+
+- **`Cache#store` crashed with `TypeError: no implicit conversion of nil into String` when `incremental: true` and `jobs >= 2`** — `Strategy::Parallel#run_batch` called `batch.each(&:strip_sources!)` before `@cache.store`, leaving `mutation.original_source = nil` for `Digest::SHA256.hexdigest`. Reordered so store runs before strip; added a defensive nil-source guard in `Cache#store` mirroring the existing one in `Cache#fetch` (#875, PR #890)
+- **`block_removal` produced unparseable mutations on block-pass arguments (`map!(&:sym)`, `index_by(&:id)`, `flat_map(&block)`)** — operator stripped the `BlockArgumentNode` from inside the call's parens, leaving a dangling open paren. Operator now skips emission when `node.block.is_a?(Prism::BlockArgumentNode)`; explicit `{}` / `do..end` blocks are unaffected (#881, PR #895)
+- **`method_body_replacement` errored at runtime when generating the `super`-replacement on methods whose enclosing class had no parent implementation** — `super`-replacement now only emitted when the original body already calls `super` (`SuperNode` or `ForwardingSuperNode`), using that as a heuristic that a super target exists in this context. Methods without `super` get only the `nil` and `self` replacements (#877, PR #892)
+
+### Documentation
+
+- **README "Installing on Rails 7.1 + Ruby 3.3" section** — covers the `cgi 0.5.0` (Rails-pinned) vs `cgi 0.5.1` (Ruby 3.3 default-gem) Bundler activation conflict, the sidecar `Gemfile.local` workaround (`eval_gemfile("Gemfile")` + add evilution + prism), the `BUNDLE_GEMFILE=Gemfile.local` invocation, and guidance on whether to commit or `.gitignore` the resulting `Gemfile.local.lock` (#882, PR #893)
+- **README MCP "After upgrading the gem: restart the MCP server" subsection** — explains that the MCP server is a long-lived stdio process the agent host spawns; `bundle update evilution` swaps the gem on disk but the running process keeps the old code in memory until restart. Symptom is opaque "Internal error" responses to flags the old build doesn't recognize (#883, PR #894)
+
 ## [0.26.0] - 2026-04-24
 
 ### Removed

data/README.md

@@ -21,6 +21,44 @@ Then: `bundle install`
 
 Or standalone: `gem install evilution`
 
+Requires `prism >= 1.5, < 2`. Older Rails apps (e.g. Rails 7.1 pins `prism 0.19`) must upgrade prism — the gemspec constraint forces bundler to resolve a compatible 1.x version. If your app pins `prism 2.x`, bundler will reject the install until evilution widens its upper bound.
+
+### Installing on Rails 7.1 + Ruby 3.3
+
+Two Bundler conflicts hit fresh installs on this stack:
+
+1. **`cgi` activation conflict.** Rails 7.1's `Gemfile.lock` pins `cgi 0.5.0`. Ruby 3.3.x ships `cgi 0.5.1` as a default gem. Loading evilution via Bundler aborts with:
+
+   ```
+   Gem::LoadError: can't activate cgi-0.5.1, already activated cgi-0.5.0.
+   Make sure all dependencies are added to Gemfile.
+   ```
+
+2. **`prism` pin.** Same lockfile pins `prism 0.19`, which lacks the `IfNode#subsequent` accessor evilution uses (older Prism releases exposed it as `consequent`). Symptom: `NoMethodError: undefined method 'subsequent' for an instance of Prism::IfNode`.
+
+Both resolve cleanly with a sidecar `Gemfile.local` that re-evaluates the project Gemfile and adds evilution + prism on top — no edits to the main `Gemfile.lock`:
+
+```ruby
+# Gemfile.local
+eval_gemfile("Gemfile")
+
+group :test, :development do
+  gem "evilution"
+  gem "prism", "~> 1.5"
+end
+```
+
+Then invoke evilution against that Gemfile:
+
+```sh
+BUNDLE_GEMFILE=Gemfile.local bundle install
+BUNDLE_GEMFILE=Gemfile.local bundle exec evilution run lib/foo.rb
+```
+
+The first command writes a sibling `Gemfile.local.lock`. Decide whether to commit or `.gitignore` it the same way you would for any developer-only Gemfile — typically gitignored when only one or two engineers run mutation testing locally, committed when CI also runs evilution against the sidecar Gemfile.
+
+The evilution gemspec already declares `prism >= 1.5, < 2`, so adding the `gem "prism"` line above is only necessary on stacks that also pin prism in `Gemfile.lock`.
+
 ## Command Reference
 
 ```
@@ -67,17 +105,30 @@ The shorter alias `evil` ships alongside `evilution` and accepts identical argum
 | `-q`, `--quiet`              | Boolean | false        | Suppress output.                                   |
 | `--stdin`                    | Boolean | false        | Read target file paths from stdin (one per line).  |
 | `--integration NAME`         | String  | `rspec`      | Test framework integration: `rspec` or `minitest`.  |
-| `--incremental`              | Boolean | false        | Cache killed/timeout results; skip unchanged mutations on re-runs. |
+| `--[no-]incremental`         | Boolean | false        | Cache killed/timeout results; skip unchanged mutations on re-runs. Pass `--no-incremental` to override `incremental: true` from the config file for one invocation (e.g. cold-cache debugging). Last flag wins when both are given. |
 | `--save-session`             | Boolean | false        | Persist results as timestamped JSON under `.evilution/results/`. |
 | `--no-progress`              | Boolean | _(enabled)_  | Disable the TTY progress bar.                      |
+| `--quiet-children`           | Boolean | false        | Redirect each forked worker's stdout/stderr to per-pid files under `tmp/evilution_children/<pid>.{out,err}` so noisy app initializers (Datadog, Bullet, etc.) don't merge with parent output. Trade-off: live worker errors only appear in the side files, not the terminal — `tail -f tmp/evilution_children/*.err` to watch them. |
+| `--quiet-children-dir DIR`   | String  | `tmp/evilution_children` | Override the directory used by `--quiet-children`.    |
 | `--isolation MODE`           | String  | `auto`       | Isolation strategy: `auto`, `fork`, or `in_process`. `auto` selects `fork` for Rails projects. See [docs/isolation.md](docs/isolation.md). |
-| `--preload FILE`             | String  | _(auto)_     | File to require in parent before forking workers (e.g. `spec/rails_helper.rb`). Auto-detected for Rails. |
+| `--preload FILE`             | String  | _(auto)_     | File to require in parent before forking workers. Auto-detect chain for Rails projects: `spec/rails_helper.rb` → `spec/spec_helper.rb` → `test/test_helper.rb`. Errors with the full chain listed if none exist; pass `--no-preload` to opt out. |
 | `--no-preload`               | Boolean | _(enabled)_  | Disable parent-process preload.                     |
 | `--skip-heredoc-literals`    | Boolean | false        | Skip all string literal mutations inside heredocs.  |
 | `--show-disabled`            | Boolean | false        | Report mutations skipped by `# evilution:disable` comments. |
 | `--fallback-full-suite`      | Boolean | false        | When no matching spec/test resolves for a mutation, run the whole test suite instead of marking it `:unresolved` and skipping. |
 | `--baseline-session PATH`    | String  | _(none)_     | Saved session file for HTML report comparison.     |
 | `-e CODE`, `--eval CODE`     | String  | _(none)_     | Inline Ruby code for `util mutation` command.      |
+| `--profile NAME`             | String  | `default`    | Operator profile: `default` or `strict`. `strict` adds aggressive truthiness mutators (e.g. replaces `x.predicate?` with `nil`) intended for pre-merge audits. |
+| `--strict`                   | Boolean | false        | Shortcut for `--profile=strict`.                    |
+
+### Operator Profiles
+
+Two profiles ship out of the box:
+
+- **`default`** — the 72 stable operators registered in `Mutator::Registry.default`. Suitable for everyday CI runs; balances coverage signal against survivor noise.
+- **`strict`** — adds extra truthiness mutators on top of `default`. Currently `PredicateToNil` (replaces every `x.predicate?` call with `nil` to surface tests that only assert truthiness rather than exact return values). Use for pre-merge audits where you want maximum sensitivity at the cost of more survivors.
+
+Set via `--profile=strict`, the `--strict` shortcut, or `profile: strict` in `.evilution.yml`.
 
 ### Exit Codes
 
@@ -323,7 +374,7 @@ The server exposes the following tools:
 |---|---|
 | `evilution-mutate` | Run mutation testing on target files with structured JSON results |
 | `evilution-session` | Inspect mutation testing history — `action: list` browses saved sessions, `action: show` displays one, `action: diff` compares two (fixed/new/persistent survivors, score delta) |
-| `evilution-info` | Discovery before mutation — `action: subjects` lists mutatable methods with mutation counts, `action: tests` resolves which specs cover given sources, `action: environment` dumps the effective config |
+| `evilution-info` | Discovery before mutation — `action: subjects` lists mutatable methods with mutation counts, `action: tests` resolves which specs cover given sources, `action: environment` dumps the effective config, `action: statuses` returns the mutation-result status glossary, `action: feedback` returns the public Discussions URL plus consent + privacy guidance for posting feedback |
 
 ### Verbosity Control
 
@@ -375,6 +426,20 @@ Pass `skip_config: true` to ignore the project config file. This skips loading `
 
 > **Note**: `.mcp.json` is gitignored by default since it is a local editor/agent configuration file.
 
+### After upgrading the gem: restart the MCP server
+
+The MCP server is a long-lived stdio process spawned by the agent host. `bundle update evilution` swaps the gem on disk but the running process keeps the old code in memory — symptom is opaque "Internal error" responses to flags or shapes the old build doesn't recognize. Restart the server (reload the workspace in Claude Code / Copilot / etc.) so the new gem loads.
+
+`evilution version` prints the gem version and the bundled `mcp` gem version on separate lines — run it in the same bundle the MCP server uses to confirm what's loaded.
+
+### Feedback channel
+
+When evilution causes friction (errors, usage problems, missing capabilities you wish were there), the MCP responses include a `feedback_url` plus `feedback_hint`. The `evilution-info` tool also exposes `action=feedback`, which returns the channel URL and posting guidance on demand.
+
+**Agents must never post on the user's behalf without explicit user permission.** Show the user exactly what you would post, get explicit approval, then post. Never include secrets, environment variables, the project name, file paths, source code, or class/method names from user code — the feedback channel is public.
+
+Discussion URL: <https://github.com/marinazzio/evilution/discussions>
+
 ## Recommended Workflows for AI Agents
 
 ### 1. Full project scan
@@ -430,6 +495,14 @@ bundle exec evilution run lib/specific_file.rb --format json
 
 Use when you know which file was modified and want to verify its test coverage.
 
+### 5a. Multi-file batch scan
+
+```bash
+bundle exec evilution run lib/models/user.rb lib/models/account.rb lib/models/order.rb
+```
+
+Pass multiple file paths on a single invocation to amortise startup cost. The framework (Rails, Sorbet, etc.) and the `preload` chain (`spec/rails_helper.rb` → `spec/spec_helper.rb` → `test/test_helper.rb`) load **once** in the parent process. When `--isolation=fork` is selected (the default `--isolation=auto` resolves to `fork` on Rails projects), every subsequent mutation across all files forks from that warmed parent — materially faster than scripting a `for f in ...; do bundle exec evilution run "$f"; done` loop, which pays the bootstrap per file. With `--isolation=in_process` (default for non-Rails projects under `auto`), there is no per-mutation fork, but the parent-process boot still runs once instead of N times. Per-file paths and line numbers are preserved in the report (`survived[].file`, HTML grouping by source file).
+
 ### 6. Fixing surviving mutants
 
 For each entry in `survived[]`:

data/lib/evilution/baseline.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require_relative "spec_resolver"
+require_relative "process_cleanup"
 
 class Evilution::Baseline
   Result = Struct.new(:failed_spec_files, :duration) do
@@ -72,7 +73,7 @@ class Evilution::Baseline
       Process.wait(pid)
       return false if data.empty?
 
-      result = Marshal.load(data) # rubocop:disable Security/MarshalLoad
+      result = Marshal.load(data)
       result[:passed]
     else
       terminate_child(pid)
@@ -81,7 +82,7 @@ class Evilution::Baseline
   end
 
   def terminate_child(pid)
-    Process.kill("TERM", pid) rescue nil # rubocop:disable Style/RescueModifier
+    Evilution::ProcessCleanup.safe_kill("TERM", pid)
     _, status = Process.waitpid2(pid, Process::WNOHANG)
     return if status
 
@@ -89,8 +90,8 @@ class Evilution::Baseline
     _, status = Process.waitpid2(pid, Process::WNOHANG)
     return if status
 
-    Process.kill("KILL", pid) rescue nil # rubocop:disable Style/RescueModifier
-    Process.wait(pid) rescue nil # rubocop:disable Style/RescueModifier
+    Evilution::ProcessCleanup.safe_kill("KILL", pid)
+    Evilution::ProcessCleanup.safe_wait(pid)
   end
 
   private

data/lib/evilution/cache.rb

@@ -27,6 +27,8 @@ class Evilution::Cache
   end
 
   def store(mutation, result_data)
+    return if mutation.original_source.nil?
+
     file_key = file_key(mutation)
     entry_key = entry_key(mutation)
     data = read_file(file_key) || {}

data/lib/evilution/child_output.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+require_relative "../evilution"
+
+module Evilution::ChildOutput
+  module_function
+
+  class << self
+    attr_accessor :log_dir
+  end
+
+  # Per-run truncation happens once in the parent (Runner#configure_child_output);
+  # within a run, multiple forks reusing the same PID (pool worker recycle, per-mutation
+  # forks) append so cross-fork output isn't lost.
+  def redirect!
+    return unless log_dir
+
+    pid = Process.pid
+    $stdout.reopen(File.join(log_dir, "#{pid}.out"), "a")
+    $stderr.reopen(File.join(log_dir, "#{pid}.err"), "a")
+    $stdout.sync = true
+    $stderr.sync = true
+  end
+end

data/lib/evilution/cli/commands/run.rb

@@ -10,6 +10,7 @@ require_relative "../../runner"
 require_relative "../../hooks"
 require_relative "../../hooks/registry"
 require_relative "../../hooks/loader"
+require_relative "../../feedback/messages"
 
 class Evilution::CLI::Commands::Run < Evilution::CLI::Command
   def call
@@ -34,10 +35,18 @@ class Evilution::CLI::Commands::Run < Evilution::CLI::Command
       @stdout.puts(JSON.generate(error_payload(error)))
       Evilution::CLI::Result.new(exit_code: 2, error: error, error_rendered: true)
     else
+      @stderr.puts(Evilution::Feedback::Messages.cli_footer) unless quiet?(config, file_options)
       Evilution::CLI::Result.new(exit_code: 2, error: error)
     end
   end
 
+  def quiet?(config, file_options)
+    return config.quiet unless config.nil?
+    return true if @options[:quiet]
+
+    file_options && file_options[:quiet]
+  end
+
   def build_hooks(config)
     return nil if config.hooks.empty?
 

data/lib/evilution/cli/commands/version.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "mcp"
 require_relative "../commands"
 require_relative "../command"
 require_relative "../dispatcher"
@@ -10,6 +11,7 @@ class Evilution::CLI::Commands::Version < Evilution::CLI::Command
 
   def perform
     @stdout.puts(Evilution::VERSION)
+    @stdout.puts("mcp gem #{::MCP::VERSION} (server compatibility)")
     0
   end
 end

data/lib/evilution/cli/parser/options_builder.rb

@@ -21,6 +21,7 @@ class Evilution::CLI::Parser::OptionsBuilder
       add_core_options(opts)
       add_filter_options(opts)
       add_flag_options(opts)
+      add_profile_options(opts)
       add_extra_flag_options(opts)
       add_session_options(opts)
       add_compare_options(opts)
@@ -70,15 +71,35 @@ class Evilution::CLI::Parser::OptionsBuilder
     opts.on("--fail-fast", "Stop after N surviving mutants " \
                            "(default: disabled; if provided without N, uses 1; use --fail-fast=N)") { @options[:fail_fast] ||= 1 }
     opts.on("--no-baseline", "Skip baseline test suite check") { @options[:baseline] = false }
-    opts.on("--incremental", "Cache killed/timeout results; skip re-running them on unchanged files") { @options[:incremental] = true }
+    opts.on("--[no-]incremental",
+            "Cache killed/timeout results; skip re-running them on unchanged files. " \
+            "Use --no-incremental to override `incremental: true` from the config file for one run.") do |v|
+      @options[:incremental] = v
+    end
     opts.on("--integration NAME", "Test integration: rspec, minitest (default: rspec)") { |i| @options[:integration] = i }
     opts.on("--isolation STRATEGY", "Isolation: auto, fork, in_process (default: auto)") { |s| @options[:isolation] = s }
     opts.on("--preload FILE", "Preload FILE in the parent process before forking " \
-                              "(default: auto-detect spec/rails_helper.rb for Rails projects)") { |f| @options[:preload] = f }
+                              "(default: auto-detect spec/rails_helper.rb -> spec/spec_helper.rb -> " \
+                              "test/test_helper.rb for Rails projects)") { |f| @options[:preload] = f }
     opts.on("--no-preload", "Disable parent-process preload even for Rails projects") { @options[:preload] = false }
     opts.on("--stdin", "Read target file paths from stdin (one per line)") { @options[:stdin] = true }
     opts.on("--suggest-tests", "Generate concrete test code in suggestions (RSpec or Minitest)") { @options[:suggest_tests] = true }
     opts.on("--no-progress", "Disable progress bar") { @options[:progress] = false }
+    opts.on("--quiet-children",
+            "Redirect each child process's stdout/stderr to per-pid files under " \
+            "tmp/evilution_children (or --quiet-children-dir DIR), keeping parent output clean.") do
+      @options[:quiet_children] = true
+    end
+    opts.on("--quiet-children-dir DIR",
+            "Directory for --quiet-children per-pid log files (default: tmp/evilution_children)") do |d|
+      @options[:quiet_children_dir] = d
+    end
+  end
+
+  def add_profile_options(opts)
+    opts.on("--profile NAME", "Operator profile: default, strict (default: default). " \
+                              "strict adds aggressive truthiness mutators for pre-merge audits.") { |p| @options[:profile] = p }
+    opts.on("--strict", "Shortcut for --profile=strict") { @options[:profile] = "strict" }
   end
 
   def add_extra_flag_options(opts)

data/lib/evilution/compare/diff_extractor/evilution.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require_relative "../diff_extractor"
+
+# Extracts {minus:, plus:} payload arrays from Evilution-format diffs.
+# Evilution diffs use "- " / "+ " line prefixes (note the trailing space) and
+# do not carry unified-diff headers or hunk markers.
+class Evilution::Compare::DiffExtractor::Evilution
+  def call(diff)
+    minus = []
+    plus = []
+    diff.to_s.each_line do |line|
+      line = line.chomp
+      if line.start_with?("- ")
+        minus << line[2..]
+      elsif line.start_with?("+ ")
+        plus << line[2..]
+      end
+    end
+    { minus: minus, plus: plus }
+  end
+end

data/lib/evilution/compare/diff_extractor/mutant.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+require_relative "../diff_extractor"
+
+# Extracts {minus:, plus:} payload arrays from Mutant unified-diff format.
+# Skips the "--- <name>", "+++ <name>", and "@@ ... @@" header lines and
+# returns each remaining payload line with its single leading "-" or "+"
+# marker stripped.
+#
+# Header detection requires a trailing space after "---"/"+++" so that a
+# payload line whose mutated source starts with "--" (emitted as "---var")
+# or "++" (emitted as "+++var") is preserved rather than misclassified as
+# a header.
+class Evilution::Compare::DiffExtractor::Mutant
+  def call(diff)
+    minus = []
+    plus = []
+    diff.to_s.each_line do |line|
+      line = line.chomp
+      next if line.start_with?("--- ", "+++ ", "@@")
+
+      if line.start_with?("-")
+        minus << line[1..]
+      elsif line.start_with?("+")
+        plus << line[1..]
+      end
+    end
+    { minus: minus, plus: plus }
+  end
+end

data/lib/evilution/compare/diff_extractor.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+require_relative "../compare"
+
+module Evilution::Compare::DiffExtractor
+end

data/lib/evilution/compare/fingerprint.rb

@@ -3,80 +3,23 @@
 require "digest"
 require_relative "../compare"
 
-module Evilution::Compare::Fingerprint
-  module_function
-
-  def extract_from_evilution_diff(diff)
-    minus = []
-    plus = []
-    diff.to_s.each_line do |line|
-      line = line.chomp
-      if line.start_with?("- ")
-        minus << line[2..]
-      elsif line.start_with?("+ ")
-        plus << line[2..]
-      end
-    end
-    { minus: minus, plus: plus }
-  end
-
-  def extract_from_mutant_diff(diff)
-    minus = []
-    plus = []
-    diff.to_s.each_line do |line|
-      line = line.chomp
-      next if line.start_with?("---", "+++", "@@")
-
-      if line.start_with?("-")
-        minus << line[1..]
-      elsif line.start_with?("+")
-        plus << line[1..]
-      end
-    end
-    { minus: minus, plus: plus }
-  end
-
-  # v1 limitation: only " and ' literals are preserved. Regex literals (/.../),
-  # heredocs, %w[], %q{} forms are treated as ordinary code — whitespace runs
-  # inside them collapse. A mutation touching whitespace inside a regex may
-  # false-match across tools.
-  # rubocop:disable Metrics/PerceivedComplexity, Style/MultipleComparison
-  def normalize_line(line)
-    out = +""
-    i = 0
-    in_literal = nil
-    last_was_space = false
-    chars = line.chars
-    while i < chars.length
-      ch = chars[i]
-      if in_literal
-        out << ch
-        if ch == "\\" && i + 1 < chars.length
-          out << chars[i + 1]
-          i += 2
-          next
-        end
-        in_literal = nil if ch == in_literal
-      elsif ch == '"' || ch == "'"
-        in_literal = ch
-        out << ch
-        last_was_space = false
-      elsif ch == " " || ch == "\t"
-        out << " " unless last_was_space || out.empty?
-        last_was_space = true
-      else
-        out << ch
-        last_was_space = false
-      end
-      i += 1
-    end
-    out.rstrip
+# Composes a stable SHA256 fingerprint from a mutation diff for cross-tool
+# matching (Evilution vs Mutant). Orchestrates two collaborators along
+# distinct change axes:
+#
+#   - extractor: parses a tool-specific diff format into {minus:, plus:}
+#   - normalizer: collapses whitespace per line so cosmetic differences
+#                 don't perturb the hash
+class Evilution::Compare::Fingerprint
+  def initialize(extractor:, normalizer:)
+    @extractor = extractor
+    @normalizer = normalizer
   end
-  # rubocop:enable Metrics/PerceivedComplexity, Style/MultipleComparison
 
-  def compute(file_path:, line:, body:)
-    minus = body[:minus].map { |l| normalize_line(l) }
-    plus  = body[:plus].map  { |l| normalize_line(l) }
+  def call(diff:, file_path:, line:)
+    body = @extractor.call(diff)
+    minus = body[:minus].map { |l| @normalizer.call(l) }
+    plus  = body[:plus].map  { |l| @normalizer.call(l) }
     payload = [file_path, line.to_s, minus.join("\n"), plus.join("\n")].join("\x00")
     Digest::SHA256.hexdigest(payload)
   end