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

Get rid of `cloakCost{,Moving}`

Open
#2,849 0 comments 0 reactions 0 assignees View on GitHub
game compat refactor
Dominant language
C++
Stars
679
Forks
290
Avg merge
3d 2h
Merged PRs (30d)
40

Description

It doesn't actually do anything (the engine does NOT use it for cloak costs), other than being re-exposed for reading back by Lua which is what customparams are for. Make PRs to games (incl. ancillaries like Circuit/Barbarian AI) first though since it should be trivial to do so.

Contributor guide

Open the contributing guide

Research direction

Start by searching the engine and game integrations for `cloakCost` and `cloakCostMoving`, including the named Circuit and Barbarian AI ancillaries. First update the affected games and ancillaries as requested, then remove the unused engine exposure and verify that Lua customparams still provide the intended readback.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
game-dev
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 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.