WIP: Ported Library Scavenger Hunt #346

Draft
LukeZurg22 wants to merge 9 commits from scavenger-hunt into master
Owner

PR Changes

Ported Changes from Wizden #43353 with some minor adjustments, such as the exclusion of Bael and transfer of the Legend of Nar'sie book to book contraband.

Reasons Behind Changes

I feel as if the bael familiar was rushed and not necessary. Further improving existing "contraband" books felt highest in my priorities atm.

Code & Technical details

The ported system & component (as of 20260603) is not going to compile, as the entity effect changes from Wizden are not ported here, yet. Note: This description will be changed later, however as of current this is barred from merging until the compiler errors can be fixed, which require the merging of Wizden's EntityEffect changes!

Images; Sprites; Media

Quite a lot, really. Mainly a new set of book sprites. (See changes)

Checklist

Breaking changes

Ingame Changelog

🆑

  • add: Ported Librarian Gameplay / Book Hunt Changes
## PR Changes Ported Changes from Wizden [#43353]( https://github.com/space-wizards/space-station-14/pull/43353) with some minor adjustments, such as the exclusion of Bael and transfer of the Legend of Nar'sie book to book contraband. ## Reasons Behind Changes I feel as if the bael familiar was rushed and not necessary. Further improving existing "contraband" books felt highest in my priorities atm. ## Code & Technical details The ported system & component (as of 20260603) is not going to compile, as the entity effect changes from Wizden are not ported here, yet. **Note: This description will be changed later, however as of current this is barred from merging until the compiler errors can be fixed, which require the merging of Wizden's EntityEffect changes!** ## Images; Sprites; Media Quite a lot, really. Mainly a new set of book sprites. (See changes) ## Checklist <!--Understand that not following all Contributor rule standards may get your PR closed at a maintainer’s discretion. Failure to follow the Contributor's ruleset may lead to your removal as a contributor.--> <!-- Confirm by placing an X in the brackets [X]: --> - [X] I read the [Pull Request and Changelog Guidelines](https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html) and [Byrd Station's Contributor Ruleset](https://docs.google.com/document/d/1oxCSRBc_2IlrcykljB5SPt783uJX7rMHTT7cImxlKFM/edit?tab=t.0#heading=h.vigrxrnffwji) am following them. - [ ] I have added media to this PR, or it does not require an ingame showcase. ## Breaking changes <!-- List any breaking changes, including namespaces, public class/method/field changes, prototype renames; and provide instructions for fixing them. This will be posted in #codebase-changes. --> **Ingame Changelog** <!-- Add a Changelog entry to make players ingame aware of new changes that could affect gameplay.--> :cl: - add: Ported Librarian Gameplay / Book Hunt Changes
Pushed Unfinished / Broken Classes.
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
Check Merge Conflicts / label (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 2s
Labels: Size / size-label (pull_request_target) Successful in 2s
Labels: Untriaged / add_label (pull_request_target) Failing after 2s
Build & Test Map Renderer / build (ubuntu-latest) (pull_request) Has been skipped
Build & Test Map Renderer / Build & Test Debug (pull_request) Has been skipped
soft-repoban check / check-author (pull_request) Successful in 2s
CRLF Check / CRLF Check (pull_request) Has been skipped
Test Packaging / Test Packaging (pull_request) Has been skipped
RGA schema validator / YAML RGA schema validator (pull_request) Has been skipped
RSI Validator / Validate RSIs (pull_request) Failing after 22s
Map file schema validator / YAML map schema validator (pull_request) Has been skipped
YAML Linter / YAML Linter (pull_request) Has been skipped
dabfc80165
Note: SpawnEntityListEntityEffect.cs and SpawnEntityListEntityEffectSystem.cs
Corrected Typo in NPCCombatSystem.Ranged.cs
Some checks failed
Check Merge Conflicts / label (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 3s
Labels: Size / size-label (pull_request_target) Successful in 2s
Build & Test Map Renderer / build (ubuntu-latest) (pull_request) Has been skipped
Build & Test Map Renderer / Build & Test Debug (pull_request) Has been skipped
CRLF Check / CRLF Check (pull_request) Has been skipped
Test Packaging / Test Packaging (pull_request) Has been skipped
RGA schema validator / YAML RGA schema validator (pull_request) Has been skipped
RSI Validator / Validate RSIs (pull_request) Failing after 22s
Map file schema validator / YAML map schema validator (pull_request) Has been skipped
YAML Linter / YAML Linter (pull_request) Has been skipped
b032c266c6
Author
Owner

@LilithArson What would you like me to do when it comes to the EntityEffectBase and so forth, the ones that we don't have from Wizden?

It's possible to retrofit this into what we have now and I did a little bit of that, but just making sure.

@LilithArson What would you like me to do when it comes to the EntityEffectBase and so forth, the ones that we don't have from Wizden? It's possible to retrofit this into what we have now and I did a little bit of that, but just making sure.
SpawnEntityListEntityEffectSystem.cs Adjustment
Some checks failed
Check Merge Conflicts / label (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 2s
Labels: Size / size-label (pull_request_target) Successful in 2s
Build & Test Map Renderer / build (ubuntu-latest) (pull_request) Has been skipped
Build & Test Map Renderer / Build & Test Debug (pull_request) Has been skipped
CRLF Check / CRLF Check (pull_request) Has been skipped
Test Packaging / Test Packaging (pull_request) Has been skipped
RGA schema validator / YAML RGA schema validator (pull_request) Has been skipped
RSI Validator / Validate RSIs (pull_request) Failing after 22s
Map file schema validator / YAML map schema validator (pull_request) Has been skipped
YAML Linter / YAML Linter (pull_request) Has been skipped
4b87288e33
LukeZurg22 changed title from WIP: Ported Library Scavenger Hunt to WIP: Ported Library Scavenger Hunt (Awaiting Ent.Eff. from Wizden) 2026-06-04 01:21:46 +00:00
LukeZurg22 changed title from WIP: Ported Library Scavenger Hunt (Awaiting Ent.Eff. from Wizden) to WIP: Ported Library Scavenger Hunt 2026-06-04 05:27:57 +00:00
Some checks failed
Check Merge Conflicts / label (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 2s
Labels: Size / size-label (pull_request_target) Successful in 2s
Build & Test Map Renderer / build (ubuntu-latest) (pull_request) Has been skipped
Build & Test Map Renderer / Build & Test Debug (pull_request) Has been skipped
CRLF Check / CRLF Check (pull_request) Has been skipped
Test Packaging / Test Packaging (pull_request) Has been skipped
RGA schema validator / YAML RGA schema validator (pull_request) Has been skipped
RSI Validator / Validate RSIs (pull_request) Failing after 22s
Map file schema validator / YAML map schema validator (pull_request) Has been skipped
YAML Linter / YAML Linter (pull_request) Has been skipped
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin scavenger-hunt:scavenger-hunt
git switch scavenger-hunt
Sign in to join this conversation.
No description provided.