refactor: replace Play component with PacketState enum

Replace standalone `Play` component with `PacketState::Play` enum variant for better
state management. Changes include:

- Remove `Play` component and update queries to use `PacketState::Play`
- Update player join logic to use `add_enum(PacketState::Play)`
- Add `#[meta]` attribute to `ChatCooldown` component

This change consolidates player state handling into the `PacketState` enum rather than
using separate marker components.

Author: andrewgazelka

crates/hyperion/src/egress/player_join/mod.rs

@@ -21,7 +21,7 @@ use valence_registry::{BiomeRegistry, RegistryCodec};
 use valence_server::entity::EntityKind;
 use valence_text::IntoText;
 
-use crate::simulation::Pitch;
+use crate::simulation::{PacketState, Pitch};
 
 mod list;
 pub use list::*;
@@ -35,7 +35,7 @@ use crate::{
         command::{get_command_packet, Command, ROOT_COMMAND},
         skin::PlayerSkin,
         util::registry_codec_raw,
-        Comms, InGameName, Play, Position, Uuid, Yaw,
+        Comms, InGameName, Position, Uuid, Yaw,
     },
     system_registry::{SystemId, PLAYER_JOINS},
     util::{SendableQuery, SendableRef},
@@ -622,7 +622,7 @@ impl Module for PlayerJoinModule {
                 let entity = world.entity_from_id(entity);
                 entity.set(skin);
 
-                entity.add::<Play>();
+                entity.add_enum(PacketState::Play);
             });
         });
     }

crates/hyperion/src/egress/stats.rs

@@ -3,15 +3,15 @@ use tracing::{error, trace_span};
 
 use crate::{
     net::Compose,
-    simulation::{blocks::Blocks, Play},
+    simulation::{blocks::Blocks, PacketState},
 };
 
 #[derive(Component)]
 pub struct StatsModule;
 
 impl Module for StatsModule {
     fn module(world: &World) {
-        let players = world.new_query::<&Play>();
+        let players = world.query::<()>().with_enum(PacketState::Play).build();
 
         // let last_frame_time_total = 0.0;
 

crates/hyperion/src/egress/sync_chunks.rs

@@ -14,7 +14,7 @@ use crate::{
     net::{Compose, DataBundle, NetworkStreamRef},
     simulation::{
         blocks::{Blocks, GetChunk},
-        ChunkPosition, Play, Position,
+        ChunkPosition, PacketState, Position,
     },
     system_registry::{GENERATE_CHUNK_CHANGES, SEND_FULL_LOADED_CHUNKS},
     util::TracingExt,
@@ -49,7 +49,7 @@ impl Module for SyncChunksModule {
             &NetworkStreamRef,
             &mut ChunkSendQueue,
         )
-        .with::<Play>()
+        .with_enum(PacketState::Play)
         .kind::<flecs::pipeline::OnUpdate>()
         .multi_threaded()
         .tracing_each_entity(
@@ -168,7 +168,7 @@ impl Module for SyncChunksModule {
         let system_id = SEND_FULL_LOADED_CHUNKS;
 
         system!("send_full_loaded_chunks", world, &Blocks($), &Compose($), &NetworkStreamRef, &mut ChunkSendQueue)
-            .with::<Play>()
+            .with_enum(PacketState::Play)
             .kind::<flecs::pipeline::OnUpdate>()
             .multi_threaded()
             .each_entity(

crates/hyperion/src/simulation/mod.rs

@@ -124,10 +124,6 @@ pub struct InGameName(Arc<str>);
 #[derive(Component, Deref, DerefMut, From, Debug, Default)]
 pub struct IgnMap(DeferredMap<Arc<str>, Entity>);
 
-/// This component is added to all players once they reach the play state. See [`PacketState::Play`].
-#[derive(Component, Default)]
-pub struct Play;
-
 /// A component that represents a Player. In the future, this should be broken up into multiple components.
 ///
 /// Why should it be broken up? The more things are broken up, the more we can take advantage of Rust borrowing rules.
@@ -514,7 +510,6 @@ impl Module for SimModule {
         world.component::<Health>().meta();
         world.component::<ChunkPosition>().meta();
         world.component::<EntityReaction>().meta();
-        world.component::<Play>();
         world.component::<ConfirmBlockSequences>();
         world.component::<metadata::Metadata>();
         world.component::<animation::ActiveAnimation>();

events/proof-of-concept/src/module/chat.rs

@@ -15,6 +15,7 @@ const CHAT_COOLDOWN_SECONDS: i64 = 15; // 15 seconds
 const CHAT_COOLDOWN_TICKS: i64 = CHAT_COOLDOWN_SECONDS * 20; // Convert seconds to ticks
 
 #[derive(Default, Component)]
+#[meta]
 pub struct ChatCooldown {
     pub expires: i64,
 }
@@ -26,7 +27,7 @@ impl Module for ChatModule {
     fn module(world: &World) {
         let system_id = SystemId(8);
 
-        world.component::<ChatCooldown>();
+        world.component::<ChatCooldown>().meta();
 
         world
             .component::<Player>()