Greptile Summary

Removes the !task.args.is_empty() / !a.args.is_empty() guards from the {{usage.*}} dependency-template rendering paths in both run.rs and deps.rs. The guards were preventing parse_usage_values_from_task from running when no CLI args were passed, even though the usage spec may define defaults; this caused deps with unrendered templates to be silently filtered out. The fix correctly scopes the work to tasks that actually contain usage-template references via the existing has_usage_deps predicate.

Confidence Score: 5/5

Safe to merge — the fix is narrowly scoped, logically correct, and has no regressions on existing tests.

Both changes are symmetric and minimal: they replace an overly-restrictive args.is_empty() guard with the already-existing has_usage_deps predicate. The only remaining finding is a P2 test gap for the specific default-value scenario, which does not affect correctness of the existing test suite.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Task invoked] --> B{has_usage_deps\nin depends/wait_for?}
    B -- No --> G[resolve_depends\nskips usage templates]
    B -- Yes --> C[parse_usage_values_from_task\nparses spec + applies defaults]
    C --> D{usage_values\nempty?}
    D -- Yes\nno defaults/required arg missing --> G
    D -- No --> E[render_depends_with_usage\nreplaces usage.* templates]
    E --> F[resolve_depends\nmatches resolved dep names]
    G --> H[Tasks run without rendered deps]
    F --> I[Tasks run with correct deps]

    style H fill:#ffcccc
    style I fill:#ccffcc

_{Reviews (3): Last reviewed commit: "[autofix.ci] apply automated fixes" | Re-trigger Greptile}