simplecov-mcp 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 41a4d8cde53da2d974436b735011f381362fa7e17e53d3ff587b41a9e66c285a
-  data.tar.gz: d22bf6d5265373c0430499c7fbac6ed249d1786e63bac0b7562964b9d9ae8958
+  metadata.gz: '08e9a29ca4d2e2daafd4620690021de136eaa2cfc2ba652340f73d8b69547d8d'
+  data.tar.gz: 784e8ef1f21e6ed4e37783a840935e909d8a6d1f6ae4c941da34d0e6635f9f95
 SHA512:
-  metadata.gz: 63aceca2e283855ee5ab6aaf16007b70baa4042e5085e976a9eecdc43f1054925d7b7749bc5ac9f4f64bed12392672f3c3f9c6006b3c844bbb5c0bcd9efa4fda
-  data.tar.gz: 599201a717dc571240c0f10b0492e4938428f1b898745dafbfbb6988bbbfb4f5505c7632e91a4d7afe16bd4b1f44876280f635c3323cf23a847441ae3c7049e1
+  metadata.gz: 52e2a95fda335f611f38cd0b97164a62e381532ceb71b330c32023be10eeb8d97e999bdc32f0dafac168f0470cac481151e78f223623e90a377fe79ef807705b
+  data.tar.gz: 42ce8d9f01b069f2e3dd0d0e817cd135298fa14d06553ef5b0771d37baf9f5bd4c194d9a9296a823966da21ec505e69b41fd489dfcfa1b0e0b74bb63949b7a21

data/README.md

@@ -92,6 +92,12 @@ echo '{"jsonrpc":"2.0","id":2,"method":"tools/call","params":{"name":"all_files_
 
 Tip: In an MCP-capable editor/agent, configure `simplecov-mcp` as a stdio server and call the same tool names with the `arguments` shown above.
 
+Response content types (MCP):
+
+- JSON data returns as a single `type: "resource"` item with `resource.mimeType: "application/json"` and the JSON string in `resource.text` (e.g., `name: "coverage_summary.json"`).
+- Human-readable strings (e.g., the table and version) return as `type: "text"`.
+- Errors return as `type: "text"` with a friendly message.
+
 Resultset resolution:
 
 - If `resultset:` is provided, it may be:
@@ -120,7 +126,7 @@ Using simplecov-mcp, show me a table of all files and their coverages.
 
 ----
 
-Using simplecov-mcp, find the uncovered code lines and report to me:
+Using simplecov-mcp, find the uncovered code lines and report to me, in a markdown file:
 
 * the most important coverage omissions to address
 * the simplest coverage omissions to address
@@ -348,6 +354,11 @@ Available tools:
   - Returns `{ files: [{"file","covered","total","percentage","stale"}, ...] }` where `stale` is a boolean.
 - `version_tool()` — returns version information
 
+Response shape and content types:
+
+- JSON tools above return a single content item `{ "type": "resource", "resource": { "mimeType": "application/json", "text": "{...}", "name": "<tool>.json" } }`.
+- `coverage_table_tool` and `version_tool` return `{ "type": "text", "text": "..." }`.
+
 Notes:
 
 - `resultset` lets clients pass a nonstandard path to `.resultset.json` directly (absolute or relative to `root`). It may be:

data/lib/simple_cov_mcp/base_tool.rb

@@ -44,6 +44,22 @@ module SimpleCovMcp
         ::MCP::Tool::Response.new([{ type: 'text', text: "Error: #{normalized.user_friendly_message}" }])
       end
 
+      # Respond with JSON as a resource to avoid clients mutating content types.
+      # The resource embeds the JSON string with a clear MIME type.
+      def self.respond_json(payload, name: 'data.json', pretty: false)
+        json = pretty ? JSON.pretty_generate(payload) : JSON.generate(payload)
+        ::MCP::Tool::Response.new([
+          {
+            'type' => 'resource',
+            'resource' => {
+              'mimeType' => 'application/json',
+              'text' => json,
+              'name' => name
+            }
+          }
+        ])
+      end
+
       private
 
       def self.log_mcp_error(error, tool_name)

data/lib/simple_cov_mcp/tools/all_files_coverage_tool.rb

@@ -53,9 +53,7 @@ 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.generate(payload) }
-          ])
+          respond_json(payload, name: 'all_files_coverage.json')
         rescue => e
           handle_mcp_error(e, 'AllFilesCoverageTool')
         end

data/lib/simple_cov_mcp/tools/coverage_detailed_tool.rb

@@ -19,7 +19,7 @@ module SimpleCovMcp
           mode = stale
           model = CoverageModel.new(root: root, resultset: resultset, staleness: mode)
           data = model.detailed_for(path)
-          ::MCP::Tool::Response.new([{ type: 'text', text: JSON.pretty_generate(data) }])
+          respond_json(data, name: 'coverage_detailed.json', pretty: true)
         rescue => e
           handle_mcp_error(e, 'CoverageDetailedTool')
         end

data/lib/simple_cov_mcp/tools/coverage_raw_tool.rb

@@ -19,7 +19,7 @@ module SimpleCovMcp
           mode = stale
           model = CoverageModel.new(root: root, resultset: resultset, staleness: mode)
           data = model.raw_for(path)
-          ::MCP::Tool::Response.new([{ type: 'text', text: JSON.pretty_generate(data) }])
+          respond_json(data, name: 'coverage_raw.json', pretty: true)
         rescue => e
           handle_mcp_error(e, 'CoverageRawTool')
         end

data/lib/simple_cov_mcp/tools/coverage_summary_tool.rb

@@ -19,7 +19,7 @@ module SimpleCovMcp
           mode = stale
           model = CoverageModel.new(root: root, resultset: resultset, staleness: mode)
           data = model.summary_for(path)
-          ::MCP::Tool::Response.new([{ type: 'text', text: JSON.pretty_generate(data) }])
+          respond_json(data, name: 'coverage_summary.json', pretty: true)
         rescue => e
           handle_mcp_error(e, 'CoverageSummaryTool')
         end

data/lib/simple_cov_mcp/tools/help_tool.rb

@@ -89,9 +89,7 @@ module SimpleCovMcp
           entries = filter_entries(entries, query) if query && !query.strip.empty?
 
           data = { query: query, tools: entries }
-          ::MCP::Tool::Response.new([
-            { 'type' => 'text', 'text' => JSON.generate(data) }
-          ])
+          respond_json(data, name: 'tools_help.json')
         rescue => e
           handle_mcp_error(e, 'HelpTool')
         end

data/lib/simple_cov_mcp/tools/uncovered_lines_tool.rb

@@ -19,7 +19,7 @@ module SimpleCovMcp
           mode = stale
           model = CoverageModel.new(root: root, resultset: resultset, staleness: mode)
           data = model.uncovered_for(path)
-          ::MCP::Tool::Response.new([{ type: 'text', text: JSON.pretty_generate(data) }])
+          respond_json(data, name: 'uncovered_lines.json', pretty: true)
         rescue => e
           handle_mcp_error(e, 'UncoveredLinesTool')
         end

data/lib/simple_cov_mcp/version.rb

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

data/spec/all_files_coverage_tool_spec.rb

@@ -28,7 +28,9 @@ RSpec.describe SimpleCovMcp::Tools::AllFilesCoverageTool do
 
     response = described_class.call(root: root, server_context: server_context)
     entry = response.payload.first
-    json = JSON.parse(entry['text'])
+    expect(entry['type']).to eq('resource')
+    expect(entry['resource']).to include('mimeType' => 'application/json')
+    json = JSON.parse(entry['resource']['text'])
 
     expect(json).to have_key('files')
     files = json['files']

data/spec/coverage_detailed_tool_spec.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'simple_cov_mcp/tools/coverage_detailed_tool'
+
+RSpec.describe SimpleCovMcp::Tools::CoverageDetailedTool do
+  let(:server_context) { instance_double('ServerContext').as_null_object }
+
+  before do
+    stub_const('MCP::Tool::Response', Struct.new(:payload))
+  end
+
+  it 'returns JSON as an application/json resource' do
+    model = instance_double(SimpleCovMcp::CoverageModel)
+    allow(SimpleCovMcp::CoverageModel).to receive(:new).and_return(model)
+    allow(model).to receive(:detailed_for).with('lib/foo.rb').and_return(
+      {
+        'file' => '/abs/path/lib/foo.rb',
+        'lines' => [
+          { 'line' => 1, 'hits' => 1, 'covered' => true },
+          { 'line' => 2, 'hits' => 0, 'covered' => false }
+        ],
+        'summary' => { 'covered' => 1, 'total' => 2, 'pct' => 50.0 }
+      }
+    )
+
+    response = described_class.call(path: 'lib/foo.rb', server_context: server_context)
+    item = response.payload.first
+    expect(item['type']).to eq('resource')
+    expect(item['resource']).to include('mimeType' => 'application/json', 'name' => 'coverage_detailed.json')
+    data = JSON.parse(item['resource']['text'])
+    expect(data).to include('file', 'lines', 'summary')
+    expect(data['lines']).to be_an(Array)
+  end
+end
+

data/spec/coverage_raw_tool_spec.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'simple_cov_mcp/tools/coverage_raw_tool'
+
+RSpec.describe SimpleCovMcp::Tools::CoverageRawTool do
+  let(:server_context) { instance_double('ServerContext').as_null_object }
+
+  before do
+    stub_const('MCP::Tool::Response', Struct.new(:payload))
+  end
+
+  it 'returns JSON as an application/json resource' do
+    model = instance_double(SimpleCovMcp::CoverageModel)
+    allow(SimpleCovMcp::CoverageModel).to receive(:new).and_return(model)
+    allow(model).to receive(:raw_for).with('lib/foo.rb').and_return(
+      {
+        'file' => '/abs/path/lib/foo.rb',
+        'lines' => [nil, 1, 0]
+      }
+    )
+
+    response = described_class.call(path: 'lib/foo.rb', server_context: server_context)
+    item = response.payload.first
+    expect(item['type']).to eq('resource')
+    expect(item['resource']).to include('mimeType' => 'application/json', 'name' => 'coverage_raw.json')
+    data = JSON.parse(item['resource']['text'])
+    expect(data).to include('file', 'lines')
+    expect(data['lines']).to be_an(Array)
+  end
+end
+

data/spec/coverage_summary_tool_spec.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'simple_cov_mcp/tools/coverage_summary_tool'
+
+RSpec.describe SimpleCovMcp::Tools::CoverageSummaryTool do
+  let(:server_context) { instance_double('ServerContext').as_null_object }
+
+  before do
+    response_class = Class.new do
+      attr_reader :payload, :meta
+      def initialize(payload, meta: nil)
+        @payload = payload
+        @meta = meta
+      end
+    end
+    stub_const('MCP::Tool::Response', response_class)
+  end
+
+  it 'returns JSON as an application/json resource' do
+    model = instance_double(SimpleCovMcp::CoverageModel)
+    allow(SimpleCovMcp::CoverageModel).to receive(:new).and_return(model)
+    allow(model).to receive(:summary_for).with('lib/foo.rb').and_return(
+      {
+        'file' => '/abs/path/lib/foo.rb',
+        'summary' => { 'covered' => 10, 'total' => 12, 'pct' => 83.33 }
+      }
+    )
+
+    response = described_class.call(path: 'lib/foo.rb', server_context: server_context)
+    item = response.payload.first
+    expect(item['type']).to eq('resource')
+    expect(item['resource']).to include('mimeType' => 'application/json', 'name' => 'coverage_summary.json')
+    data = JSON.parse(item['resource']['text'])
+    expect(data).to include('file', 'summary')
+    expect(data['summary']).to include('covered', 'total', 'pct')
+  end
+end
+

data/spec/help_tool_spec.rb

@@ -23,9 +23,9 @@ RSpec.describe SimpleCovMcp::Tools::HelpTool do
     expect(response.meta).to be_nil
 
     payload = response.payload.first
-    expect(payload['type']).to eq('text')
-
-    data = JSON.parse(payload['text'])
+    expect(payload['type']).to eq('resource')
+    expect(payload['resource']).to include('mimeType' => 'application/json')
+    data = JSON.parse(payload['resource']['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')
@@ -35,7 +35,8 @@ RSpec.describe SimpleCovMcp::Tools::HelpTool do
   it 'filters entries when a query is provided' do
     response = described_class.call(query: 'uncovered', server_context: server_context)
     payload = response.payload.first
-    data = JSON.parse(payload['text'])
+    expect(payload['type']).to eq('resource')
+    data = JSON.parse(payload['resource']['text'])
 
     expect(data['tools']).not_to be_empty
     expect(data['tools']).to all(satisfy do |entry|

data/spec/uncovered_lines_tool_spec.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'simple_cov_mcp/tools/uncovered_lines_tool'
+
+RSpec.describe SimpleCovMcp::Tools::UncoveredLinesTool do
+  let(:server_context) { instance_double('ServerContext').as_null_object }
+
+  before do
+    stub_const('MCP::Tool::Response', Struct.new(:payload))
+  end
+
+  it 'returns JSON as an application/json resource' do
+    model = instance_double(SimpleCovMcp::CoverageModel)
+    allow(SimpleCovMcp::CoverageModel).to receive(:new).and_return(model)
+    allow(model).to receive(:uncovered_for).with('lib/foo.rb').and_return(
+      {
+        'file' => '/abs/path/lib/foo.rb',
+        'uncovered' => [5, 9, 12],
+        'summary' => { 'covered' => 10, 'total' => 12, 'pct' => 83.33 }
+      }
+    )
+
+    response = described_class.call(path: 'lib/foo.rb', server_context: server_context)
+    item = response.payload.first
+    expect(item['type']).to eq('resource')
+    expect(item['resource']).to include('mimeType' => 'application/json', 'name' => 'uncovered_lines.json')
+    data = JSON.parse(item['resource']['text'])
+    expect(data).to include('file', 'uncovered', 'summary')
+    expect(data['uncovered']).to eq([5, 9, 12])
+  end
+end
+

data/spec/version_spec.rb

@@ -5,6 +5,11 @@ require 'spec_helper'
 RSpec.describe 'Version constant' do
   it 'exposes a semver-like version string' do
     expect(SimpleCovMcp::VERSION).to be_a(String)
-    expect(SimpleCovMcp::VERSION).to match(/\A\d+\.\d+\.\d+(?:[.-][0-9A-Za-z]+)?\z/)
+    # Named fragments for readability (simplified SemVer)
+    CORE  = /\d+\.\d+\.\d+/
+    ID    = /[[:alnum:].-]+/            # ASCII alnum plus dot/hyphen
+    SEMVER = /\A#{CORE.source}(?:-#{ID.source})?(?:\+#{ID.source})?\z/
+
+    expect(SimpleCovMcp::VERSION).to match(SEMVER)
   end
 end

data/spec/version_tool_spec.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'simple_cov_mcp/tools/version_tool'
+
+RSpec.describe SimpleCovMcp::Tools::VersionTool do
+  let(:server_context) { instance_double('ServerContext').as_null_object }
+
+  before do
+    stub_const('MCP::Tool::Response', Struct.new(:payload))
+  end
+
+  it 'returns a text payload with the version string' do
+    response = described_class.call(server_context: server_context)
+    item = response.payload.first
+    expect(item[:type] || item['type']).to eq('text')
+    text = item[:text] || item['text']
+    expect(text).to include('SimpleCovMcp version:')
+    expect(text).to include(SimpleCovMcp::VERSION)
+  end
+end
+

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: simplecov-mcp
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Keith R. Bennett
@@ -110,6 +110,9 @@ files:
 - spec/cli_spec.rb
 - spec/cli_table_spec.rb
 - spec/cli_usage_spec.rb
+- spec/coverage_detailed_tool_spec.rb
+- spec/coverage_raw_tool_spec.rb
+- spec/coverage_summary_tool_spec.rb
 - spec/coverage_table_tool_spec.rb
 - spec/error_handler_spec.rb
 - spec/errors_stale_spec.rb
@@ -122,8 +125,10 @@ files:
 - spec/simplecov_mcp_model_spec.rb
 - spec/spec_helper.rb
 - spec/staleness_more_spec.rb
+- spec/uncovered_lines_tool_spec.rb
 - spec/util_spec.rb
 - spec/version_spec.rb
+- spec/version_tool_spec.rb
 homepage: https://github.com/keithrbennett/simplecov-mcp
 licenses:
 - MIT