chocomint 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ff362c173d26ea61776f090575fc846b138ef7000ba0d9dd5df90adaa5fd3ef
-  data.tar.gz: bea2b7dcb44f8f7d525ed46586576ab3a6fc38a6439ab58de91f3739561b671a
+  metadata.gz: c1e486a61190ec1d3548de09e65d74ea64d42a664c3d536c0e8013f61e6e2c1c
+  data.tar.gz: b7e23eedd909b6d2dffaeaa341721e7ccbd3bc91a2556a87d2654991c9475443
 SHA512:
-  metadata.gz: 9945d52d514f88f02fe244c39915b52d8d50ee73e128416f93bab1db2f3a6a0f97c229e53fb28b79ccf6f66b5e7549e81f06fb102a9279d79d89bcea9a87273c
-  data.tar.gz: c1ab4e9ae01521a6d384eaf682b46807f3f8bbaf93ffd7c9adbfc8f8d700b13779cb768482693d326921475b2c38143b615035729084e668d3e2e06a45268325
+  metadata.gz: 5234597d4d58068f04609037aaf53d058b48a0ddfea9a12240cc64c84ace0371b5277aee1442dcea8e1b8239449a7da0e32f976aca1672f4da3b5dfa90ee6645
+  data.tar.gz: e42a3f878d613ce09722db0929d745d3aaf3fdda93698eeb691203834063a9bf3090d10bf5aca20f7797391b76305885fcf89c63e6c03c67ae58580520a46704

data/lib/chocomint/edit_handlers.rb

@@ -28,8 +28,8 @@ module Chocomint
     def handle_edit_files(req, res)
       return edit_method_guard(res) unless req.request_method == "GET"
 
-      entries = list_workspace_files
-      edit_json(res, 200, "files" => entries, "dirs" => list_workspace_empty_dirs(entries),
+      files, empty_dirs = walk_workspace_tree
+      edit_json(res, 200, "files" => files, "dirs" => empty_dirs,
                            "root" => relative_root, "abs_root" => edit_root.tr("\\", "/"))
     rescue Chocomint::Error => e
       edit_json(res, 500, "error" => e.message)
@@ -39,60 +39,60 @@ module Chocomint
     MAX_LISTED_FILES = 2000
 
     # 中身を展開しない VCS メタデータ・依存パッケージ等 (数千件で一覧を埋め尽くすため除外する)。
-    EXCLUDED_TREE_DIRS = %w[.git .hg .svn .venv venv node_modules .bundle vendor/bundle].freeze
+    # ディレクトリ名 (basename) で判定するため、"vendor/bundle" のようなパス階層は含めない。
+    EXCLUDED_TREE_DIRS = %w[.git .hg .svn .venv venv node_modules .bundle].freeze
 
-    def list_workspace_files
+    # workspace ツリーを1度だけ手動再帰し、[ファイル相対パス, 空ディレクトリ相対パス] を返す。
+    #
+    # Dir.glob("**/*") はツリー全体を舐めてから除外判定するため、node_modules を大量に
+    # 含むホームディレクトリ (例: C:\Users\<name>) を起点にすると応答が返らず WORKING
+    # DIRECTORY が空のままになる。除外ディレクトリはその配下へ降りる前に枝刈りし、
+    # MAX_LISTED_FILES 到達で打ち切ることで巨大ツリーでも実用的な時間で応答する。
+    def walk_workspace_tree
       root = edit_root
-      return [] unless File.directory?(root)
+      return [[], []] unless File.directory?(root)
 
       files = []
-      # File::FNM_DOTMATCH で "." 始まりの隠しファイル/ディレクトリも列挙対象に含める。
-      Dir.glob("**/*", File::FNM_DOTMATCH, base: root).each do |rel|
-        rel = rel.tr("\\", "/")
-        # "." / ".." など自己・親参照は除外する (FNM_DOTMATCH で拾われるため)。
-        next if [".", ".."].include?(File.basename(rel))
-        # .git などの配下は展開しない (メタデータで一覧が埋まるのを防ぐ)。
-        next if EXCLUDED_TREE_DIRS.any? { |d| rel == d || rel.start_with?("#{d}/") }
-
-        abs = File.join(root, rel)
-        next unless File.file?(abs)
-
-        files << rel
-        break if files.size >= MAX_LISTED_FILES
-      end
-      files.sort
-    end
+      # 中身を1件も持たないディレクトリ (files からツリー復元できない) を別途フロントに伝える。
+      empty_dirs = []
+      stack = [""] # root からの相対ディレクトリ (root 自身は "")
 
-    # files に1件も含まれないディレクトリ (空ディレクトリ、または空ディレクトリのみを含むディレクトリ) を列挙する。
-    # ファイル一覧からはツリー構築できないため、これらを別途フロントエンドに伝えてツリーに表示させる。
-    def list_workspace_empty_dirs(files)
-      root = edit_root
-      return [] unless File.directory?(root)
-
-      covered = files.each_with_object(Set.new) do |rel, set|
-        parts = rel.split("/")
-        parts.pop
-        prefix = +""
-        parts.each do |part|
-          prefix = prefix.empty? ? part : "#{prefix}/#{part}"
-          set << prefix
-        end
-      end
+      until stack.empty?
+        break if files.size >= MAX_LISTED_FILES
 
-      dirs = []
-      Dir.glob("**/*", File::FNM_DOTMATCH, base: root).each do |rel|
-        rel = rel.tr("\\", "/")
-        next if [".", ".."].include?(File.basename(rel))
-        next if EXCLUDED_TREE_DIRS.any? { |d| rel == d || rel.start_with?("#{d}/") }
-        next if covered.include?(rel)
+        rel_dir = stack.pop
+        abs_dir = rel_dir.empty? ? root : File.join(root, rel_dir)
 
-        abs = File.join(root, rel)
-        next unless File.directory?(abs)
+        child_count = 0
+        begin
+          Dir.each_child(abs_dir) do |name|
+            next if name == "." || name == ".."
+
+            rel = rel_dir.empty? ? name : "#{rel_dir}/#{name}"
+            abs = File.join(abs_dir, name)
+
+            if File.directory?(abs)
+              child_count += 1
+              # 除外ディレクトリは配下へ降りない (中身も一覧に出さない)。
+              next if EXCLUDED_TREE_DIRS.include?(name)
+
+              stack.push(rel)
+            elsif File.file?(abs)
+              child_count += 1
+              files << rel
+              break if files.size >= MAX_LISTED_FILES
+            end
+          end
+        rescue SystemCallError
+          # 権限不足などで開けないディレクトリはスキップする (ホーム配下で発生しうる)。
+          next
+        end
 
-        dirs << rel
-        break if dirs.size >= MAX_LISTED_FILES
+        # 子を1件も持たないディレクトリはツリーに現れないため、明示的に伝える (root は除く)。
+        empty_dirs << rel_dir if child_count.zero? && !rel_dir.empty?
       end
-      dirs.sort
+
+      [files.sort, empty_dirs.sort]
     end
 
     # ---- ファイル取得 -------------------------------------------------------

data/lib/chocomint.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Chocomint
-  VERSION = "1.0.1"
+  VERSION = "1.0.2"
 end
 
 require_relative "chocomint/errors"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chocomint
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - ひかり (Himeyama)