Skip to content

missing apoc types (real this time)#531

Merged
XeonMations merged 40 commits intoDarkPack13:masterfrom
dwinters99:new-apoc-missing-types-pr-because-i-wanted-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
Feb 9, 2026
Merged

missing apoc types (real this time)#531
XeonMations merged 40 commits intoDarkPack13:masterfrom
dwinters99:new-apoc-missing-types-pr-because-i-wanted-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one

Conversation

@dwinters99
Copy link
Copy Markdown
Member

@dwinters99 dwinters99 commented Jan 16, 2026

About The Pull Request

wolves: mundane, kinfolk, spiral edit: new deadsprites by @Major00
image

psg1 fixes
image

Why It's Good For The Game

Changelog

🆑 dwinters99, KCartridge, Major00
add: DP Wolves
add: PSG1 magazine and ammo
code: gun code organization
add: undense bin
/:cl:

…-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
…the-tfn-one-so-i-gave-up-on-the-last-one' of https://github.com/dwinters99/SecondCity into new-apoc-missing-types-pr-because-i-wanted-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
…-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
…the-tfn-one-so-i-gave-up-on-the-last-one' of https://github.com/dwinters99/SecondCity into new-apoc-missing-types-pr-because-i-wanted-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
…-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
Copy link
Copy Markdown
Member

@FalloutFalcon FalloutFalcon left a comment

Choose a reason for hiding this comment

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

nits:

Comment thread modular_darkpack/modules/npc/code/nonhuman/friendly/wolf.dm Outdated
Comment thread code/modules/mob/living/basic/pets/dog/_dog.dm Outdated
Comment thread code/modules/mob/living/basic/pets/dog/_dog.dm Outdated
…o new-apoc-missing-types-pr-because-i-wanted-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
@github-actions github-actions Bot added the Tools label Jan 20, 2026
FalloutFalcon and others added 9 commits January 20, 2026 18:25
…the-tfn-one-so-i-gave-up-on-the-last-one' of https://github.com/dwinters99/SecondCity into new-apoc-missing-types-pr-because-i-wanted-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
Co-authored-by: FalloutFalcon <86381784+FalloutFalcon@users.noreply.github.com>
Co-authored-by: FalloutFalcon <86381784+FalloutFalcon@users.noreply.github.com>
Co-authored-by: FalloutFalcon <86381784+FalloutFalcon@users.noreply.github.com>
…types-pr-because-i-wanted-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
@dwinters99 dwinters99 marked this pull request as ready for review January 27, 2026 03:43
@dwinters99 dwinters99 requested review from a team as code owners January 27, 2026 03:43
Copy link
Copy Markdown
Member

@FalloutFalcon FalloutFalcon left a comment

Choose a reason for hiding this comment

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

approved but also i made alot of changes to the pr myself so need someone else to skim it.

…-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one

// DARKPACK EDIT ADD START - NPC - (snowflake obeys_commands abstraction proc)
/mob/living/basic/pet/dog/proc/add_obey_commands()
var/static/list/pet_commands = list( // DARKPACK EDIT REMOVE START - npc - Moving this into a proc so we can have subtypes with different commands
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.

this comment matches the one on line 39 even though it does not remove here

Comment thread code/modules/mob/living/basic/pets/dog/_dog.dm
Comment on lines +1 to +10
#define COAT_BLACK 1 // wolf1
#define COAT_GRAY 2 // wolf2
#define COAT_RED 3 // wolf3
#define COAT_WHITE 4 // wolf4
#define COAT_GINGER 5 // wolf5
#define COAT_BROWN 6 // wolf6

#define TYPE_MUNDANE "wolf"
#define TYPE_KINFOLK "kinfolk"
#define TYPE_SPIRAL "spiral"
Copy link
Copy Markdown
Contributor

@chazzyjazzy chazzyjazzy Jan 30, 2026

Choose a reason for hiding this comment

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

maybe a defines folder for WTA content? just so we keep a standard?

or just, defines in the folders we've been using, idk, up to you

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The coat colors could be moved i guess but the "type" thing is snowflake for the wolves

Comment thread modular_darkpack/modules/npc/code/nonhuman/friendly/wolf.dm Outdated
Comment on lines +59 to +65
maxHealth = 120
health = 120
obj_damage = 15
melee_damage_lower = 7.5
melee_damage_upper = 7.5
attack_vis_effect = ATTACK_EFFECT_BITE
melee_attack_cooldown = 1.2 SECONDS
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.

something makes me wanna say, make these all more favorable for the wolf lol. I mean, its a WOLF!

Not opposed to this tho, it is, after all, an npc critter

Comment on lines +39 to +41
#TFN removed atmos. We have not
/obj/item/anesthetic_tank : /obj/item/tank/internals/anesthetic {@OLD}

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.

we wont be removing it on darkpack ive heard

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

i believe this comment is in reference to oldcode removing atmos and therefore needing weird new items like this that now need to be turned back into atmos items (@FalloutFalcon)

Comment on lines +45 to +46
# Apoc subtyped them under masq contracts. We didnt but honestly consider.
/obj/item/masquerade_contract/veil : /obj/item/veil_contract {@OLD}
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.

Yes

Comment on lines +1 to +3
#this assumes all old mapped "wolfkin" are infact kinfolk
/mob/living/basic/pet/dog/wolfkin : /mob/living/basic/pet/dog/wolf/kinfolk {@OLD}
/mob/living/basic/pet/dog/wolfkin/@SUBTYPES : /mob/living/basic/pet/dog/wolf/kinfolk/@SUBTYPES {@OLD}
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.

A safe assumption, im just wondering if all wolfkin mobs prior were mob/living/basic since I seem to remember a long time ago that they may have been simplemobs

not a big deal at all tho itll just mean more of a headache for anyone who wants to bring back the OLD map, like the wod13 map, lol, which likely won't happen.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think this is an oversight on either my part or @FalloutFalcon's part. they were 100% simple_mobs.

Copy link
Copy Markdown
Member

@FalloutFalcon FalloutFalcon Feb 1, 2026

Choose a reason for hiding this comment

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

Yea oversight. (the simple mob type only exists on apoc, they are evil varedits elsewhere)

Comment thread modular_darkpack/modules/weapons/code/guns.dm Outdated
Comment thread modular_darkpack/modules/weapons/code/guns.dm Outdated
dwinters99 and others added 7 commits January 30, 2026 17:32
Co-authored-by: chazzyjazzy <33268885+chazzyjazzy@users.noreply.github.com>
…-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
Co-authored-by: chazzyjazzy <33268885+chazzyjazzy@users.noreply.github.com>
Co-authored-by: chazzyjazzy <33268885+chazzyjazzy@users.noreply.github.com>
…-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
…the-tfn-one-so-i-gave-up-on-the-last-one' of https://github.com/dwinters99/SecondCity into new-apoc-missing-types-pr-because-i-wanted-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one
FalloutFalcon and others added 2 commits February 6, 2026 16:47
@XeonMations XeonMations added this pull request to the merge queue Feb 9, 2026
Merged via the queue into DarkPack13:master with commit 415d933 Feb 9, 2026
21 checks passed
darkpack13-api Bot added a commit that referenced this pull request Feb 9, 2026
github-actions Bot added a commit that referenced this pull request Feb 9, 2026
@dwinters99 dwinters99 deleted the new-apoc-missing-types-pr-because-i-wanted-to-wait-for-the-tfn-one-so-i-gave-up-on-the-last-one branch March 22, 2026 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants