fast_schema_dumper 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c7ef2b48990a434194a1ac9ddd7394fbef2a5ad40befb705209ccf01edb19c4a
-  data.tar.gz: 6360a00616008fea6f45913597ce0fc9ebbca688243bb1a204fb0abb33922ea7
+  metadata.gz: 9e74649c95b51a718a9e5d2d4583adb0eeaa752f72116e4a21235e94b7aed768
+  data.tar.gz: b17d033b4449cd527eed2b50ab3c5b5851eca822c9329fd8d698d98733bb50e9
 SHA512:
-  metadata.gz: 6a701f24b8eacb312ad56c60c96edd12aa795fa27ecf8e59e511455e55ff36efca84e6ecb9598d7ec197c0f3120a33fd487f3b1d59d34098cf469637c37aec85
-  data.tar.gz: 06cd91e908371c31c24bf90ba9d21c8f359d8a0007ceb2698a7e5712280bf191d010edf90360c72b2975b72d8df7a001bf1776591d4c23b1d9a0252301bd307c
+  metadata.gz: 8a94ed1a0bf13a02910a3a7a69f154dc1d5f5926d6bdeafedd5a241855052e6d03507825ad9cf93d2788f59a2ca19e031fd81a370fca578d4b88deed5c9db4fb
+  data.tar.gz: 53b42423236881b8887180819cee487cad61d90433750a18f5759059dd87dc1460fc98314d512c1f48d451e3d9da284b2e5e7429410d956ccd21b68781a21582

data/lib/fast_schema_dumper/fast_dumper.rb

@@ -338,8 +338,14 @@ module FastSchemaDumper
         check_clause = constraint[:check_clause]
 
         # drop redundant parentheses at the most outer level for compatibility with the original dumper
+        # Example: "((a > 0) and (b > 0))" => "(a > 0) and (b > 0)"
         if check_clause.start_with?("(") && check_clause.end_with?(")")
-          check_clause = check_clause[1..-2]
+          # Check if removing outer parens would break the expression by ensuring parentheses are balanced after removal
+          # For example, it shouldn’t remove the outer parentheses as in the following case, because doing so would break the balance: "(a > 0) and (b > 0)"
+          inner = check_clause[1..-2]
+          if balanced_parentheses?(inner)
+            check_clause = inner
+          end
         end
 
         check_clause.gsub!(/\\'/, "'") # don't escape single quotes for compatibility with the original dumper
@@ -525,5 +531,15 @@ module FastSchemaDumper
 
       idx_def
     end
+
+    def balanced_parentheses?(str)
+      depth = 0
+      str.each_char do |char|
+        depth += 1 if char == '('
+        depth -= 1 if char == ')'
+        return false if depth < 0
+      end
+      depth == 0
+    end
   end
 end

data/lib/fast_schema_dumper/version.rb

@@ -1,3 +1,3 @@
 module FastSchemaDumper
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fast_schema_dumper
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Daisuke Aritomo