Skip to content

Clarify that high-level RPC methods must be defined on Node-based scripts#11892

Open
deathtaco1408 wants to merge 2 commits into
godotengine:masterfrom
deathtaco1408:deathtaco1408-RPC-Warning
Open

Clarify that high-level RPC methods must be defined on Node-based scripts#11892
deathtaco1408 wants to merge 2 commits into
godotengine:masterfrom
deathtaco1408:deathtaco1408-RPC-Warning

Conversation

@deathtaco1408

Copy link
Copy Markdown
Contributor

Summary

This PR clarifies that Godot's high-level RPC system is node-based and that methods marked with
@rpc should be defined on scripts attached to Node-derived classes.

Why

The current multiplayer documentation explains how RPCs are used, but it does not clearly state
that defining @rpc methods only on non-Node classes is not supported. This can lead to runtime
errors and confusion for users building helper abstractions around Resource or RefCounted
scripts.

This change adds a short clarification and example so developers understand where RPC methods should
live and how to structure helper code safely.

Related context

This addresses confusion described in issue #89981.

Scope

Documentation only. No engine behavior changes.

Clarify RPC method requirements and supported patterns in high-level multiplayer documentation.
Added a note about Godot's RPC system requirements for node-based scripts.
@deathtaco1408
deathtaco1408 requested review from a team as code owners March 31, 2026 05:03
@skyace65 skyace65 added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation topic:network labels Apr 19, 2026

@Zacky3181V Zacky3181V left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting changes to meet documentation writing guidelines

Comment thread tutorials/networking/high_level_multiplayer.rst
Comment thread tutorials/networking/high_level_multiplayer.rst
Comment thread tutorials/networking/high_level_multiplayer.rst

@skyace65 skyace65 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I need to come back and look at this more fully later, but for now this is why the build is failing.

Comment thread tutorials/networking/high_level_multiplayer.rst
@deathtaco1408

Copy link
Copy Markdown
Contributor Author

Aye aye

@Calinou

Calinou commented May 16, 2026

Copy link
Copy Markdown
Member

Needs a rebase to fix merge conflits. (Note that #11983 includes the first paragraph of this PR.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:manual Issues and PRs related to the Manual/Tutorials section of the documentation enhancement topic:network

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants