fix: support aggregate field dictionaries in get_list (backport #65)

[mergify]

---

This is an automatic backport of pull request #65 done by [Mergify](https://mergify.com).

[greptile-apps]

Confidence Score: 5/5

Safe to merge — all three changed files are logically consistent, the previously flagged merge conflict is resolved, and the new test covers the fixed behavior.

The refactored coerce_field_list correctly handles the three input shapes (None, str, list) and properly threads dict entries through without modification. The type alias, updated signatures, and docstrings are all consistent with the implementation. No regressions were found in the existing code paths.

No files require special attention.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["coerce_field_list(value)"] --> B{value is None?}
    B -- Yes --> C[return None]
    B -- No --> D{value is str?}
    D -- Yes --> E[strip value]
    E --> F{stripped is empty?}
    F -- Yes --> C
    F -- No --> G[try json.loads]
    G -- success, is list --> H[value = parsed list]
    G -- fail or not list --> I["value = stripped.split(',')"]
    D -- No, already a list --> J[use value as-is]
    H --> K
    I --> K
    J --> K["list comprehension over value"]
    K --> L{entry is dict?}
    L -- Yes, non-empty --> M[keep dict as-is]
    L -- No --> N["str(entry).strip(), skip if empty"]
    M --> O[return list of FrappeSelectField]
    N --> O

Loading

%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A["coerce_field_list(value)"] --> B{value is None?}
    B -- Yes --> C[return None]
    B -- No --> D{value is str?}
    D -- Yes --> E[strip value]
    E --> F{stripped is empty?}
    F -- Yes --> C
    F -- No --> G[try json.loads]
    G -- success, is list --> H[value = parsed list]
    G -- fail or not list --> I["value = stripped.split(',')"]
    D -- No, already a list --> J[use value as-is]
    H --> K
    I --> K
    J --> K["list comprehension over value"]
    K --> L{entry is dict?}
    L -- Yes, non-empty --> M[keep dict as-is]
    L -- No --> N["str(entry).strip(), skip if empty"]
    M --> O[return list of FrappeSelectField]
    N --> O

Loading

_{Reviews (2): Last reviewed commit: "chore: resolve merge conflict" | Re-trigger Greptile}