"on attack attempt:" does not fire for sword sweep attacks

[ReddoMC]

Skript/Server Version

[16:26:18]
[Skript] Skript's aliases can be found here: https://github.com/SkriptLang/skript-aliases
[16:26:18]
[Skript] Skript's documentation can be found here: https://docs.skriptlang.org/
[16:26:18]
[Skript] Skript's tutorials can be found here: https://docs.skriptlang.org/tutorials
[16:26:18]
[Skript] Server Version: 26.1.2-74-e4e17fc (MC: 26.1.2)
[16:26:18]
[Skript] Skript Version: 2.15.3 (skriptlang-github)
[16:26:18]
[Skript] Installed Skript Addons: 
[16:26:18]
[Skript]  - skript-placeholders v1.7.1 (https://github.com/APickledWalrus/skript-placeholders)
[16:26:18]
[Skript]  - skript-yaml v1.7.2
[16:26:18]
[Skript]  - DiSky v4.29.0
[16:26:18]
[Skript]  - skript-reflect v2.6.3 (https://github.com/SkriptLang/skript-reflect)
[16:26:18]
[Skript]  - SkBee v3.25.3 (https://github.com/ShaneBeee/SkBee)
[16:26:18]
[Skript] Installed dependencies: 
[16:26:18]
[Skript]  - WorldGuard v7.0.16+2355-f7fded2

Bug Description

I am unsure if you guys can fix this but basically on attack attempt: does not get fired if the player gets hit by a sword sweep attack. This is causing players with their PVP off to still be damaged in our server.

Expected Behavior

I expect sweep to count as a player attack.

Steps to Reproduce

Just do:

on attack attempt:
brodacast "a"

and you will see sweep does not work

Errors or Screenshots

No response

Other

No response

Agreement

• I have read the guidelines above and affirm I am following them with this report.

[sovdeeth]

Do you mean it doesn't trigger for the entity being directly damaged, or nearby entities that get hit by sweep?

[TFSMads]

This is not a Skript issue.

Paper's PrePlayerAttackEntityEvent is only fired for the entity that is directly attacked by the player. Entities hit by a sword sweep attack do not trigger this event.

Adding support for sweep-attacked entities is currently not possible with the Paper API, as there is no event that is fired before the sweep attack's sounds and damage logic are processed.

Therefore, Skript cannot reliably fire attack attempt event for entities affected by sword sweep attacks with the current Paper API.

I have also opened a PR for Paper to add PrePlayerAttackEntityEvent support for entities hit by sweep attacks: PaperMC/Paper#14203