reorganize proto files a bit (#30)

Author: joshuadavidthomas

.editorconfig

@@ -9,14 +9,14 @@ trim_trailing_whitespace = true
 [{,.}{j,J}ustfile]
 indent_size = 4
 
-[*.{just,proto,py,rst,ini,md}]
+[*.{just,py,rst,ini,md}]
 indent_size = 4
 
 [*.py]
 line_length = 120
 multi_line_output = 3
 
-[*.{css,html,js,json,jsx,sass,scss,svelte,ts,tsx,yml,yaml}]
+[*.{css,html,js,json,jsx,proto,sass,scss,svelte,ts,tsx,yml,yaml}]
 indent_size = 2
 
 [*.md]

crates/djls-django/src/django.rs

@@ -23,7 +23,7 @@ impl DjangoProject {
     pub fn setup(mut python: PythonProcess) -> Result<Self, ProjectError> {
         let py = Python::setup(&mut python)?;
 
-        match check::GeoDjangoPrereqsRequest::execute(&mut python)?.result {
+        match commands::check::GeoDjangoPrereqsRequest::execute(&mut python)?.result {
             Some(messages::response::Result::CheckGeodjangoPrereqs(response)) => {
                 if !response.passed {
                     eprintln!("Warning: GeoDjango detected but GDAL is not available.");
@@ -43,7 +43,7 @@ impl DjangoProject {
             _ => Err(ProcessError::Response)?,
         }
 
-        let response = django::GetProjectInfoRequest::execute(&mut python)?;
+        let response = commands::django::GetProjectInfoRequest::execute(&mut python)?;
 
         let version = match response.result {
             Some(messages::response::Result::DjangoGetProjectInfo(response)) => {

crates/djls-ipc/src/commands.rs

@@ -13,7 +13,7 @@ pub trait IpcCommand: Default {
     }
 }
 
-impl IpcCommand for v1::check::HealthRequest {
+impl IpcCommand for v1::commands::check::HealthRequest {
     fn into_request(&self) -> messages::Request {
         messages::Request {
             command: Some(messages::request::Command::CheckHealth(*self)),
@@ -29,7 +29,7 @@ impl IpcCommand for v1::check::HealthRequest {
     }
 }
 
-impl IpcCommand for v1::check::GeoDjangoPrereqsRequest {
+impl IpcCommand for v1::commands::check::GeoDjangoPrereqsRequest {
     fn into_request(&self) -> messages::Request {
         messages::Request {
             command: Some(messages::request::Command::CheckGeodjangoPrereqs(*self)),
@@ -45,7 +45,7 @@ impl IpcCommand for v1::check::GeoDjangoPrereqsRequest {
     }
 }
 
-impl IpcCommand for v1::python::GetEnvironmentRequest {
+impl IpcCommand for v1::commands::python::GetEnvironmentRequest {
     fn into_request(&self) -> messages::Request {
         messages::Request {
             command: Some(messages::request::Command::PythonGetEnvironment(*self)),
@@ -61,7 +61,7 @@ impl IpcCommand for v1::python::GetEnvironmentRequest {
     }
 }
 
-impl IpcCommand for v1::django::GetProjectInfoRequest {
+impl IpcCommand for v1::commands::django::GetProjectInfoRequest {
     fn into_request(&self) -> messages::Request {
         messages::Request {
             command: Some(messages::request::Command::DjangoGetProjectInfo(*self)),

crates/djls-ipc/src/process.rs

@@ -79,7 +79,7 @@ impl PythonProcess {
     ) -> Result<(), ProcessError> {
         let request = messages::Request {
             command: Some(messages::request::Command::CheckHealth(
-                check::HealthRequest {},
+                commands::check::HealthRequest {},
             )),
         };
 

crates/djls-ipc/src/proto.rs

@@ -1,16 +1,16 @@
 pub mod v1 {
-    pub mod messages {
-        include!(concat!(env!("OUT_DIR"), "/djls.v1.messages.rs"));
-    }
-
-    pub mod check {
-        include!(concat!(env!("OUT_DIR"), "/djls.v1.check.rs"));
+    pub mod commands {
+        include!(concat!(env!("OUT_DIR"), "/djls.v1.commands.rs"));
     }
 
     pub mod django {
         include!(concat!(env!("OUT_DIR"), "/djls.v1.django.rs"));
     }
 
+    pub mod messages {
+        include!(concat!(env!("OUT_DIR"), "/djls.v1.messages.rs"));
+    }
+
     pub mod python {
         include!(concat!(env!("OUT_DIR"), "/djls.v1.python.rs"));
     }

crates/djls-python/src/python.rs

@@ -91,7 +91,7 @@ pub struct Python {
 
 impl Python {
     pub fn setup(python: &mut PythonProcess) -> Result<Self, PythonError> {
-        let response = python::GetEnvironmentRequest::execute(python)?;
+        let response = commands::python::GetEnvironmentRequest::execute(python)?;
         match response.result {
             Some(messages::response::Result::PythonGetEnvironment(response)) => response
                 .python

proto/v1/check.proto

@@ -0,0 +1,35 @@
+syntax = "proto3";
+
+package djls.v1.commands;
+
+import "v1/django.proto";
+import "v1/python.proto";
+
+message Check {
+  message HealthRequest {}
+  message HealthResponse {
+    bool passed = 1;
+    optional string error = 2;
+  }
+
+  message GeoDjangoPrereqsRequest {}
+  message GeoDjangoPrereqsResponse {
+    bool passed = 1;
+    optional string error = 2;
+  }
+}
+
+message Python {
+  message GetEnvironmentRequest {}
+  message GetEnvironmentResponse {
+    python.Python python = 1;
+  }
+}
+
+message Django {
+  message GetProjectInfoRequest {}
+  message GetProjectInfoResponse {
+    django.Project project = 1;
+  }
+}
+

proto/v1/commands.proto

@@ -2,14 +2,6 @@ syntax = "proto3";
 
 package djls.v1.django;
 
-// models
 message Project {
     string version = 3;
 }
-
-// commands
-message GetProjectInfoRequest {}
-
-message GetProjectInfoResponse {
-    Project project = 1;
-}

proto/v1/django.proto

@@ -2,25 +2,23 @@ syntax = "proto3";
 
 package djls.v1.messages;
 
-import "v1/check.proto";
-import "v1/django.proto";
-import "v1/python.proto";
+import "v1/commands.proto";
 
 message Request {
     oneof command {
-        check.HealthRequest check__health = 1;
-        check.GeoDjangoPrereqsRequest check__geodjango_prereqs = 2;
-        python.GetEnvironmentRequest python__get_environment = 1000;
-        django.GetProjectInfoRequest django__get_project_info = 2000;
+        commands.Check.HealthRequest check__health = 1;
+        commands.Check.GeoDjangoPrereqsRequest check__geodjango_prereqs = 2;
+        commands.Python.GetEnvironmentRequest python__get_environment = 1000;
+        commands.Django.GetProjectInfoRequest django__get_project_info = 2000;
     }
 }
 
 message Response {
     oneof result {
-        check.HealthResponse check__health = 1;
-        check.GeoDjangoPrereqsResponse check__geodjango_prereqs = 2;
-        python.GetEnvironmentResponse python__get_environment = 1000;
-        django.GetProjectInfoResponse django__get_project_info = 2000;
+        commands.Check.HealthResponse check__health = 1;
+        commands.Check.GeoDjangoPrereqsResponse check__geodjango_prereqs = 2;
+        commands.Python.GetEnvironmentResponse python__get_environment = 1000;
+        commands.Django.GetProjectInfoResponse django__get_project_info = 2000;
         Error error = 9000;
     }
 }