json-repair 0.11.1 → 0.11.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 288e3502829f51d11dbf2c3a9ab45f04dd44a1fa9ae5e00c1537f47c215aad96
-  data.tar.gz: 1ab99e5121ef3e73066157569bd85379a527cc1f44ecd7087d4f058133c4fcf0
+  metadata.gz: 2d7b2f30c2451f62471beabed342ff8360f644e4ae703fa3e0f5490e44d4f0d1
+  data.tar.gz: 254abaa4ab104a0cc6650d02f099ebb00b0600ef213b00866da266159b6c1a37
 SHA512:
-  metadata.gz: ffc4cd085d9a6aa5b45f3ee605a0fa043e20c68d63fdaab5c736acecbb17f160066d4412a76496300614545577d74d5ca4b232d4354adf625913753f8c0e8477
-  data.tar.gz: 190ea601a010b401fdf0c6aa52670a030ce81d39ebcd1af1f81c9fb2399baf6d074f3135b3b05814d1bcfcd24db53321a779b760c8eebe8d5a9ce2e029476754
+  metadata.gz: bfd417574888a7ff44a03870f48ab6e02c7bb7ae80189262e093425ba52e3943982f6ab36170c1cac6a3871e3d3379469b8319bbaa6defe8a0d473a7be6fe2a9
+  data.tar.gz: 208611ef2a09d4e3a5c91afe98334dfc7443a3db56de9624cfa82a3dea1e0c0480de18227646a96dac3c46f64af526e229f70dd9de4cd41cc1160a703b132a0b

data/.rubocop.yml

@@ -1,5 +1,15 @@
+# Merge our Exclude lists with RuboCop's defaults (vendor/**/*, tmp/**/*, …)
+# instead of replacing them — CI vendors gems into vendor/bundle, which
+# RuboCop must keep skipping.
+inherit_mode:
+  merge:
+    - Exclude
+
 AllCops:
   TargetRubyVersion: 3.0
+  Exclude:
+    # gitignored local planning notes and scratch tooling (see CLAUDE.md)
+    - docs/**/*
 
 Style/Documentation:
   Enabled: false

data/CHANGELOG.md

@@ -1,6 +1,26 @@
 # Changes
 
-### 2026-06-12 (0.11.1)
+### 2026-06-12 (0.11.2)
+
+* Fix the 0.11.0 doubled-colon repair silently mangling objects with a
+  stray junk word between pairs. `{"value_1": true, COMMENT "value_2":
+  "data"}` returned `{"value_1":true,"COMMENT":"value_2\": \"data"}`
+  (the junk word became a key and swallowed the real pair), and
+  `{ "key": "value" COMMENT "key2": "value2" }` returned a single
+  glued string value. Both shapes now raise "Object key expected"
+  again at the same positions as upstream
+  [jsonrepair](https://github.com/josdejong/jsonrepair) v3.14.0,
+  restoring the pre-0.11.0 behavior: the merge is skipped when the
+  pair already needed a missing-colon repair or the value string was
+  itself salvaged by the unescaped-quote repair — signals that the
+  pair was malformed in a way the merge would compound, not fix. The
+  salvage signal survives string concatenation: in
+  `{"a": "b" x "c" + "d": "e"}` the `+ "d"` segment no longer clears
+  it (caught in review by Copilot). All
+  0.11.0 repairs (canonical, greedy, escaped quotes, unquoted
+  keys/values) are unchanged. Go and Python `json_repair` instead
+  drop the junk word; we deliberately keep raising rather than
+  silently discarding input (see the 0.11.0 note).
 
 * Fix a `TypeError` crash on input ending in a lone backslash inside a
   string: `"abc\` now repairs to `"abc"` (likewise `"\` → `""`,

data/lib/json/repair/version.rb

@@ -2,6 +2,6 @@
 
 module JSON
   module Repair
-    VERSION = '0.11.1'
+    VERSION = '0.11.2'
   end
 end

data/lib/json/repairer.rb

@@ -32,6 +32,7 @@ module JSON
       @json = json
       @index = 0
       @output = +''
+      @repaired_unescaped_quote = false
     end
 
     def repair
@@ -295,8 +296,14 @@ module JSON
         end
 
         # repair: an object string value with unescaped quotes around a
-        # colon, like {"a": "b": "c"}
-        repair_doubled_colon if processed_value
+        # colon, like {"a": "b": "c"}. Skipped when this pair already
+        # needed a repair that makes the merge compound garbage: a
+        # missing colon (the "key" was a stray junk word, like
+        # {"v1": true, COMMENT "v2": "data"}) or a value glued together
+        # by the unescaped-quote repair (like
+        # {"k": "v" COMMENT "k2": "v2"}); both keep raising, matching
+        # upstream
+        repair_doubled_colon if processed_value && processed_colon && !@repaired_unescaped_quote
       end
 
       if @json[@index] == CLOSING_BRACE
@@ -315,9 +322,13 @@ module JSON
     # (the unescaped-quotes reading of the input). Greedy: keeps merging
     # while another `: "..."` follows. Only the string-colon-string
     # shape is repaired; anything else falls through to the regular
-    # error paths. Divergence from upstream (which raises "Object key
-    # expected" as of v3.14.0), matching the Go and Python json-repair
-    # libraries on the canonical case.
+    # error paths. The call site additionally requires the pair's colon
+    # to be present in the input and the value string to have parsed
+    # without the unescaped-quote repair (@repaired_unescaped_quote) —
+    # when either repair already fired, the pair was malformed in a way
+    # this merge would compound, not fix. Divergence from upstream
+    # (which raises "Object key expected" as of v3.14.0), matching the
+    # Go and Python json-repair libraries on the canonical case.
     def repair_doubled_colon
       loop do
         colon = @index
@@ -399,6 +410,9 @@ module JSON
     #   and fixing the string by inserting a quote there, or stopping at a
     #   stop index detected in the first iteration.
     def parse_string(stop_at_delimiter: false, stop_at_index: -1)
+      # fresh parse (the backtracking re-invocations below rebuild the
+      # string from scratch, so they reset too); see repair_doubled_colon
+      @repaired_unescaped_quote = false
       skip_escape_chars = @json[@index] == BACKSLASH
       if skip_escape_chars
         # repair: remove the first escape character
@@ -508,6 +522,7 @@ module JSON
 
           # repair unescaped quote
           str = "#{str[...o_quote]}\\#{str[o_quote..]}"
+          @repaired_unescaped_quote = true
         elsif stop_at_delimiter && unquoted_string_delimiter?(@json[@index])
           # we're in the mode to stop the string at the first delimiter
           # because there is an end quote missing
@@ -807,7 +822,12 @@ module JSON
         # repair: remove the end quote of the first string
         @output = strip_last_occurrence(@output, '"', strip_remaining_text: true)
         start = @output.length
+        # the segments form one logical string value: keep the doubled-colon
+        # guard's flag set when an earlier segment needed the unescaped-quote
+        # repair (parse_string resets it on entry)
+        repaired_earlier_segment = @repaired_unescaped_quote
         parsed_str = parse_string
+        @repaired_unescaped_quote ||= repaired_earlier_segment
         @output = if parsed_str
                     # repair: remove the start quote of the second string
                     remove_at_index(@output, start, 1)

data/sig/json/repairer.rbs

@@ -15,6 +15,8 @@ module JSON
 
     @output: ::String
 
+    @repaired_unescaped_quote: bool
+
     include Repair::StringUtils
 
     CONTROL_CHARACTERS: ::Hash[::String, "\\b" | "\\f" | "\\n" | "\\r" | "\\t"]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: json-repair
 version: !ruby/object:Gem::Version
-  version: 0.11.1
+  version: 0.11.2
 platform: ruby
 authors:
 - Aleksandr Zykov