mk_semi_lattice 0.1.1 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fd367913e384f82dbbb7b8db40d35646855a82d5006715a51fc4eaa6601bc0ca
-  data.tar.gz: fed2c0e74140c8dd15933c13ec8eb4e6466f72ac8f7f5a899c063046e62a6ce8
+  metadata.gz: 0d344bc5e5985d443acbd280d0c567b0e01be8635971e5155339d5b3b21e1b76
+  data.tar.gz: 3782664243e07f23a4e800505d08daf299963afb0df5923a489602ead09d84ad
 SHA512:
-  metadata.gz: b44216e9db854f9a28d5f9b27463f0bfd119aaa0dece1e70cdd4e35c0628ee1c5720d5f780985c35e548c32dd63f71c46af9781e7f35960d3d8ec98b680b6a3f
-  data.tar.gz: 8984c903dc85dcdf6e32bf80405d0ef6aae28718c8e776e5496554f385604a30d73bacecda8af54b7b0b6eadaaaae9ce6ef48e48844898de1adc7407f54e4bda
+  metadata.gz: 9c9d8bd624f964fe520ba5b55dbad67cdac44a0791b1693f6d3d177e5c9555675cc88a853e764cbd99eea209c7c120a0d2b1e420911a128f34c68aa8148d7105
+  data.tar.gz: f3990badd3f2fbd9d7a0c1b4a99322c8203a018d9d0e80e30018da3d8736980599a37b9bfca0a77480b71f8a01bde7edbce8097584d5d26a5480295327f02134

data/{.hyper_card → .stack}/history/20251022_0725_add_color_index.md

@@ -1,4 +1,4 @@
-daddygongon: 描かれたnodeにidも表示するoptionとして-i, -indexを追加．
+**daddygongon**: 描かれたnodeにidも表示するoptionとして-i, -indexを追加．
 
 GitHub Copilot: ### [mk_semi_lattice.rb](file:///Users/bob/git_hub/mk_semi_lattice/lib/mk_semi_lattice.rb)
 

data/.stack/history/add_icon_shape_251208.md

@@ -0,0 +1,197 @@
+# head
+date: 2025-12-08
+author: bob
+
+## User
+重複してますね．refactorしましょう．
+
+## Copilot
+重複している`@text`生成部分をまとめて、`dir`/それ以外で分岐する部分を簡潔にリファクタリングできます。
+以下のように書き換えるとスッキリします。
+
+```ruby
+# ...existing code...
+def create_graphics
+  return if @created
+  font_path = japanese_font
+
+  # テキスト生成（共通化）
+  if font_path && File.exist?(font_path)
+    @text = Text.new(label, x: x-28, y: y-10, size: 18, color: 'black', font: font_path, z: 11)
+  else
+    @text = Text.new(label, x: x-28, y: y-10, size: 18, color: 'black', z: 11)
+  end
+
+  # アイコン生成
+  @circle = if @type == 'dir'
+              Folder.new(x: x, y: y, color: NODE_COLOR, z: 10)
+            else
+              Circle.new(x: x, y: y, radius: 30, color: NODE_COLOR, z: 10)
+            end
+  @created = true
+end
+# ...existing code...
+```
+
+- `@text`の生成を一度だけにまとめました。
+- `@circle`の分岐のみを残しています。
+
+このようにすることで、重複がなくなり、保守性も向上します。
+
+## User
+pngファイルを表示するように変更．
+
+## Copilot
+`Folder`クラスでフォルダアイコンを図形からPNG画像表示に変更するには、`Rectangle`の代わりに`Image`を使います。
+下記のように修正してください。
+
+```ruby
+# ...existing code...
+class Folder
+  # フォルダアイコンをRuby2Dの画像で描画するクラス
+  attr_accessor :x, :y, :name, :label, :fixed, :linked, :dx, :dy, :type, :file_path
+  def initialize(attrs = {})
+    @x = attrs[:x]
+    @y = attrs[:y]
+    @z = attrs[:z]
+    @color = attrs[:color]
+    @image = Image.new(
+      './icons/folder.png', # PNGファイルのパス
+      x: @x-28, y: @y-20, width: 56, height: 32, z: @z
+    )
+  end
+
+  def color=(c)
+    # 画像には直接色をつけられません。必要なら画像を加工してください。
+  end
+
+  def x=(x)
+    @image.x = x - 28
+  end
+
+  def y=(y)
+    @image.y = y - 20
+  end
+end
+# ...existing code...
+```
+
+- `./icons/folder.png` をプロジェクト内に配置してください。
+- `color=` メソッドは画像には使えません（必要なら画像自体を加工してください）。
+
+この修正で、フォルダアイコンがPNG画像で表示されるようになります。
+
+## User
+codeが重複しています．親クラスとしてNodeShapeを作り，派生クラスとしてFolder2, Document, Folderを作成してください．
+
+## Copilot
+以下のように、共通部分を`NodeShape`親クラスにまとめ、`Folder2`, `Document`, `Folder`を継承する形でリファクタリングできます。
+
+```ruby
+# ...existing code...
+class NodeShape
+  attr_accessor :x, :y, :name, :label, :fixed, :linked, :dx, :dy, :type, :file_path
+
+  def initialize(attrs = {})
+    @x = attrs[:x]
+    @y = attrs[:y]
+    @z = attrs[:z]
+    @color = attrs[:color]
+    @name = attrs[:name]
+    @label = attrs[:label]
+    @fixed = attrs[:fixed]
+    @linked = attrs[:linked]
+    @dx = attrs[:dx]
+    @dy = attrs[:dy]
+    @type = attrs[:type]
+    @file_path = attrs[:file_path]
+    setup_shape
+  end
+
+  def color=(c)
+    # 派生クラスでオーバーライド
+  end
+
+  def x=(x)
+    # 派生クラスでオーバーライド
+  end
+
+  def y=(y)
+    # 派生クラスでオーバーライド
+  end
+
+  def setup_shape
+    # 派生クラスでオーバーライド
+  end
+end
+
+class Folder2 < NodeShape
+  def setup_shape
+    @image = Image.new(
+      './icons/folder.png',
+      x: @x-28, y: @y-20, width: 56, height: 36, z: @z
+    )
+  end
+
+  def color=(c)
+    # 画像には直接色をつけられません
+  end
+
+  def x=(x)
+    @image.x = x - 28
+  end
+
+  def y=(y)
+    @image.y = y - 20
+  end
+end
+
+class Document < NodeShape
+  def setup_shape
+    @image = Image.new(
+      './icons/document.png',
+      x: @x-28, y: @y-20, width: 38, height: 56, z: @z
+    )
+  end
+
+  def color=(c)
+    # 画像には直接色をつけられません
+  end
+
+  def x=(x)
+    @image.x = x - 28
+  end
+
+  def y=(y)
+    @image.y = y - 20
+  end
+end
+
+class Folder < NodeShape
+  def setup_shape
+    @folder1 = Rectangle.new(x: @x-28, y: @y-10, width: 56, height: 32, color: @color, z: @z)
+    @folder2 = Rectangle.new(x: @x-28, y: @y-20, width: 26, height: 12, color: @color, z: @z)
+  end
+
+  def color=(c)
+    @folder1.color = c
+    @folder2.color = c
+  end
+
+  def x=(x)
+    @folder1.x = x - 28
+    @folder2.x = x - 28
+  end
+
+  def y=(y)
+    @folder1.y = y - 10
+    @folder2.y = y - 20
+  end
+end
+# ...existing code...
+```
+
+- `NodeShape`で共通属性・初期化をまとめ、`setup_shape`を各派生クラスで実装します。
+- `color=`, `x=`, `y=`も必要に応じてオーバーライドします。
+
+この構造で重複が大幅に減り、拡張・保守が容易になります。

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"` となり、先頭の `=` が格納されなくなります。