Skip to content
This repository was archived by the owner on Jun 28, 2026. It is now read-only.

Anchor API rework#128

Open
metalgearsloth wants to merge 5 commits into
Space-Wizards-Federation:masterfrom
metalgearsloth:2026-06-21-rainbow6-anchor
Open

Anchor API rework#128
metalgearsloth wants to merge 5 commits into
Space-Wizards-Federation:masterfrom
metalgearsloth:2026-06-21-rainbow6-anchor

Conversation

@metalgearsloth

@metalgearsloth metalgearsloth commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Previously:

  • Anchored anchored on the spot (or at least attempted to) but was not ECS
  • AnchorEntity would always run anchoring even if the entity was already anchored.

Now:

  • AnchorEntity has overloads to move the entity and then anchor it.
  • AnchorEntity does not double-anchor and cause hellbugs if it's the same tile.
  • Reanchor got purged because I'd rather just explicitly unanchor and re-anchor to simplify content even though we take a performance hit.
  • Tests for all of the above.

I wouild've obsoleted reanchor but the issue is we'd have to raise the anchor changes in lieu of it which would almost certainly need to double-handling on content. Most forks are likely only using the anchor state change events anyway.

Content PR is Space-Wizards-Federation/space-station-14#129

@metalgearsloth metalgearsloth marked this pull request as ready for review June 21, 2026 02:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant