🐛 Validate field default ordering in oneline config

[ubmarco]

Problem

The oneline parser maps comma-separated values to needs_fields strictly by position (index). When a field without a default is defined after a field with a default, it becomes unreachable if the user omits the optional fields — exactly like Python's SyntaxError: parameter without a default follows parameter with a default.

Example invalid configuration:

needs_fields = [
  { "name" = "id" },
  { "name" = "implements", "type" = "list[str]", "default" = [] },
  { "name" = "type", "default" = "impl" },
  { "name" = "title" },  # required, but comes after optional fields
]

With this config, @need MY_ID, My Title maps My Title to implements (index 1) instead of title (index 3), and title is silently skipped.

Solution

Add check_fields_default_order() validation to OneLineCommentStyle that rejects configurations where a field without a default follows a field with a default. This is checked alongside existing schema, required-field, and uniqueness validations in check_fields_configuration().

Changes

• config.py: New check_fields_default_order() method, integrated into check_fields_configuration()
• test_analyse_config.py: Negative test case exercising the validation

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.80%. Comparing base (d30193c) to head (410e560).

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
- Coverage   89.87%   89.80%   -0.07%     
==========================================
  Files          29       29              
  Lines        2548     2560      +12     
  Branches      297      300       +3     
==========================================
+ Hits         2290     2299       +9     
- Misses        163      165       +2     
- Partials       95       96       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.