Greptile Summary

This PR re-adds support for the concatenated short-flag form (-Eval) in the environment() arg parser in src/env.rs. The existing code handled --env=val, -E=val, and -E val but silently dropped -Eval; the new branching logic adds that case by splitting at byte index 2 and checking the 2-char prefix against the known flag set.

Confidence Score: 5/5

This PR is safe to merge — the change is a targeted, well-tested fix with no regressions introduced.

No P0 or P1 issues found. The refactored branching logic correctly handles all four flag forms (-E val, --env val, -E=val, --env=val, and the new -Eval). Edge cases such as -E alone, -E=val, multi-value comma forms, and the run-subcommand boundary are all preserved.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[arg from iterator] --> B{starts with '-'?}
    B -- No --> C{in_run_subcommand?}
    C -- Yes --> D[break]
    C -- No --> E{is 'run' or 'r'?}
    E -- Yes --> F[set in_run_subcommand = true]
    E -- No --> G[continue]
    B -- Yes --> H{arg in arg_defs?\ne.g. '-E', '--env'}
    H -- Yes --> I[consume next arg as value\n'-E production']
    H -- No --> J{split_at_checked 2\nprefix in arg_defs?\nrest not starts with '='?}
    J -- Yes --> K[push rest as value\n'-Eproduction']
    J -- No --> L{split_once '='\nflag in arg_defs?}
    L -- Yes --> M[push value\n'-E=production' / '--env=production']
    L -- No --> N[ignore arg]

_{Reviews (2): Last reviewed commit: "fix(env): parse concatenated short form ..." | Re-trigger Greptile}