omk 0.5.0

A Rust runtime for Kimi CLI. Turns prompts into proof-backed engineering runs with gates, worktrees, and replay.
Documentation 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269

use anyhow::{Context, Result};
use serde::{Deserialize, Serialize};
use std::path::PathBuf;
use tracing::info;

use crate::runtime::wire_worker::ApprovalPolicy;

/// Specification for a single worker in a team
#[derive(Debug, Clone, Serialize, Deserialize)]
pub struct WorkerSpec {
    pub name: String,
    pub role: String,
    pub inbox: PathBuf,
    pub outbox: PathBuf,
    pub heartbeat: PathBuf,
    #[serde(default)]
    pub project_dir: Option<PathBuf>,
    #[serde(default, skip_serializing_if = "Option::is_none")]
    pub external_tools: Option<Vec<serde_json::Value>>,
    #[serde(default)]
    pub approval_policy: ApprovalPolicy,
    #[serde(default = "default_approval_timeout_secs")]
    pub approval_timeout_secs: u64,
}

pub fn default_approval_timeout_secs() -> u64 {
    300
}

impl WorkerSpec {
    pub async fn save(&self) -> Result<()> {
        let path = self
            .inbox
            .parent()
            .context("inbox path has no parent")?
            .join("worker-spec.json");
        let json = serde_json::to_string_pretty(self)?;
        crate::runtime::atomic::atomic_write(&path, json.as_bytes()).await?;
        info!(path = %path.display(), name = %self.name, "Saved worker spec");
        Ok(())
    }

    pub async fn load(worker_dir: &std::path::Path) -> Result<Self> {
        let path = worker_dir.join("worker-spec.json");
        let json = tokio::fs::read_to_string(&path).await?;
        let spec: Self = serde_json::from_str(&json)?;
        Ok(spec)
    }

    /// Write a task to the inbox
    pub async fn send_task(&self, task: &WorkerTask) -> Result<()> {
        // Use O_APPEND-based append (write is line-atomic on POSIX for
        // payloads under PIPE_BUF) rather than the older read-modify-write
        // `atomic_append_jsonl` helper, whose name was misleading: two
        // concurrent callers could load the same pre-image and the second
        // atomic_write would clobber the first task. POSIX `O_APPEND`
        // serializes the kernel-side seek+write per system call.
        let mut line = serde_json::to_vec(task)?;
        line.push(b'\n');
        crate::runtime::atomic::atomic_append(&self.inbox, &line).await?;
        Ok(())
    }

    /// Write a result to the outbox
    pub async fn send_result(&self, result: &WorkerResult) -> Result<()> {
        let line = format!("{}\n", serde_json::to_string(result)?);
        crate::runtime::atomic::atomic_append(&self.outbox, line.as_bytes()).await?;
        Ok(())
    }

    /// Read all completed results from outbox
    pub async fn read_results(&self) -> Result<Vec<WorkerResult>> {
        if !self.outbox.exists() {
            return Ok(vec![]);
        }
        let content = tokio::fs::read_to_string(&self.outbox).await?;
        let mut results = vec![];
        for line in content.lines() {
            if line.trim().is_empty() {
                continue;
            }
            match serde_json::from_str(line) {
                Ok(r) => results.push(r),
                Err(e) => tracing::warn!(line = %line, error = %e, "Failed to parse worker result"),
            }
        }
        Ok(results)
    }
}

#[derive(Debug, Clone, Serialize, Deserialize)]
pub struct WorkerTask {
    pub id: String,
    pub task: String,
    #[serde(default)]
    pub acceptance_criteria: Vec<String>,
    #[serde(skip_serializing_if = "Option::is_none")]
    pub context: Option<String>,
    #[serde(default, skip_serializing_if = "Option::is_none")]
    pub budget_secs: Option<u64>,
}

#[derive(Debug, Clone, Serialize, Deserialize)]
pub struct WorkerResult {
    pub task_id: String,
    pub status: ResultStatus,
    #[serde(default)]
    pub summary: String,
    #[serde(default)]
    pub artifacts: Vec<String>,
    #[serde(default)]
    pub elapsed_secs: u64,
}

#[derive(Debug, Clone, Serialize, Deserialize)]
#[serde(rename_all = "snake_case")]
pub enum ResultStatus {
    Success,
    Partial,
    Failed,
}

#[cfg(test)]
mod tests {
    use super::*;

    #[tokio::test]
    async fn test_worker_spec_roundtrip() {
        let dir = tempfile::tempdir().unwrap();
        let spec = WorkerSpec {
            name: "worker-0".to_string(),
            role: "coder".to_string(),
            inbox: dir.path().join("inbox.jsonl"),
            outbox: dir.path().join("outbox.jsonl"),
            heartbeat: dir.path().join("heartbeat.json"),
            project_dir: None,
            external_tools: None,
            approval_policy: ApprovalPolicy::default(),
            approval_timeout_secs: default_approval_timeout_secs(),
        };

        spec.save().await.unwrap();
        let loaded = WorkerSpec::load(dir.path()).await.unwrap();
        assert_eq!(loaded.name, "worker-0");
        assert_eq!(loaded.role, "coder");
    }

    #[tokio::test]
    async fn test_send_and_read_task() {
        let dir = tempfile::tempdir().unwrap();
        let spec = WorkerSpec {
            name: "worker-0".to_string(),
            role: "coder".to_string(),
            inbox: dir.path().join("inbox.jsonl"),
            outbox: dir.path().join("outbox.jsonl"),
            heartbeat: dir.path().join("heartbeat.json"),
            project_dir: None,
            external_tools: None,
            approval_policy: ApprovalPolicy::default(),
            approval_timeout_secs: default_approval_timeout_secs(),
        };

        let task = WorkerTask {
            id: "task-1".to_string(),
            task: "fix bug".to_string(),
            acceptance_criteria: vec!["tests pass".to_string()],
            context: None,
            budget_secs: None,
        };

        spec.send_task(&task).await.unwrap();

        let results = spec.read_results().await.unwrap();
        assert!(results.is_empty());

        // Simulate a result written by a worker
        let result = WorkerResult {
            task_id: "task-1".to_string(),
            status: ResultStatus::Success,
            summary: "done".to_string(),
            artifacts: vec![],
            elapsed_secs: 10,
        };
        let line = serde_json::to_string(&result).unwrap();
        tokio::fs::write(&spec.outbox, format!("{}\n", line))
            .await
            .unwrap();

        let results = spec.read_results().await.unwrap();
        assert_eq!(results.len(), 1);
        assert_eq!(results[0].task_id, "task-1");
        matches!(results[0].status, ResultStatus::Success);
    }

    #[tokio::test]
    async fn test_send_result_writes_valid_json() {
        let dir = tempfile::tempdir().unwrap();
        let spec = WorkerSpec {
            name: "worker-0".to_string(),
            role: "coder".to_string(),
            inbox: dir.path().join("inbox.jsonl"),
            outbox: dir.path().join("outbox.jsonl"),
            heartbeat: dir.path().join("heartbeat.json"),
            project_dir: None,
            external_tools: None,
            approval_policy: ApprovalPolicy::default(),
            approval_timeout_secs: default_approval_timeout_secs(),
        };

        let result = WorkerResult {
            task_id: "task-1".to_string(),
            status: ResultStatus::Success,
            summary: "done".to_string(),
            artifacts: vec!["file.rs".to_string()],
            elapsed_secs: 42,
        };

        spec.send_result(&result).await.unwrap();

        let content = tokio::fs::read_to_string(&spec.outbox).await.unwrap();
        let parsed: serde_json::Value = serde_json::from_str(content.trim()).unwrap();
        assert_eq!(parsed["task_id"], "task-1");
        assert_eq!(parsed["status"], "success");
        assert_eq!(parsed["summary"], "done");
        assert_eq!(parsed["elapsed_secs"], 42);
    }

    #[tokio::test]
    async fn test_send_result_appends_multiple_lines() {
        let dir = tempfile::tempdir().unwrap();
        let spec = WorkerSpec {
            name: "worker-0".to_string(),
            role: "coder".to_string(),
            inbox: dir.path().join("inbox.jsonl"),
            outbox: dir.path().join("outbox.jsonl"),
            heartbeat: dir.path().join("heartbeat.json"),
            project_dir: None,
            external_tools: None,
            approval_policy: ApprovalPolicy::default(),
            approval_timeout_secs: default_approval_timeout_secs(),
        };

        spec.send_result(&WorkerResult {
            task_id: "task-1".to_string(),
            status: ResultStatus::Success,
            summary: "first".to_string(),
            artifacts: vec![],
            elapsed_secs: 1,
        })
        .await
        .unwrap();

        spec.send_result(&WorkerResult {
            task_id: "task-2".to_string(),
            status: ResultStatus::Failed,
            summary: "second".to_string(),
            artifacts: vec![],
            elapsed_secs: 2,
        })
        .await
        .unwrap();

        let content = tokio::fs::read_to_string(&spec.outbox).await.unwrap();
        let lines: Vec<&str> = content.lines().collect();
        assert_eq!(lines.len(), 2);
        assert!(lines[0].contains("task-1"));
        assert!(lines[1].contains("task-2"));
    }
}