When/How should I check if my logic is running server-side?

[nocturne storm]

online, i saw some examples of events where people checked for

if (!event.player || event.player.level.isClientSide())
``` or
```js
if (event.server)

only a sparse few examples i found had these checks. are they necessary? what's the most appropriate value to check (eg: see above)? my use-case is something like below:

ItemEvents.rightClicked(event => {
    const {item, player} = event
    if (item.id == 'kubejs:example_item') {
        if (!player || player.level.isClientSide()) return
        item.count--
        event.cancel()
    }
})

[worldly walrus]

Once your ticket has been resolved, please close it with </ticket close:1054771505520717835> command!

[foggy elm]

From my experience, the check is mostly for ForgeEvent/NativeEvent, as some of them fires on both sides. Using it in a server/client script seems more likely belt and braces.

As for the method to check, I think level.isClientSide() is better than .server as there's some events not having server as one of their property.

[nocturne storm]

much appreciated!