Add missing documentation for all the shared hooks (and some other) + Fix some inaccuracies

[Winkarst-cpu]

No description provided.

[Fer3D]

🚫 Do not change or delete existing hook names; it can break a lot of code.

[Winkarst-cpu]

🚫 Do not change or delete existing hook names; it can break a lot of code.

Will see what Alex will say because having two different hook names with completely same meaning is bad.

[offlegaz2]

🚫 Do not change or delete existing hook names; it can break a lot of code.

Will see what Alex will say because having two different hook names with completely same meaning is bad.

Imagine developer writing plugin for Helix with use of area hooks and then updating his framework after your commit. What will be his reaction?

[Lucasgood5]

🚫 Do not change or delete existing hook names; it can break a lot of code.

Will see what Alex will say because having two different hook names with completely same meaning is bad.

Imagine developer writing plugin for Helix with use of area hooks and then updating his framework after your commit. What will be his reaction?

Exactly.

@Winkarst-cpu
In case we would want to change this hook, We may wanna left it but tag it as depreacated for some while before removing it.
could simply by adding documentation and an ErrorNoHaltWithStack("___ is decreacated, use ___ instead").

[Lucasgood5]

Suggested change

	--- @realm client
	function OnAreaChanged(oldID, newID)
	end
	--- @realm client
	--- @deprecated
	function OnAreaChanged(oldID, newID)
	end

[Lucasgood5]

Suggested change

	function PLUGIN:OnAreaChanged(oldID, newID)
	local client = LocalPlayer()
	function PLUGIN:OnPlayerAreaChanged(client, oldID, newID)
	function PLUGIN:OnAreaChanged(oldID, newID)
	local client = LocalPlayer()
	ErrorNoHaltWithStack("OnAreaChanged(oldID, newID) is deprecated, use OnPlayerAreaChanged(client, oldID, newID) instead")
	PLUGIN:OnPlayerAreaChanged(client, oldID, newID)
	end
	function PLUGIN:OnPlayerAreaChanged(client, oldID, newID)