Skip to content

Gas tank tweaks & fixes - #5514

Closed
redmushie wants to merge 2 commits into
ss14Starlight:starlight-devfrom
redmushie:bugfix/tall-gas-tank
Closed

Gas tank tweaks & fixes#5514
redmushie wants to merge 2 commits into
ss14Starlight:starlight-devfrom
redmushie:bugfix/tall-gas-tank

Conversation

@redmushie

@redmushie redmushie commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Short description

  • Subfloor tank
    • Now retains gas when unanchored
    • Can no longer be picked up by hand
    • Can no longer be picked up by mechs
    • Has significant mass to deter dragging it around as canister alternative
  • Tall gas tank
    • Now retains gas when unanchored
    • Can no longer be picked up by mechs
    • Entire bottom half is now clickable instead of only a small area
    • Clickable area no longer moves when rotated
    • Has significant mass to deter dragging it around as canister alternative

Why we need to add this

  • Tall gas tank fixture fixes are bugfixes
  • The other stuff is to make them better match expected behavior (I got feedback on this) and to improve their QoL while retaining balance (allowing gas retention, but with weight increase and no pickup possible).

Media (Video/Screenshots)

Subfloor tank gas retention & mass

2026-08-04.18-47-28.mp4

Tall tank: Collision, Visual targeting

2026-08-04.17-58-05.mp4

Tall tank: Unanchoring, Drag weight

2026-08-04.18-23-52.mp4

Tall tank: No mech pickup

2026-08-04.18-26-03.mp4

Checks

  • I do not require assistance to complete the PR.
  • Before posting/requesting review of a PR, I have verified that the changes work.
  • I have added screenshots/videos of the changes, or this PR does not change in-game mechanics.
  • I affirm that my changes are licensed under the MIT License and grant permission for use in this repository under its conditions.

Changelog

🆑 redmushie

  • add: Subfloor- and tall gas tanks now retain gas when unanchored.
  • add: Subfloor- and tall gas tanks are now quite heavy to drag around.
  • tweak: Subfloor- and tall gas tanks can no longer be picked up by hand or by hydraulic clamp.
  • fix: Tall gas tanks no longer have their clickable section move when oriented differently.
  • fix: Tall gas tanks are now properly clickable on their lower half.

@github-actions github-actions Bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. Changes: No C# Changes: Prototypes S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. size/S labels Aug 4, 2026
@redmushie redmushie changed the title Tall gas tank fixes Gas tank tweaks & fixes Aug 4, 2026
@redmushie
redmushie marked this pull request as ready for review August 4, 2026 17:02
@redmushie
redmushie requested a review from a team August 4, 2026 17:02
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The change adds the NoMechGrab tag and applies it to subfloor and tall gas tanks. Both tank variants retain gas when unanchored and receive updated pickup, rendering, physics, and fixture settings.

Changes

Gas tank handling

Layer / File(s) Summary
NoMechGrab tag contract and grabber filtering
Content.Server/Mech/Equipment/Components/MechGrabberComponent.cs, Resources/Prototypes/_Starlight/tags.yml
The NoMechGrab tag is documented and excluded by the mech grabber blacklist.
Subfloor gas tank handling and physics
Resources/Prototypes/_Starlight/Entities/Structures/Piping/Atmospherics/pipes.yml
Subfloor gas tanks retain gas when unanchored, prevent direct pickup, use non-colliding physics, and receive the NoMechGrab tag.
Tall gas tank handling and fixtures
Resources/Prototypes/_Starlight/Entities/Structures/Piping/Atmospherics/pipes.yml
Tall gas tanks retain gas when unanchored, render above doors, use updated fixture density and placement, add a bottom interaction fixture, and receive the NoMechGrab tag.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers: rinary1

🚥 Pre-merge checks | ✅ 8
✅ Passed checks (8 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
No Magic Numbers ✅ Passed The only system addition is the string tag NoMechGrab; the numeric edits are confined to YAML prototypes, which this check excludes.
No Hardcoded Ecs Parameters ✅ Passed The PR adds no system logic or hardcoded behavior values. MechGrabberSystem reads component.Blacklist, while tank behavior is configured through YAML component fields and fixtures. Good data-driven...
Avoid Service Locator ✅ Passed The change adds only whitelist data and YAML tags/components; it adds no service resolution. Existing dependencies use the appropriate [Dependency] injection pattern.
Title check ✅ Passed The title clearly identifies the gas tank changes and fixes covered by the pull request.
Description check ✅ Passed The description clearly and accurately explains the gas tank behavior changes, fixes, and rationale.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@redmushie
redmushie marked this pull request as draft August 4, 2026 19:27
@Rinary1

Rinary1 commented Aug 10, 2026

Copy link
Copy Markdown
Member

closed as stale

@Rinary1 Rinary1 closed this Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: C# Changes: No C# Changes: Prototypes S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants