rufio 0.61.0 → 0.62.0

data/lib/rufio/config.rb

@@ -1,11 +1,24 @@
 # frozen_string_literal: true
 
+require 'yaml'
+require 'fileutils'
+
 module Rufio
   class Config
     # Default language settings
     DEFAULT_LANGUAGE = 'en'
     AVAILABLE_LANGUAGES = %w[en ja].freeze
 
+    # 設定ディレクトリとファイルパス
+    CONFIG_DIR = File.expand_path('~/.config/rufio').freeze
+    CONFIG_RB_PATH = File.join(CONFIG_DIR, 'config.rb').freeze
+    SCRIPT_PATHS_YML = File.join(CONFIG_DIR, 'script_paths.yml').freeze
+    BOOKMARKS_YML = File.join(CONFIG_DIR, 'bookmarks.yml').freeze
+
+    # 後方互換性のためのパス（非推奨）
+    YAML_CONFIG_PATH = File.join(CONFIG_DIR, 'config.yml').freeze
+    LOCAL_YAML_PATH = './rufio.yml'
+
     # Multi-language message definitions
     MESSAGES = {
       'en' => {
@@ -179,8 +192,209 @@ module Rufio
         @current_language = nil
       end
 
+      # YAML設定を取得（キャッシュあり）
+      # @return [Hash] YAML設定
+      def yaml_config
+        @yaml_config ||= load_yaml_config(YAML_CONFIG_PATH)
+      end
+
+      # YAML設定をリロード
+      def reload_yaml_config!
+        @yaml_config = nil
+      end
+
+      # 全設定をリセット
+      def reset_config!
+        @yaml_config = nil
+        @script_paths = nil
+        @bookmarks = nil
+      end
+
+      # ========================================
+      # script_paths.yml の操作
+      # ========================================
+
+      # スクリプトパスを読み込む
+      # @param path [String] YAMLファイルのパス
+      # @return [Array<String>] 展開済みのパス配列
+      def load_script_paths(path = SCRIPT_PATHS_YML)
+        return [] unless File.exist?(path)
+
+        yaml = YAML.safe_load(File.read(path))
+        return [] unless yaml.is_a?(Array)
+
+        yaml.map { |p| File.expand_path(p) }
+      rescue StandardError => e
+        warn "Failed to load script paths: #{e.message}"
+        []
+      end
+
+      # スクリプトパスを保存
+      # @param path [String] YAMLファイルのパス
+      # @param paths [Array<String>] パス配列
+      def save_script_paths(path, paths)
+        ensure_config_directory(path)
+        File.write(path, YAML.dump(paths))
+      end
+
+      # スクリプトパスを追加
+      # @param path [String] YAMLファイルのパス
+      # @param new_path [String] 追加するパス
+      def add_script_path(path, new_path)
+        paths = load_script_paths(path)
+        expanded = File.expand_path(new_path)
+        return if paths.include?(expanded)
+
+        paths << expanded
+        save_script_paths(path, paths)
+      end
+
+      # スクリプトパスを削除
+      # @param path [String] YAMLファイルのパス
+      # @param remove_path [String] 削除するパス
+      def remove_script_path(path, remove_path)
+        paths = load_script_paths(path)
+        expanded = File.expand_path(remove_path)
+        paths.delete(expanded)
+        save_script_paths(path, paths)
+      end
+
+      # ========================================
+      # bookmarks.yml の操作
+      # ========================================
+
+      # ブックマークを読み込む
+      # @param path [String] YAMLファイルのパス
+      # @return [Array<Hash>] ブックマーク配列
+      def load_bookmarks_from_yml(path = BOOKMARKS_YML)
+        return [] unless File.exist?(path)
+
+        yaml = YAML.safe_load(File.read(path), symbolize_names: true)
+        return [] unless yaml.is_a?(Array)
+
+        yaml.select { |b| b.is_a?(Hash) && b[:path] && b[:name] }
+      rescue StandardError => e
+        warn "Failed to load bookmarks: #{e.message}"
+        []
+      end
+
+      # ブックマークを保存
+      # @param path [String] YAMLファイルのパス
+      # @param bookmarks [Array<Hash>] ブックマーク配列
+      def save_bookmarks_to_yml(path, bookmarks)
+        ensure_config_directory(path)
+        data = bookmarks.map { |b| { 'path' => b[:path], 'name' => b[:name] } }
+        File.write(path, YAML.dump(data))
+      end
+
+      # ブックマークを追加
+      # @param path [String] YAMLファイルのパス
+      # @param bookmark_path [String] ブックマークするパス
+      # @param name [String] ブックマーク名
+      def add_bookmark(path, bookmark_path, name)
+        bookmarks = load_bookmarks_from_yml(path)
+        bookmarks << { path: bookmark_path, name: name }
+        save_bookmarks_to_yml(path, bookmarks)
+      end
+
+      # ブックマークを削除
+      # @param path [String] YAMLファイルのパス
+      # @param name [String] 削除するブックマーク名
+      def remove_bookmark(path, name)
+        bookmarks = load_bookmarks_from_yml(path)
+        bookmarks.reject! { |b| b[:name] == name }
+        save_bookmarks_to_yml(path, bookmarks)
+      end
+
+      # ========================================
+      # config.rb (DSL) の操作
+      # ========================================
+
+      # config.rb を読み込む
+      # @param path [String] config.rb のパス
+      def load_config_rb(path = CONFIG_RB_PATH)
+        return unless File.exist?(path)
+
+        load path
+      rescue StandardError => e
+        warn "Failed to load config.rb: #{e.message}"
+      end
+
+      # ========================================
+      # マイグレーション
+      # ========================================
+
+      # 古い config.yml から新形式にマイグレーション
+      # @param old_config_yml [String] 古い config.yml のパス
+      # @param script_paths_yml [String] 新しい script_paths.yml のパス
+      # @param bookmarks_yml [String] 新しい bookmarks.yml のパス
+      def migrate_from_config_yml(old_config_yml, script_paths_yml, bookmarks_yml)
+        return unless File.exist?(old_config_yml)
+
+        old_config = load_yaml_config(old_config_yml)
+
+        # script_paths のマイグレーション
+        if old_config[:script_paths].is_a?(Array) && !old_config[:script_paths].empty?
+          save_script_paths(script_paths_yml, old_config[:script_paths])
+        end
+
+        # bookmarks のマイグレーション
+        if old_config[:bookmarks].is_a?(Array) && !old_config[:bookmarks].empty?
+          save_bookmarks_to_yml(bookmarks_yml, old_config[:bookmarks])
+        end
+      end
+
+      # YAML設定ファイルを読み込む
+      # @param path [String] 設定ファイルのパス
+      # @return [Hash] 設定内容（シンボルキー）
+      def load_yaml_config(path)
+        return {} unless File.exist?(path)
+
+        yaml = YAML.safe_load(File.read(path), symbolize_names: true)
+        yaml || {}
+      rescue StandardError => e
+        warn "Failed to load YAML config: #{e.message}"
+        {}
+      end
+
+      # YAML設定ファイルにセクションを保存
+      # @param path [String] 設定ファイルのパス
+      # @param key [Symbol, String] 保存するキー
+      # @param value [Object] 保存する値
+      def save_yaml_config(path, key, value)
+        ensure_config_directory(path)
+
+        existing = if File.exist?(path)
+                     YAML.safe_load(File.read(path), symbolize_names: true) || {}
+                   else
+                     {}
+                   end
+
+        existing[key.to_sym] = value
+        File.write(path, YAML.dump(stringify_keys(existing)))
+        reload_yaml_config! if path == YAML_CONFIG_PATH
+      end
+
       private
 
+      def ensure_config_directory(path)
+        dir = File.dirname(path)
+        FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
+      end
+
+      def stringify_keys(hash)
+        hash.transform_keys(&:to_s).transform_values do |value|
+          case value
+          when Hash
+            stringify_keys(value)
+          when Array
+            value.map { |v| v.is_a?(Hash) ? stringify_keys(v) : v }
+          else
+            value
+          end
+        end
+      end
+
       def detect_language
         # Only BENIYA_LANG environment variable takes precedence
         # This ensures English is default unless explicitly requested

data/lib/rufio/config_loader.rb

@@ -1,14 +1,19 @@
 # frozen_string_literal: true
 
 require 'yaml'
+require 'json'
 require_relative 'config'
-require_relative 'bookmark_storage'
 
 module Rufio
   class ConfigLoader
-    CONFIG_PATH = File.expand_path('~/.config/rufio/config.rb').freeze
-    YAML_CONFIG_PATH = File.expand_path('~/.config/rufio/config.yml').freeze
-    JSON_BOOKMARKS_PATH = File.expand_path('~/.config/rufio/bookmarks.json').freeze
+    # 新しいパス定数（Configから取得）
+    CONFIG_PATH = Config::CONFIG_RB_PATH
+    SCRIPT_PATHS_YML = Config::SCRIPT_PATHS_YML
+    BOOKMARKS_YML = Config::BOOKMARKS_YML
+
+    # 後方互換性のためのパス（非推奨）
+    YAML_CONFIG_PATH = Config::YAML_CONFIG_PATH
+    LOCAL_YAML_PATH = Config::LOCAL_YAML_PATH
 
     class << self
       def load_config
@@ -21,6 +26,7 @@ module Rufio
 
       def reload_config!
         @config = nil
+        @script_paths = nil
         load_config
       end
 
@@ -42,7 +48,6 @@ module Rufio
 
       def set_language(lang)
         Config.current_language = lang
-        # Update config if it's user-defined
         if @config
           @config[:language] = lang
         end
@@ -64,12 +69,10 @@ module Rufio
         load_config[:command_history_size] || 1000
       end
 
-      # スクリプトパスの配列を取得
+      # スクリプトパスの配列を取得（ローカル > ユーザー設定の優先順位でマージ）
       # @return [Array<String>] 展開済みのスクリプトパス
       def script_paths
-        yaml_config = load_yaml_config
-        paths = yaml_config[:script_paths] || default_script_paths
-        expand_script_paths(paths)
+        @script_paths ||= load_merged_script_paths
       end
 
       # デフォルトのスクリプトパス
@@ -85,58 +88,159 @@ module Rufio
         paths.map { |p| File.expand_path(p) }
       end
 
-      # YAML設定ファイルを読み込む
+      # スクリプトパスを追加
+      # @param path [String] 追加するパス
+      # @return [Boolean] 追加成功したか
+      def add_script_path(path)
+        expanded = File.expand_path(path)
+        current = script_paths
+        return false if current.include?(expanded)
+
+        save_script_paths_to_yaml(current + [expanded])
+        @script_paths = nil
+        true
+      end
+
+      # スクリプトパスを削除
+      # @param path [String] 削除するパス
+      # @return [Boolean] 削除成功したか
+      def remove_script_path(path)
+        expanded = File.expand_path(path)
+        current = script_paths
+        return false unless current.include?(expanded)
+
+        save_script_paths_to_yaml(current - [expanded])
+        @script_paths = nil
+        true
+      end
+
+      # YAML設定ファイルを読み込む（Config経由）
       # @param path [String, nil] 設定ファイルのパス（nilの場合はデフォルト）
       # @return [Hash] 設定内容
       def load_yaml_config(path = nil)
         config_path = path || YAML_CONFIG_PATH
-        return {} unless File.exist?(config_path)
+        Config.load_yaml_config(config_path)
+      end
+
+      # ブックマークを読み込む（新形式: bookmarks.yml）
+      # @return [Array<Hash>] ブックマークの配列
+      def load_bookmarks
+        # 新形式を優先
+        bookmarks = Config.load_bookmarks_from_yml(BOOKMARKS_YML)
+        return bookmarks unless bookmarks.empty?
 
-        yaml = YAML.safe_load(File.read(config_path), symbolize_names: true)
-        yaml || {}
+        # 後方互換: 古いconfig.ymlから読み込み
+        yaml_config = load_yaml_config
+        filter_valid_bookmarks(yaml_config[:bookmarks] || [])
+      end
+
+      # ブックマークを保存（新形式: bookmarks.yml）
+      # @param bookmarks [Array<Hash>] ブックマークの配列
+      # @return [Boolean] 保存成功したか
+      def save_bookmarks(bookmarks)
+        Config.save_bookmarks_to_yml(BOOKMARKS_YML, bookmarks)
+        true
       rescue StandardError => e
-        warn "Failed to load YAML config: #{e.message}"
-        {}
+        warn "Failed to save bookmarks: #{e.message}"
+        false
       end
 
-      # ブックマーク用のYAMLストレージを取得
-      # @return [YamlBookmarkStorage] ストレージインスタンス
+      # ブックマークストレージを取得
+      # @return [YamlBookmarkStorage] ブックマークストレージ
       def bookmark_storage
-        YamlBookmarkStorage.new(YAML_CONFIG_PATH)
+        @bookmark_storage ||= YamlBookmarkStorage.new(BOOKMARKS_YML)
       end
 
-      # 必要に応じてブックマークをJSONからYAMLに移行
-      # @param json_path [String] JSONファイルパス
-      # @param yaml_path [String] YAMLファイルパス
-      # @return [Boolean] 移行が実行されたかどうか
-      def migrate_bookmarks_if_needed(json_path = JSON_BOOKMARKS_PATH, yaml_path = YAML_CONFIG_PATH)
-        BookmarkMigrator.migrate(json_path, yaml_path)
+      # 古いconfig.ymlからのマイグレーション
+      def migrate_bookmarks_if_needed
+        # 新形式が存在する場合はスキップ
+        return if File.exist?(BOOKMARKS_YML)
+        return unless File.exist?(YAML_CONFIG_PATH)
+
+        Config.migrate_from_config_yml(YAML_CONFIG_PATH, SCRIPT_PATHS_YML, BOOKMARKS_YML)
       end
 
       private
 
+      # マージされたスクリプトパスを読み込む
+      # @return [Array<String>] マージされたスクリプトパス
+      def load_merged_script_paths
+        paths = []
+        seen = Set.new
+
+        # ローカル設定（優先）
+        if File.exist?(LOCAL_YAML_PATH)
+          local_config = load_yaml_config(LOCAL_YAML_PATH)
+          local_paths = local_config[:script_paths] || []
+          add_unique_paths(paths, seen, expand_script_paths(local_paths))
+        end
+
+        # 新形式: script_paths.yml から読み込み
+        script_paths_from_yml = Config.load_script_paths(SCRIPT_PATHS_YML)
+        add_unique_paths(paths, seen, script_paths_from_yml)
+
+        # 後方互換: 古いconfig.ymlから読み込み
+        if paths.empty?
+          user_config = load_yaml_config
+          user_paths = user_config[:script_paths] || []
+          add_unique_paths(paths, seen, expand_script_paths(user_paths))
+        end
+
+        # デフォルトパス（何も設定されていない場合）
+        if paths.empty?
+          add_unique_paths(paths, seen, default_script_paths)
+        end
+
+        paths
+      end
+
+      # ユニークなパスを追加
+      def add_unique_paths(paths, seen, new_paths)
+        new_paths.each do |path|
+          expanded = File.expand_path(path)
+          next if seen.include?(expanded)
+
+          seen.add(expanded)
+          paths << expanded
+        end
+      end
+
+      # スクリプトパスをYAMLに保存（新形式: script_paths.yml）
+      def save_script_paths_to_yaml(paths)
+        Config.save_script_paths(SCRIPT_PATHS_YML, paths)
+      end
+
+      # YAMLファイルにセクションを保存（Config経由）
+      def save_to_yaml(key, value)
+        Config.save_yaml_config(YAML_CONFIG_PATH, key, value)
+      end
+
+      def filter_valid_bookmarks(bookmarks)
+        return [] unless bookmarks.is_a?(Array)
+
+        bookmarks.select do |b|
+          b.is_a?(Hash) && b[:path] && b[:name]
+        end
+      end
+
       def load_config_file
-        # 設定ファイルを実行してグローバル定数を定義
         load CONFIG_PATH
         config = {
-          applications: Object.const_get(:APPLICATIONS),
-          colors: Object.const_get(:COLORS),
-          keybinds: Object.const_get(:KEYBINDS)
+          applications: safe_const_get(:APPLICATIONS, default_config[:applications]),
+          colors: safe_const_get(:COLORS, default_config[:colors]),
+          keybinds: safe_const_get(:KEYBINDS, default_config[:keybinds])
         }
 
-        # Load language setting if defined
         if Object.const_defined?(:LANGUAGE)
           language = Object.const_get(:LANGUAGE)
           config[:language] = language
           Config.current_language = language if Config.available_languages.include?(language.to_s)
         end
 
-        # Load scripts directory if defined
         if Object.const_defined?(:SCRIPTS_DIR)
           config[:scripts_dir] = Object.const_get(:SCRIPTS_DIR)
         end
 
-        # Load command history size if defined
         if Object.const_defined?(:COMMAND_HISTORY_SIZE)
           config[:command_history_size] = Object.const_get(:COMMAND_HISTORY_SIZE)
         end
@@ -148,6 +252,10 @@ module Rufio
         default_config
       end
 
+      def safe_const_get(name, default)
+        Object.const_defined?(name) ? Object.const_get(name) : default
+      end
+
       def default_config
         {
           applications: {
@@ -159,11 +267,11 @@ module Rufio
             :default => 'open'
           },
           colors: {
-            directory: { hsl: [220, 80, 60] },    # Blue
-            file: { hsl: [0, 0, 90] },            # Light gray
-            executable: { hsl: [120, 70, 50] },   # Green
-            selected: { hsl: [50, 90, 70] },      # Yellow
-            preview: { hsl: [180, 60, 65] }       # Cyan
+            directory: { hsl: [220, 80, 60] },
+            file: { hsl: [0, 0, 90] },
+            executable: { hsl: [120, 70, 50] },
+            selected: { hsl: [50, 90, 70] },
+            preview: { hsl: [180, 60, 65] }
           },
           keybinds: {
             quit: %w[q ESC],
@@ -182,4 +290,3 @@ module Rufio
     end
   end
 end
-

data/lib/rufio/keybind_handler.rb

@@ -51,7 +51,7 @@ module Rufio
       # Log viewer mode
       @in_log_viewer_mode = false
       @pre_log_viewer_directory = nil
-      @log_dir = File.join(Dir.home, '.config', 'rufio', 'log')
+      @log_dir = File.join(Dir.home, '.config', 'rufio', 'logs')
 
       # Preview pane focus and scroll
       @preview_focused = false
@@ -62,9 +62,8 @@ module Rufio
       @job_manager = JobManager.new(notification_manager: @notification_manager)
       @job_mode = JobMode.new(job_manager: @job_manager)
 
-      # Script path manager
-      config_file = File.expand_path('~/.config/rufio/config.yml')
-      @script_path_manager = File.exist?(config_file) ? ScriptPathManager.new(config_file) : nil
+      # Script path manager (新形式: script_paths.yml)
+      @script_path_manager = ScriptPathManager.new(Config::SCRIPT_PATHS_YML)
     end
 
     def set_directory_listing(directory_listing)
@@ -1446,8 +1445,16 @@ module Rufio
       when '3'
         show_script_paths_manager
       when '4'
-        # ブックマーク一覧表示（既存機能）
+        # ブックマーク一覧表示
+        selected_bookmark = @bookmark_manager.list_interactive
         @terminal_ui&.refresh_display
+        if selected_bookmark
+          if @bookmark_manager.path_exists?(selected_bookmark)
+            navigate_to_directory(selected_bookmark[:path])
+          else
+            show_error_and_wait('bookmark.path_not_exist', selected_bookmark[:path])
+          end
+        end
       else
         @terminal_ui&.refresh_display
       end
@@ -1460,10 +1467,8 @@ module Rufio
       current_path = @directory_listing&.current_path || Dir.pwd
 
       unless @script_path_manager
-        # ScriptPathManagerがない場合は作成
-        config_file = File.expand_path('~/.config/rufio/config.yml')
-        FileUtils.mkdir_p(File.dirname(config_file))
-        @script_path_manager = ScriptPathManager.new(config_file)
+        # ScriptPathManagerがない場合は作成（新形式: script_paths.yml）
+        @script_path_manager = ScriptPathManager.new(Config::SCRIPT_PATHS_YML)
       end
 
       if @script_path_manager.paths.include?(current_path)

data/lib/rufio/script_config_loader.rb

@@ -1,15 +1,16 @@
 # frozen_string_literal: true
 
 require 'yaml'
+require_relative 'config'
 
 module Rufio
   # 複数の設定ファイルからscript_pathsをロード・マージするクラス
   # 優先順位: ローカル > ユーザー > システム
   class ScriptConfigLoader
-    # デフォルトの設定ファイルパス
-    DEFAULT_LOCAL_PATH = './rufio.yml'
-    DEFAULT_USER_PATH = File.expand_path('~/.config/rufio/config.yml')
-    DEFAULT_SYSTEM_PATH = '/etc/rufio/config.yml'
+    # デフォルトの設定ファイルパス（新形式: script_paths.yml）
+    DEFAULT_LOCAL_PATH = Config::LOCAL_YAML_PATH
+    DEFAULT_USER_PATH = Config::SCRIPT_PATHS_YML
+    DEFAULT_SYSTEM_PATH = '/etc/rufio/script_paths.yml'
 
     # @param local_path [String, nil] ローカル設定ファイルのパス
     # @param user_path [String, nil] ユーザー設定ファイルのパス
@@ -63,25 +64,39 @@ module Rufio
 
     private
 
-    # 設定ファイルからscript_pathsを読み込む
+    # 設定ファイルからscript_pathsを読み込む（新形式対応）
     # @param path [String] 設定ファイルのパス
     # @return [Array<String>] パスの配列
     def load_paths_from_file(path)
+      # 新形式: script_paths.yml（リスト形式）
+      if path.end_with?('script_paths.yml')
+        return Config.load_script_paths(path)
+      end
+
+      # 後方互換: 古いconfig.yml形式（ハッシュ形式）
       config = load_config(path)
       config['script_paths'] || []
     end
 
-    # 設定ファイルを読み込む
+    # 設定ファイルを読み込む（後方互換用）
     # @param path [String] 設定ファイルのパス
     # @return [Hash] 設定内容
     def load_config(path)
-      return {} unless File.exist?(path)
+      config = Config.load_yaml_config(path)
+      stringify_keys(config)
+    end
 
-      yaml = YAML.safe_load(File.read(path), symbolize_names: false)
-      yaml || {}
-    rescue StandardError => e
-      warn "Warning: Failed to load config #{path}: #{e.message}"
-      {}
+    def stringify_keys(hash)
+      hash.transform_keys(&:to_s).transform_values do |value|
+        case value
+        when Hash
+          stringify_keys(value)
+        when Array
+          value.map { |v| v.is_a?(Hash) ? stringify_keys(v) : v }
+        else
+          value
+        end
+      end
     end
 
     # ハッシュを深くマージ