simplecov-mcp 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 06e96d273c6549d1f0539baa88ec863009f263e6d9f214c66157ac5e708b51a5
-  data.tar.gz: fc1503a39a31d19076f4e7827aad1d1996fd4760b381aafdf7e4b13f8891fb0c
+  metadata.gz: 41a4d8cde53da2d974436b735011f381362fa7e17e53d3ff587b41a9e66c285a
+  data.tar.gz: d22bf6d5265373c0430499c7fbac6ed249d1786e63bac0b7562964b9d9ae8958
 SHA512:
-  metadata.gz: 41125a954fca4422327a821f18ad55b8de0d473ea3c3d67bc0bb9eb6105d81aa09efcab262a09f0cedeb809adb73a0e974b36d0dcf00ceff55af8e639a573afe
-  data.tar.gz: 7ab8e6375e1645557a8dd30c72be8e37e9d4fd97de59e330db24c0f6ec92339db5488b43c5cc9dbc975edbf5184fdb3179d5425cb48d5791d2fb0650659ab439
+  metadata.gz: 63aceca2e283855ee5ab6aaf16007b70baa4042e5085e976a9eecdc43f1054925d7b7749bc5ac9f4f64bed12392672f3c3f9c6006b3c844bbb5c0bcd9efa4fda
+  data.tar.gz: 599201a717dc571240c0f10b0492e4938428f1b898745dafbfbb6988bbbfb4f5505c7632e91a4d7afe16bd4b1f44876280f635c3323cf23a847441ae3c7049e1

data/lib/simple_cov_mcp/tools/all_files_coverage_tool.rb

@@ -53,7 +53,9 @@ module SimpleCovMcp
           stale_count = files.count { |f| f['stale'] }
           ok_count = total - stale_count
           payload = { files: files, counts: { total: total, ok: ok_count, stale: stale_count } }
-          ::MCP::Tool::Response.new([{ type: 'text', text: JSON.pretty_generate(payload) }])
+          ::MCP::Tool::Response.new([
+            { 'type' => 'text', 'text' => JSON.generate(payload) }
+          ])
         rescue => e
           handle_mcp_error(e, 'AllFilesCoverageTool')
         end

data/lib/simple_cov_mcp/tools/help_tool.rb

@@ -88,7 +88,10 @@ module SimpleCovMcp
           entries = TOOL_GUIDE.map { |guide| format_entry(guide) }
           entries = filter_entries(entries, query) if query && !query.strip.empty?
 
-          ::MCP::Tool::Response.new([{ type: 'text', text: JSON.pretty_generate({ query: query, tools: entries }) }])
+          data = { query: query, tools: entries }
+          ::MCP::Tool::Response.new([
+            { 'type' => 'text', 'text' => JSON.generate(data) }
+          ])
         rescue => e
           handle_mcp_error(e, 'HelpTool')
         end

data/lib/simple_cov_mcp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SimpleCovMcp
-  VERSION = '0.2.0'
+  VERSION = '0.2.1'
 end

data/spec/all_files_coverage_tool_spec.rb

@@ -28,11 +28,11 @@ RSpec.describe SimpleCovMcp::Tools::AllFilesCoverageTool do
 
     response = described_class.call(root: root, server_context: server_context)
     entry = response.payload.first
-    json = entry[:json] || entry['json']
+    json = JSON.parse(entry['text'])
 
-    expect(json).to have_key(:files).or have_key('files')
-    files = json[:files] || json['files']
-    counts = json[:counts] || json['counts']
+    expect(json).to have_key('files')
+    files = json['files']
+    counts = json['counts']
 
     expect(files.length).to eq(2)
     expect(counts).to include('total' => 2).or include(total: 2)

data/spec/help_tool_spec.rb

@@ -10,9 +10,9 @@ RSpec.describe SimpleCovMcp::Tools::HelpTool do
     response_class = Class.new do
       attr_reader :payload, :meta
 
-      def initialize(payload, meta: nil)
+      def initialize(payload)
         @payload = payload
-        @meta = meta
+        @meta = nil
       end
     end
     stub_const('MCP::Tool::Response', response_class)
@@ -20,28 +20,28 @@ RSpec.describe SimpleCovMcp::Tools::HelpTool do
 
   it 'returns guidance for each registered tool' do
     response = described_class.call(server_context: server_context)
-    expect(response.meta[:mimeType]).to eq('application/json')
+    expect(response.meta).to be_nil
 
     payload = response.payload.first
-    expect(payload[:type]).to eq('json')
+    expect(payload['type']).to eq('text')
 
-    data = payload[:json]
-    tool_names = data[:tools].map { |entry| entry['tool'] }
+    data = JSON.parse(payload['text'])
+    tool_names = data['tools'].map { |entry| entry['tool'] }
 
     expect(tool_names).to include('coverage_summary_tool', 'uncovered_lines_tool', 'all_files_coverage_tool', 'coverage_table_tool', 'version_tool')
-    expect(data[:tools]).to all(include('use_when', 'avoid_when', 'inputs', 'example'))
+    expect(data['tools']).to all(include('use_when', 'avoid_when', 'inputs', 'example'))
   end
 
   it 'filters entries when a query is provided' do
     response = described_class.call(query: 'uncovered', server_context: server_context)
     payload = response.payload.first
-    data = payload[:json]
+    data = JSON.parse(payload['text'])
 
-    expect(data[:tools]).not_to be_empty
-    expect(data[:tools]).to all(satisfy do |entry|
+    expect(data['tools']).not_to be_empty
+    expect(data['tools']).to all(satisfy do |entry|
       combined = [entry['tool'], entry['label'], entry['use_when'], entry['avoid_when']].compact.join(' ').downcase
       combined.include?('uncovered')
     end)
-    expect(data[:tools].map { |entry| entry['tool'] }).to include('uncovered_lines_tool')
+    expect(data['tools'].map { |entry| entry['tool'] }).to include('uncovered_lines_tool')
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: simplecov-mcp
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Keith R. Bennett
@@ -13,16 +13,16 @@ dependencies:
   name: mcp
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: '0.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: '0.3'
 - !ruby/object:Gem::Dependency
   name: awesome_print
   requirement: !ruby/object:Gem::Requirement
@@ -106,7 +106,6 @@ files:
 - spec/all_files_coverage_tool_spec.rb
 - spec/base_tool_spec.rb
 - spec/cli_error_spec.rb
-- spec/cli_json_source_spec.rb
 - spec/cli_source_spec.rb
 - spec/cli_spec.rb
 - spec/cli_table_spec.rb

data/spec/cli_json_source_spec.rb

@@ -1,92 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-RSpec.describe SimpleCovMcp::CoverageCLI do
-  let(:root) { (FIXTURES / 'project1').to_s }
-
-  def run_cli_json(*argv)
-    cli = described_class.new
-    out = nil
-    silence_output do |stdout, _stderr|
-      cli.run(argv.flatten)
-      out = stdout.string
-    end
-    JSON.parse(out)
-  end
-
-  it 'includes source rows in JSON for summary --source=full' do
-    data = run_cli_json('summary', 'lib/foo.rb', '--root', root, '--resultset', 'coverage', '--json', '--source=full')
-    expect(data['file']).to eq('lib/foo.rb')
-    expect(data['source']).to be_an(Array)
-    expect(data['source'].first).to include('line', 'code', 'hits', 'covered')
-  end
-
-  it 'includes source rows in JSON for uncovered --source=uncovered' do
-    data = run_cli_json('uncovered', 'lib/foo.rb', '--root', root, '--resultset', 'coverage', '--json', '--source=uncovered', '--source-context', '1')
-    expect(data['file']).to eq('lib/foo.rb')
-    expect(data['source']).to be_an(Array)
-    # Only a subset of lines around uncovered should appear
-    lines = data['source'].map { |h| h['line'] }
-    expect(lines).to include(2) # the uncovered line
-  end
-
-  it 'includes source rows in JSON for detailed --source=full' do
-    data = run_cli_json('detailed', 'lib/foo.rb', '--root', root, '--resultset', 'coverage', '--json', '--source=full')
-    expect(data['file']).to eq('lib/foo.rb')
-    expect(data['lines']).to be_an(Array)
-    expect(data['source']).to be_an(Array)
-  end
-
-  it 'renders uncovered source with various context sizes without error' do
-    [0, -5, 50].each do |ctx|
-      out, err, status = begin
-        cli = described_class.new
-        s = nil
-        o = e = nil
-        silence_output do |stdout, stderr|
-          begin
-            cli.run(['uncovered', 'lib/foo.rb', '--root', root, '--resultset', 'coverage', '--source=uncovered', '--source-context', ctx.to_s, '--no-color'])
-            s = 0
-          rescue SystemExit => ex
-            s = ex.status
-          end
-          o = stdout.string
-          e = stderr.string
-        end
-        [o, e, s]
-      end
-      expect(status).to eq(0)
-      expect(err).to eq("")
-      expect(out).to include('File: lib/foo.rb')
-      expect(out).to include('Uncovered lines: 2')
-    end
-  end
-
-  it 'respects --color and --no-color for source rendering' do
-    # Force color on
-    out_color = begin
-      cli = described_class.new
-      silence_output do |stdout, _stderr|
-        cli.run(['summary', 'lib/foo.rb', '--root', root, '--resultset', 'coverage', '--source', '--color'])
-        stdout.string
-      end
-    end
-    # If source table is rendered, it should contain ANSI escapes when color is on
-    if out_color.include?('Line') && out_color.include?('|')
-      expect(out_color).to match(/\e\[\d+m/)
-    else
-      expect(out_color).to include('[source not available]')
-    end
-
-    # Force color off: expect no ANSI sequences
-    out_nocolor = begin
-      cli = described_class.new
-      silence_output do |stdout, _stderr|
-        cli.run(['summary', 'lib/foo.rb', '--root', root, '--resultset', 'coverage', '--source', '--no-color'])
-        stdout.string
-      end
-    end
-    expect(out_nocolor).not_to match(/\e\[\d+m/)
-  end
-end