Use RpcRequest in PubSubSubscriptionPlan

[lorisleiva]

This PR changes the configs of the executeRpcPubSubSubscriptionPlan function such that it accepts a "Subscribe Request" as a RpcRequest object instead of a subscribeMethodName and subscribeParams duo.

This mean we can now use the RpcRequestTransformer provided in a more idiomatic way.

Note: A changeset for this PR is included in #3407.

[changeset-bot]

⚠️ No Changeset found

Latest commit: a363260

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[lorisleiva]

• Replace subscriptionConfigurationHash with RpcRequest in RpcSubscriptionsPlan #3406
• Add changeset for using RpcRequest and RpcResponse in RPC Subscriptions #3407
• Use RpcRequest in PlanExecutor configs #3405
• Use RpcResponseTransformer in PubSubSubscriptionPlan #3404
• Use RpcRequest in PubSubSubscriptionPlan #3403 👈
• Use RpcRequest in createRpcMessage helper #3394
• Move RpcRequest and RpcResponse types to rpc-spec-types #3393
• Remove unused RpcSubscriptionsRequest type #3391
• Temporarily disable Node 23 CI tests #3395
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lorisleiva and the rest of your teammates on Graphite

[steveluscher]

[lorisleiva]

Merge activity

• Oct 23, 4:43 AM EDT: A user started a stack merge that includes this pull request via Graphite.
• Oct 23, 4:53 AM EDT: Graphite rebased this pull request as part of a merge.
• Oct 23, 4:54 AM EDT: A user merged this pull request with Graphite.

[github-actions]

Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up.