Skip to content

(Sepultum) Radio Station - #5560

Merged
Rinary1 merged 2 commits into
ss14Starlight:starlight-devfrom
Orhu:orhu-SepultumRadio
Aug 10, 2026
Merged

(Sepultum) Radio Station#5560
Rinary1 merged 2 commits into
ss14Starlight:starlight-devfrom
Orhu:orhu-SepultumRadio

Conversation

@Orhu

@Orhu Orhu commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Short description

Adds the Radio Station to Sepultum and takes care of some other chores.

Why we need to add this

Video did not, in fact, kill the radio star.

Media (Video/Screenshots)

Radio Station

image

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

🆑 Orhu

  • add: (Sepultum) Added Radio Station.
  • add: (Sepultum) Added Station Radios to various spots around the station.
  • tweak: (Sepultum) Reporter doors now require Journalism access to open.

- Added a radio station
- Replaced Reporter doors with journalism doors
- Added Radio Receivers around the station
@Orhu
Orhu requested a review from a team August 7, 2026 23:28
@github-actions github-actions Bot added size/XL Changes: Map Changes: No C# S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. and removed size/XL labels Aug 7, 2026
@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4dc8008f-1d37-4fb5-81a8-b7aac73d1654

📥 Commits

Reviewing files that changed from the base of the PR and between e22931e and 248383c.

📒 Files selected for processing (2)
  • Resources/Maps/_Starlight/Stations/Sepultum.yml
  • Resources/Prototypes/_Starlight/Maps/sepultum.yml

📝 Walkthrough

Summary

Adds the Sepultum radio station and registers the RadioHost job for 2 players.

The PR also:

  • Places radio receivers around the station.
  • Adds a second host spawn point.
  • Changes Reporter doors to require Journalism access.
  • Replaces an advertisement crate with a CD rack.
  • Corrects the on-air sign orientation.
  • Fixes the map prototype.

Gameplay impact

Players can operate a radio station on Sepultum. Station radios support access to the new broadcast area. Reporter door access now depends on Journalism credentials.

Risk areas

  • Reporter access may break if existing Reporter roles do not provide Journalism access.
  • The fixed RadioHost availability range may limit station operation to exactly two players.
  • Map prototype or spawn changes may affect station placement and round-start spawning.
  • Radio receiver placement and sign orientation may require in-game validation.

Guideline gaps

  • The PR context does not state why the RadioHost job must be limited to exactly 2 players.
  • The PR context does not identify whether the map changes belong in Shared or Starlight content.
  • The PR context does not mention Starlight ownership comments.
  • The PR context does not identify localization changes for station signage or job text.
  • The PR context does not document the intended breaking impact of changing Reporter door access.
  • The PR context does not explain the design rationale for the second spawn point, CD rack, or station layout changes.

Walkthrough

Sepultum Station now lists RadioHost as an available station job with exactly two positions.

Changes

Sepultum RadioHost roster

Layer / File(s) Summary
Add RadioHost positions
Resources/Prototypes/_Starlight/Maps/sepultum.yml
The Sepultum Station job roster adds two RadioHost positions.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested labels: Chore

Suggested reviewers: redmushie, rinary1

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: adding a radio station to Sepultum.
Description check ✅ Passed The description directly covers the Sepultum radio station, related map changes, and access updates.
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.

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.

@CawsForConcern CawsForConcern left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

review

missing job prototype

  • you need to add RadioHost: [ 2, 2 ] to Resources\Prototypes\_Starlight\Maps\sepultum.yml

spawnpoint

  • have at least one spawnpoint per Radio Host job slot. maps should generally aim for two Radio Host job slots.

on air sign upside down

  • flip this around to face south
Image

replace CrateRadioHostADs with CdRackFilled

  • i updated the recommendations for Radio Host, I missed there was a nice-looking CD rack for their room.

@CawsForConcern

Copy link
Copy Markdown
Contributor

Putting on DONOTMERGE until Journalism access gets fixed for Reporter - Reporter currently lacks the access, waiting on it to get fixed.

@Orhu

Orhu commented Aug 9, 2026

Copy link
Copy Markdown
Contributor Author

Will fix tmrw

- Swapped ad crate for cd rack
- added a second host spawn point
- fixed orientation of on-air sign
- fixed map prototype
@CawsForConcern

Copy link
Copy Markdown
Contributor

Dropping DONOTMERGE, this PR requires #5587 to be merged first.

@CawsForConcern CawsForConcern added T: New Content and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Aug 9, 2026

@CawsForConcern CawsForConcern left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved! good to merge once #5587 is merged.

@CawsForConcern CawsForConcern added S: Approved Status: Reviewed and approved by at least one maintainer or dev; a PR may require another approval. and removed S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. labels Aug 9, 2026
@Rinary1
Rinary1 added this pull request to the merge queue Aug 10, 2026
@coderabbitai coderabbitai Bot mentioned this pull request Aug 10, 2026
4 tasks
Merged via the queue into ss14Starlight:starlight-dev with commit 17d41c3 Aug 10, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: Map Changes: No C# Changes: Prototypes S: Approved Status: Reviewed and approved by at least one maintainer or dev; a PR may require another approval. size/XL T: New Content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants