quality_gate 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b549f0b3f39f5a51bc4c1321a8b0771ae355f4ad1a8da733e6f5cf9c8209a54e
-  data.tar.gz: a0adfdf9647271909cac928c6056ee59114a5eefee5c8173c4ecf8816bbbb5ba
+  metadata.gz: 5c2f18d9b5a74e7d19838e915bb21ab23fe18cb1d2179e0279ec848dec71e9ee
+  data.tar.gz: 8f6d82509a688cf01fb92186caca89a4c330470e05b9a5fb3200fa4c28339b55
 SHA512:
-  metadata.gz: 6f58b91553cbf70f2224831b4c21f8b2aa279a3e76dba5638445c2d11982363d4820a8d70b54b99fcc83d5b97e1d898292e472c0427b7f2e34c764b2979ab796
-  data.tar.gz: eb7bbe5a088b832b60f28751f6dfd1009a4d6ed1ce291e42c27dcd2abbbb9ccc1310e4d45992bff589723aaf1dd37c47ff9763f1edb5599aace9fbc730621d0e
+  metadata.gz: '0938c529aa703c3e0cdeaad45996e715a58b0554f4b87483d1741816056a10012567ef00410d287d15fe8d39996bf99a7cb750101275ec9b7423a044a2bf5e3d'
+  data.tar.gz: 5d822362f3278a6d219401b65379b1d4217e28db160991e7014c25e6a45403c5bd2c1f62df5766d50105d5ae40e80c50c02b5e54619ed0169dc0db723c271d77

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## [0.1.1] - 2026-09-10
+
+- Text gate output now lists each tool that ran with its status, scope, and duration, so a clean run is distinguishable from a run where nothing executed.
+- The installer no longer injects a coverage block into a test helper that already calls `SimpleCov.start`; it reports the skip and names what to confirm.
+
 ## [0.1.0] - 2026-09-08
 
 - Layered fast, verify, and audit gates with structured findings for Ruby projects.

data/README.md

@@ -45,19 +45,31 @@ The built-in fast path is meant for changed files:
 bundle exec quality_gate fast --files app/models/user.rb
 ```
 
-`verify` records coverage while it runs the full test suite, then checks changed code with Undercover. A clean `verify` or `audit` run prints this clean summary:
+`verify` records coverage while it runs the full test suite, then checks changed code with Undercover. A clean `verify` or `audit` run prints this clean summary: one line per tool that ran, naming its status, scope, and duration, then the tally:
 
 ```text
+reek         clean        project                 312ms
+test_suite   clean        test_suite           189137ms
+undercover   clean        git_diff                619ms
 0 findings, 0 tool failures
 ```
 
+(Durations above are illustrative, not measured figures.) These tool lines make a clean run visible: you see every tool that ran, not just an absence of findings.
+
 Use `--files` to select paths for RuboCop and Reek; the first path follows the option and further paths follow it as separate arguments:
 
 ```sh
 bundle exec quality_gate fast --files lib/quality_gate.rb test/test_quality_gate.rb
 ```
 
-Paths must exist, including paths from the configuration file. Any missing path returns exit `2` before tools run. A bare `fast` scans the project; it does not discover changed files automatically.
+Paths must exist, including paths from the configuration file. Any missing path returns exit `2` before tools run. A bare `fast` scans the project; it does not discover changed files automatically. RuboCop is the only default fast adapter, so a clean bare `fast` run prints one tool line then the tally:
+
+```text
+rubocop      clean        project                  842ms
+0 findings, 0 tool failures
+```
+
+(842ms above is illustrative, not a measured figure.)
 
 | Adapter | Scope with `--files` |
 | --- | --- |
@@ -82,13 +94,23 @@ The repository config must also use exactly `text` or `json` for `format`; any o
 
 ## Output contract
 
-Gate text output includes tool, severity, location when available, rule, and message:
+Gate text output prints, in order: one line per tool that ran, then findings, then the tally.
+
+When at least one tool ran, each gets a line naming the tool, its status, what it inspected, and how long it took:
+
+```text
+<tool> <status> <scope> <duration>ms
+```
+
+`status` is `clean`, `findings`, `tool_failure`, or `skipped`; `scope` describes what the tool inspected, such as `project`, `selected_files`, `git_diff`, `test_suite`, `coverage_summary`, or `lockfile`. These lines distinguish a clean run, which always names its tools, from a run where nothing executed. An empty adapter list runs no tools and prints none of these lines.
+
+Findings come next, with tool, severity, location when available, rule, and message:
 
 ```text
 <tool> <severity> <file>:<line> <rule> <message>
 ```
 
-Multiline messages retain indented detail lines. Findings without a source location omit it rather than printing `:0`. The final gate output line is the summary:
+Multiline messages retain indented detail lines. Findings without a source location omit it rather than printing `:0`. The final gate output line is unchanged, the summary:
 
 ```text
 <n> findings, <m> tool failures

data/lib/quality_gate/installation.rb

@@ -22,6 +22,7 @@ module QualityGate
     ].freeze
     MARKER_START = "# quality_gate coverage — start"
     MARKER_END = "# quality_gate coverage — end"
+    EXISTING_SIMPLECOV_START = /SimpleCov\s*\.\s*start\b/
     AGENT_CONTRACT_START = "<!-- quality_gate agent contract — start -->"
     AGENT_CONTRACT_END = "<!-- quality_gate agent contract — end -->"
     AGENT_TEMPLATE_PATHS = {
@@ -424,6 +425,7 @@ module QualityGate
     def install_coverage(path, relative_path)
       original = File.binread(path)
       return record(:unchanged, relative_path) if original.include?(MARKER_START.b)
+      return skip_existing_simplecov(relative_path) if EXISTING_SIMPLECOV_START.match?(original)
       return record_pretend_skip(relative_path) if options[:pretend]
 
       content = content_with_coverage(original)
@@ -659,6 +661,12 @@ module QualityGate
       record(:skipped, "test/test_helper.rb (missing; Minitest coverage wiring skipped)")
     end
 
+    def skip_existing_simplecov(relative_path)
+      say "Warning: coverage wiring skipped; #{relative_path} already calls SimpleCov.start. " \
+          "Confirm it enables branch coverage and loads the Undercover formatter."
+      record(:skipped, "#{relative_path} (already starts SimpleCov; coverage wiring skipped)")
+    end
+
     def record_pretend_skip(relative_path)
       record(:skipped, "#{relative_path} (pretend)")
     end

data/lib/quality_gate/reporters/text.rb

@@ -9,6 +9,8 @@ module QualityGate
       end
 
       def call(result)
+        print_checks(result.checks)
+
         result.findings.each do |finding|
           io.puts(render_finding(finding))
         end
@@ -20,6 +22,26 @@ module QualityGate
 
       attr_reader :io
 
+      def print_checks(checks)
+        return if checks.empty?
+
+        checks.each { |check| io.puts(render_check(check)) }
+      end
+
+      def render_check(check)
+        format(
+          "%<tool>-12s %<status>-12s %<scope>-18s %<duration>8dms",
+          tool: FieldSanitizer.for_text(check[:tool]),
+          status: FieldSanitizer.for_text(check[:status]),
+          scope: FieldSanitizer.for_text(check[:scope]),
+          duration: duration_ms_for(check)
+        )
+      end
+
+      def duration_ms_for(check)
+        Integer(check[:duration_ms], exception: false) || 0
+      end
+
       def render_finding(finding)
         lines = FieldSanitizer.for_text_message(finding.message).split("\n", -1)
         ["#{finding_prefix(finding)} #{lines.first}", *lines.drop(1).map { "  #{_1}" }].join("\n")

data/lib/quality_gate/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module QualityGate
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: quality_gate
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Lucian Ghinda