* Remove remote host restarting logic

* RemoteServiceName

* Remove dependency on Workspace

* Refactoring

* Fixes

Add back missing strings

Restore the 'isWriteOnly' parameter removed in previous commit, and actually set Modifiers.IsWriteOnly flag based on this parameter value. This was caught by a unit test failure.

My prior attempt to enable this as a build warning in https://github.com/dotnet/roslyn/pull/44164 did not work due to #44201. This PR actually adds the build enforcement and fixes the violations.

Removing the workaround for https://github.com/dotnet/roslyn/issues/42166 identified that IDE0051/IDE0052 were not getting enforced on the build. I will file a follow-up issue to investigate it, but this commit fixes the existing IDE0051/IDE0052 violations in IDE projects where they are set to warning.

```
# IDE0060: Remove unused parameter
dotnet_diagnostic.IDE0060.severity = warning
```

Manual fixes for IDE0059 violations for assignments of a non-constant value (we do not offer code fixes to remove such assignments which could have side effects)

This reverts commit 71716fd5.

This is to keep consistency with Project.RemoveDocuments which already
exists.