dcdpr / jp / 18877312025

58%

push

github

enhance(config): Add merge strategy support for `instructions` field (#297)

Users can now configure the `instructions` field with explicit merge
strategies, allowing finer control over how configuration layers
combine. This aligns with the existing merge strategy support for
`system_prompt`.

The instructions field now accepts either a plain array (using `replace`
strategy by default) or an object with `value` and `strategy` fields.
For example:

```toml
[assistant.instructions]
strategy = "append"
value = [{ title = "Additional instruction" }]
```

To support this, a new `MergeableVec<T>` type was introduced that
mirrors the `MergeableString` pattern, providing consistent merge
behavior across different configuration value types.

Additional changes:

Renamed string merge types for consistency and clarity:

- `StringWithMerge` → `MergeableString`
- `StringWithStrategy` → `MergedString`
- `StringMergeStrategy` → `MergedStringStrategy`

Restructured the internal merge module by splitting string and vec merge
strategies into separate submodules for better organization.

---------

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

539 of 588 new or added lines in 7 files covered. (91.67%)

7411 of 15548 relevant lines covered (47.67%)

14.84 hits per line