Hyper As Base for Rest Adapter by raphael-goetz · Pull Request #142 · code0-tech/draco

raphael-goetz · 2026-01-30T12:47:36Z

Resolves: #132

Copilot

Pull request overview

Migrates the REST adapter to use hyper directly as the HTTP server foundation (addressing issue #132) and removes the now-redundant in-repo crates/http implementation.

Changes:

Replaced the custom http crate usage in the REST adapter with a hyper-based HTTP/1 server.
Introduced request routing via RequestRoute (method + URL) and extracted server config into a dedicated module.
Removed crates/http from the workspace and updated dependencies/lockfile accordingly.

Reviewed changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 14 comments.

Show a summary per file

File	Description
`adapter/rest/src/main.rs`	Implements a `hyper`-based server loop, request handling, and flow execution response mapping.
`adapter/rest/src/route.rs`	Adds method+URL-based flow identification and slug extraction helper.
`adapter/rest/src/config.rs`	Defines HTTP server config and environment loading defaults.
`adapter/rest/Cargo.toml`	Swaps `http` workspace crate dependency for `hyper`/`hyper-util`/`http-body-util`.
`crates/base/src/store.rs`	Adds logging around flow execution requests (but currently introduces a move/compile issue).
`Cargo.toml`	Removes `crates/http` from workspace members and workspace dependency overrides.
`Cargo.lock`	Updates lockfile to reflect dependency changes and removal of the local `http` crate.
`crates/http/*`	Removes the legacy custom HTTP implementation from the repository.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-01-30T12:53:36Z

+fn json_error(status: StatusCode, msg: &str) -> Response<Full<Bytes>> {
+    let body = format!(r#"{{"error": "{}"}}"#, msg);
+    Response::builder()
+        .status(status)
+        .header("content-type", "application/json")
+        .body(Full::new(Bytes::from(body)))
+        .unwrap()


json_error interpolates msg directly into a JSON string without escaping, so quotes/newlines in msg can produce invalid JSON. Build the body via serde_json (or at least JSON-escape msg) before returning.

Copilot · 2026-01-30T12:53:36Z

+    let method = req.method().clone();
+    let path = req.uri().path().to_string();
+


path is taken from req.uri().path(), which drops the query string. Previously the request parser included ?query in path and also exposed parsed query params in the body structure, so route matching and flow inputs can change. If query strings matter for routing or inputs, use uri().path_and_query() and/or parse the query into a Value like before.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated 9 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-01-30T13:38:49Z

+        // Check if the request is matching
+        match regex::Regex::new(regex_str) {
+            Ok(regex) => {
+                log::debug!("Successfully compiled regex: {}", regex_str);
+                regex.is_match(&self.url)
+            }
+            Err(err) => {
+                log::error!("Failed to compile regex: {}", err);
+                false
+            }
+        }


identify compiles a new Regex for every candidate flow (Regex::new(regex_str)), which can become a hot path since get_possible_flow_match iterates over all matching keys/flows. Consider caching compiled regexes by pattern string (e.g., LRU/once_cell cache) or storing a precompiled regex in the flow representation if possible.

Copilot · 2026-01-30T13:38:49Z

+fn json_error(status: StatusCode, msg: &str) -> Response<Full<Bytes>> {
+    let body = format!(r#"{{"error": "{}"}}"#, msg);
+    Response::builder()
+        .status(status)
+        .header("content-type", "application/json")
+        .body(Full::new(Bytes::from(body)))
+        .unwrap()


json_error interpolates msg directly into a JSON string. If msg contains quotes/backslashes/newlines, the response becomes invalid JSON (and can lead to confusing client behavior). Build the body via serde_json (or properly escape the string) instead of manual string formatting.

Copilot · 2026-01-30T13:38:50Z

+            // headers struct
            let Value {
                kind:
-                    Some(StructValue(Struct {
-                        fields: header_fields,
+                    Some(Kind::ListValue(ListValue {
+                        values: header_fields,
                    })),
-            } = headers
+            } = headers_val
            else {
-                return None;
+                return json_error(
+                    StatusCode::INTERNAL_SERVER_ERROR,
+                    "headers was not a list of header entries",
+                );
            };


headers_val is a &Value (from fields.get(...)), but this destructuring pattern matches against an owned Value. This will not compile. Match on a reference (e.g., &Value { .. }) or use headers_val.kind.as_ref() / if let Some(Kind::ListValue(...)) = headers_val.kind.as_ref() to avoid moving out of a borrowed value.

Copilot · 2026-01-30T13:38:50Z

+            // status_code number
+            let Some(Kind::NumberValue(code)) = status_code_val.kind else {
+                return json_error(
+                    StatusCode::INTERNAL_SERVER_ERROR,
+                    "status_code was not a number",
+                );
+            };


status_code_val is a &Value, so status_code_val.kind attempts to move the kind field out of a borrowed value and will not compile. Use status_code_val.kind.as_ref() (and then map/clone as needed) to read it without moving.

Copilot · 2026-01-30T13:38:50Z

+    let method = req.method().clone();
+    let path = req.uri().path().to_string();
+


handle_request uses req.uri().path(), which drops the query string. Previously the full request target (including ?query=...) could be used for matching and/or input construction. If flow matching or downstream logic depends on query parameters, consider using uri().path_and_query() and/or parsing the query into the Value input passed to validate_and_execute_flow.

Copilot · 2026-01-30T13:38:50Z

+    let value: Option<Value> = if body.is_empty() {
+        None
+    } else {
+        match prost::Message::decode(body.as_slice()) {


prost::Message::decode(...) cannot be called directly on the Message trait because the concrete Self type is unknown; this will not compile. Decode using the concrete type (e.g., Value::decode(...) or <Value as prost::Message>::decode(...)).

Suggested change

match prost::Message::decode(body.as_slice()) {

match Value::decode(body.as_slice()) {

Copilot · 2026-01-30T13:38:51Z

        }

        let uuid = uuid::Uuid::new_v4().to_string();
+        let flow_id = flow.flow_id;


flow_id is moved out of flow (let flow_id = flow.flow_id;) and then flow is moved into convert_validation_flow(flow, ...). Since flow_id is not Copy, this leaves flow partially moved and will not compile. Use let flow_id = flow.flow_id.clone(); (or borrow it via &flow.flow_id for logging) before passing flow onward.

Suggested change

let flow_id = flow.flow_id;

let flow_id = flow.flow_id.clone();

Copilot · 2026-01-30T13:38:51Z

+    let value: Option<Value> = if body.is_empty() {
+        None
+    } else {
+        match prost::Message::decode(body.as_slice()) {
+            Ok(v) => Some(v),
+            Err(e) => {
+                log::warn!("Failed to decode request body as protobuf Value: {}", e);
+                return json_error(
+                    StatusCode::BAD_REQUEST,
+                    "Failed to decode request body as protobuf Value",
+                );
+            }
+        }
+    };


The request body is decoded as a protobuf-encoded tucana::shared::Value. For an HTTP/REST adapter this is a behavior change vs typical JSON payloads (and the prior implementation built Value from JSON). If clients send JSON, this will always return 400. Consider decoding based on Content-Type (e.g., JSON -> Value) and preserving the previous REST semantics.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

raphael-goetz added 6 commits January 29, 2026 11:29

drop: removed http crate

b140b2c

drop: removed http crate

ea7a415

feat: started to work on swtiching from http crate to hyper

8109b51

feat: made header to list

c2ceddc

feat: added logs for execution

5a88acf

ref: cargo fmt

a52ac4e

raphael-goetz requested a review from Copilot January 30, 2026 12:47

Copilot started reviewing on behalf of raphael-goetz January 30, 2026 12:47 View session

Copilot AI reviewed Jan 30, 2026

View reviewed changes

raphael-goetz and others added 8 commits January 30, 2026 13:58

Update adapter/rest/src/route.rs

f921b1a

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

Update adapter/rest/src/main.rs

ed79258

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

Update adapter/rest/src/route.rs

18342df

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

Update crates/base/src/store.rs

c70fc5d

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

Update adapter/rest/src/main.rs

24fc621

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

feat: implemented shutdown

a113543

Update adapter/rest/src/main.rs

79caae5

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

Update adapter/rest/src/main.rs

ca8d4df

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

raphael-goetz requested a review from Copilot January 30, 2026 13:33

Copilot started reviewing on behalf of raphael-goetz January 30, 2026 13:33 View session

Copilot AI reviewed Jan 30, 2026

View reviewed changes

Update adapter/rest/src/main.rs

7100a85

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>

raphael-goetz merged commit 9d625a8 into main Jan 30, 2026
4 checks passed

raphael-goetz deleted the 132-hyper-base branch January 30, 2026 15:24

		let method = req.method().clone();
		let path = req.uri().path().to_string();

	match prost::Message::decode(body.as_slice()) {
	match Value::decode(body.as_slice()) {

	let flow_id = flow.flow_id;
	let flow_id = flow.flow_id.clone();

Uh oh!

Conversation

raphael-goetz commented Jan 30, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants