fix: patching errors

[DecimalTurn]

• fix: tighten empty single-line inline tables and propagate to parent containers
• fix: unwrap InlineArrayItem when inserting into nested inline array

[Copilot]

Pull request overview

This PR fixes two patching failure modes in the TOML patcher: inserting into nested inline arrays and tightening single-line inline tables after removals (including empty tables), with new regression tests for both previously failing scenarios.

Changes:

• Unwrap InlineItem wrappers when the parent container is a nested InlineArray, so insert() receives a node with .items.
• Tighten single-line inline tables after removals even when the table becomes empty ({}), and attempt to propagate end-column shrinking to ancestor inline containers.
• Convert previously test.fails regression cases into passing tests, including corrected TOML quoting for special-character keys.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/patch.ts	Fixes parent unwrapping for nested inline-array inserts; improves inline-table tightening and adds ancestor end-position recalculation.
src/tests/patch.test.ts	Adds/updates regression tests to ensure the reported patching errors no longer occur.

Suppressed comments (1)

src/patch.ts:1688

• InlineArray recalculation only looks through KeyValue→InlineTable, but InlineArray items can directly wrap InlineTable/InlineArray values. If the last element is a nested inline container whose loc.end changed (directly or via prior recalculation), the parent array end can remain stale. Consider resolving the end column from lastItem.item when it is an InlineTable or InlineArray, and run this visitor on exit to support nested propagation.

    InlineArray: (node) => {
      if (node.items.length === 0) return;
      if (node.loc.end.line !== node.loc.start.line) return;
      const lastItem = node.items[node.items.length - 1];
      let innerEndCol = lastItem.loc.end.column;
      if (isInlineItem(lastItem) && isKeyValue(lastItem.item) && isInlineTable(lastItem.item.value)) {

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.