rspec-sprint 0.1.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f16ad949645afe84a5a8ba614a5a6e2e40f609fa76f73c2e0af72557830dbd9d
-  data.tar.gz: 8503cde0dabd4c8cccefee9d53ef55b996e2724ad21ebcb6f4f4d6b9b6eec747
+  metadata.gz: fff025e27cecb118a9aca49be7611c62cf40da2dddd17d827320883c6e232acd
+  data.tar.gz: 99e64c381ca443afea9d34bb909b9fddc9d857aadb32beec52d677040885f553
 SHA512:
-  metadata.gz: 021d7debd206e0d543a97c71692eed732b0f3bc280e13e3262c19936ee790012d3b93588f3de91f1741f19fa26ac36da322e2fd1ad87ef81f215098231f00a66
-  data.tar.gz: 0d93b9cb8eb129e2cc6451d8ad2607efcbd0a1a6108549d4c5f8a566c4fd69db39b481d1b40293462ed94bd0396b49da6a2cf573b9e38eb3c6357c22c73a9e43
+  metadata.gz: c73e4b56bc7e6a57ad70c8fec7719ed24502ffc121b99fee8d8058c13996296f68d69cf218c1d75e1da4b0078a251dda9e61769aad86354eef14fb6d8c8e0246
+  data.tar.gz: 7d44089f6ec20e24796892c8f395b4be545ee1ce87d4b327555b3620548afad35bb5a047a8ceec07ac391887b8ed8758d2f5b4ae1b4940389bd23590d03cb9f1

data/lib/rspec_sprint/cli.rb

@@ -21,9 +21,35 @@ module RspecSprint
       Requires test-prof in your bundle and `require "test_prof"` in your spec
       helper for factory diagnosis (Rule①); without it, the other rules still run.
     DESC
+    option :no_snapshot, type: :boolean, default: false,
+      desc: "Skip saving a snapshot to .rspec-sprint/"
+    option :compare_last, type: :boolean, default: false,
+      desc: "Show delta vs most recent previous snapshot"
     def doctor(*rspec_args)
       result = Collector.new(rspec_args: rspec_args).run
-      puts Doctor.diagnose(result)
+      puts Doctor.diagnose(result, no_snapshot: options[:no_snapshot],
+                                   compare_last: options[:compare_last])
+    end
+
+    desc "compare [-- RSPEC_ARGS]", "Run your suite and save results for long-term tracking"
+    long_desc <<~DESC
+      Runs your RSpec suite once and (with --save-baseline) saves the result as a
+      named baseline in .rspec-sprint/baseline.json for long-term comparison.
+
+      Unlike the rolling snapshots used by `doctor --compare-last`, the baseline
+      is a permanent reference point that is never auto-pruned.
+
+        bundle exec rspec-sprint compare --save-baseline
+        bundle exec rspec-sprint compare --save-baseline -- spec/models
+    DESC
+    option :save_baseline, type: :boolean, default: false,
+      desc: "Save results as .rspec-sprint/baseline.json"
+    option :no_snapshot, type: :boolean, default: false,
+      desc: "Skip saving a rolling snapshot to .rspec-sprint/"
+    def compare(*rspec_args)
+      result = Collector.new(rspec_args: rspec_args).run
+      puts Doctor.diagnose(result, no_snapshot: options[:no_snapshot],
+                                   save_baseline: options[:save_baseline])
     end
 
     default_command :doctor

data/lib/rspec_sprint/comparator.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+require "time"
+
+module RspecSprint
+  # Computes and formats the delta between a previous snapshot (raw JSON hash)
+  # and the current Normalizer::Snapshot.
+  module Comparator
+    NO_BASELINE_MSG = "前回のスナップショットがありません。まず `bundle exec rspec-sprint doctor` を実行してベースラインを作成してください。\n" \
+                      "(No prior snapshot. Run doctor once to create a baseline.)"
+
+    module_function
+
+    # previous_data: Hash parsed from a saved snapshot JSON (may be nil)
+    # current:       Normalizer::Snapshot
+    # Returns a formatted string to append to the doctor report.
+    def format_delta(previous_data, current)
+      return NO_BASELINE_MSG if previous_data.nil?
+
+      prev_duration   = previous_data[:suite_duration].to_f
+      curr_duration   = current.suite_duration.to_f
+      duration_delta  = curr_duration - prev_duration
+      duration_pct    = prev_duration > 0 ? (duration_delta / prev_duration * 100).round(1) : 0.0
+
+      prev_factory_pct = prev_duration > 0 ? (previous_data[:factory_time].to_f / prev_duration * 100).round(1) : 0.0
+      curr_factory_pct = (current.factory_time_ratio * 100).round(1)
+
+      prev_ts = format_timestamp(previous_data[:created_at])
+
+      sign = duration_delta >= 0 ? "+" : ""
+      "前回比 #{sign}#{duration_delta.round(1)}s (#{sign}#{duration_pct}%)  " \
+        "factory time #{prev_factory_pct}% → #{curr_factory_pct}%  [#{prev_ts}]"
+    end
+
+    def format_timestamp(iso8601)
+      return "unknown" if iso8601.nil?
+
+      Time.parse(iso8601).strftime("%Y-%m-%d %H:%M")
+    rescue ArgumentError
+      iso8601.to_s
+    end
+  end
+end

data/lib/rspec_sprint/diagnosis.rb

@@ -3,6 +3,8 @@
 require_relative "normalizer"
 require_relative "ranker"
 require_relative "formatter"
+require_relative "snapshot_store"
+require_relative "comparator"
 
 module RspecSprint
   # Pure pipeline: profiler JSON files -> normalized snapshot -> ranked findings
@@ -11,9 +13,23 @@ module RspecSprint
   module Diagnosis
     module_function
 
-    def from_files(rspec_json:, factory_prof_json: nil)
+    def from_files(rspec_json:, factory_prof_json: nil, save_snapshot: false, compare_last: false, save_baseline: false)
+      # Load previous BEFORE saving so "last" refers to the prior run, not this one.
+      previous = compare_last ? SnapshotStore.load_last : nil
+
       snapshot = Normalizer.new(rspec_json: rspec_json, factory_prof_json: factory_prof_json).call
-      Formatter.format(Ranker.call(snapshot))
+      SnapshotStore.save(snapshot) if save_snapshot
+      SnapshotStore.save_baseline(snapshot) if save_baseline
+
+      sections = [Formatter.format(Ranker.call(snapshot))]
+      sections << Comparator.format_delta(previous, snapshot) if compare_last
+      sections << baseline_saved_msg if save_baseline
+      sections.join("\n\n")
+    end
+
+    def baseline_saved_msg
+      "ベースライン保存済み: #{SnapshotStore::BASELINE_PATH}\n" \
+        "(Baseline saved — run `rspec-sprint compare` to compare future runs)"
     end
   end
 end

data/lib/rspec_sprint/doctor.rb

@@ -11,7 +11,7 @@ module RspecSprint
   module Doctor
     module_function
 
-    def diagnose(result)
+    def diagnose(result, no_snapshot: false, compare_last: false, save_baseline: false)
       return diagnosis_impossible(result) unless result.rspec_json?
 
       sections = []
@@ -19,7 +19,10 @@ module RspecSprint
       sections << factory_prof_missing_hint unless result.factory_prof?
       sections << Diagnosis.from_files(
         rspec_json: result.rspec_json_path,
-        factory_prof_json: result.factory_prof? ? result.factory_prof_path : nil
+        factory_prof_json: result.factory_prof? ? result.factory_prof_path : nil,
+        save_snapshot: !no_snapshot,
+        compare_last: compare_last,
+        save_baseline: save_baseline
       )
       sections.join("\n\n")
     end

data/lib/rspec_sprint/snapshot_store.rb

@@ -0,0 +1,115 @@
+# frozen_string_literal: true
+
+require "json"
+require "fileutils"
+require "time"
+
+module RspecSprint
+  # Persists normalized snapshots to .rspec-sprint/snapshots/ for future
+  # comparison (the seed for --compare-last / Test Performance Ledger).
+  # Failure-safe: disk errors warn and skip; doctor never crashes because of us.
+  class SnapshotStore
+    DEFAULT_DIR = ".rspec-sprint/snapshots"
+    BASELINE_PATH = ".rspec-sprint/baseline.json"
+    MAX_SNAPSHOTS = 3
+
+    def self.save(snapshot, dir: DEFAULT_DIR)
+      new(dir: dir).save(snapshot)
+    end
+
+    def self.load_last(dir: DEFAULT_DIR)
+      new(dir: dir).load_last
+    end
+
+    def self.save_baseline(snapshot, path: BASELINE_PATH)
+      new.save_baseline(snapshot, path: path)
+    end
+
+    def self.load_baseline(path: BASELINE_PATH)
+      new.load_baseline(path: path)
+    end
+
+    def initialize(dir: DEFAULT_DIR)
+      @dir = dir
+    end
+
+    def save(snapshot)
+      FileUtils.mkdir_p(@dir)
+      path = File.join(@dir, "#{timestamp}.json")
+      File.write(path, serialize(snapshot))
+      prune
+    rescue Errno::EACCES, Errno::ENOSPC => e
+      warn "rspec-sprint: snapshot not saved (#{e.class.name.split("::").last}: #{e.message})"
+    end
+
+    def list
+      Dir.glob(File.join(@dir, "*.json")).sort
+    end
+
+    # Returns the most recent snapshot as a symbol-keyed Hash, or nil if none.
+    def load_last
+      files = list
+      return nil if files.empty?
+
+      JSON.parse(File.read(files.last), symbolize_names: true)
+    rescue JSON::ParserError, Errno::ENOENT
+      nil
+    end
+
+    # Saves snapshot as the named baseline (single file, never pruned).
+    def save_baseline(snapshot, path: BASELINE_PATH)
+      FileUtils.mkdir_p(File.dirname(path))
+      File.write(path, serialize(snapshot))
+    rescue Errno::EACCES, Errno::ENOSPC => e
+      warn "rspec-sprint: baseline not saved (#{e.class.name.split("::").last}: #{e.message})"
+    end
+
+    # Returns the saved baseline as a symbol-keyed Hash, or nil if none.
+    def load_baseline(path: BASELINE_PATH)
+      JSON.parse(File.read(path), symbolize_names: true)
+    rescue JSON::ParserError, Errno::ENOENT
+      nil
+    end
+
+    private
+
+    def serialize(snapshot)
+      JSON.pretty_generate({
+        schema_version: 1,
+        created_at: Time.now.utc.iso8601,
+        suite_duration: snapshot.suite_duration,
+        factory_time: snapshot.factory_time,
+        example_count: snapshot.example_count,
+        failure_count: snapshot.failure_count,
+        factories: (snapshot.factories || []).map do |f|
+          { name: f.name, total_count: f.total_count,
+            top_level_count: f.top_level_count, total_time: f.total_time }
+        end,
+        examples: (snapshot.examples || []).map do |e|
+          { full_description: e.full_description,
+            file_path: sanitize_path(e.file_path),
+            run_time: e.run_time, status: e.status }
+        end
+      })
+    end
+
+    def sanitize_path(path)
+      path.to_s.sub(%r{\A\./}, "")
+    end
+
+    def prune
+      files = list
+      return if files.length <= MAX_SNAPSHOTS
+
+      files.first(files.length - MAX_SNAPSHOTS).each do |f|
+        File.delete(f)
+      rescue Errno::EACCES
+        nil
+      end
+    end
+
+    def timestamp
+      Time.now.strftime("%Y%m%d%H%M%S%L")
+    end
+  end
+end

data/lib/rspec_sprint/version.rb

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

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspec-sprint
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.4.0
 platform: ruby
 authors:
 - yasu551
+autorequire:
 bindir: exe
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2026-06-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: test-prof
@@ -42,6 +43,7 @@ description: |
   JSON formatter), interprets the output against opinionated heuristics, and asserts
   the top few things to fix — diagnosis plus concrete prescription, not just numbers.
   It wraps existing profilers; it does not reimplement them.
+email:
 executables:
 - rspec-sprint
 extensions: []
@@ -53,6 +55,7 @@ files:
 - lib/rspec_sprint.rb
 - lib/rspec_sprint/cli.rb
 - lib/rspec_sprint/collector.rb
+- lib/rspec_sprint/comparator.rb
 - lib/rspec_sprint/diagnosis.rb
 - lib/rspec_sprint/doctor.rb
 - lib/rspec_sprint/finding.rb
@@ -60,6 +63,7 @@ files:
 - lib/rspec_sprint/normalizer.rb
 - lib/rspec_sprint/ranker.rb
 - lib/rspec_sprint/rules.rb
+- lib/rspec_sprint/snapshot_store.rb
 - lib/rspec_sprint/version.rb
 homepage: https://github.com/yasu551/rspec-sprint
 licenses:
@@ -67,6 +71,7 @@ licenses:
 metadata:
   source_code_uri: https://github.com/yasu551/rspec-sprint
   changelog_uri: https://github.com/yasu551/rspec-sprint/blob/main/CHANGELOG.md
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -81,7 +86,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 3.5.22
+signing_key:
 specification_version: 4
 summary: Diagnose RSpec/Rails test-suite slowness and assert the top repo-specific
   fixes.