mk_semi_lattice 0.1.1

data/.hyper_card/history/20251021_2322_add_comment_edges.md

@@ -0,0 +1,252 @@
+daddygongon: init_file=='.'の時はsemi_dirがあるか確認して，なければ未実装と警告して終了．
+
+GitHub Copilot: ### [mk_semi_lattice.rb](file:///Users/bob/git_hub/mk_semi_lattice/lib/mk_semi_lattice.rb)
+
+Check if `init_file == '.'` and if so, verify that `semi_dir` exists. If not, print a warning and exit.
+
+````ruby
+# ...existing code...
+
+init_file = ARGV[0] || '.'
+parent_dir = File.dirname(init_file)
+semi_dir = File.join(parent_dir, '.semi_lattice')
+semi_lattice_yaml_path = File.join(semi_dir, "semi_lattice.yaml")
+
+if init_file == '.'
+  unless Dir.exist?(semi_dir)
+    warn "未実装: カレントディレクトリに .semi_lattice ディレクトリがありません。"
+    exit 1
+  end
+end
+
+p parent_dir
+p semi_dir
+p semi_lattice_yaml_path
+
+app = MkSemiLatticeData.new(init_file, with_semi_lattice_yaml: true)
+
+# ...existing code...
+````
+
+Made changes.
+
+daddygongon: options[:init_step] == :mk_node_edgeの時には，
+MkSemiLattice::MkNodeEdge.new(input_path: input_path, output_path: output_path)
+  file = output_path
+  app = MkSemiLatticeData.new(output_path)
+
+GitHub Copilot: ### [mk_semi_lattice.rb](file:///Users/bob/git_hub/mk_semi_lattice/lib/mk_semi_lattice.rb)
+
+Add logic to handle `options[:init_step] == :mk_node_edge`. When this is true, run `MkSemiLattice::MkNodeEdge.new(input_path: input_path, output_path: output_path)`, set `file = output_path`, and initialize `app = MkSemiLatticeData.new(output_path)`.
+
+````ruby
+# ...existing code...
+
+init_file = ARGV[0] || '.'
+parent_dir = File.dirname(init_file)
+semi_dir = File.join(parent_dir, '.semi_lattice')
+semi_lattice_yaml_path = File.join(semi_dir, "semi_lattice.yaml")
+
+if init_file == '.'
+  unless Dir.exist?(semi_dir)
+    warn "未実装: カレントディレクトリに .semi_lattice ディレクトリがありません。"
+    exit 1
+  end
+end
+
+if options[:init_step] == :mk_node_edge
+  input_path = options[:file]
+  output_path = semi_lattice_yaml_path
+  MkSemiLattice::MkNodeEdge.new(input_path: input_path, output_path: output_path)
+  file = output_path
+  app = MkSemiLatticeData.new(output_path)
+else
+  app = MkSemiLatticeData.new(init_file, with_semi_lattice_yaml: true)
+end
+
+# ...existing code...
+````
+
+Made changes.
+
+daddygongon: #file:mk_semi_lattice.rb:38-49 はARGV[0]=='.'かARGV[0]==nilおよびoptions[:file]!=nilの時に実行するように修正
+
+GitHub Copilot: ### [mk_semi_lattice.rb](file:///Users/bob/git_hub/mk_semi_lattice/lib/mk_semi_lattice.rb)
+
+Move lines 38-49 so that they are only executed when `ARGV[0]=='.'` or `ARGV[0]==nil` or `options[:file]` is not nil.
+
+````ruby
+# ...existing code...
+
+init_file, options[:init_step] = if ARGV[0]=='.' || ARGV[0].nil? || options[:file]
+  parent_dir = Dir.pwd
+  semi_dir = File.join(parent_dir, '.semi_lattice')
+  semi_lattice_yaml_path = File.join(semi_dir, "semi_lattice.yaml")
+  # ↓ このブロックのみ条件付きで実行
+  if Dir.exist?(semi_lattice_yaml_path)
+    [semi_lattice_yaml_path, :from_semi_lattice]
+  else
+    ['.', :from_dir]
+  end
+else
+  [ARGV[0], :from_semi_lattice]
+end
+
+# ...existing code...
+````
+
+Made changes.
+
+daddygongon: traverseで生成するnodeを全てHashにしてkeyをsymbolに変更．
+
+GitHub Copilot: ### [mk_node_edge.rb](file:///Users/bob/git_hub/mk_semi_lattice/lib/mk_semi_lattice/mk_node_edge.rb)
+
+`traverse`で生成するnodeを全てHashにし、keyをsymbolに変更します。  
+また、`add_edge_comments`メソッド内でnodeのkey参照もsymbolに合わせて修正します。
+
+````ruby
+# ...existing code...
+    def self.add_edge_comments(output)
+      nodes = output[:nodes]
+      edges = output[:edges]
+      id_name_map = nodes.each_with_object({}) do |node, map|
+        map[node[:id]] = node[:name]
+      end
+      # ...existing code...
+    end
+
+    def traverse(node, parent_id, nodes, edges, id_counter, current_path, top_path)
+      if node.is_a?(Hash)
+        node.each do |name, value|
+          node_id = id_counter[:val]
+          id_counter[:val] += 1
+
+          type = if name.to_s.end_with?('/')
+            'dir'
+          else
+            value.is_a?(Hash) || value.is_a?(Array) ? 'dir' : 'file'
+          end
+
+          p [current_path, name]
+          node_path = File.join(current_path, name.to_s)
+          rel_path = if node_path == top_path
+            '.'
+          else
+            rp = Pathname.new(node_path).relative_path_from(Pathname.new(top_path)).to_s
+            rp.start_with?('.') ? rp : "./#{rp}"
+          end
+
+          nodes << {
+            :id => node_id,
+            :name => name,
+            :type => type,
+            :file_path => rel_path
+          }
+          if parent_id
+            edges << {
+              :from => parent_id,
+              :to => node_id
+            }
+          end
+
+          if value.is_a?(Hash) || value.is_a?(Array)
+            traverse(value, node_id, nodes, edges, id_counter, node_path, top_path)
+          end
+        end
+      elsif node.is_a?(Array)
+        node.each do |item|
+          traverse(item, parent_id, nodes, edges, id_counter, current_path, top_path)
+        end
+      end
+    end
+# ...existing code...
+````
+
+Made changes.
+
+daddygongon: #file:mk_semi_lattice.rb:186-187 は何をしていますか？
+
+GitHub Copilot: 186-187行目（`Dir.mkdir(semi_dir) unless`）は、`.semi_lattice`ディレクトリが存在しない場合に作成しようとしています。ただし、`unless`の条件が未記載のため、文法的に不完全です。
+
+本来は  
+```ruby
+Dir.mkdir(semi_dir) unless Dir.exist?(semi_dir)
+```
+のように、ディレクトリが存在しない場合のみ作成する意図です。
+
+daddygongon: traverseで生成するnodeを全てHashにしてkeyをsymbolに変更．
+
+GitHub Copilot: ### [mk_node_edge.rb](file:///Users/bob/git_hub/mk_semi_lattice/lib/mk_semi_lattice/mk_node_edge.rb)
+
+`traverse`で生成するnodeを全てHashにし、keyをsymbolに変更します。  
+また、`add_edge_comments`メソッド内でnodeのkey参照もsymbolに合わせて修正します。
+
+````ruby
+# ...existing code...
+    def self.add_edge_comments(output)
+      nodes = output[:nodes]
+      edges = output[:edges]
+      id_name_map = nodes.each_with_object({}) do |node, map|
+        map[node[:id]] = node[:name]
+      end
+      # ...existing code...
+    end
+
+    def traverse(node, parent_id, nodes, edges, id_counter, current_path, top_path)
+      if node.is_a?(Hash)
+        node.each do |name, value|
+          node_id = id_counter[:val]
+          id_counter[:val] += 1
+
+          type = if name.to_s.end_with?('/')
+            'dir'
+          else
+            value.is_a?(Hash) || value.is_a?(Array) ? 'dir' : 'file'
+          end
+
+          p [current_path, name]
+          node_path = File.join(current_path, name.to_s)
+          rel_path = if node_path == top_path
+            '.'
+          else
+            rp = Pathname.new(node_path).relative_path_from(Pathname.new(top_path)).to_s
+            rp.start_with?('.') ? rp : "./#{rp}"
+          end
+
+          nodes << {
+            :id => node_id,
+            :name => name,
+            :type => type,
+            :file_path => rel_path
+          }
+          if parent_id
+            edges << {
+              :from => parent_id,
+              :to => node_id
+            }
+          end
+
+          if value.is_a?(Hash) || value.is_a?(Array)
+            traverse(value, node_id, nodes, edges, id_counter, node_path, top_path)
+          end
+        end
+      elsif node.is_a?(Array)
+        node.each do |item|
+          traverse(item, parent_id, nodes, edges, id_counter, current_path, top_path)
+        end
+      end
+    end
+# ...existing code...
+````
+
+Made changes.
+
+daddygongon: #file:mk_semi_lattice.rb:186-187 は何をしていますか？
+
+GitHub Copilot: 186-187行目（`Dir.mkdir(semi_dir) unless`）は、`.semi_lattice`ディレクトリが存在しない場合に作成しようとしています。ただし、`unless`の条件が未記載のため、文法的に不完全です。
+
+本来は  
+```ruby
+Dir.mkdir(semi_dir) unless Dir.exist?(semi_dir)
+```
+のように、ディレクトリが存在しない場合のみ作成する意図です。

data/.hyper_card/history/20251022_0725_add_color_index.md

@@ -0,0 +1,155 @@
+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)
+
+Add an option `-i`/`--index` to display node ids. Pass this option to the app and update node drawing to show id if enabled.
+
+````ruby
+# ...existing code...
+options = { layer: 2, init_step: :from_semi_lattice, show_index: false }
+OptionParser.new do |opts|
+  # ...existing code...
+  opts.on("-i", "--index", "Display node ids") do
+    options[:show_index] = true
+  end
+  # ...existing code...
+end.parse!
+# ...existing code...
+
+app = MkSemiLatticeData.new(input_path, 
+  with_semi_lattice_yaml: with_semi_lattice_yaml,
+  show_index: options[:show_index]
+)
+# ...existing code...
+
+update do
+  clear
+  app.edges.each(&:relax)
+  app.nodes.each { |n| n.relax(app.nodes) }
+  app.nodes.each(&:update)
+  app.edges.reverse.each(&:draw)
+  app.nodes.reverse.each do |n|
+    n.draw(app.selected == n)
+    # show node id if option enabled
+    if app.respond_to?(:show_index) && app.show_index
+      id = app.node_table.key(n)
+      Text.new(id.to_s, x: n.x + 20, y: n.y - 20, size: 16, color: 'black')
+    end
+  end
+end
+# ...existing code...
+````
+
+Made changes.
+
+daddygongon: #file:mk_semi_lattice.rb:86-89 に対応するようにMkSemiLatticeDataを修正
+
+GitHub Copilot: ### [mk_semi_lattice_graph.rb](file:///Users/bob/git_hub/mk_semi_lattice/lib/mk_semi_lattice/mk_semi_lattice_graph.rb)
+
+Update `MkSemiLatticeData` to accept and store `show_index` option, and provide it as an attribute for use in the main file.
+
+````ruby
+# ...existing code...
+
+class MkSemiLatticeData
+  attr_reader :nodes, :edges, :node_table
+  attr_accessor :selected, :shift_pressed
+  attr_accessor :show_index # add this line
+
+  def initialize(file = "dir_node_edge.yaml", with_semi_lattice_yaml: false, show_index: false)
+    @nodes = []
+    @edges = []
+    @node_table = {}
+    @selected = nil
+    @shift_pressed = false
+    @show_index = show_index # store the option
+    load_yaml_data_with_state(file, with_semi_lattice_yaml: with_semi_lattice_yaml)
+  end
+
+  # ...existing code...
+end
+# ...existing code...
+````
+
+Made changes.
+
+daddygongon: /Users/bob/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/mk_semi_lattice-0.1.0/lib/mk_semi_lattice/mk_semi_lattice_graph.rb:165:in `initialize': unknown keyword: :show_index (ArgumentError)
+とのエラー
+
+GitHub Copilot: ### [mk_semi_lattice_graph.rb](file:///Users/bob/git_hub/mk_semi_lattice/lib/mk_semi_lattice/mk_semi_lattice_graph.rb)
+
+Fix the error by adding `show_index` as an optional keyword argument to `MkSemiLatticeData#initialize` and storing it as an attribute.
+
+````ruby
+# ...existing code...
+class MkSemiLatticeData
+  attr_reader :nodes, :edges, :node_table
+  attr_accessor :selected, :shift_pressed, :show_index
+
+  def initialize(file = "dir_node_edge.yaml", with_semi_lattice_yaml: false, show_index: false)
+    @nodes = []
+    @edges = []
+    @node_table = {}
+    @selected = nil
+    @shift_pressed = false
+    @show_index = show_index
+    load_yaml_data_with_state(file, with_semi_lattice_yaml: with_semi_lattice_yaml)
+  end
+  # ...existing code...
+end
+# ...existing code...
+````
+
+Made changes.
+
+daddygongon: #file:mk_semi_lattice_graph.rb:193-226 の重複を取り除くrefactoringを掛けてください
+
+GitHub Copilot: ### [mk_semi_lattice_graph.rb](file:///Users/bob/git_hub/mk_semi_lattice/lib/mk_semi_lattice/mk_semi_lattice_graph.rb)
+
+Refactor `load_yaml_data_with_state` to remove duplication by extracting node/edge loading into a helper method.
+
+````ruby
+# ...existing code...
+
+class MkSemiLatticeData
+  # ...existing code...
+
+  def load_yaml_data_with_state(path, with_semi_lattice_yaml: false)
+    state_file = path
+    data = if with_semi_lattice_yaml && File.exist?(state_file)
+      YAML.load_file(state_file)
+    else
+      YAML.load_file(path, symbolize_names: true)
+    end
+    load_nodes_and_edges(data)
+  end
+
+  private
+
+  def load_nodes_and_edges(data)
+    nodes_data = data[:nodes]
+    edges_data = data[:edges]
+    nodes_data.each do |node_data|
+      id = node_data[:id]
+      name = node_data[:name]
+      type = node_data[:type]
+      color = node_data[:color]
+      file_path = node_data[:file_path]
+      x = node_data[:x] || rand(80..520)
+      y = node_data[:y] || rand(80..520)
+      fixed = node_data[:fixed] || false
+      label = @show_index ? "#{id}:#{name}" : name
+      add_node(id, x, y, label, type, color, file_path, fixed)
+    end
+    edges_data.each do |edge_data|
+      from_id = edge_data[:from]
+      to_id = edge_data[:to]
+      add_edge(from_id, to_id)
+    end
+  end
+  # ...existing code...
+end
+# ...existing code...
+````
+
+Made changes.