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

There's no `IsFeatureVisible(featureId)`

Open
#2,191 4 comments 0 reactions 0 assignees View on GitHub
area: Lua API enhancement good first issue
Dominant language
C++
Stars
679
Forks
290
Avg merge
3d 2h
Merged PRs (30d)
40

Description

A quick search through the codebase will reveal many places such functionality is defined, as part or helper function.

There's no easy way to check if a featureId is visible, except getting its position and checking visibility or checking if the id is included in `GetVisibleFeatures`.

See `GetFeatureDefId` (visibility checks) and `IsUnitVisible` as a reference.

Contributor guide

Open the contributing guide

Research direction

Start by searching the codebase for GetFeatureDefId, IsUnitVisible, and GetVisibleFeatures, comparing how each performs visibility checks. Add the missing IsFeatureVisible(featureId) entry point so callers can check visibility directly, then verify it covers the same visibility behavior as the referenced helpers.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.