File commands have wrong arguments (e.g. Revert File)

[bpasero]

Steps to Reproduce:

1. open a dirty file
2. add a breakpoint here
3. select File > Revert File
4. see how the second argument is typed as a URI but actually is not

@isidorn I suggest you make a pass over the file commands, there are probably more cases. I only noticed this issue because I wanted to revert a file and that failed and the error message was not bubbling up because the error message assumes the second argument is a URI.

I guess this would also happen if running a command via command palette.

[bpasero]

@isidorn actually it looks like you already fixed this issue sort of, I can no longer reproduce the error case. However, the command is still typed such as the second argument would be a URI which is dangerous because in this case where I run it from the menu the second argument is something else:

I think the commands need a URI.isURI check to make sure the argument is what was expected.

[bpasero]

@isidorn nicer than {} is to just write object as TypeScript type.

[isidorn]

Nice idea, fixed via 15e754e