Skip to content

Surgery Sound Update: Different tools, Different sounds - #13083

Open
Puckaboo2 wants to merge 26 commits into
cmss13-devs:masterfrom
Puckaboo2:surgery_sound_upd8_2
Open

Surgery Sound Update: Different tools, Different sounds#13083
Puckaboo2 wants to merge 26 commits into
cmss13-devs:masterfrom
Puckaboo2:surgery_sound_upd8_2

Conversation

@Puckaboo2

@Puckaboo2 Puckaboo2 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

About the pull request

This PR adds different sounds for different surgical tools depending on the step. Note: Not all tools have different sounds. Just some.

Maints, pls tell me in the Discord if something isn't in the changelog. I'll fix it immediately--no need to draft the PR.

Explain why it's good for the game

Did you hear a surgical drill while you were using a fire axe? No, you didn't. Not this update.

Testing Photographs and Procedure

Screenshots & Videos

new incision sounds
https://github.com/user-attachments/assets/b5b2a77c-afba-42cc-8fe4-c7f5bd54d523

New sounds for opening and repairing bones
https://github.com/user-attachments/assets/22c0ff66-9c25-44db-9638-7f984aee4be2

New sound for the Fix-O-Vein
https://github.com/user-attachments/assets/6aa494e3-64be-40a7-b2a2-b80a25b6757d

New sounds for applying skin grafts
https://github.com/user-attachments/assets/d51c53b3-1fca-419b-b27f-c0c723987fa9

New sounds for cauterizing wounds
https://github.com/user-attachments/assets/bfe5d7a8-e170-4f11-9d7b-2d347c6e43da

Changelog

🆑 Puckaboo2
code: Adds some defines for groups of surgery tools used for steps.
fix: Most tools making inappropriate sounds during surgery steps, like a knife using saw drilling noises when cutting bone, should not happen anymore.
fix: Tools that utilize toolhit sounds during failed steps have been restored.
fix: Using the IMS no longer tells you it made a bloody incision.
soundadd: The IMS, laser scalpel, bone gel, and ointment have their own sets of preop and success sounds. Some are splices of old sound files, while some are brand new.
soundadd: The Fix-O-Vein has a unique sound while using it to initiate flesh-and-vein-mending-type surgeries, while all weapons that chop instead of saw receive the same sound set of repetitive chopping when initiating a surgery step that cuts through bone.
soundadd: Cigarettes/lighters and welding tools have sounds for initiating cauterizations.
soundadd: The screwdriver uses its sound sets properly when initiating and successfully drilling rods through or applying plates on bones.
soundadd: Synthgrafts and ointment have sounds for initiating and successfully applying grafts during eschar surgeries.
soundadd: Surgery Line, cloth, and cable coils, whenever used as surgical tools, always use sewing noises to mend flesh and blood vessels.
soundtweak: Increased the volume of ointment spreading on skin a little bit so it is clearly audible.
/:cl:

@github-project-automation github-project-automation Bot moved this to Awaiting Review in Review Backlog Sep 4, 2026
@cmss13-ci cmss13-ci Bot added Sound Blast 5 minutes of bass boosted music to our players Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Code Improvement Make the code longer Fix Fix one bug, make ten more labels Sep 4, 2026
@cmss13-ci cmss13-ci Bot added the size/L Denotes a PR that changes 200-799 lines, ignoring generated files. label Sep 4, 2026
Comment thread code/__DEFINES/surgery.dm
#define SURGERY_TOOLS_PRY list(\
/obj/item/tool/surgery/retractor = SURGERY_TOOL_MULT_IDEAL,\
/obj/item/tool/surgery/hemostat = SURGERY_TOOL_MULT_SUBOPTIMAL,\
/obj/item/tool/crowbar = SURGERY_TOOL_MULT_SUBSTITUTE,\

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.

Retracting skin requires fine manipulation and a small tool head. That is to say, I don't think a crowbar would be anything more than SURGERY_TOOL_MULT_AWFUL at that task.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This was literally copied from the list of tools that the retract skin step had. That would be a balance change that would need to be approved, but I get what you are saying.

Comment thread code/__DEFINES/surgery.dm
)

///Tools used to apply a skin graft after Eschar removal surgery.
#define SURGERY_TOOLS_APPLY_GRAFT list(\

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.

Might I also suggest adding bandages to this list as an SURGERY_TOOL_MULT_BAD_SUBSTITUTE substitute? Typically burns are dressed with gauze when skin grafts aren't available/possible.

@Puckaboo2 Puckaboo2 Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'd argue gauze should be considered 'awful' because this is the 2100s and technology has advanced much further than gauze. Gauze should be a last-ditch option.

Honestly, though, that's out of scope for this PR. The only reason I'm making new datums to lists is to make it easier to group tools for sound applications. As I copy-pasted them, they remained unchanged from the original list the surgical steps had.

Comment thread code/modules/surgery/generic.dm Outdated
if(!surface_modifier)
surface_modifier = SURGERY_SURFACE_MULT_AWFUL
for(var/obj/surface in get_turf(target))
for(var/obj/surface in target.loc)

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.

Why the change? We generally prefer to use get_turf due to its ability to handle weird error states.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh, it's the reverse you guys want? That's fine. I'll fix it.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Sep 5, 2026
@cm13-github

Copy link
Copy Markdown
Contributor

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

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Sep 5, 2026
@cm13-github

Copy link
Copy Markdown
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Code Improvement Make the code longer Fix Fix one bug, make ten more size/L Denotes a PR that changes 200-799 lines, ignoring generated files. Sound Blast 5 minutes of bass boosted music to our players

Projects

Status: Awaiting Review

Development

Successfully merging this pull request may close these issues.

3 participants