Release v0.1.9: address v0.1.8 review follow-ups

CHANGELOG.md

@@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/),
 and this project adheres to [Semantic Versioning](https://semver.org/).
 
+## [0.1.9] - 2026-09-01
+
+### Fixed
+
+- Render task responses containing currency fields with the task table and its billable column instead of misclassifying them as clients.
+- Show the actual human-readable duration format in `time stop`, `time log`, and `time running` help examples.
+- Validate agent lifecycle metadata against the 4KB API limit after adding required skill and duration fields.
+
 ## [0.1.8] - 2026-09-01
 
 ### Added

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "keito-cli"
-version = "0.1.8"
+version = "0.1.9"
 edition = "2021"
 description = "AI agent time tracking CLI for Keito: capture billable human and agent work for client billing, agency projects, and AI-native services"
 license = "MIT"

src/cli/time.rs

@@ -98,7 +98,7 @@ EXAMPLE:
     \"project\": \"Acme Website\",
     \"task\": \"Development\",
     \"duration_hours\": 1.5,
-    \"duration\": \"1:30\",
+    \"duration\": \"1h 30m\",
     \"spent_date\": \"2026-03-04\",
     \"billable\": true,
     \"source\": \"cli\"
@@ -146,7 +146,7 @@ EXAMPLE:
     \"project\": \"Acme Website\",
     \"task\": \"Development\",
     \"duration_hours\": 1.5,
-    \"duration\": \"1:30\",
+    \"duration\": \"1h 30m\",
     \"spent_date\": \"2025-01-15\",
     \"date\": \"2025-01-15\",
     \"billable\": true,
@@ -367,7 +367,7 @@ EXAMPLE:
     \"source\": \"cli\",
     \"started_at\": \"2026-03-04T09:00:00Z\",
     \"elapsed_hours\": 1.5,
-    \"elapsed\": \"1:30\"
+    \"elapsed\": \"1h 30m\"
   }
 
 EXIT CODES:

src/commands/time.rs

@@ -833,7 +833,16 @@ fn prepare_agent_log_metadata(
         "duration_seconds".into(),
         Value::Number(duration_seconds.into()),
     );
-    Ok(Some(Value::Object(map)))
+    let value = Value::Object(map);
+    let size = serde_json::to_string(&value)
+        .map_err(|err| AppError::InvalidInput(format!("failed to serialize metadata: {err}")))?
+        .len();
+    if size > 4096 {
+        return Err(AppError::InvalidInput(
+            "--metadata payload must be 4KB or smaller".into(),
+        ));
+    }
+    Ok(Some(value))
 }
 
 fn insert_string_metadata(map: &mut Map<String, Value>, key: &str, value: Option<String>) {
@@ -972,4 +981,34 @@ mod tests {
         assert_eq!(metadata["skill"], "keito-time-track");
         assert_eq!(metadata["duration_seconds"], 900);
     }
+
+    #[test]
+    fn agent_log_lifecycle_metadata_checks_final_size() {
+        let raw = serde_json::json!({
+            "session_id": "session-123",
+            "padding": "x".repeat(4020),
+        })
+        .to_string();
+        assert!(raw.len() <= 4096);
+
+        let error = prepare_agent_log_metadata(
+            build_metadata(MetadataInput {
+                metadata: Some(raw),
+                session_id: None,
+                agent_id: None,
+                agent_type: None,
+                skill: None,
+            })
+            .unwrap(),
+            "agent",
+            900,
+        )
+        .unwrap_err();
+
+        assert!(matches!(
+            error,
+            AppError::InvalidInput(message)
+                if message == "--metadata payload must be 4KB or smaller"
+        ));
+    }
 }

src/output/table.rs

@@ -196,6 +196,7 @@ fn try_as_clients(arr: &[serde_json::Value]) -> Option<Vec<Client>> {
     let first = arr.first()?.as_object()?;
     if first.contains_key("name")
         && first.contains_key("currency")
+        && !first.contains_key("billable_by_default")
         && !first.contains_key("project_id")
         && !first.contains_key("task_id")
         && !first.contains_key("is_billable")
@@ -349,3 +350,31 @@ fn format_me_table(me_list: &[MeResponse]) -> String {
         "No data.".into()
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn task_with_currency_uses_task_table() {
+        let task = Task {
+            id: "task-1".into(),
+            name: "Development".into(),
+            is_active: true,
+            billable_by_default: true,
+            default_hourly_rate: Some(100.0),
+            effective_billable_rate: Some(120.0),
+            currency: Some("GBP".into()),
+            budget: None,
+            is_default: false,
+            parent_task_id: None,
+            created_at: None,
+            updated_at: None,
+        };
+
+        let output = to_table(&[task]);
+
+        assert!(output.contains("Billable"));
+        assert!(!output.contains("Currency"));
+    }
+}