Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request addresses a bug in how mise resolves task names for _default files that include file extensions. The fix ensures that the extension is correctly stripped, allowing _default files to be properly identified and their parent directory to be used as the task name. This improves the accuracy and consistency of task resolution within the system.

Highlights

• Task Name Resolution: Corrected an issue where _default files with extensions were not properly recognized, leading to incorrect task names. For example, test/_default.sh will now correctly resolve to test instead of test:_default.
• Code Logic Update: Modified the name_from_path function in src/task/mod.rs to strip file extensions from _default file names before determining the task name.
• Improved Test Coverage: Added new end-to-end and unit tests to validate the correct resolution of _default tasks, including scenarios with and without file extensions.

Greptile Summary

This PR fixes a bug where _default files with extensions (e.g., test/_default.sh) were not resolved to the parent task name — they appeared as test:_default instead of test.

• Root cause: The previous check name.strip_suffix(":_default") only matched a literal :_default suffix, so any file like _default.sh or _default.py passed through unchanged.
• Fix: The new logic uses rsplit_once(':') to split the colon-joined name into its parent path and last segment, then calls the existing strip_extension() helper to normalize the last segment before comparing it to "_default". This correctly handles all extension variants.
• Tests: Five new unit test cases are added to test_name_from_path covering no-extension, .sh, and .js variants at single and double nesting depth. The e2e test test_task_default is extended to create and run a foo/_default.sh task.
• The fix is consistent with how the rest of name_from_path operates and does not change behavior for any previously working paths.

Confidence Score: 5/5

• This PR is safe to merge — the fix is minimal, well-targeted, and thoroughly tested.
• The change is a small, focused correction to a single helper function. The new logic correctly handles both the pre-existing no-extension case (regression tested) and the new extension variants. Unit tests cover three extensions and two nesting depths; the e2e test validates end-to-end behavior. No unrelated code paths are touched.
• No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[path: foo/_default.sh] --> B[strip_prefix + join components with ':']
    B --> C["name = 'foo:_default.sh'"]
    C --> D{"rsplit_once(':')"}
    D -->|Some| E["parent='foo', last='_default.sh'"]
    E --> F{"strip_extension(last) == '_default'?"}
    F -->|Yes| G["return Ok('foo')"]
    F -->|No| H["return Ok(name)"]
    D -->|None| H
    I["OLD: name.strip_suffix(':_default')"] -->|Only matched no-extension| J["return 'foo' for 'foo:_default'"]
    I -->|Missed| K["'foo:_default.sh' returned unchanged ❌"]

_{Last reviewed commit: "[autofix.ci] apply a..."}