Pull request overview

This PR updates the schema override-path updater to support appending to lists when the override key targets an index equal to len(list), enabling CLI/YAML overrides to add new list items rather than only replacing existing ones.

Changes:

• Allow list index == len(list) to mean “append” in update_value_by_location.
• Add a property-based test to verify the new append behavior.
• Adjust the out-of-bounds index test to reflect the new valid boundary (len(list) is now allowed).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

tests/schema/test_override_dictionary.py:204

• To validate appending works for an empty list as well as non-empty lists, consider setting min_size=0 so Hypothesis includes the empty-list case.

    @settings(deadline=None)
    @given(items=st.lists(st.text(max_size=10), min_size=1, max_size=5))
    def test_list_index_at_len_appends_to_list(self, items: list[str]) -> None:

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