Add support for the dolphin file manager

[BlueDrink9]

No description provided.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 945410121e

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[FireChickenProductivity]

Just a note: (refresh | reload) it: browser.reload() is already a command in Community. I wonder if it might be worth copying that spoken form if you are using reload it and if we should at some point abstract out reloading from just being a browser thing.

[FireChickenProductivity]

Another note: file rename is defined in several contexts. We might want to put that behind a tag so we can use action overriding.
(These notes are mostly intended for the next backlog session)

[BlueDrink9]

I was surprised to find that file rename was not already in the file manager tag.

I find myself creating a reload or refresh command for quite a lot of applications, comma it definitely seems generally useful outside of just browsers. That being said, it does feel like a nuisance to have a tag for just a single command, but I suppose that's better than making users redefine the command for every application.

[nriley]

From the Community Backlog session — please address all of @FireChickenProductivity 's issues. Thanks!

[StructSeeker]

You seems to omit a from typing import Optional that cause error message in windows?
@BlueDrink9

[nriley]

Yup, sorry we did not test it! The needed changes are in another PR currently; see fab4300. @BlueDrink9 if you want to submit as a separate PR then happy to merge ASAP.

[FireChickenProductivity]

Adding the optional import actually causes another error because it makes the type signature inconsistent with the base action.

[FireChickenProductivity]

I submitted the quickest fix: #2193

[BlueDrink9]

Oops, thank you! I'm on beta so for whatever reason didn't get the error; or maybe just didn't notice it in the log. Thanks FireChicken, and sorry!