ndjson-rpc 0.0.2

//! Typed Unix-socket client. Same frame shapes as [`crate::server`].
//! One Unix-socket connection per call: the API stays a simple
//! `call(verb, args) -> result`, and a future multiplexed transport
//! can be slotted in without changing the call shape.

use std::path::{Path, PathBuf};
use std::time::Duration;

use tokio::io::{AsyncBufReadExt, AsyncWriteExt, BufReader};
use tokio::net::UnixStream;

use crate::protocol::{Request, Response, ResponseOutcome, WireError, WireErrorKind, encode_line};

/// Maximum time to wait for `UnixStream::connect` to return. The mgmt
/// daemon is on the same host as the CLI; if the kernel doesn't grant
/// the connection in five seconds something is badly wrong (daemon
/// not running, socket file stale, FS perms wrong) and operators want
/// the failure mode to be a fast error rather than a hang.
pub const CONNECT_TIMEOUT: Duration = Duration::from_secs(5);

/// Maximum time between server-sourced bytes on a one-shot call. A
/// server stall after we've sent the request shouldn't leave the
/// operator's terminal hanging; this fires `MgmtClientError::Timeout`
/// and lets the shell prompt return.
pub const ONESHOT_TIMEOUT: Duration = Duration::from_secs(30);

/// Single-shot typed Unix mgmt client. Each `call` opens a fresh
/// connection. Re-using one client for many calls works too — the
/// struct holds no persistent state across invocations.
pub struct UnixMgmtClient {
	socket_path: PathBuf,
}

impl UnixMgmtClient {
	pub fn new(socket_path: impl AsRef<Path>) -> Self {
		Self { socket_path: socket_path.as_ref().to_path_buf() }
	}

	/// Send a verb + typed args, await typed result.
	///
	/// `id` is fixed at `1` for the single-request-per-connection
	/// transport — there is no need for cross-process uniqueness on a
	/// freshly-opened socket. A future multiplexed transport will own
	/// its own id-allocation scheme.
	///
	/// # Errors
	/// I/O failure ([`MgmtClientError::Io`]); a structured server-side
	/// error ([`MgmtClientError::Server`]); or a JSON shape mismatch
	/// when decoding either the response frame or the result payload
	/// ([`MgmtClientError::Decode`]).
	pub async fn call<A, R>(&self, verb: &str, args: &A) -> Result<R, MgmtClientError>
	where
		A: serde::Serialize,
		R: for<'de> serde::Deserialize<'de>,
	{
		let stream = tokio::time::timeout(CONNECT_TIMEOUT, UnixStream::connect(&self.socket_path))
			.await
			.map_err(|_| MgmtClientError::Timeout("connect"))??;
		let (read, mut write) = stream.into_split();

		let req = Request {
			id: 1,
			verb: verb.to_string(),
			args: serde_json::to_value(args).map_err(MgmtClientError::Encode)?,
		};
		let bytes = encode_line(&req).map_err(MgmtClientError::Encode)?;
		write.write_all(&bytes).await?;
		// Half-close the write half so the server's `next_line` returns
		// `None` after the response — the server task can then drop the
		// connection cleanly.
		write.shutdown().await.ok();

		let mut lines = BufReader::new(read).lines();
		let line = tokio::time::timeout(ONESHOT_TIMEOUT, lines.next_line())
			.await
			.map_err(|_| MgmtClientError::Timeout("read"))??
			.ok_or(MgmtClientError::EmptyResponse)?;
		let response: Response = serde_json::from_str(&line).map_err(MgmtClientError::Decode)?;
		match response.outcome {
			ResponseOutcome::Result { result } => {
				serde_json::from_value(result).map_err(MgmtClientError::Decode)
			}
			ResponseOutcome::Error { error } => Err(MgmtClientError::Server(error)),
			ResponseOutcome::Event { .. } | ResponseOutcome::End { .. } => {
				// Streaming frame on a one-shot `call`. Either the verb is
				// a streaming verb (caller should use `call_stream`) or
				// the server is buggy.
				Err(MgmtClientError::Server(WireError::new(
					WireErrorKind::Internal,
					"received streaming frame on one-shot call",
				)))
			}
		}
	}

	/// Send a streaming verb and consume each `Event` frame via
	/// `on_event` until the server emits `End`, the connection drops, or
	/// the server emits `Error`.
	///
	/// Unlike [`Self::call`], this does **not** half-close the write
	/// side of the socket — keeping the write half open lets the server
	/// (or future operator-side cancellation logic) detect a clean
	/// client disconnect via socket close.
	///
	/// # Errors
	/// I/O failures, server-side `Error` frames, or a server-emitted
	/// `Result` frame on what should be a streaming verb.
	pub async fn call_stream<A, F>(
		&self,
		verb: &str,
		args: &A,
		mut on_event: F,
	) -> Result<(), MgmtClientError>
	where
		A: serde::Serialize,
		F: FnMut(serde_json::Value),
	{
		let stream = tokio::time::timeout(CONNECT_TIMEOUT, UnixStream::connect(&self.socket_path))
			.await
			.map_err(|_| MgmtClientError::Timeout("connect"))??;
		let (read, mut write) = stream.into_split();

		let req = Request {
			id: 1,
			verb: verb.to_string(),
			args: serde_json::to_value(args).map_err(MgmtClientError::Encode)?,
		};
		let bytes = encode_line(&req).map_err(MgmtClientError::Encode)?;
		write.write_all(&bytes).await?;
		// NB: do not shutdown(write) — see doc comment.

		let mut lines = BufReader::new(read).lines();
		while let Some(line) = lines.next_line().await? {
			if line.is_empty() {
				continue;
			}
			let response: Response = serde_json::from_str(&line).map_err(MgmtClientError::Decode)?;
			match response.outcome {
				ResponseOutcome::Event { event } => on_event(event),
				ResponseOutcome::End { .. } => return Ok(()),
				ResponseOutcome::Error { error } => return Err(MgmtClientError::Server(error)),
				ResponseOutcome::Result { .. } => {
					return Err(MgmtClientError::Server(WireError::new(
						WireErrorKind::Internal,
						"received one-shot Result on streaming call",
					)));
				}
			}
		}
		// Server closed mid-stream without an `End` frame — treat as
		// graceful EOF for callbacks that already consumed events.
		Ok(())
	}
}

#[derive(Debug, thiserror::Error)]
pub enum MgmtClientError {
	#[error("io: {0}")]
	Io(#[from] std::io::Error),
	#[error("encode: {0}")]
	Encode(serde_json::Error),
	#[error("decode: {0}")]
	Decode(serde_json::Error),
	#[error("empty response")]
	EmptyResponse,
	#[error("server: {kind:?} {message}", kind = .0.kind, message = .0.message)]
	Server(WireError),
	/// Non-200 HTTP response from the management endpoint. Only ever
	/// surfaced by [`crate::http_client::HttpMgmtClient`]; the Unix
	/// transport has no equivalent shape.
	#[error("http {status}: {body}")]
	Http { status: u16, body: String },
	/// Transport-level deadline expired. Phase identifies the stage:
	/// `"connect"` for a stalled accept on the mgmt socket / TCP
	/// endpoint, `"read"` for a server that accepted the request but
	/// then went silent.
	#[error("timeout: {0}")]
	Timeout(&'static str),
}

#[cfg(test)]
mod tests {
	use super::*;
	use crate::server::{DispatchOutcome, Handler, handle_conn};
	use async_trait::async_trait;
	use serde::Deserialize;
	use std::sync::Arc;

	#[derive(Debug, Deserialize)]
	struct PingResult {
		pong: bool,
		version: String,
	}

	#[derive(serde::Serialize)]
	struct NoArgs {}

	struct StubHandler;

	#[async_trait]
	impl Handler for StubHandler {
		async fn dispatch(&self, req: Request) -> DispatchOutcome {
			let result: Result<serde_json::Value, crate::protocol::WireError> = match req.verb.as_str() {
				"ping" => Ok(serde_json::json!({ "pong": true, "version": "test" })),
				"bad_shape" => Ok(serde_json::json!({ "unrelated": 1 })),
				_ => Err(WireError::new(WireErrorKind::UnknownVerb, format!("unknown {}", req.verb))),
			};
			DispatchOutcome::OneShot(result)
		}
	}

	/// Connect a duplex pair and run the server's `handle_conn` against
	/// the stub. Returns a closure-like helper bound to the test stream
	/// — used by the typed-call decode tests below.
	async fn drive_call<A, R>(verb: &str, args: A) -> Result<R, MgmtClientError>
	where
		A: serde::Serialize,
		R: for<'de> serde::Deserialize<'de>,
	{
		let (c2s_r, mut c2s_w) = tokio::io::duplex(8192);
		let (s2c_w, s2c_r) = tokio::io::duplex(8192);
		let server = tokio::spawn(handle_conn(
			c2s_r,
			s2c_w,
			Arc::new(StubHandler),
			tokio_util::sync::CancellationToken::new(),
		));

		// Client side: write the request line, half-close, read one response line.
		let req = Request {
			id: 1,
			verb: verb.to_string(),
			args: serde_json::to_value(&args).expect("args serialize"),
		};
		let bytes = encode_line(&req).expect("encode");
		c2s_w.write_all(&bytes).await.expect("write");
		drop(c2s_w);

		let mut lines = BufReader::new(s2c_r).lines();
		let line = lines
			.next_line()
			.await
			.map_err(MgmtClientError::Io)?
			.ok_or(MgmtClientError::EmptyResponse)?;
		let response: Response = serde_json::from_str(&line).map_err(MgmtClientError::Decode)?;
		// Drain the server task. (`handle_conn` returns once `next_line` returns
		// None on the read half, which happens on `drop(c2s_w)`.)
		let _ = server.await;
		match response.outcome {
			ResponseOutcome::Result { result } => {
				serde_json::from_value(result).map_err(MgmtClientError::Decode)
			}
			ResponseOutcome::Error { error } => Err(MgmtClientError::Server(error)),
			other => panic!("unexpected outcome: {other:?}"),
		}
	}

	#[tokio::test]
	async fn client_call_decodes_typed_result() {
		let result: PingResult = drive_call("ping", NoArgs {}).await.expect("ok");
		assert!(result.pong);
		assert_eq!(result.version, "test");
	}

	#[tokio::test]
	async fn client_surfaces_server_error_as_mgmt_client_error_server() {
		let err = drive_call::<_, PingResult>("nope", NoArgs {}).await.expect_err("err");
		match err {
			MgmtClientError::Server(w) => {
				assert_eq!(w.kind, crate::protocol::WireErrorKind::UnknownVerb);
			}
			other => panic!("expected Server, got {other:?}"),
		}
	}

	#[tokio::test]
	async fn client_decode_error_when_result_shape_mismatches() {
		let err = drive_call::<_, PingResult>("bad_shape", NoArgs {}).await.expect_err("err");
		assert!(matches!(err, MgmtClientError::Decode(_)), "unexpected variant: {err:?}");
	}

	struct StreamingHandler;

	#[async_trait]
	impl Handler for StreamingHandler {
		async fn dispatch(&self, _req: Request) -> DispatchOutcome {
			use crate::server::EventStream;
			struct ThreeEvents {
				remaining: Vec<serde_json::Value>,
			}
			#[async_trait]
			impl EventStream for ThreeEvents {
				async fn next_event(&mut self) -> Option<serde_json::Value> {
					self.remaining.pop()
				}
			}
			DispatchOutcome::Stream(Box::new(ThreeEvents {
				remaining: vec![
					serde_json::json!({ "i": 3 }),
					serde_json::json!({ "i": 2 }),
					serde_json::json!({ "i": 1 }),
				],
			}))
		}
	}

	#[tokio::test]
	async fn client_call_stream_invokes_callback_per_event_until_end() {
		// Drive `call_stream` against an in-process duplex pair — this
		// covers the Event → End frame sequence without spinning a real
		// Unix socket. Each emitted callback push is recorded to a Vec.
		let (c2s_r, mut c2s_w) = tokio::io::duplex(8192);
		let (s2c_w, s2c_r) = tokio::io::duplex(8192);
		let server = tokio::spawn(crate::server::handle_conn(
			c2s_r,
			s2c_w,
			Arc::new(StreamingHandler),
			tokio_util::sync::CancellationToken::new(),
		));

		// Emulate `call_stream` against the duplex pair.
		let req = Request { id: 1, verb: "tail".to_string(), args: serde_json::Value::Null };
		let bytes = encode_line(&req).expect("encode");
		c2s_w.write_all(&bytes).await.expect("write");

		let mut events = Vec::new();
		let mut lines = BufReader::new(s2c_r).lines();
		loop {
			let line = lines.next_line().await.expect("read").expect("line");
			let resp: Response = serde_json::from_str(&line).expect("parse");
			match resp.outcome {
				ResponseOutcome::Event { event } => events.push(event),
				ResponseOutcome::End { .. } => break,
				other => panic!("unexpected: {other:?}"),
			}
		}
		// Drop the client write half so the server task can return.
		drop(c2s_w);
		let _ = server.await;
		assert_eq!(events.len(), 3, "all three events consumed");
		assert_eq!(events[0]["i"], 1, "FIFO order from the stream");
		assert_eq!(events[1]["i"], 2);
		assert_eq!(events[2]["i"], 3);
	}

	#[tokio::test]
	async fn client_io_error_on_missing_socket() {
		let tmp = tempfile::tempdir().expect("tempdir");
		let path = tmp.path().join("not-there.sock");
		let client = UnixMgmtClient::new(&path);
		let err = client
			.call::<_, PingResult>("ping", &NoArgs {})
			.await
			.expect_err("must fail without a server");
		assert!(matches!(err, MgmtClientError::Io(_)), "unexpected variant: {err:?}");
	}
}