beyond-all-reason / beyond-all-reason/RecoilEngine

Let synced luarules call `Script.LuaUI`

Open
#1,130 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

area: Lua API enhancement
Dominant language
C++
Stars
683
Forks
293
Avg merge
3d 2h
Merged PRs (30d)
40

Description

The pattern where you have to SendToUnsynced first is silly, let synced luarules do that directly. Just have to make sure that return values are discarded.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Search for the implementation and call sites of Script.LuaUI, synced luarules, and SendToUnsynced, then trace how return values are handled. Confirm the intended direct call path and verify that any returned values are discarded without breaking existing synced-to-unsynced behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, lua
Domain
game-dev
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.