4 files changed, 28 insertions, 81 deletions
diff --git a/minecraft-protocol/src/packets/client_intention_packet.rs b/minecraft-protocol/src/packets/client_intention_packet.rs
deleted file mode 100644
index 424c645f..00000000
--- a/minecraft-protocol/src/packets/client_intention_packet.rs
+++ /dev/null
@@ -1,31 +0,0 @@
-use std::hash::Hash;
-
-use crate::mc_buf;
-
-use super::{ConnectionProtocol, Packet};
-
-#[derive(Hash)]
-pub struct ClientIntentionPacket<'a> {
-    pub protocol_version: u32,
-    pub hostname: &'a String,
-    pub port: u16,
-    /// 1 for status, 2 for login
-    pub intention: ConnectionProtocol,
-}
-
-// implement "Packet" for "ClientIntentionPacket"
-impl<'a> Packet for ClientIntentionPacket<'a> {
-    fn get_id(&self) -> u32 {
-        0x00
-    }
-
-    // implement "from_reader" for "ClientIntentionPacket"
-    fn write(&self, buf: &mut Vec<u8>) {
-        mc_buf::write_varint(buf, self.protocol_version as i32);
-        mc_buf::write_utf(buf, &self.hostname);
-        mc_buf::write_short(buf, self.port);
-        mc_buf::write_varint(buf, self.intention.clone() as i32);
-    }
-
-    fn parse<T: tokio::io::AsyncRead + std::marker::Unpin>(&self, buf: T) -> () {}
-}
diff --git a/minecraft-protocol/src/packets/clientbound_status_response_packet.rs b/minecraft-protocol/src/packets/clientbound_status_response_packet.rs
deleted file mode 100644
index ddca9b1f..00000000
--- a/minecraft-protocol/src/packets/clientbound_status_response_packet.rs
+++ /dev/null
@@ -1,23 +0,0 @@
-use std::hash::Hash;
-
-use super::Packet;
-
-#[derive(Hash)]
-pub struct ServerboundStatusRequestPacket {
-    // status: ServerStatus,
-    status: String,
-}
-
-// implement "Packet" for "ClientIntentionPacket"
-impl Packet for ServerboundStatusRequestPacket {
-    fn get_id(&self) -> u32 {
-        0x00
-    }
-
-    // implement "from_reader" for "ClientIntentionPacket"
-    fn write(&self, _buf: &mut Vec<u8>) {}
-    fn parse<T: tokio::io::AsyncRead + std::marker::Unpin>(&self, buf: T) -> () {
-        mc_buf::read_utf;
-        // this.status = GsonHelper.fromJson(GSON, friendlyByteBuf.readUtf(32767), ServerStatus.class);
-    }
-}
diff --git a/minecraft-protocol/src/packets/mod.rs b/minecraft-protocol/src/packets/mod.rs
index 8d943be0..5fb34743 100644
--- a/minecraft-protocol/src/packets/mod.rs
+++ b/minecraft-protocol/src/packets/mod.rs
@@ -1,21 +1,39 @@
-mod client_intention_packet;
-pub use client_intention_packet::ClientIntentionPacket;
-mod serverbound_status_request_packet;
-pub use serverbound_status_request_packet::ServerboundStatusRequestPacket;
+mod game;
+mod handshake;
+mod login;
+mod status;
+
+use async_trait::async_trait;
 use tokio::io::AsyncRead;
 
 #[derive(Debug, Clone, PartialEq, Eq, Hash)]
 pub enum ConnectionProtocol {
-    Handshaking = -1,
-    Play = 0,
+    Handshake = -1,
+    Game = 0,
     Status = 1,
     Login = 2,
 }
 
-pub trait Packet {
-    /// Get the id of the packet, this is always a byte.
-    fn get_id(&self) -> u32;
+pub enum Packet<'a> {
+    // game
+    // handshake
+    ClientIntentionPacket(&'a handshake::client_intention_packet::ClientIntentionPacket<'a>),
+    // login
+    // status
+    ServerboundStatusRequestPacket(
+        &'a status::serverbound_status_request_packet::ServerboundStatusRequestPacket,
+    ),
+    ClientboundStatusRequestPacket(
+        &'a status::clientbound_status_response_packet::ClientboundStatusRequestPacket,
+    ),
+}
 
+pub trait PacketTrait {
+    /// Return a version of the packet that you can actually use for stuff
+    fn get(&self) -> Packet;
     fn write(&self, buf: &mut Vec<u8>) -> ();
-    fn parse<T: AsyncRead + std::marker::Unpin>(&self, buf: T) -> ();
+    fn parse<T: AsyncRead + std::marker::Unpin>(
+        buf: &mut T,
+        // is using a static lifetime here a good idea? idk
+    ) -> Result<Packet<'_>, String>;
 }
diff --git a/minecraft-protocol/src/packets/serverbound_status_request_packet.rs b/minecraft-protocol/src/packets/serverbound_status_request_packet.rs
deleted file mode 100644
index ab9001f4..00000000
--- a/minecraft-protocol/src/packets/serverbound_status_request_packet.rs
+++ /dev/null
@@ -1,17 +0,0 @@
-use std::hash::Hash;
-
-use super::Packet;
-
-#[derive(Hash)]
-pub struct ServerboundStatusRequestPacket {}
-
-// implement "Packet" for "ClientIntentionPacket"
-impl Packet for ServerboundStatusRequestPacket {
-    fn get_id(&self) -> u32 {
-        0x00
-    }
-
-    // implement "from_reader" for "ClientIntentionPacket"
-    fn write(&self, _buf: &mut Vec<u8>) {}
-    fn parse<T: tokio::io::AsyncRead + std::marker::Unpin>(&self, buf: T) -> () {}
-}