daemon/api: Remove `add_history`

This request is a footgun, as it will leave commands that never return
(e.g. `shutdown`, `exit`, etc.) unrecorded.

2 files changed, 0 insertions, 31 deletions

diff --git a/crates/turtle/proto/history.proto b/crates/turtle/proto/history.proto
index 90fcf55f..c6bdd99b 100644
--- a/crates/turtle/proto/history.proto
+++ b/crates/turtle/proto/history.proto
@@ -4,7 +4,6 @@ package history;
 service History {
   rpc StartHistory(StartHistoryRequest) returns (StartHistoryReply);
   rpc EndHistory(EndHistoryRequest) returns (EndHistoryReply);
-  rpc AddHistory(AddHistoryRequest) returns (EndHistoryReply);
 
   rpc TailHistory(TailHistoryRequest) returns (stream TailHistoryReply);
 
@@ -12,12 +11,6 @@ service History {
   rpc History(HistoryRequest) returns (HistoryReply);
 }
 
-message AddHistoryRequest {
-  StartHistoryRequest start = 1;
-  int64 exit = 2;
-  Duration duration = 3;
-}
-
 message StartHistoryRequest {
   Timestamp timestamp = 1;
   string command = 2;
diff --git a/crates/turtle/src/client/mod.rs b/crates/turtle/src/client/mod.rs
index 28289652..531d8597 100644
--- a/crates/turtle/src/client/mod.rs
+++ b/crates/turtle/src/client/mod.rs
@@ -7,7 +7,6 @@ use hyper_util::rt::TokioIo;
 
 use tokio::net::UnixStream;
 
-use crate::generated::history::AddHistoryRequest;
 use crate::generated::{
     self, DAEMON_PROTOCOL_VERSION,
     control::{
@@ -235,29 +234,6 @@ impl HistoryClient {
         Ok(self.client.end_history(req).await?.into_inner())
     }
 
-    pub async fn add_history(
-        &mut self,
-        h: History,
-        duration: std::time::Duration,
-        exit: i64,
-    ) -> Result<EndHistoryReply> {
-        let req = AddHistoryRequest {
-            start: Some(StartHistoryRequest {
-                timestamp: time_to_proto_timestamp(h.timestamp),
-                command: h.command,
-                cwd: h.cwd,
-                session: h.session,
-                hostname: h.hostname,
-                author: h.author,
-                intent: h.intent,
-            }),
-            exit,
-            duration: std_to_proto_duration(duration),
-        };
-
-        Ok(self.client.add_history(req).await?.into_inner())
-    }
-
     pub async fn history(&mut self, range: Option<Range>) -> Result<Vec<History>> {
         let req = HistoryRequest {
             range: range.map(|r| generated::history::Range {