Include .yml and .markdown in Drive import file picker

[anshul-garg27]

Description

The Warp Drive import flow lets users pick workflow (YAML) and notebook (Markdown) files. The picker uses FileType::Yaml and FileType::Markdown from warpui_core::platform::file_picker, which previously listed only yaml and md. That doesn't match what the importer is willing to ingest:

• app/src/drive/import/nodes.rs (TryFrom<&Path> for FileType) accepts both .yaml and .yml for workflows.
• warp_util::file_type::is_markdown_file recognises both .md and .markdown (MARKDOWN_EXTENSIONS).
• The import modal's helper text in modal_body.rs already advertises "md, yaml, yml" to users.

The result: a user can drag-and-drop a .yml workflow or .markdown notebook and it imports cleanly, but the file picker filter hides those same files when they click "Choose files…". This change brings the picker in line with what the rest of the import path already accepts.

I left FileType::Image alone — that one has a separate user-facing contract (the theme creator body explicitly says "(.png, .jpg)"), so broadening it would need its own design decision.

Testing

• Added two unit tests covering both extensions for FileType::Yaml and FileType::Markdown.
• cargo fmt -p warpui_core -- --check
• Couldn't run the full cargo nextest locally (Metal toolchain not available on this machine), the change is well-scoped to a const-array literal so risk is low.

Server API

No server changes.

Agent Mode

Not applicable.

Changelog Entries

• Warp Drive import file picker now lists .yml and .markdown files in addition to .yaml and .md.

[oz-for-oss]

@anshul-garg27

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I approved this pull request. No matching stakeholder was found for the changed files, so no human reviewers were requested.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[oz-for-oss]

Overview

This PR expands the Drive import file picker extension filters so YAML accepts .yaml/.yml and Markdown accepts .md/.markdown, aligning picker behavior with the importer and adding focused unit coverage.

Concerns

• No blocking correctness, security, or maintainability concerns found in the changed lines.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[bnavetta]

Per repo conventions, could you put this in a file_picker_tests.rs file alongside this one? You can then use a #[path] attribute to add that file as the test module.

[bnavetta]

Left a comment on test structure, but the core change looks good!

[cla-bot]

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Anshul Garg.
This is most likely caused by a git client misconfiguration; please make sure to:

1. check if your git client is configured with an email to sign commits git config --list | grep email
2. If not, set it up using git config --global user.email email@example.com
3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

[anshul-garg27]

@bnavetta thanks! Moved the tests to file_picker_tests.rs and wired it up via #[cfg(test)] #[path = "file_picker_tests.rs"] mod tests; to match the convention used in clipboard_utils, presenter, etc.

[oz-for-oss]

@anshul-garg27

I'm checking this implementation PR for association with a likely matching ready issue.

Powered by Oz