aidp 0.16.0 → 0.17.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9bbab73392b0bc0fdcd74bb0f0e0bbf09716b2d2451b04aaa64c4ca4ccc7638e
-  data.tar.gz: af0d8b6d8937a1781b3b379ab6328ff17c5f0c96ea00c80f5886ce15b2265e2f
+  metadata.gz: 84451cdcc0886f1a70e799ab53d9b8898633f324cdd80c5f07a5f204848a3de2
+  data.tar.gz: e93982382c700564db24fac6ff1de2276e5b0301b91f6cade19aa8dfacd3f7d3
 SHA512:
-  metadata.gz: ee40afa8126a8f8d6f92a7df6fbc634c6eb054499bb7dfdec53946f05ecf8faf8c1bcae1eb02274a7441bdd7685751363e20207587df818c8bb8fe300bfbc50f
-  data.tar.gz: 010a5afdd1a06905f09a423efe3bfde2ef84aca498f939667871191b1c9174168c31807959cbc3d50e7f7ab2cc552301032c1d232f6e70331ee8918debb7aba5
+  metadata.gz: 3de783cb25c3eedc9056f77c2b294918a2f5ab46c009280897b6094dd157c45c681fc5c8fe22c6eea401ba74e0c4151c132a85e3c8ceedfef7722e9580f96f43
+  data.tar.gz: de86c77e08a8f9b837d501d8e69786b6f969c0e2a1538f1a9d2062267f01d6e8c855f0885c31d0bdc3f2c9362fa78439fccdcfe5e09a12cd1ab61282365cdc4a

data/lib/aidp/analyze/error_handler.rb

@@ -3,6 +3,18 @@
 require "logger"
 require_relative "../concurrency"
 
+begin
+  require "net/http"
+rescue LoadError
+  # Net::HTTP might not be available in all environments
+end
+
+begin
+  require "sqlite3"
+rescue LoadError
+  # SQLite3 might not be available in all environments
+end
+
 module Aidp
   module Analyze
     # Comprehensive error handling system for analyze mode
@@ -24,7 +36,7 @@ module Aidp
           context: context,
           step: step,
           retry_count: retry_count,
-          timestamp: Time.current
+          timestamp: Time.now
         }
 
         log_error(error_info)
@@ -59,7 +71,7 @@ module Aidp
         {
           status: "skipped",
           reason: error.message,
-          timestamp: Time.current
+          timestamp: Time.now
         }
       end
 
@@ -88,8 +100,8 @@ module Aidp
 
       def setup_logger(log_file, verbose)
         output_stream = log_file || @output || $stdout
-        logger = Logger.new(output_stream)
-        logger.level = verbose ? Logger::DEBUG : Logger::INFO
+        logger = ::Logger.new(output_stream)
+        logger.level = verbose ? ::Logger::DEBUG : ::Logger::INFO
         logger.formatter = proc do |severity, datetime, progname, msg|
           "#{datetime.strftime("%Y-%m-%d %H:%M:%S")} [#{severity}] #{msg}\n"
         end
@@ -97,19 +109,28 @@ module Aidp
       end
 
       def setup_recovery_strategies
-        {
-          Net::TimeoutError => :retry_with_backoff,
-          Net::HTTPError => :retry_with_backoff,
-          SocketError => :retry_with_backoff,
+        strategies = {
           Errno::ENOENT => :skip_step_with_warning,
           Errno::EACCES => :skip_step_with_warning,
           Errno::ENOSPC => :critical_error,
-          SQLite3::BusyException => :retry_with_backoff,
-          SQLite3::CorruptException => :critical_error,
           AnalysisTimeoutError => :chunk_and_retry,
           AnalysisDataError => :continue_with_partial_data,
           AnalysisToolError => :log_and_continue
         }
+
+        # Add network error classes if available
+        strategies[Net::TimeoutError] = :retry_with_backoff if defined?(Net::TimeoutError)
+
+        strategies[Net::HTTPError] = :retry_with_backoff if defined?(Net::HTTPError)
+
+        strategies[SocketError] = :retry_with_backoff if defined?(SocketError)
+
+        # Add SQLite error classes if available
+        strategies[SQLite3::BusyException] = :retry_with_backoff if defined?(SQLite3::BusyException)
+
+        strategies[SQLite3::CorruptException] = :critical_error if defined?(SQLite3::CorruptException)
+
+        strategies
       end
 
       def log_error(error_info)
@@ -250,9 +271,7 @@ module Aidp
         tool_name = context[:tool_name] || "analysis tool"
         error_msg = "#{tool_name} failed: #{error.message}"
 
-        if context[:installation_guide]
-          error_msg += "\n\nTo install #{tool_name}:\n#{context[:installation_guide]}"
-        end
+        error_msg += "\n\nTo install #{tool_name}:\n#{context[:installation_guide]}" if context[:installation_guide]
 
         raise AnalysisToolError.new(error_msg)
       end

data/lib/aidp/analyze/progress.rb

@@ -67,7 +67,7 @@ module Aidp
         end
 
         @progress = if File.exist?(@progress_file)
-          YAML.load_file(@progress_file) || {}
+          YAML.safe_load_file(@progress_file, permitted_classes: [Date, Time, Symbol], aliases: true) || {}
         else
           {}
         end

data/lib/aidp/cli.rb

@@ -241,7 +241,7 @@ module Aidp
 
         if File.exist?(config_path)
           require "yaml"
-          full_config = YAML.load_file(config_path)
+          full_config = YAML.safe_load_file(config_path, permitted_classes: [Date, Time, Symbol], aliases: true)
           logging_config = full_config["logging"] || full_config[:logging] || {}
         end
 
@@ -1722,10 +1722,10 @@ module Aidp
 
             by_source = registry.by_source
 
-            if by_source[:builtin].any?
-              display_message("Built-in Skills", type: :highlight)
+            if by_source[:template].any?
+              display_message("Template Skills", type: :highlight)
               display_message("=" * 80, type: :muted)
-              table_rows = by_source[:builtin].map do |skill_id|
+              table_rows = by_source[:template].map do |skill_id|
                 skill = registry.find(skill_id)
                 [skill_id, skill.version, skill.description[0, 60]]
               end
@@ -1735,10 +1735,10 @@ module Aidp
               display_message("", type: :info)
             end
 
-            if by_source[:custom].any?
-              display_message("Custom Skills", type: :highlight)
+            if by_source[:project].any?
+              display_message("Project Skills", type: :highlight)
               display_message("=" * 80, type: :muted)
-              table_rows = by_source[:custom].map do |skill_id|
+              table_rows = by_source[:project].map do |skill_id|
                 skill = registry.find(skill_id)
                 [skill_id, skill.version, skill.description[0, 60]]
               end
@@ -1846,6 +1846,204 @@ module Aidp
             display_message("Failed to search skills: #{e.message}", type: :error)
           end
 
+        when "preview"
+          # Preview full skill content
+          skill_id = args.shift
+
+          unless skill_id
+            display_message("Usage: aidp skill preview <skill-id>", type: :info)
+            return
+          end
+
+          begin
+            registry = Aidp::Skills::Registry.new(project_dir: Dir.pwd)
+            registry.load_skills
+
+            skill = registry.find(skill_id)
+
+            unless skill
+              display_message("Skill not found: #{skill_id}", type: :error)
+              display_message("Use 'aidp skill list' to see available skills", type: :muted)
+              return
+            end
+
+            require_relative "skills/wizard/builder"
+            require_relative "skills/wizard/template_library"
+
+            builder = Aidp::Skills::Wizard::Builder.new
+            full_content = builder.to_skill_md(skill)
+
+            # Check if this is a project skill with a matching template
+            source_info = registry.by_source[skill_id]
+            inheritance_info = ""
+            if source_info == :project
+              template_library = Aidp::Skills::Wizard::TemplateLibrary.new(project_dir: Dir.pwd)
+              template_skill = template_library.templates.find { |s| s.id == skill.id }
+              if template_skill
+                inheritance_info = " (inherits from template)"
+              end
+            elsif source_info == :template
+              inheritance_info = " (template)"
+            end
+
+            display_message("\n" + "=" * 60, type: :info)
+            display_message("Skill: #{skill.name} (#{skill.id}) v#{skill.version}#{inheritance_info}", type: :highlight)
+            display_message("=" * 60 + "\n", type: :info)
+            display_message(full_content, type: :info)
+            display_message("\n" + "=" * 60, type: :info)
+          rescue => e
+            display_message("Failed to preview skill: #{e.message}", type: :error)
+          end
+
+        when "diff"
+          # Show diff between project skill and template
+          skill_id = args.shift
+
+          unless skill_id
+            display_message("Usage: aidp skill diff <skill-id>", type: :info)
+            return
+          end
+
+          begin
+            require_relative "skills/wizard/template_library"
+            require_relative "skills/wizard/differ"
+
+            registry = Aidp::Skills::Registry.new(project_dir: Dir.pwd)
+            registry.load_skills
+
+            project_skill = registry.find(skill_id)
+
+            unless project_skill
+              display_message("Skill not found: #{skill_id}", type: :error)
+              return
+            end
+
+            # Check if it's a project skill
+            unless registry.by_source[:project].include?(skill_id)
+              display_message("Skill '#{skill_id}' is a template skill, not a project skill", type: :info)
+              display_message("Only project skills can be diffed against templates", type: :muted)
+              return
+            end
+
+            # Find the template
+            template_library = Aidp::Skills::Wizard::TemplateLibrary.new(project_dir: Dir.pwd)
+            template_skill = template_library.find(skill_id)
+
+            unless template_skill
+              display_message("No template found for skill '#{skill_id}'", type: :info)
+              display_message("This is a custom skill without a template base", type: :muted)
+              return
+            end
+
+            # Show diff
+            differ = Aidp::Skills::Wizard::Differ.new
+            diff_result = differ.diff(template_skill, project_skill)
+            differ.display(diff_result)
+          rescue => e
+            display_message("Failed to diff skill: #{e.message}", type: :error)
+          end
+
+        when "edit"
+          # Edit an existing skill
+          skill_id = args.shift
+
+          unless skill_id
+            display_message("Usage: aidp skill edit <skill-id>", type: :info)
+            return
+          end
+
+          begin
+            require_relative "skills/wizard/controller"
+
+            registry = Aidp::Skills::Registry.new(project_dir: Dir.pwd)
+            registry.load_skills
+
+            skill = registry.find(skill_id)
+
+            unless skill
+              display_message("Skill not found: #{skill_id}", type: :error)
+              display_message("Use 'aidp skill list' to see available skills", type: :muted)
+              return
+            end
+
+            # Check if it's editable (must be project skill or willing to copy template)
+            if registry.by_source[:template].include?(skill_id)
+              display_message("'#{skill_id}' is a template skill", type: :info)
+              display_message("Editing will create a project override in .aidp/skills/", type: :muted)
+            end
+
+            # Parse options
+            options = {}
+            while args.first&.start_with?("--")
+              opt = args.shift
+              case opt
+              when "--dry-run"
+                options[:dry_run] = true
+              when "--open-editor"
+                options[:open_editor] = true
+              else
+                display_message("Unknown option: #{opt}", type: :error)
+                return
+              end
+            end
+
+            # Pre-fill wizard with existing skill data
+            options[:id] = skill.id
+            options[:name] = skill.name
+            options[:edit_mode] = true
+            options[:existing_skill] = skill
+
+            # Run wizard in edit mode
+            wizard = Aidp::Skills::Wizard::Controller.new(
+              project_dir: Dir.pwd,
+              options: options
+            )
+            wizard.run
+          rescue => e
+            display_message("Failed to edit skill: #{e.message}", type: :error)
+          end
+
+        when "new"
+          # Create a new skill using the wizard
+          begin
+            require_relative "skills/wizard/controller"
+
+            # Parse options
+            options = {}
+            while args.first&.start_with?("--")
+              opt = args.shift
+              case opt
+              when "--minimal"
+                options[:minimal] = true
+              when "--dry-run"
+                options[:dry_run] = true
+              when "--yes", "-y"
+                options[:yes] = true
+              when "--id"
+                options[:id] = args.shift
+              when "--name"
+                options[:name] = args.shift
+              when "--from-template"
+                options[:from_template] = args.shift
+              when "--clone"
+                options[:clone] = args.shift
+              else
+                display_message("Unknown option: #{opt}", type: :error)
+                return
+              end
+            end
+
+            # Run wizard
+            wizard = Aidp::Skills::Wizard::Controller.new(
+              project_dir: Dir.pwd,
+              options: options
+            )
+            wizard.run
+          rescue => e
+            display_message("Failed to create skill: #{e.message}", type: :error)
+            Aidp.log_error("cli", "Skill wizard failed", error: e.message, backtrace: e.backtrace.first(5))
+          end
+
         when "validate"
           # Validate skill file format
           skill_path = args.shift
@@ -1893,19 +2091,94 @@ module Aidp
             end
           end
 
+        when "delete"
+          # Delete a project skill
+          skill_id = args.shift
+
+          unless skill_id
+            display_message("Usage: aidp skill delete <skill-id>", type: :info)
+            return
+          end
+
+          begin
+            registry = Aidp::Skills::Registry.new(project_dir: Dir.pwd)
+            registry.load_skills
+
+            skill = registry.find(skill_id)
+
+            unless skill
+              display_message("Skill not found: #{skill_id}", type: :error)
+              return
+            end
+
+            # Check if it's a project skill
+            source = registry.by_source[skill_id]
+            unless source == :project
+              display_message("Cannot delete template skill '#{skill_id}'", type: :error)
+              display_message("Only project skills in .aidp/skills/ can be deleted", type: :muted)
+              return
+            end
+
+            # Get skill directory
+            skill_dir = File.dirname(skill.source_path)
+
+            # Confirm deletion
+            require "tty-prompt"
+            prompt = TTY::Prompt.new
+            confirmed = prompt.yes?("Delete skill '#{skill.name}' (#{skill_id})? This cannot be undone.")
+
+            unless confirmed
+              display_message("Deletion cancelled", type: :info)
+              return
+            end
+
+            # Delete the skill directory
+            require "fileutils"
+            FileUtils.rm_rf(skill_dir)
+
+            display_message("✓ Deleted skill: #{skill.name} (#{skill_id})", type: :success)
+          rescue => e
+            display_message("Failed to delete skill: #{e.message}", type: :error)
+            Aidp.log_error("cli", "Skill deletion failed", error: e.message, backtrace: e.backtrace.first(5))
+          end
+
         else
           display_message("Usage: aidp skill <command>", type: :info)
           display_message("", type: :info)
           display_message("Commands:", type: :info)
           display_message("  list                List all available skills (default)", type: :info)
           display_message("  show <id>           Show detailed skill information", type: :info)
+          display_message("  preview <id>        Preview full SKILL.md content", type: :info)
+          display_message("  diff <id>           Show diff between project skill and template", type: :info)
           display_message("  search <query>      Search skills by keyword", type: :info)
+          display_message("  new [options]       Create a new skill using the wizard", type: :info)
+          display_message("  edit <id> [options] Edit an existing skill", type: :info)
+          display_message("  delete <id>         Delete a project skill", type: :info)
           display_message("  validate [path]     Validate skill file format", type: :info)
           display_message("", type: :info)
+          display_message("New Skill Options:", type: :info)
+          display_message("  --minimal           Skip optional sections", type: :info)
+          display_message("  --dry-run           Preview without saving", type: :info)
+          display_message("  --yes, -y           Skip confirmation prompts", type: :info)
+          display_message("  --id <skill_id>     Pre-set skill ID", type: :info)
+          display_message("  --name <name>       Pre-set skill name", type: :info)
+          display_message("", type: :info)
+          display_message("Edit Skill Options:", type: :info)
+          display_message("  --dry-run           Preview changes without saving", type: :info)
+          display_message("  --open-editor       Open content in $EDITOR", type: :info)
+          display_message("", type: :info)
           display_message("Examples:", type: :info)
           display_message("  aidp skill list                                # List all skills", type: :info)
           display_message("  aidp skill show repository_analyst             # Show skill details", type: :info)
+          display_message("  aidp skill preview repository_analyst          # Preview full content", type: :info)
+          display_message("  aidp skill diff my_skill                       # Show diff with template", type: :info)
           display_message("  aidp skill search git                          # Search for git-related skills", type: :info)
+          display_message("  aidp skill new                                 # Create new skill (interactive)", type: :info)
+          display_message("  aidp skill new --minimal --id my_skill         # Create with minimal prompts", type: :info)
+          display_message("  aidp skill new --from-template repo_analyst    # Inherit from template", type: :info)
+          display_message("  aidp skill new --clone my_existing_skill       # Clone existing skill", type: :info)
+          display_message("  aidp skill edit repository_analyst             # Edit existing skill", type: :info)
+          display_message("  aidp skill delete my_custom_skill              # Delete a project skill", type: :info)
           display_message("  aidp skill validate skills/my_skill/SKILL.md   # Validate specific skill", type: :info)
           display_message("  aidp skill validate                            # Validate all skills", type: :info)
         end

data/lib/aidp/config.rb

@@ -312,7 +312,7 @@ module Aidp
     end
 
     private_class_method def self.load_yaml_config(config_file)
-      YAML.load_file(config_file) || {}
+      YAML.safe_load_file(config_file, permitted_classes: [Date, Time, Symbol], aliases: true) || {}
     rescue => e
       warn "Failed to load configuration file #{config_file}: #{e.message}"
       {}

data/lib/aidp/execute/async_work_loop_runner.rb

@@ -26,6 +26,7 @@ module Aidp
         @provider_manager = provider_manager
         @config = config
         @options = options
+        @cancel_timeout = options[:cancel_timeout] || 5 # seconds to wait for graceful shutdown
         @state = WorkLoopState.new
         @instruction_queue = InstructionQueue.new
         @work_thread = nil
@@ -92,7 +93,7 @@ module Aidp
         @state.append_output("Cancellation requested, waiting for safe stopping point...", type: :warning)
 
         # Wait for thread to notice cancellation
-        @work_thread&.join(5) # 5 second timeout
+        @work_thread&.join(@cancel_timeout)
 
         if save_checkpoint && @sync_runner
           @state.append_output("Saving checkpoint before exit...", type: :info)

data/lib/aidp/execute/checkpoint.rb

@@ -40,7 +40,7 @@ module Aidp
       # Get the latest checkpoint data
       def latest_checkpoint
         return nil unless File.exist?(@checkpoint_file)
-        YAML.load_file(@checkpoint_file)
+        YAML.safe_load_file(@checkpoint_file, permitted_classes: [Date, Time, Symbol], aliases: true)
       end
 
       # Get checkpoint history for analysis

data/lib/aidp/execute/future_work_backlog.rb

@@ -218,7 +218,7 @@ module Aidp
       def load_existing_backlog
         return unless File.exist?(@backlog_file)
 
-        data = YAML.load_file(@backlog_file)
+        data = YAML.safe_load_file(@backlog_file, permitted_classes: [Date, Time, Symbol], aliases: true)
         @entries = data["entries"] || [] if data.is_a?(Hash)
         @entries = symbolize_keys_deep(@entries)
       rescue => e

data/lib/aidp/execute/progress.rb

@@ -68,7 +68,7 @@ module Aidp
         end
 
         @progress = if File.exist?(@progress_file)
-          YAML.load_file(@progress_file) || {}
+          YAML.safe_load_file(@progress_file, permitted_classes: [Date, Time, Symbol], aliases: true) || {}
         else
           {}
         end

data/lib/aidp/execute/repl_macros.rb

@@ -9,7 +9,7 @@ module Aidp
     # - /split - Divide work into smaller contracts
     # - /halt-on <pattern> - Pause on specific test failures
     class ReplMacros
-      attr_reader :pinned_files, :focus_patterns, :halt_patterns, :split_mode, :current_workstream
+      attr_reader :pinned_files, :focus_patterns, :halt_patterns, :split_mode, :current_workstream, :current_skill
 
       def initialize(project_dir: Dir.pwd)
         @pinned_files = Set.new
@@ -18,6 +18,7 @@ module Aidp
         @split_mode = false
         @project_dir = project_dir
         @current_workstream = nil
+        @current_skill = nil
         @commands = register_commands
       end
 
@@ -65,6 +66,7 @@ module Aidp
           halt_patterns: @halt_patterns,
           split_mode: @split_mode,
           current_workstream: @current_workstream,
+          current_skill: @current_skill,
           active_constraints: active_constraints_count
         }
       end
@@ -113,6 +115,30 @@ module Aidp
         true
       end
 
+      # Retrieve the current skill object, or nil if none is selected
+      #
+      # This method provides access to the full skill object (with content, providers, etc.)
+      # for the currently selected skill via `/skill use <id>`.
+      #
+      # @return [Aidp::Skills::Skill, nil] The current skill object or nil
+      #
+      # @example
+      #   repl = ReplMacros.new(project_dir: Dir.pwd)
+      #   repl.execute("/skill use repository_analyst")
+      #   skill = repl.current_skill_object
+      #   puts skill.content if skill  # => skill's markdown content
+      def current_skill_object
+        return nil unless @current_skill
+
+        require_relative "../skills"
+        registry = Aidp::Skills::Registry.new(project_dir: @project_dir)
+        registry.load_skills
+        registry.find(@current_skill)
+      rescue => e
+        Aidp.log_error("repl_macros", "Failed to load current skill object", error: e.message)
+        nil
+      end
+
       private
 
       # Register all available REPL commands
@@ -1257,25 +1283,25 @@ module Aidp
             lines = ["Available Skills:", ""]
             by_source = registry.by_source
 
-            if by_source[:builtin].any?
-              lines << "Built-in Skills:"
-              by_source[:builtin].each do |skill_id|
+            if by_source[:template].any?
+              lines << "Template Skills:"
+              by_source[:template].each do |skill_id|
                 skill = registry.find(skill_id)
                 lines << "  • #{skill_id} - #{skill.description}"
               end
               lines << ""
             end
 
-            if by_source[:custom].any?
-              lines << "Custom Skills:"
-              by_source[:custom].each do |skill_id|
+            if by_source[:project].any?
+              lines << "Project Skills:"
+              by_source[:project].each do |skill_id|
                 skill = registry.find(skill_id)
-                lines << "  • #{skill_id} - #{skill.description} [CUSTOM]"
+                lines << "  • #{skill_id} - #{skill.description} [PROJECT]"
               end
               lines << ""
             end
 
-            lines << "Use '/skill show <id>' for details"
+            lines << "Use '/skill show <id>' for details or '/skill use <id>' to activate"
 
             {
               success: true,
@@ -1412,10 +1438,53 @@ module Aidp
             }
           end
 
+        when "use"
+          # Switch to a specific skill
+          skill_id = args.shift
+
+          unless skill_id
+            return {
+              success: false,
+              message: "Usage: /skill use <skill-id>",
+              action: :none
+            }
+          end
+
+          begin
+            registry = Aidp::Skills::Registry.new(project_dir: @project_dir)
+            registry.load_skills
+
+            skill = registry.find(skill_id)
+
+            unless skill
+              return {
+                success: false,
+                message: "Skill not found: #{skill_id}\nUse '/skill list' to see available skills",
+                action: :none
+              }
+            end
+
+            # Store the current skill for the session
+            @current_skill = skill_id
+
+            {
+              success: true,
+              message: "✓ Now using skill: #{skill.name} (#{skill_id})\n\n#{skill.description}",
+              action: :switch_skill,
+              data: {skill_id: skill_id, skill: skill}
+            }
+          rescue => e
+            {
+              success: false,
+              message: "Failed to switch skill: #{e.message}",
+              action: :none
+            }
+          end
+
         else
           {
             success: false,
-            message: "Usage: /skill <command> [args]\n\nCommands:\n  list           - List all available skills\n  show <id>      - Show detailed skill information\n  search <query> - Search skills by keyword\n\nExamples:\n  /skill list\n  /skill show repository_analyst\n  /skill search git",
+            message: "Usage: /skill <command> [args]\n\nCommands:\n  list           - List all available skills\n  show <id>      - Show detailed skill information\n  search <query> - Search skills by keyword\n  use <id>       - Switch to a specific skill\n\nExamples:\n  /skill list\n  /skill show repository_analyst\n  /skill search git\n  /skill use repository_analyst",
             action: :none
           }
         end

data/lib/aidp/harness/config_validator.rb

@@ -256,7 +256,7 @@ module Aidp
         return unless @config_file
 
         begin
-          @config = YAML.load_file(@config_file) || {}
+          @config = YAML.safe_load_file(@config_file, permitted_classes: [Date, Time, Symbol], aliases: true) || {}
         rescue => e
           @config = {}
           @validation_result = {

data/lib/aidp/jobs/background_runner.rb

@@ -213,7 +213,7 @@ module Aidp
         return nil unless File.exist?(metadata_file)
 
         # Return raw metadata with times as ISO8601 strings to avoid unsafe class loading
-        YAML.load_file(metadata_file)
+        YAML.safe_load_file(metadata_file, permitted_classes: [Date, Time, Symbol], aliases: true)
       rescue
         nil
       end