dcdpr / jp / 24270450251

67%

push

github

feat(config): Add `dedup` option to `MergedVec` (#537)

A new `dedup` field on `MergedVec<T>` removes duplicate items after
merging, using order-preserving dedup.

The flag accepts `true`, `false`, or `"inherit"`. Omitting the field (or
using `"inherit"`) means "no opinion" — the flag propagates from the
previous merge in the chain. Once any config in the chain enables it,
all subsequent merges deduplicate unless a later config explicitly sets
it to `false`.

`conversation.attachments` now defaults to `dedup = true`, preventing
the same attachment from appearing more than once when multiple config
layers are merged. To opt out:

    [conversation.attachments]
    dedup = false

Several related fixes are also included: `FillDefaults` now propagates
correctly for `system_prompt_sections`, `instructions`, `attachments`,
and `PartialReasoningConfig`; `is_empty()` on `MergeableVec` and
`MergeableMap` now accounts for metadata so a `Merged` with no items but
active flags is no longer treated as empty; and `vec_dedup` is now
order-preserving instead of sorting before deduplicating.

---------

Signed-off-by: Jean Mertz <git@jeanmertz.com>

104 of 135 new or added lines in 10 files covered. (77.04%)

21404 of 32829 relevant lines covered (65.2%)

250.16 hits per line