core-watch 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ed0f842d5e77d84a3a379c48ce82d1717485b56e2c33e5e6a86fcfae69f58b4
-  data.tar.gz: f75240b2e82db55ed719ef18711ddd36501b528f27d16f28a01a8097118cf21e
+  metadata.gz: 7214e1588d42429020551b10a17cb1259b66b7a794589b4d0048403abb3da14d
+  data.tar.gz: e3437fc62e1d34e5c9e8ce3429d3dc2f07d1a7b7bf5f57c87df16b7a43802819
 SHA512:
-  metadata.gz: 14ce6c0d7b2333652ec3a4d15cc885e7d88765a04bd5b6d93acf1a9985507b94f169510b6c4a8fc90a42daef22ef3c592fa44b48804ce4ab32f20372161f1adc
-  data.tar.gz: fd21f2387f8458c9ff96586c2e91640a2123810dc91b91393b03edea85644238f8096001e3bff5c53770cb9c04d05f24eb9e2800d812d155e256a6187cdebb3d
+  metadata.gz: 57964374546b40461c94a72f958b7e5e3e3df4f3fa7503f3063a8062de789cd3e6553f7456b05fda072da746b27db1b73c77acd445e3386198e4694ea73b3a46
+  data.tar.gz: 46c5c6a2e0dd0a9203729c2c719ff004f03bacd60f08fac8c1d678c3a471d62bf67c741fd19f10a527a388be9328f0547d5631da3d94052c8572fcfc2d3c66bd

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [v0.0.2](https://github.com/metabahn/corerb/releases/tag/2021-05-22)
+
+*released on 2021-05-22*
+
+  * `chg` [#30](https://github.com/metabahn/corerb/pull/30) Performance improvements to watch snapshots ([bryanp](https://github.com/bryanp))
+
 ## [v0.0.0](https://github.com/metabahn/corerb/releases/tag/2021-04-03)
 
 *released on 2021-04-03*

data/lib/core/watch/snapshot.rb

@@ -17,8 +17,7 @@ module Core
 
       def initialize(*paths, ignore: [], strategy: self.class.default_strategy)
         @ignore = ignore
-        @directories = {}
-        @files = {}
+        @watched = {}
         @strategy = strategy.new
 
         track(*paths)
@@ -39,35 +38,33 @@ module Core
         untrackable = []
         diff = Diff.new
 
-        # The mtime of the parent directory changes when its contents have changed (e.g. a file is added or removed).
-        # So, do the minimum amount of work necessary to detect added/removed files, then detect file changes below.
-        #
-        @directories.each do |path, current|
-          if !path.exist?
-            diff.removed(path)
-            untrackable << path
-          elsif @strategy.identify(path) != current
-            path.glob("*") do |each_path|
-              unless @directories.include?(each_path) || @files.include?(each_path)
-                next if ignore?(each_path)
-
-                diff.added(each_path)
-                trackable << each_path
+        @watched.each do |path, object|
+          case object[:type]
+          when :directory
+            # The mtime of the parent directory changes when its contents have changed (e.g. a file is added or removed).
+            # So, do the minimum amount of work necessary to detect added/removed files, then detect file changes below.
+            #
+            if @strategy.identify(path) != object[:identity]
+              path.glob("*") do |each_path|
+                unless @watched.include?(each_path)
+                  next if ignore?(each_path)
+
+                  diff.added(each_path)
+                  trackable << each_path
+                end
               end
-            end
 
-            trackable << path
-          end
-        end
-
-        @files.each do |path, current|
-          if !path.exist?
-            diff.removed(path)
-            untrackable << path
-          elsif @strategy.identify(path) != current
-            diff.changed(path)
-            trackable << path
+              trackable << path
+            end
+          when :file
+            if @strategy.identify(path) != object[:identity]
+              diff.changed(path)
+              trackable << path
+            end
           end
+        rescue Errno::ENOENT
+          diff.removed(path)
+          untrackable << path
         end
 
         track(*trackable)
@@ -82,13 +79,22 @@ module Core
         paths.each do |path|
           next if ignore?(path)
 
-          if path.file?
-            @files[path] = @strategy.identify(path)
+          type = if path.file?
+            :file
           elsif path.directory?
-            existed = @directories.include?(path)
-            @directories[path] = @strategy.identify(path)
+            :directory
+          end
+
+          if type
+            object = {
+              type: type,
+              identity: @strategy.identify(path)
+            }
+
+            existed = @watched.include?(path)
+            @watched[path] = object
 
-            unless existed
+            if type == :directory && !existed
               track(*path.glob("*"))
             end
           else
@@ -101,8 +107,7 @@ module Core
       #
       private def untrack(*paths)
         paths.each do |path|
-          @files.delete(path)
-          @directories.delete(path)
+          @watched.delete(path)
         end
       end
     end

data/lib/core/watch/version.rb

@@ -2,7 +2,7 @@
 
 module Core
   module Watch
-    VERSION = "0.0.1"
+    VERSION = "0.0.2"
 
     def self.version
       VERSION

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: core-watch
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Bryan Powell
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-20 00:00:00.000000000 Z
+date: 2021-05-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: core-async