fogfish / arcnet-cli / 29356384995

80%

main: 81%

Pull #19

github

[Spec Kit] Fix BUG-002: union/append outcome must reflect actual result

mergeTexts/mergeAttrs's isListMerge branch (internal/core/merge.go)
always reported a union/append predicate as "appended" (or "created")
based on which merge behavior dispatched, never checking whether the
dispatch actually changed the value. A contribution that mergeText's own
near-duplicate detection correctly drops (or unionPredicates correctly
deduplicates) was still reported "appended", misleading arc apply
--verbose users into thinking content was added when nothing changed.

Now compares the merged value against the existing one and reports
OutcomeUnchanged for a genuine no-op, matching every other dispatch
class's own accuracy standard (spec 012 FR-019).

Adds regression coverage in internal/core/merge_test.go (exact and
near-duplicate text, duplicate Attrs list entry) and an E2E test in
cmd/arc/graph/apply_test.go reproducing the reported scenario; fixes
TestApplyReportsStepPerNode's fixture, which had encoded the buggy
expectation.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

551 of 731 new or added lines in 8 files covered. (75.38%)

3878 of 4830 relevant lines covered (80.29%)

0.93 hits per line