feat: SendRequest and SendNotification now uses ellipsis for parameters

[cmaglie]

Motivation

Improves the API of the msgpackrpc package.

Change description

Previously, the msgpackrpc.Connection had two methods requiring an array of parameters:

SendRequest(ctx context.Context, method string, params []any) (reqResult any, reqError any, err error)
SendNotification(method string, params []any) error

Now those methods do not require an explicit array anymore, but use the ellipsis ... notation:

SendRequest(ctx context.Context, method string, params ...any) (reqResult any, reqError any, err error)
SendNotification(method string, params ...any) error

This change makes the use of the API cleaner and less ambiguous, for example:

conn.SendRequest(ctx, "ping", []any{"hello", 12}) // calls ping("hello", 12)
conn.SendRequest(ctx, "ping", []any{})            // calls ping()
conn.SendRequest(ctx, "ping", nil)                // invalid call

can now be written as:

conn.SendRequest(ctx, "ping", "hello", 12) // calls ping("hello", 12)
conn.SendRequest(ctx, "ping")              // calls ping()
conn.SendRequest(ctx, "ping", nil)         // calls ping(nil)

Additional Notes

Reviewer checklist

• PR addresses a single concern.
• PR title and description are properly filled.
• Changes will be merged in main.
• Changes are covered by tests.
• Logging is meaningful in case of troubleshooting.

[dido18]

I like it 🦸