Magazines and Newspapers - #5538
Conversation
📝 WalkthroughSummaryAdds 11 newspapers, 21 magazines, and 14 revolutionary or conspiracy “Zines” ported from Ephemeral Space and adapted to Starlight lore. Gameplay and user impact
Notable implementation changes
Risk areas
Guideline gaps
WalkthroughAdded newspaper and magazine prototypes, texture metadata, categorized random spawners, and uncommon maintenance-fluff loot entries. ChangesPeriodical content and spawning
Estimated code review effort: 3 (Moderate) | ~25 minutes Suggested labels: Suggested reviewers: Sequence Diagram(s)sequenceDiagram
participant RandomSpawner
participant PeriodicalPool
participant PeriodicalEntity
RandomSpawner->>PeriodicalPool: Select configured prototype
PeriodicalPool->>PeriodicalEntity: Spawn newspaper or magazine
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 5
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@Resources/Prototypes/_ES/Entities/Markers/Spawners/Random/periodicals.yml`:
- Around line 26-59: Add MagazineFirearms to each regular periodical selector:
ESSpawnerRandomPeriodicals in
Resources/Prototypes/_ES/Entities/Markers/Spawners/Random/periodicals.yml lines
26-59, ESSpawnerRandomPeriodicalsLegit in the same file lines 85-105, and the
maintenance periodical selector in
Resources/Prototypes/Entities/Markers/Spawners/Random/maintenance.yml lines
175-223.
In `@Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml`:
- Around line 5-6: Localize every player-facing publication name, description,
issue name, and headline description using the repository’s Fluent localization
pattern. Update Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml at
lines 5-6 and 22-23, and
Resources/Prototypes/_ES/Entities/Objects/Misc/newspapers.yml at lines 5-6 and
20-21; add the corresponding Fluent entries and replace each raw value with its
localization key.
- Line 23: Correct the player-visible spelling errors in the descriptions near
the shown entries: change “Perservence” to “Perseverance,” “Vunerabilities” to
“Vulnerabilities,” “Vigiliant” to “Vigilant,” “orchastrated” to “orchestrated,”
and “competitior” to “competitor.”
In `@Resources/Prototypes/_ES/Entities/Objects/Misc/newspapers.yml`:
- Around line 2-6: Mark the BaseNewspaper prototype as abstract while preserving
its existing parent, name, and description so it remains available for
inheritance but cannot be spawned as a standalone entity.
In `@Resources/Prototypes/Entities/Markers/Spawners/Random/maintenance.yml`:
- Line 223: Remove the trailing whitespace from the comment line containing
“Starlight End” in the maintenance prototype, leaving the comment text
unchanged.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 9a9c9e06-0997-4f79-b539-d1b911b33f0a
⛔ Files ignored due to path filters (16)
Resources/Textures/_ES/Markers/spawners.rsi/legit.pngis excluded by!**/*.pngResources/Textures/_ES/Markers/spawners.rsi/magazine.pngis excluded by!**/*.pngResources/Textures/_ES/Markers/spawners.rsi/newspaper.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/cognitowood.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/cooking.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/crossword.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/earth_news.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/fashion.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/guns.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/medical.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/newspaper.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/popular_spacepeople.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/space.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/wordsearch.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/zine_conspiracy.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/zine_revolutionary.pngis excluded by!**/*.png
📒 Files selected for processing (7)
Resources/Prototypes/Entities/Markers/Spawners/Random/maintenance.ymlResources/Prototypes/_ES/Entities/Markers/Spawners/Random/periodicals.ymlResources/Prototypes/_ES/Entities/Markers/Spawners/Random/periodicals_specific.ymlResources/Prototypes/_ES/Entities/Objects/Misc/magazines.ymlResources/Prototypes/_ES/Entities/Objects/Misc/newspapers.ymlResources/Textures/_ES/Markers/spawners.rsi/meta.jsonResources/Textures/_ES/Objects/Misc/magazines.rsi/meta.json
|
github services are down, rerun tests later |
|
Adding in-hands since the tests are stalling due to github issues anyway and I have time to kill. |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml (2)
94-101: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winRemove repeated inherited
sizevalues.Each prototype inherits
size: SmallfromBaseMagazine. Keep theItemcomponent forinhandVisuals, but remove its repeatedsize: Smallfield.As per path instructions, flag redundant state updates.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml` around lines 94 - 101, Remove the redundant inherited size: Small field while preserving each prototype’s Item component and inhandVisuals entries. Apply this in Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml at lines 94-101, 114-121, 134-141, 154-161, 174-181, 194-201, 214-221, 259-266, 321-328, 366-373, and 428-435; each site requires the same removal.Source: Path instructions
58-60: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winUse explicit units for edible delays.
Change both
delay: 7andforceFeedDelay: 7to7s. This makes the duration unit explicit.
Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml#L58-L60: use7sfor both delay fields.Resources/Prototypes/_ES/Entities/Objects/Misc/newspapers.yml#L58-L60: use7sfor both delay fields.As per path instructions, use explicit units such as
5swhere the field supports them.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml` around lines 58 - 60, Use explicit seconds units for both edible delay fields: change delay and forceFeedDelay to 7s in Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml lines 58-60 and Resources/Prototypes/_ES/Entities/Objects/Misc/newspapers.yml lines 58-60.Source: Path instructions
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml`:
- Around line 13-78: Move the Starlight-specific in-hand visuals and related
prototype changes from
Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml:13-78 and
Resources/Prototypes/_ES/Entities/Objects/Misc/newspapers.yml:15-78 into
matching _Starlight prototypes, updating all references so the _ES files retain
only upstream content. Move the Starlight-owned magazine states from
Resources/Textures/_ES/Objects/Misc/magazines.rsi/meta.json:11-30 into a
dedicated _Starlight RSI and update prototype references; retain them in the
shared RSI only with a concrete technical justification.
---
Nitpick comments:
In `@Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml`:
- Around line 94-101: Remove the redundant inherited size: Small field while
preserving each prototype’s Item component and inhandVisuals entries. Apply this
in Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml at lines 94-101,
114-121, 134-141, 154-161, 174-181, 194-201, 214-221, 259-266, 321-328, 366-373,
and 428-435; each site requires the same removal.
- Around line 58-60: Use explicit seconds units for both edible delay fields:
change delay and forceFeedDelay to 7s in
Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.yml lines 58-60 and
Resources/Prototypes/_ES/Entities/Objects/Misc/newspapers.yml lines 58-60.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro Plus
Run ID: bcade0fd-9993-4f64-83c3-b93b805a2398
⛔ Files ignored due to path filters (4)
Resources/Textures/_ES/Objects/Misc/magazines.rsi/magazine-inhand-left.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/magazine-inhand-right.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/newspaper-inhand-left.pngis excluded by!**/*.pngResources/Textures/_ES/Objects/Misc/magazines.rsi/newspaper-inhand-right.pngis excluded by!**/*.png
📒 Files selected for processing (3)
Resources/Prototypes/_ES/Entities/Objects/Misc/magazines.ymlResources/Prototypes/_ES/Entities/Objects/Misc/newspapers.ymlResources/Textures/_ES/Objects/Misc/magazines.rsi/meta.json
8b1d60a
|
glad to see ephemeral space's parasites integrated into the lore. can't wait until the full gamemode is ported as well! love seeing cool things get ported :] |
I'd love to at some point yea, porting roles from ES is always a bit scary looking though considering they redid their entire role selection system. One day though. |
Short description
Ports Magazines and Newspapers from Ephemeral Space, with a number of edits to names and descriptions to work within Starlight's lore.
There are a total of 11 Newspapers, 21 Magazines, and 14 'Zines'. Newspapers and Magazines are more 'legit', while 'Zines' are conspiracy theories and revolutionary content.
Why we need to add this
A new avenue to introduce lore and worldbuilding to players mid-round. They are very immersive and fun decorations as well, for waiting rooms in Medical etc.
The Newspapers specifically can be periodically cycled out with new dates and news if we feel like it, or we can just say there's a bunch of old papers laying around.
Media (Video/Screenshots)
Checks
Changelog
🆑 Conflee, Flareguy