jhheider / edikt / 29128170084

96%

push

github

fix(jsonc): edit comma correctness — multi-del + trailing commas (#50)

* fixes a bug with multiple successive del() ops

* fix(jsonc): never manufacture an invalid trailing comma on edit

Appending a new last key/element to a multi-line JSON object/array stamped
a comma after it, and deleting the last object member left the previous
member's separator comma dangling before `}`. Both produce invalid strict
JSON; JSON5/JSONC tolerate trailing commas, so edikt's own lenient reader
round-tripped the damage and it stayed hidden until a strict consumer
(Claude Desktop, serde_json, python json) rejected the file.

Fix, format-preserving: the last appended element only gets a trailing
comma when the container was already trailing-comma style; deleting a last
member with no comma of its own strips the previous member's now-dangling
comma (mirroring delete_element for arrays). A container that already uses
trailing commas keeps that style. Adds tests for both bugs and both
preservation cases, asserting exact bytes (stronger than the lenient parser
that hid the bug).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0157vKUFSTQFdfCK3nLrxyqK

* release: bump edikt-jsonc and edikt to 0.2.2 (trailing-comma fix)

Patch bump for the JSONC edit comma-correctness fix: edikt-jsonc 0.2.1 ->
0.2.2, the edikt binary 0.2.1 -> 0.2.2, and the workspace dep pointer to
match so the published binary requires the fixed crate.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0157vKUFSTQFdfCK3nLrxyqK

---------

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

69 of 72 new or added lines in 4 files covered. (95.83%)

9605 of 9992 relevant lines covered (96.13%)

85.06 hits per line