Pass of lint for .proto files

[cmaglie]

I've used buf linter to perform linting of the .proto files.

The linter warnings rules fixed by this PR are:

• SERVICE_SUFFIX
• RPC_RESPONSE_STANDARD_NAME
• RPC_REQUEST_STANDARD_NAME
• RPC_REQUEST_RESPONSE_UNIQUE
• ENUM_VALUE_PREFIX
• ENUM_VALUE_UPPER_SNAKE_CASE
• FIELD_LOWER_SNAKE_CASE
• PACKAGE_VERSION_SUFFIX
• PACKAGE_DIRECTORY_MATCH

The changes above should not affect the behavior of the RPC API.
Anyway, those are breaking changes since they change the signature of the RPC API and the CLI JSON output in some cases.

The remaining linter warnings require a more in-depth analysis to be fixed:

cc/arduino/cli/commands/v1/lib.proto:115:5:Enum zero value name "LIBRARY_SEARCH_STATUS_FAILED" should be suffixed with "_UNSPECIFIED".
cc/arduino/cli/commands/v1/lib.proto:282:5:Enum zero value name "LIBRARY_LAYOUT_FLAT" should be suffixed with "_UNSPECIFIED".
cc/arduino/cli/commands/v1/lib.proto:289:5:Enum zero value name "LIBRARY_LOCATION_IDE_BUILTIN" should be suffixed with "_UNSPECIFIED".
cc/arduino/cli/debug/v1/debug.proto:30:24:RPC request type "DebugConfigRequest" should be named "GetDebugConfigRequest" or "DebugServiceGetDebugConfigRequest".
cc/arduino/cli/monitor/v1/monitor.proto:53:21:Enum zero value name "TARGET_TYPE_SERIAL" should be suffixed with "_UNSPECIFIED".

[kittaakos]

The new Arduino IDE cannot wait to use the new gRPC API 💄

[silvanocerza]

Seems alright to me, I'd add a task and related workflow to check formatting otherwise I fear we'll drift into badly formatted files again.

Requested changes have been made. Thanks!

[per1234]

I'd add a task and related workflow to check formatting otherwise I fear we'll drift into badly formatted files again.

I agree, and suggest doing the same with buf lint.

[silvanocerza]

Just one small change in UPGRADING.md, the rest is good.

[silvanocerza]

Perfect! 🚀