mk_semi_lattice 0.2.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 15f1e816692dca4dce63d49abf961749e3b242f96877a1bc998b2b4208e3eb83
-  data.tar.gz: 646e5f22cec25a61c7d2c8c8e9de6e31377a4d8693a1bc14f1f31724d840700c
+  metadata.gz: 0d344bc5e5985d443acbd280d0c567b0e01be8635971e5155339d5b3b21e1b76
+  data.tar.gz: 3782664243e07f23a4e800505d08daf299963afb0df5923a489602ead09d84ad
 SHA512:
-  metadata.gz: 6379e532b1dd399efd9b41ceae6aff47770520e79d0603fa413da2d4092fc78781ba6bd68af5a05e1a0870030a2fe849695f9737872d24c90835f2850af93767
-  data.tar.gz: 35d3e0c85d3a34ec4ac774339703a4026d1ebd2cfb4b2db5e52ea0ce9941f6c01cc9e223bef36d91ccfebb2f5922dd463ce47a945feeb6ba87f2af0972a3daa3
+  metadata.gz: 9c9d8bd624f964fe520ba5b55dbad67cdac44a0791b1693f6d3d177e5c9555675cc88a853e764cbd99eea209c7c120a0d2b1e420911a128f34c68aa8148d7105
+  data.tar.gz: f3990badd3f2fbd9d7a0c1b4a99322c8203a018d9d0e80e30018da3d8736980599a37b9bfca0a77480b71f8a01bde7edbce8097584d5d26a5480295327f02134

data/.stack/history/ignore_opt.md

@@ -0,0 +1,395 @@
+User: optionで-I '_stack_*|*.yaml'としてこれらのdir, file名をIgnoreする@option[:ignore]を設定するように修正．
+
+GitHub Copilot: ````ruby
+# ...existing code...
+
+      opts.on("-I", "--ignore=PATTERN", "Ignore files/dirs matching PATTERN (e.g. '_stack_*|*.yaml')") do |pattern|
+        @options[:ignore] = pattern
+      end
+
+# ...existing code...
+````
+
+この修正により、`-I` または `--ignore=PATTERN` オプションで `@options[:ignore]` にパターンが設定されます。
+
+User: @option[:ignore]のpatternが含まれるdir, file名をIgnoreするように修正．
+
+GitHub Copilot: ````ruby
+# ...existing code...
+    def dir_tree(path, depth)
+      return nil if depth < 0
+      tree = {}
+      entries =
+        if @options && (@options[:visibility] == 'all' || @options[:visibility] == 'dir_and_hidden_dir')
+          Dir.entries(path) - ['.', '..']
+        else
+          Dir.children(path)
+        end
+
+      ignore_pattern = @options && @options[:ignore] ? @options[:ignore] : nil
+      ignore_regex = ignore_pattern ? Regexp.new(ignore_pattern) : nil
+
+      entries.each do |entry|
+        next if entry.end_with?('~')
+        next if ignore_regex && entry.match(ignore_regex)
+        full = File.join(path, entry)
+        case @options && @options[:visibility]
+        when 'dir_only'
+          next if entry.start_with?('.') # 隠しディレクトリ除外
+          next unless File.directory?(full)
+        when 'dir_and_hidden_dir'
+          next unless File.directory?(full) # 隠しも含めてディレクトリのみ
+        when 'all'
+          # すべて含める（除外なし）
+        when 'yaml_exclude'
+          next if File.extname(entry) == '.yaml'
+        else # 'normal'
+          next if entry.start_with?('.')
+          next if File.file?(full) && File.extname(entry) == '.yaml' && entry != 'semi_lattice.yaml'
+        end
+        if File.symlink?(full)
+          target = File.readlink(full)
+          tree[entry] = "-> #{target}"
+        elsif File.directory?(full)
+          subtree = dir_tree(full, depth - 1)
+          if subtree
+            tree["#{entry}/"] = subtree
+          else
+            tree["#{entry}/"] = nil
+          end
+        else
+          tree[entry] = nil unless @options && (@options[:visibility] == 'dir_only' || @options[:visibility] == 'dir_and_hidden_dir')
+        end
+      end
+      tree
+    end
+# ...existing code...
+````
+
+この修正で、`@options[:ignore]` のパターンにマッチするファイル・ディレクトリ名を無視します。パターンは正規表現として扱われます。
+
+User: treeの-I optionと同様に，mk_sl -I '_stack_*|*.yaml'de
+
+GitHub Copilot: `-I '_stack_*|*.yaml'` のようなパターンで、`tree` コマンドの `-I` オプションと同様に「ワイルドカード（*）」や「|」区切りの複数パターン」をサポートするには、globパターンを正規表現に変換する必要があります。  
+以下のように修正してください。
+
+````ruby
+# ...existing code...
+    def dir_tree(path, depth)
+      return nil if depth < 0
+      tree = {}
+      entries =
+        if @options && (@options[:visibility] == 'all' || @options[:visibility] == 'dir_and_hidden_dir')
+          Dir.entries(path) - ['.', '..']
+        else
+          Dir.children(path)
+        end
+
+      # tree -I のようなワイルドカード・複数パターン対応
+      ignore_pattern = @options && @options[:ignore] ? @options[:ignore] : nil
+      ignore_regex = nil
+      if ignore_pattern
+        # '_stack_*|*.yaml' → /(?:\A_stack_.*\z|\A.*\.yaml\z)/
+        regex_str = ignore_pattern.split('|').map do |pat|
+          pat = pat.strip
+          pat = Regexp.escape(pat).gsub('\*', '.*').gsub('\?', '.')
+          "\\A#{pat}\\z"
+        end.join('|')
+        ignore_regex = Regexp.new("(?:#{regex_str})")
+      end
+
+      entries.each do |entry|
+        next if entry.end_with?('~')
+        next if ignore_regex && entry.match(ignore_regex)
+        full = File.join(path, entry)
+        case @options && @options[:visibility]
+        when 'dir_only'
+          next if entry.start_with?('.') # 隠しディレクトリ除外
+          next unless File.directory?(full)
+        when 'dir_and_hidden_dir'
+          next unless File.directory?(full) # 隠しも含めてディレクトリのみ
+        when 'all'
+          # すべて含める（除外なし）
+        when 'yaml_exclude'
+          next if File.extname(entry) == '.yaml'
+        else # 'normal'
+          next if entry.start_with?('.')
+          next if File.file?(full) && File.extname(entry) == '.yaml' && entry != 'semi_lattice.yaml'
+        end
+        if File.symlink?(full)
+          target = File.readlink(full)
+          tree[entry] = "-> #{target}"
+        elsif File.directory?(full)
+          subtree = dir_tree(full, depth - 1)
+          if subtree
+            tree["#{entry}/"] = subtree
+          else
+            tree["#{entry}/"] = nil
+          end
+        else
+          tree[entry] = nil unless @options && (@options[:visibility] == 'dir_only' || @options[:visibility] == 'dir_and_hidden_dir')
+        end
+      end
+      tree
+    end
+# ...existing code...
+````
+
+この修正で、`-I '_stack_*|*.yaml'` のような複数・ワイルドカードパターンが `tree` コマンド同様に動作します。
+
+User: #file:mk_semi_lattice_yaml.rb:117-175 がだいぶ長い関数になっています．まずは，l127-138を別関数に切り出しましょう．
+
+GitHub Copilot: ````ruby
+# ...existing code...
+    def build_ignore_regex(ignore_pattern)
+      return nil unless ignore_pattern
+      # '_stack_*|*.yaml' → /(?:\A_stack_.*\z|\A.*\.yaml\z)/
+      regex_str = ignore_pattern.split('|').map do |pat|
+        pat = pat.strip
+        pat = Regexp.escape(pat).gsub('\*', '.*').gsub('\?', '.')
+        "\\A#{pat}\\z"
+      end.join('|')
+      Regexp.new("(?:#{regex_str})")
+    end
+
+    def dir_tree(path, depth)
+      return nil if depth < 0
+      tree = {}
+      entries =
+        if @options && (@options[:visibility] == 'all' || @options[:visibility] == 'dir_and_hidden_dir')
+          Dir.entries(path) - ['.', '..']
+        else
+          Dir.children(path)
+        end
+
+      ignore_pattern = @options && @options[:ignore] ? @options[:ignore] : nil
+      ignore_regex = build_ignore_regex(ignore_pattern)
+
+      entries.each do |entry|
+        next if entry.end_with?('~')
+        next if ignore_regex && entry.match(ignore_regex)
+        full = File.join(path, entry)
+        case @options && @options[:visibility]
+        when 'dir_only'
+          next if entry.start_with?('.') # 隠しディレクトリ除外
+          next unless File.directory?(full)
+        when 'dir_and_hidden_dir'
+          next unless File.directory?(full) # 隠しも含めてディレクトリのみ
+        when 'all'
+          # すべて含める（除外なし）
+        when 'yaml_exclude'
+          next if File.extname(entry) == '.yaml'
+        else # 'normal'
+          next if entry.start_with?('.')
+          next if File.file?(full) && File.extname(entry) == '.yaml' && entry != 'semi_lattice.yaml'
+        end
+        if File.symlink?(full)
+          target = File.readlink(full)
+          tree[entry] = "-> #{target}"
+        elsif File.directory?(full)
+          subtree = dir_tree(full, depth - 1)
+          if subtree
+            tree["#{entry}/"] = subtree
+          else
+            tree["#{entry}/"] = nil
+          end
+        else
+          tree[entry] = nil unless @options && (@options[:visibility] == 'dir_only' || @options[:visibility] == 'dir_and_hidden_dir')
+        end
+      end
+      tree
+    end
+# ...existing code...
+````
+
+`l127-138` の ignore パターン処理を `build_ignore_regex` メソッドとして切り出しました。
+
+User: このあたりをskip_nameとして切り出しましょう．
+
+GitHub Copilot: ````ruby
+# ...existing code...
+    def skip_name?(entry, full, ignore_regex)
+      return true if entry.end_with?('~')
+      return true if ignore_regex && entry.match(ignore_regex)
+      case @options && @options[:visibility]
+      when 'dir_only'
+        return true if entry.start_with?('.') # 隠しディレクトリ除外
+        return true unless File.directory?(full)
+      when 'dir_and_hidden_dir'
+        return true unless File.directory?(full) # 隠しも含めてディレクトリのみ
+      when 'all'
+        # すべて含める（除外なし）
+      when 'yaml_exclude'
+        return true if File.extname(entry) == '.yaml'
+      else # 'normal'
+        return true if entry.start_with?('.')
+        return true if File.file?(full) && File.extname(entry) == '.yaml' && entry != 'semi_lattice.yaml'
+      end
+      false
+    end
+
+    def dir_tree(path, depth)
+      return nil if depth < 0
+      tree = {}
+      entries =
+        if @options && (@options[:visibility] == 'all' || @options[:visibility] == 'dir_and_hidden_dir')
+          Dir.entries(path) - ['.', '..']
+        else
+          Dir.children(path)
+        end
+
+      ignore_pattern = @options && @options[:ignore] ? @options[:ignore] : nil
+      ignore_regex = build_ignore_regex(ignore_pattern)
+
+      entries.each do |entry|
+        full = File.join(path, entry)
+        next if skip_name?(entry, full, ignore_regex)
+        if File.symlink?(full)
+          target = File.readlink(full)
+          tree[entry] = "-> #{target}"
+        elsif File.directory?(full)
+          subtree = dir_tree(full, depth - 1)
+          if subtree
+            tree["#{entry}/"] = subtree
+          else
+            tree["#{entry}/"] = nil
+          end
+        else
+          tree[entry] = nil unless @options && (@options[:visibility] == 'dir_only' || @options[:visibility] == 'dir_and_hidden_dir')
+        end
+      end
+      tree
+    end
+# ...existing code...
+````
+
+`skip_name?` メソッドとして切り出しました。
+
+User: mk_sl -I '_stack_*|*.yaml' -a
+mk_semi_lattice is running...
+/Users/bob/.rbenv/gems/3.4.0/gems/mk_semi_lattice-0.3.1/lib/mk_semi_lattice/option_manager.rb:84:in 'OptionManager#parse!': missing argument: -a (OptionParser::MissingArgument)
+とのエラーが出ます．
+
+GitHub Copilot: このエラーは、`OptionParser` で `-a` や `-d` などの短いフラグ（引数なし）と、`-I` のような引数付きオプションを同時に使った場合、`-I` の値が `'a'` などの次のフラグとして誤認識されるために発生します。
+
+**原因**  
+`OptionParser` は `-I` のような引数付きオプションの直後にスペースを空けて値を渡す場合、次の `-a` も値として解釈しようとします。  
+例:  
+`mk_sl -I _stack_*|*.yaml -a`  
+→ `-I` の値が `_stack_*|*.yaml` ではなく `_stack_*|*.yaml -a` になり、`-a` が値として消費されてしまう。
+
+**解決策**  
+- `-I` の値は必ず`=`で渡す（`-I='_stack_*|*.yaml'` または `--ignore='_stack_*|*.yaml'` のようにする）
+- もしくは、`-I '_stack_*|*.yaml'` のようにクォートで囲む（シェルによっては必要）
+
+**例:**
+```
+mk_sl -I='_stack_*|*.yaml' -a
+mk_sl --ignore='_stack_*|*.yaml' -a
+```
+または
+```
+mk_sl -a -I '_stack_*|*.yaml'
+```
+
+**補足**  
+この挙動は `OptionParser` の仕様であり、コード側での修正は不要です。  
+READMEやヘルプに「`-I` の値は `=` で渡すか、クォートで囲むこと」と明記すると親切です。
+
+---
+
+**まとめ:**  
+`-I` の値は `=` で渡すか、クォートで囲んでください。  
+例:  
+`mk_sl -I='_stack_*|*.yaml' -a`
+
+User: mk_sl -I='_stack_*|*.yaml' -a
+mk_semi_lattice is running...
+/Users/bob/.rbenv/gems/3.4.0/gems/mk_semi_lattice-0.3.1/lib/mk_semi_lattice/option_manager.rb:84:in 'OptionManager#parse!': missing argument: -a (OptionParser::MissingArgument)
+とのエラーがまだ出ています．
+
+GitHub Copilot: この問題は、`OptionParser` の `-ad` オプションが `"Show all directories including hidden ones"` という説明で定義されていますが、`-a` と `-d` の組み合わせ（`-a -d`や`-ad`）を正しく処理できていないことが原因です。
+
+**主な原因:**
+- `-ad` のような複数フラグを1つにまとめた短縮形（POSIXスタイル）を `OptionParser` で使う場合、`-a -d` ではなく `-ad` と書く必要があります。
+- しかし、`-ad` を `opts.on("-ad", ...)` で定義しているため、`-a -d` のように個別に指定した場合は `-a` だけが認識され、`-d` は未定義のフラグとして扱われてしまいます。
+
+**解決策:**
+1. `-ad` の定義を削除し、`-a` と `-d` のみを定義する。
+2. `-a` と `-d` の両方が指定された場合の処理は、既存の
+   ```ruby
+   if a_flag && d_flag
+     @options[:visibility] = 'dir_and_hidden_dir'
+   ```
+   で十分です。
+
+**修正例:**
+
+````ruby
+# ...existing code...
+      # opts.on("-ad", "Show all directories including hidden ones") do
+      #   a_flag = true
+      #   d_flag = true
+      # end
+# ...existing code...
+````
+
+**READMEやUsage例にも「-a -d のように個別に指定してください」と記載すると親切です。**
+
+---
+
+### まとめ
+
+- `-ad` の定義を削除し、`-a -d` のように個別指定で使ってください。
+- これで `-I='_stack_*|*.yaml' -a` も正しく動作します。
+
+もし `-ad` の短縮形もサポートしたい場合は、`OptionParser` ではなく手動でフラグをパースする必要がありますが、一般的には `-a -d` で十分です。
+
+User: 上記の仕様を英語のhelpにして出力するように修正．
+
+GitHub Copilot: ````ruby
+# ...existing code...
+    OptionParser.new do |opts|
+      opts.banner = <<~BANNER
+        Usage: mk_semi_lattice PATH [options]
+        Default PATH = '.'
+
+        Options:
+          -L N           Layer depth (default: 2)
+          -t, --tree=FILE    Use FILE from tree
+          -n, --node=FILE    Use FILE from node-edge
+          -i, --index        Display node ids
+          -l, --log [BOOL]   Enable/disable logging (true/false), and save to config
+          -v, --version      Show version
+          -d                 Show directories only
+          -a                 Show all (files and directories)
+          -Y                 YAML exclude mode
+          -I, --ignore=PATTERN  Ignore files/dirs matching PATTERN (e.g. '_stack_*|*.yaml')
+                               Multiple patterns can be separated by '|'.
+                               Wildcards '*' and '?' are supported (like the 'tree' command).
+                               When using -I, always use '=' or quote the pattern, e.g.:
+                                 -I='_stack_*|*.yaml' -a
+                                 --ignore='_stack_*|*.yaml' -a
+                               Do not use -ad; use -a -d for combined flags.
+      BANNER
+# ...existing code...
+````
+
+この修正で、helpメッセージに英語で仕様が明記されます。
+
+User: mk_sl -I='_stack_*|*.yaml'
+mk_semi_lattice is running...
+{layer: 2, init_step: :from_semi_lattice, show_index: false, merge: false, visibility: "normal", ignore: "=_stack_*|*.yaml"となっています．
+これを修正
+
+GitHub Copilot: ````ruby
+# ...existing code...
+      opts.on("-I PATTERN", "--ignore=PATTERN", "Ignore files/dirs matching PATTERN (e.g. '_stack_*|*.yaml')") do |pattern|
+        # Remove leading '=' if present (e.g. when user writes -I='_stack_*|*.yaml')
+        pattern = pattern.sub(/\A=/, '') if pattern
+        @options[:ignore] = pattern
+      end
+# ...existing code...
+````
+
+この修正で、`-I='_stack_*|*.yaml'` のような場合に `ignore: "_stack_*|*.yaml"` となり、先頭の `=` が格納されなくなります。