x_aeon_agents 1.0.28

data/lib/x_aeon_agents/agents/review_responder_agent.rb

@@ -0,0 +1,64 @@
+module XAeonAgents
+  module Agents
+    # Agent responsible for generating a reply to a review comment
+    class ReviewResponderAgent < ComposableAgents::Cline::Agent
+      prepend AgentDefaults
+
+      # Define input artifacts contracts
+      #
+      # @return [Hash{Symbol => Object}] Set of input artifacts description, per artifact name
+      def input_artifacts_contracts
+        super.merge(
+          {
+            conversations: {
+              description: 'All PR conversations and comments to be considered (context)',
+              type: :json
+            },
+            open_comment_for_reply: {
+              description: 'The exact comment to be replied to',
+              type: :json
+            },
+            requirements: 'The requirements that have been implemented (or "No requirements")',
+            plan: 'The implementation plan that was used to implement those requirements (or "No implementation plan")',
+            files_diffs: 'The code changes from implement_requirements workflow (or "No changes")'
+          }
+        )
+      end
+
+      # Define output artifacts contracts
+      #
+      # @return [Hash{Symbol => Object}] Set of output artifacts description, per artifact name
+      def output_artifacts_contracts
+        super.merge(
+          reply: {
+            description: 'The exact reply text to post',
+            type: :text
+          }
+        )
+      end
+
+      # Constructor
+      #
+      # @param agent_params [Hash{Symbol => Object}] Extra agent parameters
+      def initialize(**agent_params)
+        super(
+          name: 'ReviewResponder',
+          role: 'You are a review responder agent. Your role is to reply to feedback, ' \
+            'taking into account the feedback itself, and the work that has been done because of this feedback.',
+          objective: 'Generate a reply to a review comment',
+          **agent_params
+        )
+        self.constraints = <<~EO_CONSTRAINTS
+          - You are in read-only mode.
+          - Do NOT modify or write any file.
+          - The implementation work is already complete (captured in the artifacts).
+          - ONLY focus on addressing the specific comment of the `#{artifact_ref(:open_comment_for_reply)}` artifact appropriately.
+          - Do NOT answer or reply to any other comment.
+          - You already have ALL the information required.
+          - You MUST NOT ask follow-up questions.
+          - You MUST NOT ask for user confirmation.
+        EO_CONSTRAINTS
+      end
+    end
+  end
+end

data/lib/x_aeon_agents/agents/skill_generator_agent.rb

@@ -0,0 +1,95 @@
+require 'fileutils'
+require 'pathname'
+
+module XAeonAgents
+  module Agents
+    # Agent responsible for generating skill files from ERB templates.
+    class SkillGeneratorAgent < ComposableAgents::Agent
+      prepend AgentDefaults
+
+      # Define input artifacts contracts
+      #
+      # @return [Hash{Symbol => Object}] Set of input artifacts description, per artifact name
+      def input_artifacts_contracts
+        {
+          output_dir: 'Output directory for generated skills',
+          skill_names: {
+            description: 'Optional list of skill names to generate. If nil or empty, all skills are generated.',
+            optional: true
+          }
+        }
+      end
+
+      # Define output artifacts contracts
+      #
+      # @return [Hash{Symbol => Object}] Set of output artifacts description, per artifact name
+      def output_artifacts_contracts
+        { success: 'Whether the skill generation was successful' }
+      end
+
+      # Execute the agent to generate skill files from ERB templates.
+      #
+      # @param output_dir [String] Output directory for generated skills
+      # @param skill_names [Array<String>, nil] Optional list of skill names to generate.
+      #   Supports comma-separated values within each element. If nil or empty, all skills are generated.
+      # @return [Hash{Symbol => Object}] Output artifacts content
+      def run(output_dir: 'skills', skill_names: nil)
+        transformations = {
+          '.erb' => proc { |src_file| GenHelpers::ErbEvaluator.new(src_file).result }
+        }.freeze
+
+        src_dir = File.expand_path('skills.src')
+        src_pathname = Pathname.new(src_dir)
+        dest_dir = File.expand_path(output_dir)
+
+        FileUtils.mkdir_p(dest_dir)
+
+        # Normalize skill_names: flatten, split by comma, strip, compact, uniq
+        normalized_skill_names = (skill_names || [])
+          .flat_map { |name| name.split(',') }
+          .map(&:strip)
+          .reject(&:empty?)
+          .uniq
+
+        failed = false
+        Dir.glob(File.join(src_dir, '**', '*'), File::FNM_DOTMATCH)
+          .select { |f| File.file?(f) && File.basename(f) != '.skill_config.yml' }
+          .each do |src_file|
+            relative_path = Pathname.new(src_file).relative_path_from(src_pathname).to_s
+            # Determine the top-level skill directory for this file.
+            # Skip files whose top-level skill directory is not in the requested list
+            next if !normalized_skill_names.empty? && !normalized_skill_names.include?(relative_path.split('/').first)
+
+            file_ext = File.extname(relative_path)
+            dst_file = File.join(
+              dest_dir,
+              transformations.key?(file_ext) ? relative_path.sub(/#{Regexp.escape(file_ext)}$/, '') : relative_path
+            )
+            puts "Processing: #{relative_path}"
+            puts "    Output: #{dst_file}"
+            begin
+              FileUtils.mkdir_p(File.dirname(dst_file))
+              if transformations.key?(file_ext)
+                File.write(dst_file, transformations[file_ext].call(src_file))
+              else
+                FileUtils.cp(src_file, dst_file)
+              end
+              puts '    Status: ✓ Processed successfully'
+            rescue StandardError => e
+              puts "    Status: ✗ Error - #{e.message}\n    #{e.backtrace.first}"
+              failed = true
+            end
+            puts
+          end
+
+        puts
+        if failed
+          puts 'Skills generated with some errors (see above).'
+        else
+          puts 'Skills generated successfully.'
+        end
+        { success: !failed }
+      end
+    end
+  end
+end

data/lib/x_aeon_agents/agents/skill_installer_agent.rb

@@ -0,0 +1,103 @@
+require 'front_matter_parser'
+
+module XAeonAgents
+  module Agents
+    # Agent responsible for installing skills from the .skills manifest.
+    class SkillInstallerAgent < ComposableAgents::Agent
+      prepend AgentDefaults
+
+      # Define input artifacts contracts
+      #
+      # @return [Hash{Symbol => Object}] Set of input artifacts description, per artifact name
+      def input_artifacts_contracts
+        { agent: 'Agent name to be used to install skills' }
+      end
+
+      # Execute the agent to install skills from the .skills manifest.
+      #
+      # @param agent [String] Agent name to be used to install skills
+      # @return [Hash{Symbol => Object}] Output artifacts content
+      def run(agent: 'cline')
+        agent_name = agent.to_sym
+        original_no_color = ENV.fetch('NO_COLOR', nil)
+        ENV['NO_COLOR'] = '1'
+        begin
+          list_lines = `skillkit manifest`.split("\n")
+        ensure
+          ENV['NO_COLOR'] = original_no_color
+        end
+        list_lines = list_lines[(list_lines.index('Skills:') + 1)..]
+        list_lines[0..(list_lines.index('') - 1)].each_slice(2) do |repo_desc, skills_desc|
+          install_skills_recursive(
+            repo_desc[4..].strip,
+            skills_desc[12..].split(',').map(&:strip),
+            agent_name
+          )
+        end
+
+        puts
+        puts 'Skills identified in the skillkit manifest and their dependencies have been installed successfully'
+        { installed: true }
+      end
+
+      private
+
+      # Install skills using skillkit, including dependencies recursively.
+      #
+      # @param repo [String] Repository
+      # @param skills [Array<String>] Skills to install
+      # @param agent [Symbol] Agent to use
+      def install_skills_recursive(repo, skills, agent)
+        return if skills.empty?
+
+        agents_config = {
+          cline: { skills_dir: '.cline/skills' }
+        }
+        skills_dir = agents_config[agent][:skills_dir]
+
+        puts "Install skills #{repo} / #{skills.join(',')}..."
+        system "skillkit install #{repo} --yes --skills=#{skills.join(',')} --agent=#{agent}", exception: true
+        fix_skills_metadata(skills, agent)
+
+        # Resolve dependencies
+        deps_per_repo = {}
+        skills.each do |skill|
+          deps = FrontMatterParser::Parser.parse_file("#{skills_dir}/#{skill}/SKILL.md").front_matter.dig('metadata', 'dependencies')
+          next if deps.nil?
+
+          deps.each do |skill_dep|
+            skill_dep = "#{repo}:#{skill_dep}" unless skill_dep.include?(':')
+            skill_dep_repo, skill_dep_name = skill_dep.split(':')
+            unless File.exist?("#{skills_dir}/#{skill_dep_name}/SKILL.md")
+              deps_per_repo[skill_dep_repo] ||= []
+              deps_per_repo[skill_dep_repo] << skill_dep_name unless deps_per_repo[skill_dep_repo].include?(skill_dep_name)
+            end
+          end
+        end
+
+        deps_per_repo.each { |dep_repo, dep_skills| install_skills_recursive(dep_repo, dep_skills, agent) }
+      end
+
+      # Fix the .skillkit.json subpath property after skillkit install.
+      #
+      # @param skills [Array<String>] Installed skills
+      # @param agent [Symbol] Agent used
+      def fix_skills_metadata(skills, agent)
+        require 'json'
+
+        agents_config = { cline: { skills_dir: '.cline/skills' } }
+        skills_dir = agents_config[agent][:skills_dir]
+
+        skills.each do |skill_name|
+          json_file = "#{skills_dir}/#{skill_name}/.skillkit.json"
+          json = JSON.parse(File.read(json_file))
+          next if json['subpath'].start_with?('skills/')
+
+          json['subpath'] = "skills/#{json['subpath']}"
+          puts "Fix subpath of #{json_file}"
+          File.write(json_file, JSON.pretty_generate(json))
+        end
+      end
+    end
+  end
+end

data/lib/x_aeon_agents/agents/task_starter_agent.rb

@@ -0,0 +1,66 @@
+module XAeonAgents
+  module Agents
+    # Agent responsible for opening a new git worktree for a task.
+    class TaskStarterAgent < ComposableAgents::Agent
+      # Exception raised when the target worktree directory is invalid
+      # (already exists but is not a git worktree, or is a worktree on a
+      # different branch than the requested one).
+      class TaskStarterError < StandardError
+      end
+
+      prepend AgentDefaults
+
+      # Define input artifacts contracts
+      #
+      # @return [Hash{Symbol => Object}] Set of input artifacts description, per artifact name
+      def input_artifacts_contracts
+        { branch_name: 'The name of the git branch to create worktree for' }
+      end
+
+      # Define output artifacts contracts
+      #
+      # @return [Hash{Symbol => Object}] Set of output artifacts description, per artifact name
+      def output_artifacts_contracts
+        { worktree_dir: 'The directory where the worktree was created' }
+      end
+
+      # Execute the agent to open a new git worktree for a feature branch.
+      #
+      # @param branch_name [String] Name of the git branch to create worktree for
+      # @return [Hash{Symbol => Object}] Output artifacts content
+      def run(branch_name:)
+        dir = ".worktrees/#{branch_name.tr('/', '_')}"
+        puts "Setting worktree #{dir} to work on branch #{branch_name}..."
+        # Create the branch if it does not exist (without checking it out)
+        Helpers.git.branch(branch_name).create unless Helpers.git.branches.any? { |branch| branch.name == branch_name }
+        # Create the git worktree only if it does not exist yet (idempotent)
+        if File.directory?(dir)
+          # The directory already exists: it must be a git worktree for the requested branch.
+          # A git worktree has a `.git` file (a gitdir pointer), not a `.git` directory.
+          unless File.file?(File.join(dir, '.git'))
+            raise TaskStarterError, <<~EO_MSG.strip
+              Directory '#{dir}' already exists but is not a git worktree (no '.git' pointer file found).
+              Please remove it or choose a different branch name.
+            EO_MSG
+          end
+
+          # The directory is a worktree: ensure it tracks the requested branch.
+          worktree_branch = Git.open(dir).current_branch
+          if worktree_branch != branch_name
+            raise TaskStarterError, <<~EO_MSG.strip
+              Directory '#{dir}' is already a git worktree on branch '#{worktree_branch}', which differs from the requested branch '#{branch_name}'.
+              Please choose a different branch name or remove the existing worktree.
+            EO_MSG
+          end
+        else
+          # Call git worktree add on existing branches only
+          Helpers.git.lib.worktree_add(dir, branch_name)
+        end
+        # Push to remote if branch doesn't exist there yet
+        Helpers.git.push(Helpers.github_remote, branch_name, set_upstream: true)
+        Helpers.run_cmd("VSCodium.exe \"#{dir}\"")
+        { worktree_dir: dir }
+      end
+    end
+  end
+end

data/lib/x_aeon_agents/agents/tester_agent.rb

@@ -0,0 +1,44 @@
+module XAeonAgents
+  module Agents
+    # Agent responsible for fixing regressions induced by new features or fixes, while keeping initial requirements and implementation plan in mind.
+    # If decisions in the implementation plan prevent fixing regressions, modify the implementation plan and report those modifications.
+    class TesterAgent < ComposableAgents::Cline::Agent
+      prepend AgentDefaults
+
+      # Define input artifacts contracts
+      #
+      # @return [Hash{Symbol => Object}] Set of input artifacts description, per artifact name
+      def input_artifacts_contracts
+        super.merge(
+          requirements: 'The initial requirements',
+          plan: 'The implementation plan devised from the requirements',
+          files_diffs: 'The full list of files changes and differences that have been done to implement the initial ' \
+            'requirements following the implementation plan',
+          tests_output: 'The output of running the whole tests suite',
+          tests_cmd: 'The command line to be used to run the whole tests suite'
+        )
+      end
+
+      # Define output artifacts contracts
+      #
+      # @return [Hash{Symbol => Object}] Set of output artifacts description, per artifact name
+      def output_artifacts_contracts
+        super.merge(plan_modifications: 'The modification or divergence you considered from the implementation plan')
+      end
+
+      # Constructor
+      #
+      # @param agent_params [Hash{Symbol => Object}] Extra agent parameters
+      def initialize(**agent_params)
+        super(
+          name: 'Tester',
+          objective: <<~EO_OBJECTIVE,
+            Fix any regression that has been induced by new features or fixes, while keeping the initial requirements and implementation plan in mind.
+            If the decisions taken in the implementation plan prevent you from fixing regressions, modify the implementation plan and report those modifications to the user.
+          EO_OBJECTIVE
+          **agent_params
+        )
+      end
+    end
+  end
+end