rage-rb 1.26.1 → 1.28.0

data/lib/rage/cli/skills.rb

@@ -1,291 +1,293 @@
 # frozen_string_literal: true
 
-class CLISkills < Thor
-  SKILLS_DIR = "rage-framework"
-  VERSION_FILE = ".version"
-
-  desc "install", "Install skills for coding agents"
-  option :verbose, desc: "Debug output"
-  def install
-    installation_path = choose_installation_path
-    return unless installation_path
-
-    skills_version = fetch_skills_version
-    say "Downloading skills..."
-    install_skills(installation_path, skills_version)
-
-    say "#{set_color("✓", :green)} Installed Rage skills #{set_color(skills_version, :bold)} to #{set_color(installation_path, :cyan)}."
-    say "#{set_color("✓", :green)} Skills are now available to your coding agent."
-  rescue => e
-    say_error(e)
-  end
-
-  desc "update", "Update installed skills"
-  option :verbose, desc: "Debug output"
-  option :json, type: :boolean, desc: "Output JSON for programmatic use"
-  def update
-    skills_destinations = Dir.glob(".*/skills/#{SKILLS_DIR}")
-    debug { "Existing skills installations found: #{skills_destinations}" }
-
-    if skills_destinations.empty?
-      log "No existing installation found. Running fresh install...\n\n"
-      return install
+module Rage::CLI
+  class Skills < Base
+    SKILLS_DIR = "rage-framework"
+    VERSION_FILE = ".version"
+
+    desc "install", "Install skills for coding agents"
+    option :verbose, desc: "Debug output"
+    def install
+      installation_path = choose_installation_path
+      return unless installation_path
+
+      skills_version = fetch_skills_version
+      say "Downloading skills..."
+      install_skills(installation_path, skills_version)
+
+      say "#{set_color("✓", :green)} Installed Rage skills #{set_color(skills_version, :bold)} to #{set_color(installation_path, :cyan)}."
+      say "#{set_color("✓", :green)} Skills are now available to your coding agent."
+    rescue => e
+      say_error(e)
     end
 
-    skills_version = fetch_skills_version
-    updated_paths = []
+    desc "update", "Update installed skills"
+    option :verbose, desc: "Debug output"
+    option :json, type: :boolean, desc: "Output JSON for programmatic use"
+    def update
+      skills_destinations = Dir.glob(".*/skills/#{SKILLS_DIR}")
+      debug { "Existing skills installations found: #{skills_destinations}" }
 
-    skills_destinations.each do |destination|
-      version_file = File.join(destination, VERSION_FILE)
-      current_version = File.exist?(version_file) ? File.read(version_file).strip : nil
-
-      if current_version == skills_version
-        log "#{set_color(destination, :cyan)}: already up to date."
-        next
+      if skills_destinations.empty?
+        log "No existing installation found. Running fresh install...\n\n"
+        return install
       end
 
-      log "Updating #{set_color(destination, :cyan)}..."
-      install_skills(destination, skills_version)
-      updated_paths << destination
-    end
+      skills_version = fetch_skills_version
+      updated_paths = []
 
-    if updated_paths.any?
-      log "#{set_color("✓", :green)} Updated #{updated_paths.size} installation#{"s" if updated_paths.size > 1} to #{set_color(skills_version, :bold)}."
-    end
+      skills_destinations.each do |destination|
+        version_file = File.join(destination, VERSION_FILE)
+        current_version = File.exist?(version_file) ? File.read(version_file).strip : nil
 
-    json_output(
-      status: updated_paths.any? ? "updated" : "up_to_date",
-      version: skills_version,
-      paths: skills_destinations
-    )
-  rescue => e
-    if options[:json]
-      json_output(status: "error", message: e.message)
-      exit 1
-    else
-      say_error(e)
-    end
-  end
+        if current_version == skills_version
+          log "#{set_color(destination, :cyan)}: already up to date."
+          next
+        end
 
-  no_commands do
-    def debug
-      puts("* #{yield}") if options[:verbose]
-    end
+        log "Updating #{set_color(destination, :cyan)}..."
+        install_skills(destination, skills_version)
+        updated_paths << destination
+      end
+
+      if updated_paths.any?
+        log "#{set_color("✓", :green)} Updated #{updated_paths.size} installation#{"s" if updated_paths.size > 1} to #{set_color(skills_version, :bold)}."
+      end
 
-    def log(message)
+      json_output(
+        status: updated_paths.any? ? "updated" : "up_to_date",
+        version: skills_version,
+        paths: skills_destinations
+      )
+    rescue => e
       if options[:json]
-        warn(message)
+        json_output(status: "error", message: e.message)
+        exit 1
       else
-        say(message)
+        say_error(e)
       end
     end
 
-    def json_output(data)
-      return unless options[:json]
-      require "json"
-      puts JSON.generate(data)
-    end
-
-    def say_error(error)
-      say "#{set_color("𐄂 Error:", :red, :bold)} #{error.message}"
-      debug { "#{error.class}: #{error.message}\n#{error.backtrace.join("\n")}" }
-    end
+    no_commands do
+      def debug
+        puts("* #{yield}") if options[:verbose]
+      end
 
-    def choose_installation_path
-      agent_options = [
-        ["1", "Claude Code", ".claude/skills"],
-        ["2", "GitHub Copilot", ".github/skills"],
-        ["3", "Cursor", ".cursor/skills"],
-        ["4", "Amp/Codex", ".agents/skills"],
-        ["5", "Antigravity", ".agent/skills"],
-        ["6", "Gemini CLI", ".gemini/skills"],
-        ["7", "Windsurf", ".windsurf/skills"],
-        ["8", "OpenCode", ".opencode/skills"],
-        ["9", "Other", ".claude/skills"]
-      ]
-
-      display_options = ([["Option", "Coding Agent", "Installation Path"]] + agent_options).map.with_index do |(option, agent, path), index|
-        if index == 0
-          [set_color(option, :bold), set_color(agent, :bold), set_color(path, :bold)]
+      def log(message)
+        if options[:json]
+          warn(message)
         else
-          [set_color(option, :bold), agent, set_color(path, :cyan)]
+          say(message)
         end
       end
 
-      print_ansi_table(display_options)
-      agent_choice = ask(set_color("Select your coding agent (1-#{agent_options[-1][0]}):", :bold), default: "1")
+      def json_output(data)
+        return unless options[:json]
+        require "json"
+        puts JSON.generate(data)
+      end
 
-      installation_path = ".claude/skills"
+      def say_error(error)
+        say "#{set_color("𐄂 Error:", :red, :bold)} #{error.message}"
+        debug { "#{error.class}: #{error.message}\n#{error.backtrace.join("\n")}" }
+      end
 
-      agent_options.each do |option, agent, path|
-        if agent_choice == option || agent.downcase.include?(agent_choice.downcase)
-          installation_path = path
-          break
+      def choose_installation_path
+        agent_options = [
+          ["1", "Claude Code", ".claude/skills"],
+          ["2", "GitHub Copilot", ".github/skills"],
+          ["3", "Cursor", ".cursor/skills"],
+          ["4", "Amp/Codex", ".agents/skills"],
+          ["5", "Antigravity", ".agent/skills"],
+          ["6", "Gemini CLI", ".gemini/skills"],
+          ["7", "Windsurf", ".windsurf/skills"],
+          ["8", "OpenCode", ".opencode/skills"],
+          ["9", "Other", ".claude/skills"]
+        ]
+
+        display_options = ([["Option", "Coding Agent", "Installation Path"]] + agent_options).map.with_index do |(option, agent, path), index|
+          if index == 0
+            [set_color(option, :bold), set_color(agent, :bold), set_color(path, :bold)]
+          else
+            [set_color(option, :bold), agent, set_color(path, :cyan)]
+          end
         end
-      end
 
-      say "\n#{set_color("Source:", :bold)} #{set_color("https://github.com/rage-rb/skills", :cyan)}"
-      say "#{set_color("Destination:", :bold)} #{set_color(installation_path, :cyan)}"
+        print_ansi_table(display_options)
+        agent_choice = ask(set_color("Select your coding agent (1-#{agent_options[-1][0]}):", :bold), default: "1")
 
-      answer = ask(set_color("? ", :green, :bold) + set_color("Proceed with installation? [Y/n]", :bold), default: "y")
-      unless answer.downcase.start_with?("y")
-        say "Installation cancelled."
-        return nil
-      end
+        installation_path = ".claude/skills"
 
-      File.join(installation_path, SKILLS_DIR)
-    end
+        agent_options.each do |option, agent, path|
+          if agent_choice == option || agent.downcase.include?(agent_choice.downcase)
+            installation_path = path
+            break
+          end
+        end
 
-    def fetch_skills_version
-      require "json"
+        say "\n#{set_color("Source:", :bold)} #{set_color("https://github.com/rage-rb/skills", :cyan)}"
+        say "#{set_color("Destination:", :bold)} #{set_color(installation_path, :cyan)}"
 
-      manifest = begin
-        JSON.parse(fetch("https://rage-rb.github.io/skills/manifest.json"))
-      rescue => e
-        debug { "#{e.class} (#{e.message}):\n#{e.backtrace.join("\n")}" }
-        raise "Could not download skills manifest. Please check your network connection."
+        answer = ask(set_color("? ", :green, :bold) + set_color("Proceed with installation? [Y/n]", :bold), default: "y")
+        unless answer.downcase.start_with?("y")
+          say "Installation cancelled."
+          return nil
+        end
+
+        File.join(installation_path, SKILLS_DIR)
       end
 
-      major, minor, _ = Rage::VERSION.split(".").map(&:to_i)
+      def fetch_skills_version
+        require "json"
 
-      debug { "Rage::VERSION: #{Rage::VERSION}; Manifest: #{manifest["versions"]}" }
+        manifest = begin
+          JSON.parse(fetch("https://rage-rb.github.io/skills/manifest.json"))
+        rescue => e
+          debug { "#{e.class} (#{e.message}):\n#{e.backtrace.join("\n")}" }
+          raise "Could not download skills manifest. Please check your network connection."
+        end
 
-      # Find the closest matching version: same major, highest minor <= current
-      matched_major, matched_minor = manifest["versions"].
-        keys.
-        map { |v| v.split(".").map(&:to_i) }.
-        select { |_major, _| _major == major }.
-        select { |_, _minor| _minor <= minor }.
-        max_by { |_, _minor| _minor }
+        major, minor, _ = Rage::VERSION.split(".").map(&:to_i)
 
-      if matched_major && matched_minor
-        manifest["versions"]["#{matched_major}.#{matched_minor}"]
-      else
-        raise "No skills available for Rage #{major}.x."
-      end
-    end
+        debug { "Rage::VERSION: #{Rage::VERSION}; Manifest: #{manifest["versions"]}" }
 
-    def install_skills(installation_path, version)
-      require "zlib"
-      require "rubygems/package"
-      require "fileutils"
-      require "stringio"
-      require "digest"
+        # Find the closest matching version: same major, highest minor <= current
+        matched_major, matched_minor = manifest["versions"].
+          keys.
+          map { |v| v.split(".").map(&:to_i) }.
+          select { |_major, _| _major == major }.
+          select { |_, _minor| _minor <= minor }.
+          max_by { |_, _minor| _minor }
 
-      Thread.report_on_exception = false
+        if matched_major && matched_minor
+          manifest["versions"]["#{matched_major}.#{matched_minor}"]
+        else
+          raise "No skills available for Rage #{major}.x."
+        end
+      end
 
-      artifact_request = Thread.new { fetch("https://github.com/rage-rb/skills/releases/download/#{version}/skills.tar.gz") }
-      checksum_request = Thread.new { fetch("https://github.com/rage-rb/skills/releases/download/#{version}/checksums.txt") }
+      def install_skills(installation_path, version)
+        require "zlib"
+        require "rubygems/package"
+        require "fileutils"
+        require "stringio"
+        require "digest"
 
-      artifact, checksum = artifact_request.value, checksum_request.value
+        Thread.report_on_exception = false
 
-      if artifact.nil? || checksum.nil?
-        raise "Could not download the skills package. Please check your network connection and try again."
-      end
+        artifact_request = Thread.new { fetch("https://github.com/rage-rb/skills/releases/download/#{version}/skills.tar.gz") }
+        checksum_request = Thread.new { fetch("https://github.com/rage-rb/skills/releases/download/#{version}/checksums.txt") }
 
-      sha, _ = checksum.split
-      if Digest::SHA256.hexdigest(artifact) != sha
-        raise "Download verification failed. Please try again."
-      end
+        artifact, checksum = artifact_request.value, checksum_request.value
 
-      destination = File.expand_path(installation_path)
-      FileUtils.mkdir_p(destination)
+        if artifact.nil? || checksum.nil?
+          raise "Could not download the skills package. Please check your network connection and try again."
+        end
 
-      # Clear existing contents but keep the directory intact
-      Dir.children(destination).each do |child|
-        debug { "Removing #{child}" }
-        FileUtils.rm_rf(File.join(destination, child))
-      end
+        sha, _ = checksum.split
+        if Digest::SHA256.hexdigest(artifact) != sha
+          raise "Download verification failed. Please try again."
+        end
 
-      Zlib::GzipReader.wrap(StringIO.new(artifact)) do |gz|
-        Gem::Package::TarReader.new(gz) do |tar|
-          tar.each do |entry|
-            path = File.join(destination, entry.full_name)
+        destination = File.expand_path(installation_path)
+        FileUtils.mkdir_p(destination)
 
-            unless File.expand_path(path).start_with?("#{destination}/")
-              raise "Invalid archive: contains files outside the destination directory."
-            end
+        # Clear existing contents but keep the directory intact
+        Dir.children(destination).each do |child|
+          debug { "Removing #{child}" }
+          FileUtils.rm_rf(File.join(destination, child))
+        end
 
-            if entry.directory?
-              debug { "Created directory #{entry.full_name}" }
-              FileUtils.mkdir_p(path)
-            elsif entry.file?
-              debug { "Written file #{entry.full_name}" }
-              FileUtils.mkdir_p(File.dirname(path))
-              File.binwrite(path, entry.read)
+        Zlib::GzipReader.wrap(StringIO.new(artifact)) do |gz|
+          Gem::Package::TarReader.new(gz) do |tar|
+            tar.each do |entry|
+              path = File.join(destination, entry.full_name)
+
+              unless File.expand_path(path).start_with?("#{destination}/")
+                raise "Invalid archive: contains files outside the destination directory."
+              end
+
+              if entry.directory?
+                debug { "Created directory #{entry.full_name}" }
+                FileUtils.mkdir_p(path)
+              elsif entry.file?
+                debug { "Written file #{entry.full_name}" }
+                FileUtils.mkdir_p(File.dirname(path))
+                File.binwrite(path, entry.read)
+              end
             end
           end
         end
-      end
 
-      File.write(File.join(destination, VERSION_FILE), version)
-    end
+        File.write(File.join(destination, VERSION_FILE), version)
+      end
 
-    def fetch(uri, retries = 2)
-      response = request(uri)
-      return response if response
+      def fetch(uri, retries = 2)
+        response = request(uri)
+        return response if response
 
-      retries > 0 ? fetch(uri, retries - 1) : nil
-    end
+        retries > 0 ? fetch(uri, retries - 1) : nil
+      end
 
-    def request(uri, limit = 3)
-      require "net/http"
+      def request(uri, limit = 3)
+        require "net/http"
 
-      raise "Too many HTTP redirects" if limit == 0
+        raise "Too many HTTP redirects" if limit == 0
 
-      debug { "Fetching #{uri[0..100]}" }
+        debug { "Fetching #{uri[0..100]}" }
 
-      parsed_uri = URI(uri)
-      http = Net::HTTP.new(parsed_uri.host, parsed_uri.port)
-      http.open_timeout = 5
-      http.read_timeout = 5
+        parsed_uri = URI(uri)
+        http = Net::HTTP.new(parsed_uri.host, parsed_uri.port)
+        http.open_timeout = 5
+        http.read_timeout = 5
 
-      if parsed_uri.scheme == "https"
-        http.use_ssl = true
-        http.verify_mode = OpenSSL::SSL::VERIFY_PEER
-        http.cert_store = OpenSSL::X509::Store.new.tap(&:set_default_paths)
-      end
+        if parsed_uri.scheme == "https"
+          http.use_ssl = true
+          http.verify_mode = OpenSSL::SSL::VERIFY_PEER
+          http.cert_store = OpenSSL::X509::Store.new.tap(&:set_default_paths)
+        end
 
-      response = http.request(Net::HTTP::Get.new(parsed_uri))
+        response = http.request(Net::HTTP::Get.new(parsed_uri))
 
-      case response
-      when Net::HTTPSuccess
-        response.body.force_encoding("ASCII-8BIT")
-      when Net::HTTPRedirection
-        debug { "Redirected to #{response["Location"][0..100]}" }
-        request(response["Location"], limit - 1)
+        case response
+        when Net::HTTPSuccess
+          response.body.force_encoding("ASCII-8BIT")
+        when Net::HTTPRedirection
+          debug { "Redirected to #{response["Location"][0..100]}" }
+          request(response["Location"], limit - 1)
+        end
       end
-    end
 
-    # print a table stripping ANSI codes to ensure paddings are based on visible length
-    def print_ansi_table(rows)
-      return if rows.empty?
+      # print a table stripping ANSI codes to ensure paddings are based on visible length
+      def print_ansi_table(rows)
+        return if rows.empty?
 
-      widths = rows.each_with_object([]) do |row, maxima|
-        row.each_with_index do |column, index|
-          visible_width = strip_ansi(column.to_s).size
-          maxima[index] = [maxima[index] || 0, visible_width].max
+        widths = rows.each_with_object([]) do |row, maxima|
+          row.each_with_index do |column, index|
+            visible_width = strip_ansi(column.to_s).size
+            maxima[index] = [maxima[index] || 0, visible_width].max
+          end
         end
-      end
 
-      border = "+" + widths.map { |width| "-" * (width + 2) }.join("+") + "+"
-      say(border)
+        border = "+" + widths.map { |width| "-" * (width + 2) }.join("+") + "+"
+        say(border)
+
+        rows.each do |row|
+          formatted_cells = widths.each_index.map do |index|
+            cell = row[index].to_s
+            cell_padding = widths[index] - strip_ansi(cell).size
+            " #{cell}#{" " * cell_padding} "
+          end
 
-      rows.each do |row|
-        formatted_cells = widths.each_index.map do |index|
-          cell = row[index].to_s
-          cell_padding = widths[index] - strip_ansi(cell).size
-          " #{cell}#{" " * cell_padding} "
+          say("|#{formatted_cells.join("|")}|")
         end
 
-        say("|#{formatted_cells.join("|")}|")
+        say(border)
       end
 
-      say(border)
-    end
-
-    def strip_ansi(text)
-      text.gsub(/\e\[[0-9;]*m/, "")
+      def strip_ansi(text)
+        text.gsub(/\e\[[0-9;]*m/, "")
+      end
     end
   end
 end