Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TerraGov Inspector #2791

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Kr8art
Copy link
Contributor

@Kr8art Kr8art commented Jan 24, 2025

About the PR

Adds the TerraGov Inspector as a ghost role antagonist.

Why / Balance

This is meant to be a fun RP heavy antagonist that's different from existing antagonists with no mechanical depth. You tie up station's attention and resources with your presence, inspect departments and interview the criminally underpaid NanoTrasen employees about their working conditions.

The inspector's outfit/id is also chameleon whitelisted to enable syndicate impostors that are less valid than the usual CentCom impostor.

The inspector currently uses the same shuttle as the syndicate recruiter and spawns with:

  • TerraGov Jumpsuit
  • TerraGov Headset with common and command channels
  • Two different TerraGov hats
  • Administration Glasses
  • TerraGov PDA with TerraGov pen and TerraGov ID (Basic + Command access)
  • TerraGov stamp
  • TerraGov clipboard
  • White gloves and leather shoes

Technical details

  • Adds a new Ghostrole that works pretty much identical to the Syndicate recruiter on a technical level
  • Adds a whole new set of TerraGov themed sprites

To Do:

There's still a few kinks I need to iron out:

  • Define playtime requirements/whitelist (IMPORTANT)
  • Fix profession in character window.
  • Make stamp objective greentextable.
  • ID card doesn't have the inspector's name.
  • Extend random appearance to all morphotypes, instead of just humans.

Media

Sprite showcase

image

Ghostrole/Shuttle

image

Objectives

image

Stamp

image

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an ingame showcase.

Breaking changes

My sanity.

Changelog
🆑

  • add: The TerraGov Inspector is now available as a new ghost role! Strike fear into the heart of department heads as you look for healthcode violations and enforce workplace safety protocols. Credits to JustAnOrange for doing most of the shiny new sprites!

@Kr8art Kr8art requested review from a team as code owners January 24, 2025 11:44
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added Changes: YML Changes any yml files Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi labels Jan 24, 2025
Copy link
Contributor

RSI Diff Bot; head commit 89ca87c merging into 069dc1e
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/DeltaV/Clothing/Ears/Headsets/terragov.rsi

State Old New Status
alt-equipped-EARS Added
equipped-EARS Added
icon Added
icon_alt Added

Resources/Textures/DeltaV/Clothing/Head/Hats/beret_terragov.rsi

State Old New Status
equipped-HELMET-hamster Added
equipped-HELMET Added
icon Added

Resources/Textures/DeltaV/Clothing/Head/Hats/terracap.rsi

State Old New Status
equipped-HELMET-hamster Added
equipped-HELMET Added
icon Added

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpsuit/terragov.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Interface/Misc/job_icons.rsi

State Old New Status
TerraGov Added

Resources/Textures/DeltaV/Markers/jobs.rsi

State Old New Status
inspector Added

Resources/Textures/DeltaV/Objects/Devices/pda.rsi

State Old New Status
pda-terragov Added

Resources/Textures/DeltaV/Objects/Misc/stamps.rsi

State Old New Status
stamp-terragov Added

Resources/Textures/DeltaV/Objects/Misc/terragov_clipboard.rsi

State Old New Status
clipboard Added
clipboard_over Added
clipboard_paper Added
clipboard_pen Added
equipped-BELT Added
inhand-left Added
inhand-right Added

Resources/Textures/Objects/Misc/bureaucracy.rsi

State Old New Status
paper_stamp-terragov Added

Resources/Textures/Objects/Misc/id_cards.rsi

State Old New Status
idterragov Added

Resources/Textures/Objects/Misc/pens.rsi

State Old New Status
terragov_pen Added

@Bonktrauma
Copy link
Contributor

Honestly they should be focusing on making sure the station is upholding the law

@Avalon-Proto
Copy link
Contributor

Honestly they should be focusing on making sure the station is upholding the law

Are laws concerning worker safety and workplace sanitation non existent? :blunt:

@Smugman
Copy link
Contributor

Smugman commented Jan 24, 2025

I really like this concept!
An Idea, they should perhaps spawn with a "Inspection Warrant" or something like that so the Station is unable to refuse them entry, or maybe just an Announcement saying that "An Inspector from TerraGov bearing an Official Comission will be arriving soon."

Also, TerraGov handyflag, one day...

Are laws concerning worker safety and workplace sanitation non existent? :blunt:
Yes :Godo:

@Kr8art
Copy link
Contributor Author

Kr8art commented Jan 24, 2025

I really like this concept! An Idea, they should perhaps spawn with a "Inspection Warrant" or something like that so the Station is unable to refuse them entry, or maybe just an Announcement saying that "An Inspector from TerraGov bearing an Official Comission will be arriving soon."

Personally, I'd prefer to avoid adding these things to make it easier for syndicate agents to impersonate an inspector.

A legitimate inspector can still write their own inspection warrant on the shuttle before they arrive to the station and stamp it with their stamp. Since they have command access in their ID, they can also walk into the bridge and use the announcement console to make their presence known.

@Smugman
Copy link
Contributor

Smugman commented Jan 24, 2025

Personally, I'd prefer to avoid adding these things to make it easier for syndicate agents to impersonate an inspector.

A legitimate inspector can still write their own inspection warrant on the shuttle before they arrive to the station and stamp it with their stamp. Since they have command access in their ID, they can also walk into the bridge and use the announcement console to make their presence known.

Fair point. I dont think even the most ornery captain would think "Yes, I can tell the man who centcom has given command access to leave."

CentCom ops get insta-"valided", so this would be a nice middle ground.

@deltanedas
Copy link
Member

there are no space workspace safety laws so this should be a super lawyer making sure all space law violations are correctly punished imo

@ThataKat
Copy link
Contributor

Direction is looking it over. This one has big implications so it'll probably be several days, but we'll pop back in whenever we have relevant feedback.

@Smugman
Copy link
Contributor

Smugman commented Jan 27, 2025

Also, TerraGov handyflag, one day...

TerraGov handyflag, NOW!
image
image
Spirtes very much not final, i cant get the icon quiiite right...

Copy link
Contributor

@Lyndomen Lyndomen left a comment

Choose a reason for hiding this comment

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

Hi! I personally really love this concept, and think the contents bring a lot of options to our Curator team with TerraGov flavored events.

I commented a few issues, the common theme is effectively that we're not sure how this fits as an antagonist, and we're also worried it becomes routine and expected- similar to how the traveling chef is ignored in 90% of cases. We're curious if you have any ideas in how they might be driven to interact further with the station?

Some ideas we've had is a loose re framing of this to Internal Affairs, with objectives to secretly kidnap and interview crewmembers, or acting more as a vigilante to hunt down a specific Fugitive.

I'd be curious to hear your thoughts, and look forward to seeing this evolve!

Copy link
Contributor

@Radezolid Radezolid left a comment

Choose a reason for hiding this comment

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

Ensure four space indentation on your JSON files, leave the upstream ones with their current indentation.

Comment on lines +222 to +226
- type: CartridgeLoader
preinstalled:
- CrewManifestCartridge
- NotekeeperCartridge
- NewsReaderCartridge
Copy link
Contributor

Choose a reason for hiding this comment

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

This is redundant, you inherit these from the BasePDA.

Comment on lines +64 to +65
- type: Item
heldPrefix: default
Copy link
Contributor

Choose a reason for hiding this comment

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

This is redundant due to the IDCardStandard parenting.

- type: CommandStaff
- !type:AddComponentSpecial
components:
- type: PsionicBonusChance #Nyano - Summary: makes it more likely to become psionic.
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove the Nyano comment.

parent: JobIcon
id: JobIconTerraGov
icon:
sprite: /Textures/DeltaV/Interface/Misc/job_icons.rsi
Copy link
Contributor

Choose a reason for hiding this comment

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

The /Textures/ is extra.

Comment on lines +539 to +561
## DeltaV - TerraGov Clipboard
- type: entity
id: BoxFolderTerraGovClipboard
parent: BoxFolderClipboard
name: TerraGov clipboard
description: A luxurious clipboard upholstered with blue velvet. Often seen carried by TerraGov officials, seldom seen actually used.
components:
- type: Sprite
sprite: DeltaV/Objects/Misc/terragov_clipboard.rsi
layers:
- state: clipboard
- state: clipboard_paper
map: ["clipboard_paper"]
visible: false
- state: clipboard_pen
map: ["clipboard_pen"]
visible: false
- state: clipboard_over
- type: Item
sprite: DeltaV/Objects/Misc/terragov_clipboard.rsi
- type: Clothing
sprite: DeltaV/Objects/Misc/terragov_clipboard.rsi

Copy link
Contributor

Choose a reason for hiding this comment

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

Move this to the _DV namespace.

Comment on lines +59 to +69
- type: entity
name: terragov pen
parent: Pen
id: TerraGovPen
description: A fancy and bright blue pen that marks you as official TerraGov paper-pusher.
components:
- type: Sprite
state: terragov_pen
- type: Item
heldPrefix: terragov_pen

Copy link
Contributor

Choose a reason for hiding this comment

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

Move this to the _DV namespace.

@Radezolid
Copy link
Contributor

Just in case, here is an example of a correct JSON indentation:

{
"version": 1,
"license": "CC0-1.0",
"copyright": "Original work by TJohnson.",
"size": {
"x": 32,
"y": 32
},
"states": [
{
"name": "icon"
},
{
"name": "equipped-INNERCLOTHING",
"directions": 4
},
{
"name": "inhand-left",
"directions": 4
},
{
"name": "inhand-right",
"directions": 4
},
{
"name": "rolled-equipped-INNERCLOTHING",
"directions": 4
}
]
}

@ThataKat
Copy link
Contributor

ThataKat commented Feb 1, 2025

Direction update, we're still workshopping this with the author. Discussion is in a thread in the contrib channel on Discord

@Kr8art
Copy link
Contributor Author

Kr8art commented Feb 2, 2025

Since a direction doesn't want a TerraGov themed inspector any longer, I'll be removing all role related changes and make this PR purely for the sprites so they can be used for events (and maybe available in an emag'd uniform printer).

I am dismissing all the feedback regarding the files that touch upon the role, since it is not relevant any longer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files S: Merge Conflict Fix your PR! S: Needs Review size/L 256-1023 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants