foundries 0.1.4 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ee65151ab0ae7d4ef338d8dd8dec76fbf9f0746be24d99d814abdee8716c500a
-  data.tar.gz: 9a298a3ca012affe264f0467240f7a44b46decb2227f88f9eb2ca27c28f8bfad
+  metadata.gz: 12575eccffc55ee4735cc1fe6cb981ed1109d252d45b6435ab4b8333862f2663
+  data.tar.gz: 5ecb9d8707258542b76ec9e72caf4daa3619d72c0dff8f4778163a880302aa3b
 SHA512:
-  metadata.gz: 3b0eef54e2302dbda0dcc4ee7a9224c75e31f90b42498dabcdcac4a8e00318bc79bd34f343367cd2f03ab1cb969f72b01d4336c49ae4329eac5d7ebd136b64c1
-  data.tar.gz: 60f6748284f995381cc16f26a816f7f1ac9fac581948bbd0bc553c4839caf4cd9df6634982930d9017f896bdbb8d5f3edd3072aca7be19f5b9ba1620479c3a27
+  metadata.gz: bb1eba53555fdf59b449b06cef7e973516341d2806ceed56d42d2e26a43e58be877bf6459cb551b71610acb6790a7f001a3761b5098b25245bb8d3c58be6805e
+  data.tar.gz: ef7bd963c7c76e978a96ffed6562358a47cbfdb0da52e873f9613b4aecb3bcc357a4143282b7ff743ddada31e142fcdb8bd395035fa99291dbef4b890d06f419

data/CHANGELOG.md

@@ -5,19 +5,21 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
-## [0.1.4] - 2026-03-16
+## [0.1.6] - 2026-08-25
 
-### Added
+### Fixed
 
-- Recording module to analyze FactoryBot usage and suggest preset candidates (a294324)
-- parallel_tests support for Recording with per-worker output and merge rake task (8e59deb)
+- Snapshot files are read and written in binary, so captured data with non-ASCII bytes round-trips (6a7d3a3)
+- Restoring a snapshot no longer rewinds a sequence below its current value (df66836)
 
-### Changed
+## [0.1.5] - 2026-08-12
 
-- README now covers Recording feature and parallel_tests support (69fa45b)
+### Changed
 
-## [0.1.3] - 2026-03-13
+- Development dependencies are locked (1129826)
 
-### Added
+### Fixed
 
-- ancestor DSL and ancestors_for for path-based hierarchy building (7027e06)
+- Snapshot no longer caches a partial tree when a preset writes to a table that already held rows (2e9f863)
+- Lint under standard 1.56.0 (52be9eb)
+- Releases write a SHA512 checksum again (813a2ae)

data/lib/foundries/similarity/comparator.rb

@@ -28,7 +28,7 @@ module Foundries
       end
 
       def self.display_tree(tree)
-        tree.children.map(&:to_s).join(", ")
+        tree.children.join(", ")
       end
 
       private_class_method :preset_name, :display_tree

data/lib/foundries/similarity/structure_tree.rb

@@ -66,7 +66,7 @@ module Foundries
         if children.empty?
           name
         else
-          "#{name} > [#{children.map(&:to_s).join(", ")}]"
+          "#{name} > [#{children.join(", ")}]"
         end
       end
     end

data/lib/foundries/snapshot/adapters/postgres_adapter.rb

@@ -16,6 +16,10 @@ module Foundries
           @connection.select_value("SELECT NOT EXISTS (SELECT 1 FROM #{quoted(table_name)})")
         end
 
+        def count(table_name)
+          @connection.select_value("SELECT COUNT(*) FROM #{quoted(table_name)}")
+        end
+
         def capture(table_name, io)
           raw = @connection.raw_connection
           raw.copy_data("COPY #{quoted(table_name)} TO STDOUT") do
@@ -51,10 +55,14 @@ module Foundries
           )
           return unless seq
 
-          max_id = @connection.select_value(
-            "SELECT COALESCE(MAX(#{@connection.quote_column_name(pk)}), 0) FROM #{quoted(table_name)}"
+          # setval is not transactional, and rows can already be committed
+          # past the restored data (e.g. seeded outside this transaction), so
+          # the sequence must only ever move forward.
+          @connection.execute(
+            "SELECT setval(#{@connection.quote(seq)}, GREATEST(" \
+            "(SELECT COALESCE(MAX(#{@connection.quote_column_name(pk)}), 0) FROM #{quoted(table_name)}), " \
+            "(SELECT last_value FROM #{seq})))"
           )
-          @connection.execute("SELECT setval(#{@connection.quote(seq)}, #{max_id})")
         end
 
         private

data/lib/foundries/snapshot/adapters/sqlite_adapter.rb

@@ -13,7 +13,11 @@ module Foundries
         end
 
         def empty?(table_name)
-          @connection.select_value("SELECT COUNT(*) FROM #{quoted(table_name)}") == 0
+          count(table_name) == 0
+        end
+
+        def count(table_name)
+          @connection.select_value("SELECT COUNT(*) FROM #{quoted(table_name)}")
         end
 
         def capture(table_name, io)

data/lib/foundries/snapshot/store.rb

@@ -25,18 +25,41 @@ module Foundries
 
       # Record which tables are empty before the preset block runs.
       # Only these tables will be captured after the block completes.
+      #
+      # The already-populated tables are counted too, so #capture can tell
+      # whether the preset wrote into one of them. A preset that only updates
+      # existing rows leaves the count unchanged and slips past this check;
+      # presets insert, so the count is a good enough signal.
       def record_empty_tables
-        @capturable_tables = @adapter.table_names.select { |t| @adapter.empty?(t) }
+        @capturable_tables = []
+        @preexisting_counts = {}
+
+        @adapter.table_names.each do |table|
+          if @adapter.empty?(table)
+            @capturable_tables << table
+          else
+            @preexisting_counts[table] = @adapter.count(table)
+          end
+        end
       end
 
       def capture
+        spoiled = spoiled_tables
+        unless spoiled.empty?
+          warn "[Foundries] Not caching preset :#{@preset_name} — it wrote to " \
+            "#{spoiled.join(", ")}, which already held rows when it ran. " \
+            "Only tables the preset fills from empty can be snapshotted, so " \
+            "caching this would restore an incomplete tree."
+          return
+        end
+
         tables = @capturable_tables || @adapter.table_names
 
         tmp_dir = Pathname.new("#{cache_dir}.#{$$}.tmp")
         tmp_dir.mkpath
 
         tables.each do |table|
-          tmp_dir.join("#{table}.dat").open("w") do |f|
+          tmp_dir.join("#{table}.dat").open("wb") do |f|
             @adapter.capture(table, f)
           end
         end
@@ -54,7 +77,7 @@ module Foundries
             next unless file.size > 0
 
             table = file.basename(".dat").to_s
-            file.open("r") do |f|
+            file.open("rb") do |f|
               @adapter.restore(table, f)
             end
             @adapter.reset_sequence(table)
@@ -64,6 +87,17 @@ module Foundries
 
       private
 
+      # Pre-populated tables whose row count moved while the preset ran. Their
+      # new rows can't be told apart from the ones that were already there, so
+      # the snapshot would silently omit them.
+      def spoiled_tables
+        return [] unless @preexisting_counts
+
+        @preexisting_counts.filter_map do |table, count|
+          table unless @adapter.count(table) == count
+        end
+      end
+
       def cache_dir
         @cache_dir ||= Pathname.new(@storage_path).join(@preset_name)
       end

data/lib/foundries/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Foundries
-  VERSION = "0.1.4"
-  RELEASE_DATE = "2026-03-16"
+  VERSION = "0.1.6"
+  RELEASE_DATE = "2026-08-25"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foundries
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.6
 platform: ruby
 authors:
 - John Dowd
@@ -93,7 +93,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 4.0.16
 specification_version: 4
 summary: Declarative trees of related data using factory_bot
 test_files: []