Post Respawn Event by MrScopes · Pull Request #8770 · SkriptLang/Skript

MrScopes · 2026-07-12T21:19:14Z

Problem

Currently, there is no support for https://jd.papermc.io/paper/26.2/com/destroystokyo/paper/event/player/PlayerPostRespawnEvent.html
To make changes to the user's player state, you have to use

on respawn:
 wait ...
 ...

this can be inconstent. PlayerPostRespawnEvent is fired after the respawn event.
take this from PlayerRespawnEvent

Called when a player respawns.
If changing player state, see PlayerPostRespawnEvent because the player is "reset" between this event and that event and some changes won't persist.

Solution

after [player] respawn[ing]

Testing Completed

on respawn:
    broadcast "&aon respawn:"
    broadcast "&b%player% &c%respawn location%"
    if respawn location is a bed:
        broadcast "&dbed"

after respawn:
    broadcast "&aafter respawn:"
    broadcast "&b%player% &c%respawn location%"
    if respawn location is a bed:
        broadcast "&dbed"

# the example I gave for the docs
after respawn:
    if respawn location is a bed:
        broadcast "%player% respawned at their bed at %respawn location%"
    else if respawn location is a respawn anchor:
        broadcast "%player% respawned at their anchor at %respawn location%"
    else:
        broadcast "%player% respawned at %respawn location%"

Supporting Information

I also modernized the respawn location expression and condition by moving it to the player module, and I updated some of the code in the classes.

Completes: #8769
Related: none
AI assistance: CoPilot

AnOwlBe

few things

AnOwlBe

supplier!

AnOwlBe

you have my read only approval 🔥

Co-authored-by: SirSmurfy2 <82696841+Absolutionism@users.noreply.github.com>

AnOwlBe

meep

AnOwlBe · 2026-07-19T03:45:07Z

+	}
+
+	@Override
+	public boolean init(Expression<?>[] exprs, int matchedPattern, Kleenean isDelayed, ParseResult parseResult) {


Suggested change

public boolean init(Expression<?>[] exprs, int matchedPattern, Kleenean isDelayed, ParseResult parseResult) {

public boolean init(Expression<?>[] expressions, int matchedPattern, Kleenean isDelayed, ParseResult parseResult) {

This isn't required

no it isn't its just something that could be changed - hence why its a comment review not a requested changes review

Absolutionism

Almost there

Co-authored-by: SirSmurfy2 <82696841+Absolutionism@users.noreply.github.com>

sovdeeth · 2026-07-29T22:27:53Z

+		set respawn location to {example::spawn}
+	""")
+@Since("2.2-dev35")
+public class ExprRespawnLocation extends SimpleExpression<Location> implements EventRestrictedSyntax {


This should probably become a EventValueExpression and the respawn location an eventvalue. Not required for this PR but would be nice.

APickledWalrus

A few minor suggestions

APickledWalrus · 2026-08-03T20:00:30Z

+	public static void register(SyntaxRegistry syntaxRegistry) {
+		syntaxRegistry.register(SyntaxRegistry.CONDITION, SyntaxInfo.builder(CondRespawnLocation.class)
+			.supplier(CondRespawnLocation::new)
+			.addPattern("[the] respawn location (was|is)[1:(n'| no)t] [a] (:bed|respawn anchor)")


APickledWalrus · 2026-08-03T20:02:25Z

-			ExprQuitMessage::register
+			ExprQuitMessage::register,
+			ExprRespawnLocation::register,
+			CondRespawnLocation::register


Should be placed at the beginning of the list (alphabetical order)

MrScopes added 3 commits July 12, 2026 17:10

support playerpostrespawnevent

d9c5c84

update event syntax

7769165

update event syntax

5ac4b1a

MrScopes requested a review from a team as a code owner July 12, 2026 21:19

MrScopes requested review from Efnilite and Pesekjak and removed request for a team July 12, 2026 21:19

MrScopes changed the title ~~Postrespawn~~ Post Respawn Event Jul 12, 2026

AnOwlBe suggested changes Jul 12, 2026

View reviewed changes

Comment thread src/main/java/ch/njol/skript/events/SimpleEvents.java Outdated

Comment thread src/main/java/ch/njol/skript/events/SimpleEvents.java Outdated

MrScopes added 2 commits July 12, 2026 19:03

switch to playermodule over simpleevents

95db3cd

update event names

9ebf637

AnOwlBe reviewed Jul 13, 2026

View reviewed changes

Comment thread src/main/java/org/skriptlang/skript/bukkit/entity/player/PlayerModule.java

Comment thread src/main/java/org/skriptlang/skript/bukkit/entity/player/PlayerModule.java

suppliers

572cdd4

AnOwlBe approved these changes Jul 13, 2026

View reviewed changes

Comment thread src/main/java/org/skriptlang/skript/bukkit/entity/player/PlayerModule.java Outdated

update supplier

c98987c

skriptlang-automation Bot added the needs reviews A PR that needs additional reviews label Jul 14, 2026

Absolutionism requested changes Jul 16, 2026

View reviewed changes

MrScopes and others added 3 commits July 18, 2026 20:36

move nullable into the function def

4d2a236

Co-authored-by: SirSmurfy2 <82696841+Absolutionism@users.noreply.github.com>

remove final from tostring

4459fa7

Co-authored-by: SirSmurfy2 <82696841+Absolutionism@users.noreply.github.com>

fix empty line

40775b7

Co-authored-by: SirSmurfy2 <82696841+Absolutionism@users.noreply.github.com>

AnOwlBe reviewed Jul 19, 2026

View reviewed changes

Absolutionism requested changes Jul 19, 2026

View reviewed changes

MrScopes force-pushed the postrespawn branch from 94bd866 to 676e5fa Compare July 26, 2026 03:26

MrScopes and others added 6 commits July 25, 2026 23:26

fix method order

5aa4243

add empty line

c3faaf4

move supportedevents() to bottom

7d00f07

add space to cast

6a68bf5

Co-authored-by: SirSmurfy2 <82696841+Absolutionism@users.noreply.github.com>

move nullable and use ChangeMode directly

9cd9a10

Co-authored-by: SirSmurfy2 <82696841+Absolutionism@users.noreply.github.com>

move supportedEvents(), remove useless import

d8c2a78

MrScopes force-pushed the postrespawn branch from 676e5fa to d8c2a78 Compare July 26, 2026 03:26

sovdeeth approved these changes Jul 29, 2026

View reviewed changes

Absolutionism approved these changes Jul 30, 2026

View reviewed changes

skriptlang-automation Bot added feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. and removed needs reviews A PR that needs additional reviews labels Jul 30, 2026

APickledWalrus linked an issue Aug 3, 2026 that may be closed by this pull request

Post Respawn Event #8769

Open

1 task

APickledWalrus approved these changes Aug 3, 2026

View reviewed changes

	public boolean init(Expression<?>[] exprs, int matchedPattern, Kleenean isDelayed, ParseResult parseResult) {
	public boolean init(Expression<?>[] expressions, int matchedPattern, Kleenean isDelayed, ParseResult parseResult) {

	.addPattern("[the] respawn location (was\|is)[1:(n'\| no)t] [a] (:bed\|respawn anchor)")
	.addPattern("[the] respawn location (was\|is\|1:was not\|1:wasn't\|1:is not\|1:isn't) [a] (:bed\|respawn anchor)")

Uh oh!

Uh oh!

Conversation

MrScopes commented Jul 12, 2026

Problem

Solution

Testing Completed

Supporting Information

Uh oh!

AnOwlBe left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

AnOwlBe left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

AnOwlBe left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

AnOwlBe left a comment

Choose a reason for hiding this comment

Uh oh!

AnOwlBe Jul 19, 2026

Choose a reason for hiding this comment

Uh oh!

Absolutionism Jul 19, 2026

Choose a reason for hiding this comment

Uh oh!

AnOwlBe Jul 19, 2026

Choose a reason for hiding this comment

Uh oh!

Absolutionism left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sovdeeth Jul 29, 2026

Choose a reason for hiding this comment

Uh oh!

APickledWalrus left a comment

Choose a reason for hiding this comment

Uh oh!

APickledWalrus Aug 3, 2026

Choose a reason for hiding this comment

Uh oh!

APickledWalrus Aug 3, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants